@compn3rd/pi-web-code-review 0.1.1 → 0.2.0
This diff represents the content of publicly available package versions that have been released to one of the supported registries. The information contained in this diff is provided for informational purposes only and reflects changes between package versions as they appear in their respective public registries.
- package/CHANGELOG.md +41 -0
- package/README.md +24 -5
- package/dist/browser/index.js +780 -592
- package/docs/compatibility.md +29 -41
- package/docs/composer-design.md +5 -5
- package/docs/development.md +14 -3
- package/docs/screenshots/0.2.0/focus-desktop.png +0 -0
- package/docs/screenshots/0.2.0/focus-light.png +0 -0
- package/docs/screenshots/0.2.0/focus-phone.png +0 -0
- package/docs/screenshots/0.2.0/phone-compare.png +0 -0
- package/docs/screenshots/0.2.0/phone-editor.png +0 -0
- package/docs/screenshots/0.2.0/phone-files.png +0 -0
- package/docs/screenshots/0.2.0/phone-notes.png +0 -0
- package/docs/screenshots/0.2.0/phone-settings.png +0 -0
- package/docs/screenshots/0.2.0/workbench-desktop.png +0 -0
- package/docs/screenshots/0.2.0/workbench-light.png +0 -0
- package/docs/screenshots/0.2.0/workbench-phone.png +0 -0
- package/docs/screenshots/review-after.png +0 -0
- package/docs/screenshots/review-before.png +0 -0
- package/docs/usage.md +15 -5
- package/package.json +7 -5
package/CHANGELOG.md
CHANGED
|
@@ -1,5 +1,46 @@
|
|
|
1
1
|
# Changelog
|
|
2
2
|
|
|
3
|
+
## 0.2.0
|
|
4
|
+
|
|
5
|
+
### Added
|
|
6
|
+
|
|
7
|
+
- Switchable **Workbench** and **Focus** layouts in the panel’s **Review settings**, with browser-local layout and code-wrap preferences.
|
|
8
|
+
- Workbench file navigation and saved-feedback rail; Focus inline feedback, file switcher and collapsible review summary. Layout changes preserve the active file, range and draft.
|
|
9
|
+
- Phone file/notes sheets, touch-sized controls, wrapped code, numeric range disclosure, and responsive source/comparison controls.
|
|
10
|
+
- Production-bundle Chromium checks and dark/light desktop/phone screenshots. Regression coverage increases from 84 to 99 tests.
|
|
11
|
+
|
|
12
|
+
### Changed
|
|
13
|
+
|
|
14
|
+
- Require PI WEB **>=1.202610.6** and migrate the browser entry to API **v5**.
|
|
15
|
+
- Opt into public `stylesheet-integration` v1 with host shared UI primitives and theme-aware, component-scoped CSS. Server API v3 and read-only Git operations are unchanged.
|
|
16
|
+
- Compact source selection and move path/help/range details behind disclosures. Keep saved comments and unsaved drafts distinct; attaching still does not send.
|
|
17
|
+
- Update Vite/Vitest development dependencies to patched versions; dependency audit is clean.
|
|
18
|
+
- Exclude design-study source and prototype assets from the published package; ship production screenshots and user documentation.
|
|
19
|
+
|
|
20
|
+
### Validation and limits
|
|
21
|
+
|
|
22
|
+
- Typecheck, lint, 99 tests, builds, unpacked/packed artifact smoke checks, and production-bundle browser checks pass.
|
|
23
|
+
- Chromium checks cover both layouts at 360/390/430px, dark/light palettes, wrapping, dialogs, draft preservation, range editing and mocked attachment.
|
|
24
|
+
- Live-host shared styling, real phone keyboards and end-to-end send/remote acceptance remain follow-up checks. No daemon restart or installation is part of release preparation.
|
|
25
|
+
- Comment schema, IndexedDB data and composer acceptance semantics are unchanged; no comment migration is required.
|
|
26
|
+
|
|
27
|
+
## 0.1.2 — 2026-10-16
|
|
28
|
+
|
|
29
|
+
### Changed
|
|
30
|
+
|
|
31
|
+
- Open and focus comment editors directly beneath the selected code line, inside the file viewer, instead of below the entire file.
|
|
32
|
+
- Use a full-width header with saved-comment and unsaved-draft status, and move **Attach review to composer** to the header.
|
|
33
|
+
- Place file navigation beside the viewer on wide panels and stack it above the viewer on narrow panels.
|
|
34
|
+
- Highlight the active file and improve empty states, keyboard hints, focus indicators, and source-control layout.
|
|
35
|
+
- Keep changed-snapshot drafts accessible outside the code view without losing their text or allowing an invalid save.
|
|
36
|
+
|
|
37
|
+
### Validation and compatibility
|
|
38
|
+
|
|
39
|
+
- Add regression tests for inline editor placement, focus, header action placement, active-file state, and changed-snapshot draft preservation (84 tests total).
|
|
40
|
+
- Capture browser-rendered before/after screenshots with the same sample file and draft; these are standalone previews, not live-host acceptance tests.
|
|
41
|
+
- No host API, storage format, or server behavior changes. Requirements remain PI WEB `>=1.202610.3` and Node.js `>=22.19.0`.
|
|
42
|
+
- Live-host acceptance and remote-machine testing remain outstanding; see [compatibility results](docs/compatibility.md).
|
|
43
|
+
|
|
3
44
|
## 0.1.1 — 2026-10-15
|
|
4
45
|
|
|
5
46
|
Package: **`@compn3rd/pi-web-code-review`** (public npm scope).
|
package/README.md
CHANGED
|
@@ -4,20 +4,39 @@ A standalone Review tab for workspace files, staged/unstaged Git changes, and ac
|
|
|
4
4
|
|
|
5
5
|
## Requirements and installation
|
|
6
6
|
|
|
7
|
-
|
|
7
|
+
Version **0.2.0** requires **PI WEB >=1.202610.6**, Node.js **>=22.19.0**, and Git for staged/unstaged and revision-comparison review. Tested against PI WEB `1.202610.6`; later releases have not been independently verified.
|
|
8
8
|
|
|
9
9
|
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).
|
|
10
10
|
|
|
11
11
|
This package includes a server plugin: restart the target session daemon when safe, then reload the browser. Restarting interrupts active sessions and terminals; do it from outside the sessions that daemon hosts. Install the package on each remote machine where you want to review files.
|
|
12
12
|
|
|
13
|
+
## Two review layouts
|
|
14
|
+
|
|
15
|
+
Open **Review settings** in the panel header to choose:
|
|
16
|
+
|
|
17
|
+
- **Workbench** — persistent file navigation, code viewer, and saved-feedback rail on wide panels. On phones, files and notes open in sheets.
|
|
18
|
+
- **Focus** — a wide code view with inline feedback, a file switcher, and a collapsible saved-review summary.
|
|
19
|
+
|
|
20
|
+
The layout and **Wrap code** preference are saved for this browser, across workspaces. Switching layouts preserves the current file, line range, and unsaved draft. Unsaved drafts are not persisted across browser reloads or workspace changes.
|
|
21
|
+
|
|
22
|
+

