From 71776cd436a43e4ce52aaa6c98fef26368ec4fb0 Mon Sep 17 00:00:00 2001 From: abmorrill4 Date: Thu, 18 Jun 2026 11:24:40 -0500 Subject: [PATCH] Fix scope ambiguity in ponytail-audit and ponytail-review Boundaries MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit The Boundaries line opened with "Complexity only, correctness bugs, security holes, and performance go to a normal review pass." The comma after "Complexity only" fuses the in-scope item with the out-of-scope list, so a model parsing it literally can read all four categories as targets of the audit — the opposite of intent. Restate the boundary as an explicit scope fence: name what is in scope, then mark correctness/security/performance as explicitly out of scope. "Out of scope" is phrasing models reliably honor as a constraint. Also aligns the scope term with each skill's stated purpose (over-engineering). Applied to both skills/ and the .openclaw/ mirror so the two trees stay in sync. Co-Authored-By: Claude Opus 4.8 --- .openclaw/skills/ponytail-audit/SKILL.md | 5 +++-- .openclaw/skills/ponytail-review/SKILL.md | 5 +++-- skills/ponytail-audit/SKILL.md | 5 +++-- skills/ponytail-review/SKILL.md | 5 +++-- 4 files changed, 12 insertions(+), 8 deletions(-) diff --git a/.openclaw/skills/ponytail-audit/SKILL.md b/.openclaw/skills/ponytail-audit/SKILL.md index 8bc9ca8..ba82812 100644 --- a/.openclaw/skills/ponytail-audit/SKILL.md +++ b/.openclaw/skills/ponytail-audit/SKILL.md @@ -31,6 +31,7 @@ End with `net: - lines, - deps possible.` Nothing to cut: `Lean already. S ## Boundaries -Complexity only, correctness bugs, security holes, and performance go to a -normal review pass. Lists findings, applies nothing. One-shot. +Scope: over-engineering and complexity only. Correctness bugs, security holes, +and performance are explicitly out of scope — route them to a normal review +pass. Lists findings, applies nothing. One-shot. "stop ponytail-audit" or "normal mode" to revert. diff --git a/.openclaw/skills/ponytail-review/SKILL.md b/.openclaw/skills/ponytail-review/SKILL.md index 8037d68..fbb5923 100644 --- a/.openclaw/skills/ponytail-review/SKILL.md +++ b/.openclaw/skills/ponytail-review/SKILL.md @@ -44,8 +44,9 @@ If there is nothing to cut, say `Lean already. Ship.` and stop. ## Boundaries -Complexity only, correctness bugs, security holes, and performance go to a -normal review pass, not this one. A single smoke test or `assert`-based +Scope: over-engineering and complexity only. Correctness bugs, security holes, +and performance are explicitly out of scope — route them to a normal review +pass, not this one. A single smoke test or `assert`-based self-check is the ponytail minimum, not bloat, never flag it for deletion. Does not apply the fixes, only lists them. "stop ponytail-review" or "normal mode": revert to verbose review style. diff --git a/skills/ponytail-audit/SKILL.md b/skills/ponytail-audit/SKILL.md index 9c803ee..b95bb93 100644 --- a/skills/ponytail-audit/SKILL.md +++ b/skills/ponytail-audit/SKILL.md @@ -35,6 +35,7 @@ End with `net: - lines, - deps possible.` Nothing to cut: `Lean already. S ## Boundaries -Complexity only, correctness bugs, security holes, and performance go to a -normal review pass. Lists findings, applies nothing. One-shot. +Scope: over-engineering and complexity only. Correctness bugs, security holes, +and performance are explicitly out of scope — route them to a normal review +pass. Lists findings, applies nothing. One-shot. "stop ponytail-audit" or "normal mode" to revert. diff --git a/skills/ponytail-review/SKILL.md b/skills/ponytail-review/SKILL.md index f24cf07..2bbfeb9 100644 --- a/skills/ponytail-review/SKILL.md +++ b/skills/ponytail-review/SKILL.md @@ -49,8 +49,9 @@ If there is nothing to cut, say `Lean already. Ship.` and stop. ## Boundaries -Complexity only, correctness bugs, security holes, and performance go to a -normal review pass, not this one. A single smoke test or `assert`-based +Scope: over-engineering and complexity only. Correctness bugs, security holes, +and performance are explicitly out of scope — route them to a normal review +pass, not this one. A single smoke test or `assert`-based self-check is the ponytail minimum, not bloat, never flag it for deletion. Does not apply the fixes, only lists them. "stop ponytail-review" or "normal mode": revert to verbose review style.