@tekyzinc/gsd-t 2.73.23 → 2.73.25

package/CHANGELOG.md

@@ -2,6 +2,23 @@
 
 All notable changes to GSD-T are documented here. Updated with each release.
 
+## [2.73.25] - 2026-04-09
+
+### Added
+- **Undo remove** — excluded elements show as struck-through with a green ↩ restore button instead of disappearing. Click to undo before submitting.
+- **Contract deletion on submit** — when excluded elements are submitted, their contract files and source files are deleted from the project. The `/review/api/exclude` endpoint handles cleanup.
+
+## [2.73.24] - 2026-04-09
+
+### Added
+- **Remove element button** — hover over any component in the list to reveal a red × button. Clicking it excludes the element from review and auto-comments "EXCLUDED — not in Figma design". Excluded count shown in submit stats.
+
+### Fixed
+- **Comment validation removed** — all comments are now accepted (questions, exclusions, feedback). The "don't suggest specific changes" popup no longer blocks submission.
+
+### Changed
+- **Submit stats** show removed count alongside changed/commented.
+
 ## [2.73.23] - 2026-04-09
 
 ### Fixed

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@tekyzinc/gsd-t",
-  "version": "2.73.23",
+  "version": "2.73.25",
   "description": "GSD-T: Contract-Driven Development for Claude Code — 56 slash commands with headless CI/CD mode, graph-powered code analysis, real-time agent dashboard, execution intelligence, task telemetry, doc-ripple enforcement, backlog management, impact analysis, test sync, milestone archival, and PRD generation",
   "author": "Tekyz, Inc.",
   "license": "MIT",

package/scripts/gsd-t-design-review-server.js

