Skip to content

style: cargo fmt terraphim_update (CI fmt --all) - #8

Open
AlexMikhalev wants to merge 7 commits into
mainfrom
task/fmt-terraphim-update
Open

style: cargo fmt terraphim_update (CI fmt --all)#8
AlexMikhalev wants to merge 7 commits into
mainfrom
task/fmt-terraphim-update

Conversation

@AlexMikhalev

Copy link
Copy Markdown
Contributor

Unblock CI fmt --all. One-line wrap.

Live Claude Code PostToolUse sends tool_response + exitCode, not the
fixture tool_result + exit_code shape — learn hook fail-opened with
zero captures (#2704 / multi-client plan C2).

- serde alias tool_response → tool_result, exitCode → exit_code
- Auto format: normalize bash→Bash; legacy {tool,result} from #2704
- Unit tests for live Claude, lowercase bash, legacy sample

Host Phase 0 (separate): PATH agent 1.21.1, Claude post→learn hook,
OpenCode terraphim-learn plugin.
…ases (C2 multi-client hooks)' (#90) from task/learn-hook-claude-envelope-aliases into main
Phase 2 multi-client plan (disciplined design + TDD):
- AgentType::Pi, ~/.pi/agent config, install-hook docs
- packages/pi-terraphim-learn: onToolResult → learn hook (fail-open)
- design gate docs/plans/design-pi-terraphim-learn-2026-08-08.md
- install unit tests RED→GREEN (7 ok)

C2 Claude envelopes already on main (PR #90).
…Pi + package)' (#91) from task/pi-terraphim-learn-hooks into main
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