Repository navigation
code-review-rules: enroll the six URL-form sync targets in review-instructions #726
Description
Split out of #669 (item 2: "Canonical-URL pointer unverified").
Problem
Six sync targets have no root REVIEW.md, so their AGENTS.md ## Code Review Rules pointer links the canonical URL https://github.com/melodic-software/standards/blob/main/REVIEW.md. The #656 probe only showed Codex following in-repo links. Nobody has verified that the Codex reviewer fetches the URL form.
The six targets are not enrolled in the review-instructions sync component. That component syncs REVIEW.md and is already managed in ci-workflows, claude-code-account-rotation, claude-code-plugins, dotfiles, github-iac and provisioning. No recorded reason for leaving these six out was found.
| Target | Visibility |
|---|---|
.github |
public |
agent-plugins |
public |
ci-runner |
public |
claude-code-proxy |
private, presumed URL form (not readable from the cloud session) |
codex-plugins |
public |
cursor-plugins |
public |
Plan
- standards: add
review-instructionsto themanaged:list of all six targets indistribution/sync-manifest.yml, afterrepository-text. Extendexpected_review_instructions_targetsindistribution/sync-manifest.test.shto the 12-target list. This was verified on a scratch copy:sync-manifest.test.shpassed 336/336, including[196] REVIEW.md reaches every enrolled ... target, andsync-manifest.sh validateexited 0. - Each of the six repos, right after its (automerge-armed) sync PR lands
REVIEW.md: switch the AGENTS.md pointer to the managed form.This was simulated on agent-plugins:-- Org-wide criteria: [`REVIEW.md`](https://github.com/melodic-software/standards/blob/main/REVIEW.md) in `melodic-software/standards`. +- Org-wide criteria: [`REVIEW.md`](REVIEW.md), synced from `melodic-software/standards`.
- with
REVIEW.mdpresent and the pointer unchanged, the checker reportsMISSING: ... the REVIEW.md pointer must link REVIEW.md (root REVIEW.md present: true) - after the pointer edit, it reports
OK
- with
- standards: update the
components/code-review-rules/README.mdparagraph on the absent case to say that no audited target uses the URL form. Keep the URL form documented for non-target repositories.
Ordering hazard
Between a repo's sync merge and its pointer PR, the daily maintenance-audit-code-review-rules run is red for that repo. Land each pointer PR right after its sync PR merges.
Constraint
melodic-software/.github cannot be attached to cloud sessions, because repo names starting with . are refused. Its pointer edit needs a local session or a human.
Refs: #669
Activity
feat(sync): enroll the six URL-form targets in review-instructions (#917
docs(agents): point the review criteria at the synced REVIEW.md (#222)
docs(agents): point the review criteria at the synced REVIEW.md (#117)
docs(agents): point the review criteria at the synced REVIEW.md (#515)
docs(agents): point the review criteria at the synced REVIEW.md (#159)
docs(agents): point the review criteria at the synced REVIEW.md (#136)
kyle-sexton commentedon Oct 30, 2026
Done. standards#917 enrolled the six URL-form targets in
review-instructionsand updated the code-review-rules README. Each target's AGENTS.md now points to its local syncedREVIEW.md: .github#222, agent-plugins#117, ci-runner#515, claude-code-proxy#368, codex-plugins#159, cursor-plugins#136. claude-code-proxy's root-coverage check also neededREVIEW.mdas a documentation root (claude-code-proxy#366).