@diegopetrucci/pi-git-footer 0.1.2 → 0.1.4

package/.pi-fleet-tested-version

@@ -1 +1 @@
-0.79.10
+0.80.6

package/index.ts

@@ -31,6 +31,11 @@ type CommandRunner = (
 	options: { cwd: string; signal: AbortSignal },
 ) => Promise<CommandResult>;
 
+type RunCommandSafelyResult =
+	| { kind: "ok"; result: CommandResult }
+	| { kind: "transient" }
+	| { kind: "unavailable" };
+
 type TimerHandle = unknown;
 
 type Clock = {
@@ -286,6 +291,10 @@ function defaultClock(): Clock {
 	};
 }
 
+function isCommandUnavailableError(error: unknown): boolean {
+	return !!error && typeof error === "object" && (error as { code?: unknown }).code === "ENOENT";
+}
+
 class GitFooterCache {
 	private readonly cwd: () => string;
 	private readonly runner: CommandRunner;
@@ -345,7 +354,7 @@ class GitFooterCache {
 		const result = await this.fetchGitStatus();
 		if (this.disposed) return;
 		if (result.kind === "transient") return;
-		if (result.kind === "not-a-repo") {
+		if (result.kind === "not-a-repo" || result.kind === "unavailable") {
 			this.statusSnapshot = undefined;
 			this.pullRequestSnapshot = undefined;
 			this.lastSeenBranch = undefined;
@@ -371,8 +380,8 @@ class GitFooterCache {
 
 		const pr = await this.fetchPullRequest();
 		if (this.disposed) return;
-		if (pr !== undefined) this.pullRequestSnapshot = pr;
-		else if (branchChanged) this.pullRequestSnapshot = undefined;
+		if (pr.kind === "ok") this.pullRequestSnapshot = pr.pullRequest;
+		else if (pr.kind === "not-found" || pr.kind === "unavailable") this.pullRequestSnapshot = undefined;
 
 		this.emitChangeIfSnapshotsChanged(previousStatusSnapshot, previousPullRequestSnapshot);
 	}
@@ -399,32 +408,40 @@ class GitFooterCache {
 		| { kind: "ok"; status: GitStatusSnapshot }
 		| { kind: "not-a-repo" }
 		| { kind: "transient" }
+		| { kind: "unavailable" }
 	> {
 		const result = await this.runCommandSafely("git", GIT_STATUS_ARGS, this.gitTimeoutMs);
-		if (!result) return { kind: "transient" };
-		if (result.exitCode !== 0) return { kind: "not-a-repo" };
-		return { kind: "ok", status: parseGitStatusPorcelainV2(result.stdout) };
+		if (result.kind !== "ok") return result;
+		if (result.result.exitCode !== 0) return { kind: "not-a-repo" };
+		return { kind: "ok", status: parseGitStatusPorcelainV2(result.result.stdout) };
 	}
 
-	private async fetchPullRequest(): Promise<PullRequestSnapshot | undefined> {
+	private async fetchPullRequest(): Promise<
+		| { kind: "ok"; pullRequest: PullRequestSnapshot | undefined }
+		| { kind: "not-found" }
+		| { kind: "transient" }
+		| { kind: "unavailable" }
+	> {
 		const result = await this.runCommandSafely("gh", GH_PR_VIEW_ARGS, this.ghTimeoutMs);
-		if (!result || result.exitCode !== 0) return undefined;
-		return parsePullRequestJson(result.stdout);
+		if (result.kind !== "ok") return result;
+		if (result.result.exitCode !== 0) return { kind: "not-found" };
+		return { kind: "ok", pullRequest: parsePullRequestJson(result.result.stdout) };
 	}
 
 	private async runCommandSafely(
 		command: string,
 		args: readonly string[],
 		timeoutMs: number,
-	): Promise<CommandResult | undefined> {
-		if (this.disposed) return undefined;
+	): Promise<RunCommandSafelyResult> {
+		if (this.disposed) return { kind: "transient" };
 		const controller = new AbortController();
 		this.inflightControllers.add(controller);
 		const timeoutId = setTimeout(() => controller.abort(), timeoutMs);
 		try {
-			return await this.runner(command, args, { cwd: this.cwd(), signal: controller.signal });
-		} catch {
-			return undefined;
+			const result = await this.runner(command, args, { cwd: this.cwd(), signal: controller.signal });
+			return { kind: "ok", result };
+		} catch (error) {
+			return isCommandUnavailableError(error) ? { kind: "unavailable" } : { kind: "transient" };
 		} finally {
 			clearTimeout(timeoutId);
 			this.inflightControllers.delete(controller);
@@ -478,3 +495,14 @@ export default function (pi: ExtensionAPI) {
 		ctx.ui.setStatus(STATUS_KEY, undefined);
 	});
 }
+
+export const __testing = {
+	GitFooterCache,
+	GIT_STATUS_ARGS,
+	GH_PR_VIEW_ARGS,
+	parseGitStatusPorcelainV2,
+	parsePullRequestJson,
+	formatGitStatusFooterSegment,
+	formatPullRequestFooterSegment,
+	formatGitFooterStatus,
+};

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@diegopetrucci/pi-git-footer",
-  "version": "0.1.2",
+  "version": "0.1.4",
   "description": "A TLH-style git status add-on for pi's built-in footer.",
   "keywords": [
     "pi-package",
@@ -31,5 +31,9 @@
   "peerDependencies": {
     "@earendil-works/pi-coding-agent": "*",
     "@earendil-works/pi-tui": "*"
+  },
+  "type": "module",
+  "engines": {
+    "node": ">=22.19.0"
   }
 }