Skip to content

ci(release): summarize a stable release since the previous stable tag - #1491

Merged
github-actions[bot] merged 1 commit into
mainfrom
claude/release-notes-skip-rc-tags
Oct 4, 2026
Merged

github-actions[bot] merged 1 commit into
mainfrom
claude/release-notes-skip-rc-tags

Conversation

@Dumbris

@Dumbris Dumbris commented Oct 4, 2026

Copy link
Copy Markdown
Member

Pull Request

Description

The v0.70.0 release notes covered only the rc.4 → rc.5 fixes, described the release as "stability improvements", and missed profiles v3, per-client credentials and agent visibility. I rewrote them by hand after the release.

Cause. release.yml's generate-notes job finds the previous tag with git describe --tags --abbrev=0 HEAD^. After an RC cycle, the nearest tag is the last RC, so the commit range sent to the summarizer was v0.70.0-rc.4..v0.70.0, not v0.69.0..v0.70.0.

Fix. Exclude prerelease tags (--exclude '*-*') when looking up the previous tag:

  • In release.yml, the generate-notes job, which runs for stable tags only.
  • In scripts/generate-release-notes.sh, used for local runs: stable tags get the same exclusion, and RC tags keep comparing with the nearest tag of any kind.

The release notice problem is separate and not changed here. It comes from the unmerged retire PR #1362.

Testing

  • I have tested these changes locally
  • I have added/updated tests that prove my fix is effective or my feature works
  • All existing tests pass

get_previous_tag against real tags returns:

Tag Previous tag
v0.70.0 v0.69.0 (was rc.4/rc.5)
v0.70.0-rc.5 v0.70.0-rc.4
v0.70.0-rc.2 v0.70.0-rc.1
v0.69.0 v0.68.0

The workflow expression, simulated at v0.70.0, returns v0.69.0. bash -n and a YAML parse both pass. There is no test harness for the release workflow.

🤖 Generated with Claude Code

generate-notes found the previous tag with 'git describe --tags HEAD^',
which after an RC cycle is the last RC, so a stable release was
summarized as only its final RC delta: v0.70.0's notes covered rc.4..rc.5
and missed profiles v3, per-client credentials and agent visibility.

Exclude prerelease tags ('*-*') when looking up the previous tag for a
stable release, in release.yml's generate-notes job and in
scripts/generate-release-notes.sh (RC tags keep comparing with the
nearest tag of any kind).
@cloudflare-workers-and-pages

Copy link
Copy Markdown

Deploying mcpproxy-docs with  Cloudflare Pages  Cloudflare Pages

Latest commit: 10570f8
Status: ✅  Deploy successful!
Preview URL: https://402bc6fd.mcpproxy-docs.pages.dev
Branch Preview URL: https://claude-release-notes-skip-rc.mcpproxy-docs.pages.dev

View logs

@github-actions github-actions Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Approved (Model B): Paperclip review verdicts = ACCEPT and qa-gate green at this head SHA. Arming auto-merge; GitHub merges when all required checks pass.

@github-actions
github-actions Bot enabled auto-merge (squash) October 4, 2026 04:15
@github-actions
github-actions Bot merged commit a8a4f16 into main Oct 4, 2026
37 of 41 checks passed
@codecov-commenter

Copy link
Copy Markdown

⚠️ Please install the 'codecov app svg image' to ensure uploads and comments are reliably processed by Codecov.

Codecov Report

✅ All modified and coverable lines are covered by tests.

📢 Thoughts on this report? Let us know!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants