@slicemachine/manager 0.25.1-beta.2 → 0.25.2-alpha.dani-git-integration.1

package/src/lib/fetchGitHubReleaseBodyForRelease.ts

@@ -103,13 +103,18 @@ const _fetchGitHubReleaseBodyForRelease = async (
 	const cache = args.cache || {};
 
 	if (Object.keys(cache).length < 1) {
-		const releases = await fetchAllGitHubReleases({
-			repositoryOwner: args.repositoryOwner,
-			repositoryName: args.repositoryName,
-		});
+		try {
+			const releases = await fetchAllGitHubReleases({
+				repositoryOwner: args.repositoryOwner,
+				repositoryName: args.repositoryName,
+			});
 
-		for (const release of releases) {
-			cache[release.name] = release;
+			for (const release of releases) {
+				cache[release.name] = release;
+			}
+		} catch (error) {
+			// noop - Fetch all releases failed, no need to track this error in Sentry.
+			return undefined;
 		}
 	}
 

package/src/managers/SliceMachineManager.ts

@@ -17,6 +17,7 @@ import {
 	PrismicUserProfile,
 } from "../auth/PrismicAuthManager";
 import { createPrismicAuthManager } from "../auth/createPrismicAuthManager";
+import { UnauthenticatedError } from "../errors";
 
 import { API_ENDPOINTS, APIEndpoints } from "../constants/API_ENDPOINTS";
 
@@ -207,16 +208,17 @@ export class SliceMachineManager {
 								name: "__stub__",
 								message: "__stub__",
 								reason: "__stub__",
-								status: 401,
+								status: 403,
 							};
 						} else if (
+							error instanceof UnauthenticatedError ||
 							error instanceof prismicCustomTypesClient.UnauthorizedError
 						) {
 							authError = {
 								name: "__stub__",
 								message: "__stub__",
 								reason: "__stub__",
-								status: 403,
+								status: 401,
 							};
 						} else {
 							throw error;

package/src/managers/project/ProjectManager.ts

@@ -85,9 +85,9 @@ type ProjectManagerUpdateEnvironmentArgs = {
 	environment: string | undefined;
 };
 
-type ProjectManagerFetchActiveEnvironmentReturnType = {
-	activeEnvironment: Environment;
-};
+type ProjectManagerFetchActiveEnvironmentReturnType =
+	| { type: "ok"; activeEnvironment: Environment }
+	| { type: "error"; error: InvalidActiveEnvironmentError };
 
 export class ProjectManager extends BaseManager {
 	private _cachedRoot: string | undefined;
@@ -451,7 +451,7 @@ export class ProjectManager extends BaseManager {
 			this._cachedEnvironments || [],
 		);
 		if (cachedActiveEnvironment) {
-			return { activeEnvironment: cachedActiveEnvironment };
+			return { type: "ok", activeEnvironment: cachedActiveEnvironment };
 		}
 
 		// If the environment is not in the cached environments list, we
@@ -472,10 +472,13 @@ export class ProjectManager extends BaseManager {
 		);
 
 		if (!activeEnvironment) {
-			throw new InvalidActiveEnvironmentError();
+			return {
+				type: "error",
+				error: new InvalidActiveEnvironmentError(),
+			};
 		}
 
-		return { activeEnvironment };
+		return { type: "ok", activeEnvironment: activeEnvironment };
 	}
 
 	async detectVersionControlSystem(): Promise<string | "_unknown"> {

package/src/managers/telemetry/TelemetryManager.ts

@@ -116,9 +116,14 @@ export class TelemetryManager extends BaseManager {
 		if (_includeEnvironmentKind) {
 			if (this.project.checkSupportsEnvironments()) {
 				try {
-					const { activeEnvironment } =
+					const activeEnvironmentResult =
 						await this.project.fetchActiveEnvironment();
-					environmentKind = activeEnvironment.kind;
+
+					if (activeEnvironmentResult.type === "ok") {
+						environmentKind = activeEnvironmentResult.activeEnvironment.kind;
+					} else {
+						throw activeEnvironmentResult.error;
+					}
 				} catch {
 					environmentKind = "_unknown";
 				}