fix: avoid extra async config lookup when repository is provided

Combine the starter and repositoryName resolution into a single
getSliceMachineConfig() call, and only attempt it when repositoryName
is not already provided by the caller. This avoids an additional async
file traversal in the init error path, which was causing timing-sensitive
tests to fail.

Made-with: Cursor

Author: levimykel

packages/manager/src/managers/telemetry/TelemetryManager.ts

@@ -102,23 +102,18 @@ export class TelemetryManager extends BaseManager {
 	async track(args: TelemetryManagerTrackArgs): Promise<void> {
 		const { event, repository, _includeEnvironmentKind, ...properties } = args;
 		let repositoryName = repository;
+		let starter: string | undefined;
 
 		if (repositoryName === undefined) {
 			try {
-				repositoryName = await this.project.getRepositoryName();
-			} catch (error) {
-				// noop, happen only when the user is not in a project
+				const config = await this.project.getSliceMachineConfig();
+				repositoryName = config.repositoryName;
+				starter = config.starter;
+			} catch {
+				// noop, happens only when the user is not in a project
 			}
 		}
 
-		let starter: string | undefined;
-		try {
-			const config = await this.project.getSliceMachineConfig();
-			starter = config.starter;
-		} catch {
-			// noop, happen only when the user is not in a project
-		}
-
 		let environmentKind: Environment["kind"] | "_unknown" | undefined =
 			undefined;
 		if (_includeEnvironmentKind) {