sdd-cli 0.1.24 → 0.1.26

package/dist/commands/app-lifecycle.js

@@ -9,6 +9,41 @@ const fs_1 = __importDefault(require("fs"));
 const path_1 = __importDefault(require("path"));
 const child_process_1 = require("child_process");
 const config_1 = require("../config");
+function collectFilesRecursive(root, maxDepth = 8) {
+    const results = [];
+    const walk = (current, depth) => {
+        if (depth > maxDepth) {
+            return;
+        }
+        const entries = fs_1.default.readdirSync(current, { withFileTypes: true });
+        for (const entry of entries) {
+            const full = path_1.default.join(current, entry.name);
+            const rel = path_1.default.relative(root, full).replace(/\\/g, "/");
+            if (entry.isDirectory()) {
+                if ([".git", "node_modules", "dist", "build", "target", "__pycache__", ".venv", "venv"].includes(entry.name.toLowerCase())) {
+                    continue;
+                }
+                walk(full, depth + 1);
+            }
+            else {
+                results.push(rel);
+            }
+        }
+    };
+    walk(root, 0);
+    return results;
+}
+function countJsTsTests(root, maxDepth = 8) {
+    const files = collectFilesRecursive(root, maxDepth)
+        .filter((rel) => /\.(jsx?|tsx?)$/i.test(rel))
+        .filter((rel) => /\.test\.|\.spec\.|__tests__\//i.test(rel));
+    let count = 0;
+    for (const rel of files) {
+        const raw = fs_1.default.readFileSync(path_1.default.join(root, rel), "utf-8");
+        count += (raw.match(/\b(test|it)\s*\(/g) || []).length;
+    }
+    return count;
+}
 function findFileRecursive(root, predicate, maxDepth = 4) {
     const walk = (current, depth) => {
         if (depth > maxDepth) {
@@ -73,14 +108,21 @@ function countTestsRecursive(root, maxDepth = 8) {
 }
 function run(command, args, cwd) {
     let resolved = command;
-    if (process.platform === "win32" && command === "npm") {
-        resolved = "npm.cmd";
+    if (process.platform === "win32") {
+        if (command === "npm") {
+            resolved = "npm.cmd";
+        }
+        else if (command === "mvn") {
+            resolved = "mvn.cmd";
+        }
     }
     const useShell = process.platform === "win32" && resolved.toLowerCase().endsWith(".cmd");
     const result = useShell
         ? (0, child_process_1.spawnSync)([resolved, ...args].join(" "), { cwd, encoding: "utf-8", shell: true })
         : (0, child_process_1.spawnSync)(resolved, args, { cwd, encoding: "utf-8", shell: false });
-    const output = `${result.stdout || ""}${result.stderr || ""}`.trim();
+    const rawOutput = `${result.stdout || ""}${result.stderr || ""}`.trim();
+    const merged = rawOutput || (result.error ? String(result.error.message || result.error) : "");
+    const output = merged.length > 3500 ? `${merged.slice(0, 3500)}\n...[truncated]` : merged;
     return {
         ok: result.status === 0,
         command: [resolved, ...args].join(" "),
@@ -107,19 +149,58 @@ function runIfScript(cwd, script) {
         return null;
     }
 }
-function packageNeedsInstall(cwd) {
+function readPackageJson(cwd) {
     const pkgPath = path_1.default.join(cwd, "package.json");
     if (!fs_1.default.existsSync(pkgPath)) {
-        return false;
+        return null;
     }
     try {
-        const pkg = JSON.parse(fs_1.default.readFileSync(pkgPath, "utf-8"));
-        const depCount = Object.keys(pkg.dependencies ?? {}).length + Object.keys(pkg.devDependencies ?? {}).length;
-        return depCount > 0;
+        return JSON.parse(fs_1.default.readFileSync(pkgPath, "utf-8"));
     }
     catch {
+        return null;
+    }
+}
+function packageNeedsInstall(cwd) {
+    const pkg = readPackageJson(cwd);
+    if (!pkg) {
         return false;
     }
+    const depCount = Object.keys(pkg.dependencies ?? {}).length + Object.keys(pkg.devDependencies ?? {}).length;
+    return depCount > 0;
+}
+function parseGoalProfile(context) {
+    const goal = normalizeText(context?.goalText ?? "");
+    const hasJava = /\bjava\b/.test(goal);
+    const hasReact = /\breact\b/.test(goal);
+    const relationalHints = [
+        "library",
+        "biblioteca",
+        "inventario",
+        "inventory",
+        "prestamo",
+        "prestamos",
+        "loan",
+        "loans",
+        "usuario",
+        "usuarios",
+        "user",
+        "users",
+        "book",
+        "books",
+        "cita",
+        "citas",
+        "appointment",
+        "appointments",
+        "hospital",
+        "gestion",
+        "management"
+    ];
+    const relationalDataApp = relationalHints.some((hint) => goal.includes(hint));
+    return {
+        javaReactFullstack: hasJava && hasReact,
+        relationalDataApp
+    };
 }
 function basicQualityCheck(appDir) {
     const required = ["README.md"];
@@ -192,6 +273,112 @@ function advancedQualityCheck(appDir, context) {
             output: "Missing schemas.md (or equivalent schema markdown document)"
         };
     }
+    const profile = parseGoalProfile(context);
+    if (profile.relationalDataApp) {
+        const sqlSchema = findFileRecursive(appDir, (rel) => rel === "schema.sql" || rel.endsWith("/schema.sql")) ??
+            findFileRecursive(appDir, (rel) => rel.endsWith(".sql") && (rel.includes("schema") || rel.includes("init") || rel.includes("migration")), 10);
+        if (!sqlSchema) {
+            return {
+                ok: false,
+                command: "advanced-quality-check",
+                output: "Missing SQL schema file (expected schema.sql or migration .sql) for relational data app"
+            };
+        }
+        const schemaText = normalizeText(fs_1.default.readFileSync(path_1.default.join(appDir, schemaDoc), "utf-8"));
+        const readmeText = readme;
+        if (!/(postgres|postgresql|mysql|mariadb|sqlite)/.test(`${schemaText}\n${readmeText}`)) {
+            return {
+                ok: false,
+                command: "advanced-quality-check",
+                output: "Database technology not explicit in README/schemas (expected PostgreSQL/MySQL/MariaDB/SQLite)"
+            };
+        }
+    }
+    if (profile.javaReactFullstack) {
+        const hasBackendPom = fs_1.default.existsSync(path_1.default.join(appDir, "backend", "pom.xml"));
+        const hasFrontendPkg = fs_1.default.existsSync(path_1.default.join(appDir, "frontend", "package.json"));
+        if (!hasBackendPom || !hasFrontendPkg) {
+            return {
+                ok: false,
+                command: "advanced-quality-check",
+                output: "Expected Java+React fullstack structure: backend/pom.xml and frontend/package.json"
+            };
+        }
+        const frontendPkg = readPackageJson(path_1.default.join(appDir, "frontend"));
+        if (frontendPkg) {
+            const deps = { ...(frontendPkg.dependencies ?? {}), ...(frontendPkg.devDependencies ?? {}) };
+            if (typeof deps["react-query"] === "string") {
+                return {
+                    ok: false,
+                    command: "advanced-quality-check",
+                    output: "Outdated frontend dependency detected: react-query. Use @tanstack/react-query."
+                };
+            }
+        }
+        const backendRoot = path_1.default.join(appDir, "backend", "src", "main", "java");
+        if (!fs_1.default.existsSync(backendRoot)) {
+            return {
+                ok: false,
+                command: "advanced-quality-check",
+                output: "Missing backend/src/main/java for Java backend implementation"
+            };
+        }
+        const backendFiles = collectFilesRecursive(backendRoot, 12).filter((rel) => rel.toLowerCase().endsWith(".java"));
+        const hasDto = backendFiles.some((rel) => /\/dto\//i.test(`/${rel}`) || /dto\.java$/i.test(rel));
+        if (!hasDto) {
+            return {
+                ok: false,
+                command: "advanced-quality-check",
+                output: "Missing Java DTO layer (expected dto package and *Dto.java files)"
+            };
+        }
+        const hasRecord = backendFiles.some((rel) => /\brecord\b/.test(fs_1.default.readFileSync(path_1.default.join(backendRoot, rel), "utf-8")));
+        if (!hasRecord) {
+            return {
+                ok: false,
+                command: "advanced-quality-check",
+                output: "Missing Java record usage (expected at least one record for immutable transport/domain models)"
+            };
+        }
+        const serviceFiles = backendFiles.filter((rel) => /\/service\//i.test(`/${rel}`));
+        const repositoryFiles = backendFiles.filter((rel) => /\/repository\//i.test(`/${rel}`));
+        const hasServiceInterface = serviceFiles.some((rel) => /\binterface\b/.test(fs_1.default.readFileSync(path_1.default.join(backendRoot, rel), "utf-8")));
+        const hasRepositoryInterface = repositoryFiles.some((rel) => /\binterface\b/.test(fs_1.default.readFileSync(path_1.default.join(backendRoot, rel), "utf-8")));
+        if (!hasServiceInterface || !hasRepositoryInterface) {
+            return {
+                ok: false,
+                command: "advanced-quality-check",
+                output: "Missing service/repository interfaces in Java backend architecture"
+            };
+        }
+        const frontendRoot = path_1.default.join(appDir, "frontend", "src");
+        if (!fs_1.default.existsSync(frontendRoot)) {
+            return {
+                ok: false,
+                command: "advanced-quality-check",
+                output: "Missing frontend/src for React frontend implementation"
+            };
+        }
+        const frontendFiles = collectFilesRecursive(frontendRoot, 10);
+        const hasApiLayer = frontendFiles.some((rel) => /^api\//i.test(rel) || /\/api\//i.test(`/${rel}`));
+        const hasHooksLayer = frontendFiles.some((rel) => /^hooks\/use[A-Z].*\.(t|j)sx?$/i.test(rel) || /\/hooks\/use[A-Z]/.test(rel));
+        const hasComponentsLayer = frontendFiles.some((rel) => /^components\//i.test(rel) || /\/components\//i.test(`/${rel}`));
+        if (!hasApiLayer || !hasHooksLayer || !hasComponentsLayer) {
+            return {
+                ok: false,
+                command: "advanced-quality-check",
+                output: "Missing frontend layers (expected src/api, src/hooks/use*.ts(x), and src/components)"
+            };
+        }
+        const frontendTestCount = countJsTsTests(path_1.default.join(appDir, "frontend"), 10);
+        if (frontendTestCount < 3) {
+            return {
+                ok: false,
+                command: "advanced-quality-check",
+                output: `Expected at least 3 frontend tests for Java+React profile, found ${frontendTestCount}`
+            };
+        }
+    }
     const dummyLocalDoc = findFileRecursive(appDir, (rel) => rel.includes("dummylocal") && rel.endsWith(".md")) ??
         findFileRecursive(appDir, (rel) => rel.includes("dummy-local") && rel.endsWith(".md")) ??
         findFileRecursive(appDir, (rel) => rel.includes("dummy_local") && rel.endsWith(".md"));
@@ -463,6 +650,34 @@ function runAppLifecycle(projectRoot, projectName, context) {
     const build = runIfScript(appDir, "build");
     if (build)
         qualitySteps.push(build);
+    const backendDir = path_1.default.join(appDir, "backend");
+    if (fs_1.default.existsSync(path_1.default.join(backendDir, "pom.xml"))) {
+        if (hasCommand("mvn")) {
+            qualitySteps.push(run("mvn", ["-q", "test"], backendDir));
+        }
+        else {
+            qualitySteps.push({
+                ok: false,
+                command: "mvn -q test",
+                output: "Maven not available to validate Java backend"
+            });
+        }
+    }
+    const frontendDir = path_1.default.join(appDir, "frontend");
+    if (fs_1.default.existsSync(path_1.default.join(frontendDir, "package.json"))) {
+        if (packageNeedsInstall(frontendDir)) {
+            qualitySteps.push(run("npm", ["install"], frontendDir));
+        }
+        const feLint = runIfScript(frontendDir, "lint");
+        if (feLint)
+            qualitySteps.push(feLint);
+        const feTest = runIfScript(frontendDir, "test");
+        if (feTest)
+            qualitySteps.push(feTest);
+        const feBuild = runIfScript(frontendDir, "build");
+        if (feBuild)
+            qualitySteps.push(feBuild);
+    }
     qualitySteps.push(advancedQualityCheck(appDir, context));
     if (qualitySteps.length === 0) {
         qualitySteps.push(basicQualityCheck(appDir));

package/dist/commands/hello.js

@@ -418,7 +418,7 @@ async function runHello(input, runQuestions) {
                 if (!lifecycleDisabled && !lifecycle.qualityPassed) {
                     const appDir = path_1.default.join(projectRoot, "generated-app");
                     const parsedAttempts = Number.parseInt(process.env.SDD_AI_REPAIR_MAX_ATTEMPTS ?? "", 10);
-                    const maxRepairAttempts = Number.isFinite(parsedAttempts) && parsedAttempts > 0 ? parsedAttempts : 6;
+                    const maxRepairAttempts = Number.isFinite(parsedAttempts) && parsedAttempts > 0 ? parsedAttempts : 10;
                     printWhy("Quality gates failed. Attempting AI repair iterations.");
                     lifecycle.qualityDiagnostics.forEach((issue) => printWhy(`Quality issue: ${issue}`));
                     for (let attempt = 1; attempt <= maxRepairAttempts && !lifecycle.qualityPassed; attempt += 1) {

package/package.json

@@ -1,7 +1,7 @@
 {
   "name": "sdd-cli",
-  "version": "0.1.24",
-  "description": "AI-orchestrated specification-driven delivery CLI that plans, validates, and ships production-ready software projects.",
+  "version": "0.1.26",
+  "description": "AI-orchestrated specification-driven delivery CLI that plans, validates, and ships production-ready software projects.",
   "keywords": [
     "cli",
     "specification-driven-development",
@@ -25,11 +25,11 @@
   "bugs": {
     "url": "https://github.com/jdsalasca/sdd-tool/issues"
   },
-  "bin": {
-    "sdd-cli": "dist/cli.js",
-    "sdd": "dist/cli.js",
-    "sdd-tool": "dist/cli.js"
-  },
+  "bin": {
+    "sdd-cli": "dist/cli.js",
+    "sdd": "dist/cli.js",
+    "sdd-tool": "dist/cli.js"
+  },
   "main": "dist/cli.js",
   "types": "dist/cli.d.ts",
   "files": [