Skip to content
This repository was archived by the owner on Apr 3, 2024. It is now read-only.

Commit 47a6fc8

Browse files
authored
chore: remove dead code for supporting node < 10 (#1141)
1 parent 1b5314d commit 47a6fc8

File tree

3 files changed

+5
-670
lines changed

3 files changed

+5
-670
lines changed

src/agent/v8/debugapi.ts

Lines changed: 4 additions & 32 deletions
Original file line numberDiff line numberDiff line change
@@ -14,10 +14,10 @@
1414

1515
import consoleLogLevel = require('console-log-level');
1616
import * as stackdriver from '../../types/stackdriver';
17-
import {DebugAgentConfig} from '../config';
17+
import {ResolvedDebugAgentConfig} from '../config';
1818
import {ScanStats} from '../io/scanner';
1919
import {SourceMapper} from '../io/sourcemapper';
20-
import * as utils from '../util/utils';
20+
import {InspectorDebugApi} from './inspector-debugapi';
2121

2222
export interface DebugApi {
2323
set(
@@ -42,34 +42,6 @@ export interface DebugApi {
4242
numListeners_(): number;
4343
}
4444

45-
interface DebugApiConstructor {
46-
new (
47-
logger: consoleLogLevel.Logger,
48-
config: DebugAgentConfig,
49-
jsFiles: ScanStats,
50-
sourcemapper: SourceMapper
51-
): DebugApi;
52-
}
53-
54-
let debugApiConstructor: DebugApiConstructor;
55-
56-
export function willUseInspector(nodeVersion?: string) {
57-
// checking for null and undefined.
58-
// eslint-disable-next-line eqeqeq
59-
const version = nodeVersion != null ? nodeVersion : process.version;
60-
return utils.satisfies(version, '>=10');
61-
}
62-
63-
if (willUseInspector()) {
64-
// eslint-disable-next-line @typescript-eslint/no-var-requires
65-
const inspectorapi = require('./inspector-debugapi');
66-
debugApiConstructor = inspectorapi.InspectorDebugApi;
67-
} else {
68-
// eslint-disable-next-line @typescript-eslint/no-var-requires
69-
const v8debugapi = require('./legacy-debugapi');
70-
debugApiConstructor = v8debugapi.V8DebugApi;
71-
}
72-
7345
export const MODULE_WRAP_PREFIX_LENGTH = require('module')
7446
.wrap('☃')
7547
.indexOf('☃');
@@ -78,7 +50,7 @@ let singleton: DebugApi;
7850

7951
export function create(
8052
logger: consoleLogLevel.Logger,
81-
config: DebugAgentConfig,
53+
config: ResolvedDebugAgentConfig,
8254
jsFiles: ScanStats,
8355
sourcemapper: SourceMapper
8456
): DebugApi {
@@ -88,6 +60,6 @@ export function create(
8860
singleton.disconnect();
8961
}
9062

91-
singleton = new debugApiConstructor(logger, config, jsFiles, sourcemapper);
63+
singleton = new InspectorDebugApi(logger, config, jsFiles, sourcemapper);
9264
return singleton;
9365
}

0 commit comments

Comments
 (0)