backnotprop
diff --git a/‎AGENTS.md‎
Lines changed: 11 additions & 5 deletions b/‎AGENTS.md‎
Lines changed: 11 additions & 5 deletions
diff --git a/‎apps/hook/server/cli.test.ts‎
Lines changed: 1 addition & 1 deletion b/‎apps/hook/server/cli.test.ts‎
Lines changed: 1 addition & 1 deletion
diff --git a/‎apps/hook/server/cli.ts‎
Lines changed: 2 additions & 2 deletions b/‎apps/hook/server/cli.ts‎
Lines changed: 2 additions & 2 deletions
diff --git a/‎apps/hook/server/index.ts‎
Lines changed: 85 additions & 37 deletions b/‎apps/hook/server/index.ts‎
Lines changed: 85 additions & 37 deletions
diff --git a/‎apps/opencode-plugin/commands.ts‎
Lines changed: 67 additions & 19 deletions b/‎apps/opencode-plugin/commands.ts‎
Lines changed: 67 additions & 19 deletions
diff --git a/‎apps/opencode-plugin/commands/plannotator-annotate.md‎
Lines changed: 1 addition & 1 deletion b/‎apps/opencode-plugin/commands/plannotator-annotate.md‎
Lines changed: 1 addition & 1 deletion
@@ -104,6 +104,8 @@ claude --plugin-dir ./apps/hook
 | `PLANNOTATOR_SHARE` | Set to `disabled` to turn off URL sharing entirely. Default: enabled. |
 | `PLANNOTATOR_SHARE_URL` | Custom base URL for share links (self-hosted portal). Default: `https://share.plannotator.ai`. |
 | `PLANNOTATOR_PASTE_URL` | Base URL of the paste service API for short URL sharing. Default: `https://plannotator-paste.plannotator.workers.dev`. |
+| `PLANNOTATOR_JINA` | Set to `0` / `false` to disable Jina Reader for URL annotation, or `1` / `true` to enable. Default: enabled. Can also be set via `~/.plannotator/config.json` (`{ "jina": false }`) or per-invocation via `--no-jina`. |
+| `JINA_API_KEY` | Optional Jina Reader API key for higher rate limits (500 RPM vs 20 RPM unauthenticated). Free keys include 10M tokens. |
 | `PLANNOTATOR_VERIFY_ATTESTATION` | **Read by the install scripts only**, not by the runtime binary. Set to `1` / `true` to have `scripts/install.sh` / `install.ps1` / `install.cmd` run `gh attestation verify` on every install. Off by default. Can also be set persistently via `~/.plannotator/config.json` (`{ "verifyAttestation": true }`) or per-invocation via `--verify-attestation`. Requires `gh` installed and authenticated. |
 
 **Legacy:** `SSH_TTY` and `SSH_CONNECTION` are still detected when `PLANNOTATOR_REMOTE` is unset. Set `PLANNOTATOR_REMOTE=1` / `true` to force remote mode or `0` / `false` to force local mode.
@@ -152,16 +154,20 @@ Approve → "LGTM" sent to agent session
 ## Annotate Flow
 
 ```
-User runs /plannotator-annotate <file.md> command
+User runs /plannotator-annotate <file.md | file.html | https://... | folder/>
         ↓
 Claude Code: plannotator annotate subcommand runs
-OpenCode: event handler intercepts command
+OpenCode/Pi: event handler intercepts command
         ↓
-Markdown file read from disk
+Input type detected:
+  .md/.mdx   → file read from disk
+  .html/.htm → file read, converted to markdown via Turndown
+  https://   → fetched via Jina Reader (default) or fetch+Turndown (--no-jina)
+  folder/    → file browser opened, files converted on demand
         ↓
 Annotate server starts (reuses plan editor HTML with mode:"annotate")
         ↓
-User annotates markdown, provides feedback
+User annotates content, provides feedback
         ↓
 Send Annotations → feedback sent to agent session
 ```
