@peaske7/readit 0.1.8 → 0.2.1

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/store/index.ts

@@ -1,254 +0,0 @@
-import { createStore, useStore } from "zustand";
-import type { Comment, Document, Selection } from "../types";
-
-// ─── Types ───────────────────────────────────────────────────────────
-
-export interface DocumentState {
-  document: Document;
-  comments: Comment[];
-  commentsError: string | null;
-  sortedComments: Comment[];
-  selection: Selection | null;
-  pendingSelectionTop: number | undefined;
-  pendingCommentText: string;
-  highlightPositions: Record<string, number>;
-  documentPositions: Record<string, number>;
-  scrollY: number;
-  hoveredCommentId: string | undefined;
-  reanchorTarget: { commentId: string } | null;
-}
-
-export interface AppStore {
-  // Multi-document state
-  documents: Map<string, DocumentState>;
-  activeDocumentPath: string | null;
-  documentOrder: string[];
-  workingDirectory: string | null;
-
-  // Global actions
-  setWorkingDirectory: (dir: string) => void;
-  openDocument: (doc: Document, opts?: { active?: boolean }) => void;
-  closeDocument: (filePath: string) => void;
-  setActiveDocument: (filePath: string) => void;
-
-  // Per-document setters (default to active doc)
-  setComments: (comments: Comment[], filePath?: string) => void;
-  setCommentsError: (error: string | null, filePath?: string) => void;
-  setSelection: (selection: Selection | null, filePath?: string) => void;
-  setPendingSelectionTop: (top: number | undefined, filePath?: string) => void;
-  setHighlightPositions: (
-    positions: Record<string, number>,
-    filePath?: string,
-  ) => void;
-  setDocumentPositions: (
-    positions: Record<string, number>,
-    filePath?: string,
-  ) => void;
-  setScrollY: (y: number, filePath?: string) => void;
-  setHoveredCommentId: (id: string | undefined, filePath?: string) => void;
-  setReanchorTarget: (
-    target: { commentId: string } | null,
-    filePath?: string,
-  ) => void;
-  setPendingCommentText: (text: string, filePath?: string) => void;
-  updateDocumentContent: (content: string, filePath?: string) => void;
-
-  // Helpers
-  getActiveDocumentState: () => DocumentState | undefined;
-}
-
-// ─── Helpers ─────────────────────────────────────────────────────────
-
-function createInitialDocumentState(doc: Document): DocumentState {
-  return {
-    document: doc,
-    comments: [],
-    commentsError: null,
-    sortedComments: [],
-    selection: null,
-    pendingSelectionTop: undefined,
-    pendingCommentText: "",
-    highlightPositions: {},
-    documentPositions: {},
-    scrollY: 0,
-    hoveredCommentId: undefined,
-    reanchorTarget: null,
-  };
-}
-
-function sortComments(comments: Comment[]): Comment[] {
-  return [...comments].sort((a, b) => a.startOffset - b.startOffset);
-}
-
-// ─── Store Factory ───────────────────────────────────────────────────
-
-export function createAppStore() {
-  return createStore<AppStore>((set, get) => {
-    const resolveFilePath = (filePath?: string): string | null =>
-      filePath ?? get().activeDocumentPath;
-
-    const updateDocState = (
-      filePath: string,
-      updater: (state: DocumentState) => Partial<DocumentState>,
-    ) => {
-      set((prev) => {
-        const docState = prev.documents.get(filePath);
-        if (!docState) return prev;
-        const updates = updater(docState);
-        const newDocs = new Map(prev.documents);
-        newDocs.set(filePath, { ...docState, ...updates });
-        return { documents: newDocs };
-      });
-    };
-
-    return {
-      documents: new Map(),
-      activeDocumentPath: null,
-      documentOrder: [],
-      workingDirectory: null,
-
-      setWorkingDirectory: (dir) => set({ workingDirectory: dir }),
-
-      openDocument: (doc, opts) => {
-        set((prev) => {
-          const active = opts?.active ?? true;
-          const nextActive =
-            active || !prev.activeDocumentPath
-              ? doc.filePath
-              : prev.activeDocumentPath;
-
-          if (prev.documents.has(doc.filePath)) {
-            const newDocs = new Map(prev.documents);
-            const prevDoc = newDocs.get(doc.filePath)!;
-            newDocs.set(doc.filePath, {
-              ...prevDoc,
-              document: { ...prevDoc.document, ...doc },
-            });
-            return {
-              documents: newDocs,
-              activeDocumentPath: nextActive,
-            };
-          }
-          const newDocs = new Map(prev.documents);
-          newDocs.set(doc.filePath, createInitialDocumentState(doc));
-          return {
-            documents: newDocs,
-            activeDocumentPath: nextActive,
-            documentOrder: [...prev.documentOrder, doc.filePath],
-          };
-        });
-      },
-
-      closeDocument: (filePath) => {
-        set((prev) => {
-          const newDocs = new Map(prev.documents);
-          newDocs.delete(filePath);
-          const newOrder = prev.documentOrder.filter((p) => p !== filePath);
-
-          let newActive = prev.activeDocumentPath;
-          if (prev.activeDocumentPath === filePath) {
-            const oldIndex = prev.documentOrder.indexOf(filePath);
-            newActive = newOrder[oldIndex] ?? newOrder[oldIndex - 1] ?? null;
-          }
-
-          return {
-            documents: newDocs,
-            activeDocumentPath: newActive,
-            documentOrder: newOrder,
-          };
-        });
-      },
-
-      setActiveDocument: (filePath) => {
-        if (get().documents.has(filePath)) {
-          set({ activeDocumentPath: filePath });
-        }
-      },
-
-      setComments: (comments, filePath?) => {
-        const path = resolveFilePath(filePath);
-        if (!path) return;
-        updateDocState(path, () => ({
-          comments,
-          sortedComments: sortComments(comments),
-        }));
-      },
-
-      setCommentsError: (error, filePath?) => {
-        const path = resolveFilePath(filePath);
-        if (!path) return;
-        updateDocState(path, () => ({ commentsError: error }));
-      },
-
-      setSelection: (selection, filePath?) => {
-        const path = resolveFilePath(filePath);
-        if (!path) return;
-        updateDocState(path, () => ({ selection }));
-      },
-
-      setPendingSelectionTop: (top, filePath?) => {
-        const path = resolveFilePath(filePath);
-        if (!path) return;
-        updateDocState(path, () => ({ pendingSelectionTop: top }));
-      },
-
-      setHighlightPositions: (positions, filePath?) => {
-        const path = resolveFilePath(filePath);
-        if (!path) return;
-        updateDocState(path, () => ({ highlightPositions: positions }));
-      },
-
-      setDocumentPositions: (positions, filePath?) => {
-        const path = resolveFilePath(filePath);
-        if (!path) return;
-        updateDocState(path, () => ({ documentPositions: positions }));
-      },
-
-      setScrollY: (y, filePath?) => {
-        const path = resolveFilePath(filePath);
-        if (!path) return;
-        updateDocState(path, () => ({ scrollY: y }));
-      },
-
-      setHoveredCommentId: (id, filePath?) => {
-        const path = resolveFilePath(filePath);
-        if (!path) return;
-        updateDocState(path, () => ({ hoveredCommentId: id }));
-      },
-
-      setReanchorTarget: (target, filePath?) => {
-        const path = resolveFilePath(filePath);
-        if (!path) return;
-        updateDocState(path, () => ({ reanchorTarget: target }));
-      },
-
-      setPendingCommentText: (text, filePath?) => {
-        const path = resolveFilePath(filePath);
-        if (!path) return;
-        updateDocState(path, () => ({ pendingCommentText: text }));
-      },
-
-      updateDocumentContent: (content, filePath?) => {
-        const path = resolveFilePath(filePath);
-        if (!path) return;
-        updateDocState(path, (s) => ({
-          document: { ...s.document, content },
-        }));
-      },
-
-      getActiveDocumentState: () => {
-        const { documents, activeDocumentPath } = get();
-        if (!activeDocumentPath) return undefined;
-        return documents.get(activeDocumentPath);
-      },
-    };
-  });
-}
-
-// ─── Singleton + React Hook ─────────────────────────────────────────
-
-export const appStore = createAppStore();
-
-export function useAppStore<T>(selector: (state: AppStore) => T): T {
-  return useStore(appStore, selector);
-}

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