mirror of
https://github.com/garrytan/gstack.git
synced 2026-08-07 14:58:37 +02:00
Remove Opus fanout overlay nudge
This commit is contained in:
@@ -19,7 +19,6 @@ This release adds the reverse of `/codex`: external hosts can now ask Claude for
|
|||||||
|
|
||||||
- `hosts/claude.ts`: excludes the Claude outside-voice skill from Claude-host generation.
|
- `hosts/claude.ts`: excludes the Claude outside-voice skill from Claude-host generation.
|
||||||
- `test/brain-sync.test.ts`: the `GSTACK_HOME` isolation test now snapshots and preserves the real config file instead of assuming local machine state.
|
- `test/brain-sync.test.ts`: the `GSTACK_HOME` isolation test now snapshots and preserves the real config file instead of assuming local machine state.
|
||||||
- `model-overlays/opus-4-7.md`: restores the fanout nudge expected by the Opus overlay regression test.
|
|
||||||
- `claude/SKILL.md.tmpl`: uses `mktemp` for diff capture in review/challenge mode instead of a `$$`-based temp path, avoiding collisions across concurrent invocations.
|
- `claude/SKILL.md.tmpl`: uses `mktemp` for diff capture in review/challenge mode instead of a `$$`-based temp path, avoiding collisions across concurrent invocations.
|
||||||
|
|
||||||
### Changed
|
### Changed
|
||||||
|
|||||||
@@ -1,10 +1,5 @@
|
|||||||
{{INHERIT:claude}}
|
{{INHERIT:claude}}
|
||||||
|
|
||||||
**Fan out explicitly.** When independent file reads, searches, inspections,
|
|
||||||
or verification steps can run in parallel, launch them in the same turn instead
|
|
||||||
of serializing them. Keep the work bounded and avoid duplicate probes, but do
|
|
||||||
not underuse parallel tool calls when the next step benefits from a wider view.
|
|
||||||
|
|
||||||
**Effort-match the step.** Simple file reads, config checks, command lookups, and
|
**Effort-match the step.** Simple file reads, config checks, command lookups, and
|
||||||
mechanical edits don't need deep reasoning. Complete them quickly and move on. Reserve
|
mechanical edits don't need deep reasoning. Complete them quickly and move on. Reserve
|
||||||
extended thinking for genuinely hard subproblems: architectural tradeoffs, subtle bugs,
|
extended thinking for genuinely hard subproblems: architectural tradeoffs, subtle bugs,
|
||||||
|
|||||||
@@ -7,7 +7,7 @@
|
|||||||
* 3. The SDK event stream contains the types we assume (system init, assistant,
|
* 3. The SDK event stream contains the types we assume (system init, assistant,
|
||||||
* result) with the fields we destructure.
|
* result) with the fields we destructure.
|
||||||
* 4. `scripts/resolvers/model-overlay.ts` resolves `{{INHERIT:claude}}` against
|
* 4. `scripts/resolvers/model-overlay.ts` resolves `{{INHERIT:claude}}` against
|
||||||
* `opus-4-7.md` AND the resolved text contains the "Fan out explicitly" nudge.
|
* `opus-4-7.md` with no unresolved inheritance directives.
|
||||||
* 5. A local `claude` binary exists at `which claude` so binary pinning is possible.
|
* 5. A local `claude` binary exists at `which claude` so binary pinning is possible.
|
||||||
*
|
*
|
||||||
* Run: bun run scripts/preflight-agent-sdk.ts
|
* Run: bun run scripts/preflight-agent-sdk.ts
|
||||||
@@ -28,7 +28,7 @@ async function main() {
|
|||||||
failures.push(msg);
|
failures.push(msg);
|
||||||
};
|
};
|
||||||
|
|
||||||
// 1. Overlay resolver + fanout nudge text
|
// 1. Overlay resolver
|
||||||
console.log('1. Overlay resolver');
|
console.log('1. Overlay resolver');
|
||||||
const resolved = readOverlay('opus-4-7');
|
const resolved = readOverlay('opus-4-7');
|
||||||
if (!resolved) {
|
if (!resolved) {
|
||||||
@@ -40,11 +40,6 @@ async function main() {
|
|||||||
} else {
|
} else {
|
||||||
pass('no unresolved INHERIT directives');
|
pass('no unresolved INHERIT directives');
|
||||||
}
|
}
|
||||||
if (!/Fan out explicitly/i.test(resolved)) {
|
|
||||||
fail('resolved overlay does not contain "Fan out explicitly" text');
|
|
||||||
} else {
|
|
||||||
pass('fanout nudge text present in resolved overlay');
|
|
||||||
}
|
|
||||||
}
|
}
|
||||||
|
|
||||||
// 2. Local claude binary exists
|
// 2. Local claude binary exists
|
||||||
|
|||||||
@@ -82,9 +82,8 @@ describe('Opus 4.7 overlay — pacing directive', () => {
|
|||||||
expect(out).toMatch(/user approval/i);
|
expect(out).toMatch(/user approval/i);
|
||||||
});
|
});
|
||||||
|
|
||||||
test('resolved overlay keeps Fan out / Effort-match / Literal interpretation nudges', () => {
|
test('resolved overlay keeps Effort-match / Literal interpretation nudges', () => {
|
||||||
const out = generateModelOverlay(makeCtx('opus-4-7'));
|
const out = generateModelOverlay(makeCtx('opus-4-7'));
|
||||||
expect(out).toContain('Fan out explicitly');
|
|
||||||
expect(out).toContain('Effort-match the step');
|
expect(out).toContain('Effort-match the step');
|
||||||
expect(out).toContain('Literal interpretation awareness');
|
expect(out).toContain('Literal interpretation awareness');
|
||||||
});
|
});
|
||||||
|
|||||||
Reference in New Issue
Block a user