Skip to content

[pull] main from modelcontextprotocol:main#230

Merged
pull[bot] merged 8 commits intothreatcode:mainfrom
modelcontextprotocol:main
Oct 21, 2025
Merged

[pull] main from modelcontextprotocol:main#230
pull[bot] merged 8 commits intothreatcode:mainfrom
modelcontextprotocol:main

Conversation

@pull
Copy link

@pull pull bot commented Oct 21, 2025

See Commits and Changes for more details.


Created by pull[bot] (v2.0.0-alpha.4)

Can you help keep this open source service alive? 💖 Please sponsor : )

claude bot and others added 8 commits August 17, 2025 16:28
….jsonl

The Memory MCP server uses JSONL format (JSON Lines) where each line
contains a separate JSON object, but was using a .json file extension.
This caused IDE lint errors and confusion since the file is not valid JSON.

Changes:
- Update default filename in index.ts from memory.json to memory.jsonl
- Update documentation references in README.md
- Maintain backward compatibility for existing MEMORY_FILE_PATH configs

Fixes #2361

Co-authored-by: Ola Hungerford <olaservo@users.noreply.github.com>
When no custom MEMORY_FILE_PATH is set and memory.json exists but
memory.jsonl doesn't exist, automatically migrate the old file to
the new format with proper file extension.

This ensures existing users don't lose their data when upgrading
to the version with the corrected .jsonl extension.

Co-authored-by: Ola Hungerford <olaservo@users.noreply.github.com>
- Added DETECTED message when legacy memory.json file is found
- Added COMPLETED message when migration finishes successfully
- Improves visibility of backward compatibility migration process

Co-authored-by: Ola Hungerford <olaservo@users.noreply.github.com>
…20250817-1626

fix: Change memory server default filename from memory.json to memory.jsonl
@pull pull bot locked and limited conversation to collaborators Oct 21, 2025
@pull pull bot added the ⤵️ pull label Oct 21, 2025
@pull pull bot merged commit dc21983 into threatcode:main Oct 21, 2025
2 checks passed
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants