@paramour-js/next 1.0.0-rc.0 → 1.0.0-rc.1

package/dist/emit.d.ts

@@ -11,6 +11,13 @@ export interface WriteIfChangedResult {
      */
     written: boolean;
 }
+/**
+ * The line ending a user-owned file already uses. Init edits files the user
+ * owns (package.json, next.config, AGENTS.md), and writing LF text into a CRLF file
+ * leaves mixed endings, so those edits keep the file's own ending. Generated
+ * files are always written LF instead.
+ */
+export declare function lineEnding(text: string): "\n" | "\r\n";
 /**
  * Content equality that ignores CRLF vs LF. A consumer repo with git
  * `core.autocrlf=true` checks the committed LF artifact out as CRLF; that

package/dist/emit.js

@@ -1,5 +1,14 @@
 import { existsSync, mkdirSync, readFileSync, writeFileSync } from "node:fs";
 import { dirname } from "node:path";
+/**
+ * The line ending a user-owned file already uses. Init edits files the user
+ * owns (package.json, next.config, AGENTS.md), and writing LF text into a CRLF file
+ * leaves mixed endings, so those edits keep the file's own ending. Generated
+ * files are always written LF instead.
+ */
+export function lineEnding(text) {
+    return text.includes("\r\n") ? "\r\n" : "\n";
+}
 /**
  * Content equality that ignores CRLF vs LF. A consumer repo with git
  * `core.autocrlf=true` checks the committed LF artifact out as CRLF; that

package/dist/init/agents-md.js

@@ -1,5 +1,6 @@
 import { existsSync } from "node:fs";
 import { join } from "node:path";
+import { lineEnding, sameContent } from "../emit.js";
 export const AGENTS_MARKER_START = "<!-- paramour:start -->";
 export const AGENTS_MARKER_END = "<!-- paramour:end -->";
 /**
@@ -53,21 +54,22 @@ export function findAgentsFile(projectRoot) {
  * would mean guessing where the user's own text begins.
  */
 export function upsertAgentsSection(text) {
+    const eol = lineEnding(text);
+    const snippet = agentsSnippet().replaceAll("\n", eol);
     const start = text.indexOf(AGENTS_MARKER_START);
     if (start === -1) {
-        const base = text === "" || text.endsWith("\n") ? text : `${text}\n`;
-        const separator = base === "" ? "" : "\n";
+        const base = text === "" || text.endsWith("\n") ? text : `${text}${eol}`;
+        const separator = base === "" ? "" : eol;
         return {
             status: "added",
-            text: `${base}${separator}${agentsSnippet()}\n`,
+            text: `${base}${separator}${snippet}${eol}`,
         };
     }
     const end = text.indexOf(AGENTS_MARKER_END, start);
     if (end === -1)
         return { status: "unterminated", text };
     const current = text.slice(start, end + AGENTS_MARKER_END.length);
-    const snippet = agentsSnippet();
-    if (current === snippet)
+    if (sameContent(current, snippet))
         return { status: "unchanged", text };
     return {
         status: "updated",

package/dist/init/scaffold.d.ts

@@ -6,9 +6,9 @@ export interface SetupCheck {
 }
 /**
  * Insert `"paramour": "paramour generate"` into a package.json's scripts,
- * preserving the file's own indentation and trailing-newline choice.
- * Throws on malformed JSON — a broken package.json is init's one hard
- * prerequisite failure.
+ * preserving the file's own indentation, line ending, and trailing-newline
+ * choice. Throws on malformed JSON — a broken package.json is init's one
+ * hard prerequisite failure.
  */
 export declare function addPackageScript(text: string): {
     changed: boolean;

package/dist/init/scaffold.js

@@ -1,14 +1,15 @@
 import { existsSync, readFileSync } from "node:fs";
 import { join, relative } from "node:path";
+import { lineEnding } from "../emit.js";
 import { resolveRouteDirs } from "../scan.js";
 import { loadPackagedSkill } from "../skills/packaged.js";
 import { auditTarget, isOutdated } from "../skills/sync.js";
 import { detectTargets } from "../skills/targets.js";
 /**
  * Insert `"paramour": "paramour generate"` into a package.json's scripts,
- * preserving the file's own indentation and trailing-newline choice.
- * Throws on malformed JSON — a broken package.json is init's one hard
- * prerequisite failure.
+ * preserving the file's own indentation, line ending, and trailing-newline
+ * choice. Throws on malformed JSON — a broken package.json is init's one
+ * hard prerequisite failure.
  */
 export function addPackageScript(text) {
     const parsed = JSON.parse(text);
@@ -28,8 +29,12 @@ export function addPackageScript(text) {
         return { changed: false, text };
     pkg.scripts = { ...scripts, paramour: "paramour generate" };
     const indent = /^([ \t]+)"/m.exec(text)?.[1] ?? "  ";
-    const trailing = text.endsWith("\n") ? "\n" : "";
-    return { changed: true, text: JSON.stringify(pkg, null, indent) + trailing };
+    const eol = lineEnding(text);
+    const trailing = text.endsWith("\n") ? eol : "";
+    return {
+        changed: true,
+        text: JSON.stringify(pkg, null, indent).replaceAll("\n", eol) + trailing,
+    };
 }
 /**
  * The detect-and-verify summary: route dirs discoverable, both packages

package/dist/init/wrap-next-config.js

@@ -1,5 +1,6 @@
 import { existsSync } from "node:fs";
 import { join } from "node:path";
+import { lineEnding } from "../emit.js";
 const IMPORT_LINE = `import { withTypedRoutes } from "@paramour-js/next";`;
 /** {@link writtenBindings}' name for `module.exports` (no identifier has a dot). */
 const MODULE_EXPORTS = "module.exports";
@@ -131,7 +132,13 @@ export async function wrapNextConfigSource(source) {
             });
         }
         mod.exports.default = builders.functionCall(local ?? "withTypedRoutes", current);
-        return { code: generateCode(mod).code, status: "wrapped" };
+        // recast prints new nodes with LF and drops the final newline; keep the
+        // user's file as it was on both counts.
+        const eol = lineEnding(source);
+        let code = generateCode(mod).code.replaceAll("\r\n", "\n");
+        if (source.endsWith("\n") && !code.endsWith("\n"))
+            code += "\n";
+        return { code: code.replaceAll("\n", eol), status: "wrapped" };
     }
     catch {
         return { snippet: manualSnippet(), status: "manual" };

package/dist/list/discover-route-defs.d.ts

@@ -38,9 +38,10 @@ export interface RouteDefinition {
  * config globs replace the default patterns when the heuristic misfires.
  *
  * jiti evaluates matched files (the loader carry-over from config loading).
- * Known limits, both handled by the per-module degrade: tsconfig `paths`
- * aliases are not resolved (a future improvement could feed them into jiti's
- * `alias` option), and `server-only`-style imports throw outside Next.
+ * Imports resolve through the project's tsconfig `paths`, like Next's own
+ * resolution, so definitions that import shared codecs via `@/…` load. One
+ * known limit is left to the per-module degrade: `server-only`-style
+ * imports throw outside Next.
  */
 export declare function discoverRouteDefinitions(projectRoot: string, options?: {
     routeFiles?: readonly string[] | undefined;

package/dist/list/discover-route-defs.js

@@ -15,6 +15,11 @@ const IGNORE_PATTERNS = [
     "**/node_modules/**",
     "**/out/**",
 ];
+// Test files often mention defineAppRoute (they test route codecs), and
+// evaluating them outside the runner executes their top-level
+// describe/it calls and any setup side effects. These are ignored only in
+// the automatic scan: explicit `routeFiles` globs are taken as written.
+const TEST_IGNORE_PATTERNS = ["**/*.{test,spec}.*", "**/__tests__/**"];
 /** Files past this size skip the content pre-filter (bundles, lockfiles). */
 const MAX_PREFILTER_BYTES = 512 * 1024;
 /**
@@ -24,17 +29,24 @@ const MAX_PREFILTER_BYTES = 512 * 1024;
  * config globs replace the default patterns when the heuristic misfires.
  *
  * jiti evaluates matched files (the loader carry-over from config loading).
- * Known limits, both handled by the per-module degrade: tsconfig `paths`
- * aliases are not resolved (a future improvement could feed them into jiti's
- * `alias` option), and `server-only`-style imports throw outside Next.
+ * Imports resolve through the project's tsconfig `paths`, like Next's own
+ * resolution, so definitions that import shared codecs via `@/…` load. One
+ * known limit is left to the per-module degrade: `server-only`-style
+ * imports throw outside Next.
  */
 export async function discoverRouteDefinitions(projectRoot, options = {}) {
     // Dynamic imports, same stance as config.ts: only commands that actually
     // discover definitions pay for tinyglobby/jiti.
     const { glob } = await import("tinyglobby");
-    const files = await glob(options.routeFiles === undefined
-        ? DEFAULT_PATTERNS
-        : [...options.routeFiles], { absolute: true, cwd: projectRoot, ignore: IGNORE_PATTERNS });
+    const { routeFiles } = options;
+    const [patterns, ignore] = routeFiles === undefined
+        ? [DEFAULT_PATTERNS, [...IGNORE_PATTERNS, ...TEST_IGNORE_PATTERNS]]
+        : [[...routeFiles], IGNORE_PATTERNS];
+    const files = await glob(patterns, {
+        absolute: true,
+        cwd: projectRoot,
+        ignore,
+    });
     // Deterministic load order — dedupe's "first wins" must not depend on
     // filesystem enumeration order.
     files.sort();
@@ -43,6 +55,11 @@ export async function discoverRouteDefinitions(projectRoot, options = {}) {
     const jiti = createJiti(import.meta.url, {
         fsCache: false,
         interopDefault: true,
+        // A directory, not `true`: `true` searches upward from this package's
+        // own install location inside node_modules, not from the user's project.
+        // jiti's bundled get-tsconfig finds the governing tsconfig.json from
+        // here and handles `extends`, `baseUrl`, and wildcard patterns.
+        tsconfigPaths: projectRoot,
     });
     const definitions = [];
     const duplicates = [];
@@ -55,7 +72,10 @@ export async function discoverRouteDefinitions(projectRoot, options = {}) {
             mod = await jiti.import(file);
         }
         catch (error) {
-            loadFailures.push({ file: relFile, message: message(error) });
+            // First line only: Node appends a multi-line "Require stack" to
+            // MODULE_NOT_FOUND, which would bury the one-line-per-module report.
+            const [firstLine = ""] = message(error).split(/\r?\n/, 1);
+            loadFailures.push({ file: relFile, message: firstLine });
             continue;
         }
         if (typeof mod !== "object" || mod === null)

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@paramour-js/next",
-  "version": "1.0.0-rc.0",
+  "version": "1.0.0-rc.1",
   "type": "module",
   "exports": {
     ".": {
@@ -51,7 +51,7 @@
     "react-dom": "^19.2.0",
     "typescript": "^6.0.3",
     "zod": "^4.4.3",
-    "paramour": "1.0.0-rc.0"
+    "paramour": "1.0.0-rc.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>",