chore(branches): close obsolete t3code branches#11
Conversation
Backlog item #7 in docs/ops/project-status.md asked to review and merge t3code/7bcd4587 and t3code/5b6355f9. Both branches no longer exist on origin (verified via git fetch and gh api repos/:owner/:repo/branches); the underlying commits are unreachable. The only Apex/front-end work that ever landed from the t3code family was PR #1 (the marketing worker scaffold). Remove the stale references from project-status.md and move the item to Recently Completed. Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Organization UI Review profile: ASSERTIVE Plan: Pro Plus Run ID: 📒 Files selected for processing (1)
WalkthroughThis PR updates the project status documentation to reflect the completion of Poem
🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches🧪 Generate unit tests (beta)
Comment |
There was a problem hiding this comment.
Actionable comments posted: 1
Caution
Some comments are outside the diff and can’t be posted inline due to platform limitations.
⚠️ Outside diff range comments (1)
docs/ops/project-status.md (1)
184-184:⚠️ Potential issue | 🟡 Minor | ⚡ Quick winUpdate the item range to match the renumbered backlog.
After removing the t3code branches backlog item and renumbering, there are now only 7 items in the backlog (items 1-7). This line should be updated to reflect the correct range.
📝 Proposed fix
-Ordered. Each item has a verifiable Done. Items 1-5 close Phase 1; items 6-8 prep hosted ops and Phase 2. +Ordered. Each item has a verifiable Done. Items 1-5 close Phase 1; items 6-7 prep hosted ops and Phase 2.🤖 Prompt for AI Agents
Verify each finding against current code. Fix only still-valid issues, skip the rest with a brief reason, keep changes minimal, and validate. In `@docs/ops/project-status.md` at line 184, Update the backlog item range text in the sentence that currently reads "Items 1-5 close Phase 1; items 6-8 prep hosted ops and Phase 2" to reflect the renumbered backlog of seven items—change "items 6-8" to "items 6-7" so the sentence becomes "Items 1-5 close Phase 1; items 6-7 prep hosted ops and Phase 2."
🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
Inline comments:
In `@docs/ops/project-status.md`:
- Around line 232-237: Update the text under the "Close obsolete `t3code/*`
branches" section to disambiguate "this doc backlog item `#7`": replace or append
the phrase "this doc backlog item `#7`" with a clarifying qualifier such as "this
doc backlog item `#7` (former)" or "this doc backlog item `#7` — now removed" so
readers won't confuse it with the current backlog item `#7`; ensure the change
appears in the same paragraph that mentions the drives/backlog reference and
keep surrounding references (e.g., `t3code/7bcd4587`, `t3code/5b6355f9`, and PR
`#1`) intact.
---
Outside diff comments:
In `@docs/ops/project-status.md`:
- Line 184: Update the backlog item range text in the sentence that currently
reads "Items 1-5 close Phase 1; items 6-8 prep hosted ops and Phase 2" to
reflect the renumbered backlog of seven items—change "items 6-8" to "items 6-7"
so the sentence becomes "Items 1-5 close Phase 1; items 6-7 prep hosted ops and
Phase 2."
🪄 Autofix (Beta)
Fix all unresolved CodeRabbit comments on this PR:
- Push a commit to this branch (recommended)
- Create a new PR with the fixes
ℹ️ Review info
⚙️ Run configuration
Configuration used: Organization UI
Review profile: ASSERTIVE
Plan: Pro Plus
Run ID: 7af6b806-7403-4336-b8b2-92d23919284c
📒 Files selected for processing (1)
docs/ops/project-status.md
PR #2 added `permissions: administration: write` to the cleanup workflow, but `administration` is not a valid GITHUB_TOKEN scope. GitHub rejected the workflow at evaluation time, silently dropping every `pull_request.closed` event for PRs #2--#9 and accumulating eight stale `preview/pr-N` Neon branches that tripped the 10-branch free-tier cap (blocks PR #10/#11/#12 deploys with HTTP 422). Drop the invalid permission and the `deleteAnEnvironment` step that required it; rename the file so GitHub registers a fresh workflow id instead of reusing the wedged record; validate the resolved PR number is a positive integer before deleting anything. Stale Neon branches still need a one-time operator purge -- agent is forbidden from calling `neondatabase/delete-branch-action` autonomously. Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>
PR #2 added `permissions: administration: write` to the cleanup workflow, but `administration` is not a valid GITHUB_TOKEN scope. GitHub rejected the workflow at evaluation time, silently dropping every `pull_request.closed` event for PRs #2--#9 and accumulating eight stale `preview/pr-N` Neon branches that tripped the 10-branch free-tier cap (blocks PR #10/#11/#12 deploys with HTTP 422). Drop the invalid permission and the `deleteAnEnvironment` step that required it; rename the file so GitHub registers a fresh workflow id instead of reusing the wedged record; validate the resolved PR number is a positive integer before deleting anything. Stale Neon branches still need a one-time operator purge -- agent is forbidden from calling `neondatabase/delete-branch-action` autonomously. Co-authored-by: Claude Opus 4.7 <noreply@anthropic.com>
Summary
docs/ops/project-status.mdasked to review and merget3code/7bcd4587andt3code/5b6355f9.git fetch originandgh api repos/:owner/:repo/branchesthat neither branch exists on origin; underlying commits are unreachable.t3code/*family was PR feat(apex): add marketing worker at agent-paste.sh #1 (4bde837 feat(apex): add marketing worker at agent-paste.sh).Risk
Docs-only.
pnpm verifygreen (49/49 turbo tasks, all cached).Test plan
pnpm verifygh api repos/:owner/:repo/branchesconfirms not3code/*branches remainCo-Authored-By: Claude Opus 4.7 noreply@anthropic.com
Summary by CodeRabbit