From c72b893916a3274d844cb9832e15a5471b41c30f Mon Sep 17 00:00:00 2001 From: Tom Boucher Date: Thu, 30 Apr 2026 00:57:17 -0400 Subject: [PATCH] fix(test): unbreak gemini-namespacing test after #2790 skill consolidation (#2886) MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Closes #2876 follow-up — CI on main fails because the punctuation test in tests/gemini-namespacing.test.cjs hardcoded `/gsd-scan` as a known command, but #2824 (consolidate 86 → 59 skills) removed scan.md from commands/gsd/. The roster now correctly returns "scan is unknown, leave unchanged" — the conversion is right, the test fixture is stale. Swap `scan` for `health` in the punctuation test. Both are bedrock commands; the test still exercises the original intent (period vs exclamation handling on adjacent slash commands). Note added so the next consolidation reviewer knows the swap pattern. `npm test`: 5936/5936 pass. --- tests/gemini-namespacing.test.cjs | 8 ++++++-- 1 file changed, 6 insertions(+), 2 deletions(-) diff --git a/tests/gemini-namespacing.test.cjs b/tests/gemini-namespacing.test.cjs index f83ceb92c..213f991c4 100644 --- a/tests/gemini-namespacing.test.cjs +++ b/tests/gemini-namespacing.test.cjs @@ -93,8 +93,12 @@ describe('Gemini Slash Command Namespacing (Regex)', () => { }); test('converts commands ending with punctuation', () => { - const input = 'Run /gsd-help. Or /gsd-scan!'; - const expected = 'Run /gsd:help. Or /gsd:scan!'; + // Use two stable, foundational commands so this test doesn't drift when + // the roster gets consolidated (cf. #2790, which removed `scan`). `help` + // and `health` are both bedrock; if either ever gets removed, swap to + // any other entry currently in commands/gsd/. + const input = 'Run /gsd-help. Or /gsd-health!'; + const expected = 'Run /gsd:help. Or /gsd:health!'; assert.strictEqual(convertSlashCommandsToGeminiMentions(input), expected); });