@peaske7/readit 0.1.8 → 0.2.0

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");
-  });
-});

package/src/store/index.test.ts

@@ -1,242 +0,0 @@
-import { beforeEach, describe, expect, it } from "vitest";
-import type { Comment, Document } from "../types";
-import { createAppStore } from "./index";
-
-const mockDoc: Document = {
-  content: "# Hello",
-  type: "markdown",
-  filePath: "/test/file.md",
-  fileName: "file.md",
-  clean: false,
-};
-
-const mockDoc2: Document = {
-  content: "<h1>Hello</h1>",
-  type: "html",
-  filePath: "/test/file.html",
-  fileName: "file.html",
-  clean: false,
-};
-
-const mockComment = {
-  id: "c1",
-  selectedText: "hello",
-  comment: "note",
-  startOffset: 0,
-  endOffset: 5,
-  createdAt: "2026-03-13T00:00:00Z",
-} as Comment;
-
-describe("AppStore", () => {
-  let store: ReturnType<typeof createAppStore>;
-
-  beforeEach(() => {
-    store = createAppStore();
-  });
-
-  describe("openDocument", () => {
-    it("adds document to store and sets as active", () => {
-      store.getState().openDocument(mockDoc);
-      const state = store.getState();
-      expect(state.documents.has("/test/file.md")).toBe(true);
-      expect(state.activeDocumentPath).toBe("/test/file.md");
-      expect(state.documentOrder).toEqual(["/test/file.md"]);
-    });
-
-    it("does not overwrite existing document state", () => {
-      store.getState().openDocument(mockDoc);
-      store.getState().setComments([mockComment]);
-      store.getState().openDocument(mockDoc);
-      expect(
-        store.getState().documents.get("/test/file.md")!.comments,
-      ).toHaveLength(1);
-    });
-
-    it("activates existing document without overwriting", () => {
-      store.getState().openDocument(mockDoc);
-      store.getState().openDocument(mockDoc2);
-      expect(store.getState().activeDocumentPath).toBe("/test/file.html");
-      store.getState().openDocument(mockDoc);
-      expect(store.getState().activeDocumentPath).toBe("/test/file.md");
-      // Order unchanged
-      expect(store.getState().documentOrder).toEqual([
-        "/test/file.md",
-        "/test/file.html",
-      ]);
-    });
-
-    it("adds document without activating when active is false", () => {
-      store.getState().openDocument(mockDoc);
-      store.getState().openDocument(mockDoc2, { active: false });
-      expect(store.getState().activeDocumentPath).toBe("/test/file.md");
-      expect(store.getState().documentOrder).toEqual([
-        "/test/file.md",
-        "/test/file.html",
-      ]);
-    });
-
-    it("updates existing document without stealing focus when active is false", () => {
-      store.getState().openDocument(mockDoc);
-      store.getState().openDocument(mockDoc2);
-      store
-        .getState()
-        .openDocument({ ...mockDoc, content: "# Updated" }, { active: false });
-      expect(store.getState().activeDocumentPath).toBe("/test/file.html");
-      expect(
-        store.getState().documents.get("/test/file.md")!.document.content,
-      ).toBe("# Updated");
-    });
-  });
-
-  describe("closeDocument", () => {
-    it("removes document and activates right neighbor", () => {
-      store.getState().openDocument(mockDoc);
-      store.getState().openDocument(mockDoc2);
-      store.getState().setActiveDocument("/test/file.md");
-      store.getState().closeDocument("/test/file.md");
-      expect(store.getState().documents.has("/test/file.md")).toBe(false);
-      expect(store.getState().activeDocumentPath).toBe("/test/file.html");
-    });
-
-    it("activates left neighbor when closing last in order", () => {
-      store.getState().openDocument(mockDoc);
-      store.getState().openDocument(mockDoc2);
-      // mockDoc2 is active (last opened)
-      store.getState().closeDocument("/test/file.html");
-      expect(store.getState().activeDocumentPath).toBe("/test/file.md");
-    });
-
-    it("sets null when closing last document", () => {
-      store.getState().openDocument(mockDoc);
-      store.getState().closeDocument("/test/file.md");
-      expect(store.getState().activeDocumentPath).toBeNull();
-      expect(store.getState().documentOrder).toEqual([]);
-    });
-
-    it("does not change active if closing non-active tab", () => {
-      store.getState().openDocument(mockDoc);
-      store.getState().openDocument(mockDoc2);
-      // mockDoc2 is active
-      store.getState().closeDocument("/test/file.md");
-      expect(store.getState().activeDocumentPath).toBe("/test/file.html");
-    });
-
-    it("activates right neighbor when closing middle tab", () => {
-      const mockDoc3: Document = {
-        content: "# Third",
-        type: "markdown",
-        filePath: "/test/third.md",
-        fileName: "third.md",
-        clean: false,
-      };
-      store.getState().openDocument(mockDoc);
-      store.getState().openDocument(mockDoc2);
-      store.getState().openDocument(mockDoc3);
-      store.getState().setActiveDocument("/test/file.html");
-      store.getState().closeDocument("/test/file.html");
-      expect(store.getState().activeDocumentPath).toBe("/test/third.md");
-      expect(store.getState().documentOrder).toEqual([
-        "/test/file.md",
-        "/test/third.md",
-      ]);
-    });
-  });
-
-  describe("setActiveDocument", () => {
-    it("sets active document", () => {
-      store.getState().openDocument(mockDoc);
-      store.getState().openDocument(mockDoc2);
-      store.getState().setActiveDocument("/test/file.md");
-      expect(store.getState().activeDocumentPath).toBe("/test/file.md");
-    });
-
-    it("ignores unknown file paths", () => {
-      store.getState().openDocument(mockDoc);
-      store.getState().setActiveDocument("/nonexistent.md");
-      expect(store.getState().activeDocumentPath).toBe("/test/file.md");
-    });
-  });
-
-  describe("per-document setters default to active doc", () => {
-    it("setComments operates on active document", () => {
-      store.getState().openDocument(mockDoc);
-      store.getState().setComments([mockComment]);
-      const docState = store.getState().documents.get("/test/file.md")!;
-      expect(docState.comments).toEqual([mockComment]);
-      expect(docState.sortedComments).toEqual([mockComment]);
-    });
-
-    it("setComments with explicit filePath targets that document", () => {
-      store.getState().openDocument(mockDoc);
-      store.getState().openDocument(mockDoc2);
-      store.getState().setComments([mockComment], "/test/file.html");
-      expect(
-        store.getState().documents.get("/test/file.html")!.comments,
-      ).toEqual([mockComment]);
-      expect(store.getState().documents.get("/test/file.md")!.comments).toEqual(
-        [],
-      );
-    });
-
-    it("sortedComments sorts by startOffset", () => {
-      store.getState().openDocument(mockDoc);
-      const c1 = { ...mockComment, id: "c1", startOffset: 10 };
-      const c2 = { ...mockComment, id: "c2", startOffset: 2 };
-      const c3 = { ...mockComment, id: "c3", startOffset: 5 };
-      store.getState().setComments([c1, c2, c3]);
-      const docState = store.getState().documents.get("/test/file.md")!;
-      expect(docState.sortedComments.map((c) => c.id)).toEqual([
-        "c2",
-        "c3",
-        "c1",
-      ]);
-    });
-
-    it("setSelection operates on active document", () => {
-      store.getState().openDocument(mockDoc);
-      const sel = { text: "hello", startOffset: 0, endOffset: 5 };
-      store.getState().setSelection(sel);
-      expect(
-        store.getState().documents.get("/test/file.md")!.selection,
-      ).toEqual(sel);
-    });
-
-    it("setScrollY operates on active document", () => {
-      store.getState().openDocument(mockDoc);
-      store.getState().setScrollY(150);
-      expect(store.getState().documents.get("/test/file.md")!.scrollY).toBe(
-        150,
-      );
-    });
-
-    it("setters no-op when no active document", () => {
-      // No document opened
-      store.getState().setComments([mockComment]);
-      // Should not throw, just no-op
-      expect(store.getState().documents.size).toBe(0);
-    });
-  });
-
-  describe("getActiveDocumentState", () => {
-    it("returns undefined when no active document", () => {
-      expect(store.getState().getActiveDocumentState()).toBeUndefined();
-    });
-
-    it("returns active document state", () => {
-      store.getState().openDocument(mockDoc);
-      const state = store.getState().getActiveDocumentState();
-      expect(state).toBeDefined();
-      expect(state!.document.filePath).toBe("/test/file.md");
-    });
-  });
-
-  describe("updateDocumentContent", () => {
-    it("updates document content in place", () => {
-      store.getState().openDocument(mockDoc);
-      store.getState().updateDocumentContent("# Updated");
-      expect(
-        store.getState().documents.get("/test/file.md")!.document.content,
-      ).toBe("# Updated");
-    });
-  });
-});

package/src/types/index.ts

@@ -1,127 +0,0 @@
-// Anchor confidence levels - const object pattern per style guide 6.3
-export const AnchorConfidences = {
-  EXACT: "exact",
-  NORMALIZED: "normalized",
-  FUZZY: "fuzzy",
-  UNRESOLVED: "unresolved",
-} as const;
-
-export type AnchorConfidence =
-  (typeof AnchorConfidences)[keyof typeof AnchorConfidences];
-
-// Subset excluding "unresolved" for resolved anchors
-export type ResolvedAnchorConfidence = Exclude<
-  AnchorConfidence,
-  typeof AnchorConfidences.UNRESOLVED
->;
-
-export interface Comment {
-  id: string;
-  selectedText: string;
-  comment: string;
-  createdAt: string;
-  // Position info for highlighting
-  startOffset: number;
-  endOffset: number;
-  // Line hint for text-based anchoring (e.g., "L42" or "L42-45")
-  lineHint?: string;
-  // Confidence level of anchor resolution
-  anchorConfidence?: AnchorConfidence;
-  // First N chars of original text for anchor matching when selectedText is truncated
-  anchorPrefix?: string;
-}
-
-// Parsed comment file structure
-export interface CommentFile {
-  source: string; // Absolute path to source file
-  hash: string; // SHA-256 prefix (16 chars) of source content
-  version: number; // Format version
-  comments: Comment[];
-}
-
-// Anchor match result
-export interface Anchor {
-  start: number;
-  end: number;
-  line: number;
-  confidence: ResolvedAnchorConfidence;
-  distance?: number; // Levenshtein distance for fuzzy matches
-}
-
-export interface SelectionRange {
-  startOffset: number;
-  endOffset: number;
-}
-
-export interface Selection extends SelectionRange {
-  text: string;
-}
-
-export type DocumentType = "markdown" | "html";
-
-export interface Document {
-  content: string;
-  type: DocumentType;
-  filePath: string;
-  fileName: string;
-  clean: boolean;
-}
-
-// Editor scheme options - const object pattern per style guide 6.3
-export const EditorSchemes = {
-  NONE: "none",
-  VSCODE: "vscode",
-  VSCODE_INSIDERS: "vscode-insiders",
-  CURSOR: "cursor",
-} as const;
-
-export type EditorScheme = (typeof EditorSchemes)[keyof typeof EditorSchemes];
-
-// Font family options - const object pattern per style guide 6.3
-export const FontFamilies = {
-  SERIF: "serif",
-  SANS_SERIF: "sans-serif",
-} as const;
-
-export type FontFamily = (typeof FontFamilies)[keyof typeof FontFamilies];
-
-// Theme mode options - const object pattern per style guide 6.3
-export const ThemeModes = {
-  LIGHT: "light",
-  DARK: "dark",
-  SYSTEM: "system",
-} as const;
-
-export type ThemeMode = (typeof ThemeModes)[keyof typeof ThemeModes];
-
-// Layout mode options - const object pattern per style guide 6.3
-export const LayoutModes = {
-  CENTERED: "centered",
-  FULLSCREEN: "fullscreen",
-} as const;
-
-export type LayoutMode = (typeof LayoutModes)[keyof typeof LayoutModes];
-
-// Keyboard shortcut binding
-export interface ShortcutBinding {
-  key: string; // KeyboardEvent.key value, e.g. "c", "ArrowUp"
-  alt?: boolean;
-  meta?: boolean; // ⌘ on Mac, Ctrl on Windows/Linux
-  shift?: boolean;
-}
-
-// User override for a shortcut
-export interface KeybindingOverride {
-  id: string;
-  binding?: ShortcutBinding; // undefined = use default
-  enabled: boolean;
-}
-
-// Document settings stored per-file
-export interface DocumentSettings {
-  version: number;
-  fontFamily: FontFamily;
-  editorScheme?: EditorScheme;
-  keybindings?: KeybindingOverride[];
-  onboarded?: boolean;
-}