@peaske7/readit 0.1.7 → 0.2.0

package/src/lib/shortcut-registry.test.ts

@@ -1,173 +0,0 @@
-import { describe, expect, it } from "vitest";
-import {
-  DEFAULT_SHORTCUTS,
-  formatBinding,
-  isReservedBinding,
-  type KeybindingOverride,
-  matchesBinding,
-  resolveShortcuts,
-  ShortcutActions,
-  type ShortcutBinding,
-} from "./shortcut-registry";
-
-describe("DEFAULT_SHORTCUTS", () => {
-  it("defines 7 shortcuts", () => {
-    expect(DEFAULT_SHORTCUTS).toHaveLength(7);
-  });
-
-  it("has unique IDs", () => {
-    const ids = DEFAULT_SHORTCUTS.map((s) => s.id);
-    expect(new Set(ids).size).toBe(ids.length);
-  });
-
-  it("all enabled by default", () => {
-    for (const shortcut of DEFAULT_SHORTCUTS) {
-      expect(shortcut.enabled).toBe(true);
-    }
-  });
-});
-
-describe("matchesBinding", () => {
-  it("matches exact key + modifiers", () => {
-    const binding: ShortcutBinding = { key: "c", alt: true };
-    const event = new KeyboardEvent("keydown", { key: "c", altKey: true });
-    expect(matchesBinding(event, binding)).toBe(true);
-  });
-
-  it("rejects wrong key", () => {
-    const binding: ShortcutBinding = { key: "c", alt: true };
-    const event = new KeyboardEvent("keydown", { key: "v", altKey: true });
-    expect(matchesBinding(event, binding)).toBe(false);
-  });
-
-  it("rejects extra modifiers", () => {
-    const binding: ShortcutBinding = { key: "c", alt: true };
-    const event = new KeyboardEvent("keydown", {
-      key: "c",
-      altKey: true,
-      shiftKey: true,
-    });
-    expect(matchesBinding(event, binding)).toBe(false);
-  });
-
-  it("matches binding with shift", () => {
-    const binding: ShortcutBinding = { key: "c", alt: true, shift: true };
-    const event = new KeyboardEvent("keydown", {
-      key: "c",
-      altKey: true,
-      shiftKey: true,
-    });
-    expect(matchesBinding(event, binding)).toBe(true);
-  });
-
-  it("matches meta key", () => {
-    const binding: ShortcutBinding = { key: "c", meta: true };
-    const event = new KeyboardEvent("keydown", { key: "c", metaKey: true });
-    expect(matchesBinding(event, binding)).toBe(true);
-  });
-
-  it("matches shifted letter key (browser reports uppercase)", () => {
-    const binding: ShortcutBinding = { key: "c", alt: true, shift: true };
-    const event = new KeyboardEvent("keydown", {
-      key: "C",
-      altKey: true,
-      shiftKey: true,
-    });
-    expect(matchesBinding(event, binding)).toBe(true);
-  });
-
-  it("matches meta+shift letter key (browser reports uppercase)", () => {
-    const binding: ShortcutBinding = { key: "c", meta: true, shift: true };
-    const event = new KeyboardEvent("keydown", {
-      key: "C",
-      metaKey: true,
-      shiftKey: true,
-    });
-    expect(matchesBinding(event, binding)).toBe(true);
-  });
-
-  it("matches key-only binding (no modifiers)", () => {
-    const binding: ShortcutBinding = { key: "Escape" };
-    const event = new KeyboardEvent("keydown", { key: "Escape" });
-    expect(matchesBinding(event, binding)).toBe(true);
-  });
-});
-
-describe("formatBinding", () => {
-  it("formats Alt+C", () => {
-    const binding: ShortcutBinding = { key: "c", alt: true };
-    expect(formatBinding(binding, true)).toBe("Alt+C");
-  });
-
-  it("formats meta key as ⌘ on Mac", () => {
-    const binding: ShortcutBinding = { key: "c", meta: true };
-    expect(formatBinding(binding, true)).toBe("⌘+C");
-  });
-
-  it("formats meta key as Ctrl on non-Mac", () => {
-    const binding: ShortcutBinding = { key: "c", meta: true };
-    expect(formatBinding(binding, false)).toBe("Ctrl+C");
-  });
-
-  it("formats shift modifier", () => {
-    const binding: ShortcutBinding = { key: "c", meta: true, shift: true };
-    expect(formatBinding(binding, true)).toBe("⌘+Shift+C");
-  });
-
-  it("formats arrow keys with symbols", () => {
-    const binding: ShortcutBinding = { key: "ArrowUp", alt: true };
-    expect(formatBinding(binding, true)).toBe("Alt+↑");
-  });
-
-  it("formats Escape", () => {
-    const binding: ShortcutBinding = { key: "Escape" };
-    expect(formatBinding(binding, true)).toBe("Esc");
-  });
-});
-
-describe("resolveShortcuts", () => {
-  it("returns defaults when no overrides", () => {
-    const resolved = resolveShortcuts([]);
-    expect(resolved).toEqual(DEFAULT_SHORTCUTS);
-  });
-
-  it("applies enabled override", () => {
-    const overrides: KeybindingOverride[] = [
-      { id: ShortcutActions.COPY_ALL, enabled: false },
-    ];
-    const resolved = resolveShortcuts(overrides);
-    const copyAll = resolved.find((s) => s.id === ShortcutActions.COPY_ALL);
-    expect(copyAll?.enabled).toBe(false);
-  });
-
-  it("applies binding override", () => {
-    const overrides: KeybindingOverride[] = [
-      {
-        id: ShortcutActions.COPY_ALL,
-        enabled: true,
-        binding: { key: "a", meta: true },
-      },
-    ];
-    const resolved = resolveShortcuts(overrides);
-    const copyAll = resolved.find((s) => s.id === ShortcutActions.COPY_ALL);
-    expect(copyAll?.binding).toEqual({ key: "a", meta: true });
-  });
-
-  it("ignores unknown override IDs", () => {
-    const overrides: KeybindingOverride[] = [
-      { id: "unknown_action", enabled: false },
-    ];
-    const resolved = resolveShortcuts(overrides);
-    expect(resolved).toEqual(DEFAULT_SHORTCUTS);
-  });
-});
-
-describe("isReservedBinding", () => {
-  it("detects ⌘+W as reserved", () => {
-    expect(isReservedBinding({ key: "w", meta: true })).toBe(true);
-  });
-
-  it("allows Alt+C", () => {
-    expect(isReservedBinding({ key: "c", alt: true })).toBe(false);
-  });
-});

