Skip to content

test(ci): remove stale local PR steward contract after centralization - #154

Open
seonghobae wants to merge 1 commit into
mainfrom
fix/remove_stale_hourly_pr_steward_test_20260910
Open

test(ci): remove stale local PR steward contract after centralization#154
seonghobae wants to merge 1 commit into
mainfrom
fix/remove_stale_hourly_pr_steward_test_20260910

Conversation

@seonghobae

Copy link
Copy Markdown
Contributor

RCA

Protected main no longer contains .github/workflows/hourly-pr-steward.yml; PR merge/review governance was centralized in ContextualWisdomLab/.github. However services/account_unification/tests/test_hourly_pr_steward.py still unconditionally opens that deleted local workflow.

Fresh hosted evidence on unrelated #153 merge ref failed five tests with the same FileNotFoundError for that absent workflow. This is a repository baseline defect, not a .env migration 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.

@chatgpt-codex-connector

Copy link
Copy Markdown

You have reached your Codex usage limits for code reviews. You can see your limits in the Codex usage dashboard.
To continue using code reviews, add credits to your account and enable them for code reviews in your settings.

@coderabbitai

coderabbitai Bot commented Sep 9, 2026

Copy link
Copy Markdown

Warning

Review limit reached

Next included review available in 48 minutes.

Check out review usage here.

View limit details

Limit 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.

Learn how review limits work.

Review configuration:

⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Advanced

Run ID: efe99c10-c197-48b1-8163-d3c2eb6594f8

📥 Commits

Reviewing files that changed from the base of the PR and between 7d9151c and 85deb47.

📒 Files selected for processing (1)
  • services/account_unification/tests/test_hourly_pr_steward.py

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.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant