@gurulu/cli 0.4.1 → 0.4.3

package/scripts/patches/auto-instrument/sveltekit.cjs

@@ -35,7 +35,7 @@ function routeToCandidates(urlPath) {
   return out;
 }
 
-function astInstrumentFile(source, method, events) {
+function astInstrumentFile(source, method, events, opts = {}) {
   const tree = ast.parseSource(source);
   const fns = ast.findExportedFunction(tree, method);
   if (fns.length === 0) return { ok: false, reason: `${method}-not-found` };
@@ -51,7 +51,9 @@ function astInstrumentFile(source, method, events) {
   }
   const stmts = events.map((e) => ast.buildTrackStatement(e.name, e.autoProperties));
   ast.injectTrackBeforeLastReturn(target.fn, target.body, stmts);
-  ast.ensureGuruluImport(tree);
+  // Sprint D / D4 — alias-aware import.
+  const specifier = ast.resolveGuruluImportSpecifier(opts.repoRoot, opts.relPath);
+  ast.ensureGuruluImport(tree, specifier);
   const after = ast.generateSource(tree, source);
   return {
     ok: true,
@@ -103,9 +105,11 @@ function instrumentEvents(ctx, events) {
   for (const group of groups.values()) {
     const abs = path.join(ctx.repoRoot, group.relPath);
     const before = fs.readFileSync(abs, 'utf8');
+    // Sprint D / D4 — pass repoRoot + relPath to the import resolver.
+    const fileOpts = { repoRoot: (ctx && ctx.repoRoot) || null, relPath: group.relPath };
     let res;
     try {
-      res = astInstrumentFile(before, group.method, group.events);
+      res = astInstrumentFile(before, group.method, group.events, fileOpts);
     } catch (err) {
       const msg = (err && err.message) || String(err);
       // eslint-disable-next-line no-console

package/scripts/patches/auto-instrument/vue.cjs

@@ -70,7 +70,7 @@ function findEventHandler(tree) {
   return found;
 }
 
-function astInstrumentFile(source, events) {
+function astInstrumentFile(source, events, opts = {}) {
   const tree = ast.parseSource(source);
   const target = findEventHandler(tree);
   if (!target) return { ok: false, reason: 'event-handler-not-found' };
@@ -85,7 +85,9 @@ function astInstrumentFile(source, events) {
   }
   const stmts = events.map((e) => ast.buildTrackStatement(e.name, e.autoProperties));
   ast.injectTrackBeforeLastReturn(target.fn, target.body, stmts);
-  ast.ensureGuruluImport(tree);
+  // Sprint D / D4 — alias-aware import.
+  const specifier = ast.resolveGuruluImportSpecifier(opts.repoRoot, opts.relPath);
+  ast.ensureGuruluImport(tree, specifier);
   const after = ast.generateSource(tree, source);
   return {
     ok: true,
@@ -137,9 +139,11 @@ function instrumentEvents(ctx, events) {
   for (const group of groups.values()) {
     const abs = path.join(ctx.repoRoot, group.relPath);
     const before = fs.readFileSync(abs, 'utf8');
+    // Sprint D / D4 — pass repoRoot + relPath to the import resolver.
+    const fileOpts = { repoRoot: (ctx && ctx.repoRoot) || null, relPath: group.relPath };
     let res;
     try {
-      res = astInstrumentFile(before, group.events);
+      res = astInstrumentFile(before, group.events, fileOpts);
     } catch (err) {
       const msg = (err && err.message) || String(err);
       // eslint-disable-next-line no-console

package/scripts/patches/index.cjs

@@ -42,6 +42,12 @@ const PATCHERS = [
 ];
 
 const PATCHER_BY_NAME = {
+  // Sprint D / D3 — `nextjs` (bare) is the alias most agents emit. Resolve it
+  // to the App Router patcher (the modern default). The Pages Router can
+  // still be selected explicitly via `nextjs-pages`. Auto-detection (in the
+  // PATCHERS array above) handles the case where a project actually uses
+  // Pages Router and the agent passed `auto`.
+  nextjs: nextAppRouter,
   'nextjs-app': nextAppRouter,
   'nextjs-app-router': nextAppRouter,
   'nextjs-pages': nextPages,