package/src/lib/shortcut-registry.ts

@@ -1,209 +0,0 @@
-import type { KeybindingOverride, ShortcutBinding } from "../types";
-
-export type { KeybindingOverride, ShortcutBinding };
-
-export const ShortcutActions = {
-  COPY_ALL: "copyAll",
-  COPY_ALL_RAW: "copyAllRaw",
-  NAVIGATE_NEXT: "navigateNext",
-  NAVIGATE_PREVIOUS: "navigatePrevious",
-  COPY_SELECTION_RAW: "copySelectionRaw",
-  COPY_SELECTION_LLM: "copySelectionLLM",
-  CLEAR_SELECTION: "clearSelection",
-} as const;
-
-export type ShortcutAction =
-  (typeof ShortcutActions)[keyof typeof ShortcutActions];
-
-export interface ShortcutDefinition {
-  id: ShortcutAction;
-  label: string;
-  description: string;
-  defaultBinding: ShortcutBinding;
-  binding: ShortcutBinding; // resolved (default or user override)
-  enabled: boolean;
-}
-
-export const DEFAULT_SHORTCUTS: ShortcutDefinition[] = [
-  {
-    id: ShortcutActions.COPY_ALL,
-    label: "Copy All (AI)",
-    description: "Copy all comments in AI prompt format",
-    defaultBinding: { key: "c", alt: true },
-    binding: { key: "c", alt: true },
-    enabled: true,
-  },
-  {
-    id: ShortcutActions.COPY_ALL_RAW,
-    label: "Copy All (Raw)",
-    description: "Copy all comments as raw text",
-    defaultBinding: { key: "c", alt: true, shift: true },
-    binding: { key: "c", alt: true, shift: true },
-    enabled: true,
-  },
-  {
-    id: ShortcutActions.NAVIGATE_NEXT,
-    label: "Next Comment",
-    description: "Navigate to next comment",
-    defaultBinding: { key: "ArrowDown", alt: true },
-    binding: { key: "ArrowDown", alt: true },
-    enabled: true,
-  },
-  {
-    id: ShortcutActions.NAVIGATE_PREVIOUS,
-    label: "Previous Comment",
-    description: "Navigate to previous comment",
-    defaultBinding: { key: "ArrowUp", alt: true },
-    binding: { key: "ArrowUp", alt: true },
-    enabled: true,
-  },
-  {
-    id: ShortcutActions.COPY_SELECTION_RAW,
-    label: "Copy Selection",
-    description: "Copy selected text",
-    defaultBinding: { key: "c", meta: true },
-    binding: { key: "c", meta: true },
-    enabled: true,
-  },
-  {
-    id: ShortcutActions.COPY_SELECTION_LLM,
-    label: "Copy Selection (LLM)",
-    description: "Copy selected text with context for LLM",
-    defaultBinding: { key: "c", meta: true, shift: true },
-    binding: { key: "c", meta: true, shift: true },
-    enabled: true,
-  },
-  {
-    id: ShortcutActions.CLEAR_SELECTION,
-    label: "Clear Selection",
-    description: "Clear text selection",
-    defaultBinding: { key: "Escape" },
-    binding: { key: "Escape" },
-    enabled: true,
-  },
-];
-
-/**
- * Check if a KeyboardEvent matches a ShortcutBinding.
- * All modifier flags must match exactly (no extra modifiers allowed).
- */
-export function matchesBinding(
-  event: KeyboardEvent,
-  binding: ShortcutBinding,
-): boolean {
-  if (event.key.toLowerCase() !== binding.key.toLowerCase()) return false;
-  if (event.altKey !== (binding.alt ?? false)) return false;
-  if (event.metaKey !== (binding.meta ?? false)) return false;
-  if (event.shiftKey !== (binding.shift ?? false)) return false;
-  return true;
-}
-
-const KEY_DISPLAY: Record<string, string> = {
-  ArrowUp: "↑",
-  ArrowDown: "↓",
-  ArrowLeft: "←",
-  ArrowRight: "→",
-  Escape: "Esc",
-  " ": "Space",
-  Enter: "Enter",
-};
-
-/**
- * Format a ShortcutBinding for display.
- * Shows ⌘ on Mac, Ctrl on other platforms.
- */
-export function formatBinding(
-  binding: ShortcutBinding,
-  isMac: boolean,
-): string {
-  const parts: string[] = [];
-
-  if (binding.meta) parts.push(isMac ? "⌘" : "Ctrl");
-  if (binding.alt) parts.push("Alt");
-  if (binding.shift) parts.push("Shift");
-
-  const keyDisplay = KEY_DISPLAY[binding.key] ?? binding.key.toUpperCase();
-  parts.push(keyDisplay);
-
-  return parts.join("+");
-}
-
-/**
- * Merge user overrides with default shortcuts.
- * Unknown override IDs are ignored.
- */
-export function resolveShortcuts(
-  overrides: KeybindingOverride[],
-): ShortcutDefinition[] {
-  if (overrides.length === 0) return DEFAULT_SHORTCUTS;
-
-  const overrideMap = new Map(overrides.map((o) => [o.id, o]));
-
-  return DEFAULT_SHORTCUTS.map((shortcut) => {
-    const override = overrideMap.get(shortcut.id);
-    if (!override) return shortcut;
-
-    return {
-      ...shortcut,
-      binding: override.binding ?? shortcut.defaultBinding,
-      enabled: override.enabled,
-    };
-  });
-}
-
-/**
- * Browser-reserved key combos that cannot be rebound.
- */
-export const RESERVED_BINDINGS: ShortcutBinding[] = [
-  { key: "w", meta: true },
-  { key: "t", meta: true },
-  { key: "n", meta: true },
-  { key: "q", meta: true },
-  { key: "l", meta: true },
-  { key: "r", meta: true },
-  { key: "F5" },
-  { key: "F11" },
-  { key: "F12" },
-];
-
-/**
- * Check if a binding conflicts with browser-reserved shortcuts.
- */
-export function isReservedBinding(binding: ShortcutBinding): boolean {
-  return RESERVED_BINDINGS.some(
-    (reserved) =>
-      reserved.key.toLowerCase() === binding.key.toLowerCase() &&
-      (reserved.meta ?? false) === (binding.meta ?? false) &&
-      (reserved.alt ?? false) === (binding.alt ?? false) &&
-      (reserved.shift ?? false) === (binding.shift ?? false),
-  );
-}
-
-/**
- * Convert a KeyboardEvent to a ShortcutBinding (for the capture UI).
- */
-export function eventToBinding(
-  event: KeyboardEvent,
-): ShortcutBinding | undefined {
-  // Ignore standalone modifier keys
-  if (["Alt", "Shift", "Control", "Meta"].includes(event.key)) return undefined;
-
-  const binding: ShortcutBinding = { key: event.key };
-  if (event.altKey) binding.alt = true;
-  if (event.metaKey) binding.meta = true;
-  if (event.shiftKey) binding.shift = true;
-
-  return binding;
-}
-
-/**
- * Check if two bindings are equal.
- */
-export function bindingsEqual(a: ShortcutBinding, b: ShortcutBinding): boolean {
-  return (
-    a.key.toLowerCase() === b.key.toLowerCase() &&
-    (a.alt ?? false) === (b.alt ?? false) &&
-    (a.meta ?? false) === (b.meta ?? false) &&
-    (a.shift ?? false) === (b.shift ?? false)
-  );
-}

