feat: Add LU editor telemetry and machine info to the SessionStarted event

Composer/packages/electron-server/src/main.ts

@@ -1,6 +1,7 @@
 // Copyright (c) Microsoft Corporation.
 // Licensed under the MIT License.
 
+import os from 'os';
 import { join, resolve } from 'path';
 
 import { AppUpdaterSettings, UserSettings } from '@bfc/shared';
@@ -163,9 +164,14 @@ async function loadServer() {
     getARMTokenForTenant: OneAuthService.getARMTokenForTenant.bind(OneAuthService),
     getTenants: OneAuthService.getTenants.bind(OneAuthService),
     logOut: OneAuthService.signOut.bind(OneAuthService),
-    machineId,
-    sessionId,
-    composerVersion: app.getVersion(),
+    telemetryData: {
+      composerVersion: app.getVersion(),
+      machineId,
+      sessionId,
+      architecture: os.arch(),
+      cpus: os.cpus().length,
+      memory: os.totalmem(),
+    },
   });
   log(`Server started at port: ${serverPort}`);
 }

Composer/packages/lib/code-editor/src/LuEditor.tsx

@@ -1,7 +1,7 @@
 // Copyright (c) Microsoft Corporation.
 // Licensed under the MIT License.
 
-import { LuFile } from '@botframework-composer/types';
+import { LuFile, TelemetryClient } from '@botframework-composer/types';
 import styled from '@emotion/styled';
 import { EditorDidMount, Monaco } from '@monaco-editor/react';
 import { FluentTheme, NeutralColors } from '@uifabric/fluent-theme';
@@ -76,6 +76,7 @@ export interface LULSPEditorProps extends BaseEditorProps {
       }
     | string;
   toolbarHidden?: boolean;
+  telemetryClient: TelemetryClient;
   onNavigateToLuPage?: (luFileId: string, luSectionId?: string) => void;
 }
 
@@ -135,6 +136,7 @@ const LuEditor: React.FC<LULSPEditorProps> = (props) => {
     onInit: onInitProp,
     placeholder = defaultPlaceholder,
     helpURL = LU_HELP,
+    telemetryClient,
     ...restProps
   } = props;
   const luServer = languageServer || defaultLUServer;
@@ -239,7 +241,7 @@ const LuEditor: React.FC<LULSPEditorProps> = (props) => {
           if (luEdits?.scrollLine) {
             editor.revealLineInCenter(luEdits?.scrollLine);
           }
-
+          telemetryClient.track('LUEditorToolbarEntityDefinitionAdded', { entityType });
           editor.focus();
         }
       }
