Retry Smart CI merge-ref resolution against the run control base - #2404
Conversation
Codex Review SummaryThis comment shows the latest Codex review activity on this pull request.
ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings. |
|
Fresh exact-head reproduction for this repair from Alpha PR #2388:
This is the same stale control-event base defect as #2401 on a newly synchronized, current-main-containing head. Please use run 33699697079 as a real regression fixture for this PR's retry path. No Smart CI files were changed by the Alpha lane. |
|
A second in-lane PR reproduced the same stale control-event base failure after a current-base refresh:
#2299 remains unmerged. This is dependency evidence only; no request to bypass or rerun the gate. |
|
Dependency-ready checkpoint for the beta-platform-integrity owner:
Alpha PRs #2299 and #2388 are both held on reproduced #2401 stale-base failures. This is a merge-readiness handoff only; the platform lane retains merge ownership. |
Summary
pull_request_targetSmart CI run to one trusted control base (github.sha) instead of the potentially stale event payload baseThe issue's earlier SHA/tree-straddle diagnosis was corrected before implementation. A fetched local
FETCH_HEADis coherent; the observed defect was stale event base metadata versus an already-recomputed live merge ref.Verification
ERR_MODULE_NOT_FOUNDbefore the helper existednode --test scripts/ci/smart-ci/*.test.mjs: 85/85 passedsmart-ci-shadow.ymlparsed with PyYAMLnode --checkpassed for resolver and resolver spec--config-envgit diff --check origin/main...HEAD: passed33701229648, Workflow Lint33701229667, Planner Self-Test33701229556, Plan and Required Gate33701225976: passed33701225976passed against baseb66da8fd43eebad7ee3a7c21c38df7e48fe9ed97; the repaired control workflow becomes active on main after merge and remains under CI-02: Deterministic risk/impact planner with fail-closed policy, shadow mode and recall report #2326 observationDocumentation
docs/STATUS.mdupdated (issue-scoped repair; no roadmap or operating-mode change)docs/IMPLEMENTATION_MASTERPLAN.mdupdated (no roadmap or mode change)docs/TESTING_GUIDE.mdupdated (no recurring operator command changed)CLAUDE_SYNC_PACKET62f8686421e1d5274c01861e908288d7fae56c4fTracking
CI Workflow Validation
63f3816f568422c852e324ebd9621cd928b78cc9Risk Notes