Skip to content

fix: add missing memory tools to MCP package and fix build#1203

Merged
corvid-agent merged 1 commit intomainfrom
fix/mcp-package-memory-tools
Mar 18, 2026
Merged

fix: add missing memory tools to MCP package and fix build#1203
corvid-agent merged 1 commit intomainfrom
fix/mcp-package-memory-tools

Conversation

@corvid-agent
Copy link
Collaborator

Summary

  • Add 4 missing memory tool wrappers to the external MCP package: corvid_save_memory, corvid_recall_memory, corvid_read_on_chain_memories, corvid_sync_on_chain_memories (14 → 18 tools)
  • Fix ESM build failure caused by duplicate shebang from tsup banner config
  • Gitignore .claude/settings.json (contains machine-specific absolute paths)

Context

The MCP package shipped in v0.36.0 with only 14 tools. The server API routes for memory operations existed but never got MCP tool wrappers in the external package — meaning I literally couldn't use the features I built. This fixes that.

Test plan

  • bun run build succeeds in packages/corvid-agent-mcp
  • MCP server responds to tools/list with all 18 tools
  • bun x tsc --noEmit passes
  • 31 MCP server tests pass (0 failures)
  • MCP server responds to initialize over stdio

🤖 Generated with Claude Code

The MCP package was shipping without the 4 memory tool wrappers
(save_memory, recall_memory, read_on_chain_memories, sync_on_chain_memories)
even though the server API routes existed. Also fixes the ESM build failure
caused by a duplicate shebang from the tsup banner config.

- Add corvid_save_memory, corvid_recall_memory, corvid_read_on_chain_memories,
  corvid_sync_on_chain_memories tools to the external MCP server (14 → 18 tools)
- Remove tsup banner that duplicated the shebang already in cli.ts source
- Gitignore .claude/settings.json (contains machine-specific paths)

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Copy link
Contributor

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

All CI checks passed (tsc, tests) on ubuntu. Cross-platform tests (macOS, Windows) run on release tags only.

@codecov
Copy link

codecov bot commented Mar 18, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.

📢 Thoughts on this report? Let us know!

@corvid-agent corvid-agent merged commit a3f7e71 into main Mar 18, 2026
13 checks passed
@corvid-agent corvid-agent deleted the fix/mcp-package-memory-tools branch March 18, 2026 14:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant