fix(audit): require a whole-tree caller check before delete: (#679)

.openclaw/skills/ponytail-audit/SKILL.md

@@ -23,9 +23,14 @@ Same as ponytail-review:
 Deps the stdlib or platform already ships, single-implementation interfaces,
 factories with one product, wrappers that only delegate, files exporting one
 thing, dead flags and config, hand-rolled stdlib.
+Before emitting `delete:`, grep the entire tree including tests, fixtures, and
+dynamic/string references; a symbol used only by tests is still used.
 
 ## Output
 
+Open with `git rev-parse --short HEAD` (no network; `unknown base` outside a
+git repo). Re-verify each finding against current HEAD before applying; callers
+appear.
 One line per finding, ranked: `<tag> <what to cut>. <replacement>. [path]`.
 End with `net: -<N> lines, -<M> deps possible.` Nothing to cut: `Lean already. Ship.`
 

.opencode/command/ponytail-audit.md

@@ -2,4 +2,4 @@
 description: Audit the whole repo for over-engineering, what can be deleted
 ---
 
-Audit the entire repository for over-engineering only, not correctness. Scan the whole tree, not a diff. One line per finding, ranked biggest cut first: <tag> <what to cut>. <replacement>. [path]. Tags: delete (dead code/speculative feature), stdlib (reinvented standard library), native (dependency doing what the platform does), yagni (abstraction with one implementation), shrink (same logic, fewer lines). End with the net lines and dependencies removable. If nothing to cut: 'Lean already. Ship.'
+Audit the entire repository for over-engineering only, not correctness. Scan the whole tree, not a diff. Open with `git rev-parse --short HEAD` (no network; `unknown base` outside a git repo). Before emitting delete:, grep the entire tree including tests, fixtures, and dynamic/string references; a symbol used only by tests is still used. Re-verify each finding against current HEAD before applying. One line per finding, ranked biggest cut first: <tag> <what to cut>. <replacement>. [path]. Tags: delete (dead code/speculative feature), stdlib (reinvented standard library), native (dependency doing what the platform does), yagni (abstraction with one implementation), shrink (same logic, fewer lines). End with the net lines and dependencies removable. If nothing to cut: 'Lean already. Ship.'

commands/ponytail-audit.toml

@@ -1,2 +1,2 @@
 description = "Audit the whole repo for over-engineering, what can be deleted"
-prompt = "Audit the entire repository for over-engineering only, not correctness. Scan the whole tree, not a diff. One line per finding, ranked biggest cut first: <tag> <what to cut>. <replacement>. [path]. Tags: delete (dead code/speculative feature), stdlib (reinvented standard library), native (dependency doing what the platform does), yagni (abstraction with one implementation), shrink (same logic, fewer lines). End with the net lines and dependencies removable. If nothing to cut: 'Lean already. Ship.'"
+prompt = "Audit the entire repository for over-engineering only, not correctness. Scan the whole tree, not a diff. Open with `git rev-parse --short HEAD` (no network; `unknown base` outside a git repo). Before emitting delete:, grep the entire tree including tests, fixtures, and dynamic/string references; a symbol used only by tests is still used. Re-verify each finding against current HEAD before applying. One line per finding, ranked biggest cut first: <tag> <what to cut>. <replacement>. [path]. Tags: delete (dead code/speculative feature), stdlib (reinvented standard library), native (dependency doing what the platform does), yagni (abstraction with one implementation), shrink (same logic, fewer lines). End with the net lines and dependencies removable. If nothing to cut: 'Lean already. Ship.'"

skills/ponytail-audit/SKILL.md

@@ -27,9 +27,14 @@ Same as ponytail-review:
 Deps the stdlib or platform already ships, single-implementation interfaces,
 factories with one product, wrappers that only delegate, files exporting one
 thing, dead flags and config, hand-rolled stdlib.
+Before emitting `delete:`, grep the entire tree including tests, fixtures, and
+dynamic/string references; a symbol used only by tests is still used.
 
 ## Output
 
+Open with `git rev-parse --short HEAD` (no network; `unknown base` outside a
+git repo). Re-verify each finding against current HEAD before applying; callers
+appear.
 One line per finding, ranked: `<tag> <what to cut>. <replacement>. [path]`.
 End with `net: -<N> lines, -<M> deps possible.` Nothing to cut: `Lean already. Ship.`