preserve the full --inspect= flag as passed to yarn start instead of …

…treating it as a boolean
jaredpalmer · Jun 16, 2018 · 64f098c · 64f098c · jaredpalmer · Jun 16, 2018
1 parent 6aee5de
commit 64f098c
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 9 deletions.
diff --git a/packages/razzle/config/createConfig.js b/packages/razzle/config/createConfig.js
@@ -327,10 +327,10 @@ module.exports = (
       const nodeArgs = ['-r', 'source-map-support/register'];
 
       // Add --inspect or --inspect-brk flag when enabled
-      if (process.env.INSPECT_BRK_ENABLED) {
-        nodeArgs.push('--inspect-brk');
-      } else if (process.env.INSPECT_ENABLED) {
-        nodeArgs.push('--inspect');
+      if (process.env.INSPECT_BRK) {
+        nodeArgs.push(process.env.INSPECT_BRK);
+      } else if (process.env.INSPECT) {
+        nodeArgs.push(process.env.INSPECT);
       }
 
       config.plugins = [

diff --git a/packages/razzle/scripts/start.js b/packages/razzle/scripts/start.js
@@ -14,11 +14,8 @@ const setPorts = require('razzle-dev-utils/setPorts');
 
 process.noDeprecation = true; // turns off that loadQuery clutter.
 
-if (process.argv.includes('--inspect-brk')) {
-  process.env.INSPECT_BRK_ENABLED = true;
-} else if (process.argv.includes('--inspect')) {
-  process.env.INSPECT_ENABLED = true;
-}
+process.env.INSPECT_BRK = process.argv.find((arg) => arg.match(/--inspect-brk(=|$)/)) || '';
+process.env.INSPECT = process.argv.find((arg) => arg.match(/--inspect(=|$)/)) || '';
 
 function main() {
   // Optimistically, we make the console look exactly like the output of our