CalvinBackup/gstack
1
0
Fork 0
mirror of https://github.com/garrytan/gstack.git synced 2026-09-09 14:38:59 +02:00
Code Issues Packages Projects Releases Wiki Activity

fix(test): skill-e2e-bws slices the $B setup block from the Browser fallback section

Browse Source 
browse/SKILL.md no longer has '## SETUP' / '## Core QA Patterns' (Aside is the
primary driver; the $B block moved under 'Browser fallback'), so the gate test
sliced an empty block and handed the agent nothing to run. Anchor on
'### Find the `$B` binary' up to the next heading. 7/7 pass.

Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com>
This commit is contained in:
Garry Tan
2026-09-06 07:49:13 +00:00
co-authored by Claude Fable 5.1
parent 6928bf03be
commit 2649a3b6de
1 changed files with 21 additions and 9 deletions
Download Patch File Download Diff File Expand all files Collapse all files
test/skill-e2e-bws.test.ts
+21 -9
View File
@@ -86,10 +86,14 @@ Report what each command returned.`,
testConcurrentIfSelected('skillmd-setup-discovery', async () => {
// P2 (v1.2.0): the browse SETUP/binary-discovery block moved from the root
// router to browse/SKILL.md (end anchor is now ## Core QA Patterns).
// router to browse/SKILL.md; the `$B` block now sits under "Browser fallback".
const skillMd = fs.readFileSync(path.join(ROOT, 'browse', 'SKILL.md'), 'utf-8');
const setupStart = skillMd.indexOf('## SETUP');
const setupEnd = skillMd.indexOf('## Core QA Patterns');
// The `$B` setup lives in the Browser fallback section since Aside became
// the primary driver: slice from its heading to the next heading.
const setupStart = skillMd.indexOf('### Find the `$B` binary');
const nextH3 = skillMd.indexOf('\n### ', setupStart + 1);
const nextH2 = skillMd.indexOf('\n## ', setupStart + 1);
const setupEnd = [nextH3, nextH2].filter((i) => i > setupStart).sort((a, b) => a - b)[0] ?? skillMd.length;
const setupBlock = skillMd.slice(setupStart, setupEnd);
// Guard: verify we extracted a valid setup block
@@ -120,10 +124,14 @@ Report whether it worked.`,
const emptyDir = fs.mkdtempSync(path.join(os.tmpdir(), 'skill-e2e-empty-'));
// P2 (v1.2.0): the browse SETUP/binary-discovery block moved from the root
// router to browse/SKILL.md (end anchor is now ## Core QA Patterns).
// router to browse/SKILL.md; the `$B` block now sits under "Browser fallback".
const skillMd = fs.readFileSync(path.join(ROOT, 'browse', 'SKILL.md'), 'utf-8');
const setupStart = skillMd.indexOf('## SETUP');
const setupEnd = skillMd.indexOf('## Core QA Patterns');
// The `$B` setup lives in the Browser fallback section since Aside became
// the primary driver: slice from its heading to the next heading.
const setupStart = skillMd.indexOf('### Find the `$B` binary');
const nextH3 = skillMd.indexOf('\n### ', setupStart + 1);
const nextH2 = skillMd.indexOf('\n## ', setupStart + 1);
const setupEnd = [nextH3, nextH2].filter((i) => i > setupStart).sort((a, b) => a - b)[0] ?? skillMd.length;
const setupBlock = skillMd.slice(setupStart, setupEnd);
const result = await runSkillTest({
@@ -157,10 +165,14 @@ Report the exact output. Do NOT try to fix or install anything — just report w
const nonGitDir = fs.mkdtempSync(path.join(os.tmpdir(), 'skill-e2e-nogit-'));
// P2 (v1.2.0): the browse SETUP/binary-discovery block moved from the root
// router to browse/SKILL.md (end anchor is now ## Core QA Patterns).
// router to browse/SKILL.md; the `$B` block now sits under "Browser fallback".
const skillMd = fs.readFileSync(path.join(ROOT, 'browse', 'SKILL.md'), 'utf-8');
const setupStart = skillMd.indexOf('## SETUP');
const setupEnd = skillMd.indexOf('## Core QA Patterns');
// The `$B` setup lives in the Browser fallback section since Aside became
// the primary driver: slice from its heading to the next heading.
const setupStart = skillMd.indexOf('### Find the `$B` binary');
const nextH3 = skillMd.indexOf('\n### ', setupStart + 1);
const nextH2 = skillMd.indexOf('\n## ', setupStart + 1);
const setupEnd = [nextH3, nextH2].filter((i) => i > setupStart).sort((a, b) => a - b)[0] ?? skillMd.length;
const setupBlock = skillMd.slice(setupStart, setupEnd);
const result = await runSkillTest({