@algolia/wizard 0.99.0 → 0.101.0

package/dist/main.js

@@ -2447,7 +2447,7 @@ async function ensureApplication(resumeAppId) {
 // package.json
 var package_default = {
   name: "@algolia/wizard",
-  version: "0.99.0",
+  version: "0.101.0",
   description: "Magically implement Algolia functionality in your codebase",
   type: "module",
   engines: {
@@ -3167,6 +3167,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 +3246,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");
@@ -3379,6 +3397,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 +4343,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 +4369,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 +5156,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)
@@ -6633,7 +6660,8 @@ function searchInstructions(input) {
     "Style the search surfaces through the widget's class and CSS-variable overrides (or, when hand-built, the app's existing theme tokens): give the results panel and the element that actually draws the input's border and background \u2014 for a widget, the form or wrapper it renders around the bare input \u2014 non-transparent surfaces, keep that frame distinguishable from the background behind it in every supported theme, and never assume a light surface or reuse the surrounding page's colors unchanged inside the panel.",
     "The panel takes its width from the input by default, so a small input makes it unreadably narrow: give it `min-width: min(320px, calc(100vw - 2rem))` independent of the input, cap its maximum width to the viewport, and anchor it to the input edge it opens from so widening does not push it off-screen.",
     "Match the application styles closely, including light, dark, and high-contrast themes the project already supports.",
-    "Before reporting, mount the component. Do not run a type check, build, or test command: the validation pass runs the type check, after the developer agrees to it. If a tool limit is reached or an error remains unresolved, keep the implementation, report status=success, and name the remaining issue in the summary so validation can drive the next pass. Report status=fail only when no search implementation can be written, with the concrete blocker as the reason.",
+    "When you add UI text to a localized dictionary or message file, add the same keys, translated, to every locale the project supports.",
+    "Before reporting, mount the component. Do not run a type check, build, or test command unless a later instruction allows it: the validation pass runs the type check, after the developer agrees to it. If a tool limit is reached or an error remains unresolved, keep the implementation, report status=success, and name the remaining issue in the summary so validation can drive the next pass. Report status=fail only when no search implementation can be written, with the concrete blocker as the reason.",
     "The summary should be extremely concise; do not mention manual testing steps."
   ];
 }
@@ -6994,7 +7022,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) {
@@ -7059,10 +7088,16 @@ function validationRunInstructions(plan, attempt) {
     "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.",
+    "Run the type check once with the project's existing command, when one is available. Run it even when you already found other failures, and put every type error that the changes cause in additionalInstructions, so the next pass can fix them all at once.",
     "Do not use an autofix option during validation."
   ];
 }
+function retryTypeCheckInstructions(plan) {
+  if (!plan.run) return [];
+  return [
+    "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) {
   let error = void 0;
   for (; ; ) {
@@ -7231,7 +7266,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`,
@@ -7276,7 +7311,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")) {
@@ -7598,6 +7633,7 @@ ${detail}` : ""}`
           `The developer asked you to investigate this check failure: ${validation.unrelatedFailure}`
         );
       }
+      extraInstructions.push(...retryTypeCheckInstructions(buildCheckPlan));
     }
     if (searchConfigFile) {
       const ignoreStatus = await gitIgnoreStatus(
@@ -10016,7 +10052,7 @@ async function resetProjectState() {
 // package.json with { type: 'json' }
 var package_default2 = {
   name: "@algolia/wizard",
-  version: "0.99.0",
+  version: "0.101.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.99.0",
+  "version": "0.101.0",
   "description": "Magically implement Algolia functionality in your codebase",
   "type": "module",
   "engines": {