Skip to content

server-memory 0.6.3 is not available on npm #674

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Closed
BRO3886 opened this issue Feb 25, 2025 · 9 comments
Closed

server-memory 0.6.3 is not available on npm #674

BRO3886 opened this issue Feb 25, 2025 · 9 comments
Assignees
Labels
bug Something isn't working server-memory Reference implementation for the Memory MCP server - src/memory

Comments

@BRO3886
Copy link

BRO3886 commented Feb 25, 2025

Describe the bug
Unable to set the custom file path via npx since the npm registry does not have the latest version of server-memory

To Reproduce

Expected behavior
0.6.3 should be available on NPM. Thanks a lot!

@BRO3886 BRO3886 added the bug Something isn't working label Feb 25, 2025
@edobez
Copy link

edobez commented Feb 25, 2025

Related to #220

@IzumiSy
Copy link

IzumiSy commented Feb 26, 2025

Why hasn't that been published yet? I really need that.

@wcygan
Copy link

wcygan commented Apr 21, 2025

@ashwin-ant @dsp-ant @jspahrsummers can this please have some attention? The fix is there, yet users are needlessly rolling-their-own because of the lack of support.


Context:

The commit 04209ec fixes the issue, but it has not yet been published to NPM.

Package: https://www.npmjs.com/package/@modelcontextprotocol/server-memory
Collaborator 1: https://www.npmjs.com/~jspahrsummers
Collaborator 2: https://www.npmjs.com/~thedsp
Collaborator 3: https://www.npmjs.com/~ashwin-ant

@olaservo
Copy link
Member

I opened a PR to trigger a publish to NPM here: #1518

I believe there was some timing issue between when the auto-publish process was implemented and when the last code changes were made to the repo. This should force a new version.

@wcygan
Copy link

wcygan commented Apr 22, 2025

Thank you @olaservo!

I notice that the GitHub Action for typescript publish is skipped, which seems like the culprit?

Ref: https://github.com/modelcontextprotocol/servers/actions/runs/14576064001/job/40882362621

Image

Perhaps there is something wrong with the typescript publish action

Maybe it needs to have a release cut for it to trigger the publish?

  publish:
    runs-on: ubuntu-latest
    needs: [build, detect-packages]
    if: github.event_name == 'release'
    environment: release

    strategy:
      matrix:
        package: ${{ fromJson(needs.detect-packages.outputs.packages) }}
    name: Publish ${{ matrix.package }}

    permissions:
      contents: read
      id-token: write

    steps:
      - uses: actions/checkout@v4
      - uses: actions/setup-node@v4
        with:
          node-version: 22
          cache: npm
          registry-url: "https://registry.npmjs.org"

      - name: Install dependencies
        working-directory: src/${{ matrix.package }}
        run: npm ci

      - name: Publish package
        working-directory: src/${{ matrix.package }}
        run: npm publish --access public
        env:
          NODE_AUTH_TOKEN: ${{ secrets.NPM_TOKEN }}

@olaservo
Copy link
Member

@wcygan I think that's expected, since the newer automated release process is a scheduled process that checks for code changes, then auto-creates releases which then need to be approved by someone on the core team: https://github.com/modelcontextprotocol/servers/actions/workflows/release.yml

This worked recently for some other npm packages that needed to get updated, so as long as we see this one pop up in the auto-generated release tonight/tomorrow, we should be good to have the team approve the subsequent release steps.

@wcygan
Copy link

wcygan commented Apr 22, 2025

Understood. Thank you!

The addition of this version will likely be a boon for adoption of this MCP; I imagine people want to version-control their knowledge graph & share it with others.

@olaservo
Copy link
Member

Hi @wcygan looks like a new package version is now on npm: https://www.npmjs.com/package/@modelcontextprotocol/server-memory?activeTab=versions

Side note, the automated publish process switched to calendar versioning.

@olaservo olaservo added the server-memory Reference implementation for the Memory MCP server - src/memory label Apr 25, 2025
@wcygan
Copy link

wcygan commented Apr 25, 2025

Thanks @olaservo! I'm able to get it working now, looks good:

{
  "mcpServers": {
    "memory": {
      "command": "npx",
      "args": [
        "-y",
        "@modelcontextprotocol/server-memory"
      ],
      "env": {
        "MEMORY_FILE_PATH": "/Users/wcygan/path/to/memory.json"
      }
    }
  }
}

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working server-memory Reference implementation for the Memory MCP server - src/memory
Projects
None yet
Development

No branches or pull requests

5 participants