Remove Share on X button, add review quality metrics bar

Replace the Twitter share button with a metrics bar that displays
after each review: issues found, concrete fixes, lines analyzed,
review time, and line coverage percentage.

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>

public/app.js

@@ -13,8 +13,8 @@ const streamError = document.getElementById("stream-error");
 const tabButtons = document.querySelectorAll(".tab");
 const brutalityBtns = document.querySelectorAll(".brutality-btn");
 const issueBadge = document.getElementById("issue-badge");
+const reviewMetrics = document.getElementById("review-metrics");
 const copyBtn = document.getElementById("copy-btn");
-const shareXBtn = document.getElementById("share-x-btn");
 const toast = document.getElementById("toast");
 
 const GITHUB_BLOB_RE = /^https?:\/\/github\.com\/[^/]+\/[^/]+\/blob\/[^/]+\/.+$/;
@@ -240,30 +240,63 @@ function renderIssueBadge(counts) {
   issueBadge.hidden = false;
 }
 
-/* --- Share on X --- */
+/* --- Review metrics --- */
 
-function stripMarkdown(md) {
-  return md.replace(/\*\*/g, "").replace(/^#+\s*/gm, "").replace(/\n{2,}/g, "\n").trim();
-}
+function computeMetrics(markdown, totalLines, elapsedMs) {
+  // Count ```diff blocks (concrete fixes)
+  const diffBlocks = (markdown.match(/```diff/g) || []).length;
 
-function shareOnX() {
-  const url = urlInput.value.trim();
-  const mode = brutalityLevel;
-  const verdicts = stripMarkdown(currentSections.verdicts || "");
-
-  const header = `I reviewed code with Arcee AI Trinity Large in ${mode} mode and here's what Linus, Knuth, and Bjarne had to say:\n\n`;
-  const footer = `\n\nTry it yourself: https://huggingface.co/spaces/julien-c/trinity-large\n${url}`;
-  const maxVerdicts = 280 - header.length - footer.length;
-  const trimmedVerdicts = verdicts.length > maxVerdicts
-    ? verdicts.slice(0, maxVerdicts - 1) + "\u2026"
-    : verdicts;
-
-  const text = header + trimmedVerdicts + footer;
-  const shareUrl = `https://twitter.com/intent/tweet?text=${encodeURIComponent(text)}`;
-  window.open(shareUrl, "_blank", "width=600,height=400");
+  // Count total issues by priority tag
+  const issueCount = (markdown.match(/\[CRITICAL\]|\[HIGH\]|\[MEDIUM\]|\[LOW\]/gi) || []).length;
+
+  // Extract unique line numbers referenced (e.g. "Line 42", "Lines 10-20")
+  const lineRefs = new Set();
+  for (const m of markdown.matchAll(/Lines?\s+(\d+)(?:\s*[-–]\s*(\d+))?/gi)) {
+    const start = Number(m[1]);
+    const end = m[2] ? Number(m[2]) : start;
+    for (let i = start; i <= end && i <= totalLines; i++) lineRefs.add(i);
+  }
+  const coverage = totalLines > 0 ? Math.round((lineRefs.size / totalLines) * 100) : 0;
+
+  return {
+    issues: issueCount,
+    fixes: diffBlocks,
+    totalLines,
+    elapsedSec: (elapsedMs / 1000).toFixed(1),
+    coverage,
+  };
 }
 
-shareXBtn.addEventListener("click", shareOnX);
+function renderMetrics(m) {
+  reviewMetrics.textContent = "";
+
+  const items = [
+    { value: m.issues, label: "issues found" },
+    { value: m.fixes, label: "concrete fixes" },
+    { value: m.totalLines.toLocaleString(), label: "lines analyzed" },
+    { value: `${m.elapsedSec}s`, label: "review time" },
+    { value: `${m.coverage}%`, label: "line coverage" },
+  ];
+
+  for (const item of items) {
+    const span = document.createElement("span");
+    span.className = "metric-item";
+
+    const val = document.createElement("span");
+    val.className = "metric-value";
+    val.textContent = item.value;
+
+    const lbl = document.createElement("span");
+    lbl.className = "metric-label";
+    lbl.textContent = item.label;
+
+    span.appendChild(val);
+    span.appendChild(lbl);
+    reviewMetrics.appendChild(span);
+  }
+
+  reviewMetrics.hidden = false;
+}
 
 /* --- Copy to clipboard --- */
 
@@ -344,8 +377,8 @@ function startReview() {
     btn.classList.remove("has-content", "streaming");
   });
   issueBadge.hidden = true;
+  reviewMetrics.hidden = true;
   copyBtn.hidden = true;
-  shareXBtn.hidden = true;
   fullMarkdown = "";
 
   if (!url) { showInputError("Please enter a GitHub file URL."); return; }
@@ -358,6 +391,8 @@ function startReview() {
   setLoading(true);
 
   let markdown = "";
+  let reviewStartTime = performance.now();
+  let fileTotalLines = 0;
   const apiUrl = `/api/review?url=${encodeURIComponent(url)}&brutality=${encodeURIComponent(brutalityLevel)}`;
   const source = new EventSource(apiUrl);
   currentSource = source;
@@ -367,6 +402,7 @@ function startReview() {
     metaRepo.textContent = `${data.owner}/${data.repo}`;
     metaPath.textContent = data.path;
     metaBranch.textContent = data.branch;
+    fileTotalLines = data.lines || 0;
     metaEl.hidden = false;
     tabsEl.hidden = false;
     tabContent.classList.add("is-streaming");
@@ -386,11 +422,13 @@ function startReview() {
     // Final render with no streaming section
     const sections = parseSections(markdown);
     updateTabs(sections, null);
-    // Show badge and copy button
+    // Show badge, metrics, and copy button
     const counts = countIssuePriorities(markdown);
     renderIssueBadge(counts);
+    const elapsed = performance.now() - reviewStartTime;
+    const metrics = computeMetrics(markdown, fileTotalLines, elapsed);
+    renderMetrics(metrics);
     copyBtn.hidden = false;
-    shareXBtn.hidden = false;
   });
 
   source.addEventListener("error", (e) => {

public/index.html

@@ -258,6 +258,7 @@
     </div>
 
     <div id="issue-badge" hidden></div>
+    <div id="review-metrics" hidden></div>
 
     <div id="tabs" hidden>
       <nav class="tab-bar">
@@ -267,11 +268,6 @@
         <button class="tab" data-section="security">Security</button>
         <button class="tab" data-section="suggestions">Suggestions</button>
         <button class="tab" data-section="verdicts">Verdicts</button>
-        <button type="button" id="share-x-btn" hidden title="Share on X">
-          <svg viewBox="0 0 24 24" width="16" height="16" fill="currentColor">
-            <path d="M18.244 2.25h3.308l-7.227 8.26 8.502 11.24H16.17l-5.214-6.817L4.99 21.75H1.68l7.73-8.835L1.254 2.25H8.08l4.713 6.231zm-1.161 17.52h1.833L7.084 4.126H5.117z"/>
-          </svg>
-        </button>
         <button type="button" id="copy-btn" hidden>Copy Review</button>
       </nav>
       <div id="tab-content" class="tab-content"></div>

public/style.css

@@ -493,29 +493,34 @@ h1 {
   color: var(--text-muted);
 }
 
-/* ---------- Share on X Button ---------- */
+/* ---------- Review Metrics ---------- */
 
-#share-x-btn {
-  margin-left: auto;
-  padding: 6px 10px;
-  color: var(--text-muted);
+#review-metrics {
+  display: flex;
+  gap: 20px;
+  flex-wrap: wrap;
+  margin-top: 10px;
+  padding: 10px 14px;
   background: var(--surface);
   border: 1px solid var(--border);
   border-radius: var(--radius);
-  cursor: pointer;
-  transition: all 0.15s;
-  display: inline-flex;
-  align-items: center;
+  font-size: 0.85rem;
 }
 
-#share-x-btn:hover {
-  color: var(--text);
-  border-color: var(--accent);
+.metric-item {
+  display: flex;
+  align-items: baseline;
+  gap: 5px;
 }
 
-#share-x-btn:active {
-  background: var(--accent);
-  color: #fff;
+.metric-value {
+  font-weight: 700;
+  color: var(--accent);
+  font-variant-numeric: tabular-nums;
+}
+
+.metric-label {
+  color: var(--text-muted);
 }
 
 /* ---------- Copy Button ---------- */

server.js

@@ -225,7 +225,8 @@ app.get("/api/review", async (req, res) => {
     }
 
     // Send file metadata to the client
-    send("meta", { owner: meta.owner, repo: meta.repo, branch: meta.branch, path: meta.path });
+    const totalLines = code.split("\n").length;
+    send("meta", { owner: meta.owner, repo: meta.repo, branch: meta.branch, path: meta.path, lines: totalLines });
 
     // Stream from OpenRouter
     console.log(`[review] Requesting review from OpenRouter (brutality: ${brutalityLevel})…`);