feat(agents): add devin adapter

[yyovil]

Adds the devin harness, stacked on #119 (agent platform). Adapter package + Constructors() registration + resolver test.

🤖 Generated with Claude Code

Stack

1. feat(agents): agent platform — registry, activity hooks, harness allowlist #119
2. feat(agents): add grok adapter #120
3. feat(agents): add droid adapter #121
4. feat(agents): add amp adapter #122
5. feat(agents): add agy adapter #123
6. feat(agents): add crush adapter #124
7. feat(agents): add aider adapter #125
8. feat(agents): add cursor adapter #126
9. feat(agents): add qwen adapter #127
10. feat(agents): add copilot adapter #128
11. feat(agents): add goose adapter #129
12. feat(agents): add auggie adapter #130
13. feat(agents): add continue adapter #131
14. feat(agents): add devin adapter #132 👈 current
15. feat(agents): add cline adapter #133
16. feat(agents): add kimi adapter #134
17. feat(agents): add kiro adapter #135
18. feat(agents): add kilocode adapter #136
19. feat(agents): add vibe adapter #137
20. feat(agents): add pi adapter #138
21. feat(agents): add autohand adapter #139

[greptile-apps]

Greptile Summary

This PR introduces the Devin for Terminal (Cognition) agent adapter, wired into the registry alongside the existing grok, goose, and other adapters. The implementation delegates hook installation to claudecode.Plugin via Devin's documented Claude Code compatibility layer, maps AO's four permission modes onto Devin's two (auto / dangerous), and restores sessions with -r <agentSessionId>.

• New adapter (devin/devin.go): binary resolution, launch/restore command construction, permission flag mapping, and SessionInfo reading hook-written metadata — all modelled closely on the grok adapter.
• Registry + wiring test: devin.New() added to Constructors() and HarnessDevin added to the resolver smoke test, completing the integration.

Confidence Score: 4/5

Safe to merge after fixing the metadata key inconsistency in SessionInfo — title and summary will silently vanish if shared port constants are ever renamed.

The adapter correctly delegates hook installation, binary resolution, and permission mapping. The one real defect is in SessionInfo: it reads Title and Summary via private local string constants that duplicate the shared ports.MetadataKeyTitle and ports.MetadataKeySummary constants that the installed hooks actually write. Because the string values currently match, session info works today, but the devin adapter is silently decoupled from the shared vocabulary — a rename of those constants would break devin's title/summary without a compile error.

backend/internal/adapters/agent/devin/devin.go — the SessionInfo function and the devinTitleMetadataKey/devinSummaryMetadataKey const block; also update the corresponding map keys in devin_test.go.

Important Files Changed

Filename	Overview
backend/internal/adapters/agent/devin/devin.go	New Devin adapter mirroring the grok structure; SessionInfo uses private local constants instead of the shared ports.MetadataKeyTitle/ports.MetadataKeySummary that the installed hooks actually write, creating a silent divergence risk.
backend/internal/adapters/agent/devin/devin_test.go	Comprehensive unit tests covering all adapter methods; TestSessionInfoReadsHookMetadata should use ports.MetadataKeyTitle/ports.MetadataKeySummary as map keys once the production code is updated to the shared constants.
backend/internal/adapters/agent/registry/registry.go	Straightforward devin.New() insertion into the Constructors slice following the same pattern as all other adapters; no issues.
backend/internal/daemon/wiring_test.go	Adds {domain.HarnessDevin, devin} to the resolver smoke test; correct and follows the established pattern.

Sequence Diagram

sequenceDiagram
    participant AO as AO Session Manager
    participant DP as devin.Plugin
    participant CC as claudecode.Plugin
    participant DV as devin binary

    AO->>DP: GetAgentHooks(ctx, cfg)
    DP->>CC: GetAgentHooks(ctx, cfg)
    CC-->>DP: writes .claude/settings.local.json
    DP-->>AO: ok

    AO->>DP: GetLaunchCommand(ctx, cfg)
    DP->>DP: devinBinary() [resolve + cache]
    DP-->>AO: [devin, --permission-mode, auto, -p, prompt]

    AO->>DV: exec launch command
    DV-->>AO: hook payloads (SessionStart → agentSessionId, title, summary)

    AO->>DP: GetRestoreCommand(ctx, cfg)
    DP-->>AO: [devin, --permission-mode, dangerous, -r, sessionID]

    AO->>DP: SessionInfo(ctx, session)
    DP-->>AO: "SessionInfo{AgentSessionID, Title, Summary}"

Loading

_{Reviews (2): Last reviewed commit: "feat(agents): add devin adapter" | Re-trigger Greptile}