@juspay/neurolink 11.21.3 → 11.21.4

package/dist/utils/redirectDispatcher.d.ts

@@ -1,6 +1,6 @@
 import type { Dispatcher } from "undici";
 /**
- * A dispatcher that follows redirects, when composing one is safe here.
+ * A dispatcher that follows redirects.
  *
  * `getGlobalDispatcher()` returns Node's **built-in** undici dispatcher, whose
  * major tracks the runtime rather than this package's dependency. The composed
@@ -10,19 +10,30 @@ import type { Dispatcher } from "undici";
  *   node 24   built-in 7.24.4 + npm 7.28.0   request() succeeds
  *   node 22   built-in 6.28.0 + npm 7.28.0   throws "invalid onError method"
  *
- * Node 22 is this package's declared minimum, so the broken combination is not
- * exotic — it is the floor. The throw happens at request time rather than at
- * compose(), which is why it surfaces as an opaque runtime error instead of
- * something recognisably about versions.
+ * Node 22 is this package's declared minimum, so the broken combination is the
+ * floor, not an edge case.
  *
- * When the majors disagree, return the global dispatcher uncomposed. That drops
- * redirect-following from the pre-flight HEAD only. Callers already treat any
- * non-2xx HEAD — a redirect included — as untrustworthy and fall through to the
- * streaming size guard on the GET, so the size protection is unchanged and the
- * cost is one extra round trip.
+ * When the majors match, compose onto the global dispatcher — that preserves
+ * whatever the host application configured globally, such as a corporate
+ * ProxyAgent.
  *
- * Composing onto the global dispatcher rather than a fresh `Agent` is
- * deliberate: it preserves whatever the host application configured globally,
- * such as a corporate ProxyAgent.
+ * When they do not, compose onto a fresh `Agent` that npm undici owns, so the
+ * handler contract is self-consistent. Redirects still get followed.
+ *
+ * The earlier version of this function returned the global dispatcher
+ * *uncomposed* in the mismatch case, on the stated grounds that it "drops
+ * redirect-following from the pre-flight HEAD only". That was wrong: the same
+ * dispatcher feeds the real GET in fileDetector.ts and messageBuilder.ts, and
+ * that path treats any non-200 as fatal — so on Node 22 a redirecting URL threw
+ * `HTTP 302 fetching …` instead of downloading. Verified against a local
+ * redirecting server: node 24 -> 200 (followed), node 22 -> 302 (not followed).
+ *
+ * The cost of the fresh Agent is narrow and worth naming: on a runtime whose
+ * built-in undici major differs from ours, a globally-configured dispatcher
+ * (e.g. a ProxyAgent set via setGlobalDispatcher) is not inherited for these
+ * requests. Losing a proxy on one Node version is recoverable; silently failing
+ * every redirecting download is not.
+ *
+ * @param maxRedirections How many redirects to follow before giving up.
  */
 export declare function redirectFollowingDispatcher(maxRedirections: number): Dispatcher;

package/dist/utils/redirectDispatcher.js

