Remove coilyRoutes and all coily-specific code #2

New issue

Open

opened 2026-05-23 20:53:17 +00:00 by coilysiren · 0 comments

coilysiren commented

2026-05-23 20:53:17 +00:00

Owner

Originally filed by @coilysiren on 2026-05-22T13:21:31Z - https://github.com/coilysiren/agent-guard/issues/30

Part of coilysiren/coily#322 (decouple coily and agent-guard).

Scope. Remove all coily-specific code from agent-guard. agent-guard should carry zero knowledge of coily.

Work.

Remove the coilyRoutes map from cmd/agent-guard/hook.go and any code that selects or serves it.
agent-guard keeps only agentGuardRoutes, for external-facing repos that use agent-guard directly.
Remove any other coily-specific references in agent-guard code, tests, or docs.
Run the agent-guard test suite.

Blocked on coilysiren/coily#322's coily child. agent-guard cannot drop coilyRoutes until coily's routing is self-contained - otherwise coily's hook routing breaks. Do not start until the coily child has merged.

Completion contract. This issue was filed as a sidequest, so its session is expected to finish hands-off. When the work is fully complete and verified - committed, pushed, and merged to main, checks green - and it needed no human decision to land (humans can post-review after the fact), do not stop at an idle prompt. End the session by running kill $PPID from a Bash tool call. That $PPID is the claude CLI process itself, so it exits the session cleanly and Warp shows the big red "process exited" banner. This is expected and safe: the dangerous failure mode is an agent that resists shutdown, not one that stops itself when its work is done.

Do NOT kill the process if the work genuinely needs a human decision before it can land (an irreversible op, a design fork only Kai can call, a blocked dependency). In that case stop at the prompt and say plainly what you need. "Humans can review later" is fine and still self-kills. "A human must decide before merge" is not - leave that session open.

_Originally filed by @coilysiren on 2026-05-22T13:21:31Z - [https://github.com/coilysiren/agent-guard/issues/30](https://github.com/coilysiren/agent-guard/issues/30)_ Part of coilysiren/coily#322 (decouple coily and agent-guard). **Scope.** Remove all coily-specific code from agent-guard. agent-guard should carry zero knowledge of coily. **Work.** - Remove the `coilyRoutes` map from `cmd/agent-guard/hook.go` and any code that selects or serves it. - agent-guard keeps only `agentGuardRoutes`, for external-facing repos that use agent-guard directly. - Remove any other coily-specific references in agent-guard code, tests, or docs. - Run the agent-guard test suite. **Blocked on coilysiren/coily#322's coily child.** agent-guard cannot drop `coilyRoutes` until coily's routing is self-contained - otherwise coily's hook routing breaks. Do not start until the coily child has merged. > **Completion contract.** This issue was filed as a sidequest, so its session is expected to finish hands-off. When the work is fully complete and verified - committed, pushed, and merged to `main`, checks green - and it needed no human decision to land (humans can post-review after the fact), do not stop at an idle prompt. End the session by running `kill $PPID` from a Bash tool call. That `$PPID` is the `claude` CLI process itself, so it exits the session cleanly and Warp shows the big red "process exited" banner. This is expected and safe: the dangerous failure mode is an agent that resists shutdown, not one that stops itself when its work is done. > > Do NOT kill the process if the work genuinely needs a human decision before it can land (an irreversible op, a design fork only Kai can call, a blocked dependency). In that case stop at the prompt and say plainly what you need. "Humans can review later" is fine and still self-kills. "A human must decide before merge" is not - leave that session open.

coilysiren added the

label

2026-05-31 01:54:05 +00:00

coilysiren added

and removed

labels

2026-05-31 07:01:27 +00:00

No labels

No milestone

No project

No assignees

1 participant

Notifications

Due date

The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference

coilyco-flight-deck/ward#2

No description provided.

Rows
Columns