Skip to content

test: every version string agrees on every PR - #70

Merged
tonydzi merged 2 commits into
mainfrom
fix/version-consistency-ci
Nov 2, 2026
Merged

tonydzi merged 2 commits into
mainfrom
fix/version-consistency-ci

Conversation

@tonydzi

@tonydzi tonydzi commented Nov 2, 2026 •

Copy link
Copy Markdown
Owner

v0.3.4 was tagged on 2026-10-31 and never reached PyPI: the release commit changed only CHANGELOG.md, and the publish job stopped on its version check, after the tag. PyPI stayed on 0.3.3 for a week.

This runs the comparison in the normal test suite: pyproject, __init__, the three places in server.json, and the top released CHANGELOG section. A section whose header says unreleased may lead.

Red proof: with pyproject set back to 0.3.3 the test fails and names each place that differs.

🤖 Generated with Claude Code


Note

Low Risk
Test-only change with no runtime or release workflow edits; it only adds an earlier guardrail for version bumps.

Overview
Adds tests/test_version_consistent.py so CI fails before merge if release metadata is out of sync—addressing the v0.3.4 case where only CHANGELOG was bumped, the tag went out, and PyPI publish aborted too late.

test_all_versions_agree treats pyproject.toml as canonical and compares __init__.__version__, server.json (top-level version, package version, and the runtimeArguments PyPI pin), and the first CHANGELOG section whose header is not marked unreleased. Mismatches get a single assertion listing each offending location.

Companion tests lock in parsing rules: the broken v0.3.4 changelog-only bump is detected, an unreleased leading section is allowed, headers like v0.3.3.post1 / v0.4.0-rc1 do not count as the stable triple, and a stale == pin in server.json is caught.

Reviewed by Cursor Bugbot for commit 1ebdd83. Bugbot is set up for automated code reviews on this repo. Configure here.

v0.3.4 was tagged and never reached PyPI because only CHANGELOG.md was bumped;
the publish job's own version check stopped it after the tag. Same comparison now
runs in the test suite, red before merge.

Assisted-by: claude-code/claude-opus-5-5
Machine: MacPro192
Account: anton
Operator: Anton
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>

@cursor cursor Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Cursor Bugbot has reviewed your changes using default effort and found 1 potential issue.

Fix All in Cursor

❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.

Reviewed by Cursor Bugbot for commit 5b52867. Configure here.

Comment thread tests/test_version_consistent.py Resolved
Assisted-by: claude-code/claude-opus-5-5
Machine: MacPro192
Account: anton
Operator: Anton
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@tonydzi
tonydzi merged commit 30261b1 into main Nov 2, 2026
8 checks passed
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.

1 participant