@paramour-js/next 0.11.0 → 0.11.1

package/dist/app.js

@@ -1,5 +1,11 @@
 "use client";
-import { useParams, usePathname, useRouter, useSearchParams, } from "next/navigation";
+// Extensionful for the same reason as pages.ts's `next/router.js`: `next`
+// ships no `exports` map, so wherever this package is loaded through Node's
+// ESM resolver (Vitest externalizes node_modules) the bare `next/navigation`
+// dies with ERR_MODULE_NOT_FOUND. The root stub `next/navigation.js` is the
+// file the bare specifier resolves to, and Next's bundler aliases are keyed
+// on that file, so bundles see the same module either way.
+import { useParams, usePathname, useRouter, useSearchParams, } from "next/navigation.js";
 import { decodeParams, decodeSearch, ParamsDecodeError, safeDecodeParams, safeDecodeSearch, SearchDecodeError, } from "paramour";
 import { useContext } from "react";
 import { searchWireSnapshot } from "./devtools-emit.js";

package/dist/emit.d.ts

@@ -6,11 +6,18 @@ export interface WriteIfChangedResult {
      */
     previousContent: null | string;
     /**
-     * `false` on a byte-identical no-op — the signal `--check` and the watch
-     * loop hang off.
+     * `false` on a no-op (identical up to line endings) — the signal `--check`
+     * and the watch loop hang off.
      */
     written: boolean;
 }
+/**
+ * Content equality that ignores CRLF vs LF. A consumer repo with git
+ * `core.autocrlf=true` checks the committed LF artifact out as CRLF; that
+ * flip is not drift, and treating it as drift fails every strict build and
+ * `paramour check` on a fresh Windows checkout.
+ */
+export declare function sameContent(a: string, b: string): boolean;
 /** Input of {@link emitArtifact} — one union per router. */
 export interface EmitRoutes {
     appRoutes: readonly string[];
@@ -30,6 +37,8 @@ export declare function emitArtifact(routes: EmitRoutes): string;
 /**
  * Compare-before-write: a no-op regeneration must not touch the file — that
  * property is what prevents TS-server churn, watch-loop feedback, and
- * concurrent-generator races.
+ * concurrent-generator races. A file differing only in line endings is a
+ * no-op too, so a CRLF checkout keeps its endings and `git status` stays
+ * clean; a real change is written LF.
  */
 export declare function writeIfChanged(filePath: string, content: string): WriteIfChangedResult;

package/dist/emit.js

@@ -1,5 +1,14 @@
 import { existsSync, mkdirSync, readFileSync, writeFileSync } from "node:fs";
 import { dirname } from "node:path";
+/**
+ * Content equality that ignores CRLF vs LF. A consumer repo with git
+ * `core.autocrlf=true` checks the committed LF artifact out as CRLF; that
+ * flip is not drift, and treating it as drift fails every strict build and
+ * `paramour check` on a fresh Windows checkout.
+ */
+export function sameContent(a, b) {
+    return a.replaceAll("\r\n", "\n") === b.replaceAll("\r\n", "\n");
+}
 const HEADER = "// Generated by @paramour-js/next. Do not edit — regenerate with `paramour generate`.";
 /**
  * Artifact text for the per-router route unions: deterministic — sorted,
@@ -59,14 +68,17 @@ export function emitArtifact(routes) {
 /**
  * Compare-before-write: a no-op regeneration must not touch the file — that
  * property is what prevents TS-server churn, watch-loop feedback, and
- * concurrent-generator races.
+ * concurrent-generator races. A file differing only in line endings is a
+ * no-op too, so a CRLF checkout keeps its endings and `git status` stays
+ * clean; a real change is written LF.
  */
 export function writeIfChanged(filePath, content) {
     const previousContent = existsSync(filePath)
         ? readFileSync(filePath, "utf8")
         : null;
-    if (previousContent === content)
+    if (previousContent !== null && sameContent(previousContent, content)) {
         return { previousContent, written: false };
+    }
     // The outFile escape hatch may point into a not-yet-existing directory.
     mkdirSync(dirname(filePath), { recursive: true });
     writeFileSync(filePath, content);

package/dist/generate.d.ts

@@ -40,9 +40,10 @@ export interface RouterDrift {
     disappeared: string[];
 }
 /**
- * `--check`: scan to memory and byte-compare against disk — never writes. A
- * missing artifact is drift, not an error: that is exactly the
- * CI-degrades-to-world-A case the committed file exists to prevent.
+ * `--check`: scan to memory and compare against disk (line endings aside, as
+ * in {@link writeIfChanged}) — never writes. A missing artifact is drift, not
+ * an error: that is exactly the CI-degrades-to-world-A case the committed
+ * file exists to prevent.
  */
 export declare function checkArtifact(inputs: GenerateInputs): CheckResult;
 /**

package/dist/generate.js

@@ -1,5 +1,5 @@
 import { existsSync, readFileSync } from "node:fs";
-import { emitArtifact, writeIfChanged } from "./emit.js";
+import { emitArtifact, sameContent, writeIfChanged } from "./emit.js";
 import { scanRoutes } from "./scan.js";
 /**
  * Reads the per-router unions back out of a previously emitted artifact for
@@ -11,9 +11,10 @@ import { scanRoutes } from "./scan.js";
 const MEMBER_HEADER = /^\s*(appRoutes|pagesRoutes):\s*$/;
 const UNION_MEMBER = /^\s*\| "(.*)";?\s*$/;
 /**
- * `--check`: scan to memory and byte-compare against disk — never writes. A
- * missing artifact is drift, not an error: that is exactly the
- * CI-degrades-to-world-A case the committed file exists to prevent.
+ * `--check`: scan to memory and compare against disk (line endings aside, as
+ * in {@link writeIfChanged}) — never writes. A missing artifact is drift, not
+ * an error: that is exactly the CI-degrades-to-world-A case the committed
+ * file exists to prevent.
  */
 export function checkArtifact(inputs) {
     const routes = scanRoutes(inputs, inputs.pageExtensions);
@@ -21,7 +22,7 @@ export function checkArtifact(inputs) {
     const current = existsSync(inputs.artifactPath)
         ? readFileSync(inputs.artifactPath, "utf8")
         : null;
-    if (current === expected) {
+    if (current !== null && sameContent(current, expected)) {
         return {
             app: { appeared: [], disappeared: [] },
             missingFile: false,

package/dist/navigation-adapter.d.ts

@@ -12,7 +12,7 @@ import type { ParamsSource } from "paramour";
  * (`useContext(ctx) ?? realAdapter`), so neither this module nor the
  * /testing entry ever drags a `next/*` specifier into its graph. The
  * dist.test.ts bundle-hygiene invariants (/app reaches only
- * `next/navigation`, /pages only `next/router.js`, /testing neither) depend
+ * `next/navigation.js`, /pages only `next/router.js`, /testing neither) depend
  * on exactly this split — a context whose DEFAULT VALUE were the real
  * adapter would break all three.
  */

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@paramour-js/next",
-  "version": "0.11.0",
+  "version": "0.11.1",
   "type": "module",
   "exports": {
     ".": {
@@ -51,7 +51,7 @@
     "react-dom": "^19.2.0",
     "typescript": "^6.0.3",
     "zod": "^4.4.3",
-    "paramour": "0.11.0"
+    "paramour": "0.11.1"
   },
   "description": "Next.js integration for paramour: withTypedRoutes, App and Pages Router hooks, PageProps glue, and the codegen CLI.",
   "author": "Jason Paff <jasonpaff@gmail.com>",