hale-commenting-system 2.2.4 → 2.2.6

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "hale-commenting-system",
-  "version": "2.2.4",
+  "version": "2.2.6",
   "description": "An open source build scaffolding utility for web apps.",
   "repository": "https://github.com/patternfly/patternfly-react-seed.git",
   "homepage": "https://patternfly-react-seed.surge.sh",
@@ -79,8 +79,8 @@
     "@patternfly/react-core": "^6.4.0",
     "@patternfly/react-icons": "^6.4.0",
     "@patternfly/react-styles": "^6.4.0",
-    "inquirer": "^13.1.0",
-    "node-fetch": "^3.3.2",
+    "inquirer": "^8.2.6",
+    "node-fetch": "^2.7.0",
     "react": "^18",
     "react-dom": "^18",
     "sirv-cli": "^3.0.0"

package/scripts/integrate.js

@@ -11,6 +11,27 @@ const fs = require('fs');
 const path = require('path');
 const { execSync } = require('child_process');
 
+// Check Node.js version (fetch is only available natively in Node 18+)
+function checkNodeVersion() {
+  const nodeVersion = process.version;
+  const majorVersion = parseInt(nodeVersion.slice(1).split('.')[0], 10);
+  
+  if (majorVersion < 18) {
+    console.error('❌ Error: Node.js version 18 or higher is required.');
+    console.error(`   Current version: ${nodeVersion}`);
+    console.error('   The webpack middleware uses native fetch() which requires Node 18+.');
+    console.error('   Please upgrade Node.js: https://nodejs.org/\n');
+    process.exit(1);
+  }
+  
+  if (majorVersion === 18) {
+    console.log('⚠️  Warning: Node.js 18 detected. Some features may work better with Node 20+.\n');
+  }
+}
+
+// Run version check immediately
+checkNodeVersion();
+
 // Check if required dependencies are available
 let parser, traverse, generate, types;
 try {
@@ -120,6 +141,15 @@ async function prompt(questions) {
 }
 
 function findFile(filename, startDir = process.cwd()) {
+  // For index.tsx, prioritize src/app/index.tsx over src/index.tsx
+  // because src/app/index.tsx contains the App component with Router
+  if (filename === 'index.tsx') {
+    const appIndexPath = path.join(startDir, 'src', 'app', filename);
+    if (fs.existsSync(appIndexPath)) {
+      return appIndexPath;
+    }
+  }
+  
   const possiblePaths = [
     path.join(startDir, filename),
     path.join(startDir, 'src', filename),
@@ -501,8 +531,10 @@ function integrateWebpackMiddleware() {
   }
 
   // Webpack middleware template (inline since we don't have a separate template file)
+  // Note: This middleware uses native fetch() which requires Node.js 18+
   const middlewareCode = `
       // Load env vars for local OAuth/token exchange without bundling secrets into the client.
+      // Note: Requires Node.js 18+ for native fetch() support
       try {
         const dotenv = require('dotenv');
         dotenv.config({ path: path.resolve(__dirname, '.env') });