From 94be10ac32ca02a3a5e3191d8908bb0b2ec247aa Mon Sep 17 00:00:00 2001
From: Antoine du Hamel <duhamelantoine1995@gmail.com>
Date: Fri, 8 Nov 2024 16:16:15 +0000
Subject: [PATCH] util: do not mark experimental feature as deprecated

PR-URL: https://github.com/nodejs/node/pull/55740
Reviewed-By: Chengzhong Wu <legendecas@gmail.com>
Reviewed-By: Chemi Atlow <chemi@atlow.co.il>
Reviewed-By: Marco Ippolito <marcoippolito54@gmail.com>
---
 doc/api/deprecations.md                | 14 +-------------
 doc/api/util.md                        |  4 ++++
 lib/internal/util.js                   |  4 +++-
 lib/util.js                            |  4 ++--
 test/parallel/test-util-getcallsite.js |  4 ++--
 5 files changed, 12 insertions(+), 18 deletions(-)

diff --git a/doc/api/deprecations.md b/doc/api/deprecations.md
index ad371609432..bcfa2932c7e 100644
--- a/doc/api/deprecations.md
+++ b/doc/api/deprecations.md
@@ -3764,18 +3764,7 @@ Instantiating classes without the `new` qualifier exported by the `node:repl` mo
 It is recommended to use the `new` qualifier instead. This applies to all REPL classes, including
 `REPLServer` and `Recoverable`.
 
-### DEP0186: `util.getCallSite`
-
-<!-- YAML
-changes:
-  - version: REPLACEME
-    pr-url: https://github.com/nodejs/node/pull/55626
-    description: Runtime deprecation.
--->
-
-Type: Runtime
-
-The `util.getCallSite` API has been removed. Please use [`util.getCallSites()`][] instead.
+<!-- md-lint skip-deprecation DEP0186 -->
 
 [NIST SP 800-38D]: https://nvlpubs.nist.gov/nistpubs/Legacy/SP/nistspecialpublication800-38d.pdf
 [RFC 6066]: https://tools.ietf.org/html/rfc6066#section-3
@@ -3902,7 +3891,6 @@ The `util.getCallSite` API has been removed. Please use [`util.getCallSites()`][
 [`url.parse()`]: url.md#urlparseurlstring-parsequerystring-slashesdenotehost
 [`url.resolve()`]: url.md#urlresolvefrom-to
 [`util._extend()`]: util.md#util_extendtarget-source
-[`util.getCallSites()`]: util.md#utilgetcallsitesframecountoroptions-options
 [`util.getSystemErrorName()`]: util.md#utilgetsystemerrornameerr
 [`util.inspect()`]: util.md#utilinspectobject-options
 [`util.inspect.custom`]: util.md#utilinspectcustom
diff --git a/doc/api/util.md b/doc/api/util.md
index 6b7855e47ff..43b61033831 100644
--- a/doc/api/util.md
+++ b/doc/api/util.md
@@ -370,6 +370,10 @@ util.formatWithOptions({ colors: true }, 'See object %O', { foo: 42 });
 
 <!-- YAML
 added: v22.9.0
+changes:
+  - version: REPLACEME
+    pr-url: https://github.com/nodejs/node/pull/55626
+    description: The API is renamed from `util.getCallSite` to `util.getCallSites()`.
 -->
 
 * `frameCount` {number} Optional number of frames to capture as call site objects.
diff --git a/lib/internal/util.js b/lib/internal/util.js
index c6de8b31402..887ae8874e1 100644
--- a/lib/internal/util.js
+++ b/lib/internal/util.js
@@ -108,7 +108,9 @@ function getDeprecationWarningEmitter(
   return function() {
     if (!warned && shouldEmitWarning()) {
       warned = true;
-      if (code !== undefined) {
+      if (code === 'ExperimentalWarning') {
+        process.emitWarning(msg, code, deprecated);
+      } else if (code !== undefined) {
         if (!codesWarned.has(code)) {
           const emitWarning = useEmitSync ?
             require('internal/process/warning').emitWarningSync :
diff --git a/lib/util.js b/lib/util.js
index 6642d4de9d1..591a1832fec 100644
--- a/lib/util.js
+++ b/lib/util.js
@@ -436,8 +436,8 @@ module.exports = {
   // Deprecated getCallSite.
   // This API can be removed in next semver-minor release.
   getCallSite: deprecate(getCallSites,
-                         'The `util.getCallSite` API is deprecated. Please use `util.getCallSites()` instead.',
-                         'DEP0186'),
+                         'The `util.getCallSite` API has been renamed to `util.getCallSites()`.',
+                         'ExperimentalWarning'),
   getCallSites,
   getSystemErrorMap,
   getSystemErrorName,
diff --git a/test/parallel/test-util-getcallsite.js b/test/parallel/test-util-getcallsite.js
index 5c9570fe32d..34b52e9b20a 100644
--- a/test/parallel/test-util-getcallsite.js
+++ b/test/parallel/test-util-getcallsite.js
@@ -4,6 +4,6 @@ require('../common');
 const { getCallSite } = require('node:util');
 const { expectWarning } = require('../common');
 
-const warning = 'The `util.getCallSite` API is deprecated. Please use `util.getCallSites()` instead.';
-expectWarning('DeprecationWarning', warning, 'DEP0186');
+const warning = 'The `util.getCallSite` API has been renamed to `util.getCallSites()`.';
+expectWarning('ExperimentalWarning', warning);
 getCallSite();