@algolia/wizard 0.100.0 → 0.102.0

package/dist/main.js

@@ -2447,7 +2447,7 @@ async function ensureApplication(resumeAppId) {
 // package.json
 var package_default = {
   name: "@algolia/wizard",
-  version: "0.100.0",
+  version: "0.102.0",
   description: "Magically implement Algolia functionality in your codebase",
   type: "module",
   engines: {
@@ -3075,7 +3075,6 @@ var selectIndexStep = async (ctx) => {
 // src/lib/agent.ts
 import {
   ToolLoopAgent,
-  hasToolCall,
   Output as Output3,
   APICallError,
   NoOutputGeneratedError,
@@ -3167,6 +3166,22 @@ function resolveInRoot(ctx, path) {
   }
   return { ok: true, target };
 }
+function isPackageManifest(parts) {
+  if (parts.at(-1) !== "package.json") return false;
+  return parts.length === 2 || parts.length === 3 && parts[0].startsWith("@");
+}
+var DEPENDENCY_INTERNALS_REFUSAL = "Refused: do not explore node_modules. Use the Algolia SDK reference and the type checker output. To check an installed version, read node_modules/<package>/package.json.";
+function dependencyInternalsRefusal(ctx, target) {
+  if (!ctx.dependencyManifestsOnly) return null;
+  const parts = relative(ctx.root, target).split(sep);
+  const index = parts.lastIndexOf("node_modules");
+  if (index === -1 || isPackageManifest(parts.slice(index + 1))) return null;
+  return DEPENDENCY_INTERNALS_REFUSAL;
+}
+var DEPENDENCY_INTERNALS_PATH = /node_modules\/(?!\.bin\/|(?:@[^/\s'"`]+\/)?[^/\s'"`]+\/package\.json\b)/;
+function commandNamesDependencyInternals(ctx, command) {
+  return Boolean(ctx.dependencyManifestsOnly) && DEPENDENCY_INTERNALS_PATH.test(command);
+}
 async function hasSymlinkParent(ctx, target) {
   let current = ctx.root;
   const parts = relative(ctx.root, dirname2(target)).split(sep).filter(Boolean);
@@ -3230,6 +3245,8 @@ function listFilesTool(ctx) {
       }
       const resolved2 = resolveInRoot(ctx, path);
       if (!resolved2.ok) return resolved2.error;
+      const refusal = dependencyInternalsRefusal(ctx, resolved2.target);
+      if (refusal) return refusal;
       try {
         const entries = await readdir(resolved2.target, { withFileTypes: true });
         return entries.map((e) => e.isDirectory() ? `${e.name}/` : e.name).join("\n");
@@ -3271,6 +3288,19 @@ function changeDirectoryTool(ctx) {
 // src/lib/tools/reportStatus.ts
 import { tool as tool3 } from "ai";
 import z8 from "zod";
+var PLACEHOLDER_REPORT_REFUSAL = 'Refused: reportStatus got the placeholder text "placeholder" instead of a real result. The task is not finished. Continue the task, then call reportStatus with the real status, reason, and output.';
+var PLACEHOLDER_VALUE = /^\s*placeholder\s*$/i;
+function isPlaceholder(value) {
+  return typeof value === "string" && PLACEHOLDER_VALUE.test(value);
+}
+function hasPlaceholder(reason, output) {
+  if (isPlaceholder(reason)) return true;
+  if (typeof output !== "object" || output === null) return false;
+  return Object.values(output).some(isPlaceholder);
+}
+function isAcceptedReport(result) {
+  return typeof result === "object" && result !== null && "status" in result && !("rejected" in result);
+}
 function reportStatusTool(output) {
   return tool3({
     description: "Report the status of your execution. Return a reason in case of failure.",
@@ -3280,6 +3310,10 @@ function reportStatusTool(output) {
       output
     }),
     execute: async ({ status, reason, output: output2 }) => {
+      if (hasPlaceholder(reason, output2)) {
+        logger.warn({ status, reason }, "reportStatus: refused a placeholder report");
+        return { rejected: true, message: PLACEHOLDER_REPORT_REFUSAL };
+      }
       logger.info({ status, reason }, "called reportStatus tool");
       return { status, reason, output: output2 };
     }
@@ -3379,6 +3413,8 @@ function readFileTool(ctx) {
       logger.info({ filePath, readLines }, "called readFile tool");
       const resolved2 = resolveInRoot(ctx, filePath);
       if (resolved2.ok === false) return resolved2.error;
+      const refusal = dependencyInternalsRefusal(ctx, resolved2.target);
+      if (refusal) return refusal;
       try {
         const content = await readFile4(resolved2.target, "utf8");
         const safeContent = isEnvFile(resolved2.target) ? redactEnvValues(content) : content;
@@ -4323,6 +4359,8 @@ function searchFilesTool(ctx) {
       }
       const resolved2 = resolveInRoot(ctx, path);
       if (!resolved2.ok) return resolved2.error;
+      const refusal = dependencyInternalsRefusal(ctx, resolved2.target);
+      if (refusal) return refusal;
       try {
         await stat2(resolved2.target);
       } catch (err) {
@@ -4347,7 +4385,9 @@ function searchFilesTool(ctx) {
       let workerFailed = false;
       try {
         try {
-          const files = await searchableFiles(ctx.root, resolved2.target);
+          const files = (await searchableFiles(ctx.root, resolved2.target)).filter(
+            (file) => !dependencyInternalsRefusal(ctx, file)
+          );
           const rootPath = await realpath(ctx.root);
           for (const [fileIndex, file] of files.entries()) {
             const moreFilesRemain = fileIndex < files.length - LINE_NUMBER_OFFSET;
@@ -5132,6 +5172,9 @@ function runShellTool(ctx, createModel = defaultCreateModel) {
       if (isFileIOCommand(command)) {
         return "Refused: use listFiles or searchFiles to find files, and readFile to read one, instead of a shell file-listing, reading, or search command (ls/find/fd/tree/dir/locate/cat/head/tail/less/more/tac/nl/grep/egrep/fgrep/rg/ack/ag/xxd/hexdump/od/strings).";
       }
+      if (commandNamesDependencyInternals(ctx, command)) {
+        return DEPENDENCY_INTERNALS_REFUSAL;
+      }
       logger.info({ command, cwd: resolved2.target }, "called runShell tool");
       const reviewed = await serializePrompt(
         () => runReviewedCommand(ctx, command, resolved2.target)
@@ -5444,6 +5487,14 @@ function withCachedHistory(messages) {
     }
   ];
 }
+function isAcceptedReportResult(result) {
+  return result.toolName === "reportStatus" && isAcceptedReport(result.output);
+}
+function reportAccepted({
+  steps
+}) {
+  return steps.at(-1)?.toolResults?.some(isAcceptedReportResult) ?? false;
+}
 function isMissingReportRetry(kind) {
   return kind === "missingReport" || incompleteReasonOf(kind) !== void 0;
 }
@@ -5563,10 +5614,7 @@ async function runAgentAttempt(req, attempt, retryReason) {
       tools: req.tools
     }),
     toolChoice: "required",
-    stopWhen: [
-      hasToolCall("reportStatus"),
-      stepCountIs(maxSteps)
-    ],
+    stopWhen: [reportAccepted, stepCountIs(maxSteps)],
     prepareStep: ({ stepNumber, messages }) => ({
       messages: withCachedHistory(messages),
       ...stepNumber >= maxSteps - 1 && {
@@ -5648,7 +5696,7 @@ async function runAgentAttempt(req, attempt, retryReason) {
     "tool usage"
   );
   const toolResults = await stream.toolResults;
-  const report = [...toolResults].reverse().find((r) => r.toolName === "reportStatus");
+  const report = [...toolResults].reverse().find(isAcceptedReportResult);
   if (!report) {
     logger.error(
       {
@@ -6995,7 +7043,8 @@ function ingestionRetryInstruction(reason, detail) {
 }
 function validationRetryInstructions(validation) {
   return [
-    `Implementation insufficient. Address these findings before reporting completion: ${validation.additionalInstructions ?? validation.summary}`
+    `Implementation insufficient. Address these findings before reporting completion: ${validation.additionalInstructions ?? validation.summary}`,
+    "Apply these fixes first, with the fewest edits. When a finding names a fix, apply that fix instead of studying the API again."
   ];
 }
 async function pathExists(path) {
@@ -7067,7 +7116,7 @@ function validationRunInstructions(plan, attempt) {
 function retryTypeCheckInstructions(plan) {
   if (!plan.run) return [];
   return [
-    "The developer approved the type check. After your edits, run it once with the project's existing command. Do not install dependencies, use npx, dlx, or another package runner, or use an autofix option. Fix the errors that your changes cause, and name any error that remains in the summary."
+    "The developer approved the type check. Run it once with the project's existing command right after you apply the fixes, before other work. Do not install dependencies, use npx, dlx, or another package runner, or use an autofix option. Fix the errors that your changes cause, and name any error that remains in the summary."
   ];
 }
 async function resolveIngestionSource(ctx, repoRoot) {
@@ -7238,7 +7287,7 @@ async function implement(ctx, useCases = DEFAULT_IMPLEMENT_USE_CASES) {
   async function runImplementationUseCase(currentUseCase, extraInstructions = [], isRetry = false) {
     if (agentRuns > 0) ctx.recordStepExecution();
     agentRuns += 1;
-    const toolContext = currentUseCase === "ingestion" ? ingestionTools : makeToolContext(repoRoot);
+    const toolContext = currentUseCase === "ingestion" ? ingestionTools : { ...makeToolContext(repoRoot), dependencyManifestsOnly: true };
     try {
       return await runAgent({
         operation: `${currentUseCase}-implementation`,
@@ -7283,7 +7332,7 @@ async function implement(ctx, useCases = DEFAULT_IMPLEMENT_USE_CASES) {
       tools: toolsForUseCase("validation"),
       outputSchema: validationOutputSchema,
       modelProfile: "validation" /* validation */,
-      toolContext: makeToolContext(repoRoot)
+      toolContext: { ...makeToolContext(repoRoot), dependencyManifestsOnly: true }
     });
   }
   if (useCases.includes("ingestion")) {
@@ -10024,7 +10073,7 @@ async function resetProjectState() {
 // package.json with { type: 'json' }
 var package_default2 = {
   name: "@algolia/wizard",
-  version: "0.100.0",
+  version: "0.102.0",
   description: "Magically implement Algolia functionality in your codebase",
   type: "module",
   engines: {

package/docs/algolia-sdk/instantsearch-setup-js.md

@@ -95,6 +95,8 @@ Ensure the attributes you map into the component match the entity. 'title' above
 - The `item` template renders content only: **no `<a>`, no `href`, no click handler of
   your own, never `stopPropagation`.** The widget's `<li>` owns the click — it sends the
   Insights `click`, closes the panel, clears the active option.
+- Do not pass a top-level `onSelect` to `autocomplete()` to navigate. `getURL` already
+  navigates.
 - `getSearchPageURL` sends Enter with nothing highlighted to a full results page; pass it
   only if the app has one. `.getQuery` is optional.
 - TypeScript: type records through the generic, not casts inside templates —

package/docs/algolia-sdk/instantsearch-setup-react.md

@@ -87,6 +87,9 @@ Ensure the attributes you map into the component match the entity. 'title' above
 - `itemComponent` renders content only: **no `<a>`, no `href`, no `onClick` beyond the
   `onSelect` it is handed, never `stopPropagation`.** The widget's `<li>` owns the click —
   it sends the Insights `click`, closes the panel, clears the active option.
+- Do not pass a top-level `onSelect` to `<Autocomplete>` (for example to call
+  `router.push`). `getURL` already navigates. The top-level callback can receive a
+  `SyntheticEvent` instead of `{ item }`, so reading `item` from it fails the type check.
 - `getSearchPageURL` sends Enter with nothing highlighted to a full results page; pass it
   only if the app has one. `.getQuery` is optional.
 - TypeScript: type records through the generic, not casts inside `itemComponent` —

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@algolia/wizard",
-  "version": "0.100.0",
+  "version": "0.102.0",
   "description": "Magically implement Algolia functionality in your codebase",
   "type": "module",
   "engines": {