Add IDE, plugin info (#2379)

Author: helin24

pkgs/unified_analytics/CHANGELOG.md

@@ -1,6 +1,8 @@
-## 8.0.14
+## 8.0.15
+- Added IDE and plugin information to `Event.serverSession`.
 
-* Added HCPP related analytics to `Event.commandUsageValues`.
+## 8.0.14
+- Added HCPP related analytics to `Event.commandUsageValues`.
 
 ## 8.0.13
 - Added `DashEnvVar` utilities for environment variable handling.

pkgs/unified_analytics/lib/src/constants.dart

@@ -87,7 +87,7 @@ const int kMaxLogFileSize = 25 * (1 << 20);
 const String kLogFileName = 'dart-flutter-telemetry.log';
 
 /// The current version of the package, should be in line with pubspec version.
-const String kPackageVersion = '8.0.14';
+const String kPackageVersion = '8.0.15';
 
 /// The minimum length for a session.
 const int kSessionDurationMinutes = 30;

pkgs/unified_analytics/lib/src/event.dart

@@ -979,19 +979,35 @@ final class Event {
   /// * [parameters] - the names of the parameters passed to the `initialize`
   ///   request, or an empty string if the `initialize` request was not sent
   ///   or if there were no parameters given.
+  ///
+  /// * [ideName] - the name of the IDE, e.g. IntelliJ IDEA.
+  ///
+  /// * [ideVersion] - the version of the IDE platform.
+  ///
+  /// * [pluginName] - the name of the plugin, e.g. Dart.
+  ///
+  /// * [pluginVersion] - the version of the plugin.
   Event.serverSession({
     required String clientId,
     required String clientVersion,
     required int duration,
     required String flags,
     required String parameters,
+    String ideName = '',
+    String ideVersion = '',
+    String pluginName = '',
+    String pluginVersion = '',
   }) : this._(
          eventName: DashEvent.serverSession,
          eventData: {
            'clientId': clientId,
            'clientVersion': clientVersion,
            'duration': duration,
            'flags': flags,
+           'ideName': ideName,
+           'ideVersion': ideVersion,
+           'pluginName': pluginName,
+           'pluginVersion': pluginVersion,
            'parameters': parameters,
          },
        );

pkgs/unified_analytics/pubspec.yaml

@@ -5,7 +5,7 @@ description: >-
 # LINT.IfChange
 # When updating this, keep the version consistent with the changelog and the
 # value in lib/src/constants.dart.
-version: 8.0.14
+version: 8.0.15
 # LINT.ThenChange(lib/src/constants.dart)
 repository: https://github.com/dart-lang/tools/tree/main/pkgs/unified_analytics
 issue_tracker: https://github.com/dart-lang/tools/issues?q=is%3Aissue+is%3Aopen+label%3Apackage%3Aunified_analytics

pkgs/unified_analytics/test/event_test.dart

@@ -346,7 +346,40 @@ void main() {
     expect(constructedEvent.eventData['duration'], 5);
     expect(constructedEvent.eventData['flags'], 'flags');
     expect(constructedEvent.eventData['parameters'], 'parameters');
-    expect(constructedEvent.eventData.length, 5);
+    expect(constructedEvent.eventData['ideName'], '');
+    expect(constructedEvent.eventData['ideVersion'], '');
+    expect(constructedEvent.eventData['pluginName'], '');
+    expect(constructedEvent.eventData['pluginVersion'], '');
+    expect(constructedEvent.eventData.length, 9);
+  });
+
+  test('Event.serverSession constructed with optional parameters', () {
+    Event generateEvent() => Event.serverSession(
+      clientId: 'clientId',
+      clientVersion: 'clientVersion',
+      duration: 5,
+      flags: 'flags',
+      parameters: 'parameters',
+      ideName: 'ideName',
+      ideVersion: 'ideVersion',
+      pluginName: 'pluginName',
+      pluginVersion: 'pluginVersion',
+    );
+
+    final constructedEvent = generateEvent();
+
+    expect(generateEvent, returnsNormally);
+    expect(constructedEvent.eventName, DashEvent.serverSession);
+    expect(constructedEvent.eventData['clientId'], 'clientId');
+    expect(constructedEvent.eventData['clientVersion'], 'clientVersion');
+    expect(constructedEvent.eventData['duration'], 5);
+    expect(constructedEvent.eventData['flags'], 'flags');
+    expect(constructedEvent.eventData['parameters'], 'parameters');
+    expect(constructedEvent.eventData['ideName'], 'ideName');
+    expect(constructedEvent.eventData['ideVersion'], 'ideVersion');
+    expect(constructedEvent.eventData['pluginName'], 'pluginName');
+    expect(constructedEvent.eventData['pluginVersion'], 'pluginVersion');
+    expect(constructedEvent.eventData.length, 9);
   });
 
   test('Event.severityAdjustment constructed', () {