From 77662166c954b96c077b37c8df2832c02e2d2b33 Mon Sep 17 00:00:00 2001
From: Divanshu Chauhan <divkix@divkix.me>
Date: Mon, 16 Mar 2026 18:46:34 -0700
Subject: [PATCH 1/2] refactor: remove dead code identified by static analysis

Remove unused exports and private functions that were never called:

- cloudflare/index.ts: Remove unused type exports TPROptions, TPRResult
- app-ssr-entry.ts: Remove unused collectStreamChunks function
- pages-server-entry.ts: Remove unused i18n helpers (extractLocale,
  detectLocaleFromHeaders, parseCookieLocaleFromHeader)
- routing/file-matcher.ts: Remove export from internal-only
  DEFAULT_PAGE_EXTENSIONS and buildExtensionGlob
- routing/utils.ts: Remove export from internal-only routePrecedence
  and decodeRouteSegmentStrict
- shims/internal/utils.ts: Remove unused SP and ST constants

These were identified as dead code through cross-referencing exports
against imports across the entire codebase.
---
 packages/vinext/src/cloudflare/index.ts       |  2 +-
 packages/vinext/src/entries/app-ssr-entry.ts  | 20 --------
 .../vinext/src/entries/pages-server-entry.ts  | 47 -------------------
 packages/vinext/src/routing/file-matcher.ts   |  4 +-
 packages/vinext/src/routing/utils.ts          |  4 +-
 packages/vinext/src/shims/internal/utils.ts   |  3 --
 6 files changed, 5 insertions(+), 75 deletions(-)

diff --git a/packages/vinext/src/cloudflare/index.ts b/packages/vinext/src/cloudflare/index.ts
index 175d6d1ff..a17d116ae 100644
--- a/packages/vinext/src/cloudflare/index.ts
+++ b/packages/vinext/src/cloudflare/index.ts
@@ -5,4 +5,4 @@
  */
 
 export { KVCacheHandler } from "./kv-cache-handler.js";
-export { runTPR, type TPROptions, type TPRResult } from "./tpr.js";
+export { runTPR } from "./tpr.js";
diff --git a/packages/vinext/src/entries/app-ssr-entry.ts b/packages/vinext/src/entries/app-ssr-entry.ts
index 6c0057c10..c76430f9a 100644
--- a/packages/vinext/src/entries/app-ssr-entry.ts
+++ b/packages/vinext/src/entries/app-ssr-entry.ts
@@ -21,26 +21,6 @@ import * as _clientRefs from "virtual:vite-rsc/client-references";
 
 let _clientRefsPreloaded = false;
 
-/**
- * Collect all chunks from a ReadableStream into an array of text strings.
- * Used to capture the RSC payload for embedding in HTML.
- * The RSC flight protocol is text-based (line-delimited key:value pairs),
- * so we decode to text strings instead of byte arrays — this is dramatically
- * more compact when JSON-serialized into inline <script> tags.
- */
-async function collectStreamChunks(stream) {
-  const reader = stream.getReader();
-  const decoder = new TextDecoder();
-  const chunks = [];
-  while (true) {
-    const { done, value } = await reader.read();
-    if (done) break;
-    // Decode Uint8Array to text string for compact JSON serialization
-    chunks.push(decoder.decode(value, { stream: true }));
-  }
-  return chunks;
-}
-
 // React 19 dev-mode workaround (see VinextFlightRoot in handleSsr):
 //
 // In dev, Flight error decoding in react-server-dom-webpack/client.edge
diff --git a/packages/vinext/src/entries/pages-server-entry.ts b/packages/vinext/src/entries/pages-server-entry.ts
index 44be81a98..587768a50 100644
--- a/packages/vinext/src/entries/pages-server-entry.ts
+++ b/packages/vinext/src/entries/pages-server-entry.ts
@@ -536,53 +536,6 @@ function collectAssetTags(manifest, moduleIds) {
   return tags.join("\\n  ");
 }
 
