From 196be59d71e536f4824fdfe2c99d55384ec927ae Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?T=C3=82CHES?= Date: Tue, 17 Mar 2026 17:11:52 -0600 Subject: [PATCH] fix: deduplicate tierLabel/tierOrdinal exports (#988) * fix: deduplicate tierLabel and tierOrdinal exports Co-Authored-By: Claude Opus 4.6 (1M context) * fix: update tierLabel test to accept re-export pattern The test was checking source text for "export function tierLabel" which doesn't match the re-export syntax "export { tierLabel } from ...". Co-Authored-By: Claude Opus 4.6 (1M context) --------- Co-authored-by: Claude Opus 4.6 (1M context) --- src/resources/extensions/gsd/complexity.ts | 17 ++--------------- .../gsd/tests/complexity-routing.test.ts | 3 ++- 2 files changed, 4 insertions(+), 16 deletions(-) diff --git a/src/resources/extensions/gsd/complexity.ts b/src/resources/extensions/gsd/complexity.ts index c27c388be..72f9b3009 100644 --- a/src/resources/extensions/gsd/complexity.ts +++ b/src/resources/extensions/gsd/complexity.ts @@ -135,21 +135,8 @@ export function classifyUnitComplexity( // ─── Tier Helpers ───────────────────────────────────────────────────────── -export function tierLabel(tier: ComplexityTier): string { - switch (tier) { - case "light": return "L"; - case "standard": return "S"; - case "heavy": return "H"; - } -} - -export function tierOrdinal(tier: ComplexityTier): number { - switch (tier) { - case "light": return 0; - case "standard": return 1; - case "heavy": return 2; - } -} +// tierLabel and tierOrdinal are exported from complexity-classifier.ts (single source) +export { tierLabel, tierOrdinal } from "./complexity-classifier.js"; export function escalateTier(currentTier: ComplexityTier): ComplexityTier | null { switch (currentTier) { diff --git a/src/resources/extensions/gsd/tests/complexity-routing.test.ts b/src/resources/extensions/gsd/tests/complexity-routing.test.ts index 634012cd5..b9d50e3c9 100644 --- a/src/resources/extensions/gsd/tests/complexity-routing.test.ts +++ b/src/resources/extensions/gsd/tests/complexity-routing.test.ts @@ -281,7 +281,8 @@ test("unit-classify: escalateTier function exists", () => { test("unit-classify: tierLabel function exists", () => { assert.ok( - complexitySrcFull.includes("export function tierLabel"), + complexitySrcFull.includes("export function tierLabel") || + complexitySrcFull.includes("export { tierLabel"), "should export tierLabel for dashboard display", ); });