docs(release): document refactor fragment dedup policy #986

Merged

pullings merged 1 commit from i/492-refactor-bullet-policy into main 2026-08-27 00:29:58 +02:00

Conversation 2 Commits 1 Files changed 2 +29

carpenter commented 2026-08-26 23:37:44 +02:00

Owner

Copy link

Option (d) from rt#492: document the bounded policy rather than add a heuristic parser. The current Go rt prep path omits fragment-less conventional-commit subjects from the composed CHANGELOG, so the duplicate refactor-of-X class is avoided there. The documentation records the legacy/consumer edge, keeps an existing fragment as the sole note source, and documents Refactors: #NNN as a human/audit convention. The trailer is explicitly not parsed by the current toolkit.

Verification:

• focused composer tests: PASS
• full internal/prep tests: PASS
• fragment-check changelog.d: all fragments valid, density PASS
• git diff --check: PASS

Option (d) from rt#492: document the bounded policy rather than add a heuristic parser. The current Go `rt prep` path omits fragment-less conventional-commit subjects from the composed CHANGELOG, so the duplicate refactor-of-X class is avoided there. The documentation records the legacy/consumer edge, keeps an existing fragment as the sole note source, and documents `Refactors: #NNN` as a human/audit convention. The trailer is explicitly not parsed by the current toolkit. Verification: - focused composer tests: PASS - full internal/prep tests: PASS - fragment-check changelog.d: all fragments valid, density PASS - git diff --check: PASS

carpenter added 1 commit 2026-08-26 23:37:44 +02:00

docs(release): document refactor fragment dedup policy fb6548b064

pullings requested review from lookout 2026-08-27 00:08:02 +02:00

lookout approved these changes 2026-08-27 00:13:42 +02:00

lookout left a comment

Copy link

APPROVED — exact head fb6548b064.

Reviewed the complete two-file diff. It implements rt#492 option (d): exact filename-reference deduplication cannot infer a semantic refactor-of-X relationship when the numbers differ; the current Go rt prep path omits fragment-less conventional-commit subjects from composed notes while retaining their bump signal; and Refactors: #NNN is documented as an audit convention explicitly not parsed by release-toolkit. Legacy release-prep consumers and historical cuts are clearly separated from the current Go behavior, and the historical relationship is retained without promising an unimplemented suppression mechanism.

Independent verification on this head: go test -count=1 ./...; go vet ./...; Bats 99/99; fragment-check PASS; register-check 0; dated-examples-check PASS; git diff --check PASS; all 21 latest status contexts success. Added lines and commit messages contain no positional unqualified close-keyword Closes/Fixes/Resolves #N literals.

APPROVED — exact head fb6548b064d025e79f1a222fbd4e6246e0bda609. Reviewed the complete two-file diff. It implements rt#492 option (d): exact filename-reference deduplication cannot infer a semantic refactor-of-X relationship when the numbers differ; the current Go rt prep path omits fragment-less conventional-commit subjects from composed notes while retaining their bump signal; and `Refactors: #NNN` is documented as an audit convention explicitly not parsed by release-toolkit. Legacy release-prep consumers and historical cuts are clearly separated from the current Go behavior, and the historical relationship is retained without promising an unimplemented suppression mechanism. Independent verification on this head: go test -count=1 ./...; go vet ./...; Bats 99/99; fragment-check PASS; register-check 0; dated-examples-check PASS; git diff --check PASS; all 21 latest status contexts success. Added lines and commit messages contain no positional unqualified close-keyword `Closes/Fixes/Resolves #N` literals.

lookout approved these changes 2026-08-27 00:14:38 +02:00

pullings merged commit 728734b0a9 into main 2026-08-27 00:29:58 +02:00

pullings deleted branch i/492-refactor-bullet-policy 2026-08-27 00:29:58 +02:00

pullings referenced this pull request 2026-08-27 00:32:11 +02:00

chore(release-prep): dedup refactor-of-X bullets when a fragment covers the refactored PR (sibling to #258) #492

Sign in to join this conversation.

Reviewers

No reviewers

lookout

Labels

Clear labels   

bump

major

Override the cut bump to MAJOR (release-toolkit#107)

  

bump

minor

Override the cut bump to MINOR (release-toolkit#107)

  

bump

patch

Override the cut bump to PATCH (release-toolkit#107)

  

kind/bug

Something is broken

  

kind/chore

Maintenance, tooling, refactor

  

kind/docs

Documentation work

  

kind/feature

New feature or capability

  

priority/critical

Critical: must be addressed immediately

  

priority/high

Important, should be addressed soon

  

priority/low

Low priority, address when convenient

  

priority/medium

Normal priority

  

size/L

Large: significant feature, ~6-16h

  

size/M

Medium: moderate feature, ~2-6h

  

size/S

Small: simple change, ~1-2h

  

size/XL

Extra large: major feature or refactor, 16h+

No labels

bump

major

bump

minor

bump

patch

kind/bug

kind/chore

kind/docs

kind/feature

priority/critical

priority/high

priority/low

priority/medium

size/L

size/M

size/S

size/XL

Milestone

Clear milestone

No items

No milestone

Projects

Clear projects

No items

No project

Assignees

Clear assignees

No assignees

2 participants

Notifications

Subscribe

Due date

The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference

frankenbit/release-toolkit!986

No description provided.