@mrclrchtr/supi-rtk 0.1.0

package/node_modules/@mrclrchtr/supi-core/src/settings-ui.ts

@@ -0,0 +1,226 @@
+// Generic settings overlay for SuPi extensions.
+//
+// Uses pi-tui's SettingsList with scope toggle (Tab), extension grouping,
+// and search. Each extension declares its settings via registerSettings().
+
+import type { ExtensionContext } from "@earendil-works/pi-coding-agent";
+import { getSettingsListTheme } from "@earendil-works/pi-coding-agent";
+import {
+  Container,
+  Input,
+  Key,
+  matchesKey,
+  type SettingItem,
+  SettingsList,
+  Text,
+} from "@earendil-works/pi-tui";
+import {
+  getRegisteredSettings,
+  type SettingsScope,
+  type SettingsSection,
+} from "./settings-registry.ts";
+
+// ── Input submenu component ──────────────────────────────────
+
+/**
+ * Creates a pi-tui Input-backed submenu component with enter-to-confirm
+ * and escape-to-cancel handling.
+ *
+ * @param currentValue - Initial value for the text input.
+ * @param label - Label text displayed above the input.
+ * @param done - Callback invoked with the confirmed value, or undefined on cancel.
+ */
+export function createInputSubmenu(
+  currentValue: string,
+  label: string,
+  done: (selectedValue?: string) => void,
+): {
+  render: (width: number) => string[];
+  invalidate: () => void;
+  handleInput: (data: string) => boolean;
+} {
+  const input = new Input();
+  input.setValue(currentValue);
+
+  return {
+    render: (_width: number) => {
+      const lines = [`  ${label}`];
+      lines.push(...input.render(_width));
+      lines.push("  enter confirm • esc cancel");
+      return lines;
+    },
+    invalidate: () => {
+      input.invalidate();
+    },
+    handleInput: (data: string) => {
+      if (matchesKey(data, Key.escape)) {
+        done();
+        return true;
+      }
+      if (matchesKey(data, Key.enter)) {
+        done(input.getValue());
+        return true;
+      }
+      input.handleInput(data);
+      return true;
+    },
+  };
+}
+
+// ── Types ────────────────────────────────────────────────────
+
+interface OverlayState {
+  scope: SettingsScope;
+  cwd: string;
+}
+
+// ── Pure helpers ─────────────────────────────────────────────
+
+function getScopeLabel(scope: SettingsScope): string {
+  return scope === "project" ? "Project" : "Global";
+}
+
+function buildFlatItems(
+  sections: SettingsSection[],
+  scope: SettingsScope,
+  cwd: string,
+): SettingItem[] {
+  const items: SettingItem[] = [];
+  for (const section of sections) {
+    const sectionItems = section.loadValues(scope, cwd);
+    for (const item of sectionItems) {
+      items.push({
+        ...item,
+        id: `${section.id}.${item.id}`,
+        label: `${section.label}: ${item.label}`,
+      });
+    }
+  }
+  return items;
+}
+
+function findSectionAndId(
+  sections: SettingsSection[],
+  flatId: string,
+): { section: SettingsSection; itemId: string } | null {
+  const dotIndex = flatId.indexOf(".");
+  if (dotIndex === -1) return null;
+  const sectionId = flatId.slice(0, dotIndex);
+  const itemId = flatId.slice(dotIndex + 1);
+  const section = sections.find((s) => s.id === sectionId);
+  if (!section) return null;
+  return { section, itemId };
+}
+
+// ── Component ────────────────────────────────────────────────
+
+interface SettingsOverlayDeps {
+  state: OverlayState;
+  container: Container;
+  settingsList: SettingsList | null;
+  tui: Parameters<Parameters<ExtensionContext["ui"]["custom"]>[0]>[0];
+  theme: Parameters<Parameters<ExtensionContext["ui"]["custom"]>[0]>[1];
+  done: () => void;
+}
+
+function createSettingsList(deps: SettingsOverlayDeps): SettingsList {
+  const sections = getRegisteredSettings();
+  const items = buildFlatItems(sections, deps.state.scope, deps.state.cwd);
+  const onChange = (flatId: string, newValue: string) => {
+    const found = findSectionAndId(sections, flatId);
+    if (found) {
+      found.section.persistChange(deps.state.scope, deps.state.cwd, found.itemId, newValue);
+    }
+    // Re-read all values to reflect persisted changes, but keep the list
+    // instance (and its selectedIndex) intact.
+    const updatedItems = buildFlatItems(sections, deps.state.scope, deps.state.cwd);
+    for (const updated of updatedItems) {
+      const existing = items.find((i) => i.id === updated.id);
+      if (existing && existing.currentValue !== updated.currentValue) {
+        settingsList.updateValue(updated.id, updated.currentValue);
+      }
+    }
+    deps.tui.requestRender();
+  };
+  const settingsList = new SettingsList(
+    items,
+    Math.min(items.length + 4, 20),
+    getSettingsListTheme(),
+    onChange,
+    () => deps.done(),
+    { enableSearch: true },
+  );
+  return settingsList;
+}
+
+function rebuildSettingsList(deps: SettingsOverlayDeps): SettingsList {
+  const settingsList = createSettingsList(deps);
+  deps.settingsList = settingsList;
+
+  deps.container.clear();
+  deps.container.addChild(createHeaderComponent(deps));
+  deps.container.addChild(settingsList);
+
+  return settingsList;
+}
+
+function createHeaderComponent(deps: SettingsOverlayDeps): Text {
+  const { theme, state } = deps;
+  const scopeLabel = getScopeLabel(state.scope);
+  const otherScope = state.scope === "project" ? "Global" : "Project";
+  const headerText = new Text(
+    `${theme.fg("accent", theme.bold("SuPi Settings"))}  ${theme.fg("text", `Scope: ${scopeLabel}`)} ${theme.fg("dim", `(tab → ${otherScope})`)}`,
+    0,
+    0,
+  );
+  return headerText;
+}
+
+function handleScopeToggle(deps: SettingsOverlayDeps): void {
+  deps.state.scope = deps.state.scope === "project" ? "global" : "project";
+  rebuildSettingsList(deps);
+  deps.tui.requestRender();
+}
+
+// ── Entry point ──────────────────────────────────────────────
+
+export function openSettingsOverlay(ctx: ExtensionContext): void {
+  const sections = getRegisteredSettings();
+  if (sections.length === 0) {
+    ctx.ui.notify("No settings registered by SuPi extensions", "info");
+    return;
+  }
+
+  void ctx.ui.custom<void>((tui, theme, _kb, done) => {
+    const state: OverlayState = { scope: "project", cwd: ctx.cwd };
+    const container = new Container();
+
+    const deps: SettingsOverlayDeps = {
+      state,
+      container,
+      settingsList: null,
+      tui,
+      theme,
+      done,
+    };
+
+    rebuildSettingsList(deps);
+
+    const component = {
+      render: (width: number) => container.render(width),
+      invalidate: () => container.invalidate(),
+      handleInput: (data: string) => {
+        if (matchesKey(data, Key.tab)) {
+          handleScopeToggle(deps);
+          return true;
+        }
+        // Delegate input to the settings list (always set after rebuildSettingsList)
+        deps.settingsList?.handleInput?.(data);
+        deps.tui.requestRender();
+        return true;
+      },
+    };
+
+    return component;
+  });
+}

