@compn3rd/pi-web-code-review 0.1.1 → 0.1.2

package/CHANGELOG.md

@@ -1,5 +1,22 @@
 # Changelog
 
+## 0.1.2 — 2026-10-16
+
+### Changed
+
+- Open and focus comment editors directly beneath the selected code line, inside the file viewer, instead of below the entire file.
+- Use a full-width header with saved-comment and unsaved-draft status, and move **Attach review to composer** to the header.
+- Place file navigation beside the viewer on wide panels and stack it above the viewer on narrow panels.
+- Highlight the active file and improve empty states, keyboard hints, focus indicators, and source-control layout.
+- Keep changed-snapshot drafts accessible outside the code view without losing their text or allowing an invalid save.
+
+### Validation and compatibility
+
+- Add regression tests for inline editor placement, focus, header action placement, active-file state, and changed-snapshot draft preservation (84 tests total).
+- Capture browser-rendered before/after screenshots with the same sample file and draft; these are standalone previews, not live-host acceptance tests.
+- No host API, storage format, or server behavior changes. Requirements remain PI WEB `>=1.202610.3` and Node.js `>=22.19.0`.
+- Live-host acceptance and remote-machine testing remain outstanding; see [compatibility results](docs/compatibility.md).
+
 ## 0.1.1 — 2026-10-15
 
 Package: **`@compn3rd/pi-web-code-review`** (public npm scope).

package/README.md

@@ -4,7 +4,7 @@ A standalone Review tab for workspace files, staged/unstaged Git changes, and ac
 
 ## Requirements and installation
 
-Patch release: **0.1.1**. Requires **PI WEB >=1.202610.3**, Node.js **>=22.19.0**, and Git for staged/unstaged and revision-comparison review. Compatibility checks were performed against PI WEB `1.202610.3`; later releases are allowed but have not been independently tested.
+Patch release: **0.1.2**. Requires **PI WEB >=1.202610.3**, Node.js **>=22.19.0**, and Git for staged/unstaged and revision-comparison review. Compatibility checks were performed against PI WEB `1.202610.3`; later releases are allowed but have not been independently tested.
 
 Install `npm:@compn3rd/pi-web-code-review` through **Settings → Pi packages** on the target machine, then enable **Code Review** in **Settings → PI WEB plugins**. For local builds and manual installation, see the [usage guide](docs/usage.md).
 
@@ -14,9 +14,17 @@ This package includes a server plugin: restart the target session daemon when sa
 
 Open the **Review** tab, choose workspace files, staged/unstaged changes, or **Compare revisions**, and save line-range comments. Compare `main...HEAD` for branch work since its merge base, or `HEAD~3..HEAD` for accumulated work between commits. Comparison comments retain frozen commit IDs. Select a ready conversation and choose **Attach review to composer** to stage a removable **Review (N)** chip. Send it with a message or by itself. Removing the chip preserves saved comments; server acceptance clears only unchanged submitted comments.
 
+## Redesigned review panel
+
+Click a line number to open a focused comment editor directly beneath that line. The file sidebar sits beside the viewer on wide panels and stacks above it on narrow panels. Saved-comment and draft counts appear in the full-width header alongside **Attach review to composer**; only saved comments are attached.
+
+![Redesigned review panel](docs/screenshots/review-after.png)
+
+Standalone browser preview with sample content, not a live-host screenshot. [Before redesign](docs/screenshots/review-before.png).
+
 ## Validation status
 
-Typechecking, all 82 tests, builds, and unpacked/packed artifact smoke checks passed against PI WEB `1.202610.3`. Full automated verification passes; live-host acceptance testing is pending. Publication is enabled for the scoped patch release; these remaining checks are follow-up work, not claims of completed validation.
+Typechecking, all 84 tests, builds, and unpacked/packed artifact smoke checks passed against PI WEB `1.202610.3`. Full automated verification passes; live-host acceptance testing is pending. Publication is enabled for the scoped patch release; these remaining checks are follow-up work, not claims of completed validation.
 
 See the [composer design](docs/composer-design.md), [usage guide](docs/usage.md), [development guide](docs/development.md), and [compatibility results](docs/compatibility.md).
 

package/dist/browser/index.js

