claude-plan-review 0.5.0 → 0.5.2

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "claude-plan-review",
-  "version": "0.5.0",
+  "version": "0.5.2",
   "description": "Review, comment on, and approve/reject Claude Code plans in a local GitHub-style web UI — with persistent comments, version history, and diffs.",
   "type": "module",
   "bin": {

package/src/mcp.js

@@ -25,12 +25,13 @@ import { getReview, recordPlan } from "./store.js";
 
 const HERE = dirname(fileURLToPath(import.meta.url));
 const SERVER_SCRIPT = join(HERE, "server.js");
+const pkg = JSON.parse(readFileSync(join(HERE, "..", "package.json"), "utf8"));
 
 // Latest MCP protocol revision we implement. We echo the client's requested
 // version back when it sends one (the compatible behaviour for a version-
 // agnostic tools server), else advertise this.
 const PROTOCOL_VERSION = "2025-06-18";
-const SERVER_INFO = { name: "claude-plan-review", version: "0.4.0" };
+const SERVER_INFO = { name: "claude-plan-review", version: pkg.version };
 
 const sleep = (ms) => new Promise((r) => setTimeout(r, ms));
 

package/src/ui/app.js

@@ -35,7 +35,7 @@ const state = {
   docCache: new Map(), // slug → {slug,title,parent,markdown,html}
   manifestCache: new Map(), // version n → manifest|null (for the diff doc union)
   allComments: [], // every comment on the version (each carries `doc`) — drives badges
-  viewAsTree: false, // single-doc display-only outline mode
+  viewAsTree: true, // single-doc display-only outline mode (always on for single-doc plans)
   treeSection: -1, // selected outline section index (-1 = show all)
   _sections: null, // cached outline sections for the current preview
   // --- deletion ---
@@ -89,8 +89,6 @@ function showEmpty(msg) {
   $("docSidebar").hidden = true;
   $("reviewActions").hidden = true;
   $("saveActions").hidden = true;
-  $("deleteVerBtn").hidden = true;
-  $("treeToggle").hidden = true;
 }
 
 async function loadProjects() {
@@ -123,7 +121,6 @@ async function selectProject(key, wantV) {
   const fmt = (v) => `v${v.n} · ${new Date(v.createdAt).toLocaleString()}${v.n === latest ? " (current)" : ""}`;
   $("versionSel").innerHTML = versions.map((v) => `<option value="${v.n}">${esc(fmt(v))}</option>`).join("");
   $("versionSel").value = String(state.version);
-  $("deleteVerBtn").hidden = false;
 
   // base = the version right before current, if any
   state.baseVersion = versions.length > 1 ? versions[versions.findIndex((v) => v.n === state.version) - 1]?.n ?? versions[0].n : null;
@@ -139,7 +136,6 @@ async function selectProject(key, wantV) {
 async function loadVersion(n) {
   state.version = n;
   hideSelPop();
-  state.viewAsTree = false;
   state.treeSection = -1;
   state.docCache = new Map();
   state.data = await api(`/api/projects/${enc(state.key)}/versions/${n}`);
@@ -149,7 +145,9 @@ async function loadVersion(n) {
   const isTree = docs.length > 1;
   state.kind = state.data.kind || (isTree ? "tree" : "single");
   $("diffCur").textContent = n;
-  updateTreeToggle();
+  // Tree view is always on for single-doc plans (heading outline); real multi-doc
+  // trees use the doc sidebar instead, so they don't also apply the outline view.
+  state.viewAsTree = !isTree;
 
   if (isTree) {
     // sidebar badge counts come from one unscoped fetch; threads filter it per doc.
@@ -431,15 +429,6 @@ function renderHeadingTree() {
   return rows.join("");
 }
 
-function updateTreeToggle() {
-  const isTree = state.manifest && state.manifest.docs.length > 1;
-  const t = $("treeToggle");
-  t.hidden = isTree; // real doc trees get the sidebar automatically
-  if (isTree) state.viewAsTree = false;
-  t.classList.toggle("active", state.viewAsTree);
-  t.textContent = state.viewAsTree ? "Exit tree view" : "View as tree";
-}
-
 // ---------- comments ----------
 async function addComment(line, lineEnd, body, quote) {
   if (!body.trim()) return;
@@ -1025,15 +1014,6 @@ function wireUI() {
     };
   });
 
-  // view-as-tree (single-doc, display-only)
-  $("treeToggle").onclick = () => {
-    state.viewAsTree = !state.viewAsTree;
-    state.treeSection = -1;
-    updateTreeToggle();
-    renderPreview();
-    renderSidebar();
-  };
-
   // sidebar navigation — doc rows (tree) or outline sections (view-as-tree)
   $("docSidebar").addEventListener("click", (e) => {
     const row = e.target.closest("[data-doc]");
@@ -1106,8 +1086,8 @@ function wireUI() {
     await resolve("reject");
   };
 
-  // deletion + manage — delete fires immediately; the modal is blocked-only
-  $("deleteVerBtn").onclick = () => deleteVersionNow();
+  // deletion + manage — header entry points are hidden for now, but the underlying
+  // logic (deleteVersionNow / openManageModal / performDelete) is kept intact.
   $("deleteCancel").onclick = async () => {
     const pd = state.pendingDelete;
     $("deleteModal").hidden = true;
@@ -1120,7 +1100,6 @@ function wireUI() {
   };
   $("deleteForce").onclick = () => performDelete(true);
 
-  $("manageBtn").onclick = () => openManageModal();
   $("manageClose").onclick = () => ($("manageModal").hidden = true);
   $("manageList").addEventListener("click", (e) => {
     const btn = e.target.closest("[data-mg]");

package/src/ui/index.html

@@ -24,7 +24,6 @@
       <label>Version
         <select id="versionSel"></select>
       </label>
-      <button class="iconbtn" id="deleteVerBtn" title="Delete this version" aria-label="Delete this version" hidden>🗑</button>
     </div>
     <div class="saveActions" id="saveActions" hidden>
       <button class="btn" id="saveBtn">Save to gist</button>
@@ -35,7 +34,6 @@
       <button class="btn approve" id="approveBtn">Approve plan</button>
       <button class="btn reject" id="rejectBtn">Request changes</button>
     </div>
-    <button class="iconbtn" id="manageBtn" title="Manage &amp; delete plans" aria-label="Manage plans">🗄</button>
     <button class="iconbtn" id="themeToggle" title="Toggle light / dark theme" aria-label="Toggle theme">🌓</button>
   </header>
 
@@ -49,7 +47,6 @@
     <section class="panel active" id="panel-preview">
       <div class="preview-toolbar">
         <p class="selhint">Select any text in the plan to comment on it.</p>
-        <button class="btn tree-toggle" id="treeToggle" hidden>View as tree</button>
       </div>
       <div class="preview-layout">
         <aside class="doc-sidebar" id="docSidebar" hidden></aside>

package/src/ui/style.css

@@ -200,9 +200,6 @@ textarea {
 /* ---------- multi-document: sidebar + layout ---------- */
 .preview-toolbar { display: flex; align-items: center; justify-content: space-between; gap: 12px; margin: 0 0 10px; }
 .preview-toolbar .selhint { margin: 0; }
-.tree-toggle { font-size: 12px; padding: 4px 10px; flex: none; }
-.tree-toggle.active { background: var(--accent); border-color: var(--accent); color: #fff; }
-.tree-toggle.active:hover { filter: brightness(1.08); }
 
 .preview-layout { display: flex; align-items: flex-start; gap: 20px; }
 .preview-main { flex: 1; min-width: 0; }