-
Notifications
You must be signed in to change notification settings - Fork 7.6k
Description
Describe the bug
@modelcontextprotocol/server-memory has a configuration MEMORY_FILE_PATH
which by default takes value memory.json
but the content of the file is separate lines of with JSON content each. The entire file is not a JSON though.
Please change the file name to memory.jsonl
, that will make IDEs happy and will not show lint errors that lines are not separated by comma and the there is no topmost object or array element
To Reproduce
Steps to reproduce the behavior:
- Install memory mcp
- add 2 records
- open the memory.json file
- The content of the file 2 lines each containing valid json. The file itself is not of the json format
Expected behavior
Either file content is a valid JSON file with wrapper object or array having child items separated by comma, or file name has .jsonl
extension to show that the format is expected an IDEs can treat it as valid file
Logs
If applicable, add logs to help explain your problem.
Additional context
Add any other context about the problem here.