@@ -2658,6 +2658,7 @@ var ar = class extends A {
 	feedbackRead = 0;
 	connection = 0;
 	saving = !1;
+	focusEditor = !1;
 	notice = "";
 	listLoading = !1;
 	viewLoading = !1;
@@ -2670,7 +2671,10 @@ var ar = class extends A {
 		this.connection++, this.feedbackRead++, this.listing?.abort(), this.reading?.abort(), this.comparing?.abort(), super.disconnectedCallback();
 	}
 	updated(e) {
-		e.has("revision") && e.get("revision") !== void 0 && this.refresh();
+		if (e.has("revision") && e.get("revision") !== void 0 && this.refresh(), this.focusEditor) {
+			let e = this.renderRoot.querySelector(".editor textarea");
+			e !== null && (this.focusEditor = !1, e.focus({ preventScroll: !0 }), e.closest(".editor")?.scrollIntoView({ block: "nearest" }));
+		}
 	}
 	updateContext(e) {
 		let t = ir(e);
@@ -2799,7 +2803,7 @@ var ar = class extends A {
 			}
 			let a = nr(e, r, t, i);
 			if (n.aborted) return;
-			this.view = a, this.editor !== void 0 && (this.editor.path !== e || this.editor.source !== r || this.editor.hash !== a.hash) && (this.notice = "The source changed. Your unfinished comment remains below, but cannot be saved against this snapshot. Copy its text before cancelling.");
+			this.view = a, this.editor !== void 0 && (this.focusEditor = !0), this.editor !== void 0 && (this.editor.path !== e || this.editor.source !== r || this.editor.hash !== a.hash) && (this.notice = "The source changed. Your unfinished comment remains in the draft editor, but cannot be saved against this snapshot. Copy its text before cancelling.");
 		} catch (e) {
 			n.aborted || (this.view = void 0, this.fail(e));
 		} finally {
@@ -2824,7 +2828,7 @@ var ar = class extends A {
 			hash: n.hash,
 			body: "",
 			...n.comparison === void 0 ? {} : { comparison: n.comparison }
-		} : this.notice = "Save or cancel the current comment first. Shift-click extends a range on the same side.", this.requestUpdate());
+		} : this.notice = "Save or cancel the current comment first. Shift-click extends a range on the same side.", this.focusEditor = !0, this.requestUpdate());
 	}
 	async saveEditor() {
 		let e = this.editor;
@@ -2864,7 +2868,7 @@ var ar = class extends A {
 				this.notice = "Save or cancel the current comment first.", this.requestUpdate();
 				return;
 			}
-			this.editor = { ...e }, this.comparing?.abort(), this.comparing = void 0, this.source = e.source, this.comparison = e.comparison, this.view = void 0, this.entries = [], e.comparison !== void 0 && (this.baseRef = e.comparison.baseRef, this.headRef = e.comparison.headRef, this.comparisonMode = e.comparison.mode), this.loadList(), this.openFile(e.path);
+			this.editor = { ...e }, this.focusEditor = !0, this.comparing?.abort(), this.comparing = void 0, this.source = e.source, this.comparison = e.comparison, this.view = void 0, this.entries = [], e.comparison !== void 0 && (this.baseRef = e.comparison.baseRef, this.headRef = e.comparison.headRef, this.comparisonMode = e.comparison.mode), this.loadList(), this.openFile(e.path);
 		}
 	}
 	async attachReview() {
@@ -2900,20 +2904,25 @@ var ar = class extends A {
 		}} @keydown=${(e) => {
 			e.key === "Enter" && (e.ctrlKey || e.metaKey) && !e.isComposing && (e.preventDefault(), this.saveEditor());
 		}}></textarea></label>
