Remove slop

Author: cameroncooke

src/utils/execution/interactive-process.ts

@@ -73,9 +73,7 @@ function createInteractiveProcess(
 export function getDefaultInteractiveSpawner(): InteractiveSpawner {
   if (process.env.VITEST === 'true' || process.env.NODE_ENV === 'test') {
     throw new Error(
-      `🚨 REAL INTERACTIVE SPAWNER DETECTED IN TEST! 🚨\n` +
-        `This test is trying to spawn a real interactive process.\n` +
-        `Fix: Inject a mock InteractiveSpawner in your test setup.`,
+      'Interactive process spawn blocked in tests. Inject a mock InteractiveSpawner.',
     );
   }
 

src/utils/typed-tool-factory.ts

@@ -23,11 +23,8 @@ function createValidatedHandler<TParams, TContext>(
 ): (args: Record<string, unknown>) => Promise<ToolResponse> {
   return async (args: Record<string, unknown>): Promise<ToolResponse> => {
     try {
-      // Runtime validation - the ONLY safe way to cross the type boundary
-      // This provides both compile-time and runtime type safety
       const validatedParams = schema.parse(args);
 
-      // Now we have guaranteed type safety - no assertions needed!
       return await logicFunction(validatedParams, getContext());
     } catch (error) {
       if (error instanceof z.ZodError) {
@@ -45,9 +42,6 @@ function createValidatedHandler<TParams, TContext>(
  * Creates a type-safe tool handler that validates parameters at runtime
  * before passing them to the typed logic function.
  *
- * This is the ONLY safe way to cross the type boundary from the generic
- * MCP handler signature to our typed domain logic.
- *
  * @param schema - Zod schema for parameter validation
  * @param logicFunction - The typed logic function to execute
  * @param getExecutor - Function to get the command executor (must be provided)