opendoor-plugin 0.1.0 → 0.2.0

package/README.md

@@ -271,6 +271,7 @@ bottom:
 ```sh
 npm install        # install dependencies
 npm test           # vitest (unit tests under src/**/*.test.ts)
+npm run test:coverage  # v8 coverage report
 npm run typecheck  # tsc --noEmit (strict)
 npm run build      # esbuild: dist/browser/pi-web-plugin.js + dist/server-plugin.js
                    # (+ verbatim package.json manifest copy into dist/)

package/dist/browser/pi-web-plugin.js

@@ -7568,9 +7568,32 @@ var panelStyles = i`
           background: var(--pi-selection-bg);
         }
 
+        /* The per-link trash can (pointer-only, see {@link renderLinkOut}):
+           right-aligned in the flex row, mirroring .doorstop-item-add. */
+        .doorstop-link-remove {
+          display: inline-flex;
+          align-items: center;
+          align-self: center;
+          flex: 0 0 auto;
+          margin-left: auto;
+          color: var(--pi-muted);
+          cursor: pointer;
+        }
+
+        .doorstop-link-remove:hover {
+          color: var(--pi-text);
+        }
+
         .doorstop-fingerprint {
           color: var(--pi-muted);
           font-size: 11px;
+          /* The link row wraps, which could push the trash can onto a second
+             line on narrow panes. Let the fingerprint shrink and ellipsize
+             instead (min-width: 0 lifts the flex min-content floor). */
+          min-width: 0;
+          overflow: hidden;
+          text-overflow: ellipsis;
+          white-space: nowrap;
         }
 
         .doorstop-attributes {
@@ -7688,7 +7711,7 @@ var panelStyles = i`
         .doorstop-menu-items {
           position: absolute;
           right: 0;
-          top: calc(100% + 4px);
+          bottom: calc(100% + 4px);
           z-index: 3;
           min-width: 180px;
           display: grid;
@@ -7943,6 +7966,7 @@ var publishIconSvg = w`<svg width="14" height="14" viewBox="0 0 24 24" fill="non
 var gitStageIconSvg = w`<svg width="14" height="14" viewBox="0 0 24 24" fill="none" stroke="currentColor" stroke-width="2" stroke-linecap="round" stroke-linejoin="round" aria-hidden="true"><circle cx="12" cy="12" r="10"/><path d="M8 12h8"/><path d="M12 8v8"/></svg>`;
 var gitUnstageIconSvg = w`<svg width="14" height="14" viewBox="0 0 24 24" fill="none" stroke="currentColor" stroke-width="2" stroke-linecap="round" stroke-linejoin="round" aria-hidden="true"><circle cx="12" cy="12" r="10"/><path d="M8 12h8"/></svg>`;
 var gitCommitIconSvg = w`<svg width="14" height="14" viewBox="0 0 24 24" fill="none" stroke="currentColor" stroke-width="2" stroke-linecap="round" stroke-linejoin="round" aria-hidden="true"><circle cx="12" cy="12" r="10"/><path d="m9 12 2 2 4-4"/></svg>`;
+var trashIconSvg = w`<svg width="14" height="14" viewBox="0 0 24 24" fill="none" stroke="currentColor" stroke-width="2" stroke-linecap="round" stroke-linejoin="round" aria-hidden="true"><path d="M3 6h18"/><path d="M19 6v14a2 2 0 0 1-2 2H7a2 2 0 0 1-2-2V6"/><path d="M8 6V4a2 2 0 0 1 2-2h4a2 2 0 0 1 2 2v2"/><path d="M10 11v6"/><path d="M14 11v6"/></svg>`;
 
 // src/doorstop-prompts.ts
 function andJoin(values) {
@@ -8717,9 +8741,9 @@ function defineDoorstopPanelBodyElement() {
             <div class="doorstop-state-chip-row">
               ${item.stateKeys.map((key) => this.renderStateChip(key))}
             </div>
-            ${this.renderChangesSinceReview(item)}
             <h4 class="doorstop-section-title">Text</h4>
             <p class=${item.text === "" ? "doorstop-text doorstop-text-empty" : "doorstop-text"}>${item.text === "" ? "empty" : item.text}</p>
+            ${this.renderChangesSinceReview(item)}
             <h4 class="doorstop-section-title">Parent links</h4>
             ${this.renderLinksOut(item, result.index)}
             <h4 class="doorstop-section-title">Child links</h4>
@@ -8781,11 +8805,22 @@ function defineDoorstopPanelBodyElement() {
         if (item.links.length === 0) return b2`<p class="doorstop-muted">No parent links.</p>`;
         return b2`
           <div class="doorstop-links" aria-label="Parent links">
-            ${item.links.map((link) => this.renderLinkOut(link, index))}
+            ${item.links.map((link) => this.renderLinkOut(item, link, index))}
           </div>
         `;
       }
-      renderLinkOut(link, index) {
+      renderLinkOut(item, link, index) {
+        const remove = b2`
+          <span
+            class="doorstop-link-remove"
+            aria-hidden="true"
+            title=${`doorstop unlink ${item.uid} ${link.uid}`}
+            @click=${(event) => {
+          event.stopPropagation();
+          this.unlinkLink(item, link.uid);
+        }}
+          >${trashIconSvg}</span>
+        `;
         const target = index.byUid.get(link.uid);
         if (target === void 0) {
           return b2`
@@ -8793,6 +8828,7 @@ function defineDoorstopPanelBodyElement() {
               <code>${link.uid}</code>
               <span class="doorstop-chip doorstop-chip-danger">unknown</span>
               <span class="doorstop-fingerprint">not in the index — ${link.fingerprint === null ? "no recorded fingerprint" : `recorded ${shortFingerprint(link.fingerprint)}`}</span>
+              ${remove}
             </div>
           `;
         }
@@ -8811,6 +8847,7 @@ function defineDoorstopPanelBodyElement() {
             <code>${link.uid}</code>
             <span class=${suspect ? "doorstop-chip doorstop-chip-danger" : "doorstop-chip doorstop-chip-ok"}>${suspect ? "suspect" : "ok"}</span>
             <span class="doorstop-fingerprint">recorded ${shortFingerprint(link.fingerprint)} · current ${shortFingerprint(current)}</span>
+            ${remove}
           </button>
         `;
       }
@@ -8869,7 +8906,7 @@ function defineDoorstopPanelBodyElement() {
             @click=${() => {
           this.reviewItem(item);
         }}
-          >Review</button>
+          >Approve</button>
           <button
             type="button"
             class="doorstop-clear"
@@ -8879,15 +8916,6 @@ function defineDoorstopPanelBodyElement() {
           this.clearSuspects(item, suspectUids);
         }}
           >Clear suspect links</button>
-          <button
-            type="button"
-            class="doorstop-edit"
-            ?disabled=${this.runInProgress !== void 0}
-            title=${`Open ${item.uid} in the editor`}
-            @click=${() => {
-          this.editItem(item);
-        }}
-          >Edit</button>
           ${gitStatus === void 0 ? A : b2`<button
                 type="button"
                 class="doorstop-item-stage"
@@ -8906,19 +8934,6 @@ function defineDoorstopPanelBodyElement() {
           this.unstageItem(item);
         }}
               >${gitUnstageIconSvg}Unstage</button>`}
-          <div class="doorstop-op">
-            <input
-              type="text"
-              class="doorstop-target-input"
-              data-op="unlink"
-              placeholder="parent UID"
-              ${n5(this.unlinkInputRef)}
-              @keydown=${this.onTargetKeydown}
-            />
-            <button type="button" class="doorstop-unlink" title=${`doorstop unlink ${item.uid} <target>`} ?disabled=${this.runInProgress !== void 0} @click=${() => {
-          this.runTargetOp("unlink", this.unlinkInputRef, item);
-        }}>Unlink</button>
-          </div>
           <div class="doorstop-op">
             <input
               type="text"
@@ -8999,6 +9014,24 @@ function defineDoorstopPanelBodyElement() {
         if (controller === void 0 || controller.runInProgress !== void 0) return;
         void controller.runGitUnstage([item.path], `Git: unstage ${item.uid}`);
       }
+      /** Unlink one parent from one item — the same `unlink` run the palette's
+       *  free-text Unlink button dispatches, but with the target sourced from
+       *  the index (`link.uid`) instead of an input. The guard mirrors
+       *  {@link stageItem}: a run already in flight must never be overlapped.
+       *  There is no `targetError` interaction: an index-sourced UID cannot
+       *  fail the free-text validation, and `parseDoorstopRunRequest`
+       *  re-validates server-side. */
+      unlinkLink(item, target) {
+        const controller = this.controller;
+        if (controller === void 0 || controller.runInProgress !== void 0) return;
+        this.runDoorstop(
+          "unlink",
+          `Doorstop: unlink ${item.uid}`,
+          { op: "unlink", uid: item.uid, target },
+          `doorstop unlink ${item.uid} ${target}`,
+          false
+        );
+      }
       /**
        * The browser must never send an empty commit message: the server
        * rejects it, and git with an empty `-m` would hang the exec until the

package/dist/package.json

@@ -1,6 +1,6 @@
 {
   "name": "opendoor-plugin",
-  "version": "0.1.0",
+  "version": "0.2.0",
   "description": "Opendoor — a pi-web browser plugin that surfaces Doorstop requirements stored in the workspace: the document tree, per-item state (reviewed/unreviewed changes, suspect links), validation findings, and one-click doorstop CLI actions.",
   "type": "module",
   "license": "MIT",
@@ -33,6 +33,7 @@
     "build": "node scripts/build-plugin.mjs",
     "dev": "node scripts/build-plugin.mjs --watch",
     "test": "vitest run",
+    "test:coverage": "vitest run --coverage",
     "typecheck": "tsc --noEmit"
   },
   "peerDependencies": {
@@ -42,10 +43,11 @@
     "@jmfederico/pi-web": "^1.202608.2",
     "@types/js-yaml": "^4.0.9",
     "@types/node": "^24.0.0",
+    "@vitest/coverage-v8": "^4.1.11",
     "esbuild": "^0.27.7",
+    "happy-dom": "^20.11.1",
     "js-yaml": "^4.3.2",
     "lit": "^3.3.3",
-    "happy-dom": "^20.11.1",
     "typescript": "^6.0.3",
     "vitest": "^4.1.10"
   }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "opendoor-plugin",
-  "version": "0.1.0",
+  "version": "0.2.0",
   "description": "Opendoor — a pi-web browser plugin that surfaces Doorstop requirements stored in the workspace: the document tree, per-item state (reviewed/unreviewed changes, suspect links), validation findings, and one-click doorstop CLI actions.",
   "type": "module",
   "license": "MIT",
@@ -33,6 +33,7 @@
     "build": "node scripts/build-plugin.mjs",
     "dev": "node scripts/build-plugin.mjs --watch",
     "test": "vitest run",
+    "test:coverage": "vitest run --coverage",
     "typecheck": "tsc --noEmit"
   },
   "peerDependencies": {
@@ -42,10 +43,11 @@
     "@jmfederico/pi-web": "^1.202608.2",
     "@types/js-yaml": "^4.0.9",
     "@types/node": "^24.0.0",
+    "@vitest/coverage-v8": "^4.1.11",
     "esbuild": "^0.27.7",
+    "happy-dom": "^20.11.1",
     "js-yaml": "^4.3.2",
     "lit": "^3.3.3",
-    "happy-dom": "^20.11.1",
     "typescript": "^6.0.3",
     "vitest": "^4.1.10"
   }