docs: v0.9.0 — interactive HTML exports - #72
Conversation
project820
left a comment
There was a problem hiding this comment.
🤖 Psycho-842 [BOT] review
⚖️ Verdict: CONSIDER TO MERGE — Docs/version-only release notes for already-merged #71/#68; tip CI green, no unresolved P0–P2.
Summary
Bumps the package to 0.9.0 and documents interactive HTML exports (theme toggle, slide nav/print, self-contained controls, CSP network block) plus a v0.8.0 backfill for the narrowed model lineup and GPT Fast toggle from #68. No application logic in the diff — release packaging + docs only. Claims spot-checked against main (html-export-runtime.ts, model allowlist, wizard Fast mode) and match shipped behavior.
Findings (by severity)
- 🟡 P3 Low — PR body
Closes #70, but #70 is already CLOSED (closed by merged #71). Harmless no-op on merge; prefer dropping the closer or linkingRelated: #70to avoid a redundant autoclose event. - 💡 SUGGESTION — CHANGELOG compare anchor
[0.9.0]: …/compare/v0.8.0...mainis fine pre-tag; after cuttingv0.9.0, point it at the tag range (same pattern you just applied to0.8.0).
No P0–P2 findings. No overlapping open PRs on the same files. No unresolved review threads. Other-bot findings: none on this PR.
📋 Detailed analysis (CI, diff stats, verification)
Meta (measured)
- HEAD:
9b18750c7b8fbab9bb6cbc6e8137639062858670 - Base:
main· Branch:docs/v0.9.0-interactive - Diff: +43 / −7 · 6 files · mergeable=
MERGEABLE· mergeStateStatus=CLEAN - Author: project820 · Draft: false · Open PRs in repo: only #72
- Related: #71 merged (feat interactive export runtime) · #68 merged (model lineup) · #70 already closed
Files
| Path | Change |
|---|---|
package.json / package-lock.json |
0.8.0 → 0.9.0 only |
CHANGELOG.md |
New 0.9.0 section; backfill one 0.8.0 bullet; compare links |
README.md |
v0.9.0 blurb + interactive export / model notes |
docs/releases/v0.9.0.md |
ADDED — theme / slides / interactive / CSP |
docs/releases/v0.8.0.md |
Model lineup + Fast option backfill |
CI (HEAD tip)
verify(workflow CI): SUCCESS on9b18750— https://github.com/project820/notepad-ai/actions/runs/29709988043- No failed/missing required checks on tip
Docs ↔ code spot-check (main @ #71)
- Theme ☀/🌙 +
localStorage+data-theme: present insrc/main/html-export-runtime.ts - Slide keys ←/→/PageUp/PageDown/Space, prev/next, indicator, print restore: present
- CSP
connect-src 'none'; form-action 'none'; … script-src 'unsafe-inline': present - HTML export models ChatGPT Sol/Terra/Luna + Claude + Grok + local:
html-export-model-allowlist.ts - GPT Fast checkbox /
reasoningEffort: 'low':html-export-wizard.ts
Local tests
- Not re-run here — docs/version-only PR; remote tip
verifySUCCESS cited instead. (Docs/config-only exempt from full code triple-gate evidence floor.)
Adversarial pass (docs release)
- Does the version bump ship unfinished code? — No code paths changed; feature already on main via #71.
- Do release notes over-claim security or behavior? — CSP/network block and injected runtime match source; no secret/path leaks in diff.
- Broken changelog anchors / wrong closes? — Compare link pre-tag is conventional; only real nit is redundant
Closes #70(P3).
All three rejected as merge blockers.
Cross-reference
- No other open PR touches these files.
Closes #70targets an already-closed issue (see P3).- No known duplicate docs PR for v0.9.0.
Policy note
This COMMENT is advisory only — not an approval. User final sign-off still required before merge.
🧭 CONSIDER TO MERGE
🦀 Reviewed by Hermes agent BOT 🦞
project820
left a comment
There was a problem hiding this comment.
🤖 Psycho-842 [BOT] review
⚖️ Verdict: MERGE WITH CAUTION — [FINAL @9b18750] Settlement confirmed: tip CI green, no unresolved P0–P2; only leftover is P3 (redundant Closes #70).
Summary
Docs/version-only release notes for already-shipped #71/#68. Stage 1 🧭 held through monitoring; re-measured HEAD 9b18750 is still CLEAN + verify SUCCESS. Outstanding finding is P3 only → Stage 2
Findings (by severity)
- 🟡 P3 Low — still open: PR body
Closes #70but #70 is already CLOSED (via #71). Harmless no-op on merge.
No P0–P2. No unresolved review threads. No other-bot findings. No overlapping open PRs.
📋 Settlement re-measure
Settlement check (measured @ 2026-07-20 ~01:32Z)
- HEAD unchanged:
9b18750c7b8fbab9bb6cbc6e8137639062858670 - mergeable=
MERGEABLE· mergeStateStatus=CLEAN - tip CI
verify: SUCCESS (run 29709988043) - reviewThreads unresolved: 0
- stability: last commit 00:47Z, Stage 1 review 00:51Z → >20 min quiet
- docs/version-only (triple-gate code floor N/A)
Prior review: #72 (review)
Policy note: Advisory only — not an approval. User final sign-off still required.
⚠️ MERGE WITH CAUTION
🦀 Reviewed by Hermes agent BOT 🦞
What
Why
#68 shipped in the v0.8.0 tag but was absent from its documentation; #71 landed after that tag and needs a separate release section.
Verification
npm run typecheckgit diff --checkCloses #70