feat(agents): add autohand adapter

[yyovil]

Adds the autohand harness, stacked on #119 (agent platform). Adapter package + Constructors() registration + resolver test. Includes its own activity deriver.

🤖 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
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 👈 current

[greptile-apps]

Greptile Summary

Adds the autohand agent adapter, registering it in the activity-dispatch map and the adapter registry. The adapter follows the same structural pattern as existing adapters (claudecode, cline, etc.) and includes launch/restore command building, session info extraction, and hook lifecycle management.

• New package autohand provides Plugin implementing both adapters.Adapter and ports.Agent, with binary resolution, permission-mode flag mapping, and atomic config rewrites for hook install/uninstall.
• Custom UnmarshalJSON/MarshalJSON on autohandHookEntry preserves unknown user-set fields (e.g. async, filter) via an Extra map[string]json.RawMessage, addressing the field-dropping concern raised in the previous review.
• autohandBinary now gates on ctx.Err() before acquiring the mutex, ensuring a cancelled context is honoured even when the binary path is already cached — closing the second gap from the prior review.

Confidence Score: 5/5

Safe to merge; both issues surfaced in the prior review have been resolved and no new correctness problems were found.

Both previously flagged issues are fixed: unknown hook-entry fields now survive config rewrites through the custom UnmarshalJSON/MarshalJSON round-trip, and context cancellation is honoured in the cached-binary fast path of autohandBinary. The rest of the adapter — binary resolution, permission-mode mapping, atomic config writes, and the activity-state deriver — is implemented consistently with the existing adapter patterns and is well covered by the new test suite.

No files require special attention.

Important Files Changed

Filename	Overview
backend/internal/adapters/agent/autohand/hooks.go	Hook install/uninstall/query logic with atomic config writes; custom MarshalJSON/UnmarshalJSON correctly round-trips unknown user fields via Extra map.
backend/internal/adapters/agent/autohand/autohand.go	Main adapter implementing launch/restore command building, binary resolution with mutex-protected caching, and session info surfacing; context cancellation is correctly checked on all code paths including the cached-binary fast path.
backend/internal/adapters/agent/autohand/autohand_test.go	Comprehensive test suite covering argv building, permission-mode mapping, hook install/uninstall idempotency, unknown-field round-trip, and context cancellation across all methods including GetLaunchCommand with a cached binary.
backend/internal/adapters/agent/autohand/activity.go	Simple event-to-ActivityState mapper; covers all four installed hook subcommands and returns (empty, false) for unknowns.
backend/internal/adapters/agent/activitydispatch/dispatch.go	One-line addition registering autohand.DeriveActivityState in the Derivers map; no issues.
backend/internal/adapters/agent/registry/registry.go	Adds autohand.New() to Constructors(); straightforward registration matching the existing pattern.
backend/internal/daemon/wiring_test.go	Extends the resolver integration test with the HarnessAutohand → autohand mapping; no issues.

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