docs: align repository agent instructions with VNext runtime truth - #351
docs: align repository agent instructions with VNext runtime truth#351Joncallim wants to merge 3 commits into
Conversation
Codex Review SummaryThis comment shows the latest Codex review activity on this pull request.
ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings. |
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: d1325afee8
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
| - For refactors touching >3 files, run Architect before Backend/Frontend. | ||
| - Run Architect/design work for new features or cross-cutting changes; do not add | ||
| ceremonial planning calls for trivial mechanical edits. | ||
| - Always run independent Review before recommending merge of substantive PRs. |
There was a problem hiding this comment.
Require independent review for every pull request
For a small documentation, configuration, or mechanical PR, the new “substantive” qualifier permits the operator to recommend a merge without an independent Review pass. That contradicts the unchanged core responsibility requiring every pull request to be reviewed via the Review role, leaving the merge workflow ambiguous; remove the qualifier or define the same exception consistently in both rules.
AGENTS.md reference: AGENTS.md:L105-L107
Useful? React with 👍 / 👎.
Summary
Fixes a safety-relevant instruction drift found by the post-VNext repository audit (#350).
The root agent instruction surface still stated that specialist execution and local repository file application were default-on. The live runtime, README, ADRs and tests instead keep specialist/host writes hard fail-closed until an OS-enforced confined writer exists.
This PR aligns repository-agent guidance with the accepted VNext architecture without changing runtime code.
Changes
AGENTS.mdCLAUDE.mdweb/CLAUDE.mdWhy this is blocking documentation, not polish
Repository agents use these files as operating instructions. A stale statement that writes are available can cause future architecture/implementation/review work to assume an execution authority that Forge deliberately does not provide.
Validation
Documentation/instruction-only change; no runtime code.
Final head:
d1325afee81435ce0bcf44b23fd95930a72c40a0The PR remains intentionally unmerged pending explicit operator authorization.
Related: #333, #334, #336, #350