next-build-filter 0.2.1 → 0.2.3

package/lib/advanced-next-build-filter-plugin.js

@@ -60,13 +60,19 @@ class AdvancedNextBuildFilterPlugin {
         const request = resolveData.request;
         const context = resolveData.context;
 
+        // Additional safety check for webpack 4 compatibility
+        if (!request || typeof request !== 'string') {
+          return resolveData;
+        }
+
         if (this.isPageModule(request, context) && this.shouldFilterPage(request)) {
           if (this.options.verbose) {
             console.log(`📄 Intercepting page request: ${request}`);
           }
           
           // Replace with empty module
-          resolveData.request = path.resolve(__dirname, 'empty-module.js');
+          // Use require.resolve instead of path.resolve to ensure compatibility with webpack 4
+          resolveData.request = require.resolve('./empty-module.js');
           this.filteredPages.add(request);
         }
 
@@ -197,6 +203,9 @@ class AdvancedNextBuildFilterPlugin {
   }
 
   normalizePath(filePath) {
+    if (!filePath || typeof filePath !== 'string') {
+      return '';
+    }
     return filePath.replace(/\\/g, '/').toLowerCase();
   }
 }

package/lib/next-build-filter-plugin.js

@@ -64,6 +64,11 @@ class NextBuildFilterPlugin {
 
         const request = resolveData.request;
         
+        // Additional safety check for webpack 4 compatibility
+        if (typeof request !== 'string') {
+          return;
+        }
+        
         // Check if this is a page file
         if (this.isPageFile(request, resolveData.context)) {
           const normalizedRequest = this.normalizePath(request);
@@ -280,6 +285,9 @@ class NextBuildFilterPlugin {
    * Normalize file paths for consistent comparison
    */
   normalizePath(filePath) {
+    if (!filePath || typeof filePath !== 'string') {
+      return '';
+    }
     return filePath.replace(/\\/g, '/').toLowerCase();
   }
 }

package/lib/with-page-filter.js

@@ -32,14 +32,14 @@ function withPageFilter(options = {}) {
     return {
       ...nextConfig,
       
-      webpack: (config, { buildId, dev, isServer, defaultLoaders, webpack }) => {
+      webpack: (config, options) => {
         // Apply the existing webpack config if it exists
         if (nextConfig.webpack) {
-          config = nextConfig.webpack(config, { buildId, dev, isServer, defaultLoaders, webpack });
+          config = nextConfig.webpack(config, options);
         }
 
         // Don't apply filtering in development mode unless explicitly enabled
-        if (dev && !filterOptions.enableInDev) {
+        if (options.dev && !filterOptions.enableInDev) {
           return config;
         }
 
@@ -175,6 +175,9 @@ function extractRoutePath(normalizedRequest, options) {
 }
 
 function normalizePath(filePath) {
+  if (!filePath || typeof filePath !== 'string') {
+    return '';
+  }
   return filePath.replace(/\\/g, '/').toLowerCase();
 }
 

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "next-build-filter",
-  "version": "0.2.1",
+  "version": "0.2.3",
   "description": "A Next.js plugin to exclude pages/routes during build without removing files. Supports both Pages Router and App Router (Next.js 13+).",
   "main": "index.js",
   "types": "index.d.ts",
@@ -17,7 +17,7 @@
     "test:e2e": "node tests/e2e/run-e2e-tests.js",
     "test:all": "npm run test && npm run test:e2e",
     "prepublishOnly": "npm run build",
-    "postversion": "npm publish",
+    "postversion": "npm run test:all && npm publish && git push --follow-tags",
     "build": "echo \"No build process needed for now\""
   },
   "keywords": [