macaly-tagger 1.0.0 → 1.1.0

package/README.md

@@ -26,6 +26,47 @@ The loader has the following dependencies that will be installed automatically:
 
 ### Next.js Setup
 
+#### For Next.js 15.3.0+ with Turbopack
+
+Configure your `next.config.js`:
+
+```js
+/** @type {import('next').NextConfig} */
+const nextConfig = {
+  turbopack: {
+    rules: {
+      "*.{jsx,tsx}": {
+        condition: {
+          all: [
+            { not: "foreign" },  // Exclude node_modules
+            "development",        // Only in development mode
+          ],
+        },
+        loaders: [
+          {
+            loader: "macaly-tagger",
+            options: {
+              disableSourceMaps: true,  // Required to avoid Turbopack crashes
+            },
+          },
+        ],
+        as: "*",  // Preserve original file handling
+      },
+    },
+  },
+};
+
+module.exports = nextConfig;
+```
+
+> **Important Notes for Turbopack:**
+> - **`disableSourceMaps: true` is required** to prevent Turbopack crashes when processing source maps
+> - **`as: "*"`** preserves the original file type handling (don't use `"*.js"`)
+> - **Don't include `'browser'` condition** as it may exclude server components in Next.js
+> - The simple `"*.{jsx,tsx}"` glob pattern works correctly with these settings
+
+#### For Next.js with Webpack (legacy)
+
 Configure your `next.config.js`:
 
 ```js
@@ -144,6 +185,25 @@ The loader works out of the box with no configuration needed. It automatically:
 - Runs only in development mode (when configured properly)
 - Handles both regular JSX elements (`<div>`) and member expressions (`<MyLib.Button>`)
 
+### Options
+
+You can pass options to the loader:
+
+```js
+{
+  loader: 'macaly-tagger',
+  options: {
+    debug: true,              // Enable debug logging
+    disableSourceMaps: true,  // Disable source map generation (useful for Turbopack)
+  },
+}
+```
+
+| Option | Type | Default | Description |
+|--------|------|---------|-------------|
+| `debug` | `boolean` | `false` | Enable detailed console logging for debugging |
+| `disableSourceMaps` | `boolean` | `false` | Disable source map generation (recommended for Turbopack to avoid crashes) |
+
 ## Attributes Added
 
 For each JSX element, the loader adds:
@@ -171,7 +231,7 @@ Once installed, you can:
 ### Build errors?
 
 1. **Install dependencies**: Ensure all peer dependencies are installed
-2. **Check webpack version**: Requires webpack >= 4.0.0
+2. **Check webpack version**: Requires webpack >= 4.0.0 (for webpack setup)
 3. **Verify loader path**: Make sure the loader is correctly referenced
 
 ### Performance issues?
@@ -211,6 +271,12 @@ MIT License - see LICENSE file for details.
 
 ## Changelog
 
+### 1.1.0
+
+- Added `disableSourceMaps` option to prevent Turbopack crashes
+- Added comprehensive Turbopack configuration documentation
+- Tested and verified Turbopack compatibility with Next.js 15.3.0+
+
 ### 1.0.0
 
 - Initial release

package/dist/macaly-tagger-loader.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"macaly-tagger-loader.d.ts","sourceRoot":"","sources":["../src/macaly-tagger-loader.ts"],"names":[],"mappings":"AAIA,OAAO,KAAK,EACV,kBAAkB,EAInB,MAAM,SAAS,CAAC;AAuFjB,QAAA,MAAM,kBAAkB,EAAE,kBAmIzB,CAAC;AAEF,eAAe,kBAAkB,CAAC"}
+{"version":3,"file":"macaly-tagger-loader.d.ts","sourceRoot":"","sources":["../src/macaly-tagger-loader.ts"],"names":[],"mappings":"AAIA,OAAO,KAAK,EACV,kBAAkB,EAInB,MAAM,SAAS,CAAC;AAgGjB,QAAA,MAAM,kBAAkB,EAAE,kBAqIzB,CAAC;AAEF,eAAe,kBAAkB,CAAC"}

package/dist/macaly-tagger-loader.js

@@ -57,7 +57,7 @@ function isElementAlreadyTagged(attributes) {
         attr.name?.type === 'JSXIdentifier' &&
         attr.name.name === 'data-macaly-loc') ?? false);
 }
-function getJSXElementInfo(node) {
+function getJSXElementInfo(node, debug = false) {
     let tagName;
     let insertPosition;
     if (node.name?.type === 'JSXIdentifier') {
@@ -72,16 +72,25 @@ function getJSXElementInfo(node) {
         insertPosition = node.name.end ?? 0;
     }
     else {
-        console.log('[macaly-tagger] Unsupported JSX element type:', node.name?.type);
+        if (debug) {
+            console.log('[macaly-tagger] Unsupported JSX element type:', node.name?.type);
+        }
         return null;
     }
     if (!tagName || insertPosition === 0) {
-        console.log('[macaly-tagger] Missing tagName or insertPosition:', { tagName, insertPosition });
+        if (debug) {
+            console.log('[macaly-tagger] Missing tagName or insertPosition:', {
+                tagName,
+                insertPosition,
+            });
+        }
         return null;
     }
     const loc = node.loc?.start;
     if (!loc) {
-        console.log('[macaly-tagger] No location info for element:', tagName);
+        if (debug) {
+            console.log('[macaly-tagger] No location info for element:', tagName);
+        }
         return null;
     }
     return {
@@ -91,9 +100,11 @@ function getJSXElementInfo(node) {
     };
 }
 const macalyTaggerLoader = function (code) {
-    console.log('[macaly-tagger] Loader called with:', this.resourcePath);
     const callback = this.async();
     const options = this.getOptions() || {};
+    if (options.debug) {
+        console.log('[macaly-tagger] Loader called with:', this.resourcePath);
+    }
     if (!callback) {
         throw new Error('[macaly-tagger] Async callback not available');
     }
@@ -135,7 +146,7 @@ const macalyTaggerLoader = function (code) {
                             }
                             return;
                         }
-                        const elementInfo = getJSXElementInfo(jsxNode);
+                        const elementInfo = getJSXElementInfo(jsxNode, !!options.debug);
                         if (!elementInfo)
                             return;
                         const { tagName, insertPosition, location } = elementInfo;
@@ -166,7 +177,7 @@ const macalyTaggerLoader = function (code) {
             }
             return {
                 code: transformedCode,
-                map: ms.generateMap({ hires: true }),
+                map: options.disableSourceMaps ? null : ms.generateMap({ hires: true }),
             };
         }
         catch (error) {
@@ -177,7 +188,7 @@ const macalyTaggerLoader = function (code) {
     transform()
         .then((result) => {
         if (result) {
-            callback(null, result.code, result.map);
+            callback(null, result.code, result.map || undefined);
         }
         else {
             callback(null, code);

package/dist/types.d.ts

@@ -1,6 +1,7 @@
 import { LoaderDefinitionFunction } from 'webpack';
 export interface MacalyTaggerLoaderOptions {
     debug?: boolean;
+    disableSourceMaps?: boolean;
 }
 export interface SourceLocation {
     line: number;
@@ -13,7 +14,7 @@ export interface JSXElementInfo {
 }
 export interface TransformResult {
     code: string;
-    map: any;
+    map: any | null;
 }
 export type MacalyTaggerLoader = LoaderDefinitionFunction<MacalyTaggerLoaderOptions>;
 //# sourceMappingURL=types.d.ts.map

package/dist/types.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"types.d.ts","sourceRoot":"","sources":["../src/types.ts"],"names":[],"mappings":"AAAA,OAAO,EAAE,wBAAwB,EAAE,MAAM,SAAS,CAAC;AAEnD,MAAM,WAAW,yBAAyB;IACxC,KAAK,CAAC,EAAE,OAAO,CAAC;CACjB;AAED,MAAM,WAAW,cAAc;IAC7B,IAAI,EAAE,MAAM,CAAC;IACb,MAAM,EAAE,MAAM,CAAC;CAChB;AAED,MAAM,WAAW,cAAc;IAC7B,OAAO,EAAE,MAAM,CAAC;IAChB,cAAc,EAAE,MAAM,CAAC;IACvB,QAAQ,EAAE,cAAc,CAAC;CAC1B;AAED,MAAM,WAAW,eAAe;IAC9B,IAAI,EAAE,MAAM,CAAC;IACb,GAAG,EAAE,GAAG,CAAC;CACV;AAED,MAAM,MAAM,kBAAkB,GAAG,wBAAwB,CAAC,yBAAyB,CAAC,CAAC"}
+{"version":3,"file":"types.d.ts","sourceRoot":"","sources":["../src/types.ts"],"names":[],"mappings":"AAAA,OAAO,EAAE,wBAAwB,EAAE,MAAM,SAAS,CAAC;AAEnD,MAAM,WAAW,yBAAyB;IACxC,KAAK,CAAC,EAAE,OAAO,CAAC;IAChB,iBAAiB,CAAC,EAAE,OAAO,CAAC;CAC7B;AAED,MAAM,WAAW,cAAc;IAC7B,IAAI,EAAE,MAAM,CAAC;IACb,MAAM,EAAE,MAAM,CAAC;CAChB;AAED,MAAM,WAAW,cAAc;IAC7B,OAAO,EAAE,MAAM,CAAC;IAChB,cAAc,EAAE,MAAM,CAAC;IACvB,QAAQ,EAAE,cAAc,CAAC;CAC1B;AAED,MAAM,WAAW,eAAe;IAC9B,IAAI,EAAE,MAAM,CAAC;IACb,GAAG,EAAE,GAAG,GAAG,IAAI,CAAC;CACjB;AAED,MAAM,MAAM,kBAAkB,GAAG,wBAAwB,CAAC,yBAAyB,CAAC,CAAC"}

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "macaly-tagger",
-  "version": "1.0.0",
+  "version": "1.1.0",
   "description": "A webpack loader that adds location metadata to JSX elements for development debugging",
   "main": "dist/index.js",
   "types": "dist/index.d.ts",
@@ -13,22 +13,6 @@
     },
     "./package.json": "./package.json"
   },
-  "scripts": {
-    "build": "tsc -p tsconfig.build.json",
-    "build:watch": "tsc -p tsconfig.build.json --watch",
-    "clean": "rimraf dist",
-    "dev": "tsc -p tsconfig.build.json --watch",
-    "lint": "eslint src --ext .ts,.tsx",
-    "lint:fix": "eslint src --ext .ts,.tsx --fix",
-    "format": "prettier --write \"src/**/*.{ts,tsx}\"",
-    "format:check": "prettier --check \"src/**/*.{ts,tsx}\"",
-    "test": "vitest run",
-    "test:watch": "vitest",
-    "test:coverage": "vitest run --coverage",
-    "type-check": "tsc --noEmit",
-    "prepublishOnly": "npm run clean && npm run build && npm run test && npm run lint",
-    "prepack": "npm run build"
-  },
   "keywords": [
     "webpack",
     "loader",
@@ -84,5 +68,19 @@
   "bugs": {
     "url": "https://github.com/langtail/macaly-tagger/issues"
   },
-  "homepage": "https://github.com/langtail/macaly-tagger#readme"
-}
+  "homepage": "https://github.com/langtail/macaly-tagger#readme",
+  "scripts": {
+    "build": "tsc -p tsconfig.build.json",
+    "build:watch": "tsc -p tsconfig.build.json --watch",
+    "clean": "rimraf dist",
+    "dev": "tsc -p tsconfig.build.json --watch",
+    "lint": "eslint src --ext .ts,.tsx",
+    "lint:fix": "eslint src --ext .ts,.tsx --fix",
+    "format": "prettier --write \"src/**/*.{ts,tsx}\"",
+    "format:check": "prettier --check \"src/**/*.{ts,tsx}\"",
+    "test": "vitest run",
+    "test:watch": "vitest",
+    "test:coverage": "vitest run --coverage",
+    "type-check": "tsc --noEmit"
+  }
+}