@peaske7/readit 0.1.8 → 0.2.1

package/src/lib/context.test.ts

@@ -1,224 +0,0 @@
-import { describe, expect, it } from "vitest";
-import { extractContext, formatForLLM, stripHtmlTags } from "./context";
-
-describe("stripHtmlTags", () => {
-  it("removes simple HTML tags", () => {
-    expect(stripHtmlTags("<p>Hello</p>")).toBe("Hello");
-    expect(stripHtmlTags("<div><span>Nested</span></div>")).toBe("Nested");
-  });
-
-  it("removes script and style content entirely", () => {
-    expect(stripHtmlTags('<script>alert("xss")</script>text')).toBe("text");
-    expect(stripHtmlTags("<style>.foo { color: red }</style>text")).toBe(
-      "text",
-    );
-  });
-
-  it("decodes common named entities", () => {
-    expect(stripHtmlTags("&lt;tag&gt;")).toBe("<tag>");
-    expect(stripHtmlTags("&amp;&nbsp;&quot;&#39;")).toBe("& \"'");
-  });
-
-  it("decodes numeric entities (decimal)", () => {
-    expect(stripHtmlTags("&#65;&#66;&#67;")).toBe("ABC");
-  });
-
-  it("decodes numeric entities (hex)", () => {
-    expect(stripHtmlTags("&#x41;&#x42;&#x43;")).toBe("ABC");
-    expect(stripHtmlTags("&#X41;&#X42;&#X43;")).toBe("ABC"); // case-insensitive
-  });
-
-  it("handles mixed content", () => {
-    const html = "<p>Hello &amp; <strong>World</strong>!</p>";
-    expect(stripHtmlTags(html)).toBe("Hello & World!");
-  });
-
-  it("preserves plain text", () => {
-    expect(stripHtmlTags("plain text")).toBe("plain text");
-  });
-});
-
-describe("extractContext", () => {
-  it("extracts single-line selection with markers", () => {
-    const content = "line1\nline2\nline3\nline4\nline5";
-    // "line2" starts at offset 6, ends at 11
-    const result = extractContext({ content, startOffset: 6, endOffset: 11 });
-
-    expect(result.startLine).toBe(2);
-    expect(result.endLine).toBe(2);
-    // Should have >>> and <<< markers around "line2"
-    expect(result.lines.some((l) => l.includes(">>> line2 <<<"))).toBe(true);
-  });
-
-  it("includes context lines before and after", () => {
-    const content = "line1\nline2\nline3\nline4\nline5";
-    // Select "line3" at offset 12
-    const result = extractContext({
-      content,
-      startOffset: 12,
-      endOffset: 17,
-      contextLines: 2,
-    });
-
-    expect(result.startLine).toBe(3);
-    expect(result.endLine).toBe(3);
-    // Should include 2 lines before (line1, line2) and 2 after (line4, line5)
-    expect(result.lines.length).toBe(5);
-  });
-
-  it("handles multi-line selection", () => {
-    const content = "line1\nline2\nline3\nline4\nline5";
-    // Select from "line2" to "line3" (offset 6 to 17)
-    const result = extractContext({ content, startOffset: 6, endOffset: 17 });
-
-    expect(result.startLine).toBe(2);
-    expect(result.endLine).toBe(3);
-    // Start line should have >>> marker
-    expect(result.lines.some((l) => l.includes(">>>"))).toBe(true);
-    // End line should have <<< marker
-    expect(result.lines.some((l) => l.includes("<<<"))).toBe(true);
-  });
-
-  it("handles selection at start of document", () => {
-    const content = "line1\nline2\nline3";
-    // Select "line1" at start
-    const result = extractContext({ content, startOffset: 0, endOffset: 5 });
-
-    expect(result.startLine).toBe(1);
-    expect(result.endLine).toBe(1);
-    expect(result.lines[0]).toContain(">>> line1 <<<");
-  });
-
-  it("handles selection at end of document", () => {
-    const content = "line1\nline2\nline3";
-    // Select "line3" (offset 12 to 17)
-    const result = extractContext({ content, startOffset: 12, endOffset: 17 });
-
-    expect(result.startLine).toBe(3);
-    expect(result.endLine).toBe(3);
-    expect(result.lines.some((l) => l.includes(">>> line3 <<<"))).toBe(true);
-  });
-
-  it("truncates very long lines", () => {
-    const longLine = "a".repeat(250);
-    const content = `short\n${longLine}\nshort`;
-    // Select part of the long line
-    const result = extractContext({ content, startOffset: 6, endOffset: 256 });
-
-    // Long line should be truncated with ...
-    const longLineOutput = result.lines.find((l) => l.length > 100);
-    expect(longLineOutput?.endsWith("...")).toBe(true);
-  });
-
-  it("handles HTML content by stripping tags", () => {
-    const html = "<p>paragraph</p>\n<div>div content</div>";
-    // After stripping: "paragraph\ndiv content"
-    // Select "paragraph" at offset 0-9
-    const result = extractContext({
-      content: html,
-      startOffset: 0,
-      endOffset: 9,
-    });
-
-    expect(result.lines.some((l) => l.includes(">>> paragraph <<<"))).toBe(
-      true,
-    );
-  });
-
-  it("normalizes CRLF to LF", () => {
-    const content = "line1\r\nline2\r\nline3";
-    // After normalization: "line1\nline2\nline3"
-    // Select "line2" at offset 6
-    const result = extractContext({ content, startOffset: 6, endOffset: 11 });
-
-    expect(result.startLine).toBe(2);
-    expect(result.lines.some((l) => l.includes(">>> line2 <<<"))).toBe(true);
-  });
-
-  it("limits context lines to document bounds", () => {
-    const content = "only\ntwo\nlines";
-    // Select middle line with 5 context lines requested
-    const result = extractContext({
-      content,
-      startOffset: 5,
-      endOffset: 8,
-      contextLines: 5,
-    });
-
-    // Should not go beyond document bounds
-    expect(result.lines.length).toBeLessThanOrEqual(3);
-  });
-
-  it("truncates very long selections with ellipsis", () => {
-    // Create content with more than MAX_SELECTION_LINES (10) lines
-    const lines = Array.from({ length: 15 }, (_, i) => `line${i + 1}`);
-    const content = lines.join("\n");
-    // Select all content
-    const result = extractContext({
-      content,
-      startOffset: 0,
-      endOffset: content.length,
-      contextLines: 0,
-    });
-
-    // Should include ... for truncated middle
-    expect(result.lines.some((l) => l === "...")).toBe(true);
-  });
-});
-
-describe("formatForLLM", () => {
-  it("formats context with header and line range", () => {
-    const context = {
-      lines: ["before", ">>> selected <<<", "after"],
-      startLine: 5,
-      endLine: 5,
-    };
-
-    const result = formatForLLM({ context, fileName: "test.md" });
-
-    expect(result).toContain("# From: test.md");
-    expect(result).toContain("Lines 5-5:");
-    expect(result).toContain("---");
-    expect(result).toContain(">>> selected <<<");
-  });
-
-  it("includes optional comment", () => {
-    const context = {
-      lines: ["text"],
-      startLine: 1,
-      endLine: 1,
-    };
-
-    const result = formatForLLM({
-      context,
-      fileName: "test.md",
-      comment: "This needs review",
-    });
-
-    expect(result).toContain("Comment: This needs review");
-  });
-
-  it("omits comment section when not provided", () => {
-    const context = {
-      lines: ["text"],
-      startLine: 1,
-      endLine: 1,
-    };
-
-    const result = formatForLLM({ context, fileName: "test.md" });
-
-    expect(result).not.toContain("Comment:");
-  });
-
-  it("formats multi-line range correctly", () => {
-    const context = {
-      lines: [">>> start", "middle", "end <<<"],
-      startLine: 10,
-      endLine: 12,
-    };
-
-    const result = formatForLLM({ context, fileName: "doc.html" });
-
-    expect(result).toContain("Lines 10-12:");
-  });
-});

