bstack 1.3.0 → 1.5.0

package/dist/bstack.mjs

@@ -6,7 +6,7 @@ import * as v from "valibot";
 import { mkdirSync, readFileSync, renameSync, writeFileSync } from "node:fs";
 import { dirname } from "node:path";
 //#region package.json
-var version = "1.3.0";
+var version = "1.5.0";
 //#endregion
 //#region src/command.ts
 var CommandError = class extends Error {
@@ -265,9 +265,14 @@ var GitCliRepository = class {
 	}
 	pushBranches(remote, branches) {
 		const existing = this.remoteBranchOids(remote, branches.map((branch) => branch.name));
+		const updates = branches.filter((branch) => existing.get(branch.name) !== branch.oid);
+		if (updates.length === 0) return {
+			checked: branches.length,
+			updated: []
+		};
 		const leases = [];
 		const refspecs = [];
-		for (const branch of branches) {
+		for (const branch of updates) {
 			const expected = existing.get(branch.name) ?? "";
 			leases.push(`--force-with-lease=refs/heads/${branch.name}:${expected}`);
 			refspecs.push(`${branch.oid}:refs/heads/${branch.name}`);
@@ -279,7 +284,10 @@ var GitCliRepository = class {
 			remote,
 			...refspecs
 		]);
-		return branches.filter((branch) => existing.get(branch.name) !== branch.oid).map((branch) => branch.name);
+		return {
+			checked: branches.length,
+			updated: updates.map((branch) => branch.name)
+		};
 	}
 	statePath() {
 		return this.git([
@@ -510,6 +518,25 @@ var GitHubCliPlatform = class {
 	}
 };
 //#endregion
+//#region src/output.ts
+function formatSyncResult(result, dryRun) {
+	const changeCount = `${result.changes.length} change${result.changes.length === 1 ? "" : "s"}`;
+	const lines = [dryRun ? `Would sync ${changeCount} against ${result.base}:` : `Synced ${result.changes.length}-commit stack against ${result.base}:`];
+	if (dryRun) {
+		for (const change of result.changes) lines.push(`  ${change.oid.slice(0, 8)}  ${change.subject}`);
+		return lines.join("\n");
+	}
+	for (const outcome of result.outcomes) {
+		if (outcome.outcome === "closed") {
+			const { pullRequest } = outcome;
+			lines.push(`  ${outcome.outcome.padEnd(9)} #${pullRequest.number}  ${pullRequest.title} ${pullRequest.url}`);
+			continue;
+		}
+		lines.push(`  ${outcome.outcome.padEnd(9)} #${outcome.pullRequest.number}  ${outcome.change.subject} ${outcome.pullRequest.url}`);
+	}
+	return lines.join("\n");
+}
+//#endregion
 //#region src/reporter.ts
 var ConsoleReporter = class {
 	progress(message) {
@@ -584,6 +611,13 @@ var Stack = class Stack {
 		this.rewritten = rewritten;
 		this.rewrites = rewrites;
 		if (changes.length === 0) throw new Error("A stack must contain at least one change");
+		const seen = /* @__PURE__ */ new Set();
+		const duplicate = changes.find((change) => {
+			if (seen.has(change.id)) return true;
+			seen.add(change.id);
+			return false;
+		});
+		if (duplicate) throw new Error(`A stack cannot contain duplicate bstack-id ${duplicate.id}`);
 	}
 	static fromChanges(changes) {
 		return new Stack(changes);
@@ -633,7 +667,17 @@ var Stack = class Stack {
 		const removed = previous.changes.filter((change) => !currentIdSet.has(change.id));
 		const added = this.changes.filter((change) => !previousIdSet.has(change.id));
 		if (removed.length === 0) {
-			if (previousIds.every((id, index) => currentIds[index] === id)) return { kind: "full" };
+			const previousIsPrefix = previousIds.every((id, index) => currentIds[index] === id);
+			if (previousIsPrefix && added.length === 0) return { kind: "skip" };
+			if (previousIsPrefix) {
+				const stackNumber = previous.stackNumber ?? options.lookups.stackNumberForPullRequest(previous.changes[0].pullRequest);
+				if (stackNumber !== void 0) return {
+					kind: "append",
+					stackNumber,
+					branches: added.map((change) => change.remoteBranch)
+				};
+				return { kind: "full" };
+			}
 			const stackNumber = previous.stackNumber ?? options.lookups.stackNumberForPullRequest(previous.changes[0].pullRequest);
 			return stackNumber === void 0 ? { kind: "full" } : {
 				kind: "rebuild",
@@ -675,6 +719,73 @@ var Stack = class Stack {
 	}
 };
 //#endregion
+//#region src/sync-transition.ts
+function prepareStackTransition(transition, github, previous, base, remote, reporter) {
+	if (transition.kind !== "rebuild" || transition.action !== "reorder") return;
+	reporter.progress(`Preparing stack #${transition.stackNumber} for reordered branches`);
+	github.unstack(transition.stackNumber);
+	try {
+		for (const change of previous.changes) github.editPullRequestBase(github.pullRequest(change.pullRequest), base);
+	} catch (error) {
+		restorePreviousStack(github, previous, base, remote, reporter, error);
+	}
+}
+function applyStackTransition(transition, github, previous, changes, pullRequests, base, remote, draft, reporter) {
+	if (changes.length === 1) {
+		if (transition.kind === "partial") reporter.progress("Updating this down-stack prefix while preserving higher pull requests");
+		if (transition.kind === "collapse") {
+			reporter.progress(`Removing omitted pull requests from stack #${transition.stackNumber}`);
+			github.unstack(transition.stackNumber);
+			try {
+				github.editPullRequestBase(pullRequests[0], base);
+			} catch (error) {
+				restorePreviousStack(github, previous, base, remote, reporter, error);
+			}
+		}
+		return;
+	}
+	if (transition.kind === "full") {
+		reporter.progress(`Linking ${changes.length} pull requests as a native GitHub stack`);
+		github.linkStack(pullRequests.map((pullRequest) => pullRequest.number), base, remote, draft);
+		return;
+	}
+	if (transition.kind === "rebuild") {
+		reporter.progress(`Rebuilding stack #${transition.stackNumber} to ${transition.action} pull requests`);
+		if (transition.action !== "reorder") github.unstack(transition.stackNumber);
+		try {
+			github.linkStack(pullRequests.map((pullRequest) => pullRequest.number), base, remote, draft);
+		} catch (error) {
+			restorePreviousStack(github, previous, base, remote, reporter, error);
+		}
+		return;
+	}
+	if (transition.kind === "append") {
+		reporter.progress(`Appending ${transition.branches.length} pull request${transition.branches.length === 1 ? "" : "s"} to stack #${transition.stackNumber}`);
+		github.appendToStack(transition.stackNumber, transition.branches.map((branch) => {
+			const pullRequest = pullRequests[changes.findIndex((change) => change.remoteBranch === branch)];
+			if (!pullRequest) throw new Error(`Missing pull request for ${branch}`);
+			return pullRequest.number;
+		}), remote, draft);
+		return;
+	}
+	if (transition.kind === "partial") {
+		reporter.progress("Updating this down-stack prefix while preserving higher pull requests");
+		return;
+	}
+	reporter.progress("The native GitHub stack already has the correct members");
+}
+function restorePreviousStack(github, previous, base, remote, reporter, rebuildError) {
+	const rebuildMessage = rebuildError instanceof Error ? rebuildError.message : String(rebuildError);
+	reporter.progress("Rebuild failed; restoring the previous native GitHub stack");
+	try {
+		github.linkStack(previous.changes.map((change) => change.pullRequest), base, remote, true);
+	} catch (rollbackError) {
+		const rollbackMessage = rollbackError instanceof Error ? rollbackError.message : String(rollbackError);
+		throw new Error(`Stack rebuild failed: ${rebuildMessage}\nRestoring the previous stack also failed: ${rollbackMessage}`);
+	}
+	throw rebuildError;
+}
+//#endregion
 //#region src/sync.ts
 function syncStack(dependencies, options) {
 	const { repository, github, stateStore, reporter } = dependencies;
@@ -703,7 +814,8 @@ function syncStack(dependencies, options) {
 			base,
 			remote,
 			rewritten,
-			changes: [...changes]
+			changes: [...changes],
+			outcomes: []
 		};
 	}
 	reporter.progress("Reading the previous stack state");
@@ -716,24 +828,16 @@ function syncStack(dependencies, options) {
 			stackNumberForPullRequest: (pullRequest) => github.stackNumberForPullRequest(pullRequest)
 		}
 	});
+	prepareStackTransition(transition, github, previous, base, remote, reporter);
 	const isReorder = transition.kind === "rebuild" && transition.action === "reorder";
-	if (isReorder) {
-		reporter.progress(`Preparing stack #${transition.stackNumber} for reordered branches`);
-		github.unstack(transition.stackNumber);
-		try {
-			for (const change of previous.changes) github.editPullRequestBase(github.pullRequest(change.pullRequest), base);
-		} catch (error) {
-			restorePreviousStack(github, previous, base, remote, reporter, error);
-		}
-	}
-	reporter.progress(`Pushing ${changes.length} remote branch${changes.length === 1 ? "" : "es"}`);
 	let pushedBranches = /* @__PURE__ */ new Set();
 	try {
-		const updatedBranches = repository.pushBranches(remote, changes.map((change) => ({
+		const pushResult = repository.pushBranches(remote, changes.map((change) => ({
 			name: change.remoteBranch,
 			oid: change.oid
 		})));
-		pushedBranches = new Set(updatedBranches);
+		reportPushResult(reporter, pushResult);
+		pushedBranches = new Set(pushResult.updated);
 	} catch (error) {
 		if (isReorder) restorePreviousStack(github, previous, base, remote, reporter, error);
 		throw error;
@@ -748,39 +852,7 @@ function syncStack(dependencies, options) {
 		return github.createPullRequest(change, pullRequestBase, options.draft);
 	});
 	const createdBranches = new Set(changes.filter((_change, index) => existing[index] === void 0).map((change) => change.remoteBranch));
-	if (changes.length === 1) {
-		if (transition.kind === "partial") reporter.progress("Updating this down-stack prefix while preserving higher pull requests");
-		const pullRequest = pullRequests[0];
-		if (transition.kind === "collapse") {
-			reporter.progress(`Removing omitted pull requests from stack #${transition.stackNumber}`);
-			github.unstack(transition.stackNumber);
-			try {
-				github.editPullRequestBase(pullRequest, base);
-			} catch (error) {
-				restorePreviousStack(github, previous, base, remote, reporter, error);
-			}
-		}
-	} else if (transition.kind === "full") {
-		reporter.progress(`Linking ${changes.length} pull requests as a native GitHub stack`);
-		github.linkStack(pullRequests.map((pullRequest) => pullRequest.number), base, remote, options.draft);
-	} else if (transition.kind === "rebuild") {
-		reporter.progress(`Rebuilding stack #${transition.stackNumber} to ${transition.action} pull requests`);
-		if (!isReorder) github.unstack(transition.stackNumber);
-		try {
-			github.linkStack(pullRequests.map((pullRequest) => pullRequest.number), base, remote, options.draft);
-		} catch (error) {
-			restorePreviousStack(github, previous, base, remote, reporter, error);
-		}
-	} else if (transition.kind === "append") {
-		reporter.progress(`Appending ${transition.branches.length} pull request${transition.branches.length === 1 ? "" : "s"} to stack #${transition.stackNumber}`);
-		github.appendToStack(transition.stackNumber, transition.branches.map((branch) => {
-			const index = changes.findIndex((change) => change.remoteBranch === branch);
-			const pullRequest = pullRequests[index];
-			if (!pullRequest) throw new Error(`Missing pull request for ${branch}`);
-			return pullRequest.number;
-		}), remote, options.draft);
-	} else if (transition.kind === "partial") reporter.progress("Updating this down-stack prefix while preserving higher pull requests");
-	else reporter.progress("The native GitHub stack already has the correct members");
+	applyStackTransition(transition, github, previous, changes, pullRequests, base, remote, options.draft, reporter);
 	const currentIds = new Set(changes.map((change) => change.id));
 	const omittedPullRequests = transition.kind === "partial" ? [] : (previous?.changes ?? []).filter((change) => !currentIds.has(change.id)).map((change) => github.pullRequest(change.pullRequest)).filter((pullRequest) => pullRequest.state === "OPEN");
 	if (omittedPullRequests.length > 0) {
@@ -807,15 +879,25 @@ function syncStack(dependencies, options) {
 	if (stackNumber !== void 0) updatedStack.stackNumber = stackNumber;
 	writeUpdatedState(stateStore, state, previous, updatedStack);
 	reporter.progress("Saved the local stack state");
+	const synchronized = changes.map((change, index) => ({
+		...change,
+		pullRequest: pullRequests[index]
+	}));
+	const outcomes = synchronized.map((change, index) => ({
+		outcome: changeOutcome(change, index, change.pullRequest, previous, changes, base, createdBranches, pushedBranches),
+		change,
+		pullRequest: change.pullRequest
+	}));
+	outcomes.push(...omittedPullRequests.map((pullRequest) => ({
+		outcome: "closed",
+		pullRequest
+	})));
 	return {
 		base,
 		remote,
 		rewritten,
-		changes: changes.map((change, index) => ({
-			...change,
-			pullRequest: pullRequests[index],
-			outcome: changeOutcome(change, index, pullRequests[index], previous, changes, base, createdBranches, pushedBranches)
-		}))
+		changes: synchronized,
+		outcomes
 	};
 }
 function changeOutcome(change, index, pullRequest, previous, changes, base, createdBranches, pushedBranches) {
@@ -826,16 +908,12 @@ function changeOutcome(change, index, pullRequest, previous, changes, base, crea
 	const metadataChanged = pullRequest.title !== change.subject || pullRequest.body !== change.body;
 	return pushedBranches.has(change.remoteBranch) || previousBase !== currentBase || metadataChanged ? "updated" : "unchanged";
 }
-function restorePreviousStack(github, previous, base, remote, reporter, rebuildError) {
-	const rebuildMessage = rebuildError instanceof Error ? rebuildError.message : String(rebuildError);
-	reporter.progress("Rebuild failed; restoring the previous native GitHub stack");
-	try {
-		github.linkStack(previous.changes.map((change) => change.pullRequest), base, remote, true);
-	} catch (rollbackError) {
-		const rollbackMessage = rollbackError instanceof Error ? rollbackError.message : String(rollbackError);
-		throw new Error(`Stack rebuild failed: ${rebuildMessage}\nRestoring the previous stack also failed: ${rollbackMessage}`);
+function reportPushResult(reporter, result) {
+	if (result.updated.length === 0) {
+		reporter.progress(`All ${result.checked} remote branch${result.checked === 1 ? "" : "es"} already match`);
+		return;
 	}
-	throw rebuildError;
+	reporter.progress(`Updating ${result.updated.length} of ${result.checked} remote branch${result.checked === 1 ? "" : "es"}`);
 }
 function writeUpdatedState(store, state, previous, updated) {
 	const stacks = previous ? state.stacks.map((stack) => stack === previous ? updated : stack) : [...state.stacks, updated];
@@ -939,13 +1017,7 @@ function main() {
 		draft: values.draft,
 		dryRun: values["dry-run"]
 	});
-	const changeCount = `${result.changes.length} change${result.changes.length === 1 ? "" : "s"}`;
-	console.log(values["dry-run"] ? `Would sync ${changeCount} against ${result.base}:` : `Synced stack against ${result.base} (${changeCount}):`);
-	for (const change of result.changes) {
-		const outcome = change.outcome ? `${change.outcome.padEnd(9)} ` : "";
-		const destination = change.pullRequest ? ` ${change.pullRequest.url}` : "";
-		console.log(`  ${outcome}${change.oid.slice(0, 8)}  ${change.subject}${destination}`);
-	}
+	console.log(formatSyncResult(result, values["dry-run"]));
 }
 try {
 	main();

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "bstack",
-  "version": "1.3.0",
+  "version": "1.5.0",
   "description": "Create native GitHub stacked pull requests from a linear series of commits",
   "license": "MIT",
   "repository": {
@@ -32,6 +32,7 @@
     "valibot": "^1.4.2"
   },
   "devDependencies": {
+    "@total-typescript/shoehorn": "^0.1.2",
     "@types/node": "^26.0.0",
     "oxfmt": "^0.64.0",
     "oxlint": "^1.79.0",