feat(github): validate exact pull-request heads#125
Draft
Nobodyworld wants to merge 1 commit into
Draft
Conversation
Nobodyworld
commented
Jul 23, 2026
Nobodyworld
left a comment
Owner
Author
There was a problem hiding this comment.
Connector planning review complete.
The branch descends from exact merged base dcb8e283f8445dd76f215a98023197d8ed5acab3, contains only the living ExecPlan, and has successful Commitlint 29990883055 plus complete CI 29990883049.
The local implementation handoff must preserve these locked boundaries:
- manual authenticated request flow; no webhooks;
- managed PR comment only; no Checks API/status checks;
- existing repository allowlist and explicit approval gate;
- fresh
validate-switchboard@1execution independent of #121; - re-resolve head before publication and fail moved heads stale;
- credentials remain server/operator configuration;
- mocked transport tests only; no live credentials/network in automation;
- no workflow dispatch, auto-merge, repository writes, artifact upload/download, MCP, browser, or RPA scope.
Keep PR #125 draft and unmerged. Stop and report rather than rebasing, force-pushing, or opening another branch/PR.
This was referenced Jul 23, 2026
Owner
Author
|
Connector planning addendum recorded on issue #122. The local implementation must treat it as part of the locked acceptance contract. Key additions:
Copy these decisions into |
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
Begins issue #122 from the exact merged evidence baseline.
The current commit adds the living ExecPlan at
.agent/execplans/009_github_exact_pr_validation.md.Locked first slice
validate-switchboard@1path without depending on feat(evidence): reuse exact validation evidence with local availability proof #121;Forbidden scope
Planning validation
29990883055— success;29990883049— success.The complete planning-head matrix passed, including lint/pre-commit, TODO validation, Mypy, full pytest, security, full-history Gitleaks, links, coverage thresholds, and strict browser tests.
Implementation gate
Keep this PR draft and unmerged throughout local implementation and connector review. Automated tests must use mocked GitHub transport and require no live credentials or network access. Final readiness requires focused security/idempotency/stale-head tests, a server-backed request-to-comment proof, complete local validation, and fully green hosted Commitlint and CI on the implementation head.
Closes #122 only after complete implementation, validation, connector review, and explicit owner-authorized merge.