@@ -247,7 +253,7 @@ During normal plan review, an Archive sidebar tab provides the same browsing via
 
 | Endpoint              | Method | Purpose                                    |
 | --------------------- | ------ | ------------------------------------------ |
-| `/api/plan`           | GET    | Returns `{ plan, origin, mode: "annotate", filePath }` |
+| `/api/plan`           | GET    | Returns `{ plan, origin, mode: "annotate", filePath, sourceInfo? }` |
 | `/api/feedback`       | POST   | Submit annotations (body: feedback, annotations) |
 | `/api/image`          | GET    | Serve image by path query param            |
 | `/api/upload`         | POST   | Upload image, returns `{ path, originalName }` |
 
@@ -19,7 +19,7 @@ describe("CLI top-level help", () => {
     expect(output).toContain("plannotator --help");
     expect(output).toContain("plannotator [--browser <name>]");
     expect(output).toContain("plannotator review [PR_URL]");
-    expect(output).toContain("plannotator annotate <file.md | folder/>");
+    expect(output).toContain("plannotator annotate <file.md | file.html | https://... | folder/>");
     expect(output).toContain("running 'plannotator' without arguments is for hook integration");
   });
 });
 
@@ -15,7 +15,7 @@ export function formatTopLevelHelp(): string {
     "  plannotator --help",
     "  plannotator [--browser <name>]",
     "  plannotator review [PR_URL]",
-    "  plannotator annotate <file.md | folder/>",
+    "  plannotator annotate <file.md | file.html | https://... | folder/>  [--no-jina]",
     "  plannotator last",
     "  plannotator archive",
     "  plannotator sessions",
@@ -33,7 +33,7 @@ export function formatInteractiveNoArgClarification(): string {
     "",
     "For interactive use, try:",
     "  plannotator review",
-    "  plannotator annotate <file.md>",
+    "  plannotator annotate <file.md | file.html | https://...>",
     "  plannotator last",
     "  plannotator archive",
     "  plannotator sessions",
 
@@ -64,17 +64,20 @@ import {
   handleAnnotateServerReady,
 } from "@plannotator/server/annotate";
 import { type DiffType, getVcsContext, runVcsDiff, gitRuntime } from "@plannotator/server/vcs";
-import { loadConfig, resolveDefaultDiffType } from "@plannotator/shared/config";
+import { loadConfig, resolveDefaultDiffType, resolveUseJina } from "@plannotator/shared/config";
+import { htmlToMarkdown } from "@plannotator/shared/html-to-markdown";
+import { urlToMarkdown } from "@plannotator/shared/url-to-markdown";
 import { fetchRef, createWorktree, removeWorktree, ensureObjectAvailable } from "@plannotator/shared/worktree";
 import { parsePRUrl, checkPRAuth, fetchPR, getCliName, getCliInstallUrl, getMRLabel, getMRNumberLabel, getDisplayRepo } from "@plannotator/server/pr";
 import { writeRemoteShareLink } from "@plannotator/server/share-url";
 import { resolveMarkdownFile, hasMarkdownFiles } from "@plannotator/shared/resolve-file";
 import { FILE_BROWSER_EXCLUDED } from "@plannotator/shared/reference-common";
-import { statSync, rmSync, realpathSync } from "fs";
+import { statSync, rmSync, realpathSync, existsSync } from "fs";
 import { parseRemoteUrl } from "@plannotator/shared/repo";
 import { registerSession, unregisterSession, listSessions } from "@plannotator/server/sessions";
 import { openBrowser } from "@plannotator/server/browser";
 import { detectProjectName } from "@plannotator/server/project";
+import { hostnameOrFallback } from "@plannotator/shared/project";
 import { planDenyFeedback } from "@plannotator/shared/feedback-templates";
 import { readImprovementHook } from "@plannotator/shared/improvement-hooks";
 import type { Origin } from "@plannotator/shared/agents";
@@ -109,6 +112,11 @@ if (browserIdx !== -1 && args[browserIdx + 1]) {
   args.splice(browserIdx, 2);
 }
 
+// Global flag: --no-jina (disables Jina Reader for URL annotation)
+const noJinaIdx = args.indexOf("--no-jina");
+const cliNoJina = noJinaIdx !== -1;
+if (cliNoJina) args.splice(noJinaIdx, 1);
+
 if (isTopLevelHelpInvocation(args)) {
   console.log(formatTopLevelHelp());
   process.exit(0);
@@ -451,7 +459,7 @@ if (args[0] === "sessions") {
 
   let filePath = args[1];
   if (!filePath) {
-    console.error("Usage: plannotator annotate <file.md | folder/>");
+    console.error("Usage: plannotator annotate <file.md | file.html | https://... | folder/>  [--no-jina]");
     process.exit(1);
   }
 
@@ -468,50 +476,87 @@ if (args[0] === "sessions") {
     console.error(`[DEBUG] File path arg: ${filePath}`);
   }
 
-  // Check if the argument is a directory (folder annotation mode)
-  const resolvedArg = path.resolve(projectRoot, filePath);
-  let isFolder = false;
-  try {
-    isFolder = statSync(resolvedArg).isDirectory();
-  } catch {
-    // Not a directory, fall through to file resolution
-  }
-
   let markdown: string;
   let absolutePath: string;
   let folderPath: string | undefined;
   let annotateMode: "annotate" | "annotate-folder" = "annotate";
+  let sourceInfo: string | undefined;
 
-  if (isFolder) {
-    // Folder annotation mode
-    if (!hasMarkdownFiles(resolvedArg, FILE_BROWSER_EXCLUDED)) {
-      console.error(`No markdown files found in ${resolvedArg}`);
-      process.exit(1);
-    }
-    folderPath = resolvedArg;
-    absolutePath = resolvedArg;
-    markdown = "";
-    annotateMode = "annotate-folder";
-    console.error(`Folder: ${resolvedArg}`);
-  } else {
-    // Single file annotation mode
-    const resolved = resolveMarkdownFile(filePath, projectRoot);
+  // --- URL annotation ---
+  const isUrl = /^https?:\/\//i.test(filePath);
 
-    if (resolved.kind === "ambiguous") {
-      console.error(`Ambiguous filename "${resolved.input}" — found ${resolved.matches.length} matches:`);
-      for (const match of resolved.matches) {
-        console.error(`  ${match}`);
+  if (isUrl) {
+    const useJina = resolveUseJina(cliNoJina, loadConfig());
+    console.error(`Fetching: ${filePath}${useJina ? " (via Jina Reader)" : " (via fetch+Turndown)"}`);
+    try {
+      const result = await urlToMarkdown(filePath, { useJina });
+      markdown = result.markdown;
+      if (process.env.PLANNOTATOR_DEBUG) {
+        console.error(`[DEBUG] Fetched via ${result.source} (${markdown.length} chars)`);
       }
+    } catch (err) {
+      console.error(`Failed to fetch URL: ${err instanceof Error ? err.message : String(err)}`);
       process.exit(1);
     }
-    if (resolved.kind === "not_found") {
-      console.error(`File not found: ${resolved.input}`);
-      process.exit(1);
+    absolutePath = filePath; // Use URL as the "path" for display
+    sourceInfo = filePath;   // Full URL for source attribution
+  } else {
+    // Check if the argument is a directory (folder annotation mode)
+    const resolvedArg = path.resolve(projectRoot, filePath);
+    let isFolder = false;
+    try {
+      isFolder = statSync(resolvedArg).isDirectory();
+    } catch {
+      // Not a directory, fall through to file resolution
     }
 
-    absolutePath = resolved.path;
-    markdown = await Bun.file(absolutePath).text();
-    console.error(`Resolved: ${absolutePath}`);
+    if (isFolder) {
+      // Folder annotation mode (markdown + HTML files)
+      if (!hasMarkdownFiles(resolvedArg, FILE_BROWSER_EXCLUDED, /\.(mdx?|html?)$/i)) {
+        console.error(`No markdown or HTML files found in ${resolvedArg}`);
+        process.exit(1);
+      }
+      folderPath = resolvedArg;
+      absolutePath = resolvedArg;
+      markdown = "";
+      annotateMode = "annotate-folder";
+      console.error(`Folder: ${resolvedArg}`);
+    } else if (/\.html?$/i.test(resolvedArg)) {
+      // HTML file annotation mode — convert to markdown via Turndown
+      if (!existsSync(resolvedArg)) {
+        console.error(`File not found: ${filePath}`);
+        process.exit(1);
+      }
+      const htmlFile = Bun.file(resolvedArg);
+      if (htmlFile.size > 10 * 1024 * 1024) {
+        console.error(`File too large (${Math.round(htmlFile.size / 1024 / 1024)}MB, max 10MB): ${resolvedArg}`);
+        process.exit(1);
+      }
+      const html = await htmlFile.text();
+      markdown = htmlToMarkdown(html);
+      absolutePath = resolvedArg;
+      sourceInfo = path.basename(resolvedArg);
+      console.error(`Converted: ${absolutePath}`);
+    } else {
+      // Single markdown file annotation mode
+      const resolved = resolveMarkdownFile(filePath, projectRoot);
+
+      if (resolved.kind === "ambiguous") {
+        console.error(`Ambiguous filename "${resolved.input}" — found ${resolved.matches.length} matches:`);
+        for (const match of resolved.matches) {
+          console.error(`  ${match}`);
+        }
+        process.exit(1);
+      }
+      if (resolved.kind === "not_found") {
+        console.error(`File not found: ${resolved.input}`);
+        process.exit(1);
+      }
+
+      absolutePath = resolved.path;
+      markdown = await Bun.file(absolutePath).text();
+      console.error(`Resolved: ${absolutePath}`);
+    }
   }
 
   const annotateProject = (await detectProjectName()) ?? "_unknown";
@@ -523,6 +568,7 @@ if (args[0] === "sessions") {
     origin: detectedOrigin,
     mode: annotateMode,
     folderPath,
+    sourceInfo,
     sharingEnabled,
     shareBaseUrl,
     pasteApiUrl,
@@ -543,7 +589,9 @@ if (args[0] === "sessions") {
     mode: "annotate",
     project: annotateProject,
     startedAt: new Date().toISOString(),
-    label: folderPath ? `annotate-${path.basename(folderPath)}` : `annotate-${path.basename(absolutePath)}`,
+    label: folderPath
+      ? `annotate-${path.basename(folderPath)}`
+      : `annotate-${isUrl ? hostnameOrFallback(absolutePath) : path.basename(absolutePath)}`,
   });
 
   // Wait for user feedback
 
@@ -20,8 +20,12 @@ import {
 } from "@plannotator/server/annotate";
 import { getGitContext, runGitDiffWithContext } from "@plannotator/server/git";
 import { parsePRUrl, checkPRAuth, fetchPR, getCliName, getMRLabel, getMRNumberLabel, getDisplayRepo } from "@plannotator/server/pr";
-import { loadConfig, resolveDefaultDiffType } from "@plannotator/shared/config";
+import { loadConfig, resolveDefaultDiffType, resolveUseJina } from "@plannotator/shared/config";
 import { resolveMarkdownFile } from "@plannotator/shared/resolve-file";
+import { htmlToMarkdown } from "@plannotator/shared/html-to-markdown";
+import { urlToMarkdown } from "@plannotator/shared/url-to-markdown";
+import { statSync } from "fs";
+import path from "path";
 
 /** Shared dependencies injected by the plugin */
 export interface CommandDeps {
@@ -149,35 +153,79 @@ export async function handleAnnotateCommand(
   const filePath = event.properties?.arguments || event.arguments || "";
 
   if (!filePath) {
-    client.app.log({ level: "error", message: "Usage: /plannotator-annotate <file.md>" });
+    client.app.log({ level: "error", message: "Usage: /plannotator-annotate <file.md | file.html | https://...>" });
     return;
   }
 
-  client.app.log({ level: "info", message: `Opening annotation UI for ${filePath}...` });
+  let markdown: string;
+  let absolutePath: string;
+  let sourceInfo: string | undefined;
 
-  const projectRoot = process.cwd();
-  const resolved = await resolveMarkdownFile(filePath, projectRoot);
+  // --- URL annotation ---
+  const isUrl = /^https?:\/\//i.test(filePath);
 
-  if (resolved.kind === "ambiguous") {
-    client.app.log({
-      level: "error",
-      message: `Ambiguous filename "${resolved.input}" — found ${resolved.matches.length} matches:\n${resolved.matches.map((m) => `  ${m}`).join("\n")}`,
-    });
-    return;
-  }
-  if (resolved.kind === "not_found") {
-    client.app.log({ level: "error", message: `File not found: ${resolved.input}` });
-    return;
-  }
+  if (isUrl) {
+    const useJina = resolveUseJina(false, loadConfig());
+    client.app.log({ level: "info", message: `Fetching: ${filePath}${useJina ? " (via Jina Reader)" : " (via fetch+Turndown)"}...` });
+    try {
+      const result = await urlToMarkdown(filePath, { useJina });
+      markdown = result.markdown;
+    } catch (err) {
+      client.app.log({ level: "error", message: `Failed to fetch URL: ${err instanceof Error ? err.message : String(err)}` });
+      return;
+    }
+    absolutePath = filePath;
+    sourceInfo = filePath;
+  } else {
+    const projectRoot = process.cwd();
+    const resolvedArg = path.resolve(projectRoot, filePath);
 
-  const absolutePath = resolved.path;
-  client.app.log({ level: "info", message: `Resolved: ${absolutePath}` });
-  const markdown = await Bun.file(absolutePath).text();
+    if (/\.html?$/i.test(resolvedArg)) {
+      // HTML file annotation — convert to markdown via Turndown
+      let fileSize: number;
+      try {
+        fileSize = statSync(resolvedArg).size;
+      } catch {
+        client.app.log({ level: "error", message: `File not found: ${filePath}` });
+        return;
+      }
+      if (fileSize > 10 * 1024 * 1024) {
+        client.app.log({ level: "error", message: `File too large (${Math.round(fileSize / 1024 / 1024)}MB, max 10MB)` });
+        return;
+      }
+      const html = await Bun.file(resolvedArg).text();
+      markdown = htmlToMarkdown(html);
+      absolutePath = resolvedArg;
+      sourceInfo = path.basename(resolvedArg);
+      client.app.log({ level: "info", message: `Converted: ${absolutePath}` });
+    } else {
+      // Markdown file annotation
+      client.app.log({ level: "info", message: `Opening annotation UI for ${filePath}...` });
+      const resolved = await resolveMarkdownFile(filePath, projectRoot);
+
+      if (resolved.kind === "ambiguous") {
+        client.app.log({
+          level: "error",
+          message: `Ambiguous filename "${resolved.input}" — found ${resolved.matches.length} matches:\n${resolved.matches.map((m) => `  ${m}`).join("\n")}`,
+        });
+        return;
+      }
+      if (resolved.kind === "not_found") {
+        client.app.log({ level: "error", message: `File not found: ${resolved.input}` });
+        return;
+      }
+
+      absolutePath = resolved.path;
+      client.app.log({ level: "info", message: `Resolved: ${absolutePath}` });
+      markdown = await Bun.file(absolutePath).text();
+    }
+  }
 
   const server = await startAnnotateServer({
     markdown,
     filePath: absolutePath,
     origin: "opencode",
+    sourceInfo,
     sharingEnabled: await getSharingEnabled(),
     shareBaseUrl: getShareBaseUrl(),
     htmlContent,
 
@@ -1,5 +1,5 @@
 ---
-description: Open interactive annotation UI for a markdown file
+description: Open interactive annotation UI for a markdown file, HTML file, or URL
 ---
 
 The Plannotator Annotate UI has been triggered. Opening the annotation UI...