@corbat-tech/coco 2.0.0 → 2.2.0

package/dist/index.d.ts

@@ -379,7 +379,7 @@ interface Orchestrator {
 interface OrchestratorConfig {
     projectPath: string;
     provider: {
-        type: "anthropic" | "openai" | "gemini" | "kimi" | "lmstudio";
+        type: "anthropic" | "openai" | "gemini" | "kimi" | "kimi-code" | "lmstudio";
         apiKey?: string;
         model: string;
         maxTokens?: number;
@@ -500,6 +500,7 @@ declare const CocoConfigSchema: z.ZodObject<{
             openai: "openai";
             gemini: "gemini";
             kimi: "kimi";
+            "kimi-code": "kimi-code";
             lmstudio: "lmstudio";
             deepseek: "deepseek";
             mistral: "mistral";
@@ -2958,6 +2959,15 @@ declare class AnthropicProvider implements LLMProvider {
      * Ensure client is initialized
      */
     private ensureInitialized;
+    /**
+     * Extract system prompt from messages array or options.
+     *
+     * The agent-loop passes the system message as the first element of the
+     * messages array (role: "system"). convertMessages() strips it out because
+     * Anthropic requires it as a top-level parameter — but all callers forgot
+     * to also pass it via options.system. This helper bridges that gap.
+     */
+    private extractSystem;
     /**
      * Convert messages to Anthropic format
      */
@@ -3071,7 +3081,7 @@ declare class TaskError extends CocoError {
 /**
  * Supported provider types
  */
-type ProviderType = "anthropic" | "openai" | "codex" | "gemini" | "kimi" | "lmstudio" | "ollama" | "groq" | "openrouter" | "mistral" | "deepseek" | "together" | "huggingface";
+type ProviderType = "anthropic" | "openai" | "codex" | "gemini" | "kimi" | "kimi-code" | "lmstudio" | "ollama" | "groq" | "openrouter" | "mistral" | "deepseek" | "together" | "huggingface";
 /**
  * Create a provider by type
  */

package/dist/index.js

@@ -118,6 +118,8 @@ function getApiKey(provider) {
       return process.env["GEMINI_API_KEY"] ?? process.env["GOOGLE_API_KEY"];
     case "kimi":
       return process.env["KIMI_API_KEY"] ?? process.env["MOONSHOT_API_KEY"];
+    case "kimi-code":
+      return process.env["KIMI_CODE_API_KEY"];
     case "lmstudio":
       return process.env["LMSTUDIO_API_KEY"] ?? "lm-studio";
     case "ollama":
@@ -148,6 +150,8 @@ function getBaseUrl(provider) {
       return process.env["OPENAI_BASE_URL"];
     case "kimi":
       return process.env["KIMI_BASE_URL"] ?? "https://api.moonshot.ai/v1";
+    case "kimi-code":
+      return process.env["KIMI_CODE_BASE_URL"] ?? "https://api.kimi.com/coding/v1";
     case "lmstudio":
       return process.env["LMSTUDIO_BASE_URL"] ?? "http://localhost:1234/v1";
     case "ollama":
@@ -180,6 +184,8 @@ function getDefaultModel(provider) {
       return process.env["GEMINI_MODEL"] ?? "gemini-3-flash-preview";
     case "kimi":
       return process.env["KIMI_MODEL"] ?? "kimi-k2.5";
+    case "kimi-code":
+      return process.env["KIMI_CODE_MODEL"] ?? "kimi-for-coding";
     case "lmstudio":
       return process.env["LMSTUDIO_MODEL"] ?? "local-model";
     case "ollama":
@@ -220,6 +226,7 @@ var init_env = __esm({
       "codex",
       "gemini",
       "kimi",
+      "kimi-code",
       "lmstudio",
       "ollama",
       "groq",
@@ -3934,6 +3941,17 @@ var DEFAULT_QUALITY_THRESHOLDS = {
   maxIterations: 10,
   minIterations: 2
 };
+
+// src/utils/subprocess-registry.ts
+var activeSubprocesses = /* @__PURE__ */ new Set();
+function trackSubprocess(proc) {
+  activeSubprocesses.add(proc);
+  const cleanup = () => activeSubprocesses.delete(proc);
+  proc.then(cleanup, cleanup);
+  return proc;
+}
+
+// src/quality/analyzers/coverage.ts
 async function detectTestFramework(projectPath) {
   try {
     const pkgPath = join(projectPath, "package.json");
@@ -4044,12 +4062,16 @@ var CoverageAnalyzer = class {
     }
     const commands = this.buildCoverageCommand(framework, coverageTool);
     try {
-      const result = await execa(commands.command, commands.args, {
+      const proc = execa(commands.command, commands.args, {
         cwd: this.projectPath,
         reject: false,
-        timeout: 12e4
+        timeout: 12e4,
         // 2 minutes
+        cleanup: true
+        // kill process tree on parent exit
       });
+      trackSubprocess(proc);
+      const result = await proc;
       if (result.exitCode !== 0 && !result.stdout.includes("coverage")) {
         throw new Error(`Tests failed: ${result.stderr || result.stdout}`);
       }
@@ -4687,7 +4709,14 @@ var BuildVerifier = class {
       if (!SAFE_BUILD_PATTERN.test(buildCommand.trim())) {
         return {
           success: false,
-          errors: [{ file: "", line: 0, column: 0, message: `Unsafe build command rejected: ${buildCommand}` }],
+          errors: [
+            {
+              file: "",
+              line: 0,
+              column: 0,
+              message: `Unsafe build command rejected: ${buildCommand}`
+            }
+          ],
           warnings: [],
           duration: Date.now() - startTime,
           stdout: "",
@@ -4979,12 +5008,16 @@ var CorrectnessAnalyzer = class {
       return { passed: 0, failed: 0, skipped: 0 };
     }
     try {
-      const result = await execa(cmd.command, cmd.args, {
+      const proc = execa(cmd.command, cmd.args, {
         cwd: this.projectPath,
         reject: false,
-        timeout: 3e5
+        timeout: 3e5,
         // 5 minutes
+        cleanup: true
+        // kill process tree on parent exit
       });
+      trackSubprocess(proc);
+      const result = await proc;
       const output = result.stdout + "\n" + result.stderr;
       switch (framework) {
         case "vitest":
@@ -6242,9 +6275,7 @@ async function loadProjectConfig(projectPath) {
   const parsed = JSON.parse(raw);
   const result = ProjectConfigSchema.safeParse(parsed);
   if (!result.success) {
-    throw new Error(
-      `Invalid ${PROJECT_CONFIG_FILENAME} at ${configPath}: ${result.error.message}`
-    );
+    throw new Error(`Invalid ${PROJECT_CONFIG_FILENAME} at ${configPath}: ${result.error.message}`);
   }
   let config = result.data;
   if (config.extend) {
@@ -6330,10 +6361,7 @@ function mergeThresholds(base, overrides) {
 }
 function resolvedThresholds(projectConfig) {
   if (!projectConfig) return DEFAULT_QUALITY_THRESHOLDS;
-  return mergeThresholds(
-    DEFAULT_QUALITY_THRESHOLDS,
-    thresholdsFromProjectConfig(projectConfig)
-  );
+  return mergeThresholds(DEFAULT_QUALITY_THRESHOLDS, thresholdsFromProjectConfig(projectConfig));
 }
 function weightsFromProjectConfig(config) {
   const overrides = config.quality?.weights;
@@ -6416,9 +6444,7 @@ function getFileExtension(filePath) {
 function buildEvidence(dominant, counts, totalSourceFiles, files) {
   const evidence = [];
   const dominantCount = counts.get(dominant) ?? 0;
-  evidence.push(
-    `${dominantCount} of ${totalSourceFiles} source files are ${dominant}`
-  );
+  evidence.push(`${dominantCount} of ${totalSourceFiles} source files are ${dominant}`);
   const configFiles = ["tsconfig.json", "pom.xml", "build.gradle", "Cargo.toml", "go.mod"];
   for (const cfg of configFiles) {
     if (files.some((f) => path15.basename(f) === cfg)) {
@@ -6665,10 +6691,7 @@ var JavaStyleAnalyzer = class {
     for (const { path: filePath, content } of files) {
       violations.push(...this.checkFile(filePath, content));
     }
-    const deduction = violations.reduce(
-      (sum, v) => sum + (v.severity === "error" ? 10 : 5),
-      0
-    );
+    const deduction = violations.reduce((sum, v) => sum + (v.severity === "error" ? 10 : 5), 0);
     const score = Math.max(0, 100 - deduction);
     return { score, violations };
   }
@@ -6697,7 +6720,9 @@ var JavaStyleAnalyzer = class {
           severity: "error"
         });
       }
-      const methodMatch = /\b(?:public|private|protected|static)\s+(?!class|interface|enum|record|new\b)(?:void|[\w<>[\]]+)\s+([A-Z]\w*)\s*\(/.exec(line);
+      const methodMatch = /\b(?:public|private|protected|static)\s+(?!class|interface|enum|record|new\b)(?:void|[\w<>[\]]+)\s+([A-Z]\w*)\s*\(/.exec(
+        line
+      );
       if (methodMatch && !line.trim().startsWith("class") && !line.includes("class ")) {
         violations.push({
           rule: "MethodName",
@@ -6750,7 +6775,13 @@ var JavaDocumentationAnalyzer = class {
   async analyze(files) {
     const javaFiles = files ?? await findJavaFiles(this.projectPath, { srcPattern: "src/main/**/*.java" });
     if (!javaFiles.length) {
-      return { score: 100, javadocCoverage: 1, totalMethods: 0, documentedMethods: 0, undocumentedPublicMethods: [] };
+      return {
+        score: 100,
+        javadocCoverage: 1,
+        totalMethods: 0,
+        documentedMethods: 0,
+        undocumentedPublicMethods: []
+      };
     }
     const fileContents = await Promise.all(
       javaFiles.map(async (f) => ({
@@ -6762,7 +6793,13 @@ var JavaDocumentationAnalyzer = class {
   }
   analyzeContent(files) {
     if (!files.length) {
-      return { score: 100, javadocCoverage: 1, totalMethods: 0, documentedMethods: 0, undocumentedPublicMethods: [] };
+      return {
+        score: 100,
+        javadocCoverage: 1,
+        totalMethods: 0,
+        documentedMethods: 0,
+        undocumentedPublicMethods: []
+      };
     }
     let totalMethods = 0;
     let documentedMethods = 0;
@@ -6825,7 +6862,13 @@ var JavaCoverageAnalyzer = class {
   }
   parseJacocoXml(xml2) {
     if (!xml2.trim()) {
-      return { score: 0, lineCoverage: 0, branchCoverage: 0, methodCoverage: 0, reportFound: false };
+      return {
+        score: 0,
+        lineCoverage: 0,
+        branchCoverage: 0,
+        methodCoverage: 0,
+        reportFound: false
+      };
     }
     const lineCoverage = this.extractCoverage(xml2, "LINE");
     const branchCoverage = this.extractCoverage(xml2, "BRANCH");
@@ -6834,9 +6877,7 @@ var JavaCoverageAnalyzer = class {
     return { score, lineCoverage, branchCoverage, methodCoverage, reportFound: true };
   }
   extractCoverage(xml2, type) {
-    const regex = new RegExp(
-      `<counter\\s+type="${type}"\\s+missed="(\\d+)"\\s+covered="(\\d+)"`
-    );
+    const regex = new RegExp(`<counter\\s+type="${type}"\\s+missed="(\\d+)"\\s+covered="(\\d+)"`);
     const match = regex.exec(xml2);
     if (!match) return 0;
     const missed = parseInt(match[1] ?? "0", 10);
@@ -11498,7 +11539,7 @@ var AnthropicProvider = class {
           model: options?.model ?? this.config.model ?? DEFAULT_MODEL,
           max_tokens: options?.maxTokens ?? this.config.maxTokens ?? 8192,
           temperature: options?.temperature ?? this.config.temperature ?? 0,
-          system: options?.system,
+          system: this.extractSystem(messages, options?.system),
           messages: this.convertMessages(messages),
           stop_sequences: options?.stopSequences
         });
@@ -11528,7 +11569,7 @@ var AnthropicProvider = class {
           model: options?.model ?? this.config.model ?? DEFAULT_MODEL,
           max_tokens: options?.maxTokens ?? this.config.maxTokens ?? 8192,
           temperature: options?.temperature ?? this.config.temperature ?? 0,
-          system: options?.system,
+          system: this.extractSystem(messages, options?.system),
           messages: this.convertMessages(messages),
           tools: this.convertTools(options.tools),
           tool_choice: options.toolChoice ? this.convertToolChoice(options.toolChoice) : void 0
@@ -11560,7 +11601,7 @@ var AnthropicProvider = class {
         model: options?.model ?? this.config.model ?? DEFAULT_MODEL,
         max_tokens: options?.maxTokens ?? this.config.maxTokens ?? 8192,
         temperature: options?.temperature ?? this.config.temperature ?? 0,
-        system: options?.system,
+        system: this.extractSystem(messages, options?.system),
         messages: this.convertMessages(messages)
       });
       for await (const event of stream) {
@@ -11586,7 +11627,7 @@ var AnthropicProvider = class {
         model: options?.model ?? this.config.model ?? DEFAULT_MODEL,
         max_tokens: options?.maxTokens ?? this.config.maxTokens ?? 8192,
         temperature: options?.temperature ?? this.config.temperature ?? 0,
-        system: options?.system,
+        system: this.extractSystem(messages, options?.system),
         messages: this.convertMessages(messages),
         tools: this.convertTools(options.tools),
         tool_choice: options.toolChoice ? this.convertToolChoice(options.toolChoice) : void 0
@@ -11709,6 +11750,22 @@ var AnthropicProvider = class {
       });
     }
   }
+  /**
+   * Extract system prompt from messages array or options.
+   *
+   * The agent-loop passes the system message as the first element of the
+   * messages array (role: "system"). convertMessages() strips it out because
+   * Anthropic requires it as a top-level parameter — but all callers forgot
+   * to also pass it via options.system. This helper bridges that gap.
+   */
+  extractSystem(messages, optionsSystem) {
+    if (optionsSystem !== void 0) return optionsSystem;
+    const systemMsg = messages.find((m) => m.role === "system");
+    if (!systemMsg) return void 0;
+    if (typeof systemMsg.content === "string") return systemMsg.content;
+    const text = systemMsg.content.filter((b) => b.type === "text").map((b) => b.text).join("");
+    return text || void 0;
+  }
   /**
    * Convert messages to Anthropic format
    */
@@ -11869,6 +11926,7 @@ var CONTEXT_WINDOWS2 = {
   "kimi-k2.5": 262144,
   "kimi-k2-0324": 131072,
   "kimi-latest": 131072,
+  "kimi-for-coding": 131072,
   "moonshot-v1-8k": 8e3,
   "moonshot-v1-32k": 32e3,
   "moonshot-v1-128k": 128e3,
@@ -11945,7 +12003,12 @@ var LOCAL_MODEL_PATTERNS = [
   "gemma",
   "starcoder"
 ];
-var MODELS_WITH_THINKING_MODE = ["kimi-k2.5", "kimi-k2-0324", "kimi-latest"];
+var MODELS_WITH_THINKING_MODE = [
+  "kimi-k2.5",
+  "kimi-k2-0324",
+  "kimi-latest",
+  "kimi-for-coding"
+];
 var OpenAIProvider = class {
   id;
   name;
@@ -12564,6 +12627,20 @@ function createKimiProvider(config) {
   }
   return provider;
 }
+function createKimiCodeProvider(config) {
+  const provider = new OpenAIProvider("kimi-code", "Kimi Code");
+  const kimiCodeConfig = {
+    ...config,
+    baseUrl: config?.baseUrl ?? process.env["KIMI_CODE_BASE_URL"] ?? "https://api.kimi.com/coding/v1",
+    apiKey: config?.apiKey ?? process.env["KIMI_CODE_API_KEY"],
+    model: config?.model ?? "kimi-for-coding"
+  };
+  if (kimiCodeConfig.apiKey) {
+    provider.initialize(kimiCodeConfig).catch(() => {
+    });
+  }
+  return provider;
+}
 var OAUTH_CONFIGS = {
   /**
    * OpenAI OAuth (ChatGPT Plus/Pro subscriptions)
@@ -13067,7 +13144,7 @@ var GeminiProvider = class {
           temperature: options?.temperature ?? this.config.temperature ?? 0,
           stopSequences: options?.stopSequences
         },
-        systemInstruction: options?.system
+        systemInstruction: this.extractSystem(messages, options?.system)
       });
       const { history, lastMessage } = this.convertMessages(messages);
       const chat = model.startChat({ history });
@@ -13095,7 +13172,7 @@ var GeminiProvider = class {
           maxOutputTokens: options?.maxTokens ?? this.config.maxTokens ?? 8192,
           temperature: options?.temperature ?? this.config.temperature ?? 0
         },
-        systemInstruction: options?.system,
+        systemInstruction: this.extractSystem(messages, options?.system),
         tools,
         toolConfig: {
           functionCallingConfig: {
@@ -13124,7 +13201,7 @@ var GeminiProvider = class {
           maxOutputTokens: options?.maxTokens ?? this.config.maxTokens ?? 8192,
           temperature: options?.temperature ?? this.config.temperature ?? 0
         },
-        systemInstruction: options?.system
+        systemInstruction: this.extractSystem(messages, options?.system)
       });
       const { history, lastMessage } = this.convertMessages(messages);
       const chat = model.startChat({ history });
@@ -13158,7 +13235,7 @@ var GeminiProvider = class {
           maxOutputTokens: options?.maxTokens ?? this.config.maxTokens ?? 8192,
           temperature: options?.temperature ?? this.config.temperature ?? 0
         },
-        systemInstruction: options?.system,
+        systemInstruction: this.extractSystem(messages, options?.system),
         tools,
         toolConfig: {
           functionCallingConfig: {
@@ -13259,6 +13336,21 @@ var GeminiProvider = class {
       });
     }
   }
+  /**
+   * Extract system prompt from messages array or options.
+   *
+   * convertMessages() skips system-role messages ("handled via systemInstruction"),
+   * but all callers forgot to also pass it via options.system. This helper bridges
+   * that gap — mirrors the same fix applied to AnthropicProvider.
+   */
+  extractSystem(messages, optionsSystem) {
+    if (optionsSystem !== void 0) return optionsSystem;
+    const systemMsg = messages.find((m) => m.role === "system");
+    if (!systemMsg) return void 0;
+    if (typeof systemMsg.content === "string") return systemMsg.content;
+    const text = systemMsg.content.filter((b) => b.type === "text").map((b) => b.text).join("");
+    return text || void 0;
+  }
   /**
    * Convert messages to Gemini format
    */
@@ -13457,6 +13549,10 @@ async function createProvider(type, config = {}) {
       provider = createKimiProvider(mergedConfig);
       await provider.initialize(mergedConfig);
       return provider;
+    case "kimi-code":
+      provider = createKimiCodeProvider(mergedConfig);
+      await provider.initialize(mergedConfig);
+      return provider;
     case "lmstudio":
       provider = new OpenAIProvider("lmstudio", "LM Studio");
       mergedConfig.baseUrl = mergedConfig.baseUrl ?? "http://localhost:1234/v1";
@@ -13988,6 +14084,7 @@ var ProviderConfigSchema = z.object({
     "codex",
     "gemini",
     "kimi",
+    "kimi-code",
     "lmstudio",
     "ollama",
     "groq",
@@ -15890,12 +15987,16 @@ Examples:
             tool: "run_tests"
           });
       }
-      const result = await execa(command, args, {
+      const proc = execa(command, args, {
         cwd: projectDir,
         reject: false,
-        timeout: 3e5
+        timeout: 3e5,
         // 5 minute timeout
+        cleanup: true
+        // kill process tree on parent exit
       });
+      trackSubprocess(proc);
+      const result = await proc;
       const duration = performance.now() - startTime;
       return parseTestResults(
         detectedFramework,