refactor(dev): move CommandFailedError and MissingCodeDirectoryError to src/errors

All typed errors live in src/errors; the dev module was defining its own
inline.

Author: tejaskash

src/core/dev/codezip.test.ts

@@ -6,7 +6,8 @@ import { afterEach } from "bun:test";
 import type { DevLogLevel, DevServerHandle } from "../../handlers/project/dev/types";
 import type { ProjectRuntime } from "../../handlers/project/types";
 import { createSilentLogger } from "../../testing";
-import { CodeZipDevRunner, MissingCodeDirectoryError, entrypointToAsgiApp } from "./codezip";
+import { MissingCodeDirectoryError } from "../../errors";
+import { CodeZipDevRunner, entrypointToAsgiApp } from "./codezip";
 import type { SpawnServerInput } from "./process";
 
 const tempDirectories: string[] = [];

src/core/dev/codezip.ts

@@ -1,6 +1,6 @@
 import { existsSync } from "node:fs";
 import { join } from "node:path";
-import { AgentCoreCLIError, ERROR_SOURCE } from "../../errors";
+import { MissingCodeDirectoryError } from "../../errors";
 import type {
   DevRunner,
   DevServerHandle,
@@ -10,16 +10,6 @@ import type { Logger } from "../../logging";
 import { runCommand, type CommandRunner } from "./run";
 import { spawnServer, type SpawnServerInput } from "./process";
 
-/** Error raised when a runtime's code directory is missing from the project. */
-export class MissingCodeDirectoryError extends AgentCoreCLIError {
-  constructor(directory: string) {
-    super(`runtime code directory not found: ${directory}`, {
-      source: ERROR_SOURCE.USER,
-      meta: { directory },
-    });
-  }
-}
-
 type CodeZipDevRunnerConfig = {
   logger: Logger;
   /** Injectable process seams so tests never spawn uv or a real server. */

src/core/dev/index.ts

@@ -1,3 +1,3 @@
-export { CodeZipDevRunner, MissingCodeDirectoryError } from "./codezip";
+export { CodeZipDevRunner } from "./codezip";
 export { spawnServer } from "./process";
-export { runCommand, CommandFailedError, type CommandRunner } from "./run";
+export { runCommand, type CommandRunner } from "./run";

src/core/dev/run.ts

@@ -1,22 +1,9 @@
 import { spawn } from "node:child_process";
-import { AgentCoreCLIError, ERROR_SOURCE } from "../../errors";
+import { CommandFailedError } from "../../errors";
 
 // NOTE: shape-compatible with src/io/exec.ts from #1872; fold into that
 // module once it lands so the CLI has one CommandRunner.
 
-/** Error raised when a subprocess exits non-zero, carrying its captured output. */
-export class CommandFailedError extends AgentCoreCLIError {
-  constructor(command: string[], cwd: string, exitCode: number | null, output: string) {
-    super(
-      `'${command.join(" ")}' failed in ${cwd} (exit code ${exitCode ?? "unknown"}).\n\n${output.trim()}`,
-      {
-        source: ERROR_SOURCE.USER,
-        meta: { command, cwd, exitCode },
-      },
-    );
-  }
-}
-
 export type RunCommandOptions = {
   /** Working directory the command runs in. */
   cwd: string;

src/errors/errors.tsx

@@ -79,6 +79,29 @@ export class DeserializationError extends AgentCoreCLIError {
   }
 }
 
+/** Error raised when a subprocess exits non-zero, carrying its captured output. */
+export class CommandFailedError extends AgentCoreCLIError {
+  constructor(command: string[], cwd: string, exitCode: number | null, output: string) {
+    super(
+      `'${command.join(" ")}' failed in ${cwd} (exit code ${exitCode ?? "unknown"}).\n\n${output.trim()}`,
+      {
+        source: ERROR_SOURCE.USER,
+        meta: { command, cwd, exitCode },
+      },
+    );
+  }
+}
+
+/** Error raised when a runtime's code directory is missing from the project. */
+export class MissingCodeDirectoryError extends AgentCoreCLIError {
+  constructor(public readonly directory: string) {
+    super(`runtime code directory not found: ${directory}`, {
+      source: ERROR_SOURCE.USER,
+      meta: { directory },
+    });
+  }
+}
+
 /** Error raised when a command requires an AgentCore project and none encloses the working directory. */
 export class NoProjectError extends AgentCoreCLIError {
   constructor(searchPath: string, options?: Omit<AgentCoreCLIErrorOptions, "source">) {

src/errors/index.tsx

@@ -1,10 +1,12 @@
 export {
   AgentCoreCLIError,
+  CommandFailedError,
   DeserializationError,
   EmbeddedAssetNotFoundError,
   InputValidationError,
   InvalidEnvironmentError,
   InvalidProjectConfigError,
+  MissingCodeDirectoryError,
   NestedProjectError,
   NoProjectError,
   NotImplementedError,