feat: implement issue #820 — Per-repo caller stub + cross-repo redispatch - #828
Conversation
|
You have reached your Codex usage limits for code reviews. You can see your limits in the Codex usage dashboard. |
|
Note Gemini is unable to generate a review for this pull request due to the file types involved not being currently supported. |
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Organization UI Review profile: ASSERTIVE Plan: Pro Run ID: 📒 Files selected for processing (1)
📝 WalkthroughWalkthroughAdds ChangesLint Workflow Bats Registration
Estimated code review effort🎯 1 (Trivial) | ⏱️ ~2 minutes Possibly related issues
Possibly related PRs
Suggested labels
🚥 Pre-merge checks | ✅ 4 | ❌ 1❌ Failed checks (1 warning)
✅ Passed checks (4 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches🧪 Generate unit tests (beta)
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 |
Dev-Lead — review-changes (no-changes)No changes were needed for this PR. |
|
Advisory bots were rate-limited; auto-approval is withheld until they recover. pr-review-sweep will re-review this PR after 2026-06-20T06:25:27Z. |
donpetry-bot
left a comment
There was a problem hiding this comment.
Automated review — APPROVED ✓
Risk: LOW
Reviewed commit: 3cc4de3f8a4acef7e890f6f8a05a8f48fdb3936f
Review mode: triage-approved (single reviewer)
Summary
One-line CI change: registers the existing tests/test_initiative_planner_redispatch.bats in lint.yml's bats run list so the redispatch regression test is actually executed in CI. The substantive issue #820 work for this repo (redispatch.sh TARGET_REPO/-f target_repo support and the test file itself) already landed on main; this PR wires the test into CI enforcement.
Linked issue analysis
Closes #820 (Per-repo caller stub + cross-repo redispatch). That story spans two repos: the public petry-projects/.github stub (AC #1, #3) and the .github-private redispatch.sh + test work (AC #2, #4, #5). On the .github-private side, redispatch.sh already carries full TARGET_REPO support (lines 28/36/60/63/65 at head) and tests/test_initiative_planner_redispatch.bats already exists in main but was NOT in lint.yml's bats list — so it never ran in CI. This PR adds that one missing registration line, so AC #5's regression test (covering both the dogfood default target_repo=petry-projects/.github-private and the TARGET_REPO=acme/widgets fleet override) is now enforced. Verified the test passes via the green bats check.
Findings
No issues. Change is a single additive line with correct backslash line-continuation, inserted in the existing alphabetically-grouped bats list. The referenced test file exists at head (3165 bytes) and passes. Secret-scanning MCP tool was not permitted in this environment; noted and skipped — the gitleaks CI check passed and the added content is a test-file path with no secret material.
CI status
All required checks green: Lint, ShellCheck/shellcheck, bats (confirms the newly-registered test runs and passes), unit-tests, CodeQL (actions + python), SonarCloud (Quality Gate passed, 0 new issues), Agent Security Scan, gitleaks secret scan, validate-agent-profiles, AgentShield. Advisory bots (Codex, Gemini, CodeRabbit) did not produce findings — they were rate-limited or unsupported for the file type, not failures. mergeStateStatus is BLOCKED pending the required org-leads human review.
Reviewed automatically by the PR-review agent (single-reviewer mode: fable 5). Reply if you need a human review.
Dev-Lead — waiting on PR blockers (intent: review-changes)PR: #828 |
|
Note @don-petry I reviewed this PR and no code changes were needed, but it still has blocking checks or reviews (failing or cancelled checks, or changes-requested reviews), so I cannot mark it done yet. I'll re-check automatically. |
Dev-Lead — fix-bot-comment (no-changes)Agent reasoning |
|
Advisory bots were rate-limited; auto-approval is withheld until they recover. pr-review-sweep will re-review this PR after 2026-06-20T07:55:38Z. |
|
Dev-Lead — review-changes (no-changes)No changes were needed for this PR. |
|
Advisory bots were rate-limited; auto-approval is withheld until they recover. pr-review-sweep will re-review this PR after 2026-06-20T08:04:10Z. |



Closes #820
Implemented by dev-lead agent. Please review.
Summary by CodeRabbit