@posthog/webpack-plugin 1.1.1 → 1.1.2

package/dist/index.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"index.d.ts","sourceRoot":"","sources":["../src/index.ts"],"names":[],"mappings":"AAAA,OAAO,EAAE,MAAM,EAAgB,MAAM,eAAe,CAAA;AACpD,OAAO,EAAE,YAAY,EAAiB,oBAAoB,EAAE,MAAM,UAAU,CAAA;AAC5E,OAAO,EAAE,WAAW,EAAS,MAAM,SAAS,CAAA;AAG5C,cAAc,UAAU,CAAA;AAExB,qBAAa,oBAAoB;IAC7B,cAAc,EAAE,oBAAoB,CAAA;IACpC,MAAM,EAAE,MAAM,CAAA;gBAEF,YAAY,EAAE,YAAY;IAatC,KAAK,CAAC,QAAQ,EAAE,GAAG,GAAG,IAAI;IAsBpB,iBAAiB,CAAC,WAAW,EAAE,WAAW,EAAE,MAAM,EAAE,oBAAoB,GAAG,OAAO,CAAC,IAAI,CAAC;CAqCjG"}
+{"version":3,"file":"index.d.ts","sourceRoot":"","sources":["../src/index.ts"],"names":[],"mappings":"AAAA,OAAO,EAAE,MAAM,EAAgB,MAAM,eAAe,CAAA;AACpD,OAAO,EAAE,YAAY,EAAiB,oBAAoB,EAAE,MAAM,UAAU,CAAA;AAC5E,OAAO,EAAE,WAAW,EAAS,MAAM,SAAS,CAAA;AAI5C,cAAc,UAAU,CAAA;AAExB,qBAAa,oBAAoB;IAC7B,cAAc,EAAE,oBAAoB,CAAA;IACpC,MAAM,EAAE,MAAM,CAAA;gBAEF,YAAY,EAAE,YAAY;IAatC,KAAK,CAAC,QAAQ,EAAE,GAAG,GAAG,IAAI;IAmBpB,iBAAiB,CAAC,WAAW,EAAE,WAAW,EAAE,MAAM,EAAE,oBAAoB,GAAG,OAAO,CAAC,IAAI,CAAC;CA+CjG"}

package/dist/index.js

@@ -8,6 +8,9 @@ var __webpack_modules__ = {
     },
     "@posthog/core/process": function(module) {
         module.exports = require("@posthog/core/process");
+    },
+    path: function(module) {
+        module.exports = require("path");
     }
 };
 var __webpack_module_cache__ = {};
@@ -59,6 +62,8 @@ var __webpack_exports__ = {};
     var _posthog_core__WEBPACK_IMPORTED_MODULE_0__ = __webpack_require__("@posthog/core");
     var _config__WEBPACK_IMPORTED_MODULE_1__ = __webpack_require__("./config");
     var _posthog_core_process__WEBPACK_IMPORTED_MODULE_2__ = __webpack_require__("@posthog/core/process");
