@fission-ai/openspec 1.14.0 → 1.14.2

package/dist/cli/index.js

@@ -1,34 +1,22 @@
-import { asStatus } from '../commands/shared-output.js';
 import { Command, Option } from 'commander';
 import { createRequire } from 'module';
-import ora from 'ora';
 import path from 'path';
 import { fileURLToPath } from 'url';
 import { existsSync, promises as fs } from 'fs';
 import { AI_TOOLS, TOOL_ID_ALIASES } from '../core/config.js';
-import { UpdateCommand } from '../core/update.js';
-import { getAvailableCliUpdate, displayCliUpdateNote, shouldOfferUpgrade, getInstallDir, offerCliUpgrade, rerunUpdateWithUpgradedCli, displayUpgradeCommand, isSourceCheckout, checkForCliUpdate, getCliInstallInfo, getCliUpdateCommand, canSelfUpgrade, buildVersionReportLines, } from '../core/version-check.js';
-import { ListCommand } from '../core/list.js';
-import { ArchiveCommand } from '../core/archive.js';
-import { ViewCommand } from '../core/view.js';
-import { resolveRootForCommand, toRootOutput } from '../core/root-selection.js';
-import { registerSpecCommand } from '../commands/spec.js';
-import { ChangeCommand } from '../commands/change.js';
-import { ValidateCommand } from '../commands/validate.js';
-import { ShowCommand } from '../commands/show.js';
-import { CompletionCommand } from '../commands/completion.js';
-import { FeedbackCommand } from '../commands/feedback.js';
-import { registerConfigCommand } from '../commands/config.js';
-import { registerSchemaCommand } from '../commands/schema.js';
-import { registerStoreCommand } from '../commands/store.js';
-import { registerDoctorCommand } from '../commands/doctor.js';
-import { registerContextCommand } from '../commands/context.js';
-import { registerWorksetCommand } from '../commands/workset.js';
-import { statusCommand, BATCH_STATUS_FAILURE_PAYLOAD, instructionsCommand, applyInstructionsCommand, archiveInstructionsCommand, templatesCommand, schemasCommand, newChangeCommand, DEFAULT_SCHEMA, } from '../commands/workflow/index.js';
-import { maybeShowTelemetryNotice, trackCommand, shutdown } from '../telemetry/index.js';
-import { maybeShowCompletionTip } from '../core/completion-tip.js';
+import { registerSpecCommand } from './commands/spec.js';
+import { registerConfigCommand } from './commands/config.js';
+import { registerSchemaCommand } from './commands/schema.js';
+import { registerStoreCommand } from './commands/store.js';
+import { registerDoctorCommand } from './commands/doctor.js';
+import { registerContextCommand } from './commands/context.js';
+import { registerWorksetCommand } from './commands/workset.js';
+import { DEFAULT_SCHEMA } from '../commands/workflow/default-schema.js';
 import { COMMON_FLAGS } from '../core/completions/shared-flags.js';
-import { isInteractive } from '../utils/interactive.js';
+// Startup cost: every command's implementation, and the packages it uses,
+// loads with `await import()` inside its action. This module and
+// ./commands/ only define commands (names, options, help), so `--version`,
+// `--help`, and each command load no other command's implementation.
 const STORE_OPTION_DESCRIPTION = COMMON_FLAGS.store.description;
 // Deliberate rejection path: --store-path stays registered (hidden) so the
 // resolver can explain that registering the path is the supported route,
@@ -37,14 +25,16 @@ const STORE_OPTION_DESCRIPTION = COMMON_FLAGS.store.description;
 function hiddenStorePathOption() {
     return new Option('--store-path <path>', 'Not supported; register the path with "openspec store register <path>" and use --store <id>').hideHelp();
 }
