Fix skill parity: sync .codex afx skill copies missed by #1143 - #1332
Closed
waleedkadous wants to merge 1 commit into
Closed
Fix skill parity: sync .codex afx skill copies missed by #1143#1332waleedkadous wants to merge 1 commit into
waleedkadous wants to merge 1 commit into
Conversation
…nly update PR #1143 (cron exitCode) updated .claude/skills/afx/SKILL.md in both trees but not the .codex twins — its CI green predated the skill-parity tests (1280 T17, 1273 wait-discipline), so main went red at merge. Mechanical sync, md5-verified identical, parity suites green.
Contributor
Author
|
Superseded: PR #1320 merged first carrying an identical .codex parity fix (1273's phase-7 tests caught the same #1143 gap independently; both fixes produce byte-identical results — md5-verified on main post-merge, parity suites green). Two independent detections and fixes of one defect within the hour; the cross-tree guards are earning their keep. |
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
1 participant
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.
PR #1143 updated
.claude/skills/afx/SKILL.mdin both trees (the new### afx croncondition-environment docs) but not the.codextwins. Its 6/6 CI green was from July 6 — predating the skill-parity guards the repo has since grown (1280's T17, 1273's wait-discipline parity test) — so main went red at merge: 3 failures across two parity suites.Mechanical fix: copy the two .claude files over their .codex twins. md5-verified identical; parity tests green locally.
Gate-check lesson recorded by the architect: a stale CI green must be re-validated against invariants added since the run — zero drift of the touched files is not zero drift of the repo's expectations.
🤖 Generated with Claude Code