fix(ci): remove dead cross-repo repair-review-feedback job - #43
Conversation
pr-review-fix-scheduler.yml in ContextualWisdomLab/.github was hardened
to reject any caller whose github.repository is not
ContextualWisdomLab/.github itself. RankWeave's repair-review-feedback
job called it cross-repository at a pinned commit SHA, a shape that
check can never satisfy, so every hourly run failed with zero jobs
scheduled ("workflow file issue") for as long as the hardening has been
in place -- 19+ consecutive failures observed over 18+ hours.
Review-feedback repair moves to a central rankweave-hourly-review-repair.yml
caller in ContextualWisdomLab/.github (companion PR), matching the pattern
already proven for every other product repository in the organization.
Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01K8VWdPkLwNYHGAKWowfueJ
|
Warning Review limit reachedNext included review available in 38 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: Pro Plus Run ID: 📒 Files selected for processing (1)
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Pro Plus Run ID: 📒 Files selected for processing (3)
🚧 Files skipped from review as they are similar to previous changes (1)
Included review availability: Your plan provides up to 1 included review per hour; 0 remain after this review. 📝 WalkthroughWalkthrough시간별 상용화 루프에서 고정 커밋 기반 Changes시간별 상용화 루프 거버넌스
Estimated code review effort: 3 (Moderate) | ~20 minutes Merge Risk: ⚪ Minimal · up to This change removes a failed CI job and updates its dependency conditions; no actionable merge-blocking risk remains after normal checks and review. 🚥 Pre-merge checks | ✅ 3 | ❌ 2❌ Failed checks (2 warnings)
✅ Passed checks (3 passed)
Full details: Docstring CoverageExplanation Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%. Docstring coverage is scoped to functions touched by this diff. Analyzed 10 functions across 1 files. (2 skipped: 2 unsupported.) ✨ Finishing Touches 💡 1📝 Generate docstrings 💡
🧪 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 |
![devin-ai-integration[bot]](https://avatars.githubusercontent.com/in/811515?s=60&v=4){kind=small}
This comment was marked as resolved.
This comment was marked as resolved.
Sorry, something went wrong.
This comment was marked as resolved.
This comment was marked as resolved.
Sorry, something went wrong.
This comment was marked as resolved.
This comment was marked as resolved.
Sorry, something went wrong.
| "read": { | ||
| "*": "allow", | ||
| ".git/**": "deny", | ||
| "opencode.json": "deny", | ||
| ".env": "deny", | ||
| ".env.*": "deny" | ||
| "*": "deny", | ||
| "AGENTS.md": "allow", | ||
| "ARCHITECTURE.md": "allow", | ||
| "CHANGELOG.md": "allow", | ||
| "CLAUDE.md": "allow", | ||
| "LICENSE": "allow", | ||
| "README.md": "allow", | ||
| "pyproject.toml": "allow", | ||
| "uv.lock": "allow", | ||
| "docs/**": "allow", | ||
| "src/**": "allow", | ||
| "tests/**": "allow" |
There was a problem hiding this comment.
📝 Info: PR scope broader than title
Beyond removing the dead job, the diff rewrites both OpenCode read permission blocks to deny-all-with-allowlist and deletes the entire PR_MESSAGE.md title/body pipeline in favor of a static title. Both are captured by the CHANGELOG and new tests, so they read as intentional, but the change is wider than the title suggests.
Was this helpful? React with 👍 or 👎 to provide feedback.
1 participant
Summary
hourly-commercialization-loop.yml'srepair-review-feedbackjob calledContextualWisdomLab/.github'spr-review-fix-scheduler.ymlcross-repository at a pinned commit SHA.github.repository == ContextualWisdomLab/.github). A cross-repository caller can never satisfy that check, so every hourly run has failed with zero jobs scheduled ("This run likely failed because of a workflow file issue") for as long as the hardening has been in place -- confirmed via 19 consecutive failed runs over the last 18+ hours (gh run list,jobsAPI returningtotal_count: 0).develop-next-product-gap'sneeds/ifto depend only oninspect-pr-queueandrevalidate-pr-queue(unaffected, sincepr-review-merge-scheduler.yml's own trust check is a no-op and still accepts cross-repo callers).rankweave-hourly-review-repair.ymlcaller matching the pattern already used by every other product repository, so review-feedback repair keeps running without RankWeave needing to track a pinned SHA for this reusable workflow ever again.Test plan
actionlintclean (pre-existing shellcheck style nits only, unchanged)python -m pytest -q tests/test_hourly_commercialization_workflow.py(16 tests)python -m coverage run -m pytest -q-- 661 passed, 100% line/branch coveragepython -m ruff check .clean🤖 Generated with Claude Code
https://claude.ai/code/session_01K8VWdPkLwNYHGAKWowfueJ
Summary by CodeRabbit
버그 수정
문서