Lane 8 page P4: a draft, Suggested wiki updates and Wiki health (WIKI-9, WIKI-10, WIKI-16) #921

Merged
claude-bot merged 6 commits from build/lane8-p4-drafts into main 2026-09-29 07:46:13 +00:00
Collaborator

Lane 8 page P4 (#734). Frontend only. Replaces WikiDraftReview.jsx and WikiProposals.jsx and deletes what only they used (legacyDraftBody.jsx, BlockedState.jsx, ENTRY_TYPE_COLOURS).

  • A draft (WIKI-9): the page beside the draft, each kind's heading and Approve, the stages in plain words, typed stats with how sure, Send it back, and Throw it away (which asks first).
  • A player's suggestion (IS-054) opens on the same page and is compared line by line with the page as she reads it (GET /lore/:id?as_user=). There is no fallback to the GM's text: if the page is no longer shared with her, it says so. Accepting merges as her on the server.
  • Suggested wiki updates (WIKI-10): where each session's suggestions stand, Pages and Relationships counted, one action at a time, failures shown on the row.
  • Wiki health (WIKI-16): five lists, each row with its fix; Merge starts IS-060's merge draft.
  • The Contents gains Wiki health for a GM, and Suggested wiki updates carries its count ("?" if it couldn't be counted).

Render check (real 390×844 and 1440×900 in the e2e stack, light and dark, as Dana; I looked at the shots): the first pass found two faults, both fixed in eb00c29b:

  • "Priya Raghunathan change" was missing its possessive, in the heading and in the Throw it away line.
  • A word wider than a phone ("Harbourmaster") pushed the page 4 px sideways. The shared Doc heading now breaks such a word; words that fit are never split.

After the fix: 20 of 20 shots have no sideways scroll. On a page with a :::gm block, Priya's side of her suggestion leaves the GM line out.

"Review the merge…" on a player's suggestion row matches the drawing (lane6c, p.kind === 'Update' || p.who).

Frontend lint, 2725 tests and the build pass. The backend is untouched; its Tuesday test failure is fixed by #919.

Part of #734.

🤖 Generated with Claude Code

Lane 8 page P4 (#734). Frontend only. Replaces `WikiDraftReview.jsx` and `WikiProposals.jsx` and deletes what only they used (`legacyDraftBody.jsx`, `BlockedState.jsx`, `ENTRY_TYPE_COLOURS`). - **A draft (WIKI-9):** the page beside the draft, each kind's heading and Approve, the stages in plain words, typed stats with how sure, Send it back, and Throw it away (which asks first). - **A player's suggestion (IS-054)** opens on the same page and is compared line by line with the page **as she reads it** (`GET /lore/:id?as_user=`). There is no fallback to the GM's text: if the page is no longer shared with her, it says so. Accepting merges as her on the server. - **Suggested wiki updates (WIKI-10):** where each session's suggestions stand, Pages and Relationships counted, one action at a time, failures shown on the row. - **Wiki health (WIKI-16):** five lists, each row with its fix; Merge starts IS-060's merge draft. - The Contents gains Wiki health for a GM, and Suggested wiki updates carries its count ("?" if it couldn't be counted). **Render check** (real 390×844 and 1440×900 in the e2e stack, light and dark, as Dana; I looked at the shots): the first pass found two faults, both fixed in eb00c29b: - "Priya Raghunathan change" was missing its possessive, in the heading and in the Throw it away line. - A word wider than a phone ("Harbourmaster") pushed the page 4 px sideways. The shared `Doc` heading now breaks such a word; words that fit are never split. After the fix: 20 of 20 shots have no sideways scroll. On a page with a `:::gm` block, Priya's side of her suggestion leaves the GM line out. "Review the merge…" on a player's suggestion row matches the drawing (`lane6c`, `p.kind === 'Update' || p.who`). Frontend lint, 2725 tests and the build pass. The backend is untouched; its Tuesday test failure is fixed by #919. Part of #734. 🤖 Generated with [Claude Code](https://claude.com/claude-code)
Lane 8 page P4. Replaces WikiDraftReview.jsx and WikiProposals.jsx, and
deletes what only they used (legacyDraftBody.jsx, BlockedState.jsx,
ENTRY_TYPE_COLOURS).

- Draft (WIKI-9): the page as the GM reads it beside the draft (the
  table's view, which never holds a block); each kind's heading, line and
  Approve; the four stages, stopped waiting, too soon to start over, did
  not finish in plain words, nothing to convert; typed stats with how
  sure, loose rows with who sees each; Send it back; Throw it away asks.
- A player's suggestion (IS-054) opens here too, compared line by line
  against the page as she reads it ("See it as"), never the GM's text.
- Suggestions (WIKI-10): where each session's suggestions stand, Pages
  and Relationships counted, one action at a time, failures on the row,
  a receipt with a way to the new page, Edit first, Discard asks.
- Health (WIKI-16): five lists, each row with its fix; Merge starts
  IS-060's merge draft; Keep both is kept on this device.
- Contents: Wiki health for a GM; Suggested wiki updates carries the
  queue's count, "?" when it could not be counted (WK-005, WK-017).
- Front page: "The full page" opens Wiki health; players' suggestions
  are in Waiting for you.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
fix(webapp): a suggestion says whose it is; a long name can't push a page sideways (#734)
Some checks failed
CI / Summarisation accuracy eval harness (stub provider) (pull_request) Successful in 59s
CI / Bot tests and audit (pull_request) Successful in 2m6s
CI / Backend lint (ruff) (pull_request) Successful in 32s
CI / Feedback site tests (pull_request) Successful in 46s
CI / Bot/backend version sync (pull_request) Successful in 30s
CI / Docker image build (pull_request) Successful in 55s
CI / Does this change touch the recording path? (pull_request) Successful in 21s
CI / Synthetic session harness (no GPU, no LLM) (pull_request) Has been skipped
CI / Backend migration, tests, and audit (pull_request) Failing after 12m6s
CI / Frontend tests, audit, and build (pull_request) Successful in 31m49s
CI / End-to-end journeys (desktop, phone, keyboard-only) (pull_request) Successful in 22m53s
eb00c29b72
"Priya Raghunathan's change" (and its Throw it away line) had lost the
possessive. The Doc heading breaks a word wider than the screen
("Harbourmaster" at 390 px ran 4 px off the edge); words that fit are
never split. Adds P4's changelog fragment.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Merge remote-tracking branch 'origin/main' into build/lane8-p4-drafts
All checks were successful
CI / Summarisation accuracy eval harness (stub provider) (pull_request) Successful in 1m9s
CI / Bot tests and audit (pull_request) Successful in 2m10s
CI / Backend lint (ruff) (pull_request) Successful in 33s
CI / Feedback site tests (pull_request) Successful in 46s
CI / Bot/backend version sync (pull_request) Successful in 29s
CI / Docker image build (pull_request) Successful in 30s
CI / Does this change touch the recording path? (pull_request) Successful in 21s
CI / Synthetic session harness (no GPU, no LLM) (pull_request) Has been skipped
CI / Backend migration, tests, and audit (pull_request) Successful in 19m37s
CI / Frontend tests, audit, and build (pull_request) Successful in 22m49s
CI / End-to-end journeys (desktop, phone, keyboard-only) (pull_request) Successful in 16m22s
1db86a75ac
Merge remote-tracking branch 'origin/main' into build/lane8-p4-drafts
Some checks failed
CI / Summarisation accuracy eval harness (stub provider) (pull_request) Successful in 1m8s
CI / Backend lint (ruff) (pull_request) Successful in 1m0s
CI / Feedback site tests (pull_request) Successful in 1m29s
CI / Bot/backend version sync (pull_request) Successful in 48s
CI / Docker image build (pull_request) Successful in 51s
CI / Does this change touch the recording path? (pull_request) Successful in 25s
CI / Synthetic session harness (no GPU, no LLM) (pull_request) Has been skipped
CI / Bot tests and audit (pull_request) Successful in 2m23s
CI / Backend migration, tests, and audit (pull_request) Successful in 18m49s
CI / Frontend tests, audit, and build (pull_request) Successful in 20m32s
CI / End-to-end journeys (desktop, phone, keyboard-only) (pull_request) Has been cancelled
49a60dac3d
# Conflicts:
#	webapp/frontend/src/App.jsx
#	webapp/frontend/src/shell/useContents.js
Merge remote-tracking branch 'origin/main' into build/lane8-p4-drafts
All checks were successful
CI / Summarisation accuracy eval harness (stub provider) (pull_request) Successful in 2m53s
CI / Bot tests and audit (pull_request) Successful in 4m40s
CI / Backend lint (ruff) (pull_request) Successful in 1m17s
CI / Feedback site tests (pull_request) Successful in 2m0s
CI / Bot/backend version sync (pull_request) Successful in 1m6s
CI / Does this change touch the recording path? (pull_request) Successful in 32s
CI / Synthetic session harness (no GPU, no LLM) (pull_request) Has been skipped
CI / Docker image build (pull_request) Successful in 1m1s
CI / End-to-end journeys (desktop, phone, keyboard-only) (pull_request) Successful in 17m31s
CI / Backend migration, tests, and audit (pull_request) Successful in 39m49s
CI / Frontend tests, audit, and build (pull_request) Successful in 40m31s
a7a43a9c42
# Conflicts:
#	webapp/frontend/src/App.jsx
Sign in to join this conversation.
No description provided.