@@ -1,4 +1,4 @@
-import { getGlobalDispatcher, interceptors } from "undici";
+import { Agent, getGlobalDispatcher, interceptors } from "undici";
 /**
  * The major of the `undici` this package depends on.
  *
@@ -8,7 +8,7 @@ import { getGlobalDispatcher, interceptors } from "undici";
  */
 const NPM_UNDICI_MAJOR = 7;
 /**
- * A dispatcher that follows redirects, when composing one is safe here.
+ * A dispatcher that follows redirects.
  *
  * `getGlobalDispatcher()` returns Node's **built-in** undici dispatcher, whose
  * major tracks the runtime rather than this package's dependency. The composed
@@ -18,26 +18,36 @@ const NPM_UNDICI_MAJOR = 7;
  *   node 24   built-in 7.24.4 + npm 7.28.0   request() succeeds
  *   node 22   built-in 6.28.0 + npm 7.28.0   throws "invalid onError method"
  *
- * Node 22 is this package's declared minimum, so the broken combination is not
- * exotic — it is the floor. The throw happens at request time rather than at
- * compose(), which is why it surfaces as an opaque runtime error instead of
- * something recognisably about versions.
+ * Node 22 is this package's declared minimum, so the broken combination is the
+ * floor, not an edge case.
  *
- * When the majors disagree, return the global dispatcher uncomposed. That drops
- * redirect-following from the pre-flight HEAD only. Callers already treat any
- * non-2xx HEAD — a redirect included — as untrustworthy and fall through to the
- * streaming size guard on the GET, so the size protection is unchanged and the
- * cost is one extra round trip.
+ * When the majors match, compose onto the global dispatcher — that preserves
+ * whatever the host application configured globally, such as a corporate
+ * ProxyAgent.
  *
- * Composing onto the global dispatcher rather than a fresh `Agent` is
- * deliberate: it preserves whatever the host application configured globally,
- * such as a corporate ProxyAgent.
+ * When they do not, compose onto a fresh `Agent` that npm undici owns, so the
+ * handler contract is self-consistent. Redirects still get followed.
+ *
+ * The earlier version of this function returned the global dispatcher
+ * *uncomposed* in the mismatch case, on the stated grounds that it "drops
+ * redirect-following from the pre-flight HEAD only". That was wrong: the same
+ * dispatcher feeds the real GET in fileDetector.ts and messageBuilder.ts, and
+ * that path treats any non-200 as fatal — so on Node 22 a redirecting URL threw
+ * `HTTP 302 fetching …` instead of downloading. Verified against a local
+ * redirecting server: node 24 -> 200 (followed), node 22 -> 302 (not followed).
+ *
+ * The cost of the fresh Agent is narrow and worth naming: on a runtime whose
+ * built-in undici major differs from ours, a globally-configured dispatcher
+ * (e.g. a ProxyAgent set via setGlobalDispatcher) is not inherited for these
+ * requests. Losing a proxy on one Node version is recoverable; silently failing
+ * every redirecting download is not.
+ *
+ * @param maxRedirections How many redirects to follow before giving up.
  */
 export function redirectFollowingDispatcher(maxRedirections) {
-    const globalDispatcher = getGlobalDispatcher();
     const builtinMajor = Number.parseInt(process.versions.undici?.split(".")[0] ?? "", 10);
-    if (!Number.isFinite(builtinMajor) || builtinMajor !== NPM_UNDICI_MAJOR) {
-        return globalDispatcher;
-    }
-    return globalDispatcher.compose(interceptors.redirect({ maxRedirections }));
+    const base = Number.isFinite(builtinMajor) && builtinMajor === NPM_UNDICI_MAJOR
+        ? getGlobalDispatcher()
+        : new Agent();
+    return base.compose(interceptors.redirect({ maxRedirections }));
 }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@juspay/neurolink",
-  "version": "11.21.3",
+  "version": "11.21.4",
   "packageManager": "pnpm@10.15.1",
   "description": "TypeScript AI SDK with 24+ LLM providers behind one consistent API. MCP-native (connect any MCP server), voice TTS/STT/realtime, RAG, agents, memory, context compaction. OpenAI · Anthropic · Gemini · Bedrock · Azure · Ollama · DeepSeek · NVIDIA NIM and more.",
   "author": {
@@ -42,6 +42,7 @@
     "check": "svelte-kit sync && svelte-check --tsconfig ./tsconfig.json && tsc --noEmit --strict",
     "check:ci-scripts": "svelte-kit sync && tsc -p tsconfig.ci-scripts.json",
     "check:test-parse": "node scripts/parse-check-tests.mjs",
+    "check:tools-tests": "svelte-kit sync && tsc -p tsconfig.tools-tests.json",
     "check:watch": "svelte-kit sync && svelte-check --tsconfig ./tsconfig.json --watch",
     "typecheck": "tsc --noEmit",
     "modelServer": "tsx scripts/modelServer.ts",
@@ -475,6 +476,7 @@
     "@sveltejs/package": "^2.5.7",
     "@sveltejs/vite-plugin-svelte": "^7.0.0",
     "@types/cors": "^2.8.19",
+    "@types/diff": "^8.0.0",
     "@types/express": "^5.0.6",
     "@types/fluent-ffmpeg": "^2.1.28",
     "@types/js-yaml": "^4.0.9",
@@ -491,8 +493,10 @@
     "@vercel/ncc": "^0.38.4",
     "concurrently": "^9.2.1",
     "conventional-changelog-conventionalcommits": "^9.1.0",
+    "diff": "^9.0.0",
     "esbuild": "^0.28.1",
     "eslint": "^10.0.2",
+    "glob": "^13.0.6",
     "husky": "^9.1.7",
     "playwright": "^1.58.2",
     "prettier": "^3.8.1",