sproutkit-code 0.16.0 → 0.17.0

package/dist/index.js

@@ -30557,6 +30557,147 @@ var require_llm = __commonJS({
   }
 });
 
+// ../desktop/dist/main/lib/qa.js
+var require_qa = __commonJS({
+  "../desktop/dist/main/lib/qa.js"(exports2) {
+    "use strict";
+    Object.defineProperty(exports2, "__esModule", { value: true });
+    exports2.staticChecks = staticChecks;
+    exports2.reviewWithClaude = reviewWithClaude;
+    exports2.fixIssues = fixIssues;
+    var llm_1 = require_llm();
+    function staticChecks(args) {
+      const { code, photoCount, phone, businessName } = args;
+      const issues = [];
+      const deadHrefs = code.match(/href\s*=\s*["'`]\s*#\s*["'`]/g)?.length ?? 0;
+      if (deadHrefs > 0) {
+        issues.push({
+          severity: "warn",
+          code: "dead_href",
+          detail: `${deadHrefs} placeholder href="#" link(s) \u2014 nav/CTAs should point somewhere real (tel:, /book, or a section id).`
+        });
+      }
+      if (/lorem ipsum|todo:|placeholder text/i.test(code)) {
+        issues.push({
+          severity: "error",
+          code: "placeholder_copy",
+          detail: "Placeholder copy (lorem ipsum / TODO) left in the page."
+        });
+      }
+      if (businessName && !code.includes(businessName.split(" ")[0])) {
+        issues.push({
+          severity: "error",
+          code: "missing_business_name",
+          detail: `The business name ("${businessName}") doesn't appear in the page.`
+        });
+      }
+      if (phone && !code.includes("tel:")) {
+        issues.push({
+          severity: "warn",
+          code: "missing_phone_cta",
+          detail: "A phone number is on file but there's no tel: link to call it."
+        });
+      }
+      const referenced = new Set((code.match(/\/photo(\d+)\.jpg/g) ?? []).map((m) => Number(m.match(/\d+/)[0])));
+      const overRef = [...referenced].filter((n) => n > photoCount);
+      if (overRef.length > 0) {
+        issues.push({
+          severity: "error",
+          code: "broken_image_ref",
+          detail: `References /photo${overRef.join(", /photo")}.jpg but only ${photoCount} photo(s) were shipped \u2014 those images will be broken.`
+        });
+      }
+      if (photoCount > 0 && referenced.size === 0 && !code.includes("PHOTOS")) {
+        issues.push({
+          severity: "warn",
+          code: "unused_photos",
+          detail: `${photoCount} real photo(s) were shipped but the page uses none of them.`
+        });
+      }
+      if (/>\s*(Opening Hours|Hours)\s*<\/[a-z0-9]+>\s*<\/(div|section)>/i.test(code)) {
+        issues.push({
+          severity: "warn",
+          code: "empty_section",
+          detail: "An Opening Hours heading appears with no content under it \u2014 omit the section instead of rendering it empty."
+        });
+      }
+      return issues;
+    }
+    async function reviewWithClaude(args) {
+      const prompt = `You are reviewing a generated single-page React site (App.jsx) for a real local business before it's shown to the owner. Find only genuine DEFECTS \u2014 things that would look broken, unfinished, or wrong to that owner.
+
+Business: "${args.businessName}" (${args.businessType})
+
+Look for: sections that render empty or with placeholder content; copy that doesn't match this business/industry; links that go nowhere; text that's cut off mid-sentence; duplicated sections; contact details that contradict each other; obviously broken layout (e.g. a grid with one item, overlapping fixed elements).
+Do NOT report style preferences, colour choices, or "could be improved" suggestions. Only defects.
+
+Respond with ONLY compact JSON, no prose:
+{"issues":[{"severity":"error"|"warn","code":"short_slug","detail":"one sentence"}]}
+If there are no real defects, return {"issues":[]}.
+
+App.jsx:
+\`\`\`jsx
+${args.code.slice(0, 45e3)}
+\`\`\``;
+      try {
+        const res = await (0, llm_1.generateText)({
+          apiKey: args.apiKey,
+          prompt,
+          maxTokens: 2048,
+          timeoutMs: 18e4
+        });
+        const raw = res.text.replace(/^```(?:json)?\s*/i, "").replace(/\s*```\s*$/, "").trim();
+        const match = raw.match(/\{[\s\S]*\}/);
+        if (!match)
+          return [];
+        const parsed = JSON.parse(match[0]);
+        if (!Array.isArray(parsed.issues))
+          return [];
+        return parsed.issues.filter((i) => !!i && typeof i.detail === "string").slice(0, 12).map((i) => ({
+          severity: i.severity === "error" ? "error" : "warn",
+          code: String(i.code || "issue").slice(0, 40),
+          detail: String(i.detail).slice(0, 300)
+        }));
+      } catch {
+        return [];
+      }
+    }
+    async function fixIssues(args) {
+      if (args.issues.length === 0)
+        return null;
+      const list = args.issues.map((i, n) => `${n + 1}. [${i.severity}] ${i.detail}`).join("\n");
+      const prompt = `Fix these specific defects in the React component below. Change ONLY what's needed to fix them \u2014 do not redesign, restyle, rewrite copy that isn't faulty, or remove working sections.
+
+Defects:
+${list}
+
+Rules: keep the same design language and structure; if a section can't be filled with real content, remove the section entirely rather than leaving it empty; never invent image URLs (only use the existing PHOTOS entries); the file must remain a single component ending in \`export default App;\`.
+
+Return ONLY the complete corrected App.jsx \u2014 no markdown fences, no commentary.
+
+\`\`\`jsx
+${args.code.slice(0, 45e3)}
+\`\`\``;
+      try {
+        const res = await (0, llm_1.generateText)({
+          apiKey: args.apiKey,
+          prompt,
+          maxTokens: 64e3,
+          timeoutMs: 3e5
+        });
+        const cleaned = res.text.replace(/^```[a-z]*\s*/i, "").replace(/\s*```$/, "").trim();
+        if (!cleaned.includes("export default App"))
+          return null;
+        if (cleaned.length < args.code.length * 0.5)
+          return null;
+        return cleaned;
+      } catch {
+        return null;
+      }
+    }
+  }
+});
+
 // ../desktop/dist/main/lib/models.js
 var require_models = __commonJS({
   "../desktop/dist/main/lib/models.js"(exports2) {
@@ -79518,6 +79659,7 @@ var require_builder = __commonJS({
     var runtime_1 = require_runtime();
     var db_1 = require_db();
     var llm_1 = require_llm();
+    var qa_1 = require_qa();
     var fs3 = __importStar(require("fs"));
     var path3 = __importStar(require("path"));
     var models_1 = require_models();
@@ -79760,7 +79902,7 @@ var require_builder = __commonJS({
       }
     }
     async function buildAndDeploySite(args) {
-      const { lead, anthropicKey, vercelToken, placesKey, customInstructions, onProgress, includeCms, blobToken, reuseAdminPassword, skipContentExtraction, multipage, pages } = args;
+      const { lead, anthropicKey, vercelToken, placesKey, customInstructions, onProgress, includeCms, blobToken, reuseAdminPassword, skipContentExtraction, multipage, pages, skipQa } = args;
       const pageList = multipage && (!pages || pages.length === 0) ? ["Home", "Services", "About", "Contact"] : pages ?? [];
       if (includeCms && !blobToken) {
         throw new Error("CMS is enabled but no Vercel Blob token is set. Add it in Settings \u2192 Vercel Blob token.");
@@ -80073,6 +80215,35 @@ ${prompt}` : prompt;
         }
       }
       appCode = appCode.replace("REPLACE_PHOTOS", JSON.stringify(photoPaths));
+      if (!skipQa) {
+        onProgress("Reviewing the site for defects\u2026");
+        const staticIssues = (0, qa_1.staticChecks)({
+          code: appCode,
+          photoCount: photoFiles.length,
+          phone: lead.phone ?? "",
+          businessName: lead.name
+        });
+        const modelIssues = await (0, qa_1.reviewWithClaude)({
+          code: appCode,
+          businessName: lead.name,
+          businessType: lead.type ?? "local business",
+          apiKey: anthropicKey
+        });
+        const issues = [...staticIssues, ...modelIssues];
+        if (issues.length > 0) {
+          const errors = issues.filter((i) => i.severity === "error").length;
+          onProgress(`QA found ${issues.length} issue${issues.length === 1 ? "" : "s"}${errors ? ` (${errors} serious)` : ""} \u2014 fixing\u2026`);
+          const fixed = await (0, qa_1.fixIssues)({ code: appCode, issues, apiKey: anthropicKey });
+          if (fixed) {
+            appCode = fixed;
+            onProgress("QA fixes applied.");
+          } else {
+            onProgress("QA fix didn't come back clean \u2014 deploying the original.");
+          }
+        } else {
+          onProgress("QA passed.");
+        }
+      }
       onProgress("Deploying to Vercel...");
       const indexHtml = `<!DOCTYPE html>
 <html lang="en"><head><meta charset="UTF-8"><meta name="viewport" content="width=device-width,initial-scale=1"><title>${lead.name}</title></head>
@@ -92918,6 +93089,10 @@ var require_chat_tools = __commonJS({
             multipage: {
               type: "boolean",
               description: "When true, build a MULTI-PAGE site (Home/Services/About/Contact as real routed pages with a nav) instead of a single long-scroll page. Use when the user asks for a multipage site or a bigger site with separate pages."
+            },
+            skipQa: {
+              type: "boolean",
+              description: "Skip the post-build QA review (a review + fix pass that catches broken links, empty sections, broken image refs). Only set true when the user explicitly wants maximum speed/volume over quality."
             }
           },
           required: ["id"]
@@ -93047,6 +93222,7 @@ var require_chat_tools = __commonJS({
           customInstructions: args.instructions,
           includeCms: args.includeCms,
           multipage: args.multipage,
+          skipQa: args.skipQa,
           blobToken,
           onProgress: (msg) => args.ctx.emitProgress?.({
             channel: "sites:progress",
@@ -93339,6 +93515,7 @@ Interested?`;
             instructions: input.instructions,
             includeCms: !!input.includeCms,
             multipage: !!input.multipage,
+            skipQa: input.skipQa === true,
             ctx
           });
           return { ...result, id };
@@ -93356,6 +93533,7 @@ Interested?`;
             id,
             instructions,
             multipage: !!input.multipage,
+            skipQa: input.skipQa === true,
             ctx
           });
           return { ...result, id };
@@ -94419,7 +94597,7 @@ var require_package2 = __commonJS({
   "package.json"(exports2, module2) {
     module2.exports = {
       name: "sproutkit-code",
-      version: "0.16.0",
+      version: "0.17.0",
       description: "Sproutkit in your terminal \u2014 find leads, build sites, run outreach, all via chat.",
       bin: {
         sproutkit: "./dist/index.js"
@@ -95576,7 +95754,8 @@ async function cmdBuild(argv) {
         id,
         instructions: flags.instructions ? String(flags.instructions) : void 0,
         includeCms: !!flags.cms,
-        multipage: !!flags.multipage
+        multipage: !!flags.multipage,
+        skipQa: !!flags["skip-qa"]
       },
       makeCtx({ json, status })
     );
@@ -95661,7 +95840,7 @@ async function cmdRegenerate(argv) {
   try {
     const result = await runTool(
       "regenerate_site",
-      { id, instructions, multipage: !!flags.multipage },
+      { id, instructions, multipage: !!flags.multipage, skipQa: !!flags["skip-qa"] },
       makeCtx({ json, status })
     );
     if (result && result.ok === false) {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "sproutkit-code",
-  "version": "0.16.0",
+  "version": "0.17.0",
   "description": "Sproutkit in your terminal — find leads, build sites, run outreach, all via chat.",
   "bin": {
     "sproutkit": "./dist/index.js"