Send the real plugin version during login - #96
Merged
ishaanxgupta merged 1 commit intoSep 27, 2026
Merged
Conversation
The auth URL hardcoded cli_version=2.0.10, so every release reported itself as 2.0.10. Use PLUGIN_VERSION, which is synced from package.json.
ishaanxgupta
added a commit
that referenced
this pull request
Sep 27, 2026
Fixes #97. - Describe what the plugin does today: profile-only first-message context, direct recall by default, the real `[SUPERMEMORY]` and `<supermemory-context>` blocks, `search` scope and `limit` semantics, container tag precedence, `filterPrompt`, capture defaults, and `SUPERMEMORY_DEBUG`. - Document the installer steps once, including the config-directory and in-place config handling from #92 and #94. - The generated /supermemory-login command no longer claims a fixed port, and the tool's help lists `limit` for search. - Add a CI workflow running typecheck, tests, and build on pull requests and pushes to main. - Bump to 2.0.16 so the CLI fixes in #92, #94, and #96 publish together.
ishaanxgupta
approved these changes
Sep 27, 2026
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.
Fixes #95
Problem
loginsendscli_versionto Supermemory, but it was hardcoded as"2.0.10"insrc/services/auth.ts, so every release since then reports itself as 2.0.10. The correct version was already available: since #65,scripts/sync-version.mjsgeneratesPLUGIN_VERSIONfrompackage.jsonon every build, and the update check andsm_plugin_versionalready use it. This was the one spot left over, added in #60 just before #65 landed.Fix
Use the synced constant instead of the literal:
cli_versionnow always matchespackage.json, including future releases, with no manual edit needed. It's imported from../version.jsdirectly (not viaconfig.ts) to keepauth.tsfree of config side effects.Before / after
package.jsonis2.0.15.loginwas pointed at a dead local auth URL with a 3 s timeout, so nothing reached Supermemory:Testing
loginon Windows 11 on both branches (output above).dist/cli.jsonmaincontainsPLUGIN_VERSION = "2.0.15"alongside the literalcli_version: "2.0.10". This branch usescli_version: PLUGIN_VERSION.bun run typecheckis clean andbun testpasses (33 tests).auth.tsthan Keep login waiting when the browser cannot be opened #90, so the two merge independently.