repl: runtime deprecate instantiating without new

nodejs · Oct 26, 2024 · 23a70c0 · 23a70c0
1 parent f0e5b6a
commit 23a70c0
Show file tree

Hide file tree

Showing 6 changed files with 32 additions and 15 deletions.
diff --git a/doc/api/deprecations.md b/doc/api/deprecations.md
@@ -3748,14 +3748,17 @@ It is recommended to use the `new` qualifier instead. This applies to all Zlib c
 
 <!-- YAML
 changes:
+  - version: REPLACEME
+    pr-url: https://github.com/nodejs/node/pull/54869
+    description: Runtime deprecation.
   - version:
     - v22.9.0
     - v20.18.0
     pr-url: https://github.com/nodejs/node/pull/54842
     description: Documentation-only deprecation.
 -->
 
-Type: Documentation-only
+Type: Runtime
 
 Instantiating classes without the `new` qualifier exported by the `node:repl` module is deprecated.
 It is recommended to use the `new` qualifier instead. This applies to all REPL classes, including

diff --git a/lib/internal/util.js b/lib/internal/util.js
@@ -97,7 +97,7 @@ function isError(e) {
 // each one once.
 const codesWarned = new SafeSet();
 
-let validateString;
+const lazyValidateString = getLazy(() => require('internal/validators').validateString);
 
 function getDeprecationWarningEmitter(
   code, msg, deprecated, useEmitSync,
@@ -145,12 +145,8 @@ function pendingDeprecate(fn, msg, code) {
 // Returns a modified function which warns once by default.
 // If --no-deprecation is set, then it is a no-op.
 function deprecate(fn, msg, code, useEmitSync) {
-  // Lazy-load to avoid a circular dependency.
-  if (validateString === undefined)
-    ({ validateString } = require('internal/validators'));
-
   if (code !== undefined)
-    validateString(code, 'code');
+    lazyValidateString()(code, 'code');
 
   const emitDeprecationWarning = getDeprecationWarningEmitter(
     code, msg, deprecated, useEmitSync,
@@ -179,6 +175,15 @@ function deprecate(fn, msg, code, useEmitSync) {
   return deprecated;
 }
 
+function deprecateInstantation(target, code, ...args) {
+  if (code !== undefined)
+    lazyValidateString()(code, 'code');
+
+  getDeprecationWarningEmitter(code, `Instantiating ${target.name} without the 'new' keyword has been deprecated.`, target)();
+
+  return ReflectConstruct(target, args);
+}
+
 function decorateErrorStack(err) {
   if (!(isError(err) && err.stack) || err[decorated_private_symbol])
     return;
@@ -873,6 +878,7 @@ module.exports = {
   defineLazyProperties,
   defineReplaceableLazyAttribute,
   deprecate,
+  deprecateInstantation,
   emitExperimentalWarning,
   encodingsMap,
   exposeInterface,

diff --git a/lib/repl.js b/lib/repl.js
@@ -111,6 +111,7 @@ const {
   decorateErrorStack,
   isError,
   deprecate,
+  deprecateInstantation,
   SideEffectFreeRegExpPrototypeSymbolReplace,
   SideEffectFreeRegExpPrototypeSymbolSplit,
 } = require('internal/util');
@@ -262,12 +263,7 @@ function REPLServer(prompt,
                     ignoreUndefined,
                     replMode) {
   if (!(this instanceof REPLServer)) {
-    return new REPLServer(prompt,
-                          stream,
-                          eval_,
-                          useGlobal,
-                          ignoreUndefined,
-                          replMode);
+    return deprecateInstantation(REPLServer, 'DEP0185', prompt, stream, eval_, useGlobal, ignoreUndefined, replMode);
   }
 
   let options;
@@ -1848,6 +1844,8 @@ function defineDefaultCommands(repl) {
 }
 
 function Recoverable(err) {
+  if (!(this instanceof Recoverable))
+    return deprecateInstantation(Recoverable, 'DEP0185', err);
   this.err = err;
 }
 ObjectSetPrototypeOf(Recoverable.prototype, SyntaxErrorPrototype);

diff --git a/test/parallel/test-repl-preview-without-inspector.js b/test/parallel/test-repl-preview-without-inspector.js
@@ -65,7 +65,7 @@ function runAndWait(cmds, repl) {
   return promise;
 }
 
-const repl = REPLServer({
+const repl = new REPLServer({
   prompt: PROMPT,
   stream: new REPLStream(),
   ignoreUndefined: true,

diff --git a/test/parallel/test-repl-preview.js b/test/parallel/test-repl-preview.js
@@ -57,7 +57,7 @@ function runAndWait(cmds, repl) {
 }
 
 async function tests(options) {
-  const repl = REPLServer({
+  const repl = new REPLServer({
     prompt: PROMPT,
     stream: new REPLStream(),
     ignoreUndefined: true,

diff --git a/test/parallel/test-repl.js b/test/parallel/test-repl.js
@@ -1035,3 +1035,13 @@ function event(ee, expected) {
     }));
   });
 }
+
+{
+  const server = repl.REPLServer();
+  common.expectWarning({
+    DeprecationWarning: {
+      DEP0185: 'Instantiating REPLServer without the \'new\' keyword has been deprecated.',
+    }
+  });
+  server.emit('line', '.exit');
+}