Skip to content

Add ACP model helper#457

Draft
developersdigest wants to merge 1 commit into
vercel:mainfrom
developersdigest:add-acp-model-helper
Draft

Add ACP model helper#457
developersdigest wants to merge 1 commit into
vercel:mainfrom
developersdigest:add-acp-model-helper

Conversation

@developersdigest

Copy link
Copy Markdown

Summary

Adds an eve/acp helper for using Agent Client Protocol agents as AI SDK language models in defineAgent({ model }).

import { acp } from "eve/acp";
import { defineAgent } from "eve";

export default defineAgent({
  model: acp("opencode"),
});

The helper keeps the default path small while allowing model selection and arbitrary ACP commands:

model: acp("opencode", "opencode-go/kimi-k2.7-code")
model: acp({ command: "kimi", args: ["acp"], model: "moonshot-v1-auto" })

Notes

  • Adds presets for OpenCode, Gemini, Codex via codex-acp, and Claude via claude-code-acp.
  • Leaves @mcpc-tech/acp-ai-provider as an optional peer dependency so the base eve install stays lightweight.
  • Defaults persistSession to false; local smoke testing showed true keeps the ACP child process alive after one-shot generateText() calls.

Testing

  • PATH=/Users/j/.nvm/versions/node/v24.13.0/bin:$PATH pnpm --filter eve exec vitest run --config vitest.unit.config.ts src/public/acp/index.test.ts
  • PATH=/Users/j/.nvm/versions/node/v24.13.0/bin:$PATH pnpm --filter eve run typecheck
  • PATH=/Users/j/.nvm/versions/node/v24.13.0/bin:$PATH pnpm docs:check

External smoke tests in a throwaway app:

  • acp("opencode") through generateText() returned eve acp smoke ok.
  • acp("opencode", { model: "opencode-go/kimi-k2.7-code" }) through generateText() returned eve acp smoke ok.
  • Codex via brokk-codex-acp accepted initialize + session/new, but emitted Codex app-server logs, so it is not used as the clean default.
  • Installed Claude Code did not expose a native ACP subcommand; this PR uses the Zed adapter command name for the preset.

Signed-off-by: Developers Digest <124798203+developersdigest@users.noreply.github.com>
@vercel

vercel Bot commented Jul 1, 2026

Copy link
Copy Markdown
Contributor

@developersdigest is attempting to deploy a commit to the Vercel Team on Vercel.

A member of the Team first needs to authorize it.

@socket-security

Copy link
Copy Markdown

Review the following changes in direct dependencies. Learn more about Socket for GitHub.

Diff Package Supply Chain
Security
Vulnerability Quality Maintenance License
Addednpm/​@​mcpc-tech/​acp-ai-provider@​0.3.37910010091100

View full report

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