Skip to content

Commit 716ee15

Browse files
Trottrichardlau
Trott
authored and
richardlau
committed
debugger: rename internal library for clarity
When I moved these files from node-inspect to Node.js core, I put them in lib/internal/inspector. That was a mistake. They should be in lib/internal/debugger. PR-URL: #39080 Backport-PR-URL: #39446 Reviewed-By: Luigi Pinca <luigipinca@gmail.com> Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
1 parent b7ee9d8 commit 716ee15

File tree

5 files changed

+6
-6
lines changed

5 files changed

+6
-6
lines changed

lib/internal/inspector/_inspect.js renamed to lib/internal/debugger/_inspect.js

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -43,8 +43,8 @@ const console = require('internal/console/global');
4343

4444
const { 0: InspectClient, 1: createRepl } =
4545
[
46-
require('internal/inspector/inspect_client'),
47-
require('internal/inspector/inspect_repl'),
46+
require('internal/debugger/inspect_client'),
47+
require('internal/debugger/inspect_repl'),
4848
];
4949

5050
const debuglog = util.debuglog('inspect');

lib/internal/inspector/inspect_client.js renamed to lib/internal/debugger/inspect_client.js

File renamed without changes.

lib/internal/inspector/inspect_repl.js renamed to lib/internal/debugger/inspect_repl.js

File renamed without changes.

‎lib/internal/main/inspect.js‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -18,5 +18,5 @@ markBootstrapComplete();
1818

1919
// Start the debugger agent.
2020
process.nextTick(() => {
21-
require('internal/inspector/_inspect').start();
21+
require('internal/debugger/_inspect').start();
2222
});

‎node.gyp‎

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -125,6 +125,9 @@
125125
'lib/internal/crypto/sig.js',
126126
'lib/internal/crypto/util.js',
127127
'lib/internal/constants.js',
128+
'lib/internal/debugger/_inspect.js',
129+
'lib/internal/debugger/inspect_client.js',
130+
'lib/internal/debugger/inspect_repl.js',
128131
'lib/internal/dgram.js',
129132
'lib/internal/dns/promises.js',
130133
'lib/internal/dns/utils.js',
@@ -148,9 +151,6 @@
148151
'lib/internal/heap_utils.js',
149152
'lib/internal/histogram.js',
150153
'lib/internal/idna.js',
151-
'lib/internal/inspector/_inspect.js',
152-
'lib/internal/inspector/inspect_client.js',
153-
'lib/internal/inspector/inspect_repl.js',
154154
'lib/internal/inspector_async_hook.js',
155155
'lib/internal/js_stream_socket.js',
156156
'lib/internal/linkedlist.js',

0 commit comments

Comments
 (0)