Skip to content

docs: a port's PR title is its CHANGELOG entry (runbook Step 4.5, ADR 0003 D8) - #45

Merged
iancooper merged 3 commits into
mainfrom
docs/runbook-changelog-via-pr-title
Sep 28, 2026
Merged

iancooper merged 3 commits into
mainfrom
docs/runbook-changelog-via-pr-title

Conversation

@iancooper

@iancooper iancooper commented Sep 27, 2026 •

Copy link
Copy Markdown
Member

Fixes a runbook defect found during upstream sync #1. Refs #28.

Step 4.5 of .agent_instructions/upstream_sync.md said to "append a CHANGELOG.md entry" for each port. But CHANGELOG.md is generated at release: after-release.yml runs eng/update-changelog.ps1, which builds the version's section from the GitHub release notes (one bullet per merged PR title) and inserts it below <!-- next-release -->. There's no unreleased section to append to, and a hand edit would sit outside any version heading.

Step 4.5 now says:

  • The PR title is the changelog entry, so write it for a Fences user.
  • Don't hand-edit CHANGELOG.md.
  • The upstream SHA goes in the commit message and the PR body, not the title.
  • The "never rewrite Polly's history" rule stays.

This follows the decision recorded on #28. Ports #42, #43 and #44 already work this way.

ADR 0003 D8 is amended to match. It said a port "adds a CHANGELOG.md entry". It now says the PR title is the entry and ports don't hand-edit the file. An amendment note under ## Status records what changed and why. This is the first ADR amendment in the repo: it keeps Accepted and adds a dated note, following the convention in .claude/commands/adr/write_adr_metadata.md.

markdownlint is clean. pyspelling reports nothing new (its only hits are the gitignored PROMPT.md and baseline/README.md).

🤖 Generated with Claude Code

Runbook Step 4.5 said to append a CHANGELOG.md entry by hand, but the file is
generated at release by eng/update-changelog.ps1 from the GitHub release notes
(PR titles), inserted below <!-- next-release -->. There is no unreleased
section to append to. Decided during upstream sync #1 (#28).

Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
D8 said a port adds a CHANGELOG.md entry, but the file is generated at
release from merged PR titles and has no unreleased section to add to. D8 now
matches runbook Step 4.5, with an amendment note under Status.

Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
@iancooper iancooper changed the title docs: upstream sync runbook, a port's PR title is its CHANGELOG entry docs: a port's PR title is its CHANGELOG entry (runbook Step 4.5, ADR 0003 D8) Sep 27, 2026
@iancooper
iancooper merged commit c41497a into main Sep 28, 2026
39 of 41 checks passed
@iancooper
iancooper deleted the docs/runbook-changelog-via-pr-title branch September 28, 2026 22:43
@github-actions github-actions Bot locked and limited conversation to collaborators Sep 28, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant