From 0b8254bfa059bf8c063bcc0f9e0614fbd6ee462e Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?T=C3=82CHES?= Date: Wed, 18 Mar 2026 11:23:54 -0600 Subject: [PATCH] refactor: remove commands.ts re-exports, import directly from submodules (#1204) Co-authored-by: Claude Opus 4.6 (1M context) --- src/resources/extensions/gsd/commands.ts | 6 ------ src/resources/extensions/gsd/index.ts | 3 ++- src/resources/extensions/gsd/tests/gsd-inspect.test.ts | 2 +- 3 files changed, 3 insertions(+), 8 deletions(-) diff --git a/src/resources/extensions/gsd/commands.ts b/src/resources/extensions/gsd/commands.ts index 47df5b898..04643bb4c 100644 --- a/src/resources/extensions/gsd/commands.ts +++ b/src/resources/extensions/gsd/commands.ts @@ -45,12 +45,6 @@ import { handleCleanupBranches, handleCleanupSnapshots, handleSkip, handleDryRun import { handleDoctor, handleSteer, handleCapture, handleTriage, handleKnowledge, handleRunHook, handleUpdate, handleSkillHealth } from "./commands-handlers.js"; import { handleLogs } from "./commands-logs.js"; -// ─── Re-exports (preserve public API surface) ─────────────────────────────── -export { handlePrefs, handlePrefsMode, handlePrefsWizard, ensurePreferencesFile, handleImportClaude, buildCategorySummaries, serializePreferencesToFrontmatter, yamlSafeString, configureMode } from "./commands-prefs-wizard.js"; -export { TOOL_KEYS, loadToolApiKeys, getConfigAuthStorage, handleConfig } from "./commands-config.js"; -export { type InspectData, formatInspectOutput, handleInspect } from "./commands-inspect.js"; -export { handleCleanupBranches, handleCleanupSnapshots, handleSkip, handleDryRun } from "./commands-maintenance.js"; -export { handleDoctor, handleSteer, handleCapture, handleTriage, handleKnowledge, handleRunHook, handleUpdate, handleSkillHealth } from "./commands-handlers.js"; export function dispatchDoctorHeal(pi: ExtensionAPI, scope: string | undefined, reportText: string, structuredIssues: string): void { const workflowPath = process.env.GSD_WORKFLOW_PATH ?? join(process.env.HOME ?? "~", ".pi", "GSD-WORKFLOW.md"); diff --git a/src/resources/extensions/gsd/index.ts b/src/resources/extensions/gsd/index.ts index cc1b60d92..f738d4f27 100644 --- a/src/resources/extensions/gsd/index.ts +++ b/src/resources/extensions/gsd/index.ts @@ -27,7 +27,8 @@ import { createBashTool, createWriteTool, createReadTool, createEditTool, isTool import { Type } from "@sinclair/typebox"; import { debugLog, debugTime } from "./debug-logger.js"; -import { registerGSDCommand, loadToolApiKeys } from "./commands.js"; +import { registerGSDCommand } from "./commands.js"; +import { loadToolApiKeys } from "./commands-config.js"; import { registerExitCommand } from "./exit-command.js"; import { registerWorktreeCommand, getWorktreeOriginalCwd, getActiveWorktreeName } from "./worktree-command.js"; import { getActiveAutoWorktreeContext } from "./auto-worktree.js"; diff --git a/src/resources/extensions/gsd/tests/gsd-inspect.test.ts b/src/resources/extensions/gsd/tests/gsd-inspect.test.ts index 83daad70d..947313c09 100644 --- a/src/resources/extensions/gsd/tests/gsd-inspect.test.ts +++ b/src/resources/extensions/gsd/tests/gsd-inspect.test.ts @@ -3,7 +3,7 @@ // Tests the pure formatInspectOutput function with known data. import { createTestContext } from './test-helpers.ts'; -import { formatInspectOutput, type InspectData } from '../commands.ts'; +import { formatInspectOutput, type InspectData } from '../commands-inspect.ts'; const { assertEq, assertTrue, assertMatch, report } = createTestContext();