Normalizes machineId across VS Code clones

Author: d13

src/env/browser/machine.ts

@@ -0,0 +1,4 @@
+// This is intentional as the browser environment doesn't have access to MAC addresses
+export function getMac(): string | undefined {
+	return undefined;
+}

src/env/node/machine.ts

@@ -0,0 +1,21 @@
+import { networkInterfaces } from 'os';
+
+// Sourced from https://github.com/bevry/getmac/blob/master/source/index.ts
+// There's issues with importing 'getmac' directly, so we referenced the relevant code here
+
+const zeroRegex = /(?:[0]{1,2}[:-]){5}[0]{1,2}/;
+export function getMac(): string | undefined {
+	const list = networkInterfaces();
+
+	for (const parts of Object.values(list)) {
+		// for some reason beyond me, this is needed to satisfy typescript
+		// fix https://github.com/bevry/getmac/issues/100
+		if (!parts) continue;
+		for (const part of parts) {
+			if (zeroRegex.test(part.mac) === false) {
+				return part.mac;
+			}
+		}
+	}
+	return undefined;
+}

src/extension.ts

@@ -23,6 +23,7 @@ import { registerPartnerActionRunners } from './partners';
 import { executeCommand, registerCommands } from './system/-webview/command';
 import { configuration, Configuration } from './system/-webview/configuration';
 import { setContext } from './system/-webview/context';
+import { getMachineId } from './system/-webview/machine';
 import { Storage } from './system/-webview/storage';
 import { deviceCohortGroup } from './system/-webview/vscode';
 import { isTextDocument } from './system/-webview/vscode/documents';
@@ -56,7 +57,7 @@ export async function activate(context: ExtensionContext): Promise<GitLensApi |
 							env.appName
 						} (${codeVersion}) on the ${isWeb ? 'web' : 'desktop'}; language='${
 							env.language
-						}', logLevel='${logLevel}', defaultDateLocale='${defaultDateLocale}' (${env.machineId}|${
+						}', logLevel='${logLevel}', defaultDateLocale='${defaultDateLocale}' (${getMachineId()}|${
 							env.sessionId
 						})`,
 					);
@@ -110,7 +111,7 @@ export async function activate(context: ExtensionContext): Promise<GitLensApi |
 		log: {
 			message: ` activating in ${env.appName} (${codeVersion}) on the ${isWeb ? 'web' : 'desktop'}; language='${
 				env.language
-			}', logLevel='${logLevel}', defaultDateLocale='${defaultDateLocale}' (${env.uriScheme}|${env.machineId}|${
+			}', logLevel='${logLevel}', defaultDateLocale='${defaultDateLocale}' (${env.uriScheme}|${getMachineId()}|${
 				env.sessionId
 			})`,
 			//${context.extensionRuntime !== ExtensionRuntime.Node ? ' in a webworker' : ''}

src/plus/gk/subscriptionService.ts

@@ -48,6 +48,7 @@ import type { RepositoriesChangeEvent } from '../../git/gitProviderService';
 import { executeCommand, registerCommand } from '../../system/-webview/command';
 import { configuration } from '../../system/-webview/configuration';
 import { setContext } from '../../system/-webview/context';
+import { getMachineId } from '../../system/-webview/machine';
 import { openUrl } from '../../system/-webview/vscode/uris';
 import { createFromDateDelta, fromNow } from '../../system/date';
 import { gate } from '../../system/decorators/-webview/gate';
@@ -1025,7 +1026,7 @@ export class SubscriptionService implements Disposable {
 				id: session.account.id,
 				platform: getPlatform(),
 				gitlensVersion: this.container.version,
-				machineId: env.machineId,
+				machineId: getMachineId(),
 				sessionId: env.sessionId,
 				vscodeEdition: env.appName,
 				vscodeHost: env.appHost,

src/system/-webview/machine.ts

@@ -0,0 +1,10 @@
+import { env } from 'vscode';
+import { getMac } from '@env/machine';
+import { isWeb } from '@env/platform';
+
+export function getMachineId(): string {
+	if (isWeb) {
+		return env.machineId;
+	}
+	return getMac() ?? env.machineId;
+}

src/system/-webview/vscode.ts

@@ -8,9 +8,10 @@ import { getDistributionGroup } from '../string';
 import { satisfies } from '../version';
 import { executeCoreCommand } from './command';
 import { configuration } from './configuration';
+import { getMachineId } from './machine';
 import { exists } from './vscode/uris';
 
-export const deviceCohortGroup = getDistributionGroup(env.machineId);
+export const deviceCohortGroup = getDistributionGroup(getMachineId());
 
 let _hostExecutablePath: string | undefined;
 export async function getHostExecutablePath(): Promise<string> {

src/telemetry/telemetry.ts

@@ -6,6 +6,7 @@ import { getPlatform } from '@env/platform';
 import type { Source, TelemetryEventData, TelemetryEvents, TelemetryGlobalContext } from '../constants.telemetry';
 import type { Container } from '../container';
 import { configuration } from '../system/-webview/configuration';
+import { getMachineId } from '../system/-webview/machine';
 
 export interface TelemetryContext {
 	env: string;
@@ -99,7 +100,7 @@ export class TelemetryService implements Disposable {
 				env: container.env,
 				extensionId: container.id,
 				extensionVersion: container.version,
-				machineId: env.machineId,
+				machineId: getMachineId(),
 				sessionId: env.sessionId,
 				language: env.language,
 				platform: getPlatform(),