mirror of
https://github.com/garrytan/gstack.git
synced 2026-10-31 22:14:25 +01:00
fix(overlay): claude.md dedicated-tools nudge stops naming tools the host may lack (s01 M1)
The nudge named Glob and Grep, which some Claude hosts do not expose. It now names the host's dedicated file tools and its search tools when it has them, keeping the prefer-dedicated-tools behavior the claude-dedicated-tools-vs-bash overlay eval measures. Updates the prose comment at test/gen-skill-docs.test.ts:643 (comment only, no assertion change). The matching comment in test/fixtures/overlay-nudges.ts is left to its owner.
This commit is contained in:
1 parent
c24344af37
commit
949200ec8f
2 files changed
+4
-3
No files matched your search
@@ -6,5 +6,6 @@ turns out to be unnecessary, mark it skipped with a one-line reason.
|
||||
non-trivial new features), briefly state your approach before executing. This lets
|
||||
the user course-correct cheaply instead of mid-flight.
|
||||
|
||||
**Dedicated tools over Bash.** Prefer Read, Edit, Write, Glob, Grep over shell
|
||||
equivalents (cat, sed, find, grep). The dedicated tools are cheaper and clearer.
|
||||
**Dedicated tools over Bash.** Prefer the host's dedicated file tools (Read, Edit,
|
||||
Write, and its search tools when it has them) over shell equivalents (cat, sed,
|
||||
find, grep). The dedicated tools are cheaper and clearer.
|
||||
@@ -640,7 +640,7 @@ describe('gen-skill-docs', () => {
|
||||
// Should not have Edit, Glob, or Grep in allowed-tools.
|
||||
// Scope to frontmatter (between the first two --- lines) — the body can
|
||||
// legitimately mention these tool names in prose (e.g., Claude model
|
||||
// overlay says "prefer Read, Edit, Write, Glob, Grep over Bash").
|
||||
// overlay names the host's dedicated file and search tools).
|
||||
const fmMatch = qaOnlyContent.match(/^---\n([\s\S]*?)\n---/);
|
||||
expect(fmMatch).not.toBeNull();
|
||||
const frontmatter = fmMatch![1];
|
||||
|
||||
Reference in new issue
Block a user