@selesai/code 0.7.0 → 0.8.1

package/dist/extensions/pi-powerline-footer/tests/fixed-editor-border.test.ts

@@ -0,0 +1,39 @@
+import { describe, expect, it } from "vitest";
+import { initTheme, theme } from "../../../modes/interactive/theme/theme.ts";
+
+/**
+ * The powerline fixed-editor render draws its own border. It must respect the
+ * editor's `borderColor` (set by interactive-mode to the bash-mode / thinking
+ * color) instead of always using the gray "sep" color, so typing `!` visibly
+ * switches the input field to bash mode.
+ */
+function powerlineBorderColor(editor: { borderColor: unknown }, s: string): string {
+  return typeof editor.borderColor === "function"
+    ? (editor.borderColor as (str: string) => string)(s)
+    : `\x1b[38;5;244m${s}\x1b[0m`;
+}
+
+describe("powerline fixed-editor border respects editor.borderColor", () => {
+  it("uses the bash-mode color when interactive-mode sets it (typing !)", () => {
+    initTheme("dark");
+    const editorWithBashBorder = {
+      borderColor: theme.getBashModeBorderColor(),
+    };
+    const border = powerlineBorderColor(editorWithBashBorder, "─".repeat(5));
+    // Bash mode is green in the theme; the border must carry a color code.
+    expect(border).toContain("\x1b[");
+    expect(border).toMatch(/[0-9]+;[0-9]+;[0-9]+m/);
+    // And it must NOT be the plain gray "sep" code.
+    expect(border).not.toContain("\x1b[38;5;244m");
+  });
+
+  it("falls back to the gray sep color when borderColor is not a function", () => {
+    const border = powerlineBorderColor({ borderColor: "x" }, "─".repeat(5));
+    expect(border).toBe("\x1b[38;5;244m─────\x1b[0m");
+  });
+});
+
+function getBashModeBorderColor(): (str: string) => string {
+  // Mirrors theme.getBashModeBorderColor(): colors with the theme's "bashMode" color.
+  return (str: string) => `\x1b[38;5;34m${str}\x1b[39m`;
+}

package/dist/extensions/pi-powerline-footer/tests/git-status.test.ts

@@ -0,0 +1,92 @@
+import test from "node:test";
+import assert from "node:assert/strict";
+import { detectGitHost, getCurrentBranch, getGitStatus, invalidateGitBranch, invalidateGitStatus, subscribeGitUpdates } from "../git-status.ts";
+
+const sleep = (ms: number) => new Promise((resolve) => setTimeout(resolve, ms));
+
+// Background fetches spawn git with up to 500ms timeouts; give them room.
+const FETCH_SETTLE_MS = 1200;
+
+test("git status supports disabling extension git polling", () => {
+  assert.deepEqual(getGitStatus("main", "off"), {
+    branch: "main",
+    staged: 0,
+    unstaged: 0,
+    untracked: 0,
+  });
+});
+
+test("invalidateGitStatus serves stale counts while refreshing (no flicker to zeros)", async () => {
+  getGitStatus("provider-branch"); // kick off the initial background fetch
+  await sleep(FETCH_SETTLE_MS);
+
+  const seeded = getGitStatus("provider-branch");
+
+  invalidateGitStatus();
+  const afterInvalidate = getGitStatus("provider-branch");
+  assert.deepEqual(
+    { staged: afterInvalidate.staged, unstaged: afterInvalidate.unstaged, untracked: afterInvalidate.untracked },
+    { staged: seeded.staged, unstaged: seeded.unstaged, untracked: seeded.untracked },
+  );
+
+  // The background refresh converges to real data again (repo is unchanged).
+  await sleep(FETCH_SETTLE_MS);
+  const refreshed = getGitStatus("provider-branch");
+  assert.deepEqual(
+    { staged: refreshed.staged, unstaged: refreshed.unstaged, untracked: refreshed.untracked },
+    { staged: seeded.staged, unstaged: seeded.unstaged, untracked: seeded.untracked },
+  );
+});
+
+test("invalidateGitBranch keeps serving the last known branch while refreshing", async () => {
+  getGitStatus("provider-fallback"); // seed branch cache
+  await sleep(FETCH_SETTLE_MS);
+
+  const realBranch = getCurrentBranch("provider-fallback");
+  // Once fetched, the cached branch no longer falls back to the provider
+  // (null in non-git directories, the actual branch name inside a repo).
+  assert.notEqual(realBranch, "provider-fallback");
+
+  invalidateGitBranch();
+  assert.equal(getCurrentBranch("provider-fallback"), realBranch);
+});
+
+test("git cache completion notifies active footer subscribers", async () => {
+  let updates = 0;
+  const unsubscribe = subscribeGitUpdates(() => { updates += 1; });
+  try {
+    invalidateGitStatus();
+    invalidateGitBranch();
+    getGitStatus("provider-fallback");
+    await sleep(FETCH_SETTLE_MS);
+    assert.ok(updates > 0);
+  } finally {
+    unsubscribe();
+  }
+});
+
+test("detectGitHost recognizes known hosts over SSH and HTTPS", () => {
+  assert.equal(detectGitHost("git@github.com:owner/repo.git"), "github");
+  assert.equal(detectGitHost("https://github.com/owner/repo.git"), "github");
+  assert.equal(detectGitHost("ssh://git@gitlab.com/owner/repo.git"), "gitlab");
+  assert.equal(detectGitHost("https://gitlab.com/owner/repo"), "gitlab");
+  assert.equal(detectGitHost("git@bitbucket.org:owner/repo.git"), "bitbucket");
+  assert.equal(detectGitHost("https://user@bitbucket.org/owner/repo.git"), "bitbucket");
+});
+
+test("detectGitHost normalizes www and sub-domains", () => {
+  assert.equal(detectGitHost("https://www.github.com/owner/repo"), "github");
+  assert.equal(detectGitHost("git@ssh.github.com:owner/repo.git"), "github");
+});
+
+test("detectGitHost treats unknown or self-hosted remotes as a generic host", () => {
+  assert.equal(detectGitHost("git@git.example.com:owner/repo.git"), "other");
+  assert.equal(detectGitHost("https://gitea.mycorp.dev/owner/repo.git"), "other");
+  assert.equal(detectGitHost("/srv/git/local.git"), "other");
+});
+
+test("detectGitHost returns null when there is no remote", () => {
+  assert.equal(detectGitHost(null), null);
+  assert.equal(detectGitHost(""), null);
+  assert.equal(detectGitHost("   "), null);
+});

