Updated DWDS to include a boolean flag that enables debugging support when set to true (#2601)

* updated Dwds with bool that only enableDebuggingSupport when set to true

* updated Changelog

* formatted file

* updated dwds version in pubspec to 24.3.8

* updated version.dart to 24.3.8

* updated docstring for DWDSInjector class

* updated docstring for DWDSInjector class

* modified docstring

Author: jyameo

dwds/CHANGELOG.md

@@ -1,3 +1,7 @@
+## 24.3.8
+
+- Updated DWDS to include a boolean flag that enables debugging support only when set to true. [#60289](https://github.com/dart-lang/sdk/issues/60289)
+
 ## 24.3.7
 
 - The registered extension `reassemble` is now no longer called when calling

dwds/lib/dart_web_debug_service.dart

@@ -66,6 +66,7 @@ class Dwds {
     required Stream<BuildResult> buildResults,
     required ConnectionProvider chromeConnection,
     required ToolConfiguration toolConfiguration,
+    bool enableDebuggingSupport = true,
   }) async {
     globalToolConfiguration = toolConfiguration;
     final debugSettings = toolConfiguration.debugSettings;
@@ -117,7 +118,10 @@ class Dwds {
       _logger.info('Serving DevTools at $uri\n');
     }
 
-    final injected = DwdsInjector(extensionUri: extensionUri);
+    final injected = DwdsInjector(
+      extensionUri: extensionUri,
+      enableDebuggingSupport: enableDebuggingSupport,
+    );
 
     final devHandler = DevHandler(
       chromeConnection,

dwds/lib/src/handlers/injector.dart

@@ -26,14 +26,29 @@ const mainExtensionMarker = '/* MAIN_EXTENSION_MARKER */';
 
 const _clientScript = 'dwds/src/injected/client';
 
-/// Handles injecting the DWDS client and embedding debugging related
-/// information.
+/// This class is responsible for modifying the served JavaScript files
+/// to include the injected DWDS client, enabling debugging capabilities
+/// and source mapping when running in a browser environment.
+///
+/// The `_enableDebuggingSupport` flag determines whether debugging-related
+/// functionality should be included:
+/// - When `true`, the DWDS client is injected, enabling debugging features.
+/// - When `false`, debugging support is disabled, meaning the application will
+///   run without debugging.
+///
+/// This separation allows for scenarios where debugging is not needed or
+/// should be explicitly avoided.
 class DwdsInjector {
   final Future<String>? _extensionUri;
   final _devHandlerPaths = StreamController<String>();
   final _logger = Logger('DwdsInjector');
+  final bool _enableDebuggingSupport;
 
-  DwdsInjector({Future<String>? extensionUri}) : _extensionUri = extensionUri;
+  DwdsInjector({
+    Future<String>? extensionUri,
+    bool enableDebuggingSupport = true,
+  }) : _extensionUri = extensionUri,
+       _enableDebuggingSupport = enableDebuggingSupport;
 
   /// Returns the embedded dev handler paths.
   ///
@@ -95,13 +110,17 @@ class DwdsInjector {
           await globalToolConfiguration.loadStrategy.trackEntrypoint(
             entrypoint,
           );
-          body = await _injectClientAndHoistMain(
-            body,
-            appId,
-            devHandlerPath,
-            entrypoint,
-            await _extensionUri,
-          );
+          // If true, inject the debugging client and hoist the main function
+          // to enable debugging support.
+          if (_enableDebuggingSupport) {
+            body = await _injectClientAndHoistMain(
+              body,
+              appId,
+              devHandlerPath,
+              entrypoint,
+              await _extensionUri,
+            );
+          }
           body += await globalToolConfiguration.loadStrategy.bootstrapFor(
             entrypoint,
           );

dwds/lib/src/version.dart

@@ -1,6 +1,6 @@
 name: dwds
 # Every time this changes you need to run `dart run build_runner build`.
-version: 24.3.7
+version: 24.3.8
 description: >-
   A service that proxies between the Chrome debug protocol and the Dart VM
   service protocol.