package/src/lib/context.ts

@@ -1,193 +0,0 @@
-/**
- * Context extraction and formatting utilities for LLM clipboard copy.
- */
-
-interface ContextResult {
-  lines: string[]; // Lines with >>> markers inserted
-  startLine: number; // 1-based line number
-  endLine: number; // 1-based line number
-}
-
-interface FormatOptions {
-  context: ContextResult;
-  fileName: string;
-  comment?: string;
-}
-
-/**
- * Parameters for context extraction.
- * Using object destructuring per style guide §3.5 for clarity.
- */
-export interface ExtractContextParams {
-  content: string;
-  startOffset: number;
-  endOffset: number;
-  contextLines?: number;
-}
-
-const DEFAULT_CONTEXT_LINES = 2;
-const MAX_SELECTION_LINES = 10;
-const MAX_LINE_LENGTH = 200;
-
-/**
- * Strip HTML tags to get plain text matching TreeWalker offset calculation.
- */
-export function stripHtmlTags(html: string): string {
-  // Remove script/style content entirely
-  let text = html.replace(/<(script|style)[^>]*>[\s\S]*?<\/\1>/gi, "");
-  // Remove all HTML tags
-  text = text.replace(/<[^>]+>/g, "");
-  // Decode common named entities
-  text = text.replace(/&nbsp;/g, " ");
-  text = text.replace(/&lt;/g, "<");
-  text = text.replace(/&gt;/g, ">");
-  text = text.replace(/&amp;/g, "&");
-  text = text.replace(/&quot;/g, '"');
-  text = text.replace(/&#39;/g, "'");
-  // Decode numeric entities (decimal and hex)
-  text = text.replace(/&#(\d+);/g, (_, code) =>
-    String.fromCharCode(Number.parseInt(code, 10)),
-  );
-  text = text.replace(/&#x([0-9a-f]+);/gi, (_, code) =>
-    String.fromCharCode(Number.parseInt(code, 16)),
-  );
-  return text;
-}
-
-/**
- * Detect if content is HTML based on presence of HTML tags.
- */
-function isHtml(content: string): boolean {
-  return /<[a-z][\s\S]*>/i.test(content);
-}
-
-/**
- * Truncate a line if it exceeds max length.
- */
-function truncateLine(
-  line: string,
-  maxLength: number = MAX_LINE_LENGTH,
-): string {
-  if (line.length <= maxLength) return line;
-  return `${line.slice(0, maxLength - 3)}...`;
-}
-
-/**
- * Extract context around a selection using character offsets.
- * Returns lines with >>> and <<< markers inserted at selection boundaries.
- */
-export function extractContext({
-  content,
-  startOffset,
-  endOffset,
-  contextLines = DEFAULT_CONTEXT_LINES,
-}: ExtractContextParams): ContextResult {
-  // For HTML, strip tags to match offset calculation
-  const textContent = isHtml(content) ? stripHtmlTags(content) : content;
-  // Normalize CRLF to LF for consistent offset calculation
-  const normalizedContent = textContent.replace(/\r\n/g, "\n");
-
-  const lines = normalizedContent.split("\n");
-  let currentOffset = 0;
-  let startLineIndex = -1;
-  let endLineIndex = -1;
-  let startCharInLine = 0;
-  let endCharInLine = 0;
-
-  // Find lines containing the selection
-  for (let i = 0; i < lines.length; i++) {
-    const lineLength = lines[i].length;
-    const lineEnd = currentOffset + lineLength;
-
-    if (startLineIndex === -1 && lineEnd >= startOffset) {
-      startLineIndex = i;
-      startCharInLine = startOffset - currentOffset;
-    }
-
-    if (lineEnd >= endOffset) {
-      endLineIndex = i;
-      endCharInLine = endOffset - currentOffset;
-      break;
-    }
-
-    currentOffset += lineLength + 1; // +1 for newline
-  }
-
-  // Handle edge case: couldn't find lines
-  if (startLineIndex === -1) startLineIndex = 0;
-  if (endLineIndex === -1) endLineIndex = lines.length - 1;
-
-  // Calculate context range
-  const contextStart = Math.max(0, startLineIndex - contextLines);
-  const contextEnd = Math.min(lines.length - 1, endLineIndex + contextLines);
-
-  // Build output lines with markers
-  const outputLines: string[] = [];
-  const selectionSpan = endLineIndex - startLineIndex + 1;
-  const shouldTruncateMiddle = selectionSpan > MAX_SELECTION_LINES;
-
-  for (let i = contextStart; i <= contextEnd; i++) {
-    let line = lines[i];
-
-    // Handle truncation for very long selections
-    if (shouldTruncateMiddle) {
-      const showStart = startLineIndex + 2;
-      const showEnd = endLineIndex - 2;
-
-      if (i > showStart && i < showEnd) {
-        if (i === showStart + 1) {
-          outputLines.push("...");
-        }
-        continue;
-      }
-    }
-
-    // Insert markers for selection boundaries
-    if (i === startLineIndex && i === endLineIndex) {
-      // Single line selection
-      const before = line.slice(0, startCharInLine);
-      const selected = line.slice(startCharInLine, endCharInLine);
-      const after = line.slice(endCharInLine);
-      line = `${before}>>> ${selected} <<<${after}`;
-    } else if (i === startLineIndex) {
-      // Start of multi-line selection
-      const before = line.slice(0, startCharInLine);
-      const selected = line.slice(startCharInLine);
-      line = `${before}>>> ${selected}`;
-    } else if (i === endLineIndex) {
-      // End of multi-line selection
-      const selected = line.slice(0, endCharInLine);
-      const after = line.slice(endCharInLine);
-      line = `${selected} <<<${after}`;
-    }
-
-    outputLines.push(truncateLine(line));
-  }
-
-  return {
-    lines: outputLines,
-    startLine: startLineIndex + 1, // 1-based
-    endLine: endLineIndex + 1, // 1-based
-  };
-}
-
-/**
- * Format extracted context for LLM clipboard copy.
- */
-export function formatForLLM({
-  context,
-  fileName,
-  comment,
-}: FormatOptions): string {
-  const header = `# From: ${fileName}`;
-  const lineRange = `Lines ${context.startLine}-${context.endLine}:`;
-  const body = ["---", ...context.lines, "---"].join("\n");
-
-  const parts = [header, "", lineRange, body];
-
-  if (comment) {
-    parts.push("", `Comment: ${comment}`);
-  }
-
-  return parts.join("\n");
-}

package/src/lib/editor-links.ts

@@ -1,59 +0,0 @@
-import { type EditorScheme, EditorSchemes } from "../types";
-
-// Known source file extensions — kept broad to cover common languages
-const EXTENSIONS =
-  "ts|tsx|js|jsx|mjs|cjs|json|md|mdx|css|scss|html|htm|py|rs|go|rb|java|kt|yml|yaml|toml|sh|bash|zsh|sql|graphql|gql|vue|svelte|astro|c|cpp|h|hpp|cs|swift|zig|lua|ex|exs|erl|hrl|elm|clj|cljs|ml|mli|fs|fsx|r|jl|dart|tf|hcl|proto|xml|svg";
-
-// Match file paths with known extensions and optional :line[:col] suffix
-// Requires at least one `/` to avoid matching bare filenames like "README.md"
-const FILE_PATH_RE = new RegExp(
-  `^(?:\\.\\.\\/|\\.\\/)?((?:[\\w@.-]+\\/)+[\\w.-]+\\.(?:${EXTENSIONS}))(?::(\\d+)(?::(\\d+))?)?$`,
-);
-
-export interface FilePathMatch {
-  path: string;
-  line?: number;
-  col?: number;
-}
-
-export function parseFilePath(text: string): FilePathMatch | undefined {
-  const match = FILE_PATH_RE.exec(text.trim());
-  if (!match) return undefined;
-
-  return {
-    path: match[1],
-    line: match[2] ? Number.parseInt(match[2], 10) : undefined,
-    col: match[3] ? Number.parseInt(match[3], 10) : undefined,
-  };
-}
-
-export function buildEditorUri(
-  scheme: EditorScheme,
-  absolutePath: string,
-  line?: number,
-  col?: number,
-): string {
-  if (scheme === EditorSchemes.NONE) return "";
-
-  // vscode://file/path:line:col
-  let uri = `${scheme}://file/${absolutePath}`;
-  if (line !== undefined) {
-    uri += `:${line}`;
-    if (col !== undefined) {
-      uri += `:${col}`;
-    }
-  }
-  return uri;
-}
-
-export function resolveAbsolutePath(
-  relativePath: string,
-  workingDirectory: string,
-): string {
-  if (relativePath.startsWith("/")) return relativePath;
-
-  const base = workingDirectory.endsWith("/")
-    ? workingDirectory
-    : `${workingDirectory}/`;
-  return `${base}${relativePath}`;
-}

package/src/lib/highlight/colors.ts

@@ -1,37 +0,0 @@
-/**
- * Color palette for comment highlights.
- * Colors are assigned by document position (top-to-bottom).
- */
-
-export const COMMENT_COLORS = [
-  {
-    name: "amber",
-    bg: "rgba(245, 222, 160, 0.5)",
-    bgFocused: "rgba(228, 195, 110, 0.65)",
-    border: "#c9a84a",
-    text: "#8b6914",
-  },
-  {
-    name: "blue",
-    bg: "rgba(168, 196, 228, 0.5)",
-    bgFocused: "rgba(130, 168, 210, 0.65)",
-    border: "#5b7fa8",
-    text: "#3d5f8a",
-  },
-  {
-    name: "green",
-    bg: "rgba(170, 210, 170, 0.5)",
-    bgFocused: "rgba(130, 185, 135, 0.65)",
-    border: "#5a9a62",
-    text: "#3d6e45",
-  },
-  {
-    name: "rose",
-    bg: "rgba(225, 180, 185, 0.5)",
-    bgFocused: "rgba(205, 145, 155, 0.65)",
-    border: "#b86b78",
-    text: "#8a4a55",
-  },
-] as const;
-
-export type CommentColor = (typeof COMMENT_COLORS)[number];

package/src/lib/highlight/index.ts

@@ -1,23 +0,0 @@
-// Highlighter (unified adapter)
-
-export type { CommentColor } from "./colors";
-// Colors
-export { COMMENT_COLORS } from "./colors";
-export type {
-  Highlighter,
-  HighlighterOptions,
-  HoverHandler,
-  PositionChangeHandler,
-  SelectionHandler,
-} from "./highlighter";
-export { createHighlighter } from "./highlighter";
-
-// Script builder (needed by IframeContainer)
-export { buildIframeScript } from "./script-builder";
-
-// Types (public API)
-export type {
-  HighlightComment,
-  HighlightPositions,
-  HighlightStyle,
-} from "./types";