Skip to content

docs: make Copilot primary executor in agent instructions#138

Open
Copilot wants to merge 2 commits intomainfrom
copilot/update-copilot-instructions
Open

docs: make Copilot primary executor in agent instructions#138
Copilot wants to merge 2 commits intomainfrom
copilot/update-copilot-instructions

Conversation

Copy link
Copy Markdown
Contributor

Copilot AI commented May 2, 2026

Codex issue comments are not verifiable GitHub artifacts; only a PR, branch, commit, or merge constitutes delivery. These docs lacked an explicit executor policy to encode that distinction.

Changes

AGENTS.md

  • New ## Executor policy section declaring Copilot as primary executor for bounded issue-first PR work
  • Codex scoped to reviewer / cross-repo analyst / backup — explicitly not the default executor
  • Delivery definition: GitHub must show a PR, branch, commit, or merge; comments alone do not count
  • Issue body called out as the sole task spec (Copilot may not see post-assignment comments)

.github/copilot-instructions.md

  • Matching ## Executor policy section prepended, reinforcing the same delivery and source-of-truth rules
  • Codex comment caveat surfaced at the point Copilot reads its own instructions

All existing high-risk path cautions preserved verbatim in both files.

- Add Executor policy section to AGENTS.md: Copilot is primary executor,
  Codex is reviewer/cross-repo analyst/backup only
- Clarify that delivery requires a PR, branch, commit, or merge (not comments)
- Add matching Executor policy section to .github/copilot-instructions.md
- State issue body is the complete source of truth
- Preserve all existing high-risk path cautions

Closes #(issue)

Agent-Logs-Url: https://github.com/SociOS-Linux/source-os/sessions/a6af174e-35f5-4c87-bd5d-1013a45761fd

Co-authored-by: mdheller <21163552+mdheller@users.noreply.github.com>
Copilot AI changed the title [WIP] Update Copilot as primary executor in agent instructions docs: make Copilot primary executor in agent instructions May 2, 2026
Copilot AI requested a review from mdheller May 2, 2026 20:10
@mdheller mdheller marked this pull request as ready for review May 3, 2026 01:53
Copy link
Copy Markdown
Member

mdheller commented May 5, 2026

Replay review:

This executor-policy update is strategically correct, but the PR is non-mergeable against current main; do not merge stale history.

Preserve the useful policy:

  • Copilot is the primary executor for bounded issue-first PR work;
  • Codex is reviewer / cross-repo analyst / backup, not default executor;
  • issue comments alone are not delivery artifacts;
  • the issue body must contain the full task spec because Copilot may not see later comments.

Recommended action: replay this policy into current AGENTS.md and .github/copilot-instructions.md, then align the wording with the AgentPlane Agentic PR Control Plane replay. Close this PR as superseded after replacement lands.

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.

repo hygiene: make Copilot primary executor in agent instructions

2 participants