Repository navigation
fix(release): drop nine changelog fragments a stacked merge re-added after release - #11141
Conversation
Nine docs-7537-b-move fragments were consumed and deleted by the release commit 4f1bccf (#11088), then re-added by the squash of #11062: its stacked branch kept them from layer commit 7131210, and its merge of main (f87335d, merge base 9b6151a) predated their add and delete on main, so the merge kept the branch's copies. Release PR #11112 then released them a second time, and changelog-parity-bump failed with REPEATED CHANGELOG BODY in eight plugins; planning repeats the same bullet without tripping the check. Refs: #11112 Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Codex Review SummaryThis comment shows the latest Codex review activity on this pull request.
ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings. |
|
claude-review reviewed No findings. The diff only deletes nine What I checked myself:
Author-claimed and not verified by me: the history of #11062, #11088 and #11056, the The PR states no closing issue, only |
|
claude-security-review reviewed No security issues found. The diff deletes nine
critical-findings: 0 |
Refs: #11112
Summary
Release PR #11112 fails
check-plugins->changelog-parity-bumpwithREPEATED CHANGELOG BODYin eight plugins: architecture, code-tidying, corrections, coupling, harness-config, improvement, review and session-flow. It releases ninedocs-7537-b-move-*fragments a second time. Release #11088 (4f1bccff20) already consumed and deleted them. The ninth fragment is planning's. Planning's new entry repeats a bullet that shipped in 0.126.5, but the check misses it because 0.126.5 also has a### Fixedsection.Fix
This PR deletes the nine stale fragments. It changes no release code: the release cleanup deleted every fragment it consumed (
git show --name-status 4f1bccff20).How they came back: #11062 is a stacked layer of the #7537 split.
713121026e.f87335dd0b, has merge base9b6151ae38. That base predates both the fragments' add on main (#11056,2de0fb5e42) and their deletion (4f1bccff20).5b2ca68e19re-added them.Verification
scripts/check-changelog-parity.sh --check-bump origin/mainpasses on this branch.scripts/release-plugins.sh --date 2026-10-30on this branch's tree and committed the result. On that commit,CHANGELOG_HEAD_REF=release/plugins scripts/check-changelog-parity.sh --check-bump <this branch>reports that every changed version has its entry, with exit 0. The simulated release bumps only autonomy, guardrails, prototype, review (fromfix-security-lane-s10-bar), work-items and the pr-pipeline convention.scripts/check-affected.sh: 7 passed, 8 skipped.Related
.github/scripts/release-publish-plugins/decide-rebuild.sh:28-31), and the next scheduled run,41 */3 * * *, rebuilds it. Aworkflow_dispatchwithaction: buildrebuilds it sooner.check-changelog-fragments.shthat rejects a fragment path a release commit already deleted would catch this case.🤖 Generated with Claude Code