@algolia/wizard 0.64.0 → 0.66.0

package/dist/main.js

@@ -1760,7 +1760,7 @@ function identify(traits) {
 // package.json
 var package_default = {
   name: "@algolia/wizard",
-  version: "0.64.0",
+  version: "0.66.0",
   description: "Magically implement Algolia functionality in your codebase",
   type: "module",
   engines: {
@@ -1804,17 +1804,17 @@ var package_default = {
     "lint-staged": "^17.0.8",
     tsx: "^4.22.3",
     typescript: "^6.0.3",
-    vitest: "^4.1.8"
+    vitest: "^4.1.11"
   },
   dependencies: {
-    "@ai-sdk/anthropic": "^3.0.81",
-    "@ai-sdk/openai-compatible": "^2.0.47",
+    "@ai-sdk/anthropic": "^3.0.118",
+    "@ai-sdk/openai-compatible": "^2.0.75",
     "@hono/node-server": "^2.0.10",
     "@segment/analytics-node": "^3.1.0",
-    ai: "^6.0.190",
+    ai: "^6.0.283",
     "cross-keychain": "^1.1.0",
     dotenv: "^17.4.2",
-    hono: "^4.12.27",
+    hono: "^4.13.8",
     ink: "^7.0.5",
     "ink-picture": "^2.1.0",
     "ink-spinner": "^5.0.0",
@@ -4567,7 +4567,7 @@ ${formatCompletedSteps(ctx.completedSteps)}`,
 
 // src/actions/implement.ts
 import z28 from "zod";
-import { mkdir as mkdir5, readFile as readFile8 } from "node:fs/promises";
+import { access, mkdir as mkdir5, readFile as readFile8 } from "node:fs/promises";
 import { join as join10, relative as relative6 } from "node:path";
 
 // src/lib/git.ts
@@ -4732,12 +4732,13 @@ var implementationOutputSchema = z28.object({
   // won't end up gitignored (it's public, meant to be committed).
   searchConfigFile: z28.string().optional()
 });
-var verificationOutputSchema = z28.object({
+var validationOutputSchema = z28.object({
   summary: z28.string(),
   sufficient: z28.boolean(),
-  additionalInstructions: z28.string().optional()
+  additionalInstructions: z28.string().optional(),
+  unrelatedFailure: z28.string().optional()
 });
-var MAX_IMPLEMENT_VERIFICATION_ATTEMPTS = 3;
+var MAX_IMPLEMENT_VALIDATION_ATTEMPTS = 3;
 var DEFAULT_IMPLEMENT_USE_CASES = ["ingestion", "search"];
 var INGEST_DIR = ".algolia-wizard";
 var INGESTION_SOURCE_PROMPT = "What data do you want to index?";
@@ -4746,6 +4747,9 @@ var INGESTION_SOURCE = {
   fileUpload: "Import a data file (CSV, JSON, TSV)",
   generated: "Generate sample data"
 };
+var BUILD_CHECK_PROMPT = "Run type-check and lint now? This can take several minutes on a large repository.";
+var MISSING_DEPENDENCIES_PROMPT = "Dependencies are not installed. Install them and run type-check and lint now?";
+var UNRELATED_FAILURE_PROMPT = "These checks are still failing and appear unrelated to the changes. Continue the investigation?";
 var FILE_UPLOAD_PATH_PROMPT = "Enter the path to your JSON, CSV, or TSV file (relative to the project root, or absolute):";
 var JS_LANGUAGES = ["javascript", "typescript", "jsx", "tsx", "node"];
 function lower(entries) {
@@ -4870,19 +4874,19 @@ function searchInstructions(input) {
     "The summary should be extremely concise; do not mention manual testing steps."
   ];
 }
-function verificationInstructions(input) {
+function validationInstructions(input) {
   return [
-    "Verify the Algolia implementation changes.",
-    `Verification tools found in the codebase: ${JSON.stringify(input.findings.verification ?? [])}.`,
-    "Run the project's own checks (lint, type check, tests) via runShell, using the project's commands \u2014 its task runner, manifest scripts, etc. Run every check that applies, not just the first.",
-    "If a check fails because packages or modules are missing, install the dependencies via runShell and re-run it rather than changing the code.",
-    "For issues caused by the implementation, make minimal fixes with writeFile and re-run the checks.",
-    "Do not make speculative fixes when no checks exist, a check cannot run, or failures are unrelated to these changes \u2014 note the limitation in your summary.",
-    "Do not add new Algolia functionality here; only validate and make minimal correctness fixes.",
-    `Do not modify "${input.ingestDir}/" unless a check reports an actionable issue in its files.`,
-    "Always call reportStatus with status=success once verification has run, even when sufficient=false.",
-    "Set sufficient=true only when the implementation is complete and checks pass (or fail for a clearly unrelated reason).",
-    "Set sufficient=false when the implementation is incomplete or has implementation-caused failures; include concrete additionalInstructions for the next pass."
+    "Validate the Algolia search implementation by inspecting the project source.",
+    "Confirm that the application renders exactly one search bar. Check for an old search bar and duplicate mounts.",
+    `Confirm that the search bar is mounted in ${input.searchLocation ? `"${input.searchLocation}"` : "the best always-rendered shared layout location"}.`,
+    "Confirm that the search component is reachable across the application and fits the surrounding layout.",
+    "Do not run the test suite or any test command.",
+    "Do not modify files. Report each implementation issue as a concrete instruction for the next implementation pass.",
+    `Available type-check and lint tools: ${JSON.stringify(input.findings.verification ?? [])}.`,
+    "Always call reportStatus with status=success after validation, even when sufficient=false.",
+    "Set sufficient=true only when the search bar is unique, correctly placed, and structurally complete.",
+    "Set sufficient=false for an implementation issue. Include concrete additionalInstructions for the next pass.",
+    "If a check fails only in untouched code, set unrelatedFailure to a concise failure description."
   ];
 }
 var IMPLEMENT_CONFIG = {
@@ -4894,9 +4898,9 @@ var IMPLEMENT_CONFIG = {
     title: "Algolia search",
     buildInstructions: searchInstructions
   },
-  verification: {
-    title: "Algolia verification",
-    buildInstructions: verificationInstructions
+  validation: {
+    title: "Algolia validation",
+    buildInstructions: validationInstructions
   }
 };
 var useCaseToolMap = {
@@ -4909,7 +4913,7 @@ var useCaseToolMap = {
     "notifyUser"
   ],
   search: [...FS_READ_TOOLS, "writeFile", "runShell", "notifyUser"],
-  verification: [...FS_READ_TOOLS, "writeFile", "runShell", "notifyUser"]
+  validation: [...FS_READ_TOOLS, "runShell", "notifyUser"]
 };
 function toolsForUseCase(useCase, ingestionSource) {
   const tools = useCaseToolMap[useCase];
@@ -4929,7 +4933,7 @@ function buildAgentInstructions(useCase, input, extraInstructions = []) {
   ];
 }
 function formatSummary(useCase, summary) {
-  const label = useCase === "ingestion" ? "Ingestion" : useCase === "search" ? "Search" : "Verification";
+  const label = useCase === "ingestion" ? "Ingestion" : useCase === "search" ? "Search" : "Validation";
   return `${label}: ${summary}`;
 }
 function parseIngestRecordCount(output) {
@@ -5021,9 +5025,62 @@ function makeToolContext(root, env = async () => ({})) {
     createShellContext({ env, approve: storeApproval(root) })
   );
 }
-function verificationRetryInstructions(verification) {
+function validationRetryInstructions(validation) {
   return [
-    `Implementation insufficient. Address these findings before reporting completion: ${verification.additionalInstructions ?? verification.summary}`
+    `Implementation insufficient. Address these findings before reporting completion: ${validation.additionalInstructions ?? validation.summary}`
+  ];
+}
+async function pathExists(path) {
+  try {
+    await access(path);
+    return true;
+  } catch {
+    return false;
+  }
+}
+async function resolveBuildCheckPlan(ctx, repoRoot, input) {
+  if (!input.findings.verification?.length) {
+    return { run: false, installDependencies: false };
+  }
+  const installDependencies = isJsProject(input.language) && !await pathExists(join10(repoRoot, "node_modules"));
+  const accepted = Boolean(
+    await ctx.requestUserInput({
+      prompt: installDependencies ? MISSING_DEPENDENCIES_PROMPT : BUILD_CHECK_PROMPT,
+      promptType: "acceptReject",
+      options: ["Yes", "No"],
+      messages: []
+    })
+  );
+  return {
+    run: accepted,
+    installDependencies: accepted && installDependencies
+  };
+}
+function installCheckInstructions(plan, attempt) {
+  if (!plan.installDependencies) {
+    return [];
+  }
+  if (attempt === 1) {
+    return [
+      "Install dependencies once with the project's declared package manager before any check.",
+      "Do not try alternate install commands. If installation fails, stop the checks and report the limitation."
+    ];
+  }
+  return ["Dependencies were already installed. Do not install them again."];
+}
+function validationRunInstructions(plan, changedFiles, attempt) {
+  if (!plan.run) {
+    return [
+      "The developer chose to skip build checks. Do not install dependencies or run shell checks."
+    ];
+  }
+  return [
+    "The developer approved the optional build checks.",
+    ...installCheckInstructions(plan, attempt),
+    "Do not use npx, dlx, or another package runner that can download an undeclared tool.",
+    "Run the type check once with the project's existing command, when one is available.",
+    `Lint only these changed files: ${JSON.stringify(changedFiles)}. Do not run a full-project lint. If this list is empty, do not run lint.`,
+    "Do not use an autofix option during validation."
   ];
 }
 async function resolveIngestionSource(ctx, repoRoot) {
@@ -5192,13 +5249,17 @@ async function implement(ctx, useCases = DEFAULT_IMPLEMENT_USE_CASES) {
       toolContext: currentUseCase === "ingestion" ? ingestionTools ?? searchTools : searchTools
     });
   }
-  async function runVerificationUseCase() {
+  async function runValidationUseCase(extraInstructions) {
     if (agentRuns > 0) ctx.recordStepExecution();
     agentRuns += 1;
     return runAgent({
-      instructions: buildAgentInstructions("verification", input),
-      tools: toolsForUseCase("verification"),
-      outputSchema: verificationOutputSchema,
+      instructions: buildAgentInstructions(
+        "validation",
+        input,
+        extraInstructions
+      ),
+      tools: toolsForUseCase("validation"),
+      outputSchema: validationOutputSchema,
       toolContext: searchTools
     });
   }
@@ -5307,6 +5368,7 @@ ${detail}` : ""}`
   let searchConfigFile;
   if (useCases.includes("search")) {
     let extraInstructions = [];
+    let buildCheckPlan;
     const confirmedFrameworks = language.frameworks ?? [];
     const detectedFrameworks = scan.frameworks ?? [];
     const framework = confirmedFrameworks[0]?.name ?? "unknown";
@@ -5331,15 +5393,15 @@ ${detail}` : ""}`
       });
     };
     useWizard.getState().clearWrittenFiles();
-    for (let attempt = 1; attempt <= MAX_IMPLEMENT_VERIFICATION_ATTEMPTS; attempt++) {
+    for (let attempt = 1; attempt <= MAX_IMPLEMENT_VALIDATION_ATTEMPTS; attempt++) {
       if (attempt > 1) {
         logger.info(
           {
             attempt,
-            maxAttempts: MAX_IMPLEMENT_VERIFICATION_ATTEMPTS,
+            maxAttempts: MAX_IMPLEMENT_VALIDATION_ATTEMPTS,
             extraInstructions
           },
-          "implement: retrying search implementation after failed verification"
+          "implement: retrying search implementation after failed validation"
         );
       }
       const searchResult = await runImplementationUseCase(
@@ -5350,21 +5412,55 @@ ${detail}` : ""}`
       if (searchResult.searchConfigFile) {
         searchConfigFile = searchResult.searchConfigFile;
       }
-      const verification = await runVerificationUseCase();
-      summaries.push(formatSummary("verification", verification.summary));
-      if (verification.sufficient) {
+      buildCheckPlan ??= await resolveBuildCheckPlan(ctx, repoRoot, input);
+      const searchFilesChanged = [
+        ...new Set(useWizard.getState().writtenFiles)
+      ].map((file) => relative6(repoRoot, file));
+      const validation = await runValidationUseCase(
+        validationRunInstructions(buildCheckPlan, searchFilesChanged, attempt)
+      );
+      summaries.push(formatSummary("validation", validation.summary));
+      const finalAttempt = attempt === MAX_IMPLEMENT_VALIDATION_ATTEMPTS;
+      let investigateUnrelated = false;
+      if (validation.unrelatedFailure && !finalAttempt) {
+        investigateUnrelated = Boolean(
+          await ctx.requestUserInput({
+            prompt: UNRELATED_FAILURE_PROMPT,
+            promptType: "acceptReject",
+            options: ["Yes", "No"],
+            messages: [validation.unrelatedFailure]
+          })
+        );
+        if (!investigateUnrelated) {
+          summaries.push("Validation: Skipped unrelated check failures.");
+          if (validation.sufficient) {
+            ctx.setUserInput("implementation", "success");
+            trackComponentGenerated(true, attempt);
+            break;
+          }
+        }
+      }
+      if (validation.sufficient && (!validation.unrelatedFailure || finalAttempt)) {
+        if (validation.unrelatedFailure) {
+          summaries.push("Validation: Unrelated check failures remain.");
+        }
         ctx.setUserInput("implementation", "success");
         trackComponentGenerated(true, attempt);
         break;
       }
-      if (attempt === MAX_IMPLEMENT_VERIFICATION_ATTEMPTS) {
+      if (finalAttempt) {
         ctx.setUserInput("implementation", "fail");
         trackComponentGenerated(false, attempt);
         throw new Error(
-          `Implementation verification failed after ${MAX_IMPLEMENT_VERIFICATION_ATTEMPTS} attempts: ${verification.additionalInstructions ?? verification.summary}`
+          `Unable to validate the implementation after ${MAX_IMPLEMENT_VALIDATION_ATTEMPTS} attempts: ${validation.additionalInstructions ?? validation.unrelatedFailure ?? validation.summary}`
+        );
+      }
+      extraInstructions = validation.sufficient ? [] : validationRetryInstructions(validation);
+      if (validation.unrelatedFailure && investigateUnrelated) {
+        extraInstructions.push(
+          `The developer asked you to investigate this check failure: ${validation.unrelatedFailure}`
         );
       }
-      extraInstructions = verificationRetryInstructions(verification);
     }
     if (searchConfigFile) {
       const ignoreStatus = await gitIgnoreStatus(
@@ -7074,7 +7170,7 @@ function delay(ms) {
 // package.json with { type: 'json' }
 var package_default2 = {
   name: "@algolia/wizard",
-  version: "0.64.0",
+  version: "0.66.0",
   description: "Magically implement Algolia functionality in your codebase",
   type: "module",
   engines: {
@@ -7118,17 +7214,17 @@ var package_default2 = {
     "lint-staged": "^17.0.8",
     tsx: "^4.22.3",
     typescript: "^6.0.3",
-    vitest: "^4.1.8"
+    vitest: "^4.1.11"
   },
   dependencies: {
-    "@ai-sdk/anthropic": "^3.0.81",
-    "@ai-sdk/openai-compatible": "^2.0.47",
+    "@ai-sdk/anthropic": "^3.0.118",
+    "@ai-sdk/openai-compatible": "^2.0.75",
     "@hono/node-server": "^2.0.10",
     "@segment/analytics-node": "^3.1.0",
-    ai: "^6.0.190",
+    ai: "^6.0.283",
     "cross-keychain": "^1.1.0",
     dotenv: "^17.4.2",
-    hono: "^4.12.27",
+    hono: "^4.13.8",
     ink: "^7.0.5",
     "ink-picture": "^2.1.0",
     "ink-spinner": "^5.0.0",

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@algolia/wizard",
-  "version": "0.64.0",
+  "version": "0.66.0",
   "description": "Magically implement Algolia functionality in your codebase",
   "type": "module",
   "engines": {
@@ -44,17 +44,17 @@
     "lint-staged": "^17.0.8",
     "tsx": "^4.22.3",
     "typescript": "^6.0.3",
-    "vitest": "^4.1.8"
+    "vitest": "^4.1.11"
   },
   "dependencies": {
-    "@ai-sdk/anthropic": "^3.0.81",
-    "@ai-sdk/openai-compatible": "^2.0.47",
+    "@ai-sdk/anthropic": "^3.0.118",
+    "@ai-sdk/openai-compatible": "^2.0.75",
     "@hono/node-server": "^2.0.10",
     "@segment/analytics-node": "^3.1.0",
-    "ai": "^6.0.190",
+    "ai": "^6.0.283",
     "cross-keychain": "^1.1.0",
     "dotenv": "^17.4.2",
-    "hono": "^4.12.27",
+    "hono": "^4.13.8",
     "ink": "^7.0.5",
     "ink-picture": "^2.1.0",
     "ink-spinner": "^5.0.0",