Skip to content

Bump to 1.9.1; single-source version; force version from release tag …#5

Open
Robbie1977 wants to merge 1 commit into
mainfrom
chore/bump-1.9.1
Open

Bump to 1.9.1; single-source version; force version from release tag …#5
Robbie1977 wants to merge 1 commit into
mainfrom
chore/bump-1.9.1

Conversation

@Robbie1977

Copy link
Copy Markdown

Summary

Fixes the reported version (preview showed 1.8.1 after the #4 release) and stops it drifting again.

Root cause

The version lived in three places that had drifted apart:

  • src/index.ts — hardcoded const VERSION = '1.8.1' (what the server reports)
  • package.json1.8.1
  • server.json1.5.0 (stale)

#4 shipped without bumping any of them.

Changes

  • Bump to 1.9.1: package.json and server.json (was 1.5.0).
  • src/index.ts: VERSION is now imported from package.json (resolveJsonModule is on; real tsc emit + runtime require('../package.json') verified), so there's a single source of truth.
  • docker.yml: new Set version from release tag step (refs/tags/v*) that forces package.json + server.json to the tag before the image build — so the deployed server reports the released version. Mirrors the existing step in publish-mcp.yml (which already sets server.json for the registry).

Effect

  • On main/branch builds: reports the committed version (now 1.9.1).
  • On a v* release tag: package.json/server.json/reported VERSION are all forced to the tag automatically — no more manual three-place edits.

tsc --noEmit clean; docker.yml YAML valid.

…in Docker build

- package.json 1.8.1 -> 1.9.1; server.json 1.5.0 -> 1.9.1 (was stale).
- src/index.ts: VERSION is now imported from package.json (was a hardcoded
  '1.8.1' that drifted from package.json and server.json — the cause of the
  wrong reported version).
- docker.yml: add a 'Set version from release tag' step (on refs/tags/v*) that
  forces package.json + server.json to the tag before the image build, so the
  deployed server reports the released version. Mirrors publish-mcp.yml.
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