-
Notifications
You must be signed in to change notification settings - Fork 0
Open
Labels
Description
Bug
PR #70 updated all documentation to replace Crush with Goose as the primary local execution driver, but four stale Crush references remain in the Go source and are now contradicted by the updated docs.
Affected Lines (cmd/shellforge/main.go)
| Line | Content | Type |
|---|---|---|
| 94 | Run a governed agent (claude, copilot, codex, gemini, crush) |
CLI help text (shown to users) |
| 519 | fmt.Fprintln(os.Stderr, "Available drivers: claude, copilot, codex, gemini, crush") |
Runtime error message |
| 783 | // Used by Crush fork to check actions before execution. |
Code comment |
| 809 | // Map Crush tool names to ShellForge tool names |
Code comment |
Impact
- Users running
shellforge run --helpseecrushas a valid driver - Running
shellforge runwith an invalid driver showscrushas available option, misleading users who followed the updated docs - Code comments imply
cmdEvaluateis Crush-specific, obscuring that it is the generic evaluate endpoint now used by Goose viagovern-shell.sh
Fix
- Lines 94 and 519: replace
crushwithgoosein the driver lists - Line 783: update comment to reflect
govern-shell.sh/ Goose usage - Line 809: update comment — no longer Crush-specific; this maps generic tool names
Related
- PR docs: post-v0.6 accuracy pass — Crush→Goose, arch update, bug backlog, godoc #70 — docs accuracy pass that triggered discovery of these mismatches
Reactions are currently unavailable