package/node_modules/@mrclrchtr/supi-core/src/terminal.ts

@@ -0,0 +1,60 @@
+/**
+ * Shared terminal title formatting and signaling utilities.
+ *
+ * Centralized place for pi title convention (π prefix), completion (✓)
+ * and waiting (●) indicators, and the audible terminal bell.
+ */
+import path from "node:path";
+
+/** Unicode checkmark shown when the agent finishes a turn. */
+export const DONE_SYMBOL = "\u2713";
+/** Unicode dot shown when waiting for user input. */
+export const WAITING_SYMBOL = "\u25CF";
+
+/** Minimal UI surface needed for title operations. */
+export interface TitleTarget {
+  ui: {
+    setTitle?(title: string): void;
+  };
+}
+
+/**
+ * Format pi's canonical terminal title from session name and cwd.
+ * Falls back gracefully when either is missing.
+ *
+ * @example
+ *   formatTitle("my-session", "/home/projects/foo")  // "π - my-session - foo"
+ *   formatTitle(undefined, "/home/projects/foo")      // "π - foo"
+ *   formatTitle("my-session")                         // "π - my-session"
+ *   formatTitle()                                     // "π"
+ */
+export function formatTitle(sessionName?: string, cwd?: string): string {
+  const base = cwd ? path.basename(cwd) : undefined;
+  if (sessionName && base) return `π - ${sessionName} - ${base}`;
+  if (sessionName) return `π - ${sessionName}`;
+  if (base) return `π - ${base}`;
+  return "π";
+}
+
+/** Sound the audible terminal bell (ASCII BEL). */
+export function signalBell(): void {
+  process.stdout.write("\x07");
+}
+
+/**
+ * Set the terminal title to indicate the agent is waiting for user input.
+ * Prefixes with ● and sounds the terminal bell.
+ */
+export function signalWaiting(ctx: TitleTarget, title: string): void {
+  ctx.ui.setTitle?.(`${WAITING_SYMBOL}  ${title}`);
+  signalBell();
+}
+
+/**
+ * Set the terminal title to indicate the agent turn has completed.
+ * Prefixes with ✓ and sounds the terminal bell.
+ */
+export function signalDone(ctx: TitleTarget, title: string): void {
+  ctx.ui.setTitle?.(`${DONE_SYMBOL} ${title}`);
+  signalBell();
+}

