chore: bump MCP to 1.1.0; include CLI core version in User-Agent - #37
Conversation
User-Agent on outbound API requests now reports both the MCP server version and the resolved @absmartly/cli core version: ABsmartly-MCP-Server/1.1.0 (CLI-core/1.7.0) Why bother: - Backend logs / dashboards can now tell which MCP version is calling and which CLI core it shipped with. Previously the UA only carried the MCP version, so when a payload-shape question came up, there was no way to know which CLI core (and therefore which experimentToInput/build-from-template behavior) produced it. - The MCP and CLI core ship on different cadences. Pinning both in the UA makes "which version regressed this?" answerable from request logs alone. scripts/sync-version.js (the prebuild/predev/predeploy hook) now reads the installed @absmartly/cli's package.json at build time and writes a CLI_CORE_VERSION constant into src/version.ts alongside MCP_VERSION. If node_modules hasn't been installed yet (fresh checkout, only happens before npm ci), the constant becomes "unknown" — better than crashing the build. Version bump from 1.0.0 → 1.1.0 (minor). Earlier PR #29 added param validation to execute_command which is an observable change to MCP tool behavior (rejects unknown params with "did you mean" suggestions instead of silently no-op'ing), and the catalog gained new params on the experiment-transition commands (note, unarchive, required reason). Both are additive — no breaking changes — hence minor not major. manifest.json (the DXT extension manifest) bumped to match. package-lock.json picks up the new top-level version automatically.
|
Caution Review failedThe pull request is closed. ℹ️ Recent review info⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Pro Run ID: ⛔ Files ignored due to path filters (1)
📒 Files selected for processing (5)
WalkthroughThe package version is bumped from Estimated code review effort🎯 1 (Trivial) | ⏱️ ~4 minutes Possibly related PRs
Suggested reviewers
Poem
✨ Finishing Touches📝 Generate docstrings
🧪 Generate unit tests (beta)
Comment |
Why
Outbound API requests now report both versions in the User-Agent:
```
ABsmartly-MCP-Server/1.1.0 (CLI-core/1.7.0)
```
The MCP and CLI core ship on different cadences. Pinning both in the UA makes "which version regressed this?" answerable from backend request logs alone — previously the UA only carried the MCP version, so when a payload-shape question came up there was no way to know which CLI core (and therefore which `experimentToInput` / `buildPayloadFromTemplate` behavior) produced it.
How
Verification
```
$ ./scripts/ensure-dxt-bundle.sh && npm run typecheck
Updated src/version.ts to MCP=1.1.0, CLI core=1.7.0
Test plan
Summary by CodeRabbit