@@ -620,6 +620,48 @@ const server = http.createServer((req, res) => {
     return;
   }
 
+  if (pathname === "/review/api/exclude" && req.method === "POST") {
+    // Remove excluded elements: delete contract files and remove from queue
+    let body = "";
+    req.on("data", (chunk) => { body += chunk; });
+    req.on("end", () => {
+      try {
+        const { excludedIds } = JSON.parse(body);
+        const removed = [];
+        for (const id of excludedIds) {
+          const item = reviewQueue.find(q => q.id === id);
+          if (!item || !item.sourcePath) continue;
+          const match = item.sourcePath.match(/src\/components\/(\w+)\/(\w+)\.vue$/);
+          if (!match) continue;
+          const [, tier, name] = match;
+          const kebab = name.replace(/([a-z0-9])([A-Z])/g, "$1-$2").toLowerCase();
+          const contractPath = path.join(PROJECT_DIR, ".gsd-t", "contracts", "design", tier, `${kebab}.contract.md`);
+          if (fs.existsSync(contractPath)) {
+            fs.unlinkSync(contractPath);
+            removed.push({ id, contract: contractPath });
+          }
+          // Remove source file too
+          const srcPath = path.join(PROJECT_DIR, item.sourcePath);
+          if (fs.existsSync(srcPath)) {
+            fs.unlinkSync(srcPath);
+            removed.push({ id, source: srcPath });
+          }
+        }
+        // Remove from queue
+        for (let i = reviewQueue.length - 1; i >= 0; i--) {
+          if (excludedIds.includes(reviewQueue[i].id)) reviewQueue.splice(i, 1);
+        }
+        broadcast("queue-update", reviewQueue);
+        res.writeHead(200, { "Content-Type": "application/json", "Access-Control-Allow-Origin": "*" });
+        res.end(JSON.stringify({ ok: true, removed }));
+      } catch (err) {
+        res.writeHead(400, { "Content-Type": "application/json" });
+        res.end(JSON.stringify({ error: err.message }));
+      }
+    });
+    return;
+  }
+
   if (pathname === "/review/api/write-source" && req.method === "POST") {
     // Apply CSS property changes back to source files
     let body = "";

package/scripts/gsd-t-design-review.html

@@ -195,6 +195,14 @@
 
   .component-name { flex: 1; overflow: hidden; text-overflow: ellipsis; white-space: nowrap; }
   .component-type { font-size: 10px; color: #94a3b8; }
+  .remove-btn, .undo-remove-btn {
+    display: none; background: none; border: none; font-size: 14px;
+    cursor: pointer; padding: 0 4px; line-height: 1; opacity: 0.6;
+  }
+  .remove-btn { color: #ef4444; }
+  .undo-remove-btn { color: #22c55e; display: block; }
+  .remove-btn:hover, .undo-remove-btn:hover { opacity: 1; }
+  .component-item:hover .remove-btn { display: block; }
 
   /* ── Submit bar ─────────────────────────────────── */
   .submit-bar {
@@ -801,6 +809,7 @@
   let inspectActive = false;
   const changes = new Map();     // componentId → [{path, property, oldValue, newValue}]
   const comments = new Map();   // componentId → string
+  const excluded = new Set();   // componentIds removed from review
   let currentElementPath = null;
   let currentStyles = null;
 
@@ -952,14 +961,45 @@
         statusClass = "rejected";
       }
 
+      const isExcluded = excluded.has(item.id);
+
       const div = document.createElement("div");
-      div.className = `component-item${idx === selectedIdx ? " selected" : ""}`;
-      div.innerHTML = `
-        <div class="component-status ${statusClass}"></div>
-        <div class="component-name">${item.name || item.id}</div>
-        <div class="component-type">${item.type || ""}</div>
-      `;
-      div.addEventListener("click", () => selectComponent(idx));
+      div.className = `component-item${idx === selectedIdx ? " selected" : ""}${isExcluded ? " excluded" : ""}`;
+
+      if (isExcluded) {
+        div.innerHTML = `
+          <div class="component-status" style="background:#ef4444"></div>
+          <div class="component-name" style="text-decoration:line-through;opacity:0.5">${item.name || item.id}</div>
+          <button class="undo-remove-btn" title="Restore element">↩</button>
+        `;
+        div.querySelector(".undo-remove-btn").addEventListener("click", (e) => {
+          e.stopPropagation();
+          excluded.delete(item.id);
+          comments.delete(item.id);
+          renderComponentList();
+          updateSubmitStats();
+        });
+      } else {
+        div.innerHTML = `
+          <div class="component-status ${statusClass}"></div>
+          <div class="component-name">${item.name || item.id}</div>
+          <div class="component-type">${item.type || ""}</div>
+          <button class="remove-btn" title="Remove from review">×</button>
+        `;
+        div.querySelector(".component-name").addEventListener("click", () => selectComponent(idx));
+        div.querySelector(".component-status").addEventListener("click", () => selectComponent(idx));
+        div.querySelector(".remove-btn").addEventListener("click", (e) => {
+          e.stopPropagation();
+          excluded.add(item.id);
+          comments.set(item.id, "EXCLUDED — not in Figma design");
+          if (selectedIdx === idx) {
+            const remaining = filteredQueue.filter(q => !excluded.has(q.id));
+            selectedIdx = remaining.length > 0 ? filteredQueue.indexOf(remaining[0]) : -1;
+          }
+          renderComponentList();
+          updateSubmitStats();
+        });
+      }
       componentList.appendChild(div);
     });
   }
@@ -2110,21 +2150,20 @@
   // ── Submit ────────────────────────────────────────
   function updateSubmitStats() {
     const total = queue.length;
+    const excludedCount = excluded.size;
     const changed = Array.from(changes.keys()).filter(id => (changes.get(id) || []).length > 0).length;
-    const commented = comments.size;
-
-    if (changed > 0 || commented > 0) {
-      submitStats.innerHTML = `
-        <span><span class="component-status changed" style="display:inline-block"></span> ${changed} changed</span>
-        <span><span class="component-status rejected" style="display:inline-block"></span> ${commented} commented</span>
-        <span style="color:var(--text-dim)">${total} total</span>
-      `;
-    } else {
-      submitStats.innerHTML = `<span style="color:var(--text-dim)">${total} elements — no changes</span>`;
-    }
-
-    submitAll.textContent = changed > 0 || commented > 0
-      ? `Submit (${changed} changes, ${commented} comments)`
+    const commented = Array.from(comments.keys()).filter(id => !excluded.has(id)).length;
+
+    const parts = [];
+    if (changed > 0) parts.push(`<span><span class="component-status changed" style="display:inline-block"></span> ${changed} changed</span>`);
+    if (commented > 0) parts.push(`<span><span class="component-status rejected" style="display:inline-block"></span> ${commented} commented</span>`);
+    if (excludedCount > 0) parts.push(`<span style="color:#ef4444">${excludedCount} removed</span>`);
+    parts.push(`<span style="color:var(--text-dim)">${total - excludedCount} of ${total}</span>`);
+    submitStats.innerHTML = parts.join(" ");
+
+    const actionCount = changed + commented + excludedCount;
+    submitAll.textContent = actionCount > 0
+      ? `Submit (${changed} changes, ${commented} comments, ${excludedCount} removed)`
       : "Submit — Approve All";
   }
 
@@ -2136,28 +2175,7 @@
       else comments.delete(filteredQueue[selectedIdx].id);
     }
 
-    // Check for non-actionable comments (documentation, not change requests)
-    const actionWords = /change|make|set|move|add|remove|reduce|increase|fix|use|switch|replace|adjust|align|center|should be|needs to|too |bigger|smaller|wider|narrower|thicker|thinner|lighter|darker|bolder|px|rem|%|#[0-9a-f]/i;
-    const docComments = [];
-    for (const [compId, comment] of comments) {
-      if (!actionWords.test(comment)) {
-        const item = queue.find(q => q.id === compId);
-        docComments.push(item ? item.name : compId);
-      }
-    }
-    if (docComments.length > 0) {
-      const proceed = confirm(
-        "These comments don't suggest specific changes:\n\n" +
-        docComments.map(n => "  \u2022 " + n).join("\n") +
-        "\n\nComments should describe what to change, e.g.:\n" +
-        '  "make padding 8px"\n  "use darker blue"\n  "reduce gap between title and chart"\n\n' +
-        "Non-actionable comments will be discarded.\n\nSubmit anyway?"
-      );
-      if (!proceed) return;
-      for (const [compId, comment] of comments) {
-        if (!actionWords.test(comment)) comments.delete(compId);
-      }
-    }
+    // All comments are accepted — questions, exclusions, and change requests alike
 
     // Build feedback: each element gets its changes and comments
     const feedback = queue.map(item => ({
@@ -2193,6 +2211,15 @@
             body: JSON.stringify({ changes: allChanges }),
           });
         }
+
+        // Delete contracts and source files for excluded elements
+        if (excluded.size > 0) {
+          await fetch("/review/api/exclude", {
+            method: "POST",
+            headers: { "Content-Type": "application/json" },
+            body: JSON.stringify({ excludedIds: Array.from(excluded) }),
+          });
+        }
       }
     } catch (err) {
       submitAll.textContent = "Error — retry";