package/package.json

@@ -0,0 +1,41 @@
+{
+  "name": "@mrclrchtr/supi-rtk",
+  "version": "0.1.0",
+  "description": "SuPi RTK extension — transparent bash command rewriting via RTK for token savings",
+  "license": "MIT",
+  "repository": {
+    "type": "git",
+    "url": "git+https://github.com/mrclrchtr/supi.git"
+  },
+  "publishConfig": {
+    "access": "public"
+  },
+  "keywords": [
+    "pi-package",
+    "pi",
+    "pi-coding-agent"
+  ],
+  "files": [
+    "src/**/*.ts",
+    "README.md"
+  ],
+  "dependencies": {
+    "@mrclrchtr/supi-core": "workspace:*"
+  },
+  "bundledDependencies": [
+    "@mrclrchtr/supi-core"
+  ],
+  "peerDependencies": {
+    "@earendil-works/pi-coding-agent": "*"
+  },
+  "devDependencies": {
+    "vitest": "^4.1.4",
+    "@mrclrchtr/supi-test-utils": "workspace:*"
+  },
+  "pi": {
+    "extensions": [
+      "./src/rtk.ts"
+    ]
+  },
+  "main": "src/index.ts"
+}

package/src/guards.ts

@@ -0,0 +1,74 @@
+import { existsSync, readFileSync } from "node:fs";
+import { join } from "node:path";
+
+// rtk-ai/rtk#665, rtk-ai/rtk#1489 — upstream biome → rtk lint routing gap
+const BIOME_RE = /biome(?:\s|$)/;
+// rtk-ai/rtk#1367, rtk-ai/rtk#1604 — rg → grep rewrite is lossy (grep has no -g, -U, --glob, etc.)
+const RG_RE = /^rg(?:\s|$)/;
+const PACKAGE_LINT_RE = /^(?:pnpm|npm|yarn|bun)\s+(?:run\s+)?lint(?:\s|$)/;
+const BIOME_CONFIG_FILES = ["biome.json", "biome.jsonc"];
+
+/**
+ * Normalize a command string by stripping leading shell wrappers that
+ * would interfere with `^`-anchored guard regexes:
+ * - `cd /path && command`
+ * - `rtk command`  (from a previous RTK rewrite pass)
+ */
+function stripShellWrappers(command: string): string {
+  let s = command.trimStart();
+  // Strip leading `cd /some/path && ` or `cd /some/path;`
+  while (/^cd\s+\S+(?:\s*[;&]\s*|\s+&&\s+)/.test(s)) {
+    s = s.replace(/^cd\s+\S+(?:\s*[;&]\s*|\s+&&\s+)/, "");
+  }
+  // Strip leading `rtk ` prefix from a prior rewrite attempt
+  s = s.replace(/^rtk\s+/, "");
+  return s.trimStart();
+}
+
+/**
+ * Return whether SuPi should bypass RTK's rewrite registry for commands with
+ * known lossy rewrites. RTK 0.37.x collapses several Biome invocations into
+ * `rtk lint ...`, which can drop the `biome` subcommand and produce misleading
+ * lint-wrapper warnings. Passing these forms through preserves correctness while
+ * upstream rewrite routing is fixed.
+ *
+ * TODO: Remove this workaround after rtk-ai/rtk#665 and rtk-ai/rtk#1489 are
+ * closed and the `pnpm exec biome ...` routing gap is fixed upstream.
+ *
+ * `rg` (ripgrep) commands are also guarded: RTK rewrites `rg` to `rtk grep` or
+ * native `grep`, but doesn't translate ripgrep-specific flags (`-g`, `-U`,
+ * `--glob`, `--type`, etc.), producing broken commands like `grep -g '*.ts'`.
+ * See rtk-ai/rtk#1367 and rtk-ai/rtk#1604.
+ *
+ * TODO: Remove this workaround after RTK properly handles rg-native flags.
+ */
+export function shouldBypassRtkRewrite(command: string, cwd: string): boolean {
+  const normalized = stripShellWrappers(command);
+  if (hasRtkDisabledPrefix(normalized)) return true;
+  if (BIOME_RE.test(normalized)) return true;
+  if (RG_RE.test(normalized)) return true;
+  if (PACKAGE_LINT_RE.test(normalized) && projectUsesBiome(cwd)) return true;
+  return false;
+}
+
+function hasRtkDisabledPrefix(command: string): boolean {
+  if (command.startsWith("RTK_DISABLED=1 ")) return true;
+  return command.startsWith("env RTK_DISABLED=1 ");
+}
+
+function projectUsesBiome(cwd: string): boolean {
+  if (BIOME_CONFIG_FILES.some((file) => existsSync(join(cwd, file)))) {
+    return true;
+  }
+
+  try {
+    const packageJson = JSON.parse(readFileSync(join(cwd, "package.json"), "utf-8")) as {
+      scripts?: Record<string, unknown>;
+    };
+    return Object.values(packageJson.scripts ?? {}).some(
+      (script) => typeof script === "string" && /(^|\s)biome(\s|$)/.test(script),
+    );
+  } catch {
+    return false;
+  }
+}

package/src/index.ts

@@ -0,0 +1 @@
+export { default } from "./rtk.ts";

package/src/rewrite.ts

@@ -0,0 +1,113 @@
+import { execFileSync } from "node:child_process";
+
+export type RtkRewriteFailureReason =
+  | "timeout"
+  | "unavailable"
+  | "non-zero-exit"
+  | "empty-output"
+  | "error";
+
+export type RtkRewriteResult =
+  | {
+      kind: "rewritten" | "unchanged";
+      command: string;
+      durationMs: number;
+      stdout: string;
+      stderr?: string;
+    }
+  | {
+      kind: "failed";
+      reason: RtkRewriteFailureReason;
+      durationMs: number;
+      stdout?: string;
+      stderr?: string;
+      exitCode?: number;
+      errorMessage?: string;
+    };
+
+interface ExecErrorShape {
+  code?: string;
+  status?: number;
+  signal?: string;
+  stdout?: string | Buffer;
+  stderr?: string | Buffer;
+  message?: string;
+}
+
+function toText(value: string | Buffer | undefined): string | undefined {
+  return typeof value === "string" ? value : value?.toString("utf-8");
+}
+
+function classifyError(err: ExecErrorShape): RtkRewriteFailureReason {
+  const message = err.message?.toLowerCase() ?? "";
+  if (err.code === "ENOENT") return "unavailable";
+  if (err.code === "ETIMEDOUT" || message.includes("timeout") || message.includes("timed out")) {
+    return "timeout";
+  }
+  if (typeof err.status === "number") return "non-zero-exit";
+  return "error";
+}
+
+/**
+ * Rewrite a bash command through RTK's `rtk rewrite` CLI with structured diagnostics.
+ *
+ * Non-zero RTK exits can still emit a usable rewrite on stdout; those are treated
+ * as successful rewrites so callers preserve existing RTK behavior.
+ */
+export function rtkRewriteDetailed(command: string, timeoutMs: number): RtkRewriteResult {
+  const started = Date.now();
+  try {
+    const stdout = execFileSync("rtk", ["rewrite", command], {
+      encoding: "utf-8",
+      timeout: timeoutMs,
+    });
+    const durationMs = Date.now() - started;
+    const rewritten = stdout.trim();
+    if (!rewritten) {
+      return { kind: "failed", reason: "empty-output", durationMs, stdout };
+    }
+    return {
+      kind: rewritten === command ? "unchanged" : "rewritten",
+      command: rewritten,
+      durationMs,
+      stdout,
+    };
+  } catch (error: unknown) {
+    const err = error as ExecErrorShape;
+    const stdout = toText(err.stdout);
+    const stderr = toText(err.stderr);
+    const durationMs = Date.now() - started;
+    const rewritten = stdout?.trim();
+    if (rewritten) {
+      return {
+        kind: rewritten === command ? "unchanged" : "rewritten",
+        command: rewritten,
+        durationMs,
+        stdout: stdout ?? "",
+        stderr,
+      };
+    }
+    return {
+      kind: "failed",
+      reason: classifyError(err),
+      durationMs,
+      stdout,
+      stderr,
+      exitCode: err.status,
+      errorMessage: err.message,
+    };
+  }
+}
+
+/**
+ * Rewrite a bash command through RTK's `rtk rewrite` CLI.
+ *
+ * @param command The original shell command.
+ * @param timeoutMs Timeout in milliseconds for the rewrite call.
+ * @returns The rewritten command string, or `undefined` if RTK could not rewrite it
+ *          (timeout, binary missing, or non-zero exit without usable stdout).
+ */
+export function rtkRewrite(command: string, timeoutMs: number): string | undefined {
+  const result = rtkRewriteDetailed(command, timeoutMs);
+  return result.kind === "failed" ? undefined : result.command;
+}