Codeman/test/keyboard-shortcuts.test.ts

import { readFileSync } from 'node:fs';
import { describe, expect, it } from 'vitest';

const appSource = readFileSync('src/web/public/app.js', 'utf8');
const terminalUiSource = readFileSync('src/web/public/terminal-ui.js', 'utf8');
const helpHtml = readFileSync('src/web/public/index.html', 'utf8');
const readme = readFileSync('README.md', 'utf8');

describe('keyboard shortcuts', () => {
  it('uses physical Option+number keys so macOS special characters do not break tab switching', () => {
    expect(appSource).toContain('e.code ||');
    expect(appSource).toContain('Digit([1-9])');
    expect(appSource).toContain('parseInt(digitMatch[1], 10) - 1');
  });

  it('provides Option+bracket shortcuts for previous and next session', () => {
    // Registry entries (so they can be rebound), matched by physical key code.
    expect(appSource).toMatch(/id: 'previous-session'[\s\S]*?code: 'BracketLeft'[\s\S]*?action: 'previousSession'/);
    expect(appSource).toMatch(/id: 'next-session-alt'[\s\S]*?code: 'BracketRight'[\s\S]*?action: 'nextSession'/);
    expect(appSource).toContain('previousSession: () => this.prevSession()');
    expect(appSource).toContain('nextSession: () => this.nextSession()');
  });

  it('suppresses xterm PTY injection for the same physical Alt nav codes (no ESC leak)', () => {
    // terminal-ui.js must gate its xterm pass-through on the SAME physical e.code set the
    // app.js handler consumes; otherwise Alt+[ / Alt+] (and Option+digit on remapped macOS
    // layouts) switch tabs AND inject ESC<char> into the focused terminal. Keep in sync.
    expect(terminalUiSource).toContain('/^(Digit[1-9]|BracketLeft|BracketRight|KeyK)$/.test(ev.code');
  });

  it('documents the Alt/Option shortcuts in help and README', () => {
    expect(helpHtml).toContain('<kbd>Alt/Option</kbd>+<kbd>[</kbd>');
    expect(helpHtml).toContain('<kbd>Alt/Option</kbd>+<kbd>]</kbd>');
    expect(helpHtml).toContain('<kbd>Alt/Option</kbd>+<kbd>1-9</kbd>');
    expect(readme).toContain('`Alt/Option+[` / `Alt/Option+]`');
    expect(readme).toContain('`Alt/Option+1`-`Alt/Option+9`');
  });

  it('documents the Command-K open-session palette in help and README', () => {
    expect(appSource).toContain('this.openCommandPalette()');
    expect(helpHtml).toContain('<kbd>Ctrl/Cmd/Option</kbd>+<kbd>K</kbd>');
    expect(readme).toMatch(/\| `Ctrl\/Cmd\/Option\+K`\s+\| Find open session or start a new one\s+\|/);
  });

  it('gates the palette chord in the xterm custom key handler (no 0x0b kill-line into the PTY)', () => {
    // The document-level capture handler opens the palette, but preventDefault()
    // does NOT stop xterm from evaluating Ctrl+K into 0x0b and writing it to the
    // live PTY — terminal-ui.js must return false for the palette chord.
    expect(terminalUiSource).toMatch(/ev\.type === 'keydown' && this\.shouldOpenCommandPaletteFromShortcut\?\.\(ev\)/);
  });

  it('dispatches document shortcuts through the shortcut registry (rebind/disable aware)', () => {
    // The legacy hardcoded SHORTCUTS table must stay gone — dispatch goes through
    // getShortcutRegistry() + matchesShortcutEvent() so overrides and per-shortcut
    // disables (App Settings → Shortcuts) actually take effect.
    expect(appSource).not.toContain('const SHORTCUTS = [');
    expect(appSource).toContain('const SHORTCUT_ACTIONS = this._shortcutActions();');
    expect(appSource).toContain('for (const shortcut of this.getShortcutRegistry())');
    expect(appSource).toContain('if (this.matchesShortcutEvent(e, shortcut))');
    expect(appSource).toContain('if (shortcut.disabled || !shortcut.action) continue;');
  });

  it('keeps the interrupt when Ctrl+C copies a selection (#211)', () => {
    // The xterm handler owns this decision, and the no-selection path must fall
    // through with NO preventDefault so xterm still evaluates Ctrl+C into 0x03.
    expect(terminalUiSource).toContain('this.shouldCopyTerminalSelectionFromShortcut?.(ev)');
    // The CLEANED selection is what decides. A drag across the blank part of a row
    // selects real padding spaces, so the raw text is truthy and testing it would
    // spend the press on a copy of nothing — the same lost interrupt this test
    // guards, reached by a different door.
    //
    // The RAW selection is what travels on, because copyTerminalSelection cleans
    // again on its own and the margin strip is not idempotent. Passing the
    // cleaned string dedented every claude and codex copy twice; see
    // test/terminal-copy-clean.test.ts for the branch's own pin.
    expect(terminalUiSource).toMatch(/const selection = this\.cleanedTerminalSelection\(raw\);/);
    expect(terminalUiSource).toMatch(/if \(selection\.trim\(\)\) \{/);
    expect(terminalUiSource).toContain('void this.copyTerminalSelection(raw);');
    expect(appSource).toContain("id: 'copy-selection'");
  });

  it('documents the terminal copy shortcut in help and README', () => {
    expect(helpHtml).toContain('<kbd>Ctrl</kbd>+<kbd>C</kbd>');
    expect(helpHtml).toContain('<kbd>Ctrl</kbd>+<kbd>Shift</kbd>+<kbd>C</kbd>');
    expect(readme).toContain('`Ctrl/Cmd+C`');
    expect(readme).toContain('`Ctrl+Shift+C`');
  });
});