[PR Ready] Fix: Add tenant headers X-OpenViking-Account and X-OpenViking-User

[libinghui123]

PR #2 is ready for review

I have submitted PR #2 which fixes the 400 "ROOT requests to tenant-scoped APIs" error in the claude-code-memory-plugin.

Changes:

• Adds X-OpenViking-Account and X-OpenViking-User headers to all API requests
• Reads account/user from claude_code config section in ov.conf

Files changed:

• config.mjs, auto-recall.mjs, debug-recall.mjs, auto-capture.mjs, debug-capture.mjs, memory-server.ts, memory-server.js

Testing: Verified with debug logs - all requests return 200 with correct headers.

Please review: #2

Thank you!

[Castor6]

Thanks for the PR! This issue is related to volcengine/OpenViking#1306, and I have a plan to fix it properly in the main repo.

The planned approach:

Instead of reusing the server-side ov.conf for client connection info, I plan to introduce a separate client config file for the Claude Code plugin:

• Default path: ~/.openviking/claude-code-memory-plugin/config.json
• Two modes:
  • local mode: reads from local server config (~/.openviking/ov.conf)
  • remote mode: baseUrl, apiKey (for connecting to remote OpenViking server)

This approach provides better multi-tenant support and cleaner separation between server and client configurations. It follows the pattern used by openclaw's client config.

So I won't be merging this PR, but thanks for raising the issue and the investigation!

[Castor6]

@libinghui123 Hi, I have updated this in the new version. For migration instructions, please check the #4