perf(mcp)\!: drop shortStatus from git_inventory output

BREAKING CHANGE: InventoryEntryJson no longer carries `shortStatus`;
the markdown output drops the trailing `short:` block. `branchStatus`
already contains the full `git status --short -b` body (branch header
line followed by porcelain entries), so emitting `shortStatus`
separately was shipping the porcelain lines twice.

gitStatusSnapshotAsync loses its `shortLine` / `shortOk` fields to
match — they have no remaining callers.

Author: AlbinoGeek

src/server/git.ts

@@ -147,20 +147,13 @@ function gitStatusFailText(r: { stderr: string; stdout: string }): string {
 
 export async function gitStatusSnapshotAsync(cwd: string): Promise<{
   branchLine: string;
-  shortLine: string;
   branchOk: boolean;
-  shortOk: boolean;
 }> {
   const r = await spawnGitAsync(cwd, ["status", "--short", "-b"]);
   if (!r.ok) {
-    const text = gitStatusFailText(r);
-    return { branchOk: false, shortOk: false, branchLine: text, shortLine: text };
+    return { branchOk: false, branchLine: gitStatusFailText(r) };
   }
-  const full = r.stdout.trimEnd();
-  const nl = full.indexOf("\n");
-  const branchLine = full;
-  const shortLine = nl >= 0 ? full.slice(nl + 1) : "";
-  return { branchOk: true, shortOk: true, branchLine, shortLine };
+  return { branchOk: true, branchLine: r.stdout.trimEnd() };
 }
 
 export async function gitStatusShortBranchAsync(

src/server/inventory.ts

@@ -7,7 +7,6 @@ export type InventoryEntryJson = {
   label: string;
   path: string;
   branchStatus: string;
-  shortStatus: string;
   detached: boolean;
   headAbbrev: string;
   upstreamMode: "auto" | "fixed";
@@ -33,7 +32,6 @@ export function makeSkipEntry(
     label,
     path: abs,
     branchStatus: "",
-    shortStatus: "",
     detached: false,
     headAbbrev: "",
     upstreamMode,
@@ -50,10 +48,7 @@ export function buildInventorySectionMarkdown(e: InventoryEntryJson): string[] {
     return [`## ${e.label}`, `path: ${e.path}`, "```text", e.skipReason, "```", ``];
   }
   const lines: string[] = [];
-  lines.push(e.branchStatus);
-  lines.push("");
-  lines.push("short:");
-  lines.push(e.shortStatus || "(clean)");
+  lines.push(e.branchStatus || "(clean)");
   lines.push("");
   if (e.detached) {
     lines.push("branch: (detached HEAD)");
@@ -76,7 +71,6 @@ function buildEntry(params: {
   label: string;
   absPath: string;
   branchStatus: string;
-  shortStatus: string;
   detached: boolean;
   headAbbrev: string;
   upstreamMode: "auto" | "fixed";
@@ -89,7 +83,6 @@ function buildEntry(params: {
     label: params.label,
     path: params.absPath,
     branchStatus: params.branchStatus,
-    shortStatus: params.shortStatus,
     detached: params.detached,
     headAbbrev: params.headAbbrev || "(unknown)",
     upstreamMode: params.upstreamMode,
@@ -112,10 +105,9 @@ export async function collectInventoryEntry(
   ]);
 
   const branchStatus = snap.branchLine;
-  const shortStatus = snap.shortLine;
   const headAbbrev = headR.ok ? headR.stdout.trim() : "";
   const detached = !headR.ok || headAbbrev === "HEAD" || headAbbrev.endsWith("/HEAD");
-  const base = { label, absPath, branchStatus, shortStatus, detached, headAbbrev };
+  const base = { label, absPath, branchStatus, detached, headAbbrev };
 
   if (fixedRemote !== undefined && fixedBranch !== undefined) {
     const ref = `${fixedRemote}/${fixedBranch}`;