Skip to content

docs: remove hardcoded reviewer model requirements - #71

Merged
deku2026 merged 1 commit into
mainfrom
plan/provider-neutral-prompts
Oct 28, 2026
Merged

deku2026 merged 1 commit into
mainfrom
plan/provider-neutral-prompts

Conversation

@deku2026

@deku2026 deku2026 commented Oct 27, 2026 •

Copy link
Copy Markdown
Contributor

The execution prompts required Opus 5.5, model-ID confirmation and fixed Claude attribution, causing agents to stop and request a separate Claude login. Remove those obsolete constraints at the user's request.

Align the coordinator, worker and reviewer prompts, execution policy, delivery procedure, AGENTS.md and active handoff instructions with the AD-0014 amendment. Available agents can be used without a required provider or model. Independent review, serial delivery and required host checks remain; attribution must match actual contributors and tools. Dated historical review records are preserved.

Validation at 700bad8cec117cda44ce4cec6cbcf49e2906c6dc: git diff --check passed, and the independent review approved this exact head with no findings. After Design #38 merged, native Windows and Linux python eng/verify.py run-and-report were rerun and both passed the full profile (Design-main refresh, plan validation, traceability and tool tests). Both summaries record the new Design main 63cf9865be44f3427eff081f507275e5ec6f65d0; both required statuses report success. This satisfies the Design-main revalidation requirement before merging.

Companions: Design #38, already merged, and ArcForges #59.

Prepared with Codex.

Co-Authored-By: Codex <noreply@openai.com>

Copy link
Copy Markdown
Contributor Author

Independent review record — APPROVE at 700bad8cec117cda44ce4cec6cbcf49e2906c6dc.

A fresh independent Codex reviewer completed this review; the coordinator is posting its record. No blocking or non-blocking findings.

Reviewed the complete six-file diff, AGENTS.md, execution policy, delivery procedure and entry prompts. The coordinator/worker/reviewer and active handoff instructions consistently remove the required model, model-ID confirmation, separate-client dependency and false attribution, while preserving independent review and other execution gates. The remaining Opus reference records an actual past review.

Distinct checks through Windows MCP:

  • git rev-parse HEAD and git status --porcelain: exact reviewed head and clean worktree.
  • git diff --check f9d386c2daa55131f1136cc71f4d0225b323f46b 700bad8cec117cda44ce4cec6cbcf49e2906c6dc: passed.
  • git diff --no-ext-diff f9d386c2daa55131f1136cc71f4d0225b323f46b 700bad8cec117cda44ce4cec6cbcf49e2906c6dc: complete diff reviewed.
  • Targeted git grep searches found no surviving active restricted-model, model-ID or false-attribution requirement in the reviewed policy documents.

The reviewer independently confirmed arcforges/windows=success and arcforges/linux=success on this exact SHA through GitHub. Full suites were not repeated because verification tooling is unchanged and the required statuses already cover them.

Residual consideration: Both current verification legs used Design main 973934c658f62967be4b6cc40d0526d877ee4383. Merge Design #38 first and rerun both Plan legs against the new Design main before merging this PR.

@deku2026
deku2026 merged commit 66b3453 into main Oct 28, 2026
3 checks passed
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