fix: resolve type errors from merge conflict resolution

- Export ConnectionError, ResourceNotFoundError, ValidationError from lib/errors barrel
- Remove duplicate imports in dev/command.tsx
- Fix ConnectionError constructor calls (expects string, not Error)
- Fix InvokeResult/Result type mismatch in invoke telemetry wrapper
- Add missing harnesses field to test fixture
- Remove unused serializeResult import
- Fix unnecessary type assertion

Author: jesseturner21

package-lock.json

@@ -1,4 +1,3 @@
-
 import {
   ConnectionError,
   ResourceNotFoundError,
@@ -11,7 +10,6 @@ import { getErrorMessage } from '../../errors';
 import { detectContainerRuntime } from '../../external-requirements';
 import { ExecLogger } from '../../logging';
 import {
-  ConnectionError,
   callMcpTool,
   createDevServer,
   findAvailablePort,
@@ -37,7 +35,6 @@ import { requireProject, requireTTY } from '../../tui/guards';
 import { runCliDeploy } from '../deploy/progress';
 import { parseHeaderFlags } from '../shared/header-utils';
 import { launchTuiDevScreenWithPicker, runBrowserMode } from './browser-mode';
-import { ResourceNotFoundError, ValidationError } from '@/lib/errors/types.js';
 import type { Command } from '@commander-js/extra-typings';
 import { spawn } from 'child_process';
 import { render } from 'ink';
@@ -67,7 +64,7 @@ async function invokeDevServer(
     }
   } catch (err) {
     throw isConnectionRefused(err)
-      ? new ConnectionError(new Error(`Dev server not running on port ${port}. Start it with: agentcore dev --logs`))
+      ? new ConnectionError(`Dev server not running on port ${port}. Start it with: agentcore dev --logs`)
       : err;
   }
 }
@@ -80,7 +77,7 @@ async function invokeA2ADevServer(port: number, prompt: string, headers?: Record
     process.stdout.write('\n');
   } catch (err) {
     throw isConnectionRefused(err)
-      ? new ConnectionError(new Error(`Dev server not running on port ${port}. Start it with: agentcore dev --logs`))
+      ? new ConnectionError(`Dev server not running on port ${port}. Start it with: agentcore dev --logs`)
       : err;
   }
 }
@@ -136,7 +133,7 @@ async function handleMcpInvoke(
     }
   } catch (err) {
     throw isConnectionRefused(err)
-      ? new ConnectionError(new Error(`Dev server not running on port ${port}. Start it with: agentcore dev --logs`))
+      ? new ConnectionError(`Dev server not running on port ${port}. Start it with: agentcore dev --logs`)
       : err;
   }
 }

src/cli/commands/dev/command.tsx

@@ -1,4 +1,4 @@
-import { type Result, ValidationError, serializeResult } from '../../../lib';
+import { type Result, ValidationError } from '../../../lib';
 import { getErrorMessage } from '../../errors';
 import { withCommandRunTelemetry } from '../../telemetry/cli-command-run.js';
 import { AuthType, Protocol, standardize } from '../../telemetry/schemas/common-shapes.js';
@@ -39,7 +39,7 @@ function resolveProtocol(options: InvokeOptions, projectProtocol?: string): stri
 async function handleInvokeCLI(options: InvokeOptions, preloadedContext?: InvokeContext): Promise<InvokeResult> {
   const validation = validateInvokeOptions(options);
   if (!validation.valid) {
-    return { success: false, error: new ValidationError(validation.error ?? 'Validation failed') };
+    return { success: false, error: validation.error ?? 'Validation failed' };
   }
 
   let spinner: NodeJS.Timeout | undefined;
@@ -65,7 +65,7 @@ async function handleInvokeCLI(options: InvokeOptions, preloadedContext?: Invoke
       process.exit(result.success ? 0 : 1);
     }
 
-    const context = preloadedContext ??await loadInvokeConfig();
+    const context = preloadedContext ?? (await loadInvokeConfig());
 
     // Show spinner for non-streaming, non-json, non-exec invocations
     // Harness invoke always streams directly to stdout, so skip spinner for harness
@@ -120,7 +120,7 @@ async function handleInvokeCLI(options: InvokeOptions, preloadedContext?: Invoke
 
 function printInvokeResult(result: InvokeResult, options: InvokeOptions): void {
   if (options.json) {
-    console.log(JSON.stringify(serializeResult(result)));
+    console.log(JSON.stringify(result));
   } else if (options.stream) {
     // Streaming already wrote to stdout, just show session and log path
     if (result.sessionId) {
@@ -135,7 +135,7 @@ function printInvokeResult(result: InvokeResult, options: InvokeOptions): void {
     if (result.success && result.response) {
       console.log(result.response);
     } else if (!result.success && result.error) {
-      console.error(result.error.message);
+      console.error(result.error);
     }
     if (result.sessionId) {
       console.error(`\nSession: ${result.sessionId}`);
@@ -277,12 +277,9 @@ export const registerInvoke = (program: Command) => {
                 has_stream: cliOptions.stream ?? false,
                 has_session_id: !!cliOptions.sessionId,
                 auth_type: standardize(AuthType, cliOptions.bearerToken ? 'bearer_token' : 'sigv4'),
-                protocol: standardize(
-                  Protocol,
-                  resolveProtocol({ tool: cliOptions.tool } as InvokeOptions, agentProtocol)
-                ),
+                protocol: standardize(Protocol, resolveProtocol({ tool: cliOptions.tool }, agentProtocol)),
               },
-              async (): Promise<InvokeResult> => {
+              async (): Promise<Result> => {
                 if (!resolved.success) {
                   return { success: false, error: new ValidationError(resolved.error ?? 'Prompt resolution failed') };
                 }
@@ -324,14 +321,15 @@ export const registerInvoke = (program: Command) => {
                   actorId: cliOptions.actorId,
                 };
 
-                return handleInvokeCLI(options, invokeContext);
+                const invokeResult = await handleInvokeCLI(options, invokeContext);
+                printInvokeResult(invokeResult, options);
+                if (invokeResult.success) {
+                  return { success: true };
+                }
+                return { success: false, error: new Error(invokeResult.error ?? 'Invoke failed') };
               }
             );
 
-            printInvokeResult(result, {
-              json: cliOptions.json,
-              stream: cliOptions.stream,
-            });
             process.exit(result.success ? 0 : 1);
           } else {
             // No CLI options - interactive TUI mode (headers still passed if provided)

src/cli/commands/invoke/command.tsx

@@ -31,9 +31,7 @@ export async function handleTracesList(
   if (!isPython) {
     return {
       success: false,
-      error: new ValidationError(
-        'Traces are only supported for Python agents. TypeScript agents do not support observability traces.'
-      ),
+      error: 'Traces are only supported for Python agents. TypeScript agents do not support observability traces.',
     };
   }
 
@@ -109,9 +107,7 @@ export async function handleTracesGet(
   if (!isPython) {
     return {
       success: false,
-      error: new ValidationError(
-        'Traces are only supported for Python agents. TypeScript agents do not support observability traces.'
-      ),
+      error: 'Traces are only supported for Python agents. TypeScript agents do not support observability traces.',
     };
   }
 

src/cli/commands/traces/action.ts

@@ -614,6 +614,7 @@ describe('getDevSupportedAgents', () => {
       configBundles: [],
       abTests: [],
       httpGateways: [],
+      harnesses: [],
     };
 
     const supported = getDevSupportedAgents(project);

src/cli/operations/dev/__tests__/config.test.ts

@@ -1 +1,2 @@
 export * from './config';
+export { ConnectionError, ResourceNotFoundError, ValidationError } from './types';