package/dist/extensions/pi-powerline-footer/tests/guide.test.ts

@@ -22,7 +22,7 @@ const features: GuideFeature[] = [
 ];
 
 
-test("guide opens fully for each real session start", () => {
+test("explicit full guide opens for each real session start", () => {
   for (const reason of ["startup", "new", "resume", "fork"]) {
     assert.equal(resolveGuideDisplayMode({ reason, hasUI: true, guideMode: "full" }), "full");
   }
@@ -35,8 +35,10 @@ test("guide dismissal and compact preferences change automatic display", () => {
   assert.equal(resolveGuideDisplayMode({ reason: "startup", hasUI: false, guideMode: "full" }), "none");
 });
 
-test("quiet startup does not suppress the default blocking guide", () => {
-  assert.equal(resolveGuideDisplayMode({ reason: "startup", hasUI: true, guideMode: "full" }), "full");
+test("guide defaults to a compact header so fixed-editor mode stays active", () => {
+  const preferences = getGuidePreferences({});
+  assert.equal(preferences.mode, "compact");
+  assert.equal(resolveGuideDisplayMode({ reason: "startup", hasUI: true, guideMode: preferences.mode }), "compact");
 });
 
 test("full and compact guide content expose README commands and examples", () => {
@@ -63,7 +65,10 @@ test("blocking guide dismisses only from its overlay input handler", () => {
 
   assert.equal(GUIDE_DISMISS_HINT, "Press any key to continue");
   assert.match(source, /guideOverlayBlocking = true;[\s\S]*handleInput: \(\) => dismiss\(\),/);
-  assert.doesNotMatch(source, /scheduleDismissWelcome|welcomeDismissScheduler|welcome-dismiss/);
+  // The blocking guide cannot be dismissed by the welcome scheduler: dismissWelcome
+  // bails while guideOverlayBlocking is set, so only the overlay's input handler
+  // (which clears the flag first) can close it.
+  assert.match(source, /function dismissWelcome\(ctx: any\) \{[\s\S]*?if \(guideOverlayBlocking\) return;/);
 });
 
 test("README-derived guide entries stay in the feature map", () => {
@@ -94,9 +99,9 @@ test("guide dismissal persists only the mode and version marker", () => {
   }
 });
 
-test("invalid persisted guide settings fall back to the full guide", () => {
+test("invalid persisted guide settings fall back to the compact guide", () => {
   assert.deepEqual(getGuidePreferences({ selesaiGuide: { mode: "unexpected", lastSeenVersion: 42 } }), {
-    mode: "full",
+    mode: "compact",
     lastSeenVersion: undefined,
   });
 });

package/dist/extensions/pi-powerline-footer/tests/jump-shortcuts.test.ts

@@ -1,213 +1,49 @@
 import test from "node:test";
 import assert from "node:assert/strict";
-import { readFileSync } from "node:fs";
-import { KEYBINDINGS } from "../node_modules/@selesai/code/dist/core/keybindings.js";
-import {
-  isSupportedSuperShortcut,
-  matchesConfiguredShortcut,
-  shortcutConflictKey,
-} from "../shortcuts.ts";
 
-const source = readFileSync(new URL("../index.ts", import.meta.url), "utf-8");
+import { isSupportedSuperShortcut, matchesConfiguredShortcut, shortcutConflictKey } from "../shortcuts.ts";
+import { resolveShortcutConfig } from "../index.ts";
 
-const powerlineShortcutKeys = new Set([
-  "stashHistory",
-  "copyEditor",
-  "cutEditor",
-  "jumpPreviousUserMessage",
-  "jumpNextUserMessage",
-  "jumpPreviousLlmMessage",
-  "jumpNextLlmMessage",
-  "jumpChatBottom",
-  "scrollChatUp",
-  "scrollChatDown",
-  "editorStart",
-  "editorEnd",
-]);
-
-function normalizeShortcut(shortcut: string): string {
-  const parts = shortcut.trim().toLowerCase().split("+");
-  if (parts.length <= 1) return parts[0] ?? "";
-
-  const modifierRank = new Map(["ctrl", "alt", "super", "shift"].map((modifier, index) => [modifier, index]));
-  return [
-    ...parts.slice(0, -1).sort((a, b) => (modifierRank.get(a) ?? 99) - (modifierRank.get(b) ?? 99)),
-    parts[parts.length - 1],
-  ].join("+");
-}
-
-function powerlineDefaults(): Map<string, string> {
-  const defaults = new Map<string, string>();
-  for (const match of source.matchAll(/^  ([a-zA-Z0-9]+): "([^"]+)",?$/gm)) {
-    const key = match[1];
-    const value = match[2];
-    if (key && value && powerlineShortcutKeys.has(key)) {
-      defaults.set(key, value);
-    }
-  }
-  return defaults;
-}
-
-test("chat jump shortcuts are configurable and route through fixed editor scrolling", () => {
-  const defaults = powerlineDefaults();
-  assert.equal(defaults.get("jumpPreviousUserMessage"), "ctrl+shift+u");
-  assert.equal(defaults.get("jumpNextUserMessage"), "ctrl+shift+i");
-  assert.equal(defaults.get("jumpPreviousLlmMessage"), "ctrl+alt+,");
-  assert.equal(defaults.get("jumpNextLlmMessage"), "ctrl+alt+.");
-  assert.equal(defaults.get("jumpChatBottom"), "ctrl+shift+g");
-  assert.equal(defaults.get("scrollChatUp"), "super+up");
-  assert.equal(defaults.get("scrollChatDown"), "super+down");
-  assert.equal(defaults.get("editorStart"), "super+shift+up");
-  assert.equal(defaults.get("editorEnd"), "super+shift+down");
-  assert.match(source, /const CHAT_JUMP_SHORTCUTS:/);
-  assert.match(source, /shortcutKey: "jumpPreviousUserMessage"/);
-  assert.match(source, /shortcutKey: "jumpNextUserMessage"/);
-  assert.match(source, /shortcutKey: "jumpPreviousLlmMessage"/);
-  assert.match(source, /shortcutKey: "jumpNextLlmMessage"/);
-  assert.match(source, /shortcutKey: "jumpChatBottom"/);
-  assert.match(source, /pi\.registerShortcut\(resolvedShortcuts\[shortcutKey\]/);
-  assert.match(source, /function collectChatMessageStartLines\(role: ChatJumpRole\): number\[\]/);
-  assert.match(source, /componentName === "UserMessageComponent"/);
-  assert.match(source, /componentName === "SkillInvocationMessageComponent"/);
-  assert.match(source, /componentName === "AssistantMessageComponent"/);
-  assert.match(source, /fixedEditorCompositor\.jumpToPreviousRootTarget\(targets\)/);
-  assert.match(source, /fixedEditorCompositor\.jumpToNextRootTarget\(targets\)/);
-  assert.match(source, /fixedEditorCompositor\.jumpToRootBottom\(\)/);
-  assert.match(source, /function getChatJumpShortcutAction\(data: string\): ChatJumpShortcutAction \| null/);
-  assert.match(source, /let resolvedShortcuts = resolveShortcutConfig\(startupSettings\)/);
-  assert.match(source, /resolvedShortcuts = resolveShortcutConfig\(settings\)/);
-  assert.match(source, /keyboardScrollShortcuts: \{\n\s+up: resolvedShortcuts\.scrollChatUp,\n\s+down: resolvedShortcuts\.scrollChatDown,/);
-  assert.match(source, /editorBoundaryShortcuts: \{\n\s+start: resolvedShortcuts\.editorStart,\n\s+end: resolvedShortcuts\.editorEnd,/);
-  assert.match(source, /modifier === "cmd" \|\| modifier === "command" \? "super" : modifier/);
-  assert.match(source, /shortcutUsesSuper\(normalizedShortcut\) && !isSupportedSuperShortcut\(normalizedShortcut\)/);
-  assert.match(source, /jumpToChatMessage\(ctx, action\.action\.role, action\.action\.direction\)/);
+test("surviving editor shortcuts resolve alongside chat jumps", () => {
+  const resolved = resolveShortcutConfig({});
+  assert.equal(resolved.stashHistory, "ctrl+alt+h");
+  assert.equal(resolved.copyEditor, "ctrl+alt+c");
+  assert.equal(resolved.cutEditor, "ctrl+alt+x");
+  assert.equal(resolved.ideaCapture, null);
+  assert.equal(resolved.editorStart, "super+shift+up");
+  assert.equal(resolved.editorEnd, "super+shift+down");
+  assert.equal(resolved.jumpPreviousUserMessage, "ctrl+shift+u");
+  assert.equal(resolved.jumpChatBottom, "ctrl+shift+g");
+  assert.equal(resolved.scrollChatUp, "super+up");
+  assert.equal(resolved.scrollChatDown, "super+down");
 });
 
-test("super shortcut matching rejects plain keys and unsupported command aliases", () => {
-  assert.equal(matchesConfiguredShortcut("c", "super+c"), false);
-  assert.equal(matchesConfiguredShortcut("X", "super+shift+x"), false);
-  assert.equal(matchesConfiguredShortcut("\x1b[A", "super+up"), false);
+test("super shortcut matching and conflict normalization remain supported", () => {
   assert.equal(matchesConfiguredShortcut("\x1b[1;9A", "super+up"), true);
-  assert.equal(matchesConfiguredShortcut("\x1b[1;10A", "super+shift+up"), true);
-  assert.equal(matchesConfiguredShortcut("\x1b[122;9u", "super+z"), false);
-  assert.equal(isSupportedSuperShortcut("super+c"), false);
-  assert.equal(isSupportedSuperShortcut("super+shift+x"), false);
-  assert.equal(isSupportedSuperShortcut("super+z"), false);
+  assert.equal(matchesConfiguredShortcut("c", "super+c"), false);
   assert.equal(isSupportedSuperShortcut("super+up"), true);
+  assert.equal(isSupportedSuperShortcut("super+z"), false);
   assert.equal(shortcutConflictKey("super+home"), "super+up");
-  assert.equal(shortcutConflictKey("super+end"), "super+down");
-  assert.equal(shortcutConflictKey("super+shift+home"), "super+shift+up");
-  assert.equal(shortcutConflictKey("super+shift+end"), "super+shift+down");
 });
 
-test("editor submits follow the fixed chat viewport to bottom", () => {
-  assert.match(source, /function followSubmittedEditorToBottom\(\): void/);
-  assert.match(source, /onEditorSubmit: \(\) => followSubmittedEditorToBottom\(\)/);
-  assert.match(source, /Object\.defineProperty\(editor, "onSubmit"/);
-  assert.match(source, /followSubmittedEditorToBottom\(\);\n\s+handler\(text\);/);
-  assert.match(source, /keybindings\.matches\(data, "app\.message\.followUp"\)/);
-});
-
-test("thinking level changes invalidate powerline status rendering", () => {
-  assert.match(source, /let currentThinkingLevel: string \| null = null/);
-  assert.match(source, /pi\.on\("thinking_level_select", async \(event, ctx\) => \{\n\s+currentCtx = ctx;\n\s+currentThinkingLevel = getThinkingLevelFn\?\.\(\) \?\? \(typeof event\.level === "string" \? event\.level : null\);\n\s+requestImmediateStatusRender\(\{ deferDuringTyping: false \}\);\n\s+\}\);/);
-  assert.match(source, /if \(e\.type === "thinking_level_change" && typeof e\.thinkingLevel === "string"\) \{\n\s+thinkingLevelFromSession = e\.thinkingLevel;/);
-  assert.match(source, /const thinkingLevel = currentThinkingLevel \?\? thinkingLevelFromSession \?\? getThinkingLevelFn\?\.\(\) \?\? "off"/);
-});
+test("idea capture shortcut remains configurable", () => {
+  const resolved = resolveShortcutConfig({
+    powerlineShortcuts: {
+      ideaCapture: "ctrl+alt+i",
+    },
+  });
 
-test("context usage changes repaint from live streaming message usage", () => {
-  assert.match(source, /const CONTEXT_STATUS_RENDER_MS = 250/);
-  assert.match(source, /function getUsageTokenTotal\(usage: SessionAssistantUsage\): number/);
-  assert.match(source, /const totalTokens = "totalTokens" in usage && typeof usage\.totalTokens === "number"/);
-  assert.match(source, /return totalTokens \|\| usage\.input \+ usage\.output \+ usage\.cacheRead \+ usage\.cacheWrite/);
-  assert.match(source, /let liveAssistantUsage: SessionAssistantUsage \| null = null/);
-  assert.doesNotMatch(source, /const requestContextStatusRender/);
-  assert.match(source, /lastUserPrompt = "";\n\s+isStreaming = false;\n\s+liveAssistantUsage = null;\n\s+stashedEditorText = null;/);
-  assert.match(source, /pi\.on\("agent_start", async \(_event, ctx\) => \{\n\s+isStreaming = true;\n\s+liveAssistantUsage = null;\n\s+onVibeAgentStart\(\);\n\s+dismissWelcome\(ctx\);\n\s+currentCtx = ctx;\n\s+\}\);/);
-  assert.match(source, /pi\.on\("message_update", async \(event, ctx\) => \{\n\s+if \(isSessionAssistantMessage\(event\.message\)\n\s+&& event\.message\.stopReason !== "error"\n\s+&& event\.message\.stopReason !== "aborted"\n\s+&& getUsageTokenTotal\(event\.message\.usage\) > 0\) \{\n\s+liveAssistantUsage = event\.message\.usage;\n\s+currentCtx = ctx;\n\s+layoutDirty = true;\n\s+statusRenderScheduler\.schedule\(CONTEXT_STATUS_RENDER_MS\);\n\s+\}\n\s+\}\);/);
-  assert.match(source, /pi\.on\("message_end", async \(event, ctx\) => \{\n\s+currentCtx = ctx;\n\s+if \(isSessionAssistantMessage\(event\.message\)\) \{\n\s+if \(event\.message\.stopReason === "error" \|\| event\.message\.stopReason === "aborted"\) \{\n\s+liveAssistantUsage = null;\n\s+\} else if \(getUsageTokenTotal\(event\.message\.usage\) > 0\) \{\n\s+liveAssistantUsage = event\.message\.usage;\n\s+\}\n\s+\}\n\s+requestImmediateStatusRender\(\{ deferDuringTyping: false \}\);\n\s+\}\);/);
-  assert.match(source, /pi\.on\("agent_end", async \(_event, ctx\) => \{\n\s+isStreaming = false;\n\s+liveAssistantUsage = null;\n\s+currentCtx = ctx;/);
-  assert.match(source, /pi\.on\("session_tree", async \(_event, ctx\) => \{\n\s+currentCtx = ctx;\n\s+currentThinkingLevel = null;\n\s+liveAssistantUsage = null;\n\s+requestImmediateStatusRender\(\{ deferDuringTyping: false \}\);\n\s+\}\);/);
-  assert.match(source, /if \(getUsageTokenTotal\(m\.usage\) > 0\) \{\n\s+lastAssistant = m;\n\s+\}/);
-  assert.match(source, /const coreContextUsage = isStreaming && liveAssistantUsage \? null : readCoreContextUsage\(ctx\)/);
-  assert.match(source, /const contextTokens = coreContextUsage\?\.contextTokens \?\? \(latestUsage \? getUsageTokenTotal\(latestUsage\) : 0\)/);
+  assert.equal(resolved.ideaCapture, "ctrl+alt+i");
 });
 
-test("extension status changes invalidate powerline status rendering", () => {
-  assert.match(source, /let forceNextLayoutRecompute = false/);
-  assert.match(source, /let restoreFooterStatusRepaintHook: \(\(\) => void\) \| null = null/);
-  assert.match(source, /const requestImmediateStatusRender = \(options: \{ deferDuringTyping\?: boolean \} = \{\}\) => \{/);
-  assert.match(source, /if \(options\.deferDuringTyping !== false && Date\.now\(\) - lastEditorInputAt < EDITOR_STATUS_DEFER_MS\) \{\n\s+statusRenderScheduler\.schedule\(\);\n\s+return;\n\s+\}/);
-  assert.match(source, /forceNextLayoutRecompute = true;\n\s+statusRenderScheduler\.cancel\(\);\n\s+statusRenderScheduler\.schedule\(0\);/);
-  assert.match(source, /const installFooterStatusRepaintHook = \(footerData: ReadonlyFooterDataProvider\) => \{/);
-  assert.match(source, /setExtensionStatus\?: \(key: string, text: string \| undefined\) => void/);
-  assert.match(source, /const setExtensionStatusAndRepaint = function setExtensionStatusAndRepaint/);
-  assert.match(source, /originalSetExtensionStatus\.call\(this, key, text\);\n\s+requestImmediateStatusRender\(\);/);
-  assert.match(source, /installFooterStatusRepaintHook\(footerData\);/);
-  assert.match(source, /if \(writableFooterData\.setExtensionStatus === setExtensionStatusAndRepaint\) \{\n\s+writableFooterData\.setExtensionStatus = originalSetExtensionStatus;/);
-  assert.match(source, /if \(clearExtensionStatusesAndRepaint && writableFooterData\.clearExtensionStatuses === clearExtensionStatusesAndRepaint\)/);
-  assert.match(source, /restoreFooterStatusRepaintHook\?\.\(\);\n\s+restoreFooterStatusRepaintHook = null;/);
-});
-
-test("fixed editor captures Pi status messages with the editor cluster", () => {
-  assert.match(source, /let fixedStatusContainer: any = null/);
-  assert.match(source, /const statusContainerCandidate = tuiChildren\[editorContainerMatch\.index - 2\] \?\? null/);
-  assert.match(source, /fixedStatusContainer = statusContainerCandidate && typeof statusContainerCandidate\.render === "function"/);
-  assert.match(source, /compositor\.renderHidden\(fixedStatusContainer, width\)\.filter\(\(line\) => visibleWidth\(line\) > 0\)/);
-  assert.match(source, /statusLines: \[\.\.\.aboveWidgetLines, \.\.\.renderPowerlineStatusLines\(width\), \.\.\.statusContainerLines\]/);
-  assert.match(source, /if \(fixedStatusContainer\?\.render\) compositor\.hideRenderable\(fixedStatusContainer\)/);
-  assert.match(source, /fixedStatusContainer = null/);
-});
-
-test("shutdown cleanup resets terminal modes even before compositor install", () => {
-  assert.match(source, /import \{ emergencyTerminalModeReset, TerminalSplitCompositor \}/);
-  assert.match(source, /const hadCompositor = fixedEditorCompositor !== null/);
-  assert.match(source, /if \(!hadCompositor && options\?\.resetExtendedKeyboardModes\)/);
-  assert.match(source, /process\.stdout\.write\(emergencyTerminalModeReset\(\)\)/);
-});
-
-test("powerline shortcut defaults do not claim reserved Pi shortcuts", () => {
-  const reservedKeys = new Map<string, string>();
-  for (const [id, definition] of Object.entries(KEYBINDINGS)) {
-    const keys = definition.defaultKeys === undefined
-      ? []
-      : Array.isArray(definition.defaultKeys)
-        ? definition.defaultKeys
-        : [definition.defaultKeys];
-    for (const key of keys) {
-      reservedKeys.set(normalizeShortcut(key), id);
-    }
-  }
-
-  for (const [name, shortcut] of powerlineDefaults()) {
-    const conflict = reservedKeys.get(normalizeShortcut(shortcut));
-    assert.equal(conflict, undefined, `${name} default ${shortcut} conflicts with ${conflict}`);
-  }
-});
-
-test("powerline fallback routing rejects reserved Pi shortcut defaults", () => {
-  assert.doesNotMatch(source, /KeybindingsManager/);
-  assert.match(source, /TUI_KEYBINDINGS/);
-  assert.match(source, /const APP_RESERVED_SHORTCUTS = \[/);
-  assert.match(source, /"alt\+enter"/);
-  assert.match(source, /"alt\+up"/);
-  assert.match(source, /"alt\+down"/);
-  assert.match(source, /"ctrl\+s"/);
-  assert.match(source, /"shift\+l"/);
-  assert.match(source, /for \(const definition of Object\.values\(TUI_KEYBINDINGS\)\)/);
-  assert.doesNotMatch(source, /RESERVED_TUI_KEYBINDING_IDS/);
-  assert.match(source, /const EXTRA_RESERVED_SHORTCUTS = \["alt\+s"\] as const/);
-  assert.match(source, /const SHORTCUT_MODIFIER_ORDER = \["ctrl", "alt", "super", "shift"\] as const/);
-  assert.match(source, /const SHORTCUT_MODIFIERS = new Set\(SHORTCUT_MODIFIER_ORDER\)/);
-  assert.match(source, /modifierRank\.get\(a\)/);
-  assert.match(source, /configuredToggleShortcut && !reservedShortcuts\(\)\.has\(shortcutUsageKey\(configuredToggleShortcut\)\)/);
-});
+test("editor boundary shortcuts remain configurable", () => {
+  const resolved = resolveShortcutConfig({
+    powerlineShortcuts: {
+      editorStart: "ctrl+shift+y",
+      editorEnd: "ctrl+shift+x",
+    },
+  });
 
-test("powerline shortcuts have terminal-input fallback routing", () => {
-  assert.match(source, /function getPowerlineShortcutAction\(data: string\): PowerlineShortcutAction \| null/);
-  assert.match(source, /matchesConfiguredShortcut\(data, resolvedShortcuts\.stashHistory\)/);
-  assert.match(source, /matchesConfiguredShortcut\(data, resolvedShortcuts\.copyEditor\)/);
-  assert.match(source, /matchesConfiguredShortcut\(data, resolvedShortcuts\.cutEditor\)/);
-  assert.match(source, /matchesConfiguredShortcut\(data, bashModeSettings\.toggleShortcut\)/);
-  assert.match(source, /runPowerlineShortcut\(ctx, powerlineShortcutAction\)/);
+  assert.equal(resolved.editorStart, "ctrl+shift+y");
+  assert.equal(resolved.editorEnd, "ctrl+shift+x");
 });

package/dist/extensions/pi-powerline-footer/tests/mode-aware-trigger.test.ts

@@ -0,0 +1,79 @@
+import { describe, expect, it } from "vitest";
+import { Editor } from "@earendil-works/pi-tui";
+import { createInlineSkillAutocompleteProvider } from "../../../extensions/inline-skills.ts";
+import { createInlineSubagentAutocompleteProvider } from "../../../extensions/pi-subagents/src/slash/inline-subagents.ts";
+import { ModeAwareAutocompleteProvider, BashAutocompleteProvider, OneOffBashAutocompleteProvider } from "../bash-mode/completion.ts";
+
+const fakeTui = { requestRender() {}, terminal: { rows: 50 } } as never;
+const pi = {
+	getCommands: () => [
+		{ name: "skill:grill-me", description: "Grill", source: "skill", sourceInfo: { path: "C:/x/grill.md" } },
+		{ name: "skill:research", description: "Research", source: "skill", sourceInfo: { path: "C:/x/research.md" } },
+	],
+	on: () => {},
+} as never;
+
+const baseProvider: any = {
+	triggerCharacters: [],
+	async getSuggestions() {
+		return null;
+	},
+	applyCompletion(lines: string[], cursorLine: number, cursorCol: number) {
+		return { lines, cursorLine, cursorCol };
+	},
+	shouldTriggerFileCompletion() {
+		return true;
+	},
+};
+
+const popupValues = (e: Editor) =>
+	(e as any).autocompleteState ? ((e as any).autocompleteList?.items ?? []).map((i: any) => i.value) : null;
+
+async function typeInto(e: Editor, text: string) {
+	for (const ch of text) {
+		(e as any).insertCharacter(ch);
+		await (e as any).autocompleteRequestTask;
+		await new Promise((r) => setTimeout(r, 80));
+	}
+}
+
+function buildChain() {
+	let provider: any = baseProvider;
+	for (const wrap of [
+		(c: any) => createInlineSkillAutocompleteProvider(pi, c),
+		(c: any) => createInlineSubagentAutocompleteProvider({ baseCwd: process.cwd() }, c),
+	]) {
+		provider = wrap(provider);
+	}
+	provider.triggerCharacters = [...new Set(["$", "#"])];
+	return provider;
+}
+
+describe("powerline ModeAware wrapper must forward trigger characters", () => {
+	it("keeps $ after wrapping and shows skills for the user's three inputs", async () => {
+		const chain = buildChain();
+		// powerline: editor.installAutocompleteProvider(new ModeAware(chain, bash, oneOffBash))
+		const wrapped = new ModeAwareAutocompleteProvider(chain, new BashAutocompleteProvider(), new OneOffBashAutocompleteProvider(), () => false);
+
+		const e = new Editor(fakeTui as any, { borderColor: "x" } as any);
+		e.setAutocompleteProvider(wrapped as any);
+		console.log("triggerChars after wrap:", (e as any).autocompleteTriggerCharacters);
+		expect((e as any).autocompleteTriggerCharacters).toContain("$");
+
+		await typeInto(e, "$gril");
+		console.log("'$gril' ->", JSON.stringify(popupValues(e)));
+		expect(popupValues(e) ?? []).toContain("$grill-me");
+
+		const e2 = new Editor(fakeTui as any, { borderColor: "x" } as any);
+		e2.setAutocompleteProvider(wrapped as any);
+		await typeInto(e2, "asdf $gril");
+		console.log("'asdf $gril' ->", JSON.stringify(popupValues(e2)));
+		expect(popupValues(e2) ?? []).toContain("$grill-me");
+
+		const e3 = new Editor(fakeTui as any, { borderColor: "x" } as any);
+		e3.setAutocompleteProvider(wrapped as any);
+		await typeInto(e3, "asdf #builder $gril");
+		console.log("'asdf #builder $gril' ->", JSON.stringify(popupValues(e3)));
+		expect(popupValues(e3) ?? []).toContain("$grill-me");
+	});
+});

package/dist/extensions/pi-powerline-footer/tests/paths.test.ts

@@ -0,0 +1,63 @@
+import test from "node:test";
+import assert from "node:assert/strict";
+import { mkdirSync, mkdtempSync, rmSync } from "node:fs";
+import { tmpdir } from "node:os";
+import { join } from "node:path";
+import { getAgentPath, getAgentSessionDirs, getLegacyPiPath } from "../paths.ts";
+
+function withTemporaryPathEnv(run: (home: string, agentDir: string) => void): void {
+  const root = mkdtempSync(join(tmpdir(), "powerline-paths-"));
+  const home = join(root, "home");
+  const agentDir = join(root, "custom-agent");
+  const originalHome = process.env.HOME;
+  const originalUserProfile = process.env.USERPROFILE;
+  const originalAgentDir = process.env.SELESAI_CODING_AGENT_DIR;
+
+  try {
+    mkdirSync(home, { recursive: true });
+    process.env.HOME = home;
+    delete process.env.USERPROFILE;
+    run(home, agentDir);
+  } finally {
+    if (originalHome === undefined) delete process.env.HOME;
+    else process.env.HOME = originalHome;
+    if (originalUserProfile === undefined) delete process.env.USERPROFILE;
+    else process.env.USERPROFILE = originalUserProfile;
+    if (originalAgentDir === undefined) delete process.env.SELESAI_CODING_AGENT_DIR;
+    else process.env.SELESAI_CODING_AGENT_DIR = originalAgentDir;
+    rmSync(root, { recursive: true, force: true });
+  }
+}
+
+test("agent dir uses non-empty SELESAI_CODING_AGENT_DIR", () => {
+  withTemporaryPathEnv((_home, agentDir) => {
+    process.env.SELESAI_CODING_AGENT_DIR = agentDir;
+
+    assert.equal(getAgentPath("settings.json"), join(agentDir, "settings.json"));
+  });
+});
+
+test("agent dir falls back to HOME config dir agent for empty env values", () => {
+  withTemporaryPathEnv((home) => {
+    process.env.SELESAI_CODING_AGENT_DIR = "";
+
+    assert.equal(getAgentPath("settings.json"), join(home, ".selesai", "agent", "settings.json"));
+    assert.equal(getAgentPath("sessions"), join(home, ".selesai", "agent", "sessions"));
+  });
+});
+
+test("agent sessions include legacy config-dir sessions only when it exists", () => {
+  withTemporaryPathEnv((home, agentDir) => {
+    process.env.SELESAI_CODING_AGENT_DIR = agentDir;
+    assert.deepEqual(getAgentSessionDirs(), [join(agentDir, "sessions")]);
+
+    mkdirSync(join(home, ".selesai", "sessions"), { recursive: true });
+    assert.deepEqual(getAgentSessionDirs(), [join(agentDir, "sessions"), join(home, ".selesai", "sessions")]);
+  });
+});
+
+test("legacy path resolves under the fork config dir name", () => {
+  withTemporaryPathEnv((home) => {
+    assert.equal(getLegacyPiPath("sessions"), join(home, ".selesai", "sessions"));
+  });
+});