util.inspect({level: "info"}) exits the debug session

[learnerbyheart]

When debugging in node.js 5.3.0 the script bug.js

var util = require('util');
util.inspect({level: "info "});
console.log("Test");

with node debug bug.js, the debugging session exits. I tried the same in node.js 4.2.4 and it doesn't. When debugging the difference in lib/util.js, line 324 in 5.3.0 looks like this:

var promiseInternals = inspectPromise(value);

and line 319 in 4.2.4 looks like:

var promiseInternals = value instanceof Promise && inspectPromise(value);

As value is not an instance of Promise, inspectPromise(value) is called in 5.3.0. Then inspectPromise calls ensureDebugIsInitialized() which on the other hand calls require('vm').runInDebugContext('Debug'). This causes the debugging session to exit. The issue was tested in Ubuntu 14.04 LTS.

[mscdex]

Can you clarify what you're doing step-by-step? You mention running that script with node debug but where does the util.inspect({level: 'info'}) come into play? I ran the script as you say, it breaks on the first statement and continuing execution at the debug> prompt results in "Finished" logged to stdout as expected.

[learnerbyheart]

Sorry, that was an copy and paste fault by me. I updated my initial comment where the right bug.js script is described.

[learnerbyheart]

But the before mentioned script bug2.js

var vm = require('vm');
vm.runInDebugContext('Debug');
console.log("Finished");

exits after vm.runInDebugContext('Debug'); when I run node debug bug2.js.

[mscdex]

I'm not able to reproduce that on master:

$ cat bug.js 
var vm = require('vm');
vm.runInDebugContext('Debug');
console.log("Finished");
$ ./node debug bug.js
< Debugger listening on port 5858
connecting to 127.0.0.1:5858 ... ok
break in bug.js:1
> 1 var vm = require('vm');
  2 vm.runInDebugContext('Debug');
  3 console.log("Finished");
debug> c
< Finished
debug> .exit

[mscdex]

Similarly with your updated code example, I cannot reproduce the problem there either on master:

$ cat bug2.js 
var util = require('util');
util.inspect({level: "info "});
console.log("Test");
$ ./node debug bug2
< Debugger listening on port 5858
connecting to 127.0.0.1:5858 ... ok
break in bug2.js:1
> 1 var util = require('util');
  2 util.inspect({level: "info "});
  3 console.log("Test");
debug> c
< Test
debug> .exit

[mscdex]

Perhaps there's something that needs to be backported to 5.x yet?

[learnerbyheart]

I debugged the script

var vm = require('vm');
vm.runInDebugContext('Debug');
console.log("Finished");

in a new workspace in cloud9 (https://c9.io/) which has node 4.1.1 and Ubuntu 14.04 LTS, on my private Laptop which has node 5.3.0 and Ubuntu 15.10 and on a new VM with a fresh Ubuntu 14.04 LTS and node 4.2.4. In every environment the debugging exits after vm.runInDebugContext('Debug');. Usually I code in WebStorm which exited with exit code 139 when debugging it. When debugging it via node-inspector as well with cloud9 the error message was a segmentation fault.
Does it has something to do with Ubuntu as the operating system? Which os did you use? Does the master branch differs from the 5.3.0 tag?

[mscdex]

Yes, master represents the next major version. Necessary commits are backported to versioned branches (e.g. v4.x, v5.x, etc.).

It shouldn't be OS-related. I don't know offhand what needs to be backported, there's a fair amount of different stuff in master (including a new "major" v8 version).

[Trott]

On OSX, it's broken for me in Node 5.3.0 but works fine with current master.

5.3.0:

$ cat test.js
var vm = require('vm');
vm.runInDebugContext('Debug');
console.log("Finished");
$ node debug test.js 
< Debugger listening on port 5858
connecting to 127.0.0.1:5858 ... ok
break in test.js:1
> 1 var vm = require('vm');
  2 vm.runInDebugContext('Debug');
  3 console.log("Finished");
debug> c
program terminated
debug> .exit

master:

$ ~/io.js/node debug test.js 
< Debugger listening on port 5858
connecting to 127.0.0.1:5858 ... ok
break in test.js:1
> 1 var vm = require('vm');
  2 vm.runInDebugContext('Debug');
  3 console.log("Finished");
debug> c
< Finished
debug> .exit

[Trott]

Maybe it was fixed by 25776f3?

[bnoordhuis]

25776f3 is almost certainly the fix. (EDIT: And hasn't been released yet.)

[learnerbyheart]

In which release will it be inculded? 5.4.0? When will this release be published?

[Trott]

@LucasMoody It should be in the next release which will either be 5.3.1 or 5.4.0 depending on what else is in the release.

For the stable release line (currently 5.x), the goal is to have a release every 1-2 weeks. The release team seems to be meeting that goal. There's probably not much activity this week, though, as it's a major holiday in much of the world. So add one more week maybe.

So, the unofficial upshot from my non-release-team-member analysis would be: A release with the fix should be out by January 15, and probably sooner.