-      <div class="toolbar"><button ?disabled=${this.saving || !t || e.body.trim() === ""} @click=${() => {
+      <div class="toolbar"><button class="primary" ?disabled=${this.saving || !t || e.body.trim() === ""} @click=${() => {
 			this.saveEditor();
 		}}>Save comment</button>
       <button ?disabled=${this.saving} @click=${() => {
 			this.editor = void 0, this.notice = "", this.requestUpdate();
-		}}>Cancel edit</button></div>
+		}}>Cancel edit</button><span class="hint">Ctrl / ⌘ + Enter to save</span></div>
     </section>`;
 	}
 	render() {
-		let e = this.view;
-		return T`<section aria-label="Code Review">
-      <h2>Code Review</h2>
-      <p>Save comments, then attach a review chip to the selected conversation. Failed sends retain feedback; server acceptance clears unchanged submitted comments.</p>
-      <div class="toolbar"><label>Source <select .value=${this.source} @change=${(e) => {
+		let e = this.view, t = this.editor !== void 0 && this.editor.hash === e?.hash && rr(this.editor, e);
+		return T`<section class="review-panel" aria-label="Code Review">
+      <header class="review-header"><div><h2>Code Review</h2><p class="hint">Review files. Comment in context. Share when ready.</p></div>
+        <div class="review-actions"><span>${this.comments.length} saved · ${this.editor === void 0 ? "No draft" : "1 unsaved draft"}</span>
+        <button class="primary" ?disabled=${this.saving || this.comments.length === 0} @click=${() => {
+			this.attachReview();
+		}}>Attach review to composer</button>
+        <span class="hint">Saved comments only · send from the conversation</span></div>
+      </header>
+      <div class="source-controls"><div class="toolbar"><label>Source <select .value=${this.source} @change=${(e) => {
 			e.target instanceof HTMLSelectElement && this.selectSource(e.target.value);
 		}}>
         <option value="files">Files (including untracked)</option><option value="git-unstaged">Git unstaged</option><option value="git-staged">Git staged</option><option value="git-compare">Compare revisions</option>
@@ -2942,6 +2951,7 @@ var ar = class extends A {
 		}}><option value="merge-base">Branch changes since merge base (A...B)</option><option value="direct">Between revisions (A..B)</option></select></label>
         <button>Compare</button>
       </form><p>Compare committed work only. Use main → HEAD with merge base for a branch review, or a commit → HEAD with between revisions for accumulated work. Branch names, tags, commit IDs, and HEAD~3 are supported. Compare or Refresh resolves revisions again; opening files uses the frozen commits shown below.</p>` : null}
+      </div>
       ${this.error === "" ? null : T`<p role="alert">${this.error}</p>`}
       ${Object.values(this.feedbackErrors).every((e) => e === "") ? null : T`
         ${Object.values(this.feedbackErrors).filter(Boolean).map((e) => T`<p role="alert">${e}</p>`)}
@@ -2949,35 +2959,35 @@ var ar = class extends A {
 			this.reportFeedbackError("", "acceptance"), this.reportFeedbackError("", "action"), this.reportFeedbackError("", "read");
 		}}>Dismiss feedback warning</button>`}
       <p role="status" aria-live="polite">${this.listLoading || this.viewLoading || this.comparing !== void 0 ? "Loading…" : this.notice}</p>
-      <details class="file-list" open><summary>${this.source === "files" ? this.directory || "Workspace files" : `${Qn(this.source, this.comparison)} files`}</summary>
+      <div class="review-workspace"><nav aria-label="Review files"><details class="file-list" open><summary>${this.source === "files" ? this.directory || "Workspace files" : `${Qn(this.source, this.comparison)} files`}</summary>
         ${this.source === "files" && this.directory !== "" ? T`<button @click=${() => {
 			this.directory = this.directory.split("/").slice(0, -1).join("/"), this.loadList();
 		}}>Parent directory</button>` : null}
-        ${this.entries.map((e) => T`<button ?disabled=${this.listLoading} @click=${() => {
-			e.type === "directory" ? (this.directory = e.path, this.loadList()) : this.openFile(e.path);
-		}}>${e.type === "directory" ? "📁 " : ""}${e.name}</button>`)}
+        ${this.entries.map((t) => T`<button aria-current=${t.path === e?.path ? "page" : "false"} ?disabled=${this.listLoading} @click=${() => {
+			t.type === "directory" ? (this.directory = t.path, this.loadList()) : this.openFile(t.path);
+		}}>${t.type === "directory" ? "📁 " : ""}${t.name}</button>`)}
         ${!this.listLoading && this.comparing === void 0 && this.entries.length === 0 ? T`<p>No files in this view.</p>` : null}
-      </details>
-      ${e === void 0 ? null : T`<h3>${e.path} — ${Qn(e.source, e.comparison)}</h3><p>Click a line number, then Shift-click to extend its range, or edit the range below (works on touch screens).</p>
-        <div class="code" aria-label="Review source" aria-busy=${String(this.viewLoading)}>${e.rows.map((t) => {
-			let n = t.line === void 0 || t.side === void 0 ? void 0 : {
+      </details></nav><main class="viewer">
+      ${e === void 0 ? T`<div class="empty-state"><h3>${this.viewLoading ? "Opening file…" : "Choose a file to start reviewing"}</h3><p>Select a file from the list or open a workspace-relative path above.</p></div>` : T`<div class="file-heading"><h3>${e.path}</h3><span class="hint">${Qn(e.source, e.comparison)}</span><p class="hint">Click a line number to comment. Shift-click to extend the range, or use the editor’s line fields.</p></div>
+        <div class="code" aria-label="Review source" aria-busy=${String(this.viewLoading)}>${e.rows.map((n) => {
+			let r = n.line === void 0 || n.side === void 0 ? void 0 : {
 				path: e.path,
 				source: e.source,
-				side: t.side,
-				start: t.line,
-				end: t.line
-			}, r = n !== void 0 && this.editor?.path === e.path && this.editor.source === e.source && this.editor.hash === e.hash && this.editor.side === t.side && n.start >= this.editor.start && n.start <= this.editor.end, i = n === void 0 ? [] : this.comments.filter((n) => n.path === e.path && n.source === e.source && n.hash === e.hash && n.side === t.side && n.start === t.line);
-			return T`<div class=${`row ${t.kind}${r ? " selected" : ""}`}>
-            ${n === void 0 ? T`<span class="line"></span>` : T`<button class="line" aria-label=${`${t.side === "old" ? "Deleted" : "Current"} line ${String(t.line)}`} aria-pressed=${String(r)} ?disabled=${this.viewLoading} @click=${(e) => {
-				this.selectLine(t, e.shiftKey);
-			}}>${t.line}</button>`}
-            <code>${t.text || " "}</code></div>${i.map((e) => T`<div class="inline-comment">${fn(T, e.body)}</div>`)}`;
+				side: n.side,
+				start: n.line,
+				end: n.line
+			}, i = r !== void 0 && this.editor?.path === e.path && this.editor.source === e.source && this.editor.hash === e.hash && this.editor.side === n.side && r.start >= this.editor.start && r.start <= this.editor.end, a = r === void 0 ? [] : this.comments.filter((t) => t.path === e.path && t.source === e.source && t.hash === e.hash && t.side === n.side && t.start === n.line);
+			return T`<div class=${`row ${n.kind}${i ? " selected" : ""}`}>
+            ${r === void 0 ? T`<span class="line"></span>` : T`<button class="line" aria-label=${`${n.side === "old" ? "Deleted" : "Current"} line ${String(n.line)}`} aria-pressed=${String(i)} ?disabled=${this.viewLoading} @click=${(e) => {
+				this.selectLine(n, e.shiftKey);
+			}}>${n.line}</button>`}
+            <code>${n.text || " "}</code></div>${t && this.editor?.side === n.side && this.editor?.start === n.line ? this.renderEditor() : null}${a.map((e) => T`<div class="inline-comment">${fn(T, e.body)}</div>`)}`;
 		})}</div>${e.rows.length === 0 ? T`<p>No text lines to review.</p>` : null}`}
-      ${this.renderEditor()}
-      <h3>Saved comments (${this.comments.length})</h3>
-      <div class="toolbar"><button ?disabled=${this.saving || this.comments.length === 0} @click=${() => {
-			this.attachReview();
-		}}>Attach review to composer</button></div>
+      ${t ? null : this.renderEditor()}
+      </main></div>
+      <section class="saved-review" aria-label="Saved review"><h3>Saved comments (${this.comments.length})</h3>
+      <p class="hint">Saved locally until sent. Failed sends retain feedback; acceptance clears unchanged submitted comments.</p>
+      ${this.comments.length === 0 ? T`<p class="hint">No comments yet. Click a line number in the file viewer to add one.</p>` : null}
       ${this.comments.map((t) => T`<article><strong>${$n(t)}</strong>
         ${t.path === e?.path && t.source === e.source && t.hash !== e.hash ? T`<p class="stale">Source changed since this comment was saved. Verify its coordinates before sending.</p>` : null}
         <div class="markdown">${fn(T, t.body)}</div>
@@ -2988,18 +2998,46 @@ var ar = class extends A {
 		}}>Remove comment</button></div>
       </article>`)}
       <details><summary>Feedback Markdown</summary><textarea aria-label="Feedback Markdown" readonly rows="8" .value=${er(this.comments)}></textarea></details>
-    </section>`;
+    </section></section>`;
 	}
 	static styles = o`
     :host { display: block; color: var(--pi-text, #ddd); font: 14px system-ui, sans-serif; }
-    section { padding: 12px; } h2 { margin-top: 0; } p { line-height: 1.5; }
+    * { box-sizing: border-box; }
+    :host { container-type: inline-size; width: 100%; min-width: 0; }
+    .review-panel { padding: 16px; } h2, h3 { margin: 0; } p { line-height: 1.5; }
+    .hint { color: var(--pi-muted, #aaa); font-size: 12px; }
+    .review-header { display: flex; justify-content: space-between; align-items: center; gap: 16px; flex-wrap: wrap; border-bottom: 1px solid var(--pi-border, #666); padding-bottom: 16px; }
+    .review-header p { margin: 6px 0 0; }
+    .review-actions { display: flex; flex-direction: column; align-items: flex-end; gap: 6px; }
+    .source-controls { display: flex; flex-wrap: wrap; align-items: end; gap: 8px 20px; padding: 8px 0; }
+    .source-controls form { flex: 1; } .source-controls label { flex: 1; min-width: 140px; }
+    .source-controls > p { flex-basis: 100%; margin: 0; font-size: 12px; color: var(--pi-muted, #aaa); }
+    .review-workspace { display: grid; grid-template-columns: minmax(160px, 22%) minmax(0, 1fr); border: 1px solid var(--pi-border, #666); border-radius: 8px; overflow: hidden; }
+    nav { background: var(--pi-surface, #222); border-right: 1px solid var(--pi-border, #666); min-width: 0; padding: 12px; }
+    .viewer { min-width: 0; }
+    .file-heading { padding: 12px 16px; overflow-wrap: anywhere; } .file-heading p { margin-bottom: 0; }
+    .empty-state { padding: 48px 24px; text-align: center; color: var(--pi-muted, #aaa); }
+    .saved-review { margin-top: 24px; } summary { cursor: pointer; overflow-wrap: anywhere; }
+    [role=status]:empty { display: none; }
+    @container (max-width: 640px) {
+      .review-panel { padding: 10px; } .review-workspace { grid-template-columns: minmax(0, 1fr); }
+      nav { border-right: 0; border-bottom: 1px solid var(--pi-border, #666); }
+      .review-actions { align-items: flex-start; } .file-list { max-height: 150px !important; }
+    }
     .toolbar { display: flex; flex-wrap: wrap; align-items: end; gap: 8px; margin: 8px 0; }
     label { display: flex; flex-direction: column; gap: 4px; }
     button, select, input, textarea { color: inherit; background: var(--pi-surface, #222); border: 1px solid var(--pi-border, #666); border-radius: 4px; padding: 6px; font: inherit; }
-    button { cursor: pointer; } button:disabled { opacity: .5; cursor: default; } button:focus-visible { outline: 2px solid var(--pi-accent, #6cf); }
+    button { cursor: pointer; } button:hover:not(:disabled) { border-color: var(--pi-accent, #6cf); }
+    .primary { background: var(--pi-accent, #6cf); color: var(--pi-surface, #222); border-color: transparent; font-weight: 600; }
+    :is(button, input, select, textarea):focus-visible { outline: 2px solid var(--pi-accent, #6cf); outline-offset: 2px; }
+    button:disabled { opacity: .5; cursor: default; } button:focus-visible { outline: 2px solid var(--pi-accent, #6cf); }
     textarea { box-sizing: border-box; width: 100%; resize: vertical; } input[type=number] { width: 6em; }
-    .file-list { max-height: 220px; overflow: auto; margin: 12px 0; } .file-list button { display: block; text-align: left; margin: 3px 0; overflow-wrap: anywhere; }
-    .code { max-height: 55vh; overflow: auto; border: 1px solid var(--pi-border, #666); }
+    .file-list { max-height: 65vh; overflow: auto; } .file-list button { display: block; width: 100%; text-align: left; margin: 4px 0; overflow-wrap: anywhere; border-color: transparent; background: transparent; }
+    .file-list button[aria-current=page] { background: var(--pi-selection-bg, #336699); border-color: var(--pi-accent, #6cf); }
+    .code { max-height: 65vh; overflow: auto; border-top: 1px solid var(--pi-border, #666); }
+    .code .editor, .code .inline-comment { position: sticky; left: 0; width: calc(100% - 24px); margin: 10px 12px; background: var(--pi-surface, #222); }
+    .editor { border-left: 3px solid var(--pi-accent, #6cf) !important; }
+    .editor strong { display: block; margin-bottom: 8px; }
     .row { display: flex; min-width: max-content; } .row code { white-space: pre; padding: 2px 8px; font: 12px ui-monospace, monospace; }
     .line { flex: 0 0 4em; border-radius: 0; padding: 2px; font: 12px ui-monospace, monospace; text-align: right; }
     .add { background: #22883325; } .remove { background: #bb222225; } .meta { color: var(--pi-muted, #999); }

package/docs/compatibility.md

@@ -24,6 +24,10 @@ No production API migration was required for this release:
 
 The supported dependency range is `>=1.202610.3`; the lockfile records a concrete resolved version. Compatibility was checked against `1.202610.3`, not every later version. One test access was made null-safe because panel `state` is optional in the public contract. No production API migration was needed. The subsequent `0.1.1` patch adds the Review icon, lint cleanup, and branch/commit comparisons described in the [changelog](../CHANGELOG.md). Revision comparison uses only the existing scoped package-peer API and read-only Git runner; no additional host API is required.
 
+## UI patch 0.1.2
+
+The redesigned panel uses the same host APIs, storage format, and server operations as 0.1.1. Standalone Edge screenshots show the old and new components with identical sample content ([before](screenshots/review-before.png), [after](screenshots/review-after.png)). Browser rendering was checked separately from the DOM tests; this does not establish live-host behavior or narrow-panel acceptance.
+
 ## Validation
 
 After `npm ci --ignore-scripts`:
@@ -32,7 +36,7 @@ After `npm ci --ignore-scripts`:
 | --- | --- |
 | `npm run typecheck` | Passed |
 | `npm run lint` | Passed |
-| `npm test` | Passed: 6 files, 82 tests |
+| `npm test` | Passed: 6 files, 84 tests |
 | `npm run build` | Passed |
 | `npm run check:artifacts` | Passed |
 | `npm pack` and smoke check of extracted package | Passed |
@@ -50,6 +54,6 @@ All 38 original lint findings are resolved. Async disposal and callback-mutated
 1. Automated verification and packed smoke checks pass after lint cleanup; storage/ownership guards remain in place.
 2. Validate comparison controls and saved comparison comments on a live host, including moving branches and upgrading all older tabs. Complete the separate live-host checklist in [the release gate](composer-design.md#release-gate), especially successful/failed/chips-only sends, steer/follow-up, replacement during Send, selection changes, two-tab acknowledgements, and remote scoping. The newest host also adds pending-message controls; confirm locally queued messages do not clear saved feedback before server admission.
 3. The declared host range is `>=1.202610.3`. Only `1.202610.3` was checked here; later versions are allowed but have not been independently verified.
-4. Publication is enabled for the `0.1.1` patch release as `@compn3rd/pi-web-code-review`. The remaining validation is follow-up work; discovered defects will be addressed in patch releases.
+4. Publication is enabled for the `0.1.2` patch release as `@compn3rd/pi-web-code-review`. The remaining validation is follow-up work; discovered defects will be addressed in patch releases.
 
 Automated verification did not install the plugin into a live PI WEB instance or restart a daemon. Live-host testing remains separate from package publication.

package/docs/composer-design.md

@@ -4,9 +4,9 @@
 
 Target the public API on `jmfederico/pi-web` main at `830dc179`, including composer chips introduced in `02d24ec6`. The source imports `PluginPromptChip` directly from `@jmfederico/pi-web/plugin-api`; it does not recreate the missing API through a local shim or access private host code.
 
-The development dependency accepts PI WEB `>=1.202610.3`; compatibility was checked against published `1.202610.3`. Composer chips shipped in `1.202610.2`; the `1.202610.3` public declarations match this design. No speculative version, Git dependency, or local shim is used. Typechecking, all 82 tests, builds, and unpacked/packed artifact smoke checks pass. Full automated verification passes. See [compatibility results](compatibility.md).
+The development dependency accepts PI WEB `>=1.202610.3`; compatibility was checked against published `1.202610.3`. Composer chips shipped in `1.202610.2`; the `1.202610.3` public declarations match this design. No speculative version, Git dependency, or local shim is used. Typechecking, all 84 tests, builds, and unpacked/packed artifact smoke checks pass. Full automated verification passes. See [compatibility results](compatibility.md).
 
-The `0.1.1` patch release is enabled for publication. Live-host checks remain follow-up work rather than publication gates. Live-host runtime testing and installation are still deferred; the checks above validate declarations and standalone artifacts, not end-to-end send admission in a running host.
+The `0.1.2` patch release is enabled for publication. Live-host checks remain follow-up work rather than publication gates. Live-host runtime testing and installation are still deferred; the checks above validate declarations and standalone artifacts, not end-to-end send admission in a running host.
 
 Reference: [upstream composer documentation](https://github.com/jmfederico/pi-web/blob/830dc179/docs/plugins.md#attach-text-context-to-a-message).
 
@@ -49,6 +49,6 @@ Current progress against published PI WEB `1.202610.3`:
 2. Re-read the published contract and reconcile any differences from the recorded main revision. Do not assume main's design is frozen.
 3. Run `npm run verify` at the package root, followed by an actual packed-package smoke check. All 82 unit/DOM cases have passed, as have unpacked and packed artifact smoke checks. All 38 lint findings have been resolved and full `verify` now passes. The dev-only `fake-indexeddb` factory exercises the actual adapter on two connections, transaction aborts, migration precedence/empty markers, and open retry/late-success cleanup. Dependencies were installed with `--ignore-scripts`; no host installation or daemon restart was performed.
 4. In a separate test installation, exercise successful and failed sends, chips-only messages, steer/follow-up, selection changes, repeat attachment during an in-flight send, local edits/deletes, simultaneous two-tab saves/acknowledgements, failed/blocked/corrupt migration, retained legacy backups, persistent storage/acknowledgement warnings across remounts, async navigation/disposal, LAN HTTP without Web Locks, BroadcastChannel failure, Git with `diff.submodule=diff`, reload/restaging, disabled-plugin cleanup, and remote-machine scoping. Confirm the server-accepted callback clears only the correct saved feedback. Do not restart the developer's active session daemon as part of this check.
-5. Publication is enabled for version `0.1.1` as `@compn3rd/pi-web-code-review`, with PI WEB `>=1.202610.3` supported. Keep outstanding validation documented and address discovered defects in patch releases.
+5. Publication is enabled for version `0.1.2` as `@compn3rd/pi-web-code-review`, with PI WEB `>=1.202610.3` supported. Keep outstanding validation documented and address discovered defects in patch releases.
 
 There is no scheduled release watcher or automatic installation/restart associated with this gate.

package/docs/development.md

@@ -6,9 +6,9 @@ This repository starts from the plugin snapshot in [CompN3rd/pi-web at `d0c7a212
 
 ## Verification status
 
-The development dependency accepts PI WEB `>=1.202610.3`, which includes the public composer-chip API. The lockfile records the concrete version used for reproducible development. There are no local replacement declarations. Typechecking, all 82 tests, the build, and unpacked/packed artifact smoke checks pass. Full `verify` passes; live-host testing is still pending. See [compatibility results](compatibility.md).
+The development dependency accepts PI WEB `>=1.202610.3`, which includes the public composer-chip API. The lockfile records the concrete version used for reproducible development. There are no local replacement declarations. Typechecking, all 84 tests, the build, and unpacked/packed artifact smoke checks pass. Full `verify` passes; live-host testing is still pending. See [compatibility results](compatibility.md).
 
-The `0.1.1` patch release is enabled for publication with live-host testing tracked as follow-up work in the [validation checklist](composer-design.md#release-gate). Reproduce validation with:
+The `0.1.2` patch release is enabled for publication with live-host testing tracked as follow-up work in the [validation checklist](composer-design.md#release-gate). Reproduce validation with:
 
 ```sh
 npm ci --ignore-scripts
@@ -27,4 +27,4 @@ Prepared cases cover actual adapter serialization on two IndexedDB connections,
 - The Markdown renderer is adapted from PI WEB's MIT-licensed Captain's Log example. It renders known lexer tokens through Lit text bindings, not unsafe HTML.
 - Browser dependencies are bundled under `dist/browser`; server JavaScript remains outside that public directory. The whole plugin can be built separately from the host after the dependency gate is satisfied.
 
-Version `0.1.1` is packaged for public npm distribution as `@compn3rd/pi-web-code-review`. Release publication is separate from host installation and activation. Do not publish the PI WEB host to ship this plugin.
+Version `0.1.2` is packaged for public npm distribution as `@compn3rd/pi-web-code-review`. Release publication is separate from host installation and activation. Do not publish the PI WEB host to ship this plugin.

package/docs/usage.md

@@ -1,6 +1,6 @@
 # Code Review workflow
 
-Version **0.1.1** requires **PI WEB >=1.202610.3**. Automated tests, builds, and artifact smoke checks passed against `1.202610.3`; live-host testing remains outstanding. See [compatibility results](compatibility.md) for the validation limits.
+Version **0.1.2** requires **PI WEB >=1.202610.3**. Automated tests, builds, and artifact smoke checks passed against `1.202610.3`; live-host testing remains outstanding. See [compatibility results](compatibility.md) for the validation limits.
 
 ## Install the standalone package
 
@@ -40,7 +40,7 @@ For remote workspaces, install and enable the package on each target machine. Gi
 
 1. Select a workspace and open the **Review** tab (or **Open Code Review** in the action palette).
 2. Choose **Files**, **Git unstaged**, **Git staged**, or **Compare revisions**. Files includes untracked files. Browse directories or enter a workspace-relative file path. For committed work across multiple commits, use the comparison controls below.
-3. Click a line number. Shift-click extends its range on the same side; on touch screens, edit **Start line** and **End line** instead. Deleted diff lines use old-side coordinates; context and additions use new-side coordinates. Ranges cannot span omitted diff context.
+3. Click a line number to open and focus the comment editor directly below it inside the viewer. Shift-click extends its range on the same side; on touch screens, edit **Start line** and **End line** instead. Deleted diff lines use old-side coordinates; context and additions use new-side coordinates. Ranges cannot span omitted diff context.
 4. Enter Markdown and choose **Save comment**. Ctrl/Cmd+Enter also saves; ordinary Enter inserts a newline.
 5. Review the saved list. **Edit comment** reloads its source before allowing a save; **Remove comment** deletes that comment.
 
@@ -79,7 +79,7 @@ Refreshing or host file invalidation reloads the current snapshot (in comparison
 
 ## Attach feedback to the composer
 
-Choose a ready, unarchived conversation in this workspace, then select **Attach review to composer**. One removable **Review (N)** chip contains a snapshot of all saved comments. Repeating the action replaces that conversation's review chip, not the text at its cursor. You can inspect the payload under **Feedback Markdown**.
+Choose a ready, unarchived conversation in this workspace, then select **Attach review to composer** in the panel header. The header shows saved-comment and unsaved-draft counts; save your draft first if you want it included. One removable **Review (N)** chip contains a snapshot of all saved comments. Repeating the action replaces that conversation's review chip, not the text at its cursor. You can inspect the payload under **Feedback Markdown**.
 
 Send the chip with a message or by itself. Failed sends keep it and the saved comments for retry. Server acceptance clears only unchanged comments from the submitted snapshot; later edits and new comments survive. Acceptance does not mean the assistant has finished answering. Removing the composer chip only detaches it—saved comments remain available.
 

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@compn3rd/pi-web-code-review",
-  "version": "0.1.1",
+  "version": "0.1.2",
   "publishConfig": {
     "access": "public"
   },