Polish document workspace and shared design system #423
- MA
Polish the document workspace after live design feedback, with shared tokens and component variants owning the approved visual settings.
- Document: Lora 400 headings, 650px prose, 56px desktop/24px mobile gutters and 60px top padding. Research drafts, progress and reports now follow document flow at prose width. Restore rounded panel corners and make the title/menu controls a compact split control. - Navigation: move Chat to the right while preserving mounted state and keyboard order; tighten sidebar, header, menu and tooltip typography. Align sidebar labels and controls, preserve icon proportions, and use Option-N for new documents with typing/modal guards. Fetch failures use an inline warning with retry that preserves drafts. - Shared controls: finite icon-button, menu, count, empty-state and notice recipes; matching compact targets, square selected counts, circular empty states and lighter button outlines. Decision markers scroll with prose and use the lighter hover outline. Strengthen token checks without adding aesthetic exceptions. - SeeCode: remove code-block chrome and hints, place playback below the diagram, improve measured typography, and derive narrower layouts targeting 1100px where feasible. Keep source/connections stable and offer readable zoom/panning. Fix tagged UML classes clipping longer methods.
Agentation is development-only. Temporary jigs and planning material are excluded; obsolete placement code and implementation-mirror tests are removed. The unused standalone MCP dependency is removed from the app, resolving its Docker native-install failure.
Validation: two independent reviews with fixes; focused unit/geometry/contract tests, Chromium interaction/layout/recovery tests, formatting/lint/token checks, TypeScript, production build/bundle budget and full Docker build. Production output excludes Agentation.
The actual workspace components show Chat on the right, serif headings and compact controls.

Decision counts are square, evenly inset, and white when selected.

The warning reserves space above the workspace; the real chat draft remains available.

Research sits between authored paragraphs and uses the same prose column.

The actual diagram renderer shows a narrower graph at native scale with playback below it.

CI follow-up: align browser assertions with the approved 12px header glyphs and prose-width, inline research drafts; declare the empty-state icon’s rendered 24px size; wait for native diagram scrolling to finish. Keep private research keystrokes out of Lexical and restore editor focus after dismissal. The reopening test also checks for browser errors.
Retire PR CI Fixer, Rebase open PRs, Readiness Coordinator, and Readiness Worker. Remove their workflow sources, generated workflows, maintenance scripts, and unused instructions. Standard CI, issue triage, and Weekly React Effect Review remain. All four maintenance workflows were disabled immediately.

The language-menu focus test now installs its clock before navigation and pauses at a fixed future time, eliminating a runner/browser clock race. Five consecutive local browser repetitions passed.
Production preview exclusion: move the audit-only destructive-icon selector from the shared theme into the development audit stylesheet. The production build and exclusion check pass across all 429 assets. CI now builds once and runs that check before browser integration.
MaggieAppletonrenamed Polish document typography and workspace controls to Polish document workspace and shared design systemlast month on GitHub
MaggieAppletonauto-rebase-enabledlast month on GitHub
github-actions[bot]added label maintenance:workinglast month on GitHub
<!-- chopin:pr-maintenance:v1 --> **PR maintenance: working**
rebase in progress.
Inspected head: 59fd55353d05cb2d02e4fd4cf74230cb9dd454c2
Base head: 025d98e324f888a40854a316dce199ec0cb3db64
[PR readiness runs](https://github.com/githubnext/chopin/actions/workflows/pr-readiness.yml) · [CI run](https://github.com/githubnext/chopin/actions/runs/37906560311) <!-- chopin:pr-maintenance-auth:eyJwYXlsb2FkIjp7InNjaGVtYVZlcnNpb24iOjEsInJlcG9zaXRvcnkiOiJnaXRodWJuZXh0L2Nob3BpbiIsInJldmlzaW9uIjowLCJwcnMiOnsiNDIzIjp7ImJvZHkiOiI8IS0tIGNob3Bpbjpwci1tYWludGVuYW5jZTp2MSAtLT5cbioqUFIgbWFpbnRlbmFuY2U6IHdvcmtpbmcqKlxuXG5yZWJhc2UgaW4gcHJvZ3Jlc3MuXG5cbkluc3BlY3RlZCBoZWFkOiA1OWZkNTUzNTNkMDVjYjJkMDJlNGZkNGNmNzQyMzBjYjlkZDQ1NGMyXG5cbkJhc2UgaGVhZDogMDI1ZDk4ZTMyNGY4ODhhNDA4NTRhMzE2ZGNlMTk5ZWMwY2IzZGI2NFxuXG5bUFIgcmVhZGluZXNzIHJ1bnNdKGh0dHBzOi8vZ2l0aHViLmNvbS9naXRodWJuZXh0L2Nob3Bpbi9hY3Rpb25zL3dvcmtmbG93cy9wci1yZWFkaW5lc3MueW1sKSDCtyBbQ0kgcnVuXShodHRwczovL2dpdGh1Yi5jb20vZ2l0aHVibmV4dC9jaG9waW4vYWN0aW9ucy9ydW5zLzM3OTA2NTYwMzExKSIsImJsb2NrZXJJZCI6bnVsbH19fSwic2lnbmF0dXJlIjoiZGY3YjljZjBiNjY4ODE4ZTU1YWQyZmVkYTQ5NGZiMTQwMWIzMjY0ZTg3YmQ3YmQ4MmYyNDdmYzkzODQ0NmJkMSJ9 -->
MaggieAppletonreferenced this in Run PR readiness on open non-draft PRs and provision test databases #424last month on GitHub
github-actions[bot]removed label maintenance:workinglast month on GitHub
github-actions[bot]added label needs-human-attentionlast month on GitHub
MaggieAppletonmentionedlast month on GitHub
MaggieAppletonsubscribedlast month on GitHub
MaggieAppletonreferenced this in Exclude inherited base changes from rebase reviews #425last month on GitHub
github-actions[bot]removed label needs-human-attentionlast month on GitHub
github-actions[bot]added label maintenance:workinglast month on GitHub
MaggieAppletonhead-ref-force-pushed bd5758a27603last month on GitHub
MaggieAppletonremoved label maintenance:workinglast month on GitHub
MaggieAppletonmerged b28882d25334last month on GitHub
MaggieAppletonhead-ref-deletedlast month on GitHub