git-stack-cli 2.5.3 → 2.6.1

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "git-stack-cli",
-  "version": "2.5.3",
+  "version": "2.6.1",
   "description": "",
   "author": "magus",
   "license": "MIT",
@@ -36,7 +36,7 @@
     "test": "bun test",
     "test:watch": "pnpm run test --watch",
     "test:types": "tsc",
-    "test:all": "pnpm run prettier:check && pnpm run lint:check && pnpm run test:types",
+    "test:all": "pnpm run prettier:check && pnpm run lint:check && pnpm run test:types && pnpm run test",
     "prepublishOnly": "bun run scripts/npm-prepublishOnly.ts"
   },
   "dependencies": {

package/scripts/bun-build.ts

@@ -111,6 +111,11 @@ if (!WATCH) {
       continue;
     }
 
+    // ignore bun build files
+    if (filename.includes(".bun-build")) {
+      continue;
+    }
+
     log(`⚠️ Change ${filename}`);
 
     if (VERBOSE) {

package/src/app/App.tsx

@@ -14,6 +14,7 @@ import { Main } from "~/app/Main";
 import { Output } from "~/app/Output";
 import { Providers } from "~/app/Providers";
 import { RebaseCheck } from "~/app/RebaseCheck";
+import { RequireBranch } from "~/app/RequireBranch";
 import { Store } from "~/app/Store";
 import { VerboseDebugInfo } from "~/app/VerboseDebugInfo";
 import { Fixup } from "~/commands/Fixup";
@@ -112,11 +113,13 @@ function MaybeMain() {
       <DependencyCheck>
         <DirtyCheck>
           <GatherMetadata>
-            <LocalCommitStatus>
-              <DetectInitialPR>
-                <Main />
-              </DetectInitialPR>
-            </LocalCommitStatus>
+            <RequireBranch>
+              <LocalCommitStatus>
+                <DetectInitialPR>
+                  <Main />
+                </DetectInitialPR>
+              </LocalCommitStatus>
+            </RequireBranch>
           </GatherMetadata>
         </DirtyCheck>
       </DependencyCheck>

package/src/app/AutoUpdate.tsx

@@ -6,6 +6,7 @@ import { Brackets } from "~/app/Brackets";
 import { Command } from "~/app/Command";
 import { FormatText } from "~/app/FormatText";
 import { YesNoPrompt } from "~/app/YesNoPrompt";
+import { assertNever } from "~/core/assertNever";
 import { cli } from "~/core/cli";
 import { colors } from "~/core/colors";
 import { fetch_json } from "~/core/fetch_json";
@@ -28,7 +29,7 @@ type State = {
   error: null | Error;
   local_version: null | string;
   latest_version: null | string;
-  status: "init" | "prompt" | "install" | "done" | "exit";
+  status: "init" | "prompt" | "install" | "done";
   is_brew_bun_standalone: boolean;
 };
 
@@ -61,7 +62,24 @@ export function AutoUpdate(props: Props) {
   }
 
   React.useEffect(() => {
-    let status: State["status"] = "done";
+    switch (state.status) {
+      case "init":
+      case "prompt":
+      case "install":
+        break;
+
+      case "done": {
+        props.onDone?.();
+        break;
+      }
+
+      default:
+        assertNever(state.status);
+    }
+  }, [state.status]);
+
+  React.useEffect(() => {
+    let status: State["status"] = "init";
     let latest_version: string | null = null;
     let is_brew_bun_standalone = false;
 
@@ -73,17 +91,13 @@ export function AutoUpdate(props: Props) {
         throw new Error("Auto update requires process.env.CLI_VERSION to be set");
       }
 
-      if (is_output) {
-        handle_output(<Ink.Text key="init">Checking for latest version...</Ink.Text>);
-      }
-
       const timeout_ms = is_finite_value(props.timeoutMs) ? props.timeoutMs : 2 * 1000;
 
       const npm_json = await Promise.race([
         fetch_json(`https://registry.npmjs.org/${props.name}`),
 
         sleep(timeout_ms).then(() => {
-          throw new Error("Timeout");
+          throw new Error("AutoUpdate timeout");
         }),
       ]);
 
@@ -126,8 +140,13 @@ export function AutoUpdate(props: Props) {
       }
 
       const semver_result = semver_compare(latest_version, local_version);
+      if (props_ref.current.verbose) {
+        handle_output(<Ink.Text dimColor>{JSON.stringify({ semver_result })}</Ink.Text>);
+      }
 
       if (semver_result === 0) {
+        status = "done";
+
         if (is_output) {
           handle_output(
             <Ink.Text>
@@ -138,15 +157,12 @@ export function AutoUpdate(props: Props) {
         return;
       }
 
-      if (semver_result === -1) {
-        // latest version is less than or equal to local version, skip auto update
-        throw new Error(
-          `latest version < local_version, skipping auto update [${latest_version} < ${local_version}]`,
-        );
+      if (semver_result === 1) {
+        // trigger yes no prompt
+        status = "prompt";
       }
 
-      // trigger yes no prompt
-      status = "prompt";
+      throw new Error("AutoUpdate failed");
     }
 
     const onError = props_ref.current.onError || (() => {});
@@ -156,9 +172,6 @@ export function AutoUpdate(props: Props) {
         patch({ status, local_version, latest_version, is_brew_bun_standalone });
       })
       .catch((error) => {
-        patch({ status, error, local_version, latest_version, is_brew_bun_standalone });
-        onError(error);
-
         if (props_ref.current.verbose) {
           handle_output(
             <Ink.Text key="error" color={colors.red}>
@@ -166,9 +179,11 @@ export function AutoUpdate(props: Props) {
             </Ink.Text>,
           );
         }
-      })
-      .finally(() => {
-        props.onDone?.();
+
+        // ensure we always exit
+        status = "done";
+        patch({ status, error, local_version, latest_version, is_brew_bun_standalone });
+        onError(error);
       });
   }, []);
 
@@ -180,54 +195,60 @@ export function AutoUpdate(props: Props) {
       case "prompt": {
         let install_command = "";
         if (state.is_brew_bun_standalone) {
-          install_command = `npm install -g ${props.name}@latest`;
+          install_command = "brew install magus/git-stack/git-stack";
         } else {
-          install_command = "brew upgrade magus/git-stack/git-stack";
+          install_command = `npm install -g ${props.name}@latest`;
         }
 
         return (
           <YesNoPrompt
             message={
               <Ink.Box flexDirection="column">
-                <Ink.Text color={colors.yellow}>
+                <Ink.Box flexDirection="column">
+                  <Ink.Text color={colors.yellow}>
+                    <FormatText
+                      wrapper={<Ink.Text />}
+                      message="New version available {latest_version}"
+                      values={{
+                        latest_version: <Brackets>{state.latest_version}</Brackets>,
+                      }}
+                    />
+                    ,
+                  </Ink.Text>
+                  <Ink.Text> </Ink.Text>
+                  <Command>{install_command}</Command>
+                  <Ink.Text> </Ink.Text>
+                </Ink.Box>
+                <Ink.Box>
                   <FormatText
-                    wrapper={<Ink.Text />}
-                    message="New version available {latest_version}, would you like to update?"
-                    values={{
-                      latest_version: <Brackets>{state.latest_version}</Brackets>,
-                    }}
+                    wrapper={<Ink.Text color={colors.yellow} />}
+                    message="Would you like to run the above command to update?"
                   />
-                  ,
-                </Ink.Text>
-                <Ink.Text> </Ink.Text>
-                <Command>{install_command}</Command>
-                <Ink.Text> </Ink.Text>
-                <FormatText
-                  wrapper={<Ink.Text color={colors.yellow} />}
-                  message="Would you like to run the above command to update?"
-                />
+                </Ink.Box>
               </Ink.Box>
             }
             onYes={async () => {
-              handle_output(
-                <FormatText
-                  key="install"
-                  wrapper={<Ink.Text />}
-                  message="Installing {name}@{version}..."
-                  values={{
-                    name: <Ink.Text color={colors.yellow}>{props.name}</Ink.Text>,
-                    version: <Ink.Text color={colors.blue}>{state.latest_version}</Ink.Text>,
-                  }}
-                />,
-              );
+              handle_output(<Command>{install_command}</Command>);
 
               patch({ status: "install" });
 
-              await cli(install_command);
+              await cli(install_command, {
+                env: {
+                  ...process.env,
+                  HOMEBREW_COLOR: "1",
+                },
+                onOutput: (data: string) => {
+                  handle_output(<Ink.Text>{data}</Ink.Text>);
+                },
+              });
 
-              patch({ status: "exit" });
+              handle_output(
+                <Ink.Text key="done">
+                  ✅ Installed <Brackets>{state.latest_version}</Brackets>
+                </Ink.Text>,
+              );
 
-              handle_output(<Ink.Text key="done">Auto update done.</Ink.Text>);
+              patch({ status: "done" });
             }}
             onNo={() => {
               patch({ status: "done" });
@@ -239,9 +260,6 @@ export function AutoUpdate(props: Props) {
       case "install":
         return null;
 
-      case "exit":
-        return null;
-
       case "done":
         return props.children;
     }

package/src/app/GatherMetadata.tsx

@@ -67,33 +67,10 @@ async function run() {
 
     actions.debug(`master_branch = ${master_branch}`);
 
-    const branch_name = (await cli("git rev-parse --abbrev-ref HEAD")).stdout;
-
-    // handle detahed head state
-    if (branch_name === "HEAD") {
-      actions.error("Must run within a branch.");
-      actions.exit(0);
-      return;
-    }
-
-    // handle when there are no detected changes
-    if (`origin/${branch_name}` === master_branch) {
-      actions.error("Must run within a branch.");
-      actions.exit(0);
-      return;
-    }
-
     const head = (await cli("git rev-parse HEAD")).stdout;
+    const branch_name = (await cli("git rev-parse --abbrev-ref HEAD")).stdout;
     const merge_base = (await cli(`git merge-base HEAD ${master_branch}`)).stdout;
 
-    // handle when there are no detected changes
-    if (head === merge_base) {
-      actions.newline();
-      actions.output(<Ink.Text color={colors.gray}>No changes detected.</Ink.Text>);
-      actions.exit(0);
-      return;
-    }
-
     // git@github.com:magus/git-multi-diff-playground.git
     // https://github.com/magus/git-multi-diff-playground.git
     const origin_url = (await cli(`git config --get remote.origin.url`)).stdout;
@@ -107,6 +84,7 @@ async function run() {
       state.master_branch = master_branch;
       state.head = head;
       state.branch_name = branch_name;
+      state.merge_base = merge_base;
     });
   } catch (err) {
     actions.error("Unable to gather git metadata.");

package/src/app/Main.tsx

@@ -6,7 +6,6 @@ import { ManualRebase } from "~/app/ManualRebase";
 import { PostRebaseStatus } from "~/app/PostRebaseStatus";
 import { PreLocalMergeRebase } from "~/app/PreLocalMergeRebase";
 import { PreManualRebase } from "~/app/PreManualRebase";
-import { PreSelectCommitRanges } from "~/app/PreSelectCommitRanges";
 import { SelectCommitRanges } from "~/app/SelectCommitRanges";
 import { Status } from "~/app/Status";
 import { Store } from "~/app/Store";
@@ -32,9 +31,6 @@ export function Main() {
     case "pre-local-merge-rebase":
       return <PreLocalMergeRebase />;
 
-    case "pre-select-commit-ranges":
-      return <PreSelectCommitRanges />;
-
     case "select-commit-ranges":
       return <SelectCommitRanges />;
 

package/src/app/ManualRebase.tsx

@@ -59,6 +59,11 @@ async function run() {
       commit.title = group_from_map.title;
     }
 
+    // // capture commit_range for GitReviseTodo test
+    // // doc-link capture-git-revise-todo
+    // console.debug(JSON.stringify(commit_range, null, 2));
+    // throw new Error("STOP");
+
     await GitReviseTodo.execute({
       rebase_group_index: 0,
       rebase_merge_base: merge_base,

package/src/app/MultiSelect.tsx

@@ -14,6 +14,7 @@ type Props<T> = {
   disabled?: boolean;
   disableSelect?: boolean;
   maxWidth?: number;
+  startIndex?: number;
 };
 
 type Item<T> = {
@@ -67,11 +68,13 @@ export function MultiSelect<T>(props: Props<T>) {
       for (let i = props.items.length - 1; i >= 0; i--) {
         const item = props.items[i];
 
-        if (!item.disabled) {
-          last_enabled = i;
+        if (item.disabled) {
+          continue;
         }
 
-        if (!item.selected && !item.disabled) {
+        last_enabled = i;
+
+        if (!item.selected) {
           return i;
         }
       }
@@ -80,7 +83,7 @@ export function MultiSelect<T>(props: Props<T>) {
         return last_enabled;
       }
 
-      return 0;
+      return props.startIndex || 0;
     },
   );
 
@@ -188,7 +191,6 @@ type ItemRowProps = {
 
 function ItemRow(props: ItemRowProps) {
   let color;
-  let bold;
   let underline;
   let dimColor;
 
@@ -197,14 +199,8 @@ function ItemRow(props: ItemRowProps) {
     underline = true;
   }
 
-  if (props.selected) {
-    // color = "";
-    bold = true;
-  }
-
   if (props.disabled) {
     color = "";
-    bold = false;
     underline = false;
     dimColor = true;
   }
@@ -215,7 +211,7 @@ function ItemRow(props: ItemRowProps) {
 
       <Ink.Box width={props.maxWidth}>
         <Ink.Text
-          bold={bold}
+          // force line break
           underline={underline}
           color={color}
           dimColor={dimColor}

package/src/app/RequireBranch.tsx

@@ -0,0 +1,67 @@
+import * as React from "react";
+
+import * as Ink from "ink-cjs";
+
+import { Await } from "~/app/Await";
+import { Store } from "~/app/Store";
+import { colors } from "~/core/colors";
+import { invariant } from "~/core/invariant";
+
+type Props = {
+  children: React.ReactNode;
+};
+
+export function RequireBranch(props: Props) {
+  const fallback = <Ink.Text color={colors.yellow}>Gathering local git information…</Ink.Text>;
+
+  return (
+    <Await fallback={fallback} function={run}>
+      {props.children}
+    </Await>
+  );
+}
+
+async function run() {
+  const state = Store.getState();
+  const actions = Store.getState().actions;
+  const master_branch = state.master_branch;
+  const head = state.head;
+  const branch_name = state.branch_name;
+  const merge_base = state.merge_base;
+
+  invariant(head, "head must exist");
+  invariant(branch_name, "branch_name must exist");
+  invariant(merge_base, "merge_base must exist");
+
+  try {
+    // handle detahed head state
+    if (branch_name === "HEAD") {
+      actions.error("Must run within a branch.");
+      actions.exit(0);
+      return;
+    }
+
+    // handle when there are no detected changes
+    if (`origin/${branch_name}` === master_branch) {
+      actions.error("Must run within a branch.");
+      actions.exit(0);
+      return;
+    }
+
+    // handle when there are no detected changes
+    if (head === merge_base) {
+      actions.newline();
+      actions.output(<Ink.Text color={colors.gray}>No changes detected.</Ink.Text>);
+      actions.exit(0);
+      return;
+    }
+  } catch (err) {
+    actions.error("Unable to detect branch changes.");
+
+    if (err instanceof Error) {
+      actions.error(err.message);
+    }
+
+    actions.exit(17);
+  }
+}