Add Veil Cash Base MCP plugin

[apexethdev]

Summary

• Adds skills/base-mcp/plugins/veil.md for Veil Cash — a privacy pool for ETH and USDC on Base mainnet.
• Registers the plugin in skills/base-mcp/SKILL.md and adds discovery tags privacy and shielded-payments to references/plugin-spec.md.
• MCP package: @veil-cash/mcp@0.2.1 (veildotcash/veil-mcp).

Integration model

• integration: external-mcp, stdio-only v1 — install local @veil-cash/mcp beside Base MCP.
• requires.externalMcp.url: null by design — private VEIL_KEY, ZK proof building, and balance decryption must stay on the user's machine. A hosted HTTP MCP would be custodial and would centralize activity metadata.
• Public flows (register/deposit): Veil MCP prepare tools → Base MCP send_calls.
• Private flows (withdraw, transfer, x402, consolidate): Veil relay inside Veil MCP → Base MCP submission none.
• Chat-only connector surfaces (Claude.ai web, ChatGPT): documented as unsupported on v1 — no hosted URL.

Please confirm whether externalMcp.url: null is acceptable for stdio npm MCPs, or if Base prefers a different frontmatter representation.

Test plan

• Plugin matches plugin-spec.md authoring checklist
• ## Submission documents both send_calls (public) and none (private relay)
• ## Surface Routing states chat-only stop behavior
• Example prompts cover read, deposit, private withdraw, x402, and chat-only fallback

Made with Cursor

[cb-heimdall]

🟡 Heimdall Review Status

Requirement	Status	More Info
Reviews	🟡 0/1	Denominator calculation Show calculation 1 if user is bot 0 1 if user is external 0 2 if repo is sensitive 0 From .codeflow.yml 1 Additional review requirements Show calculation Max 0 0 From CODEOWNERS 0 Global minimum 0 Max 1 1 1 if commit is unverified 1 Sum 2

[stephancill]

Thanks for the submission. The plugin is spec-conformant, the MCP exposes the documented tools, and the deposit calldata maps correctly into send_calls. A few changes before merge:

Required

• Don't add the plugin to the registry/inventory. Please revert the SKILL.md plugins-table row and, in references/plugin-spec.md, the "Existing Plugin Conformance" table row and the "Veil Cash ships at 0.2.1" line. The registry and that conformance inventory are maintainer-owned and will be updated when the program is ready. Keep your net-new tag additions (privacy, shielded-payments) in the vocabulary list, and limit the rest of the PR to plugins/veil.md.

Minor

• version is set to 0.2.1 to mirror the package. The spec defines version as the plugin-doc version (independent of upstream), but mirroring the package is reasonable — noting it only in case that wasn't intentional.
• Register-before-deposit is a hard prerequisite: a deposit submitted before the deposit key is registered reverts. Make this explicit in ## Orchestration (or batch the register + deposit calls), rather than leaving it implied.

I'm proposing a change to the plugin spec to support defining local MCP servers in the frontmatter in #78

[apexethdev]

Thanks for the detailed review @stephancill — pushed an update addressing the feedback:

Required (done)

• Reverted the maintainer-owned registry/inventory edits: SKILL.md is fully back to master, and in references/plugin-spec.md the "Existing Plugin Conformance" table row and the "Veil Cash ships at 0.2.1" line are reverted (the count is back to "7 native plugins"). The PR is now limited to plugins/veil.md plus the net-new privacy / shielded-payments tag-vocabulary additions, as requested.
• Made register-before-deposit an explicit hard prerequisite in ## Orchestration > Deposit (a deposit submitted before the deposit key is registered reverts), rather than leaving it implied.

Minor

• Kept version: 0.2.1 to mirror the package — intentional, per your note.

Re: #78 — I've rebased plugins/veil.md's frontmatter onto the local-MCP schema you proposed in #78: requires.externalMcp now uses transport: stdio with command / args (pinned) / env (names only) instead of the url: null placeholder, and the local-exec risk tag + a ### local-exec Risks subsection are added. That means this PR now depends on #78 landing first — it references the stdio schema and local-exec tag that #78 introduces. Happy to adjust if you'd prefer a different sequencing.

[youssefea]

Thanks again for putting this together, @apexethdev. One operational note: this PR includes one or more commits that GitHub marks as unverified. We require verified commits for external plugin submissions so we can rely on GitHub signature and provenance checks before merging code into base/skills.

Could you please open a new PR with verified commits? I cannot merge this PR as-is. Thank you.

[apexethdev]

Opened #105 as the replacement with verified (signed) commits, per @youssefea's request — GitHub now marks the commit as Verified. Same content as here, freshly branched off current master (so it also picks up the merged #78 stdio schema). Closing this one in favor of #105.