package/src/lib/utils.test.ts

@@ -1,110 +0,0 @@
-import { createElement } from "react";
-import { describe, expect, it } from "vitest";
-import { getTextContent, slugify } from "./utils";
-
-describe("slugify", () => {
-  it("converts text to lowercase", () => {
-    expect(slugify("Hello World")).toBe("hello-world");
-  });
-
-  it("replaces spaces with hyphens", () => {
-    expect(slugify("hello world")).toBe("hello-world");
-  });
-
-  it("removes special characters", () => {
-    expect(slugify("Hello, World!")).toBe("hello-world");
-  });
-
-  it("handles multiple spaces", () => {
-    expect(slugify("hello   world")).toBe("hello-world");
-  });
-
-  it("handles leading/trailing whitespace", () => {
-    expect(slugify("  hello world  ")).toBe("hello-world");
-  });
-
-  it("preserves underscores", () => {
-    expect(slugify("hello_world")).toBe("hello_world");
-  });
-
-  it("handles hyphens", () => {
-    expect(slugify("hello-world")).toBe("hello-world");
-  });
-
-  it("collapses multiple hyphens", () => {
-    expect(slugify("hello---world")).toBe("hello-world");
-  });
-});
-
-describe("getTextContent", () => {
-  it("returns string children directly", () => {
-    expect(getTextContent("Hello")).toBe("Hello");
-  });
-
-  it("converts number children to string", () => {
-    expect(getTextContent(123)).toBe("123");
-  });
-
-  it("joins array of strings", () => {
-    expect(getTextContent(["Hello", " ", "World"])).toBe("Hello World");
-  });
-
-  it("extracts text from React element", () => {
-    const element = createElement("strong", null, "Bold");
-    expect(getTextContent(element)).toBe("Bold");
-  });
-
-  it("extracts text from nested React elements", () => {
-    const element = createElement(
-      "span",
-      null,
-      "Hello ",
-      createElement("strong", null, "World"),
-    );
-    expect(getTextContent(element)).toBe("Hello World");
-  });
-
-  it("handles mixed array of strings and elements", () => {
-    const children = ["Hello ", createElement("strong", null, "World"), "!"];
-    expect(getTextContent(children)).toBe("Hello World!");
-  });
-
-  it("handles deeply nested elements", () => {
-    const element = createElement(
-      "div",
-      null,
-      createElement("span", null, createElement("strong", null, "Deep")),
-    );
-    expect(getTextContent(element)).toBe("Deep");
-  });
-
-  it("returns empty string for null", () => {
-    expect(getTextContent(null)).toBe("");
-  });
-
-  it("returns empty string for undefined", () => {
-    expect(getTextContent(undefined)).toBe("");
-  });
-
-  it("returns empty string for boolean", () => {
-    expect(getTextContent(true)).toBe("");
-    expect(getTextContent(false)).toBe("");
-  });
-
-  it("handles markdown-style inline formatting", () => {
-    // Simulates what react-markdown produces for "## Hello **World**"
-    const children = ["Hello ", createElement("strong", null, "World")];
-    expect(getTextContent(children)).toBe("Hello World");
-  });
-
-  it("handles complex markdown with multiple formatting", () => {
-    // Simulates "## Hello **World** and _italic_"
-    const children = [
-      "Hello ",
-      createElement("strong", null, "World"),
-      " and ",
-      createElement("em", null, "italic"),
-    ];
-    expect(getTextContent(children)).toBe("Hello World and italic");
-  });
-});