-
Notifications
You must be signed in to change notification settings - Fork 280
Closed as not planned
Closed as not planned
Copy link
Labels
Description
We looked at 5 open PRs — 2 look great and are ready for review, and 3 need a closer look before merge.
Ready to review 🟢
These PRs passed all checks and are ready for maintainer eyes.
| PR | Title | Author | Lines | Quality |
|---|---|---|---|---|
| #20115 | Rename safe-inputs to mcp-scripts |
@Copilot |
3,851 | lgtm ✨ |
| #19890 | fix: expose GITHUB_TOKEN env var for cross-repo git CLI operations | @Copilot |
158 | lgtm ✨ |
Needs a closer look 🟡
These PRs are on-topic but have gaps to address before merge.
| PR | Title | Author | Lines | Quality |
|---|---|---|---|---|
| #20119 | [log] Add debug logging to MCP inspection, workflow resolution, and repo memory | @github-actions[bot] |
45 | needs-work |
| #20109 | fix: add missing campaign_discovery.cjs module | @Copilot |
197 | needs-work |
| #19744 | feat: portable GH_AW_HOME for self-hosted runner support | @Mossaka |
31,443 | needs-work |
Key gaps:
- [log] Add debug logging to MCP inspection, workflow resolution, and repo memory #20119 — Missing
make agent-finish/go test ./...confirmation in validation block. Logging additions are clean; just needs test suite sign-off. - fix: inline campaign discovery logic in campaign workflow step #20109 — Missing
campaign_discovery.test.cjs. Every other.cjsmodule inactions/setup/js/has a paired test file; this one needs one too. - feat: portable GH_AW_HOME for self-hosted runner support #19744 — Bundles 3 unrelated features: GH_AW_HOME portability, OpenCode engine integration, and compiler improvements. The
.changeset/patch-add-opencode-engine.mddescribes a completely different feature than the PR title. Recommend splitting into focused PRs before review.
Evaluated: 5 · Skipped: n/a (pre-filter file unavailable) · Run: 22829752108
Generated by Contribution Check · ◷
- expires on Mar 9, 2026, 9:07 PM UTC
Reactions are currently unavailable