<@U0BME18CF7S> hey. please review 6335 from SDK, a...
# automations-debug
c
@fresh-machine-37269 hey. please review 6335 from SDK, and approve or request changes or comment as you think fit
šŸ‘€ 1
f
šŸ¤– On it! View progress here: oss-agent-canvas.ngrok.dev/conversations/…
āœ… Done! Review complete — I went through PR 6335 ("ci: remove review artifacts in the release PR") and approved it āœ… What it does: Moves
.pr/
evidence cleanup from a post-merge cleanup PR into the release-prep PR itself (implements #6334). The cleanup script is staged before the version commit in
prepare-release.yml
, the
cleanup-after-merge
job and its stale exemptions are removed, and the release-tree guard stays with updated remediation messaging. What I verified: • Ran the 5 new regression tests locally against real Git fixtures — all pass (they exercise the actual workflow steps, including symlink safety, nested binaries, and the modify/delete-conflict refresh path — nice coverage) •
ruff check
,
ruff format --check
, and
pyright
all clean on the new/changed Python files • All 39 CI checks green on head
afca1172
, and the docs/skills/AGENTS.md updates are consistent with the new lifecycle • No unrelated path can be deleted — the script only stages tracked root
.pr/
entries and never executes or traverses evidence Two non-blocking rollout notes (already flagged in the PR description): open cleanup PR #6304 needs manual reconciliation, and existing release branches need the documented refresh runbook.
c
So I merged those two on the sdk, is there anything else to do for the next release branch or wip branch if any?