@cairnvibe/indexer 0.2.1 → 0.2.2

package/dist/ensure-transpile.js

@@ -0,0 +1,113 @@
+"use strict";
+// `cairn setup`'s other departure from `init`'s "never touch an existing
+// file" rule, for the same class of reason as inject-widget.ts: without
+// this, @cairnvibe/sdk and @cairnvibe/core — which ship raw, untranspiled
+// .tsx/.ts as their main entry deliberately, so bundlers apply the
+// *consuming* project's own JSX/TS settings — have no instruction telling
+// Next.js to transform that source at all. Found live, not theoretical:
+// a real project on Next.js 16 + Turbopack failed cold with "Unknown
+// module type" on @cairnvibe/sdk/src/index.tsx the moment `next dev` ran,
+// because nothing in that project's next.config.ts listed it in
+// transpilePackages. Fixing that one project by hand isn't the fix — every
+// consuming project needs this, automatically, which is what this does.
+var __importDefault = (this && this.__importDefault) || function (mod) {
+    return (mod && mod.__esModule) ? mod : { "default": mod };
+};
+Object.defineProperty(exports, "__esModule", { value: true });
+exports.ensureTranspilePackages = ensureTranspilePackages;
+const node_fs_1 = __importDefault(require("node:fs"));
+const node_path_1 = __importDefault(require("node:path"));
+const ts_morph_1 = require("ts-morph");
+const REQUIRED_PACKAGES = ["@cairnvibe/sdk", "@cairnvibe/core"];
+const CONFIG_CANDIDATES = ["next.config.ts", "next.config.mjs", "next.config.js", "next.config.cjs"];
+function findConfigFile(absDir) {
+    for (const name of CONFIG_CANDIDATES) {
+        const p = node_path_1.default.join(absDir, name);
+        if (node_fs_1.default.existsSync(p))
+            return p;
+    }
+    return null;
+}
+/** Resolves `export default X` / `module.exports = X` down to the actual
+ * object literal, following one level of `const nextConfig = {...}`
+ * indirection — covers the two shapes basically every real Next.js
+ * config file uses. Anything else (a config wrapped in a plugin function
+ * call like `withSentryConfig(nextConfig)`) returns null on purpose —
+ * safely falling back to printed instructions beats guessing which
+ * argument of an arbitrary function call is the "real" config. */
+function resolveConfigObject(sf, expr) {
+    if (!expr)
+        return null;
+    if (expr.getKind() === ts_morph_1.SyntaxKind.ObjectLiteralExpression)
+        return expr;
+    if (expr.getKind() === ts_morph_1.SyntaxKind.Identifier) {
+        const varDecl = sf.getVariableDeclaration(expr.getText());
+        const init = varDecl?.getInitializer();
+        if (init?.getKind() === ts_morph_1.SyntaxKind.ObjectLiteralExpression)
+            return init;
+    }
+    return null;
+}
+function findExportedConfigObject(sf) {
+    // `export default X;`
+    const defaultExport = sf.getExportAssignments()[0];
+    if (defaultExport) {
+        const resolved = resolveConfigObject(sf, defaultExport.getExpression());
+        if (resolved)
+            return resolved;
+    }
+    // `module.exports = X;`
+    const moduleExports = sf
+        .getDescendantsOfKind(ts_morph_1.SyntaxKind.BinaryExpression)
+        .find((b) => b.getOperatorToken().getText() === "=" && b.getLeft().getText() === "module.exports");
+    if (moduleExports) {
+        const resolved = resolveConfigObject(sf, moduleExports.getRight());
+        if (resolved)
+            return resolved;
+    }
+    return null;
+}
+function ensureTranspilePackages(dir) {
+    const absDir = node_path_1.default.resolve(dir);
+    const existing = findConfigFile(absDir);
+    if (!existing) {
+        // No config at all yet — the simplest possible one, ESM since that's
+        // what every current Next.js version accepts for a fresh project.
+        const newPath = node_path_1.default.join(absDir, "next.config.mjs");
+        node_fs_1.default.writeFileSync(newPath, `/** @type {import('next').NextConfig} */\nconst nextConfig = {\n  transpilePackages: ${JSON.stringify(REQUIRED_PACKAGES)},\n};\n\nexport default nextConfig;\n`);
+        return { ok: true, filePath: newPath, created: true };
+    }
+    const relTarget = node_path_1.default.relative(absDir, existing) || existing;
+    const manualHint = `add manually: transpilePackages: ${JSON.stringify(REQUIRED_PACKAGES)}`;
+    try {
+        const project = new ts_morph_1.Project({ useInMemoryFileSystem: false, skipAddingFilesFromTsConfig: true });
+        const sf = project.addSourceFileAtPath(existing);
+        const configObject = findExportedConfigObject(sf);
+        if (!configObject) {
+            return { ok: false, reason: `couldn't confidently find the config object in ${relTarget} (it may be wrapped in a plugin function) — ${manualHint}` };
+        }
+        const existingProp = configObject.getProperty("transpilePackages");
+        if (existingProp?.getKind() === ts_morph_1.SyntaxKind.PropertyAssignment) {
+            const initializer = existingProp.asKindOrThrow(ts_morph_1.SyntaxKind.PropertyAssignment).getInitializer();
+            if (initializer?.getKind() !== ts_morph_1.SyntaxKind.ArrayLiteralExpression) {
+                return { ok: false, reason: `${relTarget}'s transpilePackages isn't a plain array — ${manualHint}` };
+            }
+            const arr = initializer.asKindOrThrow(ts_morph_1.SyntaxKind.ArrayLiteralExpression);
+            const current = arr.getElements().map((e) => e.getText().replace(/^["']|["']$/g, ""));
+            const toAdd = REQUIRED_PACKAGES.filter((p) => !current.includes(p));
+            if (toAdd.length === 0) {
+                return { ok: false, reason: `${relTarget} already lists these packages — leaving it alone` };
+            }
+            for (const pkg of toAdd)
+                arr.addElement(`"${pkg}"`);
+        }
+        else {
+            configObject.addPropertyAssignment({ name: "transpilePackages", initializer: JSON.stringify(REQUIRED_PACKAGES) });
+        }
+        sf.saveSync();
+        return { ok: true, filePath: existing };
+    }
+    catch (err) {
+        return { ok: false, reason: `couldn't safely modify ${relTarget} (${err.message}) — ${manualHint}` };
+    }
+}

package/dist/inject-widget.js

@@ -1,11 +1,29 @@
 "use strict";
 // `cairn setup`'s one departure from `init`'s "never touch an existing
-// file" rule — but only for this one, narrow, reversible edit (adding an
-// import + one JSX tag), and only ever via a real AST parse, never blind
-// string splicing. Any structure this doesn't recognize falls back to
-// printing the two-line manual instruction instead of guessing — the
-// same "don't corrupt what you don't understand" discipline `init` uses
-// for whole files, applied here at the node level.
+// file" rule — but only for this one, narrow, reversible edit (a new
+// wrapper component file, plus one import + one JSX tag in the real
+// layout), and only ever via a real AST parse, never blind string
+// splicing. Any structure this doesn't recognize falls back to printing
+// the two-line manual instruction instead of guessing — the same "don't
+// corrupt what you don't understand" discipline `init` uses for whole
+// files, applied here at the node level.
+//
+// Real bug this fixes, found by testing against an actual project, not
+// a synthetic fixture: an earlier version inserted `<Copilot
+// onDo={(action, target) => {...}} />` directly into app/layout.tsx.
+// layout.tsx is a Server Component by default (App Router) — React
+// Server Components cannot accept a plain inline function as a prop on
+// a Client Component, which <Copilot/> is ("Event handlers cannot be
+// passed to Client Component props"). The working example app
+// (examples/demo-app/components/CopilotWithActions.tsx) already solved
+// this the right way: a small "use client" wrapper component that
+// *defines* onDo itself, so no function ever crosses the server/client
+// boundary — layout.tsx only ever references the wrapper by name, with
+// zero function props. This module now generates that same wrapper
+// instead of inlining Copilot directly, for both App Router and Pages
+// Router (Pages Router doesn't strictly need it — no RSC boundary
+// there — but the same shape avoids a special case and matches the one
+// real, working example this project has).
 var __importDefault = (this && this.__importDefault) || function (mod) {
     return (mod && mod.__esModule) ? mod : { "default": mod };
 };
@@ -14,7 +32,27 @@ exports.injectWidget = injectWidget;
 const node_fs_1 = __importDefault(require("node:fs"));
 const node_path_1 = __importDefault(require("node:path"));
 const ts_morph_1 = require("ts-morph");
-const WIDGET_JSX = `<Copilot registeredActions={[]} onDo={(action, target) => { /* run it through your own auth */ }} />`;
+const WRAPPER_COMPONENT_NAME = "CairnCopilot";
+function wrapperSource() {
+    return `"use client";
+
+import { Copilot } from "@cairnvibe/sdk";
+
+// A small client wrapper so the layout/app file (a server component, for
+// metadata etc.) never has to pass a function prop across the server/client
+// boundary — see the comment in inject-widget.ts for why that fails.
+export function ${WRAPPER_COMPONENT_NAME}() {
+  return (
+    <Copilot
+      registeredActions={[]}
+      onDo={(action, target) => {
+        // run it through your own auth
+      }}
+    />
+  );
+}
+`;
+}
 function findLayoutFile(absDir, framework) {
     const candidates = framework === "next-app-router"
         ? ["app/layout.tsx", "app/layout.jsx"]
@@ -26,17 +64,34 @@ function findLayoutFile(absDir, framework) {
     }
     return null;
 }
+function toPosixRelativeImport(fromFile, toFileNoExt) {
+    let rel = node_path_1.default.relative(node_path_1.default.dirname(fromFile), toFileNoExt).split(node_path_1.default.sep).join("/");
+    if (!rel.startsWith("."))
+        rel = `./${rel}`;
+    return rel;
+}
 function injectWidget(dir, framework) {
     const absDir = node_path_1.default.resolve(dir);
     const target = findLayoutFile(absDir, framework);
     if (!target) {
-        return { injected: false, reason: "no app/layout.tsx or pages/_app.tsx found — add <Copilot/> manually" };
+        return { injected: false, reason: "no app/layout.tsx or pages/_app.tsx found — add the widget manually" };
     }
     const relTarget = node_path_1.default.relative(absDir, target) || target;
     const original = node_fs_1.default.readFileSync(target, "utf8");
-    if (original.includes("@cairnvibe/sdk") || original.includes("<Copilot")) {
-        return { injected: false, reason: `${relTarget} already references Copilot — leaving it alone` };
+    if (original.includes(WRAPPER_COMPONENT_NAME) || original.includes("@cairnvibe/sdk") || original.includes("<Copilot")) {
+        return { injected: false, reason: `${relTarget} already references the widget — leaving it alone` };
+    }
+    // The wrapper always matches the layout file's own extension (.tsx stays
+    // .tsx, .jsx stays .jsx — a .tsx file dropped into a plain-JS project has
+    // no type checker configured for it and would just confuse tooling).
+    const ext = node_path_1.default.extname(target); // ".tsx" or ".jsx"
+    const wrapperPath = node_path_1.default.join(absDir, "components", `${WRAPPER_COMPONENT_NAME}${ext}`);
+    if (!node_fs_1.default.existsSync(wrapperPath)) {
+        node_fs_1.default.mkdirSync(node_path_1.default.dirname(wrapperPath), { recursive: true });
+        node_fs_1.default.writeFileSync(wrapperPath, wrapperSource());
     }
+    const importPath = toPosixRelativeImport(target, wrapperPath.slice(0, -ext.length));
+    const widgetJsx = `<${WRAPPER_COMPONENT_NAME} />`;
     try {
         const project = new ts_morph_1.Project({
             useInMemoryFileSystem: false,
@@ -44,9 +99,9 @@ function injectWidget(dir, framework) {
             compilerOptions: { jsx: ts_morph_1.ts.JsxEmit.ReactJSX, allowJs: true, esModuleInterop: true, target: ts_morph_1.ts.ScriptTarget.ES2022 },
         });
         const sf = project.addSourceFileAtPath(target);
-        const hasImport = sf.getImportDeclarations().some((d) => d.getModuleSpecifierValue() === "@cairnvibe/sdk");
+        const hasImport = sf.getImportDeclarations().some((d) => d.getModuleSpecifierValue() === importPath);
         if (!hasImport) {
-            sf.addImportDeclaration({ moduleSpecifier: "@cairnvibe/sdk", namedImports: ["Copilot"] });
+            sf.addImportDeclaration({ moduleSpecifier: importPath, namedImports: [WRAPPER_COMPONENT_NAME] });
         }
         // `insertText` at a position, not `replaceWithText` on a node — the latter
         // asks ts-morph to structurally reconcile old vs. new trees, which fails
@@ -62,7 +117,7 @@ function injectWidget(dir, framework) {
         if (bodyOpening) {
             const closing = bodyOpening.getParentIfKind(ts_morph_1.SyntaxKind.JsxElement)?.getClosingElement();
             if (closing) {
-                sf.insertText(closing.getStart(), `${WIDGET_JSX}\n      `);
+                sf.insertText(closing.getStart(), `${widgetJsx}\n      `);
                 inserted = true;
             }
         }
@@ -72,7 +127,7 @@ function injectWidget(dir, framework) {
                 .getDescendantsOfKind(ts_morph_1.SyntaxKind.JsxExpression)
                 .find((e) => e.getExpression()?.getText() === "children");
             if (childrenExpr) {
-                sf.insertText(childrenExpr.getEnd(), `\n      ${WIDGET_JSX}`);
+                sf.insertText(childrenExpr.getEnd(), `\n      ${widgetJsx}`);
                 inserted = true;
             }
         }
@@ -89,20 +144,22 @@ function injectWidget(dir, framework) {
                 // first insertion changes the source text underneath it.
                 const start = componentTag.getStart();
                 const end = componentTag.getEnd();
-                sf.insertText(end, `\n      ${WIDGET_JSX}\n    </>`);
+                sf.insertText(end, `\n      ${widgetJsx}\n    </>`);
                 sf.insertText(start, `<>\n      `);
                 inserted = true;
             }
         }
         if (!inserted) {
-            return { injected: false, reason: `couldn't find a safe spot in ${relTarget} — add <Copilot/> manually` };
+            return { injected: false, reason: `couldn't find a safe spot in ${relTarget} — add the widget manually` };
         }
         sf.saveSync();
-        return { injected: true, filePath: target };
+        return { injected: true, filePath: target, wrapperPath };
     }
     catch (err) {
         // Never leave a half-written file — ts-morph only writes on saveSync(),
         // so a thrown error here means the original file on disk is untouched.
-        return { injected: false, reason: `couldn't safely modify ${relTarget} (${err.message}) — add <Copilot/> manually` };
+        // The wrapper component file, if it was just created, is still valid
+        // and harmless on its own — it's just not referenced from anywhere yet.
+        return { injected: false, reason: `couldn't safely modify ${relTarget} (${err.message}) — add the widget manually` };
     }
 }

package/dist/setup.js

@@ -22,6 +22,7 @@ const node_path_1 = __importDefault(require("node:path"));
 const node_child_process_1 = require("node:child_process");
 const init_1 = require("./init");
 const inject_widget_1 = require("./inject-widget");
+const ensure_transpile_1 = require("./ensure-transpile");
 const prompt_1 = require("./prompt");
 const l1_scan_1 = require("./l1-scan");
 const l2_reachability_1 = require("./l2-reachability");
@@ -225,12 +226,25 @@ async function runSetup(dir) {
     const framework = init.framework;
     const inject = (0, inject_widget_1.injectWidget)(dir, framework);
     if (inject.injected) {
-        console.log((0, ui_1.green)(`✓ wired <Copilot/> into ${node_path_1.default.relative(absDir, inject.filePath)}`));
+        console.log((0, ui_1.green)(`✓ wired the widget into ${node_path_1.default.relative(absDir, inject.filePath)} (via a new components/CairnCopilot.tsx wrapper)`));
     }
     else {
-        console.log(`\n<Copilot/> not auto-wired (${inject.reason}). Add it yourself:`);
+        console.log(`\nWidget not auto-wired (${inject.reason}). Add it yourself:`);
         console.log('  import { Copilot } from "@cairnvibe/sdk";');
         console.log("  <Copilot registeredActions={[]} onDo={(action, target) => { /* run it */ }} />");
+        console.log("  (in a \"use client\" component — see examples/demo-app/components/CopilotWithActions.tsx for why)");
+    }
+    // 5b. @cairnvibe/sdk and @cairnvibe/core ship raw TS/TSX as their main
+    // entry deliberately — bundlers need transpilePackages to know to
+    // transform it. Without this, real projects fail cold at `next dev`
+    // with "Unknown module type", not something a demo on a fresh project
+    // would ever surface (this repo's own next.config.js already has it).
+    const transpile = (0, ensure_transpile_1.ensureTranspilePackages)(dir);
+    if (transpile.ok) {
+        console.log((0, ui_1.green)(`✓ ${transpile.created ? "created" : "updated"} ${node_path_1.default.relative(absDir, transpile.filePath)} with transpilePackages`));
+    }
+    else {
+        console.log(`\ntranspilePackages not auto-added (${transpile.reason})`);
     }
     // 6. Build the manifest once now, if we actually have a usable key — no
     // point trying (and failing loudly) with nothing to call. On failure,

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@cairnvibe/indexer",
-  "version": "0.2.1",
+  "version": "0.2.2",
   "description": "Cairn's analyzer and installer (the `cairn` CLI) — scans Next.js source or crawls any running app, and scaffolds the backend either way.",
   "license": "MIT",
   "publishConfig": { "access": "public" },