-// i18n helpers
-function extractLocale(url) {
-  if (!i18nConfig) return { locale: undefined, url, hadPrefix: false };
-  const pathname = url.split("?")[0];
-  const parts = pathname.split("/").filter(Boolean);
-  const query = url.includes("?") ? url.slice(url.indexOf("?")) : "";
-  if (parts.length > 0 && i18nConfig.locales.includes(parts[0])) {
-    const locale = parts[0];
-    const rest = "/" + parts.slice(1).join("/");
-    return { locale, url: (rest || "/") + query, hadPrefix: true };
-  }
-  return { locale: i18nConfig.defaultLocale, url, hadPrefix: false };
-}
-
-function detectLocaleFromHeaders(headers) {
-  if (!i18nConfig) return null;
-  const acceptLang = headers.get("accept-language");
-  if (!acceptLang) return null;
-  const langs = acceptLang.split(",").map(function(part) {
-    const pieces = part.trim().split(";");
-    const q = pieces[1] ? parseFloat(pieces[1].replace("q=", "")) : 1;
-    return { lang: pieces[0].trim().toLowerCase(), q: q };
-  }).sort(function(a, b) { return b.q - a.q; });
-  for (let k = 0; k < langs.length; k++) {
-    const lang = langs[k].lang;
-    for (let j = 0; j < i18nConfig.locales.length; j++) {
-      if (i18nConfig.locales[j].toLowerCase() === lang) return i18nConfig.locales[j];
-    }
-    const prefix = lang.split("-")[0];
-    for (let j = 0; j < i18nConfig.locales.length; j++) {
-      const loc = i18nConfig.locales[j].toLowerCase();
-      if (loc === prefix || loc.startsWith(prefix + "-")) return i18nConfig.locales[j];
-    }
-  }
-  return null;
-}
-
-function parseCookieLocaleFromHeader(cookieHeader) {
-  if (!i18nConfig || !cookieHeader) return null;
-  const match = cookieHeader.match(/(?:^|;\\s*)NEXT_LOCALE=([^;]*)/);
-  if (!match) return null;
-  var value;
-  try { value = decodeURIComponent(match[1].trim()); } catch (e) { return null; }
-  if (i18nConfig.locales.indexOf(value) !== -1) return value;
-  return null;
-}
-
 // Lightweight req/res facade for getServerSideProps and API routes.
 // Next.js pages expect ctx.req/ctx.res with Node-like shapes.
 function createReqRes(request, url, query, body) {
diff --git a/packages/vinext/src/routing/file-matcher.ts b/packages/vinext/src/routing/file-matcher.ts
index 7d03e22bf..9a63c6bb1 100644
--- a/packages/vinext/src/routing/file-matcher.ts
+++ b/packages/vinext/src/routing/file-matcher.ts
@@ -1,6 +1,6 @@
 import { glob } from "node:fs/promises";
 
-export const DEFAULT_PAGE_EXTENSIONS = ["tsx", "ts", "jsx", "js"] as const;
+const DEFAULT_PAGE_EXTENSIONS = ["tsx", "ts", "jsx", "js"] as const;
 
 function escapeRegex(value: string): string {
   return value.replace(/[.*+?^${}()|[\]\\]/g, "\\$&");
@@ -18,7 +18,7 @@ export function normalizePageExtensions(pageExtensions?: readonly string[] | nul
   return filtered.length > 0 ? [...filtered] : [...DEFAULT_PAGE_EXTENSIONS];
 }
 
-export function buildExtensionGlob(stem: string, extensions: readonly string[]): string {
+function buildExtensionGlob(stem: string, extensions: readonly string[]): string {
   if (extensions.length === 1) {
     return `${stem}.${extensions[0]}`;
   }
diff --git a/packages/vinext/src/routing/utils.ts b/packages/vinext/src/routing/utils.ts
index 005515c49..5d7217732 100644
--- a/packages/vinext/src/routing/utils.ts
+++ b/packages/vinext/src/routing/utils.ts
@@ -21,7 +21,7 @@
  *     E.g. /:locale/blog/:path+ (with infix "blog") correctly beats /:locale/:path+
  *     even when both share the same "locale-test" static prefix.
  */
-export function routePrecedence(pattern: string): number {
+function routePrecedence(pattern: string): number {
   const parts = pattern.split("/").filter(Boolean);
   let score = 0;
 
@@ -105,7 +105,7 @@ export function decodeRouteSegment(segment: string): string {
 /**
  * Strict variant for request pipelines that should reject malformed percent-encoding.
  */
-export function decodeRouteSegmentStrict(segment: string): string {
+function decodeRouteSegmentStrict(segment: string): string {
   return encodePathDelimiters(decodeURIComponent(segment));
 }
 
diff --git a/packages/vinext/src/shims/internal/utils.ts b/packages/vinext/src/shims/internal/utils.ts
index b449c4e17..ce282d020 100644
--- a/packages/vinext/src/shims/internal/utils.ts
+++ b/packages/vinext/src/shims/internal/utils.ts
@@ -64,6 +64,3 @@ export function getURL(): string {
   }
   return "http://localhost/";
 }
-
-export const SP = typeof performance !== "undefined";
-export const ST = SP && typeof performance.mark === "function";

From d8fc66d143a14636a8ae5c69b58b9ef64cbd5347 Mon Sep 17 00:00:00 2001
From: Divanshu Chauhan <divkix@divkix.me>
Date: Mon, 16 Mar 2026 18:55:55 -0700
Subject: [PATCH 2/2] test: update snapshots to reflect removed dead code

---
 .../entry-templates.test.ts.snap              | 67 -------------------
 1 file changed, 67 deletions(-)

diff --git a/tests/__snapshots__/entry-templates.test.ts.snap b/tests/__snapshots__/entry-templates.test.ts.snap
index 47a984f3e..1bfbe95a9 100644
--- a/tests/__snapshots__/entry-templates.test.ts.snap
+++ b/tests/__snapshots__/entry-templates.test.ts.snap
@@ -18395,26 +18395,6 @@ import * as _clientRefs from "virtual:vite-rsc/client-references";
 
 let _clientRefsPreloaded = false;
 
-/**
- * Collect all chunks from a ReadableStream into an array of text strings.
- * Used to capture the RSC payload for embedding in HTML.
- * The RSC flight protocol is text-based (line-delimited key:value pairs),
- * so we decode to text strings instead of byte arrays — this is dramatically
- * more compact when JSON-serialized into inline <script> tags.
- */
-async function collectStreamChunks(stream) {
-  const reader = stream.getReader();
-  const decoder = new TextDecoder();
-  const chunks = [];
-  while (true) {
-    const { done, value } = await reader.read();
-    if (done) break;
-    // Decode Uint8Array to text string for compact JSON serialization
-    chunks.push(decoder.decode(value, { stream: true }));
-  }
-  return chunks;
-}
-
 // React 19 dev-mode workaround (see VinextFlightRoot in handleSsr):
 //
 // In dev, Flight error decoding in react-server-dom-webpack/client.edge
@@ -19328,53 +19308,6 @@ function collectAssetTags(manifest, moduleIds) {
   return tags.join("\\n  ");
 }
 
-// i18n helpers
-function extractLocale(url) {
-  if (!i18nConfig) return { locale: undefined, url, hadPrefix: false };
-  const pathname = url.split("?")[0];
-  const parts = pathname.split("/").filter(Boolean);
-  const query = url.includes("?") ? url.slice(url.indexOf("?")) : "";
-  if (parts.length > 0 && i18nConfig.locales.includes(parts[0])) {
-    const locale = parts[0];
-    const rest = "/" + parts.slice(1).join("/");
-    return { locale, url: (rest || "/") + query, hadPrefix: true };
-  }
-  return { locale: i18nConfig.defaultLocale, url, hadPrefix: false };
-}
-
-function detectLocaleFromHeaders(headers) {
-  if (!i18nConfig) return null;
-  const acceptLang = headers.get("accept-language");
-  if (!acceptLang) return null;
-  const langs = acceptLang.split(",").map(function(part) {
-    const pieces = part.trim().split(";");
-    const q = pieces[1] ? parseFloat(pieces[1].replace("q=", "")) : 1;
-    return { lang: pieces[0].trim().toLowerCase(), q: q };
-  }).sort(function(a, b) { return b.q - a.q; });
-  for (let k = 0; k < langs.length; k++) {
-    const lang = langs[k].lang;
-    for (let j = 0; j < i18nConfig.locales.length; j++) {
-      if (i18nConfig.locales[j].toLowerCase() === lang) return i18nConfig.locales[j];
-    }
-    const prefix = lang.split("-")[0];
-    for (let j = 0; j < i18nConfig.locales.length; j++) {
-      const loc = i18nConfig.locales[j].toLowerCase();
-      if (loc === prefix || loc.startsWith(prefix + "-")) return i18nConfig.locales[j];
-    }
-  }
-  return null;
-}
-
-function parseCookieLocaleFromHeader(cookieHeader) {
-  if (!i18nConfig || !cookieHeader) return null;
-  const match = cookieHeader.match(/(?:^|;\\s*)NEXT_LOCALE=([^;]*)/);
-  if (!match) return null;
-  var value;
-  try { value = decodeURIComponent(match[1].trim()); } catch (e) { return null; }
-  if (i18nConfig.locales.indexOf(value) !== -1) return value;
-  return null;
-}
-
 // Lightweight req/res facade for getServerSideProps and API routes.
 // Next.js pages expect ctx.req/ctx.res with Node-like shapes.
 function createReqRes(request, url, query, body) {