gh-stack 0.19.3 → 0.19.5

package/README.md

@@ -650,6 +650,7 @@ per-stack files plus git-native branch config, rather than a single JSON blob:
   deleted/<stack>.json         tombstones (recoverable)
   snapshots/<ts>__<stack>.json append-only, retained per-stack
   restack-state.json           the last restack/sync/join run: resume point + action log
+  push-failure.log             full output of the last refused push (a failing pre-push hook, say)
 ```
 
 A single `active/<stack>.json` looks like:

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "gh-stack",
-  "version": "0.19.3",
+  "version": "0.19.5",
   "description": "Stacked PR manager for squash-merge workflows",
   "author": "Kiliman <kiliman@gmail.com>",
   "license": "MIT",
@@ -51,11 +51,11 @@
     ]
   },
   "devDependencies": {
-    "@types/bun": "1.3.10",
+    "@types/bun": "1.4.4",
     "husky": "^9.1.7",
     "lint-staged": "^17.6.0",
-    "oxfmt": "^0.74.0",
-    "oxlint": "^1.89.0"
+    "oxfmt": "^0.75.0",
+    "oxlint": "^1.90.0"
   },
   "peerDependencies": {
     "typescript": "^5"

package/src/lib/git.ts

@@ -1,6 +1,7 @@
 // Git command helpers using Bun's shell API
 import { $ } from "bun";
 import { createHash } from "node:crypto";
+import { mkdir } from "node:fs/promises";
 
 export const STACK_SYNC_TAG_PREFIX = "stack-sync-base";
 export const STACK_SYNC_TAG_GLOB = `${STACK_SYNC_TAG_PREFIX}-*`;
@@ -153,9 +154,18 @@ export type PushResult =
        * moved since the last fetch. `rejected`: anything else.
        */
       reason: "not-integrated" | "stale-lease" | "rejected";
-      stderr: string;
+      /**
+       * git's stdout and stderr together. A local pre-push hook's output lands
+       * on either, depending on the git version (stdout as of git 2.56).
+       */
+      output: string;
+      /** Where a `rejected` push's full output was saved, when that worked. */
+      logFile?: string;
     };
 
+/** How many lines of a failed push's output `pushFailureHint` shows. */
+const PUSH_OUTPUT_TAIL = 20;
+
 /**
  * The one way gh-stack force-pushes a branch (restack, sync, merge's advance,
  * submit, join). Sets upstream, and works whether or not the branch exists on
@@ -179,17 +189,35 @@ export async function forcePushWithLease(
   const lease = opts.expect
     ? `--force-with-lease=refs/heads/${branch}:${opts.expect}`
     : "--force-with-lease";
-  const { exitCode, stderr } = await $`git push -u ${lease} --force-if-includes origin ${branch}`
-    .nothrow()
-    .quiet();
+  const { exitCode, stdout } =
+    await $`git push -u ${lease} --force-if-includes origin ${branch} 2>&1`.nothrow().quiet();
   if (exitCode === 0) return { ok: true };
-  const text = stderr.toString();
-  const reason = text.includes("remote ref updated since checkout")
+  const output = stdout.toString();
+  const reason = output.includes("remote ref updated since checkout")
     ? "not-integrated"
-    : text.includes("stale info")
+    : output.includes("stale info")
       ? "stale-lease"
       : "rejected";
-  return { ok: false, reason, stderr: text };
+  if (reason !== "rejected") return { ok: false, reason, output };
+  return { ok: false, reason, output, logFile: await savePushFailure(output) };
+}
+
+/**
+ * Save a failed push's full output to `.git/.gh-stack/push-failure.log`. A
+ * failing pre-push hook (a test suite, say) can print a thousand lines, so
+ * `pushFailureHint` shows only the end and points here for the rest. Best
+ * effort: a failed write returns undefined.
+ */
+async function savePushFailure(text: string): Promise<string | undefined> {
+  try {
+    const dir = `${await gitDir()}/.gh-stack`;
+    await mkdir(dir, { recursive: true });
+    const file = `${dir}/push-failure.log`;
+    await Bun.write(file, text);
+    return file;
+  } catch {
+    return undefined;
+  }
 }
 
 /** Why a force-push was refused, and what to do, in gh-stack terms (plain lines). */
@@ -208,13 +236,19 @@ export function pushFailureHint(branch: string, result: PushResult): string[] {
         `origin/${branch} moved since your last fetch, so the lease refused.`,
         `Look at what's there first: git fetch origin ${branch} && git log ${branch}..origin/${branch}`,
       ];
-    default:
-      return [
-        result.stderr
-          .trim()
-          .split("\n")
-          .find((l) => l.includes("rejected") || l.startsWith("error")) ?? result.stderr.trim(),
-      ];
+    default: {
+      // Show the end of git's output, not just its "failed to push" line: a
+      // local pre-push hook prints its reason above that line, and a remote
+      // rejection names the rule that refused the push.
+      const lines = result.output.split("\n").filter((l) => l.trim() !== "");
+      const tail = lines.slice(-PUSH_OUTPUT_TAIL);
+      const hint = ["git push failed. The end of its output (a pre-push hook prints here too):"];
+      hint.push(...tail.map((l) => `  ${l.trimEnd()}`));
+      if (result.logFile && lines.length > tail.length) {
+        hint.push(`Full output (${lines.length} lines): ${result.logFile}`);
+      }
+      return hint;
+    }
   }
 }
 

package/src/lib/paths.ts

@@ -7,6 +7,7 @@
 //     deleted/<stack>.json         tombstones (recoverable)
 //     snapshots/<ts>__<stack>.json append-only pre-operation branch tips
 //     restack-state.json           last restack/sync/join run: resume point + action log
+//     push-failure.log             full output of the last rejected push (git.forcePushWithLease)
 //
 // Everything lives inside `.git/` and is never committed — gh-stack is a
 // single-developer tool, so there's no need to share metadata across clones.

package/src/lib/using-gh-stack.md

@@ -296,6 +296,11 @@ it (its PR would be diffed against the parent's old history); the run lists it
 as held and exits 1 until `--resume` gets the parent pushed. `merge` and `join`
 refuse to run while a restack/sync is pending.
 
+**Why did the push fail?** gh-stack prints the last lines of `git push`'s
+output, which includes a pre-push hook's own messages, and saves all of it to
+`.git/.gh-stack/push-failure.log`. Fix the cause (a failing check, say), then
+`--resume`.
+
 **"Origin has commits that the local branches don't"** (restack/sync exit 1
 before changing anything): a branch's copy on origin has commits it never had,
 often the squash commit a `merge --collapse` added on GitHub. Run the