@@ -248,11 +250,15 @@ const LuEditor: React.FC<LULSPEditorProps> = (props) => {
   );
 
   const insertEntity = useCallback(
-    (entityName: string) => {
+    (entityName: string, entityType: string, method: 'toolbar' | 'floatingMenu' = 'toolbar') => {
       if (editor) {
         const edits = computeInsertLuEntityEdits(entityName, editor);
         if (edits) {
           editor.executeEdits('toolbarMenu', edits);
+          telemetryClient.track('LUEditorToolbarEntityTagAdded', {
+            entityType: entityType !== 'prebuilt' ? entityType : entityName,
+            method,
+          });
           editor.focus();
         }
       }

Composer/packages/lib/code-editor/src/lu/InsertEntityButton.tsx

@@ -30,7 +30,7 @@ const buttonStyles = {
 };
 
 type Props = {
-  onInsertEntity: (entityName: string) => void;
+  onInsertEntity: (entityName: string, eventType: string) => void;
   labelingMenuVisible: boolean;
   insertEntityDisabled: boolean;
   tagEntityDisabled: boolean;
@@ -47,7 +47,7 @@ export const InsertEntityButton = React.memo((props: Props) => {
     (_, item?: IContextualMenuItem) => {
       const entity = item?.data as LuEntity;
       if (entity) {
-        onInsertEntity(entity.Name);
+        onInsertEntity(entity.Name, entity.Type);
       }
     },
     [onInsertEntity]

Composer/packages/lib/code-editor/src/lu/LuEditorToolbar.tsx

@@ -29,7 +29,7 @@ type Props = {
   luFile?: LuFile;
   labelingMenuVisible: boolean;
   onDefineEntity: (entityType: ToolbarLuEntityType, entityName?: string) => void;
-  onInsertEntity: (entityName: string) => void;
+  onInsertEntity: (entityName: string, entityType: string) => void;
 };
 
 export const LuEditorToolbar = React.memo((props: Props) => {

Composer/packages/lib/code-editor/src/lu/LuLabelingMenu.tsx

@@ -15,7 +15,7 @@ type Props = {
   editor: any;
   luFile?: LuFile;
   onMenuToggled?: (visible: boolean) => void;
-  onInsertEntity: (entityName: string) => void;
+  onInsertEntity: (entityName: string, entityType: string, method: 'toolbar' | 'floatingMenu') => void;
 };
 
 export const LuLabelingMenu = ({ editor, luFile, onMenuToggled, onInsertEntity }: Props) => {
@@ -79,7 +79,7 @@ export const LuLabelingMenu = ({ editor, luFile, onMenuToggled, onInsertEntity }
     (_, item?: IContextualMenuItem) => {
       const entity = item?.data as LuEntity;
       if (entity) {
-        onInsertEntity(entity.Name);
+        onInsertEntity(entity.Name, entity.Type, 'floatingMenu');
       }
       setMenuTargetElm(null);
     },

Composer/packages/server/src/services/telemetry.ts

@@ -18,9 +18,11 @@ const getTelemetryContext = () => {
   const electronContext = useElectronContext();
 
   if (electronContext) {
-    const { sessionId, machineId, composerVersion } = electronContext;
+    const {
+      telemetryData: { composerVersion, sessionId, machineId, architecture, cpus, memory },
+    } = electronContext;
     const { telemetry = {} } = SettingsService.getSettings();
-    return { sessionId, userId: machineId, telemetry, composerVersion };
+    return { sessionId, userId: machineId, telemetry, composerVersion, architecture, cpus, memory };
   }
 
   return {};
@@ -39,7 +41,7 @@ if (instrumentationKey) {
   // do not collect the user's machine name
   AppInsights.defaultClient.context.tags[AppInsights.defaultClient.context.keys.cloudRoleInstance] = '';
   AppInsights.defaultClient.addTelemetryProcessor((envelope: AppInsights.Contracts.Envelope, context): boolean => {
-    const { sessionId, telemetry, composerVersion, userId } = getTelemetryContext();
+    const { sessionId, telemetry, composerVersion, userId, architecture, cpus, memory } = getTelemetryContext();
 
     if (!telemetry?.allowDataCollection) {
       return false;
@@ -70,6 +72,20 @@ if (instrumentationKey) {
         data.baseData.properties.composerVersion = composerVersion;
       }
 
+      if (data.baseData.name === 'SessionStarted') {
+        if (architecture) {
+          data.baseData.properties.architecture = architecture;
+        }
+
+        if (cpus !== undefined) {
+          data.baseData.properties.cpu = cpu;
+        }
+
+        if (memory !== undefined) {
+          data.baseData.properties.memory = memory;
+        }
+      }
+
       // remove PII
       for (const property of piiProperties) {
         if (data.baseData.properties[property] !== undefined) {

Composer/packages/server/src/utility/electronContext.ts

@@ -10,9 +10,14 @@ export type ElectronContext = {
   getARMTokenForTenant: (tenantId: string) => Promise<string>;
   getTenants: () => Promise<AzureTenant[]>;
   logOut: () => void;
-  machineId: string;
-  sessionId: string;
-  composerVersion: string;
+  telemetryData: {
+    composerVersion: string;
+    machineId: string;
+    sessionId: string;
+    architecture: string;
+    cpus: number;
+    memory: number;
+  };
 };
 
 let context;

Composer/packages/types/src/telemetry.ts

@@ -135,6 +135,11 @@ type LgEditorEvents = {
   };
 };
 
+type LuEditorEvents = {
+  LUEditorToolbarEntityTagAdded: { entityType: string; method: 'toolbar' | 'floatingMenu' };
+  LUEditorToolbarEntityDefinitionAdded: { entityType: string };
+};
+
 type WebChatEvents = {
   WebChatPaneOpened: undefined;
   WebChatPaneClosed: undefined;
@@ -185,7 +190,8 @@ export type TelemetryEvents = ApplicationEvents &
   AppSettingsEvents &
   PageView &
   LgEditorEvents &
-  WebChatEvents;
+  WebChatEvents &
+  LuEditorEvents;
 
 export type TelemetryEventName = keyof TelemetryEvents;
 

Composer/packages/ui-plugins/luis/src/LuisIntentEditor.tsx

@@ -81,6 +81,7 @@ const LuisIntentEditor: React.FC<FieldProps<string>> = (props) => {
       luFile={luFile}
       luOption={{ fileId: luFile.id, sectionId: luIntent.Name, projectId, luFeatures }}
       placeholder={placeholder || inlineModePlaceholder}
+      telemetryClient={shellApi.telemetryClient}
       value={luIntent.Body}
       onChange={commitChanges}
       onChangeSettings={handleSettingsChange}