@codexa/cli 9.0.1 → 9.0.3

package/gates/validator.ts

@@ -5,6 +5,7 @@ import { validateAgainstStandards, printValidationResult } from "./standards-val
 import { runTypecheck, printTypecheckResult } from "./typecheck-validator";
 import { extractUtilitiesFromFile } from "../commands/patterns";
 import { findDuplicateUtilities } from "../db/schema";
+import { GateError } from "../errors";
 
 export interface GateResult {
   passed: boolean;
@@ -63,8 +64,8 @@ const GATES: Record<string, GateCheck[]> = {
     },
     {
       check: "checkpoint-filled",
-      message: "Checkpoint obrigatorio",
-      resolution: "Forneca --checkpoint 'resumo do que foi feito'",
+      message: "Checkpoint obrigatorio (min 30 chars, 5 palavras)",
+      resolution: "Forneca --checkpoint 'resumo detalhado do que foi feito' (min 30 caracteres e 5 palavras)",
     },
     {
       check: "files-exist",
@@ -164,9 +165,15 @@ function executeCheck(check: string, context: any): { passed: boolean; details?:
     }
 
     case "checkpoint-filled": {
-      return {
-        passed: !!context.checkpoint && context.checkpoint.length >= 10,
-      };
+      const cp = context.checkpoint?.trim() || "";
+      if (cp.length < 30) {
+        return { passed: false, details: "Checkpoint deve ter pelo menos 30 caracteres" };
+      }
+      const wordCount = cp.split(/\s+/).filter((w: string) => w.length > 1).length;
+      if (wordCount < 5) {
+        return { passed: false, details: "Checkpoint deve ter pelo menos 5 palavras" };
+      }
+      return { passed: true };
     }
 
     case "all-tasks-done": {
@@ -192,12 +199,35 @@ function executeCheck(check: string, context: any): { passed: boolean; details?:
       if (!context.files || context.files.length === 0) return { passed: true };
 
       // v8.0: Validar não apenas existência, mas conteúdo mínimo
+      // v9.2: Validar que arquivo foi modificado DURANTE a task
       const issues: string[] = [];
 
+      // Buscar started_at da task para comparacao temporal
+      let taskStartTime: number | null = null;
+      if (context.taskId) {
+        const taskRow = db.query("SELECT started_at FROM tasks WHERE id = ?").get(context.taskId) as any;
+        if (taskRow?.started_at) {
+          taskStartTime = new Date(taskRow.started_at).getTime();
+        }
+      }
+
       for (const file of context.files) {
         const validation = validateFileContent(file);
         if (!validation.valid) {
           issues.push(`${file}: ${validation.reason}`);
+          continue;
+        }
+        // Verificar que arquivo foi tocado durante a task
+        if (taskStartTime) {
+          try {
+            const stat = statSync(file);
+            const mtime = stat.mtimeMs;
+            if (mtime < taskStartTime) {
+              issues.push(`${file}: arquivo nao foi modificado durante esta task (mtime anterior ao start)`);
+            }
+          } catch {
+            // statSync falhou — arquivo pode nao existir (ja reportado por validateFileContent)
+          }
         }
       }
 
@@ -409,14 +439,16 @@ export function enforceGate(command: string, context: any = {}): void {
   const result = validateGate(command, context);
 
   if (!result.passed) {
-    console.error(`\nBLOQUEADO: ${result.reason}`);
-    console.error(`Resolva: ${result.resolution}\n`);
-    process.exit(1);
+    throw new GateError(
+      result.reason || "Gate falhou",
+      result.resolution || "Verifique o estado atual",
+      command
+    );
   }
 }
 
 // v8.0: Validar conteúdo de arquivos criados (não apenas existência)
-interface FileValidationResult {
+export interface FileValidationResult {
   valid: boolean;
   reason?: string;
 }
@@ -456,7 +488,7 @@ function validateFileContent(filePath: string): FileValidationResult {
   }
 }
 
-function validateByExtension(ext: string, content: string): FileValidationResult {
+export function validateByExtension(ext: string, content: string): FileValidationResult {
   const trimmed = content.trim();
 
   switch (ext) {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@codexa/cli",
-  "version": "9.0.1",
+  "version": "9.0.3",
   "description": "Orchestrated workflow system for Claude Code - manages feature development through parallel subagents with structured phases, gates, and quality enforcement.",
   "type": "module",
   "bin": {
@@ -16,6 +16,8 @@
   ],
   "scripts": {
     "cli": "bun run workflow.ts",
+    "test": "bun test",
+    "test:watch": "bun test --watch",
     "install": "npm publish --access public"
   },
   "engines": {

package/protocol/process-return.ts

@@ -11,7 +11,7 @@
 
 import { getDb } from "../db/connection";
 import { SubagentReturn, Knowledge } from "./subagent-protocol";
-import { addReasoning, addGraphRelation, getRelatedFiles, upsertUtility } from "../db/schema";
+import { addReasoning, addGraphRelation, upsertUtility, getNextDecisionId } from "../db/schema";
 import { extractUtilitiesFromFile, inferScopeFromPath } from "../commands/patterns";
 
 interface ProcessResult {
@@ -26,18 +26,6 @@ interface ProcessResult {
   errors: string[];
 }
 
-function getNextDecisionId(specId: string): string {
-  const db = getDb();
-  const last = db
-    .query("SELECT id FROM decisions WHERE spec_id = ? ORDER BY created_at DESC LIMIT 1")
-    .get(specId) as any;
-
-  if (!last) return "DEC-001";
-
-  const num = parseInt(last.id.replace("DEC-", "")) + 1;
-  return `DEC-${num.toString().padStart(3, "0")}`;
-}
-
 /**
  * Processa o retorno de um subagent e registra automaticamente
  * todos os dados extraidos no banco
@@ -85,87 +73,32 @@ export function processSubagentReturn(
     }
   }
 
-  // 2. Registrar Decisions
+  // 2. Registrar Decisions (com retry para collision de IDs concorrentes)
+  const savedDecisionIds: string[] = [];
   if (data.decisions_made && data.decisions_made.length > 0) {
     for (const dec of data.decisions_made) {
-      try {
-        const decisionId = getNextDecisionId(specId);
-        db.run(
-          `INSERT INTO decisions (id, spec_id, task_ref, title, decision, rationale, status, created_at)
-           VALUES (?, ?, ?, ?, ?, ?, 'active', ?)`,
-          [decisionId, specId, taskNumber, dec.title, dec.decision, dec.rationale || null, now]
-        );
-        result.decisionsAdded++;
-      } catch (e) {
-        result.errors.push(`Erro ao registrar decision: ${(e as Error).message}`);
-      }
-    }
-  }
-
-  // 2b. v8.3: Deteccao automatica de contradicoes entre decisoes
-  if (data.decisions_made && data.decisions_made.length > 0) {
-    try {
-      const existingDecisions = db.query(
-        "SELECT id, title, decision FROM decisions WHERE spec_id = ? AND status = 'active'"
-      ).all(specId) as any[];
-
-      const allTaskFiles = [...data.files_created, ...data.files_modified];
-
-      for (const newDec of data.decisions_made) {
-        for (const existingDec of existingDecisions) {
-          // Skip se mesma decisao (recem adicionada)
-          if (existingDec.title === newDec.title) continue;
-
-          const newDecLower = `${newDec.title} ${newDec.decision}`.toLowerCase();
-          const existDecLower = `${existingDec.title} ${existingDec.decision}`.toLowerCase();
-
-          // Verificar overlap de arquivos
-          const existingDecFiles = getRelatedFiles(existingDec.id, "decision");
-          const fileOverlap = allTaskFiles.some(f => existingDecFiles.includes(f));
-
-          if (fileOverlap) {
-            // Heuristica de keywords opostos
-            const opposites = [
-              ['usar', 'nao usar'], ['usar', 'evitar'],
-              ['use', 'avoid'], ['add', 'remove'],
-              ['incluir', 'excluir'], ['habilitar', 'desabilitar'],
-              ['enable', 'disable'], ['create', 'delete'],
-              ['client', 'server'], ['sync', 'async'],
-            ];
-
-            let isContradiction = false;
-            for (const [word1, word2] of opposites) {
-              if ((newDecLower.includes(word1) && existDecLower.includes(word2)) ||
-                  (newDecLower.includes(word2) && existDecLower.includes(word1))) {
-                isContradiction = true;
-                break;
-              }
-            }
-
-            if (isContradiction) {
-              const newDecId = db.query(
-                "SELECT id FROM decisions WHERE spec_id = ? AND title = ? ORDER BY created_at DESC LIMIT 1"
-              ).get(specId, newDec.title) as any;
-
-              if (newDecId) {
-                addGraphRelation(specId, {
-                  sourceType: "decision",
-                  sourceId: newDecId.id,
-                  targetType: "decision",
-                  targetId: existingDec.id,
-                  relation: "contradicts",
-                  strength: 0.7,
-                  metadata: { reason: "Arquivos sobrepostos com semantica oposta" },
-                });
-                result.relationsAdded++;
-              }
-            }
+      let retries = 3;
+      let inserted = false;
+      while (retries > 0 && !inserted) {
+        try {
+          const decisionId = getNextDecisionId(specId);
+          db.run(
+            `INSERT INTO decisions (id, spec_id, task_ref, title, decision, rationale, status, created_at)
+             VALUES (?, ?, ?, ?, ?, ?, 'active', ?)`,
+            [decisionId, specId, taskNumber, dec.title, dec.decision, dec.rationale || null, now]
+          );
+          savedDecisionIds.push(decisionId);
+          result.decisionsAdded++;
+          inserted = true;
+        } catch (e: any) {
+          if (e.message?.includes("UNIQUE constraint") && retries > 1) {
+            retries--;
+            continue;
           }
+          result.errors.push(`Erro ao registrar decision: ${(e as Error).message}`);
+          break;
         }
       }
-    } catch (e) {
-      // Nao-critico: nao falhar processamento por deteccao de contradicoes
-      result.errors.push(`Aviso contradicoes: ${(e as Error).message}`);
     }
   }
 
@@ -331,10 +264,9 @@ export function processSubagentReturn(
       result.relationsAdded++;
     }
 
-    // Relação: decision -> arquivos (se houver decisions)
-    if (data.decisions_made) {
-      for (const dec of data.decisions_made) {
-        const decisionId = getNextDecisionId(specId);
+    // Relação: decision -> arquivos (usa IDs salvos na seção 2, não gera novos)
+    if (savedDecisionIds.length > 0) {
+      for (const decisionId of savedDecisionIds) {
         for (const file of [...data.files_created, ...data.files_modified]) {
           addGraphRelation(specId, {
             sourceType: "decision",