Skip to content

fix(version): resync server.json/package.json to the released 0.1.2 - #10

Merged
jmagar merged 1 commit into
mainfrom
fix/version-drift
Jul 29, 2026
Merged

fix(version): resync server.json/package.json to the released 0.1.2#10
jmagar merged 1 commit into
mainfrom
fix/version-drift

Conversation

@jmagar

@jmagar jmagar commented Jul 28, 2026

Copy link
Copy Markdown
Collaborator

Unblocks the stale release PR #4.

server.json and packages/gotify-rmcp/package.json were hand-bumped to 0.1.3 in b096afa (2026-07-12). No 0.1.3 was ever released — there is no v0.1.3 tag, Cargo.toml is 0.1.2, and .release-please-manifest.json is 0.1.2. The repo's real version is 0.1.2, so those two files come back to it.

Why it went unnoticed: this repo carried an 87-line variant of check-version-sync.sh that never looks at server.json. It reported OK — all 1 files at v0.1.2 while server.json sat at 0.1.3. The script is forked into 4 variants across the fleet:

variant repos checks server.json
87-line rgotify, rtailscale, runifi no
97-line synapse, rarcane yes
43-line rapprise yes (and the only one in CI)
102-line yarr yes

Replaced with the 97-line variant — it now reports 4 files and fails on injected drift (verified with a negative control). Wired into npm-package.yml, which runs on pull_request and push to main.

Note rtailscale and runifi carry the same blind 87-line script and should get the same treatment.

server.json and packages/gotify-rmcp/package.json were hand-bumped to
0.1.3 in b096afa ("chore: publish MCP Registry metadata") on 2026-07-12.
No 0.1.3 was ever released: there is no v0.1.3 tag, Cargo.toml is 0.1.2,
and .release-please-manifest.json is 0.1.2. The repo's real version is
0.1.2, so bring those two files back to it.

The drift survived because this repo carried an 87-line variant of
check-version-sync.sh that never looks at server.json — it reported
"OK — all 1 files at v0.1.2" while server.json sat at 0.1.3. Replace it
with the 97-line variant already used by synapse and rarcane, which
checks all four version-bearing files; it now reports 4 and fails on
injected drift (verified).

Wire it into npm-package.yml, which runs on pull_request and on push to
main, so this cannot drift silently again. rapprise is the only repo in
the fleet that already gated this in CI.
@jmagar
jmagar merged commit 2dd84ee into main Jul 29, 2026
5 checks passed
@jmagar
jmagar deleted the fix/version-drift branch July 29, 2026 00:46
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