[rush] Differentiate remote and local execution in telemetry.

common/changes/@microsoft/rush/cobuild-telemetry-skew_2024-05-30-20-57.json

@@ -0,0 +1,10 @@
+{
+  "changes": [
+    {
+      "packageName": "@microsoft/rush",
+      "comment": "Adds a new `wasExecutedOnThisMachine` property to operation telemetry events, to simplify reporting about cobuilt operations.",
+      "type": "none"
+    }
+  ],
+  "packageName": "@microsoft/rush"
+}

common/reviews/api/rush-lib.api.md

@@ -885,6 +885,7 @@ export interface ITelemetryOperationResult {
     nonCachedDurationMs?: number;
     result: string;
     startTimestampMs?: number;
+    wasExecutedOnThisMachine?: boolean;
 }
 
 // @public

libraries/rush-lib/src/cli/scriptActions/PhasedScriptAction.ts

@@ -804,7 +804,7 @@ export class PhasedScriptAction extends BaseScriptAction<IPhasedCommandConfig> {
       executionManagerOptions
     );
 
-    const { isInitial, isWatch } = options.executeOperationsContext;
+    const { isInitial, isWatch, cobuildConfiguration } = options.executeOperationsContext;
 
     let success: boolean = false;
     let result: IExecutionResult | undefined;
@@ -902,6 +902,9 @@ export class PhasedScriptAction extends BaseScriptAction<IPhasedCommandConfig> {
             startTimestampMs: startTime,
             endTimestampMs: endTime,
             nonCachedDurationMs: operationResult.nonCachedDurationMs,
+            wasExecutedOnThisMachine:
+              !operationResult.cobuildRunnerId ||
+              operationResult.cobuildRunnerId === cobuildConfiguration?.cobuildRunnerId,
             result: operationResult.status,
             dependencies: Array.from(getNonSilentDependencies(operation)).sort()
           };

libraries/rush-lib/src/logic/Telemetry.ts

@@ -71,6 +71,12 @@ export interface ITelemetryOperationResult {
    * Duration in milliseconds when the operation does not hit cache
    */
   nonCachedDurationMs?: number;
+
+  /**
+   * Was this operation built on this machine? If so, the duration can be calculated from `startTimestampMs` and `endTimestampMs`.
+   *  If not, you should use the metrics from the machine that built it.
+   */
+  wasExecutedOnThisMachine?: boolean;
 }
 
 /**