refactor(plugins): rename Plugin.experimental to Plugin.optIn (#9802)

The `experimental` static flag never signaled experimental status - it
only decides whether a plugin is enabled by default or requires an
explicit opt-in (via DD_TRACE_<NAME>_ENABLED or tracer.use()). Rename
it to `optIn` so the name matches its only behavior.

Author: rochdev

packages/datadog-plugin-electron/src/index.js

@@ -7,7 +7,7 @@ const ElectronNetPlugin = require('./net')
 
 class ElectronPlugin extends CompositePlugin {
   static id = 'electron'
-  static experimental = DD_MAJOR >= 7
+  static optIn = DD_MAJOR >= 7
   static get plugins () {
     return {
       net: ElectronNetPlugin,

packages/datadog-plugin-fs/src/index.js

@@ -5,7 +5,7 @@ const TracingPlugin = require('../../dd-trace/src/plugins/tracing')
 class FsPlugin extends TracingPlugin {
   static id = 'fs'
   static operation = 'operation'
-  static experimental = true
+  static optIn = true
 
   bindStart (ctx) {
     if (!this.activeSpan) return { noop: true }

packages/datadog-plugin-fs/test/index.spec.js

@@ -25,8 +25,8 @@ describe('Plugin', () => {
       assert.strictEqual(plugins['node:fs'], FsPlugin)
     })
 
-    it('marks the fs plugin experimental so it stays disabled by default', () => {
-      assert.strictEqual(FsPlugin.experimental, true)
+    it('marks the fs plugin opt-in so it stays disabled by default', () => {
+      assert.strictEqual(FsPlugin.optIn, true)
     })
   })
 

packages/datadog-plugin-nats/src/index.js

@@ -8,7 +8,7 @@ class NatsPlugin extends CompositePlugin {
   static id = 'nats'
   // Disabled by default — users must opt in via DD_TRACE_NATS_ENABLED=true
   // or `tracer.use('nats')`. Matches the feature parity dashboard policy.
-  static experimental = true
+  static optIn = true
   static get plugins () {
     return {
       producer: ProducerPlugin,

packages/dd-trace/src/config/major-overrides.js

@@ -61,7 +61,7 @@ function applyMajorOverrides (supportedConfigurations, majorVersion) {
 
   if (majorVersion >= 7) {
     // The Electron plugin moved to the Electron SDK and is opt-in (disabled by default) from v7 on,
-    // while remaining enabled by default on earlier majors (see ElectronPlugin.experimental).
+    // while remaining enabled by default on earlier majors (see ElectronPlugin.optIn).
     const electronEntry = supportedConfigurations.DD_TRACE_ELECTRON_ENABLED?.[0]
     if (electronEntry) electronEntry.default = 'false'
   }

packages/dd-trace/src/plugin_manager.js

@@ -61,9 +61,9 @@ function maybeEnable (Plugin) {
 function getEnabled (Plugin) {
   const envName = `DD_TRACE_${Plugin.id.toUpperCase()}_ENABLED`
   // skipDefault: only an explicitly configured value should drive enablement here. A registered
-  // default of `false` (e.g. an experimental plugin like `nats`) must not be read as an explicit
+  // default of `false` (e.g. an opt-in plugin like `nats`) must not be read as an explicit
   // "disabled via configuration option" — that path both logs a misleading line and nulls the
-  // plugin class, bypassing the experimental opt-in handled by `loadPlugin`.
+  // plugin class, bypassing the opt-in handled by `loadPlugin`.
   return getValueFromEnvSources(normalizePluginEnvName(envName), true)
 }
 
@@ -102,7 +102,7 @@ module.exports = class PluginManager {
     }
     const pluginConfig = this._configsByName[name] || {
       enabled: this._tracerConfig.plugins !== false &&
-        (!Plugin.experimental || isTrue(getEnabled(Plugin))),
+        (!Plugin.optIn || isTrue(getEnabled(Plugin))),
     }
 
     // extracts predetermined configuration from tracer and combines it with plugin-specific config

packages/dd-trace/test/plugin_manager.spec.js

@@ -68,7 +68,7 @@ describe('Plugin Manager', () => {
       },
       seven: {},
       eight: class Eight extends FakePlugin {
-        static experimental = true
+        static optIn = true
         static id = 'eight'
       },
       graphql: class Graphql extends FakePlugin {
@@ -96,7 +96,7 @@ describe('Plugin Manager', () => {
 
     // Mirrors getValueFromEnvSources: an explicit env value wins, otherwise the registered
     // default is returned unless the caller passes skipDefault. registeredDefaults lets a test
-    // model a plugin whose default-enabled flag is `false` (e.g. an experimental plugin).
+    // model a plugin whose default-enabled flag is `false` (e.g. an opt-in plugin).
     registeredDefaults = {}
     PluginManager = proxyquire.noPreserveCache()('../src/plugin_manager', {
       './plugins': { ...plugins, '@noCallThru': true },
@@ -309,7 +309,7 @@ describe('Plugin Manager', () => {
       })
     })
 
-    describe('with an experimental plugin', () => {
+    describe('with an opt-in plugin', () => {
       it('should disable the plugin by default', () => {
         pm.configure(makeTracerConfig())
         loadChannel.publish({ name: 'eight' })