Skip to content

docs(adr): agents over ACP — spike, finding 13 and ADR 0013 - #68

Open
FreshlyBrewedCode wants to merge 2 commits into
mainfrom
t3code/investigate-canvas-acp-adapter
Open

FreshlyBrewedCode wants to merge 2 commits into
mainfrom
t3code/investigate-canvas-acp-adapter

Conversation

@FreshlyBrewedCode

@FreshlyBrewedCode FreshlyBrewedCode commented Oct 2, 2026 •

Copy link
Copy Markdown
Owner

Base layer of the ACP stack (epic #62).

  • Spike:
    • src/runtime/acp-adapter.ts: a spike-grade ACP adapter behind the existing AgentAdapter.
    • spikes/13-acp-agents.ts: drives the real runtime (startRun) on opencode and Claude Code.
    • Bumps @tanstack/ai to 0.64 and adds @tanstack/ai-acp, @agentclientprotocol/sdk and
      @agentclientprotocol/claude-agent-acp. The tanstack bump was a no-op for the existing
      suite.
  • Finding 13 (docs/findings/13-acp-agents.md) — both agents:
    • completed a two-step workflow with a chosen model, and structured output parsed on the
      first try;
    • had permission answered client-side;
    • cancelled in 2–3 ms with no process left.
    • Also: model ids are per agent, and usage means different things per agent. The addendum
      probes how to exclude host settings.
  • ADR 0013 (proposed): one ACP adapter; agent and model as separate fields, and factory
    always sends a model; host settings ignored by default; permission answered by the client;
    context and cost fields; one process per step over stdio.

Layers above: #63, #64, #65, #66, #67.

🤖 Generated with Claude Code

FreshlyBrewedCode and others added 2 commits October 2, 2026 17:41
A spike-grade ACP adapter behind the existing AgentAdapter seam, a runner
script that drives the real runtime with it, and the finding. Bumps
@tanstack/ai to 0.64 and adds @tanstack/ai-acp, @agentclientprotocol/sdk and
@agentclientprotocol/claude-agent-acp.

Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
One ACP adapter for opencode and Claude Code. Agent and model become
separate fields and factory always sends the model. Runs ignore host agent
settings by default (configurable per agent). Headless permission is answered
by the client, and context and cost get their own fields on
AgentStepFinished. Adds the settings-isolation probe to finding 13.

Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
@FreshlyBrewedCode FreshlyBrewedCode changed the title t3code/investigate canvas acp adapter docs(adr): agents over ACP — spike, finding 13 and ADR 0013 Oct 2, 2026
@FreshlyBrewedCode
FreshlyBrewedCode added this pull request to stack #70 October 2, 2026 19:28

This branch has not been deployed

No deployments
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