Skip to content

feat: expose arka primitives over MCP#13

Open
Genmin wants to merge 1 commit into
kcolbchain:mainfrom
Genmin:feat/mcp-server
Open

feat: expose arka primitives over MCP#13
Genmin wants to merge 1 commit into
kcolbchain:mainfrom
Genmin:feat/mcp-server

Conversation

@Genmin
Copy link
Copy Markdown

@Genmin Genmin commented May 1, 2026

Summary

  • add arka::mcp with an object-safe ArkaTool trait and a minimal stdio JSON-RPC server for initialize, tools/list, and tools/call
  • add MCP tools for native balance reads, swap quote/build metadata, and AgentAccount deposit/balance operations
  • add the arka-mcp-server binary and README wiring for Claude Desktop
  • cover the server with no-network unit tests plus a stdio integration test that spawns the binary and drives MCP requests

Addresses #11.

Validation

  • cargo fmt --check
  • cargo build --bin arka-mcp-server
  • cargo test --lib
  • cargo test --test mcp_stdio
  • cargo test --all-targets
  • git diff --check

Signed-off-by: Genmin <joey@joeyroth.com>
@abhicris
Copy link
Copy Markdown
Contributor

Welcome to kcolbchain, @Genmin — glad you're here. 🌱

Here's what happens from this PR:

  1. Our automated review looks for obvious issues (tests, secrets, size) within a couple of hours.
  2. If it's clean and CI passes, we merge without back-and-forth.
  3. If we need changes, we'll leave a specific comment — not a generic nit. Push another commit and we re-review.

While you wait:

  • Run the repo's tests locally (see the repo README.md).
  • Keep the PR scoped to one concern — bigger PRs land slower.
  • Don't commit tokens or .env contents.

What happens after your first merge

Thanks for writing the code. We're building this to last.

@abhicris
Copy link
Copy Markdown
Contributor

🤖 Audit verdict: safe

Clean MCP server implementation with proper input validation, no credential leakage, no malicious patterns, and comprehensive test coverage; agent deposit operations use InMemoryAgentAccount (testing-only).

Audited by the kcolbchain PR pipeline. See pipeline docs.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants