Repository navigation
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Add collaboration rules, a skill candidate and an optional exact-head Codex
prerequisite to the read-only reusable observer. This includes workflow code.
Why
Progress and review rules have diverged: source progress must follow current PR
state, and review evidence must precede merge. The historical eleven-item batch
also needs an explicit record of missing identities rather than guessed PRs.
Changes
unresolved findings before merge; preserve independent review and native gates.
merge_authorized remains false. Existing callers and App identities are unchanged.
are unavailable and explicitly unassigned; the eleven-PR batch is incomplete.
Testing
fail-before/pass-after marker, transport, drift and concurrency cases.
1 existing README advisory; identical on the indexed base tree.
skill creator quick validator and git diff --check pass.
Risk and rollback
The marker is necessary evidence only; it cannot prove base/scope acceptance,
native CI or owner authorization. Two reads cannot make merge atomic. The
existing dedicated bot must revalidate before disposition. The skill is not
installed and no caller, issue, required check, branch protection or ruleset is
changed. Roll back through a separately reviewed revert of these eight files.
No labels, merge, deploy, Secret/IAM changes, production switch changes or trading
operation is included; production switch values were not inspected.
Breaking changes
None for existing shadow callers: the new input defaults to false. Actual bot,
issue and required-check adoption remains separate and is not applied here.
Checklist
AI assistance
Codex, GPT-6.1 Sol (host configured).
Publication checker
Checker:
publication-check/2.0-advisory; build2947dd7867b4; matching lines: 35; advisory findings: 35.The full tracked candidate was scanned; all eight changed files separately have 0 matches. First 20 masked full-tree entries follow. Existing matches are unchanged; no credential is introduced.