WIP multiple branches at commit

Signed-off-by: Kipras Melnikovas <kipras@kipras.org>

Author: kiprasmel

branchSequencer.ts

@@ -8,6 +8,7 @@ import { getWantedCommitsWithBranchBoundariesOurCustomImpl } from "./git-stacked
 import { createExecSyncInRepo } from "./util/execSyncInRepo";
 import { Termination } from "./util/error";
 import { assertNever } from "./util/assertNever";
+import { sequentialResolve } from "./util/sequentialResolve";
 
 import { parseNewGoodCommands } from "./parse-todo-of-stacked-rebase/parseNewGoodCommands";
 import { GoodCommand, GoodCommandStacked } from "./parse-todo-of-stacked-rebase/validator";
@@ -25,7 +26,7 @@ export type GetBranchesCtx = BranchRefs & {
 };
 export type SimpleBranchAndCommit = {
 	commitSHA: string | null;
-	branchEndFullName: string;
+	branchEndFullName: string[];
 	// branchExistsYet: boolean; // TODO
 };
 export type GetBoundariesInclInitial = (
@@ -107,7 +108,7 @@ const getBoundariesInclInitialByParsingNotYetAppliedState: GetBoundariesInclInit
 		.map(
 			(cmd): SimpleBranchAndCommit => ({
 				commitSHA: cmd.commitSHAThatBranchPointsTo,
-				branchEndFullName: cmd.targets![0],
+				branchEndFullName: [cmd.targets![0]],
 			})
 		);
 };
@@ -122,7 +123,7 @@ const getBoundariesInclInitialWithSipleBranchTraversal: GetBoundariesInclInitial
 			.filter((b) => !!b.branchEnd)
 			.map(
 				(boundary): SimpleBranchAndCommit => ({
-					branchEndFullName: boundary.branchEnd!.name(), // TS ok because of the filter
+					branchEndFullName: boundary.branchEnd!.map((x) => x.name()), // TS ok because of the filter
 					commitSHA: boundary.commit.sha(),
 				})
 			)
@@ -277,7 +278,7 @@ export const branchSequencer: BranchSequencer = async ({
 			currentBranch,
 		})
 	).map((boundary) => {
-		boundary.branchEndFullName = boundary.branchEndFullName.replace("refs/heads/", "");
+		boundary.branchEndFullName = boundary.branchEndFullName.map((x) => x.replace("refs/heads/", ""));
 		assert(boundary.branchEndFullName);
 
 		/**
@@ -289,7 +290,7 @@ export const branchSequencer: BranchSequencer = async ({
 		// if (!Git.Branch.lookup(repo, targetBranch, Git.Branch.BRANCH.LOCAL)) {
 		// 	execSyncInRepo();
 		// }
-		if (boundary.branchEndFullName.startsWith("refs/remotes/")) {
+		if (boundary.branchEndFullName.some((x) => x.startsWith("refs/remotes/"))) {
 			/**
 			 * TODO - probably should handle this "checkout remote branch locally" logic
 			 * in a better place than here,
@@ -327,7 +328,7 @@ export const branchSequencer: BranchSequencer = async ({
 			 * before doing the checkouts.
 			 *
 			 */
-			boundary.branchEndFullName = boundary.branchEndFullName.replace(/refs\/remotes\/[^/]+\//, "");
+			boundary.branchEndFullName = boundary.branchEndFullName.map((x) => x.replace(/refs\/remotes\/[^/]+\//, ""));
 		}
 
 		// console.log({ targetCommitSHA, target: targetBranch });
@@ -390,13 +391,17 @@ export const branchSequencer: BranchSequencer = async ({
 		// await Git.Checkout.tree(repo, targetBranch as any); // TODO TS FIXME
 		execSyncInRepo(`${gitCmd} checkout ${targetBranch}`); // f this
 
-		await actionInsideEachCheckedOutBranch({
-			repo, //
-			targetBranch,
-			targetCommitSHA,
-			isLatestBranch,
-			execSyncInRepo,
-		});
+		await sequentialResolve(
+			targetBranch.map((x) => async () =>
+				await actionInsideEachCheckedOutBranch({
+					repo, //
+					targetBranch: x,
+					targetCommitSHA,
+					isLatestBranch,
+					execSyncInRepo,
+				})
+			)
+		);
 
 		return goNext();
 	}

git-stacked-rebase.ts

@@ -887,31 +887,34 @@ async function createInitialEditTodoOfGitStackedRebase(
 	const rebaseTodo = commitsWithBranchBoundaries
 		.map(({ commit, commitCommand, branchEnd }, i) => {
 			if (i === 0) {
-				assert(!!branchEnd, `very first commit has a branch (${commit.sha()}).`);
+				assert(!!branchEnd?.length, `very first commit has a branch (${commit.sha()}).`);
+				assert.strictEqual(branchEnd.length, 1, "must be only a single initial branch");
 
 				// return [];
 				return [
 					// `pick ${commit.sha()} ${commit.summary()}`,
 					/**
 					 * TODO refs/REMOTES/* instead of refs/HEADS/*
 					 */
-					`branch-end-initial ${branchEnd.name()}`, //
+					`branch-end-initial ${branchEnd[0].name()}`, //
 				];
 			}
 
 			if (i === commitsWithBranchBoundaries.length - 1) {
-				assert(!!branchEnd, `very last commit has a branch. sha = ${commit.sha()}`);
+				assert(!!branchEnd?.length, `very last commit has a branch. sha = ${commit.sha()}`);
 
 				return [
 					`${commitCommand} ${commit.sha()} ${commit.summary()}`,
-					`branch-end-last ${branchEnd.name()}`, //
+					// `branch-end-last ${branchEnd.name()}`, //
+					`branch-end-last ${initialBranch.name()}`,
 				];
 			}
 
-			if (branchEnd) {
+			if (branchEnd?.length) {
 				return [
 					`${commitCommand} ${commit.sha()} ${commit.summary()}`,
-					`branch-end ${branchEnd.name()}`, //
+					// `branch-end ${branchEnd.name()}`, //
+					`branch-end ${currentBranch.name()}`, //
 				];
 			}
 
@@ -996,7 +999,7 @@ function callAll(keyToFunctionMap: KeyToFunctionMap) {
 type CommitAndBranchBoundary = {
 	commit: Git.Commit;
 	commitCommand: RegularRebaseEitherCommandOrAlias;
-	branchEnd: Git.Reference | null;
+	branchEnd: Git.Reference[] | null;
 };
 
 export async function getWantedCommitsWithBranchBoundariesOurCustomImpl(
@@ -1355,7 +1358,7 @@ async function extendCommitsWithBranchEnds(
 		{
 			commit: c,
 			commitCommand: commandOrAliasNames[i] || "pick",
-			branchEnd: !matchedRefs.length ? null : matchedRefs[0],
+			branchEnd: !matchedRefs.length ? null : matchedRefs,
 		}
 	);