|
|
23
|
+
|
|
24
|
+
[Focus layout](docs/screenshots/0.2.0/focus-desktop.png) · [Phone Workbench](docs/screenshots/0.2.0/workbench-phone.png) · [Phone Focus](docs/screenshots/0.2.0/focus-phone.png) · [Phone settings](docs/screenshots/0.2.0/phone-settings.png)
|
|
25
|
+
|
|
26
|
+
Screenshots render the production bundle with sample backend data, not a live PI WEB host. Phone captures use Chromium at phone viewport sizes, not real devices.
|
|
27
|
+
|
|
13
28
|
## Review workflow
|
|
14
29
|
|
|
15
|
-
|
|
30
|
+
Choose **Files**, **Unstaged**, **Staged**, or **Compare**. 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.
|
|
31
|
+
|
|
32
|
+
Click or tap a line number to open an inline comment editor. Shift-click extends the range; on phones, expand **Line range** to edit the coordinates. Save your comments, select a ready conversation, and choose **Attach review to composer**. This stages a removable **Review (N)** chip; it does not send a message or include unsaved drafts. Removing the chip preserves saved comments. Server acceptance clears only unchanged submitted comments.
|
|
33
|
+
|
|
34
|
+
## Validation
|
|
16
35
|
|
|
17
|
-
|
|
36
|
+
Typechecking, lint, **99 tests**, builds, and unpacked/packed artifact smoke checks pass. Production-bundle Chromium checks cover dark/light palettes, phone widths **360/390/430px**, wrapping, settings, draft preservation, sheets, keyboard modality, range editing and mocked composer attachment. `npm audit` reports no known vulnerabilities in the resolved dependency tree.
|
|
18
37
|
|
|
19
|
-
|
|
38
|
+
Live-host shared styling, real phone keyboards, remote-machine behavior and end-to-end send acceptance remain follow-up validation; they are not claimed as completed.
|
|
20
39
|
|
|
21
|
-
See the [
|
|
40
|
+
See the [usage guide](docs/usage.md), [development guide](docs/development.md), [compatibility results](docs/compatibility.md), and [composer design](docs/composer-design.md).
|
|
22
41
|
|
|
23
42
|
Adapted from the Code Review UI branch and PI WEB's MIT-licensed plugin examples; see [LICENSE](LICENSE).
|