mu-coding 0.15.0 → 0.16.1

package/src/tui/update/runUpdateInTui.ts

@@ -1,127 +0,0 @@
-/**
- * TUI-friendly update runner — same semantics as `mu update` from the CLI,
- * but never writes to stdout/stderr (Ink owns the terminal). Progress and
- * results are surfaced as toasts via `uiService.notify`.
- *
- * Mirrors `cli/update.ts` but uses `child_process.execFile` with `stdio:
- * 'pipe'` so subprocess output is buffered, not streamed to the TUI's tty.
- */
-
-import { execFile } from 'node:child_process';
-import { realpathSync } from 'node:fs';
-import { promisify } from 'node:util';
-import { ensureDataDir } from '../../cli/install';
-import { invalidateUpdateCheckCache } from '../../runtime/startupUpdateCheck';
-import { listConfiguredNpmPlugins, PACKAGE_NAME } from '../../runtime/updateCheck';
-import type { InkUIService } from '../plugins/InkUIService';
-
-const execFileAsync = promisify(execFile);
-
-export type UpdateScope = 'all' | 'plugins' | 'self';
-
-interface SelfInstallStrategy {
-  manager: 'bun' | 'npm' | 'pnpm' | 'yarn' | 'unknown';
-  command?: [string, string[]];
-}
-
-function detectSelfInstall(): SelfInstallStrategy {
-  let bin = process.argv[1] ?? '';
-  try {
-    bin = realpathSync(bin);
-  } catch {
-    // keep raw argv
-  }
-  const norm = bin.replace(/\\/g, '/');
-  if (norm.includes('/.bun/') || norm.includes('/bun/install/')) {
-    return { manager: 'bun', command: ['bun', ['add', '-g', `${PACKAGE_NAME}@latest`]] };
-  }
-  if (norm.includes('/pnpm/')) {
-    return { manager: 'pnpm', command: ['pnpm', ['add', '-g', `${PACKAGE_NAME}@latest`]] };
-  }
-  if (norm.includes('/.yarn/') || norm.includes('/yarn/global/')) {
-    return { manager: 'yarn', command: ['yarn', ['global', 'add', `${PACKAGE_NAME}@latest`]] };
-  }
-  if (norm.includes('/npm/') || norm.includes('node_modules/.bin/mu')) {
-    return { manager: 'npm', command: ['npm', ['i', '-g', `${PACKAGE_NAME}@latest`]] };
-  }
-  return { manager: 'unknown' };
-}
-
-async function updatePlugin(name: string, dataDir: string): Promise<boolean> {
-  try {
-    await execFileAsync('bun', ['update', '--latest', name], { cwd: dataDir });
-    return true;
-  } catch {
-    return false;
-  }
-}
-
-async function updatePlugins(ui: InkUIService): Promise<{ ok: number; failed: number; total: number }> {
-  const dataDir = ensureDataDir();
-  const names = listConfiguredNpmPlugins();
-  if (names.length === 0) return { ok: 0, failed: 0, total: 0 };
-
-  let ok = 0;
-  let failed = 0;
-  for (const name of names) {
-    ui.notify(`Updating ${name}…`, 'info');
-    if (await updatePlugin(name, dataDir)) ok += 1;
-    else {
-      failed += 1;
-      ui.notify(`Failed to update ${name}`, 'error');
-    }
-  }
-  return { ok, failed, total: names.length };
-}
-
-async function updateSelf(ui: InkUIService): Promise<boolean> {
-  const strategy = detectSelfInstall();
-  if (!strategy.command) {
-    ui.notify(`Cannot auto-detect mu's installer. Re-install manually: bun add -g ${PACKAGE_NAME}@latest`, 'warning');
-    return false;
-  }
-  const [bin, args] = strategy.command;
-  ui.notify(`Updating mu via ${strategy.manager}…`, 'info');
-  try {
-    await execFileAsync(bin, args);
-    ui.notify('mu updated. Restart your session to pick up the new version.', 'success');
-    return true;
-  } catch (err) {
-    const message = err instanceof Error ? err.message.split('\n')[0] : String(err);
-    ui.notify(`Failed to update mu: ${message}`, 'error');
-    return false;
-  }
-}
-
-/**
- * Entry point used by the `/update` slash command. Fire-and-forget — the
- * caller voids the returned promise. All progress / errors land in toasts.
- */
-export async function runUpdateInTui(scope: UpdateScope, ui: InkUIService): Promise<void> {
-  if (scope === 'plugins') {
-    const { ok, failed, total } = await updatePlugins(ui);
-    if (total === 0) ui.notify('No npm plugins configured.', 'info');
-    else if (failed === 0) ui.notify(`Updated ${ok}/${total} plugin${total === 1 ? '' : 's'}.`, 'success');
-    else ui.notify(`Plugins: ${ok} updated, ${failed} failed.`, 'warning');
-    invalidateUpdateCheckCache();
-    return;
-  }
-
-  if (scope === 'self') {
-    await updateSelf(ui);
-    invalidateUpdateCheckCache();
-    return;
-  }
-
-  // 'all'
-  const plugins = await updatePlugins(ui);
-  const selfOk = await updateSelf(ui);
-  if (plugins.total === 0 && selfOk) {
-    // mu-only success already toasted; nothing to add
-  } else if (plugins.failed === 0 && selfOk) {
-    ui.notify('Update complete.', 'success');
-  } else {
-    ui.notify('Update finished with errors — see prior messages.', 'warning');
-  }
-  invalidateUpdateCheckCache();
-}

package/src/utils/clipboard.ts

@@ -1,97 +0,0 @@
-import { execFileSync, execSync } from 'node:child_process';
-import { existsSync, readFileSync, statSync, unlinkSync } from 'node:fs';
-import { tmpdir } from 'node:os';
-import { join } from 'node:path';
-import type { ImageAttachment } from 'mu-core';
-
-const CLIPBOARD_TIMEOUT = 3000;
-
-function tryExecFile(file: string, args: string[]): boolean {
-  try {
-    execFileSync(file, args, { stdio: 'pipe', timeout: CLIPBOARD_TIMEOUT });
-    return true;
-  } catch {
-    return false;
-  }
-}
-
-function tryShell(command: string): boolean {
-  try {
-    execSync(command, { stdio: 'pipe', timeout: CLIPBOARD_TIMEOUT });
-    return true;
-  } catch {
-    return false;
-  }
-}
-
-function commandExists(name: string): boolean {
-  return tryExecFile('which', [name]);
-}
-
-function extractMacImage(tmpFile: string): boolean {
-  if (commandExists('pngpaste')) {
-    return tryExecFile('pngpaste', [tmpFile]);
-  }
-  // Each `-e` is a discrete script line — passing as separate args avoids shell quoting traps.
-  return tryExecFile('osascript', [
-    '-e',
-    'tell application "System Events"',
-    '-e',
-    'set imgData to the clipboard as «class PNGf»',
-    '-e',
-    `set fp to open for access POSIX file "${tmpFile}" with write permission`,
-    '-e',
-    'write imgData to fp',
-    '-e',
-    'close access fp',
-    '-e',
-    'end tell',
-  ]);
-}
-
-function extractLinuxImage(tmpFile: string): boolean {
-  // xclip / wl-paste write to stdout; redirection still requires a shell.
-  // tmpFile is constructed from tmpdir() + a numeric timestamp, so quoting is safe.
-  if (tryShell(`xclip -selection clipboard -t image/png -o > "${tmpFile}"`)) {
-    return true;
-  }
-  return tryShell(`wl-paste --type image/png > "${tmpFile}"`);
-}
-
-function extractPlatformImage(tmpFile: string): boolean {
-  if (process.platform === 'darwin') {
-    return extractMacImage(tmpFile);
-  }
-  if (process.platform === 'linux') {
-    return extractLinuxImage(tmpFile);
-  }
-  return false;
-}
-
-function readTmpAsAttachment(tmpFile: string): ImageAttachment | null {
-  if (!existsSync(tmpFile)) {
-    return null;
-  }
-  if (statSync(tmpFile).size === 0) {
-    unlinkSync(tmpFile);
-    return null;
-  }
-  const buffer = readFileSync(tmpFile);
-  unlinkSync(tmpFile);
-  return { data: buffer.toString('base64'), mimeType: 'image/png', name: 'clipboard.png' };
-}
-
-export function readClipboardImage(): ImageAttachment | null {
-  const tmpFile = join(tmpdir(), `mu-clip-${Date.now()}.png`);
-  try {
-    if (!extractPlatformImage(tmpFile)) {
-      return null;
-    }
-    return readTmpAsAttachment(tmpFile);
-  } catch {
-    if (existsSync(tmpFile)) {
-      unlinkSync(tmpFile);
-    }
-    return null;
-  }
-}

package/src/utils/diff.test.ts

@@ -1,56 +0,0 @@
-import { describe, expect, it } from 'bun:test';
-import { computeDiff, renderDiff } from './diff';
-
-describe('computeDiff', () => {
-  it('emits only context lines when texts are identical', () => {
-    const diff = computeDiff('a\nb\nc', 'a\nb\nc');
-    expect(diff.lines.every((l) => l.type === 'context')).toBe(true);
-    expect(diff.lines.find((l) => l.type === 'old')).toBeUndefined();
-    expect(diff.lines.find((l) => l.type === 'new')).toBeUndefined();
-    expect(diff.totalOldLines).toBe(3);
-    expect(diff.totalNewLines).toBe(3);
-  });
-
-  it('detects an added line in the middle', () => {
-    const diff = computeDiff('a\nb\nc', 'a\nx\nb\nc');
-    const types = diff.lines.map((l) => l.type);
-    expect(types).toContain('new');
-    expect(diff.lines.find((l) => l.type === 'new')?.value).toBe('x');
-  });
-
-  it('detects a removed line', () => {
-    const diff = computeDiff('a\nb\nc', 'a\nc');
-    expect(diff.lines.find((l) => l.type === 'old')?.value).toBe('b');
-  });
-
-  it('returns no lines when either side exceeds the size limit', () => {
-    const huge = Array.from({ length: 600 }, (_, i) => `line${i}`).join('\n');
-    const diff = computeDiff(huge, huge.replace('line0', 'lineX'));
-    expect(diff.lines).toEqual([]);
-    expect(diff.totalOldLines).toBe(600);
-  });
-
-  it('emits up to CONTEXT_LINES of leading/trailing context', () => {
-    const diff = computeDiff('a\nb\nc\nd\ne', 'a\nb\nc\nX\ne');
-    const contextValues = diff.lines.filter((l) => l.type === 'context').map((l) => l.value);
-    expect(contextValues).toContain('c');
-    expect(contextValues).toContain('e');
-  });
-});
-
-describe('renderDiff', () => {
-  it('prefixes added/removed/context lines and reports truncation', () => {
-    const diff = computeDiff('a\nb', 'a\nc');
-    const { lines, truncated } = renderDiff(diff, 100);
-    expect(lines.some((l) => l.startsWith('-'))).toBe(true);
-    expect(lines.some((l) => l.startsWith('+'))).toBe(true);
-    expect(truncated).toBe(false);
-  });
-
-  it('reports truncated when diff exceeds maxLines', () => {
-    const diff = computeDiff('a', Array.from({ length: 50 }, (_, i) => `n${i}`).join('\n'));
-    const { lines, truncated } = renderDiff(diff, 5);
-    expect(lines).toHaveLength(5);
-    expect(truncated).toBe(true);
-  });
-});

package/src/utils/diff.ts

@@ -1,81 +0,0 @@
-// Lightweight diff for edit_file tool output.
-// Uses prefix/suffix matching — sufficient for small, localized edits.
-
-export interface DiffLine {
-  type: 'context' | 'old' | 'new';
-  value: string;
-}
-
-export interface DiffResult {
-  lines: DiffLine[];
-  totalOldLines: number;
-  totalNewLines: number;
-}
-
-const CONTEXT_LINES = 3;
-const MAX_LINES = 500;
-
-export function computeDiff(oldText: string, newText: string): DiffResult {
-  const oldLines = oldText.split('\n');
-  const newLines = newText.split('\n');
-
-  if (oldLines.length > MAX_LINES || newLines.length > MAX_LINES) {
-    return { lines: [], totalOldLines: oldLines.length, totalNewLines: newLines.length };
-  }
-
-  // Find common prefix
-  let prefixLen = 0;
-  const minLen = Math.min(oldLines.length, newLines.length);
-  while (prefixLen < minLen && oldLines[prefixLen] === newLines[prefixLen]) {
-    prefixLen++;
-  }
-
-  // Find common suffix (don't overlap with prefix)
-  let suffixLen = 0;
-  const maxSuffix = Math.min(oldLines.length - prefixLen, newLines.length - prefixLen);
-  while (
-    suffixLen < maxSuffix &&
-    oldLines[oldLines.length - 1 - suffixLen] === newLines[newLines.length - 1 - suffixLen]
-  ) {
-    suffixLen++;
-  }
-
-  const result: DiffLine[] = [];
-
-  // Context from prefix (last N lines)
-  const ctxStart = Math.max(0, prefixLen - CONTEXT_LINES);
-  for (let i = ctxStart; i < prefixLen; i++) {
-    result.push({ type: 'context', value: oldLines[i] });
-  }
-
-  // Removed lines
-  for (let i = prefixLen; i < oldLines.length - suffixLen; i++) {
-    result.push({ type: 'old', value: oldLines[i] });
-  }
-
-  // Added lines
-  for (let i = prefixLen; i < newLines.length - suffixLen; i++) {
-    result.push({ type: 'new', value: newLines[i] });
-  }
-
-  // Context from suffix (first N lines)
-  const ctxEnd = Math.min(suffixLen, CONTEXT_LINES);
-  for (let i = 0; i < ctxEnd; i++) {
-    const idx = oldLines.length - suffixLen + i;
-    result.push({ type: 'context', value: oldLines[idx] });
-  }
-
-  return { lines: result, totalOldLines: oldLines.length, totalNewLines: newLines.length };
-}
-
-export function renderDiff(diff: DiffResult, maxLines: number): { lines: string[]; truncated: boolean } {
-  const result: string[] = [];
-  const capped = diff.lines.slice(0, maxLines);
-
-  for (const line of capped) {
-    const prefix = line.type === 'old' ? '-' : line.type === 'new' ? '+' : ' ';
-    result.push(`${prefix} ${line.value}`);
-  }
-
-  return { lines: result, truncated: diff.lines.length > maxLines };
-}