sdd-tool 1.4.0 → 1.4.1

package/dist/cli/index.js

@@ -105,6 +105,21 @@ var init_messages = __esm({
 });
 
 // src/errors/base.ts
+function getErrorMessage(error2, fallback = "\uC54C \uC218 \uC5C6\uB294 \uC624\uB958\uAC00 \uBC1C\uC0DD\uD588\uC2B5\uB2C8\uB2E4") {
+  if (error2 instanceof Error) {
+    return error2.message;
+  }
+  if (typeof error2 === "string") {
+    return error2;
+  }
+  if (error2 && typeof error2 === "object" && "message" in error2) {
+    return String(error2.message);
+  }
+  return fallback;
+}
+function getResultErrorMessage(error2, fallback = "\uC54C \uC218 \uC5C6\uB294 \uC624\uB958\uAC00 \uBC1C\uC0DD\uD588\uC2B5\uB2C8\uB2E4") {
+  return error2?.message ?? fallback;
+}
 var SddError, FileSystemError, ValidationError, ChangeError;
 var init_base = __esm({
   "src/errors/base.ts"() {
@@ -17717,7 +17732,7 @@ async function executeSyncCommand(specId, options, projectRoot) {
     ...options,
     specId
   });
-  if (!syncResult.success) {
+  if (!syncResult.success || !syncResult.data) {
     return failure(syncResult.error || new Error("\uB3D9\uAE30\uD654 \uAC80\uC99D \uC2E4\uD328"));
   }
   return success({
@@ -17756,7 +17771,7 @@ function registerSyncCommand(program2) {
         }
       }
       if (result.data.result.missing.length > 0 && !options.json) {
-        process.exit(1);
+        process.exit(ExitCode.VALIDATION_ERROR);
       }
       process.exit(ExitCode.SUCCESS);
     } catch (error2) {
@@ -18570,6 +18585,7 @@ async function executeDiff(projectRoot, options = {}) {
 }
 
 // src/cli/commands/diff.ts
+init_errors();
 function registerDiffCommand(program2) {
   program2.command("diff [commit1] [commit2]").description("\uC2A4\uD399 \uBCC0\uACBD\uC0AC\uD56D \uC2DC\uAC01\uD654").option("--staged", "\uC2A4\uD14C\uC774\uC9D5\uB41C \uBCC0\uACBD\uC0AC\uD56D\uB9CC \uD45C\uC2DC").option("--stat", "\uBCC0\uACBD \uD1B5\uACC4 \uC694\uC57D \uD45C\uC2DC").option("--name-only", "\uBCC0\uACBD\uB41C \uD30C\uC77C\uBA85\uB9CC \uD45C\uC2DC").option("--json", "JSON \uD615\uC2DD\uC73C\uB85C \uCD9C\uB825").option("--no-color", "\uCEEC\uB7EC \uCD9C\uB825 \uBE44\uD65C\uC131\uD654").option("-s, --spec <id>", "\uD2B9\uC815 \uC2A4\uD399\uB9CC \uBE44\uAD50").action(async (commit1, commit2, options) => {
     const projectRoot = process.cwd();
@@ -18584,10 +18600,12 @@ function registerDiffCommand(program2) {
       commit2
     });
     if (!result.success) {
-      console.error(`\uC624\uB958: ${result.error?.message}`);
-      process.exit(1);
+      error(getErrorMessage(result.error));
+      process.exit(ExitCode.GENERAL_ERROR);
+    }
+    if (result.data?.output) {
+      console.log(result.data.output);
     }
-    console.log(result.data?.output);
   });
 }
 
@@ -19579,6 +19597,7 @@ function formatExportResult(result) {
 }
 
 // src/cli/commands/export.ts
+init_errors();
 function registerExportCommand(program2) {
   program2.command("export [specId...]").description("\uC2A4\uD399\uC744 HTML, JSON \uB4F1 \uB2E4\uC591\uD55C \uD615\uC2DD\uC73C\uB85C \uB0B4\uBCF4\uB0B4\uAE30").option("-f, --format <format>", "\uCD9C\uB825 \uD615\uC2DD (html, json, markdown, pdf)", "html").option("-o, --output <path>", "\uCD9C\uB825 \uD30C\uC77C \uACBD\uB85C").option("--theme <theme>", "\uD14C\uB9C8 (light, dark)", "light").option("--all", "\uC804\uCCB4 \uC2A4\uD399 \uB0B4\uBCF4\uB0B4\uAE30", false).option("--toc", "\uBAA9\uCC28 \uD3EC\uD568", true).option("--no-toc", "\uBAA9\uCC28 \uC81C\uC678").option("--include-constitution", "Constitution \uD3EC\uD568", false).option("--include-changes", "\uBCC0\uACBD \uC81C\uC548 \uD3EC\uD568", false).option("--json", "JSON \uD615\uC2DD \uCD9C\uB825 (\uACB0\uACFC \uBA54\uD0C0\uC815\uBCF4)").action(async (specIds, options) => {
     const projectRoot = process.cwd();
@@ -19598,7 +19617,10 @@ function registerExportCommand(program2) {
       console.log(formatExportResult(result));
     }
     if (!result.success) {
-      process.exit(1);
+      if (result.error) {
+        error("\uB0B4\uBCF4\uB0B4\uAE30 \uC2E4\uD328", result.error);
+      }
+      process.exit(ExitCode.GENERAL_ERROR);
     }
   });
 }
@@ -19985,6 +20007,7 @@ function registerDomainCommand(program2) {
 init_types();
 init_fs();
 import chalk3 from "chalk";
+init_errors();
 async function executeContextSet(domainIds, options, projectPath) {
   const root = projectPath || await findSddRoot(process.cwd());
   if (!root) {
@@ -20078,18 +20101,18 @@ function registerContextCommand(program2) {
   context.command("set").description("\uC791\uC5C5 \uCEE8\uD14D\uC2A4\uD2B8 \uC124\uC815").argument("<domains...>", "\uD65C\uC131\uD654\uD560 \uB3C4\uBA54\uC778 ID \uBAA9\uB85D").option("--no-include-deps", "\uC758\uC874\uC131 \uB3C4\uBA54\uC778 \uC790\uB3D9 \uD3EC\uD568 \uBE44\uD65C\uC131\uD654").action(async (domains, opts) => {
     const result = await executeContextSet(domains, { includeDeps: opts.includeDeps });
     if (!result.success) {
-      console.error(chalk3.red(`\uC624\uB958: ${result.error.message}`));
-      process.exit(1);
+      error(getResultErrorMessage(result.error));
+      process.exit(ExitCode.GENERAL_ERROR);
     }
-    console.log(chalk3.green("\uCEE8\uD14D\uC2A4\uD2B8\uAC00 \uC124\uC815\uB418\uC5C8\uC2B5\uB2C8\uB2E4."));
+    success2("\uCEE8\uD14D\uC2A4\uD2B8\uAC00 \uC124\uC815\uB418\uC5C8\uC2B5\uB2C8\uB2E4.");
     console.log("");
     console.log(formatContextInfo(result.data));
   });
   context.command("show").description("\uD604\uC7AC \uCEE8\uD14D\uC2A4\uD2B8 \uD45C\uC2DC").option("--json", "JSON \uD615\uC2DD\uC73C\uB85C \uCD9C\uB825").action(async (opts) => {
     const result = await executeContextShow();
     if (!result.success) {
-      console.error(chalk3.red(`\uC624\uB958: ${result.error.message}`));
-      process.exit(1);
+      error(getResultErrorMessage(result.error));
+      process.exit(ExitCode.GENERAL_ERROR);
     }
     if (opts.json) {
       console.log(JSON.stringify(result.data, null, 2));
@@ -20100,43 +20123,43 @@ function registerContextCommand(program2) {
   context.command("clear").description("\uCEE8\uD14D\uC2A4\uD2B8 \uD574\uC81C").action(async () => {
     const result = await executeContextClear();
     if (!result.success) {
-      console.error(chalk3.red(`\uC624\uB958: ${result.error.message}`));
-      process.exit(1);
+      error(getResultErrorMessage(result.error));
+      process.exit(ExitCode.GENERAL_ERROR);
     }
-    console.log(chalk3.green("\uCEE8\uD14D\uC2A4\uD2B8\uAC00 \uD574\uC81C\uB418\uC5C8\uC2B5\uB2C8\uB2E4."));
+    success2("\uCEE8\uD14D\uC2A4\uD2B8\uAC00 \uD574\uC81C\uB418\uC5C8\uC2B5\uB2C8\uB2E4.");
   });
   context.command("add").description("\uCEE8\uD14D\uC2A4\uD2B8\uC5D0 \uB3C4\uBA54\uC778 \uCD94\uAC00").argument("<domain>", "\uCD94\uAC00\uD560 \uB3C4\uBA54\uC778 ID").option("--no-include-deps", "\uC758\uC874\uC131 \uB3C4\uBA54\uC778 \uC790\uB3D9 \uD3EC\uD568 \uBE44\uD65C\uC131\uD654").action(async (domain, opts) => {
     const result = await executeContextAdd(domain, { includeDeps: opts.includeDeps });
     if (!result.success) {
-      console.error(chalk3.red(`\uC624\uB958: ${result.error.message}`));
-      process.exit(1);
+      error(getResultErrorMessage(result.error));
+      process.exit(ExitCode.GENERAL_ERROR);
     }
-    console.log(chalk3.green(`\uB3C4\uBA54\uC778 "${domain}"\uC774(\uAC00) \uCEE8\uD14D\uC2A4\uD2B8\uC5D0 \uCD94\uAC00\uB418\uC5C8\uC2B5\uB2C8\uB2E4.`));
+    success2(`\uB3C4\uBA54\uC778 "${domain}"\uC774(\uAC00) \uCEE8\uD14D\uC2A4\uD2B8\uC5D0 \uCD94\uAC00\uB418\uC5C8\uC2B5\uB2C8\uB2E4.`);
     console.log("");
     console.log(formatContextInfo(result.data));
   });
   context.command("remove").alias("rm").description("\uCEE8\uD14D\uC2A4\uD2B8\uC5D0\uC11C \uB3C4\uBA54\uC778 \uC81C\uAC70").argument("<domain>", "\uC81C\uAC70\uD560 \uB3C4\uBA54\uC778 ID").action(async (domain) => {
     const result = await executeContextRemove(domain);
     if (!result.success) {
-      console.error(chalk3.red(`\uC624\uB958: ${result.error.message}`));
-      process.exit(1);
+      error(getResultErrorMessage(result.error));
+      process.exit(ExitCode.GENERAL_ERROR);
     }
-    console.log(chalk3.green(`\uB3C4\uBA54\uC778 "${domain}"\uC774(\uAC00) \uCEE8\uD14D\uC2A4\uD2B8\uC5D0\uC11C \uC81C\uAC70\uB418\uC5C8\uC2B5\uB2C8\uB2E4.`));
+    success2(`\uB3C4\uBA54\uC778 "${domain}"\uC774(\uAC00) \uCEE8\uD14D\uC2A4\uD2B8\uC5D0\uC11C \uC81C\uAC70\uB418\uC5C8\uC2B5\uB2C8\uB2E4.`);
     console.log("");
     console.log(formatContextInfo(result.data));
   });
   context.command("specs").description("\uCEE8\uD14D\uC2A4\uD2B8\uC5D0 \uD3EC\uD568\uB41C \uC2A4\uD399 \uBAA9\uB85D").option("--json", "JSON \uD615\uC2DD\uC73C\uB85C \uCD9C\uB825").action(async (opts) => {
     const result = await executeContextSpecs(opts);
     if (!result.success) {
-      console.error(chalk3.red(`\uC624\uB958: ${result.error.message}`));
-      process.exit(1);
+      error(getResultErrorMessage(result.error));
+      process.exit(ExitCode.GENERAL_ERROR);
     }
     if (opts.json) {
       console.log(JSON.stringify(result.data, null, 2));
     } else {
       const { active, readOnly } = result.data;
       if (active.length === 0 && readOnly.length === 0) {
-        console.log(chalk3.yellow("\uCEE8\uD14D\uC2A4\uD2B8\uC5D0 \uC2A4\uD399\uC774 \uC5C6\uC2B5\uB2C8\uB2E4."));
+        warn("\uCEE8\uD14D\uC2A4\uD2B8\uC5D0 \uC2A4\uD399\uC774 \uC5C6\uC2B5\uB2C8\uB2E4.");
         return;
       }
       console.log(chalk3.bold("\uCEE8\uD14D\uC2A4\uD2B8 \uC2A4\uD399 \uBAA9\uB85D:"));
@@ -20159,8 +20182,8 @@ function registerContextCommand(program2) {
   context.action(async () => {
     const result = await executeContextShow();
     if (!result.success) {
-      console.error(chalk3.red(`\uC624\uB958: ${result.error.message}`));
-      process.exit(1);
+      error(getResultErrorMessage(result.error));
+      process.exit(ExitCode.GENERAL_ERROR);
     }
     console.log(formatContextInfo(result.data));
   });
@@ -20171,6 +20194,7 @@ init_fs();
 import path57 from "path";
 import chalk10 from "chalk";
 init_errors();
+init_types();
 
 // src/integrations/serena/types.ts
 var SymbolKindNames = {
@@ -20801,12 +20825,12 @@ function inferDomainsFromDirectories(directories, files) {
   }).sort(([, a], [, b]) => b.files.length - a.files.length).slice(0, 10);
   const totalFiles = files.length;
   return sortedDomains.map(([name, info2]) => {
+    const estimatedSymbolCount = info2.files.length * 5;
     const domain = {
       name,
       path: info2.path,
       fileCount: info2.files.length,
-      symbolCount: 0
-      // TODO: 심볼 분석 시 업데이트
+      symbolCount: estimatedSymbolCount
     };
     return {
       ...domain,
@@ -22896,16 +22920,12 @@ var aiAssistant = new AIAssistant();
 
 // src/cli/commands/reverse.ts
 import { promises as fs38 } from "fs";
-async function handleScan(targetPath, options) {
-  const sddRoot = await findSddRoot();
+async function executeScanCommand(targetPath, options, projectRoot) {
+  const sddRoot = projectRoot || await findSddRoot();
   if (!sddRoot) {
-    error("SDD \uD504\uB85C\uC81D\uD2B8\uB97C \uCC3E\uC744 \uC218 \uC5C6\uC2B5\uB2C8\uB2E4. `sdd init`\uC744 \uBA3C\uC800 \uC2E4\uD589\uD558\uC138\uC694.");
-    process.exit(ExitCode.GENERAL_ERROR);
+    return failure(new Error("SDD \uD504\uB85C\uC81D\uD2B8\uB97C \uCC3E\uC744 \uC218 \uC5C6\uC2B5\uB2C8\uB2E4. `sdd init`\uC744 \uBA3C\uC800 \uC2E4\uD589\uD558\uC138\uC694."));
   }
   const scanPath2 = targetPath ? path57.resolve(targetPath) : sddRoot;
-  if (!options.quiet) {
-    info(`\uC2A4\uCE94 \uC911: ${scanPath2}`);
-  }
   const scanResult = await scanProject(scanPath2, {
     depth: options.depth,
     include: options.include,
@@ -22913,15 +22933,51 @@ async function handleScan(targetPath, options) {
     language: options.language
   });
   if (!scanResult.success) {
-    error(scanResult.error.message);
-    process.exit(ExitCode.GENERAL_ERROR);
+    return failure(scanResult.error);
   }
   const result = scanResult.data;
   const sddPath = path57.join(sddRoot, ".sdd");
-  const metaResult = await addScanToMeta(sddPath, result);
-  if (!metaResult.success && !options.quiet) {
-    warn("\uC2A4\uCE94 \uBA54\uD0C0\uB370\uC774\uD130 \uC800\uC7A5 \uC2E4\uD328");
+  await addScanToMeta(sddPath, result);
+  let domainsCreated = 0;
+  let domainsSkipped = 0;
+  const shouldCreateDomains = options.createDomains !== false;
+  if (shouldCreateDomains && result.summary.suggestedDomains.length > 0) {
+    const domainService = createDomainService(sddRoot);
+    const existingDomainsResult = await domainService.list();
+    const existingDomainIds = existingDomainsResult.success ? existingDomainsResult.data.map((d) => d.id) : [];
+    for (const suggested of result.summary.suggestedDomains) {
+      if (existingDomainIds.includes(suggested.name)) {
+        domainsSkipped++;
+        continue;
+      }
+      const createResult = await domainService.create(suggested.name, {
+        description: `${suggested.name} \uB3C4\uBA54\uC778 (reverse scan\uC73C\uB85C \uC790\uB3D9 \uC0DD\uC131)`,
+        path: suggested.path
+      });
+      if (createResult.success) {
+        domainsCreated++;
+      }
+    }
   }
+  return success({
+    result,
+    sddRoot,
+    sddPath,
+    domainsCreated,
+    domainsSkipped
+  });
+}
+async function handleScan(targetPath, options) {
+  if (!options.quiet) {
+    const scanPath2 = targetPath ? path57.resolve(targetPath) : process.cwd();
+    info(`\uC2A4\uCE94 \uC911: ${scanPath2}`);
+  }
+  const commandResult = await executeScanCommand(targetPath, options);
+  if (!commandResult.success) {
+    error(commandResult.error.message);
+    process.exit(ExitCode.GENERAL_ERROR);
+  }
+  const { result, sddRoot, sddPath, domainsCreated, domainsSkipped } = commandResult.data;
   if (options.compare) {
     const lastScan = await getLastScan(sddPath);
     if (lastScan) {
@@ -22947,37 +23003,16 @@ async function handleScan(targetPath, options) {
       error(`\uACB0\uACFC \uC800\uC7A5 \uC2E4\uD328: ${error2}`);
     }
   }
-  const shouldCreateDomains = options.createDomains !== false;
-  if (shouldCreateDomains && result.summary.suggestedDomains.length > 0) {
-    const domainService = createDomainService(sddRoot);
-    const existingDomainsResult = await domainService.list();
-    const existingDomainIds = existingDomainsResult.success ? existingDomainsResult.data.map((d) => d.id) : [];
-    let createdCount = 0;
-    let skippedCount = 0;
-    for (const suggested of result.summary.suggestedDomains) {
-      if (existingDomainIds.includes(suggested.name)) {
-        skippedCount++;
-        continue;
-      }
-      const createResult = await domainService.create(suggested.name, {
-        description: `${suggested.name} \uB3C4\uBA54\uC778 (reverse scan\uC73C\uB85C \uC790\uB3D9 \uC0DD\uC131)`,
-        path: suggested.path
-      });
-      if (createResult.success) {
-        createdCount++;
-      }
+  if (!options.quiet && (domainsCreated > 0 || domainsSkipped > 0)) {
+    console.log("");
+    console.log(chalk10.bold("\u{1F4C1} \uB3C4\uBA54\uC778 \uC790\uB3D9 \uC0DD\uC131:"));
+    if (domainsCreated > 0) {
+      console.log(chalk10.green(`   \u2705 ${domainsCreated}\uAC1C \uB3C4\uBA54\uC778 \uC0DD\uC131\uB428`));
     }
-    if (!options.quiet && (createdCount > 0 || skippedCount > 0)) {
-      console.log("");
-      console.log(chalk10.bold("\u{1F4C1} \uB3C4\uBA54\uC778 \uC790\uB3D9 \uC0DD\uC131:"));
-      if (createdCount > 0) {
-        console.log(chalk10.green(`   \u2705 ${createdCount}\uAC1C \uB3C4\uBA54\uC778 \uC0DD\uC131\uB428`));
-      }
-      if (skippedCount > 0) {
-        console.log(chalk10.dim(`   \u23ED\uFE0F  ${skippedCount}\uAC1C \uB3C4\uBA54\uC778 \uC774\uBBF8 \uC874\uC7AC (\uAC74\uB108\uB700)`));
-      }
-      console.log("");
+    if (domainsSkipped > 0) {
+      console.log(chalk10.dim(`   \u23ED\uFE0F  ${domainsSkipped}\uAC1C \uB3C4\uBA54\uC778 \uC774\uBBF8 \uC874\uC7AC (\uAC74\uB108\uB700)`));
     }
+    console.log("");
   }
   if (!options.skipSerenaCheck) {
     const serenaCheck = await ensureSerenaAvailable("scan", { skipSerenaCheck: true, quiet: true });
@@ -22992,14 +23027,52 @@ async function handleScan(targetPath, options) {
     console.log("");
   }
 }
-async function handleExtract(targetPath, options) {
-  const sddRoot = await findSddRoot();
+async function executeExtractCommand(targetPath, options, onProgress, projectRoot) {
+  const sddRoot = projectRoot || await findSddRoot();
   if (!sddRoot) {
-    error("SDD \uD504\uB85C\uC81D\uD2B8\uB97C \uCC3E\uC744 \uC218 \uC5C6\uC2B5\uB2C8\uB2E4. `sdd init`\uC744 \uBA3C\uC800 \uC2E4\uD589\uD558\uC138\uC694.");
-    process.exit(ExitCode.GENERAL_ERROR);
+    return failure(new Error("SDD \uD504\uB85C\uC81D\uD2B8\uB97C \uCC3E\uC744 \uC218 \uC5C6\uC2B5\uB2C8\uB2E4. `sdd init`\uC744 \uBA3C\uC800 \uC2E4\uD589\uD558\uC138\uC694."));
   }
   const extractPath = targetPath ? path57.resolve(targetPath) : sddRoot;
+  const scanResult = await scanProject(extractPath, { depth: 5 });
+  if (!scanResult.success) {
+    return failure(scanResult.error);
+  }
+  const extractResult = await extractSpecs(scanResult.data, {
+    depth: options.depth || "medium",
+    ai: options.ai,
+    domain: options.domain
+  }, onProgress);
+  if (!extractResult.success) {
+    return failure(extractResult.error);
+  }
+  const result = extractResult.data;
+  if (result.specs.length === 0) {
+    return success({
+      specs: [],
+      symbolCount: result.symbolCount,
+      skippedCount: result.skippedCount,
+      overallConfidence: result.overallConfidence
+    });
+  }
+  const sddPath = path57.join(sddRoot, ".sdd");
+  const saveResult = await saveExtractedSpecs(sddPath, result, "json");
+  if (!saveResult.success) {
+    return failure(saveResult.error);
+  }
+  await updateExtractionStatus(sddPath, {
+    extractedCount: result.specs.length,
+    pendingReviewCount: result.specs.length
+  });
+  return success({
+    specs: result.specs.map((s) => ({ id: s.id, confidence: s.confidence })),
+    symbolCount: result.symbolCount,
+    skippedCount: result.skippedCount,
+    overallConfidence: result.overallConfidence
+  });
+}
+async function handleExtract(targetPath, options) {
   if (!options.quiet) {
+    const extractPath = targetPath ? path57.resolve(targetPath) : process.cwd();
     info(`\uCD94\uCD9C \uC911: ${extractPath}`);
     if (options.depth) {
       info(`\uAE4A\uC774: ${options.depth}`);
@@ -23008,45 +23081,24 @@ async function handleExtract(targetPath, options) {
       info("AI \uCD94\uB860 \uD65C\uC131\uD654\uB428");
     }
   }
-  const scanResult = await scanProject(extractPath, {
-    depth: 5
-  });
-  if (!scanResult.success) {
-    error(scanResult.error.message);
-    process.exit(ExitCode.GENERAL_ERROR);
-  }
-  const extractResult = await extractSpecs(scanResult.data, {
-    depth: options.depth || "medium",
-    ai: options.ai,
-    domain: options.domain
-  }, (progress) => {
+  const commandResult = await executeExtractCommand(targetPath, options, (progress) => {
     if (!options.quiet) {
       process.stdout.write(`\r   \uCC98\uB9AC \uC911: ${progress.processedSymbols}/${progress.totalSymbols} \uC2EC\uBCFC, ${progress.specsGenerated} \uC2A4\uD399 \uC0DD\uC131\uB428`);
     }
   });
-  if (!extractResult.success) {
-    error(extractResult.error.message);
+  if (!commandResult.success) {
+    error(commandResult.error.message);
     process.exit(ExitCode.GENERAL_ERROR);
   }
   if (!options.quiet) {
     console.log("");
   }
-  const result = extractResult.data;
+  const result = commandResult.data;
   if (result.specs.length === 0) {
     warn("\uCD94\uCD9C\uB41C \uC2A4\uD399\uC774 \uC5C6\uC2B5\uB2C8\uB2E4. \uD504\uB85C\uC81D\uD2B8\uC5D0 \uBD84\uC11D \uAC00\uB2A5\uD55C \uC2EC\uBCFC\uC774 \uC5C6\uAC70\uB098 Serena MCP\uAC00 \uD544\uC694\uD569\uB2C8\uB2E4.");
     console.log(chalk10.dim("\n\u{1F4A1} Serena MCP\uB97C \uC5F0\uACB0\uD558\uBA74 \uC2EC\uBCFC \uC218\uC900 \uBD84\uC11D\uC774 \uAC00\uB2A5\uD569\uB2C8\uB2E4."));
     return;
   }
-  const sddPath = path57.join(sddRoot, ".sdd");
-  const saveResult = await saveExtractedSpecs(sddPath, result, "json");
-  if (!saveResult.success) {
-    error(saveResult.error.message);
-    process.exit(ExitCode.GENERAL_ERROR);
-  }
-  await updateExtractionStatus(sddPath, {
-    extractedCount: result.specs.length,
-    pendingReviewCount: result.specs.length
-  });
   console.log("");
   console.log(chalk10.bold("\u{1F4C4} \uC2A4\uD399 \uCD94\uCD9C \uC644\uB8CC"));
   console.log("\u2500".repeat(40));
@@ -23067,64 +23119,96 @@ async function handleExtract(targetPath, options) {
   console.log("   sdd reverse review    # \uCD94\uCD9C\uB41C \uC2A4\uD399 \uB9AC\uBDF0");
   console.log("");
 }
-async function handleReview(specId, options) {
-  const sddRoot = await findSddRoot();
+async function executeReviewCommand(specId, options, projectRoot) {
+  const sddRoot = projectRoot || await findSddRoot();
   if (!sddRoot) {
-    error("SDD \uD504\uB85C\uC81D\uD2B8\uB97C \uCC3E\uC744 \uC218 \uC5C6\uC2B5\uB2C8\uB2E4.");
-    process.exit(ExitCode.GENERAL_ERROR);
+    return failure(new Error("SDD \uD504\uB85C\uC81D\uD2B8\uB97C \uCC3E\uC744 \uC218 \uC5C6\uC2B5\uB2C8\uB2E4."));
   }
   const sddPath = path57.join(sddRoot, ".sdd");
   const draftsPath = path57.join(sddPath, ".reverse-drafts");
   if (!await fileExists(draftsPath)) {
-    warn("\uCD94\uCD9C\uB41C \uC2A4\uD399\uC774 \uC5C6\uC2B5\uB2C8\uB2E4. `sdd reverse extract`\uB97C \uBA3C\uC800 \uC2E4\uD589\uD558\uC138\uC694.");
-    return;
+    return success({ action: "no_drafts" });
   }
   const loadResult = await loadReviewList(sddPath);
   if (!loadResult.success) {
-    error(loadResult.error.message);
-    process.exit(ExitCode.GENERAL_ERROR);
+    return failure(loadResult.error);
   }
   const items = loadResult.data;
   if (items.length === 0) {
-    warn("\uB9AC\uBDF0\uD560 \uC2A4\uD399\uC774 \uC5C6\uC2B5\uB2C8\uB2E4.");
-    return;
+    return success({ action: "empty" });
   }
   if (specId) {
     const item = items.find((i) => i.specId === specId || i.specId.endsWith(`/${specId}`));
     if (!item) {
-      error(`\uC2A4\uD399\uC744 \uCC3E\uC744 \uC218 \uC5C6\uC2B5\uB2C8\uB2E4: ${specId}`);
-      process.exit(ExitCode.GENERAL_ERROR);
+      return failure(new Error(`\uC2A4\uD399\uC744 \uCC3E\uC744 \uC218 \uC5C6\uC2B5\uB2C8\uB2E4: ${specId}`));
     }
     if (options.approve) {
       const result = await approveSpec(sddPath, item.specId);
-      if (result.success) {
-        success2(`\uC2B9\uC778\uB428: ${item.specId}`);
-        console.log("");
-        console.log(chalk10.bold("\u{1F4A1} \uB2E4\uC74C \uB2E8\uACC4:"));
-        console.log("   sdd reverse finalize --all    # \uC2B9\uC778\uB41C \uC2A4\uD399 \uD655\uC815");
-      } else {
-        error(result.error.message);
-        process.exit(ExitCode.GENERAL_ERROR);
+      if (!result.success) {
+        return failure(result.error);
       }
-      return;
+      return success({ action: "approved", specId: item.specId });
     }
     if (options.reject) {
       const result = await rejectSpec(sddPath, item.specId, options.reason || "\uC0AC\uC6A9\uC790\uC5D0 \uC758\uD574 \uAC70\uBD80\uB428");
-      if (result.success) {
-        success2(`\uAC70\uBD80\uB428: ${item.specId}`);
-      } else {
-        error(result.error.message);
-        process.exit(ExitCode.GENERAL_ERROR);
+      if (!result.success) {
+        return failure(result.error);
       }
-      return;
+      return success({ action: "rejected", specId: item.specId });
     }
-    console.log(formatSpecDetail(item));
-    console.log(chalk10.bold("\u{1F4A1} \uC791\uC5C5:"));
-    console.log(`   sdd reverse review ${specId} --approve    # \uC2B9\uC778`);
-    console.log(`   sdd reverse review ${specId} --reject     # \uAC70\uBD80`);
-    console.log("");
-    return;
+    return success({ action: "detail", specId: item.specId, sddPath });
+  }
+  return success({ action: "list", sddPath });
+}
+async function handleReview(specId, options) {
+  const commandResult = await executeReviewCommand(specId, options);
+  if (!commandResult.success) {
+    error(commandResult.error.message);
+    process.exit(ExitCode.GENERAL_ERROR);
   }
+  const result = commandResult.data;
+  switch (result.action) {
+    case "no_drafts":
+      warn("\uCD94\uCD9C\uB41C \uC2A4\uD399\uC774 \uC5C6\uC2B5\uB2C8\uB2E4. `sdd reverse extract`\uB97C \uBA3C\uC800 \uC2E4\uD589\uD558\uC138\uC694.");
+      return;
+    case "empty":
+      warn("\uB9AC\uBDF0\uD560 \uC2A4\uD399\uC774 \uC5C6\uC2B5\uB2C8\uB2E4.");
+      return;
+    case "approved":
+      success2(`\uC2B9\uC778\uB428: ${result.specId}`);
+      console.log("");
+      console.log(chalk10.bold("\u{1F4A1} \uB2E4\uC74C \uB2E8\uACC4:"));
+      console.log("   sdd reverse finalize --all    # \uC2B9\uC778\uB41C \uC2A4\uD399 \uD655\uC815");
+      return;
+    case "rejected":
+      success2(`\uAC70\uBD80\uB428: ${result.specId}`);
+      return;
+    case "detail":
+      if (result.sddPath && result.specId) {
+        const detailLoadResult = await loadReviewList(result.sddPath);
+        if (detailLoadResult.success) {
+          const detailItem = detailLoadResult.data.find(
+            (i) => i.specId === result.specId || i.specId.endsWith(`/${result.specId}`)
+          );
+          if (detailItem) {
+            console.log(formatSpecDetail(detailItem));
+            console.log(chalk10.bold("\u{1F4A1} \uC791\uC5C5:"));
+            console.log(`   sdd reverse review ${result.specId} --approve    # \uC2B9\uC778`);
+            console.log(`   sdd reverse review ${result.specId} --reject     # \uAC70\uBD80`);
+            console.log("");
+          }
+        }
+      }
+      return;
+    case "list":
+      break;
+  }
+  const sddRoot = await findSddRoot();
+  if (!sddRoot) return;
+  const sddPath = path57.join(sddRoot, ".sdd");
+  const loadResult = await loadReviewList(sddPath);
+  if (!loadResult.success) return;
+  const items = loadResult.data;
   console.log(formatReviewList(items));
   const pending = items.filter((i) => i.status === "pending");
   if (pending.length > 0) {
@@ -23139,55 +23223,78 @@ async function handleReview(specId, options) {
     console.log("");
   }
 }
-async function handleFinalize(specId, options) {
-  const sddRoot = await findSddRoot();
+async function executeFinalizeCommand(specId, options, projectRoot) {
+  const sddRoot = projectRoot || await findSddRoot();
   if (!sddRoot) {
-    error("SDD \uD504\uB85C\uC81D\uD2B8\uB97C \uCC3E\uC744 \uC218 \uC5C6\uC2B5\uB2C8\uB2E4.");
-    process.exit(ExitCode.GENERAL_ERROR);
-  }
-  if (!options.quiet) {
-    if (specId) {
-      info(`\uD655\uC815 \uC911: ${specId}`);
-    } else if (options.all) {
-      info("\uC2B9\uC778\uB41C \uBAA8\uB4E0 \uC2A4\uD399 \uD655\uC815 \uC911");
-    } else if (options.domain) {
-      info(`\uB3C4\uBA54\uC778 \uD655\uC815 \uC911: ${options.domain}`);
-    }
+    return failure(new Error("SDD \uD504\uB85C\uC81D\uD2B8\uB97C \uCC3E\uC744 \uC218 \uC5C6\uC2B5\uB2C8\uB2E4."));
   }
-  let result;
   if (specId) {
     const finalizeResult = await finalizeById(sddRoot, specId);
     if (!finalizeResult.success) {
-      error(finalizeResult.error.message);
-      process.exit(ExitCode.GENERAL_ERROR);
+      return failure(finalizeResult.error);
     }
-    result = {
-      finalized: [finalizeResult.data],
-      skipped: [],
-      errors: []
-    };
-  } else if (options.domain) {
+    return success({
+      action: "single",
+      data: {
+        finalized: [finalizeResult.data],
+        skipped: [],
+        errors: []
+      }
+    });
+  }
+  if (options.domain) {
     const finalizeResult = await finalizeDomain(sddRoot, options.domain);
     if (!finalizeResult.success) {
-      error(finalizeResult.error.message);
-      process.exit(ExitCode.GENERAL_ERROR);
+      return failure(finalizeResult.error);
     }
-    result = finalizeResult.data;
-  } else if (options.all) {
+    return success({
+      action: "domain",
+      data: finalizeResult.data
+    });
+  }
+  if (options.all) {
     const finalizeResult = await finalizeAllApproved(sddRoot);
     if (!finalizeResult.success) {
-      error(finalizeResult.error.message);
-      process.exit(ExitCode.GENERAL_ERROR);
+      return failure(finalizeResult.error);
     }
-    result = finalizeResult.data;
-  } else {
-    warn("\uD655\uC815\uD560 \uB300\uC0C1\uC744 \uC9C0\uC815\uD558\uC138\uC694:");
-    console.log("   sdd reverse finalize <spec-id>     # \uD2B9\uC815 \uC2A4\uD399");
-    console.log("   sdd reverse finalize --all         # \uBAA8\uB4E0 \uC2B9\uC778 \uC2A4\uD399");
-    console.log("   sdd reverse finalize -d <domain>   # \uD2B9\uC815 \uB3C4\uBA54\uC778");
-    return;
+    return success({
+      action: "all",
+      data: finalizeResult.data
+    });
+  }
+  return success({ action: "no_target" });
+}
+async function handleFinalize(specId, options) {
+  if (!options.quiet) {
+    if (specId) {
+      info(`\uD655\uC815 \uC911: ${specId}`);
+    } else if (options.all) {
+      info("\uC2B9\uC778\uB41C \uBAA8\uB4E0 \uC2A4\uD399 \uD655\uC815 \uC911");
+    } else if (options.domain) {
+      info(`\uB3C4\uBA54\uC778 \uD655\uC815 \uC911: ${options.domain}`);
+    }
+  }
+  const commandResult = await executeFinalizeCommand(specId, options);
+  if (!commandResult.success) {
+    error(commandResult.error.message);
+    process.exit(ExitCode.GENERAL_ERROR);
+  }
+  const result = commandResult.data;
+  switch (result.action) {
+    case "single":
+    case "domain":
+    case "all":
+      if (result.data) {
+        console.log(formatFinalizeResult(result.data));
+      }
+      return;
+    case "no_target":
+      warn("\uD655\uC815\uD560 \uB300\uC0C1\uC744 \uC9C0\uC815\uD558\uC138\uC694:");
+      console.log("   sdd reverse finalize <spec-id>     # \uD2B9\uC815 \uC2A4\uD399");
+      console.log("   sdd reverse finalize --all         # \uBAA8\uB4E0 \uC2B9\uC778 \uC2A4\uD399");
+      console.log("   sdd reverse finalize -d <domain>   # \uD2B9\uC815 \uB3C4\uBA54\uC778");
+      return;
   }
-  console.log(formatFinalizeResult(result));
 }
 function showReverseHelp() {
   console.log(`
@@ -23222,9 +23329,20 @@ ${chalk10.bold("\uC6CC\uD06C\uD50C\uB85C\uC6B0:")}
   4. finalize \u2192 \uC815\uC2DD \uC2A4\uD399\uC73C\uB85C \uBCC0\uD658
 `);
 }
-async function handleCheckSerena() {
-  const result = await ensureSerenaAvailable("check", { quiet: false });
+async function executeCheckSerenaCommand() {
+  const result = await ensureSerenaAvailable("check", { quiet: true });
   if (result.success) {
+    return success({ available: true });
+  }
+  return success({ available: false });
+}
+async function handleCheckSerena() {
+  const commandResult = await executeCheckSerenaCommand();
+  if (!commandResult.success) {
+    console.log(createInstallGuide());
+    process.exit(ExitCode.GENERAL_ERROR);
+  }
+  if (commandResult.data.available) {
     console.log(chalk10.green("\u2705 Serena MCP \uC0AC\uC6A9 \uAC00\uB2A5"));
   } else {
     console.log(createInstallGuide());