output telemtry from local results view on state changes

Author: robertbrignull

extensions/ql-vscode/src/view/results/alert-table.tsx

@@ -30,6 +30,7 @@ import {
 import { vscode } from "../vscode-api";
 import { isWholeFileLoc, isLineColumnLoc } from "../../pure/bqrs-utils";
 import { ScrollIntoViewHelper } from "./scroll-into-view-helper";
+import { sendTelemetry } from "../common/telemetry";
 
 export type PathTableProps = ResultTableProps & {
   resultSet: InterpretedResultSet<SarifInterpretationData>;
@@ -64,6 +65,9 @@ export class PathTable extends React.Component<PathTableProps, PathTableState> {
           expanded.add(str);
         }
       }
+      if (expanded) {
+        sendTelemetry("local-results-alert-table-path-expanded");
+      }
       return { expanded };
     });
     e.stopPropagation();
@@ -185,6 +189,7 @@ export class PathTable extends React.Component<PathTableProps, PathTableState> {
           ...previousState,
           selectedItem: resultKey,
         }));
+        sendTelemetry("local-results-alert-table-path-selected");
       };
     };
 

extensions/ql-vscode/src/view/results/raw-results-table.tsx

@@ -18,6 +18,7 @@ import { ResultRow } from "../../pure/bqrs-cli-types";
 import { onNavigation } from "./results";
 import { tryGetResolvableLocation } from "../../pure/bqrs-utils";
 import { ScrollIntoViewHelper } from "./scroll-into-view-helper";
+import { sendTelemetry } from "../common/telemetry";
 
 export type RawTableProps = ResultTableProps & {
   resultSet: RawTableResultSet;
@@ -44,6 +45,7 @@ export class RawTable extends React.Component<RawTableProps, RawTableState> {
       ...prev,
       selectedItem: { row, column },
     }));
+    sendTelemetry("local-results-raw-results-table-selected");
   }
 
   render(): React.ReactNode {

extensions/ql-vscode/src/view/results/result-tables.tsx

@@ -26,6 +26,7 @@ import {
   openFile,
 } from "./result-table-utils";
 import { vscode } from "../vscode-api";
+import { sendTelemetry } from "../common/telemetry";
 
 const FILE_PATH_REGEX = /^(?:.+[\\/])*(.+)$/;
 
@@ -165,6 +166,9 @@ export class ResultTables extends React.Component<
       this.setState({
         problemsViewSelected: e.target.checked,
       });
+      if (e.target.checked) {
+        sendTelemetry("local-results-show-results-in-problems-view");
+      }
       if (resultsPath !== undefined) {
         vscode.postMessage({
           t: "toggleDiagnostics",
@@ -199,6 +203,10 @@ export class ResultTables extends React.Component<
     return parsedResultSets.pageNumber * parsedResultSets.pageSize;
   }
 
+  sendResultsPageChangedTelemetry() {
+    sendTelemetry("local-results-alert-table-page-changed");
+  }
+
   renderPageButtons(): JSX.Element {
     const { parsedResultSets } = this.props;
     const selectedTable = this.state.selectedTable;
@@ -217,6 +225,7 @@ export class ResultTables extends React.Component<
 
     const onChange = (e: React.ChangeEvent<HTMLInputElement>) => {
       this.setState({ selectedPage: e.target.value });
+      this.sendResultsPageChangedTelemetry();
     };
     const choosePage = (input: string) => {
       const pageNumber = parseInt(input);
@@ -239,13 +248,15 @@ export class ResultTables extends React.Component<
         pageNumber: Math.max(parsedResultSets.pageNumber - 1, 0),
         selectedTable,
       });
+      this.sendResultsPageChangedTelemetry();
     };
     const nextPage = (e: React.MouseEvent<HTMLButtonElement, MouseEvent>) => {
       vscode.postMessage({
         t: "changePage",
         pageNumber: Math.min(parsedResultSets.pageNumber + 1, numPages - 1),
         selectedTable,
       });
+      this.sendResultsPageChangedTelemetry();
     };
 
     const openQuery = () => {
@@ -334,6 +345,7 @@ export class ResultTables extends React.Component<
             nonemptyRawResults={nonemptyRawResults}
             showRawResults={() => {
               this.setState({ selectedTable: SELECT_TABLE_NAME });
+              sendTelemetry("local-results-show-raw-results");
             }}
             offset={this.getOffset()}
           />