Skip to content

fix(devin): connect T3 tools for new and resumed sessions - #1

Closed
ProtonDev-sys wants to merge 1 commit into
mzakyali:feat/devin-upstream-portfrom
ProtonDev-sys:fix/devin-live-integration
Closed

fix(devin): connect T3 tools for new and resumed sessions#1
ProtonDev-sys wants to merge 1 commit into
mzakyali:feat/devin-upstream-portfrom
ProtonDev-sys:fix/devin-live-integration

Conversation

@ProtonDev-sys

Copy link
Copy Markdown

Devin can chat through ACP but ignores session/new.mcpServers, so T3's preview tools never reach the agent. The runtime also constructed additionalDirectories that the generated RPC encoder silently dropped.

Connect T3's tools through Devin's MCP extension using a private configuration directory owned by the provider session. Preserve additional workspace roots when creating, loading, and resuming sessions, and remove the temporary credentials when the session closes. This reuses the existing ACP request and HTTP paths.

This is a focused follow-up to the existing Devin integration, targeting its feature branch rather than duplicating the provider implementation.

Validation:

  • 95 focused tests passed across the Devin adapter/provider/configuration and ACP client/agent/protocol suites.
  • Server and effect-acp type checks, lint on changed files, and the production web/server build passed.
  • Devin CLI 3000.10.21 on Windows: real preview tool calls passed with Adaptive and SWE-1.7; the extended smoke test resumed a session with a fresh credential, called the tool again, and read/wrote project files.
  • Built-app browser check: selected Devin's SWE-1.7 Lightning model, sent a request through the composer, and verified the resulting file and assistant reply.

Model and harness: GPT-6 in Codex.

@github-actions github-actions Bot added vouch:unvouched PR author is not yet trusted in the VOUCHED list. size:L labels Sep 10, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

size:L vouch:unvouched PR author is not yet trusted in the VOUCHED list.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant