@xia-sc/dsh-git 0.5.2 → 0.6.0

package/lib/index.js

@@ -84,8 +84,15 @@ const LLM_TIMEOUT_MS = 120000;
 const LLM_DIFF_MAX_CHARS = 12000;
 /** Diff-stat characters sent to the model. */
 const LLM_STAT_MAX_CHARS = 2000;
-/** Output cap for one generated commit message. */
-const LLM_MAX_TOKENS = 256;
+/**
+ * Output cap for one generated commit message. A reasoning model spends this
+ * budget on its thinking BEFORE any text exists and reports the two together as
+ * `completion_tokens`, so a cap sized for the message alone truncates every
+ * call that thinks a little. 8192 leaves room for a long think plus a
+ * subject+body; the model still stops on its own well before it (an unset cap
+ * would fall back to the adapter's own default, which is larger still).
+ */
+const LLM_MAX_TOKENS = 8192;
 /** Accepted `generateMessage` modes. */
 const GENERATE_MODES = ["staged", "unstaged", "all"];
 /** Default mode: the only one whose content is what `commit` will actually record. */
@@ -535,7 +542,7 @@ async function gitFetch(rawCwd, rawRemote, signal) {
 		return fail("fetch-failed", cleanMessage(error), {});
 	}
 	const detail = [out.stdout, out.stderr].join("\n").trim();
-	return ok({ message: detail !== "" ? detail.slice(0, 800) : "fetch complete" });
+	return ok({ message: detail.slice(0, 800) });
 }
 
 /**
@@ -555,7 +562,7 @@ async function gitPull(rawCwd, signal) {
 		return fail("pull-failed", cleanMessage(error), {});
 	}
 	const detail = [out.stdout, out.stderr].join("\n").trim();
-	return ok({ message: detail !== "" ? detail.slice(0, 800) : "pull complete" });
+	return ok({ message: detail.slice(0, 800) });
 }
 
 /** Read the repo's configured author (user.name / user.email) or null. */
@@ -624,7 +631,7 @@ async function gitPush(rawCwd, signal) {
 		return fail("push-failed", cleanMessage(error), {});
 	}
 	const detail = [out.stdout, out.stderr].join("\n").trim();
-	return ok({ message: detail !== "" ? detail.slice(0, 800) : "push complete" });
+	return ok({ message: detail.slice(0, 800) });
 }
 
 /**
@@ -759,7 +766,9 @@ async function gitStage(rawCwd, signal) {
 		return fail("stage-failed", cleanMessage(error), {});
 	}
 	const detail = [out.stdout, out.stderr].join("\n").trim();
-	return ok({ message: detail !== "" ? detail.slice(0, 800) : "all changes staged" });
+	// `message` is git's own output and nothing else: an operation git says
+	// nothing about reports an empty string, and the panel words its own summary.
+	return ok({ message: detail.slice(0, 800) });
 }
 
 /**
@@ -1049,7 +1058,8 @@ function generationMessage(text) {
  * @param route - `{ provider, model }` from {@link resolveLlmRoute}.
  * @param signal - cancellation (browser abort plus the server ceiling).
  * @returns the trimmed message.
- * @throws on a terminal stream failure or empty output.
+ * @throws on a terminal stream failure, an output cap hit before any text
+ *   existed (`llm-truncated`), or genuinely empty output (`llm-empty`).
  */
 async function requestCommitMessage(ctx, prompt, route, signal) {
 	const options = {
@@ -1076,6 +1086,16 @@ async function requestCommitMessage(ctx, prompt, route, signal) {
 	const ordered = (map) => [...map.entries()].sort((left, right) => left[0] - right[0]).map((entry) => entry[1]);
 	const message = (blocks.size > 0 ? ordered(blocks) : ordered(deltas)).join("").trim();
 	if (message === "") {
+		// "Nothing came back" and "the budget ran out before any text did" are
+		// different failures with different fixes; a reasoning model hits the
+		// second one routinely (its thinking shares `completion_tokens` with the
+		// message), so report the cap instead of a misleading generic empty.
+		if (finish !== undefined && finish.kind === "max-tokens") {
+			throw Object.assign(
+				new Error(`the model hit the ${LLM_MAX_TOKENS}-token output cap before writing any commit message (a reasoning model spends that budget on its thinking first)`),
+				{ pluginCode: "llm-truncated" }
+			);
+		}
 		throw Object.assign(new Error("the model returned no commit message"), { pluginCode: "llm-empty" });
 	}
 	return message;

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@xia-sc/dsh-git",
-  "version": "0.5.2",
+  "version": "0.6.0",
   "description": "Complete Git management for the DeepSeek Harness Web GUI: branch switch, new-branch-from-base, fetch, pull, stage-all, commit (with AI-drafted message), push, status, and a click-to-open diff viewer (staged/unstaged, resizable) — as a collapsible floating panel that follows the current session's workspace.",
   "type": "module",
   "main": "lib/index.js",
@@ -12,7 +12,8 @@
   "files": [
     "lib",
     "cordis.patch.yml",
-    "README.en.md"
+    "README.en.md",
+    "LICENSE"
   ],
   "license": "MIT",
   "author": "xia-sc",