fix(errors): swap tty error to invalidEnvironmentError

Author: Hweinstock

src/globalConfig/accessor.tsx

@@ -76,7 +76,7 @@ export class DefaultGlobalConfigAccessor implements GlobalConfigAccessor {
   private async writeToConfigFile(data: GlobalConfigFileData): Promise<GlobalConfigFileData> {
     const dataParseResult = globalConfigFileSchema.safeParse(data);
     if (!dataParseResult.success) {
-      throw new InternalValidationError("failed to valide config data before writing to file", {
+      throw new InternalValidationError("failed to validate config data before writing to file", {
         cause: dataParseResult.error,
       });
     }

src/tui/index.tsx

@@ -11,6 +11,7 @@ import {
 import type { AppIO } from "../io";
 import type { Core } from "../handlers/types";
 import { JsonKey } from "../handlers/keys";
+import { AgentCoreCLIError, ERROR_SOURCE, type AgentCoreCLIErrorOptions } from "../errors";
 
 // renderJson pretty-prints a value as indented JSON. It is the output
 // counterpart to renderTui: handlers call it to emit machine-readable results
@@ -43,7 +44,7 @@ export async function renderTuiAt(
   io: AppIO,
 ): Promise<void> {
   if (!io.stdin.isTTY || !io.stdout.isTTY) {
-    throw new TypeError("interactive mode requires a TTY on stdin and stdout");
+    throw new InvalidEnvironmentError("interactive mode requires a TTY on stdin and stdout");
   }
 
   // alternateScreen switches the terminal to its alternate buffer so the TUI
@@ -74,3 +75,10 @@ export function renderTui(core: Core, io: AppIO): DefaultHandle {
     await renderTuiAt(ctx.require(PathKey), ctx, core, io);
   };
 }
+
+/** Error raised when detecting an invalid environment */
+export class InvalidEnvironmentError extends AgentCoreCLIError {
+  constructor(message?: string, options?: Omit<AgentCoreCLIErrorOptions, "source">) {
+    super(message, { ...options, source: ERROR_SOURCE.USER });
+  }
+}

src/tui/tui.test.tsx

@@ -1,6 +1,6 @@
 import { test, expect, describe } from "bun:test";
 import { createRootHandler } from "../handlers";
-import { renderJson } from "./index";
+import { InvalidEnvironmentError, renderJson } from "./index";
 import {
   createSilentLogger,
   TestCoreClient,
@@ -89,9 +89,7 @@ describe("TUI stream boundary", () => {
         globalConfigAccessor: new TestGlobalConfigAccessor(),
       });
 
-      await expect(root.route(["node", "agentcore"])).rejects.toThrow(
-        "interactive mode requires a TTY on stdin and stdout",
-      );
+      await expect(root.route(["node", "agentcore"])).rejects.toThrow(InvalidEnvironmentError);
     },
   );