release: sync SDK + MCP to omnigraph-server v0.8.1#14
Open
aaltshuler wants to merge 1 commit into
Open
Conversation
Scripted pipeline sync (sync-spec -> generate -> check-versions): the v0.8.1 openapi delta is version-metadata only — no route or schema changes, so no generated-type churn. Package versions and the MCP health assertion follow the engine version. Engine release notes: https://github.com/ModernRelay/omnigraph/releases/tag/v0.8.1 (note: this engine release is binaries-only; crates.io publication resumes at 0.9.0 — irrelevant to this SDK, which consumes the tagged openapi.json).
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Scripted sync to the v0.8.1 engine tag, following #13's pattern:
omnigraph.serverVersion→ 0.8.1;sync-specfetched the tag'sopenapi.json(check-driftclean).info.versionline, sogenerateproduced no type changes (onlyversion.gen.ts).check-versionsgreen); the MCP health test's pinnedsdkServerVersionupdated.Engine v0.8.1 highlights relevant to SDK users: filtered
nearest()/bm25()now applies filters before the search (result sets change — previously missing rows appear), undirected traversal$a <edge> $bin .gq, enum widening in schema migration. All are query/engine-level — no SDK API changes.Greptile Summary
This PR is a scripted version bump from 0.8.0 to 0.8.1 across the SDK and MCP packages, syncing to the omnigraph-server v0.8.1 engine tag. The OpenAPI spec delta is limited to
info.version, so no type generation occurred and no wire-surface changes were introduced.package.json(serverVersion), bothpackage.jsonmanifests,spec/openapi.json,packages/sdk/src/version.gen.ts, and the MCP health test'ssdkServerVersionassertion.packages/mcp/src/version.gen.tswas not regenerated: thegen-versionscript updates both the SDK and MCP version files together, but only the SDK file appears in the diff —MCP_PACKAGE_VERSIONstill reads\"0.8.0\", so the MCP server will advertise the wrong version in the MCPinitializehandshake.'0.8.0', keeping it consistent with the stale generated file but masking the discrepancy from CI.Confidence Score: 3/5
The MCP server will ship advertising version 0.8.0 to clients even though the package declares 0.8.1, because the auto-generated version constant was not regenerated.
The
packages/mcp/src/version.gen.tsfile was not regenerated — it remains at MCP_PACKAGE_VERSION = "0.8.0" while everything else moved to 0.8.1. The MCP server embeds this constant directly in the MCP protocol handshake, so every client that connects will receive the wrong version. The test at line 94 was left unchanged, meaning this discrepancy passes CI silently.packages/mcp/src/version.gen.ts needs to be regenerated via
pnpm gen-version, and the test assertion in packages/mcp/test/server.test.ts line 94 needs to be updated to '0.8.1'.Important Files Changed
Flowchart
%%{init: {'theme': 'neutral'}}%% flowchart TD A["root package.json omnigraph.serverVersion = 0.8.1"] --> B["scripts/gen-version.ts"] C["packages/mcp/package.json version = 0.8.1"] --> B B --> D["packages/sdk/src/version.gen.ts SERVER_VERSION = 0.8.1 (updated)"] B --> E["packages/mcp/src/version.gen.ts MCP_PACKAGE_VERSION = 0.8.0 (NOT regenerated)"] D --> F["SDK reports correct version"] E --> G["McpServer init handshake reports 0.8.0 instead of 0.8.1"] E --> H["Test line 94 still asserts 0.8.0 masks the stale value"]%%{init: {'theme': 'base', 'themeVariables': {"darkMode": true, "background": "#0d1117", "primaryColor": "#21262d", "primaryTextColor": "#e6edf3", "primaryBorderColor": "#8b949e", "lineColor": "#8b949e", "textColor": "#e6edf3", "edgeLabelBackground": "#161b22", "actorBkg": "#21262d", "actorBorder": "#8b949e", "actorTextColor": "#e6edf3", "actorLineColor": "#8b949e", "signalColor": "#8b949e", "signalTextColor": "#e6edf3", "noteBkgColor": "#373320", "noteBorderColor": "#d4a72c", "noteTextColor": "#f0e6c0", "labelBoxBkgColor": "#21262d", "labelBoxBorderColor": "#8b949e", "labelTextColor": "#e6edf3", "loopTextColor": "#e6edf3", "activationBkgColor": "#30363d", "activationBorderColor": "#8b949e"}}}%% flowchart TD A["root package.json omnigraph.serverVersion = 0.8.1"] --> B["scripts/gen-version.ts"] C["packages/mcp/package.json version = 0.8.1"] --> B B --> D["packages/sdk/src/version.gen.ts SERVER_VERSION = 0.8.1 (updated)"] B --> E["packages/mcp/src/version.gen.ts MCP_PACKAGE_VERSION = 0.8.0 (NOT regenerated)"] D --> F["SDK reports correct version"] E --> G["McpServer init handshake reports 0.8.0 instead of 0.8.1"] E --> H["Test line 94 still asserts 0.8.0 masks the stale value"]Comments Outside Diff (1)
packages/mcp/test/server.test.ts, line 94 (link)The test checks the version reported by the MCP server in the
initializehandshake, which is sourced fromMCP_PACKAGE_VERSIONinpackages/mcp/src/version.gen.ts. Onceversion.gen.tsis regenerated to0.8.1, this assertion will fail. Update it now so it stays consistent with the package bump.Note: If this suggestion doesn't match your team's coding style, reply to this and let me know. I'll remember it for next time!
Reviews (1): Last reviewed commit: "release: sync SDK + MCP to omnigraph-ser..." | Re-trigger Greptile