pi-powerline-footer 0.2.21 → 0.2.23

package/CHANGELOG.md

@@ -2,6 +2,23 @@
 
 ## [Unreleased]
 
+## [0.2.23] - 2026-02-06
+
+### Fixed
+- **Slash command autocomplete not appearing** — Custom editor created during `session_start` never received the autocomplete provider because pi v0.52.7 moved `setupAutocomplete()` to run after extensions load. The `handleInput` override now detects the missing provider on first keystroke, re-triggers `setEditorComponent` (which succeeds because the provider exists by then), and forwards the keystroke to the new editor. Users without editor-replacing extensions were unaffected.
+
+## [0.2.22] - 2026-01-31
+
+### Fixed
+- **Detached HEAD flickering** — Git branch segment no longer oscillates between showing "detached" and hiding every 500ms when HEAD is detached
+  - Root cause: two competing branch detection methods (provider reads `.git/HEAD` → `"detached"`, extension runs `git branch --show-current` → empty/null) fought via a `??` fallback that leaked the provider value on every cache expiry
+  - Branch cache now returns stale value while refreshing instead of falling through to provider
+  - Detached HEAD now shows the short commit SHA (e.g., `abc1234 (detached)`) instead of bare "detached"
+
+### Changed
+- **Extracted `runGit` helper** — Consolidated duplicated process-spawning logic from `fetchGitBranch` and `fetchGitStatus` into a shared helper
+- `fetchGitBranch` now distinguishes "not a git repo" (null, early exit) from "detached HEAD" (empty string, SHA lookup) — avoids spawning a wasteful second process for non-git directories
+
 ## [0.2.21] - 2026-01-31
 
 ### Changed

package/git-status.ts

@@ -60,12 +60,9 @@ function parseGitStatusOutput(output: string): { staged: number; unstaged: numbe
   return { staged, unstaged, untracked };
 }
 
-/**
- * Fetch current git branch asynchronously
- */
-async function fetchGitBranch(): Promise<string | null> {
+function runGit(args: string[], timeoutMs = 200): Promise<string | null> {
   return new Promise((resolve) => {
-    const proc = spawn("git", ["branch", "--show-current"], {
+    const proc = spawn("git", args, {
       stdio: ["ignore", "pipe", "pipe"],
     });
 
@@ -84,67 +81,40 @@ async function fetchGitBranch(): Promise<string | null> {
     });
 
     proc.on("close", (code) => {
-      if (code !== 0) {
-        finish(null);
-        return;
-      }
-      const branch = stdout.trim();
-      finish(branch || null); // Empty string means detached HEAD
+      finish(code === 0 ? stdout.trim() : null);
     });
 
     proc.on("error", () => {
       finish(null);
     });
 
-    // Timeout after 200ms
     const timeoutId = setTimeout(() => {
       proc.kill();
       finish(null);
-    }, 200);
+    }, timeoutMs);
   });
 }
 
 /**
- * Fetch git status asynchronously
+ * Fetch current git branch asynchronously.
+ * For detached HEAD, returns the short commit SHA (matches provider's "detached" behavior).
  */
-async function fetchGitStatus(): Promise<{ staged: number; unstaged: number; untracked: number } | null> {
-  return new Promise((resolve) => {
-    const proc = spawn("git", ["status", "--porcelain"], {
-      stdio: ["ignore", "pipe", "pipe"],
-    });
-
-    let stdout = "";
-    let resolved = false;
-
-    const finish = (result: { staged: number; unstaged: number; untracked: number } | null) => {
-      if (resolved) return;
-      resolved = true;
-      clearTimeout(timeoutId);
-      resolve(result);
-    };
-
-    proc.stdout.on("data", (data) => {
-      stdout += data.toString();
-    });
-
-    proc.on("close", (code) => {
-      if (code !== 0) {
-        finish(null);
-        return;
-      }
-      finish(parseGitStatusOutput(stdout));
-    });
+async function fetchGitBranch(): Promise<string | null> {
+  const branch = await runGit(["branch", "--show-current"]);
+  if (branch === null) return null;
+  if (branch) return branch;
 
-    proc.on("error", () => {
-      finish(null);
-    });
+  const sha = await runGit(["rev-parse", "--short", "HEAD"]);
+  return sha ? `${sha} (detached)` : "detached";
+}
 
-    // Timeout after 500ms
-    const timeoutId = setTimeout(() => {
-      proc.kill();
-      finish(null);
-    }, 500);
-  });
+/**
+ * Fetch git status asynchronously
+ */
+async function fetchGitStatus(): Promise<{ staged: number; unstaged: number; untracked: number } | null> {
+  const output = await runGit(["status", "--porcelain"], 500);
+  if (output === null) return null;
+  return parseGitStatusOutput(output);
 }
 
 /**
@@ -174,8 +144,8 @@ export function getCurrentBranch(providerBranch: string | null): string | null {
     });
   }
 
-  // Return cached branch, or fall back to provider
-  return cachedBranch?.branch ?? providerBranch;
+  // Return stale cache while refreshing; only use provider before first fetch
+  return cachedBranch ? cachedBranch.branch : providerBranch;
 }
 
 /**

package/index.ts

@@ -550,14 +550,23 @@ export default function powerlineFooter(pi: ExtensionAPI) {
   function setupCustomEditor(ctx: any) {
     // Import CustomEditor dynamically and create wrapper
     import("@mariozechner/pi-coding-agent").then(({ CustomEditor }) => {
-      ctx.ui.setEditorComponent((tui: any, editorTheme: any, keybindings: any) => {
+      let currentEditor: any = null;
+      let autocompleteFixed = false;
+
+      const editorFactory = (tui: any, editorTheme: any, keybindings: any) => {
         // Create custom editor that overrides render for status bar below content
         const editor = new CustomEditor(tui, editorTheme, keybindings);
+        currentEditor = editor;
         
-        // Override handleInput to dismiss welcome on first keypress
         const originalHandleInput = editor.handleInput.bind(editor);
         editor.handleInput = (data: string) => {
-          // Dismiss welcome overlay/header on first keypress (use setTimeout to avoid re-entrancy)
+          if (!autocompleteFixed && !(editor as any).autocompleteProvider) {
+            autocompleteFixed = true;
+            ctx.ui.setEditorComponent(editorFactory);
+            currentEditor?.handleInput(data);
+            return;
+          }
+          // Dismiss welcome overlay/header (use setTimeout to avoid re-entrancy)
           setTimeout(() => dismissWelcome(ctx), 0);
           originalHandleInput(data);
         };
@@ -632,7 +641,9 @@ export default function powerlineFooter(pi: ExtensionAPI) {
         };
         
         return editor;
-      });
+      };
+
+      ctx.ui.setEditorComponent(editorFactory);
 
       // Set up footer data provider access (needed for git branch, extension statuses)
       // Status bar is rendered inside the editor override, footer is empty

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "pi-powerline-footer",
-  "version": "0.2.21",
+  "version": "0.2.23",
   "description": "Powerline-style status bar extension for pi coding agent",
   "type": "module",
   "bin": {