Skip to content

Isolate CPAN report PR merges from tester checkout - #1123

Merged
fglock merged 1 commit into
masterfrom
docs/isolate-cpan-report-merge
Aug 25, 2026
Merged

Isolate CPAN report PR merges from tester checkout#1123
fglock merged 1 commit into
masterfrom
docs/isolate-cpan-report-merge

Conversation

@fglock

@fglock fglock commented Aug 25, 2026

Copy link
Copy Markdown
Owner

Summary

  • require CPAN report PR creation, inspection, and merge to run from a neutral non-worktree directory
  • require an explicit GitHub repository argument so merge-time branch cleanup cannot mutate the tester checkout
  • forbid chronological incident logs in AGENTS.md and skills; integrate reusable prevention into the relevant guidance instead
  • broaden the shell-quoting guidance to prevent command substitution from backticks in any double-quoted argument

Live-test result

PR #1122 successfully published and merged a lock-consistent report snapshot. The report checkout had no newer edits to lose, but gh pr merge --delete-branch switched it to master and fast-forwarded it. This correction prevents that local side effect.

Validation

  • skill validator: pass
  • make check-links: pass, 0 errors
  • direct offline link check for AGENTS.md and the skill: pass, 0 errors
  • full build and runtime tests not run; documentation and non-executable skill instructions only

@fglock
fglock force-pushed the docs/isolate-cpan-report-merge branch from 15da3ae to 94d6b60 Compare August 25, 2026 15:35
@fglock
fglock merged commit ccd0677 into master Aug 25, 2026
2 checks passed
@fglock
fglock deleted the docs/isolate-cpan-report-merge branch August 25, 2026 15:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant