@agentforge/cli 0.16.3 → 0.16.4

package/dist/index.js

@@ -75,6 +75,30 @@ var Logger = class {
   }
 };
 var logger = new Logger();
+
+// src/utils/command-errors.ts
+function getErrorMessage(error) {
+  if (error instanceof Error) {
+    return error.message;
+  }
+  if (typeof error === "string") {
+    return error;
+  }
+  if (typeof error === "object" && error !== null && "message" in error && typeof error.message === "string") {
+    return error.message;
+  }
+  return String(error);
+}
+function exitWithCommandError(error, options = {}) {
+  if (options.spinnerFailureText) {
+    logger.failSpinner(options.spinnerFailureText);
+  }
+  if (options.logError !== false) {
+    const message = options.message ?? getErrorMessage(error);
+    logger.error(options.prefix ? `${options.prefix}: ${message}` : message);
+  }
+  process.exit(1);
+}
 async function promptProjectSetup(defaults = {}) {
   return inquirer.prompt([
     {
@@ -434,13 +458,11 @@ async function createCommand(projectName, options) {
   try {
     logger.header("\u{1F680} Create AgentForge Project");
     if (!projectName) {
-      logger.error("Project name is required");
-      process.exit(1);
+      return exitWithCommandError("Project name is required");
     }
     const targetPath = path12.join(process.cwd(), projectName);
     if (!await isEmptyDir(targetPath)) {
-      logger.error(`Directory ${projectName} already exists and is not empty`);
-      process.exit(1);
+      return exitWithCommandError(`Directory ${projectName} already exists and is not empty`);
     }
     const answers = await promptProjectSetup({
       projectName,
@@ -483,7 +505,7 @@ async function createCommand(projectName, options) {
       try {
         await installDependencies(targetPath, answers.packageManager);
         logger.succeedSpinner("Dependencies installed");
-      } catch (error) {
+      } catch {
         logger.failSpinner("Failed to install dependencies");
         logger.warn("You can install them manually later");
       }
@@ -494,7 +516,7 @@ async function createCommand(projectName, options) {
         await initGitRepository(targetPath);
         await createInitialCommit(targetPath);
         logger.succeedSpinner("Git repository initialized");
-      } catch (error) {
+      } catch {
         logger.failSpinner("Failed to initialize git");
         logger.warn("You can initialize it manually later");
       }
@@ -510,8 +532,7 @@ async function createCommand(projectName, options) {
     logger.newLine();
     logger.info("Happy coding! \u{1F389}");
   } catch (error) {
-    logger.error(`Failed to create project: ${error.message}`);
-    process.exit(1);
+    return exitWithCommandError(error, { prefix: "Failed to create project" });
   }
 }
 
@@ -531,9 +552,9 @@ async function devCommand(options) {
     await runScript(cwd, "dev", packageManager);
     logger.succeedSpinner("Development server started");
   } catch (error) {
-    logger.failSpinner("Failed to start development server");
-    logger.error(error.message);
-    process.exit(1);
+    return exitWithCommandError(error, {
+      spinnerFailureText: "Failed to start development server"
+    });
   }
 }
 
@@ -559,9 +580,7 @@ async function buildCommand(options) {
     logger.newLine();
     logger.success("\u2728 Production build ready!");
   } catch (error) {
-    logger.failSpinner("Build failed");
-    logger.error(error.message);
-    process.exit(1);
+    return exitWithCommandError(error, { spinnerFailureText: "Build failed" });
   }
 }
 
@@ -587,9 +606,7 @@ async function testCommand(options) {
     await runScript(cwd, script, packageManager);
     logger.succeedSpinner("Tests completed");
   } catch (error) {
-    logger.failSpinner("Tests failed");
-    logger.error(error.message);
-    process.exit(1);
+    return exitWithCommandError(error, { spinnerFailureText: "Tests failed" });
   }
 }
 
@@ -606,7 +623,7 @@ async function lintCommand(options) {
     try {
       await runScript(cwd, options.fix ? "lint:fix" : "lint", packageManager);
       logger.succeedSpinner("Linting completed");
-    } catch (error) {
+    } catch {
       logger.failSpinner("Linting found issues");
       if (!options.fix) {
         logger.info("Run with --fix to automatically fix issues");
@@ -617,15 +634,14 @@ async function lintCommand(options) {
       try {
         await runScript(cwd, "format", packageManager);
         logger.succeedSpinner("Formatting completed");
-      } catch (error) {
+      } catch {
         logger.failSpinner("Formatting failed");
       }
     }
     logger.newLine();
     logger.success("\u2728 Code quality check completed!");
   } catch (error) {
-    logger.error(error.message);
-    process.exit(1);
+    return exitWithCommandError(error);
   }
 }
 async function agentCreateCommand(name, options) {
@@ -666,8 +682,7 @@ async function agentCreateCommand(name, options) {
       answers.generateTests ? `Run ${chalk4.cyan(`pnpm test tests/agents/${answers.name}.test.ts`)} to test your agent` : ""
     ].filter(Boolean));
   } catch (error) {
-    logger.error(`Failed to create agent: ${error.message}`);
-    process.exit(1);
+    return exitWithCommandError(error, { prefix: "Failed to create agent" });
   }
 }
 function generateAgentContent(name, pattern, description) {
@@ -850,8 +865,7 @@ async function agentCreateReusableCommand(name, options) {
     logger.newLine();
     logger.info(chalk4.gray("\u{1F4A1} Tip: See examples/vertical-agents/ for reference implementations"));
   } catch (error) {
-    logger.error(`Failed to create reusable agent: ${error.message}`);
-    process.exit(1);
+    return exitWithCommandError(error, { prefix: "Failed to create reusable agent" });
   }
 }
 function kebabToPascal(str) {
@@ -899,8 +913,7 @@ async function agentListCommand(options) {
       logger.info(`Use ${chalk4.cyan("--verbose")} for more details`);
     }
   } catch (error) {
-    logger.error(`Failed to list agents: ${error.message}`);
-    process.exit(1);
+    return exitWithCommandError(error, { prefix: "Failed to list agents" });
   }
 }
 function extractPattern(content) {
@@ -924,7 +937,7 @@ async function agentTestCommand(name, options) {
     if (!await pathExists(testFile)) {
       logger.error(`Test file not found: ${testFile}`);
       logger.info(`Create tests with: ${chalk4.cyan(`agentforge agent:create ${name} --test`)}`);
-      process.exit(1);
+      return exitWithCommandError(`Test file not found: ${testFile}`, { logError: false });
     }
     logger.info(`Testing agent: ${chalk4.cyan(name)}`);
     logger.info(`Watch mode: ${options.watch ? "Yes" : "No"}`);
@@ -936,9 +949,7 @@ async function agentTestCommand(name, options) {
     await runScript(cwd, testCommand2, packageManager);
     logger.succeedSpinner("Tests completed");
   } catch (error) {
-    logger.failSpinner("Tests failed");
-    logger.error(error.message);
-    process.exit(1);
+    return exitWithCommandError(error, { spinnerFailureText: "Tests failed" });
   }
 }
 async function agentDeployCommand(name, options) {
@@ -972,10 +983,11 @@ async function agentDeployCommand(name, options) {
     logger.newLine();
     logger.info(chalk4.dim("For detailed deployment guides, see:"));
     logger.info(chalk4.dim("https://tvscoundrel.github.io/agentforge/guide/advanced/deployment"));
-    process.exit(1);
+    return exitWithCommandError("Automated agent deployment is not yet implemented", {
+      logError: false
+    });
   } catch (error) {
-    logger.error(error.message);
-    process.exit(1);
+    return exitWithCommandError(error);
   }
 }
 async function toolCreateCommand(name, options) {
@@ -1016,8 +1028,7 @@ async function toolCreateCommand(name, options) {
     ];
     logger.list(nextSteps.filter(Boolean));
   } catch (error) {
-    logger.error(`Failed to create tool: ${error.message}`);
-    process.exit(1);
+    return exitWithCommandError(error, { prefix: "Failed to create tool" });
   }
 }
 function generateToolContent(name, category, description) {
@@ -1165,8 +1176,7 @@ async function toolListCommand(options) {
       logger.info(`Use ${chalk4.cyan("--verbose")} for more details`);
     }
   } catch (error) {
-    logger.error(`Failed to list tools: ${error.message}`);
-    process.exit(1);
+    return exitWithCommandError(error, { prefix: "Failed to list tools" });
   }
 }
 function extractCategory(content) {
@@ -1185,7 +1195,7 @@ async function toolTestCommand(name, options) {
     if (!await pathExists(testFile)) {
       logger.error(`Test file not found: ${testFile}`);
       logger.info(`Create tests with: ${chalk4.cyan(`agentforge tool:create ${name} --test`)}`);
-      process.exit(1);
+      return exitWithCommandError(`Test file not found: ${testFile}`, { logError: false });
     }
     logger.info(`Testing tool: ${chalk4.cyan(name)}`);
     logger.info(`Watch mode: ${options.watch ? "Yes" : "No"}`);
@@ -1197,9 +1207,7 @@ async function toolTestCommand(name, options) {
     await runScript(cwd, testCommand2, packageManager);
     logger.succeedSpinner("Tests completed");
   } catch (error) {
-    logger.failSpinner("Tests failed");
-    logger.error(error.message);
-    process.exit(1);
+    return exitWithCommandError(error, { spinnerFailureText: "Tests failed" });
   }
 }
 async function toolPublishCommand(name, options) {
@@ -1221,9 +1229,10 @@ async function toolPublishCommand(name, options) {
         await runScript(toolPath, "test", packageManager);
         logger.succeedSpinner("Tests passed");
       } catch (error) {
-        logger.failSpinner("Tests failed");
-        logger.error("Cannot publish tool with failing tests");
-        process.exit(1);
+        return exitWithCommandError(error, {
+          spinnerFailureText: "Tests failed",
+          message: "Cannot publish tool with failing tests"
+        });
       }
     } else {
       logger.info("\u26A0\uFE0F  Skipping tests (no test script found)");
@@ -1234,8 +1243,7 @@ async function toolPublishCommand(name, options) {
         await runScript(toolPath, "build", packageManager);
         logger.succeedSpinner("Build completed");
       } catch (error) {
-        logger.failSpinner("Build failed");
-        process.exit(1);
+        return exitWithCommandError(error, { spinnerFailureText: "Build failed" });
       }
     } else {
       logger.info("\u26A0\uFE0F  Skipping build (no build script found)");
@@ -1253,20 +1261,23 @@ async function toolPublishCommand(name, options) {
         logger.succeedSpinner("Published to npm");
       }
     } catch (error) {
+      const errorMessage = getErrorMessage(error);
       logger.failSpinner("Publishing failed");
-      if (error.message.includes("ENEEDAUTH") || error.message.includes("E401")) {
+      if (errorMessage.includes("ENEEDAUTH") || errorMessage.includes("E401")) {
         logger.error("Not authenticated with npm");
         logger.info("Run: npm login");
-      } else if (error.message.includes("E403")) {
+      } else if (errorMessage.includes("E403")) {
         logger.error("Permission denied - you may not have access to publish this package");
         logger.info("Check package name and npm organization permissions");
-      } else if (error.message.includes("EPUBLISHCONFLICT") || error.message.includes("E409")) {
+      } else if (errorMessage.includes("EPUBLISHCONFLICT") || errorMessage.includes("E409")) {
         logger.error("Version already published");
         logger.info("Update the version in package.json before publishing");
       } else {
-        logger.error(error.message);
+        logger.error(errorMessage);
       }
-      process.exit(1);
+      return exitWithCommandError(error, {
+        logError: false
+      });
     }
     logger.newLine();
     if (options.dryRun) {
@@ -1280,9 +1291,7 @@ async function toolPublishCommand(name, options) {
       logger.info("Users can now install with: npm install " + name);
     }
   } catch (error) {
-    logger.failSpinner("Publishing failed");
-    logger.error(error.message);
-    process.exit(1);
+    return exitWithCommandError(error, { spinnerFailureText: "Publishing failed" });
   }
 }
 async function resolveToolPath(name) {
@@ -1335,29 +1344,25 @@ async function resolveToolPath(name) {
     "Providing a path to the tool package directory",
     "Have the tool in a standard location (./tools/<name>, ./packages/<name>)"
   ]);
-  process.exit(1);
+  return exitWithCommandError(`Could not find tool package: ${name}`, { logError: false });
 }
 async function validateToolPath(toolPath, expectedName) {
   if (!await fs.pathExists(toolPath)) {
-    logger.error(`Tool directory not found: ${toolPath}`);
-    process.exit(1);
+    return exitWithCommandError(`Tool directory not found: ${toolPath}`);
   }
   const packageJsonPath = path12.join(toolPath, "package.json");
   if (!await fs.pathExists(packageJsonPath)) {
-    logger.error(`package.json not found in: ${toolPath}`);
     logger.info("Tool packages must have a package.json file");
-    process.exit(1);
+    return exitWithCommandError(`package.json not found in: ${toolPath}`);
   }
   let packageJson;
   try {
     packageJson = await fs.readJson(packageJsonPath);
   } catch (error) {
-    logger.error(`Failed to read package.json: ${error.message}`);
-    process.exit(1);
+    return exitWithCommandError(error, { prefix: "Failed to read package.json" });
   }
   if (!packageJson.name) {
-    logger.error('package.json must have a "name" field');
-    process.exit(1);
+    return exitWithCommandError('package.json must have a "name" field');
   }
   const packageName = packageJson.name;
   const nameMatches = packageName === expectedName || packageName === `@agentforge/${expectedName}` || packageName.endsWith(`/${expectedName}`);