@aiready/cli 0.12.12 → 0.12.16

package/src/index.ts

@@ -6,6 +6,7 @@ import {
   GLOBAL_SCAN_OPTIONS,
   GLOBAL_INFRA_OPTIONS,
   COMMON_FINE_TUNING_OPTIONS,
+  initializeParsers,
 } from '@aiready/core';
 import type {
   AnalysisResult,
@@ -92,19 +93,44 @@ const TOOL_PACKAGE_MAP: Record<string, string> = {
 };
 
 /**
- * Sanitize tool configuration by removing global options (except rootDir)
+ * Deeply sanitizes a configuration object by removing infrastructure keys like rootDir.
+ * Works recursively to clean up nested tool configurations to ensure compatibility
+ * with the AIReadyConfig schema for portable configuration files.
  */
-function sanitizeToolConfig(config: any): any {
-  if (!config || typeof config !== 'object') return config;
-  const sanitized = { ...config };
-  GLOBAL_INFRA_OPTIONS.forEach((key: string) => {
-    if (key !== 'rootDir') {
-      delete (sanitized as any)[key];
+function sanitizeConfigRecursive(obj: any): any {
+  if (!obj || typeof obj !== 'object' || Array.isArray(obj)) return obj;
+
+  const sanitized: any = {};
+  const infraToStrip = [
+    'rootDir',
+    'onProgress',
+    'progressCallback',
+    'streamResults',
+    'batchSize',
+    'useSmartDefaults',
+  ];
+
+  for (const [key, value] of Object.entries(obj)) {
+    // Skip infrastructure keys that shouldn't be in a portable config file
+    if (infraToStrip.includes(key)) continue;
+
+    if (typeof value === 'object' && value !== null && !Array.isArray(value)) {
+      sanitized[key] = sanitizeConfigRecursive(value);
+    } else {
+      sanitized[key] = value;
     }
-  });
+  }
+
   return sanitized;
 }
 
+/**
+ * Sanitize tool configuration by removing global options to match AIReadyConfig schema
+ */
+function sanitizeToolConfig(config: any): any {
+  return sanitizeConfigRecursive(config);
+}
+
 /**
  * AIReady Unified Analysis
  * Orchestrates all registered tools via the ToolRegistry.
@@ -112,6 +138,9 @@ function sanitizeToolConfig(config: any): any {
 export async function analyzeUnified(
   options: UnifiedAnalysisOptions
 ): Promise<UnifiedAnalysisResult> {
+  // Initialize language parsers
+  await initializeParsers();
+
   const startTime = Date.now();
   const requestedTools = options.tools || [
     'patterns',
@@ -169,9 +198,9 @@ export async function analyzeUnified(
 
     try {
       // Sanitize options for metadata tracking (remove functions/internal keys)
-      const sanitizedConfig = { ...options };
-      delete (sanitizedConfig as any).onProgress;
-      delete (sanitizedConfig as any).progressCallback;
+      const sanitizedOptions = { ...options };
+      delete (sanitizedOptions as any).onProgress;
+      delete (sanitizedOptions as any).progressCallback;
 
       // 1. Start with sanitized global subset
       const toolOptions: any = {
@@ -286,8 +315,9 @@ export async function analyzeUnified(
     }
   }
 
-  // Finalize configuration for metadata to match AIReadyConfig structure
-  result.summary.config = {
+  // Finalize configuration for metadata to match AIReadyConfig schema
+  // We use sanitizeConfigRecursive to ensure no internal keys (like rootDir) remain
+  result.summary.config = sanitizeConfigRecursive({
     scan: {
       tools: requestedTools,
       include: options.include,
@@ -295,10 +325,7 @@ export async function analyzeUnified(
     },
     // Use 'tools' for tool-specific configurations to match AIReadyConfig
     tools: result.summary.toolConfigs,
-    // Keep top-level options for backward compatibility
-    rootDir: options.rootDir,
-    useSmartDefaults: options.useSmartDefaults,
-  };
+  });
 
   result.summary.executionTime = Date.now() - startTime;
   return result;