mirror of
https://github.com/diegosouzapw/OmniRoute.git
synced 2026-09-13 18:32:12 +03:00
docs(agents): protected-surface merge rule — operator approval for agent-instruction files (#12253)
PR #11770 (2026-09-01) added a CLAUDE.md section instructing every AI agent to clone and execute a third-party setup script; a merge campaign swept it into the release branch with no human risk review (reverted in #12249). Review focus now carries the rule: PRs touching CLAUDE.md / AGENTS.md / GEMINI.md / llm.txt / skills SKILL.md files are HOLD until explicit per-PR operator approval — CI validates code, not instruction-surface intent.
This commit is contained in:
committed by
GitHub
parent
d53da4fdc8
commit
30a26d9dcb
@@ -494,6 +494,12 @@ Why this matters: fixing bug A while opening bug B is worse than not fixing at a
|
||||
pipeline, and A2A skills.
|
||||
- Do not close a contributor pull request after using its code; merge it through GitHub so
|
||||
the contributor receives credit.
|
||||
- **Never merge a PR that touches an agent-instruction surface without explicit operator
|
||||
approval** — `CLAUDE.md`, `AGENTS.md`, `GEMINI.md`, `llm.txt` (+ mirrors) and
|
||||
`skills/**/SKILL.md` are executed as authority by every AI session; a merged instruction
|
||||
compromises every future agent run. Check with `gh pr diff <N> --name-only` before any
|
||||
merge. Incident record: PR #11770 (2026-09-01) told agents to execute a third-party
|
||||
setup script and was swept in by a merge campaign; reverted in #12249.
|
||||
|
||||
---
|
||||
|
||||
|
||||
Reference in New Issue
Block a user