Repository navigation
feat(changelog): skip internal PRs, fold follow-up fixes, group related PRs - #837
Merged
Merged
Conversation
Internal changes (CI, build, tests, release tooling) are dropped by title prefix, then a Claude triage call flags other internal PRs and finds fix PRs that finish a feature from the same release. Follow-ups get no entry of their own; they go to Claude as context for the feature's entry. Everything left out is recorded in changelog.ai.txt so reruns agree. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
The triage call now also returns groups of PRs that deliver one change, such as the API and Admin parts of a feature. Each group is sent to Claude as one item and becomes one entry with all its PRs linked. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
This branch was successfully deployed
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
What changed
The changelog generator had three problems.
Skipped PRslist (#5776, #5986, #6013, #6018).scripts/generate-changelog.tsnow triages new PRs before it generates anything:ci,buildortesttype, aci,build,release,e2eortestscope, orchore: merge ...are dropped without an API call.build-toolsis not matched, since it covers user-facing fixes like #5989.report_triagetool sees all remaining new PRs plus the titles of PRs handled on earlier runs. It returns the other internal PRs,{pr, parent}follow-up pairs, and groups of related PRs. The script ignores anything that doesn't point at a real PR of the release, resolves chains (fix of a fix → the feature) and ignores cycles.## Skipped PRsinchangelog.ai.txtwith the reason, e.g.6018 - ci(e2e): ...; internal (\ci` change), skipped automatically.` Reviewers can see what was dropped, and the next run doesn't re-decide.Smaller fixes along the way:
changelog.ai.txtis only scaffolded when missing. Before, a fresh run overwrote it.changelog.mdxexists. Before, it was decided by whether any PR was already known, which crashed whenchangelog.ai.txthad skips but nochangelog.mdxexisted.--dry-runprints what would be left out. WithCLAUDE_KEYset, it runs the triage call too.Testing
--dry-runfor 6.5.0 with its changelog files moved aside: the title filter drops #5776, #5843, #5986, #6013 and #6018, all of which reviewers had removed or which are merges.appendSkippedPRsagainst the real 6.5.0changelog.ai.txtand a missing file, and checked thatextractSkippedPRNumbersreads the result back.resolveTriagewith chains, a cycle, an unknown parent, an internal PR also marked as a follow-up, overlapping groups, and groups that shrink to one PR. Checked theformatItemoutput for a grouped item with a follow-up.CLAUDE_KEYwas available locally. The first real run (or a--dry-runwith the key) will show how it classifies.Workspace: docs.webiny.com · changelog filtering and grouping
🤖 Generated with Claude Code