test(ci): remove stale local PR steward contract after centralization - #154
test(ci): remove stale local PR steward contract after centralization#154seonghobae wants to merge 1 commit into
Conversation
|
You have reached your Codex usage limits for code reviews. You can see your limits in the Codex usage dashboard. |
|
Warning Review limit reachedNext included review available in 48 minutes. View limit detailsLimit details: You’ve used the included review currently available. You've used all free OSS reviews for now. Wait for the free limit to reset to keep reviewing this public repository. Review configuration: ⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Advanced Run ID: 📒 Files selected for processing (1)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
RCA
Protected
mainno longer contains.github/workflows/hourly-pr-steward.yml; PR merge/review governance was centralized inContextualWisdomLab/.github. Howeverservices/account_unification/tests/test_hourly_pr_steward.pystill unconditionally opens that deleted local workflow.Fresh hosted evidence on unrelated #153 merge ref failed five tests with the same
FileNotFoundErrorfor that absent workflow. This is a repository baseline defect, not a.envmigration failure.Repair
Delete only the stale static contract test. Do not recreate the completed local merge-steward workflow, weaken current CI, or alter
.github/workflows/hourly-product-development.yml. The current central governance workflows and their own conformance tests remain authoritative.This follows the repository rule that completed self-modifying/source-fix workflow copies are removed after central ownership is established; retaining a test for a deleted local owner creates a false failing contract.
Verification
This PR must run the complete current-head CI/security/review gates. The deletion does not exempt any production Python line/branch from the existing 100% coverage requirement and does not change application behavior.