-function failWithError(error, json) {
+async function failWithError(error, json) {
     // The agent contract: every --json failure leaves exactly one JSON
     // document on stdout (the command's null-shape plus a status array).
     if (json?.enabled) {
+        const { asStatus } = await import('../commands/shared-output.js');
         console.log(JSON.stringify({ ...(json.payload ?? {}), status: [asStatus(error, json.fallbackCode ?? 'command_error')] }, null, 2));
         process.exitCode = 1;
         return;
     }
+    const { default: ora } = await import('ora');
     ora().fail(`Error: ${error.message}`);
     // Resolution and store errors carry a pasteable fix - never drop it.
     const fix = error.diagnostic?.fix;
@@ -100,6 +90,13 @@ export function isJsonRun(command) {
 export function isCompletionRun(commandPath) {
     return commandPath.split(':')[0] === 'completion' || commandPath === '__complete';
 }
+/**
+ * True for the hidden `__complete` resolver alone. It runs on every Tab press,
+ * so the hooks skip telemetry and the completion tip for it entirely.
+ */
+export function isTabCompletionRun(commandPath) {
+    return commandPath === '__complete';
+}
 /**
  * True when the first-run completions tip must be deferred rather than shown.
  *
@@ -124,6 +121,7 @@ program
     .option('--json', 'Output as JSON')
     .option('--check', 'Check the registry for a newer version')
     .action(async (options) => {
+    const { getCliInstallInfo, checkForCliUpdate, getCliUpdateCommand, canSelfUpgrade, buildVersionReportLines, } = await import('../core/version-check.js');
     const install = getCliInstallInfo();
     const update = options.check ? await checkForCliUpdate() : undefined;
     const command = update?.status === 'available' ? getCliUpdateCommand(install) : null;
@@ -158,9 +156,16 @@ program.hook('preAction', async (thisCommand, actionCommand) => {
     if (opts.color === false) {
         process.env.NO_COLOR = '1';
     }
+    // `__complete` runs on every Tab press, with stderr sent to /dev/null by the
+    // generated scripts. Tracking it would hold each press open for the request,
+    // and the first-run notice would be marked seen without anyone reading it.
+    if (isTabCompletionRun(getCommandPath(actionCommand))) {
+        return;
+    }
     // Show first-run telemetry notice (if not seen). It's written to stderr, so it
     // never pollutes stdout — but --json runs still defer it (see isJsonRun) so the
     // very first invocation stays free of any incidental output on either stream.
+    const { maybeShowTelemetryNotice, trackCommand } = await import('../telemetry/index.js');
     await maybeShowTelemetryNotice({ silent: isJsonRun(actionCommand) });
     // Track command execution (use actionCommand to get the actual subcommand)
     const commandPath = getCommandPath(actionCommand);
@@ -168,6 +173,11 @@ program.hook('preAction', async (thisCommand, actionCommand) => {
 });
 // Shutdown telemetry after command completes
 program.hook('postAction', async (_thisCommand, actionCommand) => {
+    // Nothing to flush (preAction tracked nothing), and the tip is always
+    // deferred for completion runs, so skip loading either module.
+    if (isTabCompletionRun(getCommandPath(actionCommand))) {
+        return;
+    }
     // Show the first-run shell-completions tip (on stderr, so piped stdout stays
     // clean). postAction, not preAction: the tip trails the command's own output
     // instead of pushing an error message or `init`'s setup summary down the
@@ -175,6 +185,7 @@ program.hook('postAction', async (_thisCommand, actionCommand) => {
     // `openspec completion ...`, and a stderr that is not a terminal (agents and
     // pipes would otherwise silently burn the user's one-shot tip).
     try {
+        const { maybeShowCompletionTip } = await import('../core/completion-tip.js');
         await maybeShowCompletionTip({
             silent: shouldDeferCompletionTip(actionCommand, Boolean(process.stderr.isTTY)),
         });
@@ -182,6 +193,7 @@ program.hook('postAction', async (_thisCommand, actionCommand) => {
     finally {
         // The flush runs even if the hint throws: parse() is synchronous, so a
         // rejection here has no catch anywhere above it.
+        const { shutdown } = await import('../telemetry/index.js');
         await shutdown();
     }
 });
@@ -236,7 +248,7 @@ program
         await initCommand.execute(targetPath);
     }
     catch (error) {
-        failWithError(error);
+        await failWithError(error);
         process.exit(1);
     }
 });
@@ -257,7 +269,7 @@ program
         await initCommand.execute('.');
     }
     catch (error) {
-        failWithError(error);
+        await failWithError(error);
         process.exit(1);
     }
 });
@@ -267,6 +279,11 @@ program
     .option('--force', 'Force update even when tools are up to date')
     .action(async (targetPath = '.', options) => {
     try {
+        const [{ getInstallDir, isSourceCheckout, getAvailableCliUpdate, shouldOfferUpgrade, displayCliUpdateNote, offerCliUpgrade, rerunUpdateWithUpgradedCli, displayUpgradeCommand, }, { isInteractive }, { UpdateCommand },] = await Promise.all([
+            import('../core/version-check.js'),
+            import('../utils/interactive.js'),
+            import('../core/update.js'),
+        ]);
         const installDir = getInstallDir();
         // Running from a clone: the version is whatever the branch says, so any
         // upgrade advice would be noise. Decided before the request, so a
@@ -318,7 +335,7 @@ program
         }
     }
     catch (error) {
-        failWithError(error);
+        await failWithError(error);
         process.exit(1);
     }
 });
@@ -338,6 +355,10 @@ program
         if (options?.specs && (options.archived || options.all)) {
             throw new Error('--archived and --all can only be used when listing changes.');
         }
+        const [{ resolveRootForCommand, toRootOutput }, { ListCommand }] = await Promise.all([
+            import('../core/root-selection.js'),
+            import('../core/list.js'),
+        ]);
         const root = await resolveRootForCommand(options ?? {}, {
             json: options?.json,
             failurePayload: options?.specs ? { specs: [], root: null } : { changes: [], root: null },
@@ -360,7 +381,7 @@ program
         });
     }
     catch (error) {
-        failWithError(error, {
+        await failWithError(error, {
             enabled: options?.json,
             payload: options?.specs ? { specs: [], root: null } : { changes: [], root: null },
             fallbackCode: 'list_error',
@@ -378,6 +399,10 @@ program
         // Implicit cwd fallback stays enabled so `view` keeps accepting the same
         // directories as `list`/`status` — notably pre-config.yaml `openspec/`
         // dirs. ViewCommand still reports a missing openspec/ directory itself.
+        const [{ resolveRootForCommand }, { ViewCommand }] = await Promise.all([
+            import('../core/root-selection.js'),
+            import('../core/view.js'),
+        ]);
         const root = await resolveRootForCommand(options ?? {});
         if (!root) {
             return;
@@ -386,7 +411,7 @@ program
         await viewCommand.execute(root.path);
     }
     catch (error) {
-        failWithError(error);
+        await failWithError(error);
         process.exit(1);
     }
 });
@@ -408,6 +433,7 @@ changeCmd
     .option('--no-interactive', 'Disable interactive prompts')
     .action(async (changeName, options) => {
     try {
+        const { ChangeCommand } = await import('../commands/change.js');
         const changeCommand = new ChangeCommand();
         await changeCommand.show(changeName, options);
     }
@@ -424,6 +450,7 @@ changeCmd
     .action(async (options) => {
     try {
         console.error('Warning: "openspec change list" is deprecated. Use "openspec list".');
+        const { ChangeCommand } = await import('../commands/change.js');
         const changeCommand = new ChangeCommand();
         await changeCommand.list(options);
     }
@@ -440,6 +467,7 @@ changeCmd
     .option('--no-interactive', 'Disable interactive prompts')
     .action(async (changeName, options) => {
     try {
+        const { ChangeCommand } = await import('../commands/change.js');
         const changeCommand = new ChangeCommand();
         // validate() already sets process.exitCode, and Node honours it at
         // natural exit. Calling process.exit() here would skip commander's
@@ -464,11 +492,12 @@ program
     .addOption(hiddenStorePathOption())
     .action(async (changeName, options) => {
     try {
+        const { ArchiveCommand } = await import('../core/archive.js');
         const archiveCommand = new ArchiveCommand();
         await archiveCommand.execute(changeName, options);
     }
     catch (error) {
-        failWithError(error);
+        await failWithError(error);
         process.exit(1);
     }
 });
@@ -497,11 +526,12 @@ program
     .addOption(hiddenStorePathOption())
     .action(async (itemName, options) => {
     try {
+        const { ValidateCommand } = await import('../commands/validate.js');
         const validateCommand = new ValidateCommand();
         await validateCommand.execute(itemName, options);
     }
     catch (error) {
-        failWithError(error, { enabled: options?.json, fallbackCode: 'validate_error' });
+        await failWithError(error, { enabled: options?.json, fallbackCode: 'validate_error' });
         process.exit(1);
     }
 });
@@ -528,11 +558,12 @@ program
     .allowUnknownOption(true)
     .action(async (itemName, options) => {
     try {
+        const { ShowCommand } = await import('../commands/show.js');
         const showCommand = new ShowCommand();
         await showCommand.execute(itemName, options ?? {});
     }
     catch (error) {
-        failWithError(error, { enabled: options?.json, fallbackCode: 'show_error' });
+        await failWithError(error, { enabled: options?.json, fallbackCode: 'show_error' });
         process.exit(1);
     }
 });
@@ -543,11 +574,12 @@ program
     .option('--body <text>', 'Detailed description for the feedback')
     .action(async (message, options) => {
     try {
+        const { FeedbackCommand } = await import('../commands/feedback.js');
         const feedbackCommand = new FeedbackCommand();
         await feedbackCommand.execute(message, options);
     }
     catch (error) {
-        failWithError(error);
+        await failWithError(error);
         process.exit(1);
     }
 });
@@ -560,11 +592,12 @@ completionCmd
     .description('Generate completion script for a shell (outputs to stdout)')
     .action(async (shell) => {
     try {
+        const { CompletionCommand } = await import('../commands/completion.js');
         const completionCommand = new CompletionCommand();
         await completionCommand.generate({ shell });
     }
     catch (error) {
-        failWithError(error);
+        await failWithError(error);
         process.exit(1);
     }
 });
@@ -574,11 +607,12 @@ completionCmd
     .option('--verbose', 'Show detailed installation output')
     .action(async (shell, options) => {
     try {
+        const { CompletionCommand } = await import('../commands/completion.js');
         const completionCommand = new CompletionCommand();
         await completionCommand.install({ shell, verbose: options?.verbose });
     }
     catch (error) {
-        failWithError(error);
+        await failWithError(error);
         process.exit(1);
     }
 });
@@ -588,11 +622,12 @@ completionCmd
     .option('-y, --yes', 'Skip confirmation prompts')
     .action(async (shell, options) => {
     try {
+        const { CompletionCommand } = await import('../commands/completion.js');
         const completionCommand = new CompletionCommand();
         await completionCommand.uninstall({ shell, yes: options?.yes });
     }
     catch (error) {
-        failWithError(error);
+        await failWithError(error);
         process.exit(1);
     }
 });
@@ -602,6 +637,7 @@ program
     .description('Output completion data in machine-readable format (internal use)')
     .action(async (type) => {
     try {
+        const { CompletionCommand } = await import('../commands/completion.js');
         const completionCommand = new CompletionCommand();
         await completionCommand.complete({ type });
     }
@@ -625,14 +661,17 @@ program
     .addOption(hiddenStorePathOption())
     .action(async (options) => {
     try {
+        const { statusCommand } = await import('../commands/workflow/status.js');
         await statusCommand(options);
     }
     catch (error) {
-        failWithError(error, {
+        await failWithError(error, {
             enabled: options.json,
             // The batch null-shape; the single-change failure shape is
             // pre-existing contract and stays payload-free.
-            payload: options.all ? BATCH_STATUS_FAILURE_PAYLOAD : undefined,
+            payload: options.all
+                ? (await import('../commands/workflow/status.js')).BATCH_STATUS_FAILURE_PAYLOAD
+                : undefined,
             fallbackCode: 'change_error',
         });
         process.exit(1);
@@ -649,6 +688,7 @@ program
     .addOption(hiddenStorePathOption())
     .action(async (artifactId, options) => {
     try {
+        const { applyInstructionsCommand, archiveInstructionsCommand, instructionsCommand } = await import('../commands/workflow/instructions.js');
         // Workflow instruction surfaces are reserved command branches, not artifacts.
         if (artifactId === 'apply') {
             await applyInstructionsCommand(options);
@@ -661,7 +701,7 @@ program
         }
     }
     catch (error) {
-        failWithError(error, { enabled: options.json, fallbackCode: 'change_error' });
+        await failWithError(error, { enabled: options.json, fallbackCode: 'change_error' });
         process.exit(1);
     }
 });
@@ -673,10 +713,11 @@ program
     .option('--json', 'Output as JSON mapping artifact IDs to template paths')
     .action(async (options) => {
     try {
+        const { templatesCommand } = await import('../commands/workflow/templates.js');
         await templatesCommand(options);
     }
     catch (error) {
-        failWithError(error);
+        await failWithError(error);
         process.exit(1);
     }
 });
@@ -689,10 +730,11 @@ program
     .addOption(hiddenStorePathOption())
     .action(async (options) => {
     try {
+        const { schemasCommand } = await import('../commands/workflow/schemas.js');
         await schemasCommand(options);
     }
     catch (error) {
-        failWithError(error, {
+        await failWithError(error, {
             enabled: options.json,
             payload: { schemas: [], root: null },
             fallbackCode: 'schemas_error',
@@ -717,10 +759,11 @@ newCmd
     .addOption(new Option('--areas <names>', 'No longer supported').hideHelp())
     .action(async (name, options) => {
     try {
+        const { newChangeCommand } = await import('../commands/workflow/new-change.js');
         await newChangeCommand(name, options);
     }
     catch (error) {
-        failWithError(error);
+        await failWithError(error);
         process.exit(1);
     }
 });

package/dist/commands/completion.js

@@ -1,4 +1,3 @@
-import ora from 'ora';
 import { CompletionFactory } from '../core/completions/factory.js';
 import { COMMAND_REGISTRY } from '../core/completions/command-registry.js';
 import { detectShell } from '../utils/shell-detection.js';
@@ -93,6 +92,9 @@ export class CompletionCommand {
     async installForShell(shell, verbose) {
         const generator = CompletionFactory.createGenerator(shell);
         const installer = CompletionFactory.createInstaller(shell);
+        // Loaded here, not at the top: `__complete` shares this module and runs on
+        // every Tab press, so it must not pay for the spinner.
+        const { default: ora } = await import('ora');
         const spinner = ora(`Installing ${shell} completion script...`).start();
         try {
             // Generate the completion script
@@ -187,6 +189,7 @@ export class CompletionCommand {
                 return;
             }
         }
+        const { default: ora } = await import('ora');
         const spinner = ora(`Uninstalling ${shell} completion script...`).start();
         try {
             const result = await installer.uninstall();

package/dist/commands/config.d.ts

@@ -1,4 +1,3 @@
-import { Command } from 'commander';
 import { GlobalConfig } from '../core/global-config.js';
 import type { Profile, Delivery } from '../core/global-config.js';
 /**
@@ -40,11 +39,21 @@ export declare function formatWorkflowSummary(workflows: readonly string[], prof
  * Build a user-facing diff summary between two profile states.
  */
 export declare function diffProfileState(before: ProfileState, after: ProfileState): ProfileStateDiff;
-/**
- * Register the config command and all its subcommands.
- *
- * @param program - The Commander program instance
- */
-export declare function registerConfigCommand(program: Command): void;
+export declare function configPathCommand(): void;
+export declare function configListCommand(options: {
+    json?: boolean;
+}): void;
+export declare function configGetCommand(key: string): void;
+export declare function configSetCommand(key: string, value: string, options: {
+    string?: boolean;
+    allowUnknown?: boolean;
+}): void;
+export declare function configUnsetCommand(key: string): void;
+export declare function configResetCommand(options: {
+    all?: boolean;
+    yes?: boolean;
+}): Promise<void>;
+export declare function configEditCommand(): Promise<void>;
+export declare function configProfileCommand(preset?: string): Promise<void>;
 export {};
 //# sourceMappingURL=config.d.ts.map