+    var path__WEBPACK_IMPORTED_MODULE_3__ = __webpack_require__("path");
+    var path__WEBPACK_IMPORTED_MODULE_3___default = /*#__PURE__*/ __webpack_require__.n(path__WEBPACK_IMPORTED_MODULE_3__);
     var __WEBPACK_REEXPORT_OBJECT__ = {};
     for(var __WEBPACK_IMPORT_KEY__ in _config__WEBPACK_IMPORTED_MODULE_1__)if ([
         "default",
@@ -69,8 +74,6 @@ var __webpack_exports__ = {};
     __webpack_require__.d(__webpack_exports__, __WEBPACK_REEXPORT_OBJECT__);
     class PosthogWebpackPlugin {
         apply(compiler) {
-            var _compiler_options_devtool;
-            compiler.options.devtool = null != (_compiler_options_devtool = compiler.options.devtool) ? _compiler_options_devtool : this.resolvedConfig.sourcemaps.enabled ? 'source-map' : void 0;
             const onDone = async (stats, callback)=>{
                 callback = callback || (()=>{});
                 try {
@@ -88,15 +91,17 @@ var __webpack_exports__ = {};
             const outputDirectory = compilation.outputOptions.path;
             const args = [
                 'sourcemap',
-                'process',
-                '--directory',
-                outputDirectory
+                'process'
             ];
-            for (const chunk of compilation.chunks)for (const file of chunk.files)args.push('--include', `**/${file}`);
+            const chunkArray = Array.from(compilation.chunks);
+            if (0 == chunkArray.length) return;
+            const filePaths = chunkArray.flatMap((chunk)=>Array.from(chunk.files).map((file)=>path__WEBPACK_IMPORTED_MODULE_3___default().resolve(outputDirectory, file)));
+            const [commonDirectory, relativeFilePaths] = splitFilePaths(filePaths);
+            args.push('--directory', commonDirectory);
+            for (const chunkPath of relativeFilePaths)args.push('--include', `**/${chunkPath}`);
             if (config.sourcemaps.project) args.push('--project', config.sourcemaps.project);
             if (config.sourcemaps.version) args.push('--version', config.sourcemaps.version);
             if (config.sourcemaps.deleteAfterUpload) args.push('--delete-after');
-            this.logger.info(args);
             await (0, _posthog_core_process__WEBPACK_IMPORTED_MODULE_2__.spawnLocal)(config.cliBinaryPath, args, {
                 cwd: process.cwd(),
                 env: {
@@ -119,6 +124,27 @@ var __webpack_exports__ = {};
     function assertValue(value, message) {
         if (!value) throw new Error(message);
     }
+    function splitFilePaths(absolutePaths) {
+        if (!absolutePaths || 0 === absolutePaths.length) return [
+            process.cwd(),
+            []
+        ];
+        let commonDir = path__WEBPACK_IMPORTED_MODULE_3___default().dirname(absolutePaths[0]);
+        const isCommonAncestor = (candidate)=>absolutePaths.every((p)=>{
+                const rel = path__WEBPACK_IMPORTED_MODULE_3___default().relative(candidate, p);
+                return '' === rel || !rel.startsWith('..') && !path__WEBPACK_IMPORTED_MODULE_3___default().isAbsolute(rel);
+            });
+        while(!isCommonAncestor(commonDir)){
+            const parent = path__WEBPACK_IMPORTED_MODULE_3___default().dirname(commonDir);
+            if (parent === commonDir) break;
+            commonDir = parent;
+        }
+        const relativePaths = absolutePaths.map((p)=>path__WEBPACK_IMPORTED_MODULE_3___default().relative(commonDir, p).replace(/\\/g, '/'));
+        return [
+            commonDir,
+            relativePaths
+        ];
+    }
 })();
 exports.PosthogWebpackPlugin = __webpack_exports__.PosthogWebpackPlugin;
 for(var __webpack_i__ in __webpack_exports__)if (-1 === [

package/dist/index.mjs

@@ -1,11 +1,10 @@
 import { createLogger } from "@posthog/core";
 import { resolveConfig } from "./config.mjs";
 import { spawnLocal } from "@posthog/core/process";
+import path from "path";
 export * from "./config.mjs";
 class PosthogWebpackPlugin {
     apply(compiler) {
-        var _compiler_options_devtool;
-        compiler.options.devtool = null != (_compiler_options_devtool = compiler.options.devtool) ? _compiler_options_devtool : this.resolvedConfig.sourcemaps.enabled ? 'source-map' : void 0;
         const onDone = async (stats, callback)=>{
             callback = callback || (()=>{});
             try {
@@ -23,15 +22,17 @@ class PosthogWebpackPlugin {
         const outputDirectory = compilation.outputOptions.path;
         const args = [
             'sourcemap',
-            'process',
-            '--directory',
-            outputDirectory
+            'process'
         ];
-        for (const chunk of compilation.chunks)for (const file of chunk.files)args.push('--include', `**/${file}`);
+        const chunkArray = Array.from(compilation.chunks);
+        if (0 == chunkArray.length) return;
+        const filePaths = chunkArray.flatMap((chunk)=>Array.from(chunk.files).map((file)=>path.resolve(outputDirectory, file)));
+        const [commonDirectory, relativeFilePaths] = splitFilePaths(filePaths);
+        args.push('--directory', commonDirectory);
+        for (const chunkPath of relativeFilePaths)args.push('--include', `**/${chunkPath}`);
         if (config.sourcemaps.project) args.push('--project', config.sourcemaps.project);
         if (config.sourcemaps.version) args.push('--version', config.sourcemaps.version);
         if (config.sourcemaps.deleteAfterUpload) args.push('--delete-after');
-        this.logger.info(args);
         await spawnLocal(config.cliBinaryPath, args, {
             cwd: process.cwd(),
             env: {
@@ -54,4 +55,25 @@ class PosthogWebpackPlugin {
 function assertValue(value, message) {
     if (!value) throw new Error(message);
 }
+function splitFilePaths(absolutePaths) {
+    if (!absolutePaths || 0 === absolutePaths.length) return [
+        process.cwd(),
+        []
+    ];
+    let commonDir = path.dirname(absolutePaths[0]);
+    const isCommonAncestor = (candidate)=>absolutePaths.every((p)=>{
+            const rel = path.relative(candidate, p);
+            return '' === rel || !rel.startsWith('..') && !path.isAbsolute(rel);
+        });
+    while(!isCommonAncestor(commonDir)){
+        const parent = path.dirname(commonDir);
+        if (parent === commonDir) break;
+        commonDir = parent;
+    }
+    const relativePaths = absolutePaths.map((p)=>path.relative(commonDir, p).replace(/\\/g, '/'));
+    return [
+        commonDir,
+        relativePaths
+    ];
+}
 export { PosthogWebpackPlugin };

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@posthog/webpack-plugin",
-  "version": "1.1.1",
+  "version": "1.1.2",
   "description": "Webpack plugin for Posthog 🦔",
   "repository": {
     "type": "git",
@@ -16,7 +16,7 @@
   "module": "./dist/index.mjs",
   "types": "./dist/index.d.ts",
   "dependencies": {
-    "@posthog/cli": "~0.5.13",
+    "@posthog/cli": "~0.5.16",
     "@posthog/core": "1.6.0"
   },
   "files": [

package/src/index.ts

@@ -2,6 +2,7 @@ import { Logger, createLogger } from '@posthog/core'
 import { PluginConfig, resolveConfig, ResolvedPluginConfig } from './config'
 import { Compilation, Stats } from 'webpack'
 import { spawnLocal } from '@posthog/core/process'
+import path from 'path'
 
 export * from './config'
 
@@ -23,9 +24,6 @@ export class PosthogWebpackPlugin {
     }
 
     apply(compiler: any): void {
-        compiler.options.devtool =
-            compiler.options.devtool ?? (this.resolvedConfig.sourcemaps.enabled ? 'source-map' : undefined)
-
         const onDone = async (stats: Stats, callback: any): Promise<void> => {
             callback = callback || (() => {})
             try {
@@ -46,13 +44,25 @@ export class PosthogWebpackPlugin {
 
     async processSourceMaps(compilation: Compilation, config: ResolvedPluginConfig): Promise<void> {
         const outputDirectory = compilation.outputOptions.path
-        const args = ['sourcemap', 'process', '--directory', outputDirectory]
 
-        for (const chunk of compilation.chunks) {
-            // chunk.files is a Set in webpack 5
-            for (const file of chunk.files) {
-                args.push('--include', `**/${file}`)
-            }
+        // chunks are output outside of the output directory for server chunks
+        const args = ['sourcemap', 'process']
+        const chunkArray = Array.from(compilation.chunks)
+
+        if (chunkArray.length == 0) {
+            // No chunks generated, skipping sourcemap processing.
+            return
+        }
+
+        const filePaths = chunkArray.flatMap((chunk) =>
+            Array.from(chunk.files).map((file) => path.resolve(outputDirectory, file))
+        )
+        const [commonDirectory, relativeFilePaths] = splitFilePaths(filePaths)
+
+        args.push('--directory', commonDirectory)
+
+        for (const chunkPath of relativeFilePaths) {
+            args.push('--include', `**/${chunkPath}`)
         }
 
         if (config.sourcemaps.project) {
@@ -67,8 +77,6 @@ export class PosthogWebpackPlugin {
             args.push('--delete-after')
         }
 
-        this.logger.info(args)
-
         await spawnLocal(config.cliBinaryPath, args, {
             cwd: process.cwd(),
             env: {
@@ -88,3 +96,36 @@ function assertValue(value: any, message: string): void {
         throw new Error(message)
     }
 }
+
+// Convert a list of absolute file path to a common absolute directory ancestor path and relative path
+function splitFilePaths(absolutePaths: string[]): [string, string[]] {
+    if (!absolutePaths || absolutePaths.length === 0) {
+        return [process.cwd(), []]
+    }
+
+    // Start with the directory of the first path and walk up until we find a common ancestor
+    let commonDir = path.dirname(absolutePaths[0])
+
+    const isCommonAncestor = (candidate: string): boolean => {
+        return absolutePaths.every((p) => {
+            const rel = path.relative(candidate, p)
+            // If rel starts with '..' or is absolute, p is not inside candidate
+            return rel === '' || (!rel.startsWith('..') && !path.isAbsolute(rel))
+        })
+    }
+
+    // If the first candidate isn't a common ancestor, walk up the directory tree
+    while (!isCommonAncestor(commonDir)) {
+        const parent = path.dirname(commonDir)
+        if (parent === commonDir) {
+            // reached filesystem root
+            break
+        }
+        commonDir = parent
+    }
+
+    // Compute relative paths from the common directory, normalize to forward slashes for globs
+    const relativePaths = absolutePaths.map((p) => path.relative(commonDir, p).replace(/\\/g, '/'))
+
+    return [commonDir, relativePaths]
+}