From c3322a59525f70a1d479c875c7a44ec2a9762f96 Mon Sep 17 00:00:00 2001 From: garrytan Date: Tue, 3 Nov 2026 20:23:35 +0000 Subject: [PATCH] after-pin: default skill_routing on (#3938) Co-authored-by: capy-ai[bot] <230910855+capy-ai[bot]@users.noreply.github.com> --- changelog.d/capy-gst-1088-flip-routing.md | 1 + lib/skill-route.ts | 2 +- test/setup-skill-hooks-gate.test.ts | 6 +++--- 3 files changed, 5 insertions(+), 4 deletions(-) create mode 100644 changelog.d/capy-gst-1088-flip-routing.md diff --git a/changelog.d/capy-gst-1088-flip-routing.md b/changelog.d/capy-gst-1088-flip-routing.md new file mode 100644 index 000000000..5a46335b7 --- /dev/null +++ b/changelog.d/capy-gst-1088-flip-routing.md @@ -0,0 +1 @@ +The skill routing hook is now on by default (GST-1088): a prompt that fits an installed gstack skill gets a one-line hint naming it. Turn it off with `gstack-config set skill_routing off` or `./setup --no-skill-routing`. diff --git a/lib/skill-route.ts b/lib/skill-route.ts index 69ad89c43..71356716d 100644 --- a/lib/skill-route.ts +++ b/lib/skill-route.ts @@ -36,7 +36,7 @@ export const SWITCH_ENV: Record = { /** Each switch's one default. setup, gstack-config and gstack-skill-route read it through `gstack-skill-route --default `; the hooks read it here. */ export const SWITCH_DEFAULT: Record = { - skill_routing: 'off', + skill_routing: 'on', skill_contracts: 'off', }; diff --git a/test/setup-skill-hooks-gate.test.ts b/test/setup-skill-hooks-gate.test.ts index 3af9dd5e1..979003bb8 100644 --- a/test/setup-skill-hooks-gate.test.ts +++ b/test/setup-skill-hooks-gate.test.ts @@ -189,9 +189,9 @@ describe('SWITCH_DEFAULT is the one default (a flip PR changes one line)', () => expect(config()).toBe(''); } - test('pin: skill_routing defaults to off today (a flip PR changes lib/skill-route.ts line 39 and this pin)', () => { - expect(original.split('\n')[38]).toBe(" skill_routing: 'off',"); - expect(SWITCH_DEFAULT.skill_routing).toBe('off'); + test('pin: skill_routing defaults to on today (a flip PR changes lib/skill-route.ts line 39 and this pin)', () => { + expect(original.split('\n')[38]).toBe(" skill_routing: 'on',"); + expect(SWITCH_DEFAULT.skill_routing).toBe('on'); }); test('pin: skill_contracts defaults to off today (a flip PR changes lib/skill-route.ts line 40 and this pin)', () => {