githubnext/chopinPublic

Let's compose rich plans together

Polish document workspace and shared design system #423

MergedMaggieAppletonmerged 21 commits intomainfromrefs/pull/423/head
  1. 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.

    ![Document on the left and Chat on the right with compact header controls](https://github.com/user-attachments/assets/3006e893-bba8-451e-bef1-658653494f59)

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

    ![Unselected and selected Decisions buttons with square count badges and even spacing](https://github.com/user-attachments/assets/d367c268-6f53-4fd1-93e1-26e7a74c3761)

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

    ![Inline recovery warning, rounded workspace panels and preserved chat draft](https://github.com/user-attachments/assets/5b510f5c-9d07-4c31-a342-da651bb8b565)

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

    ![Research composer aligned with prose in normal document flow](https://github.com/user-attachments/assets/8a176a39-dfd7-4e22-b9de-f3438ec60902)

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

    ![Readable diagram with narrower layout and bottom-left playback](https://github.com/user-attachments/assets/6f03c3d6-0da9-45ac-8d0a-6140e4f3eadc)

    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.

    ![Four PR maintenance workflows removed while standard CI and scheduled review remain](https://github.com/user-attachments/assets/7bd45a4b-25b3-4369-95d4-f7e8913b7ea3)

    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.

  2. MaggieAppletonrenamed Polish document typography and workspace controls to Polish document workspace and shared design systemlast month on GitHub

  3. MaggieAppletonauto-rebase-enabledlast month on GitHub

  4. github-actions[bot]botadded label maintenance:workinglast month on GitHub

  5. <!-- 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 -->

  6. MaggieAppletonreferenced this in Run PR readiness on open non-draft PRs and provision test databases #424last month on GitHub

  7. github-actions[bot]botremoved label maintenance:workinglast month on GitHub

  8. github-actions[bot]botadded label needs-human-attentionlast month on GitHub

  9. MaggieAppletonmentionedlast month on GitHub

  10. MaggieAppletonsubscribedlast month on GitHub

  11. MaggieAppletonreferenced this in Exclude inherited base changes from rebase reviews #425last month on GitHub

  12. github-actions[bot]botremoved label needs-human-attentionlast month on GitHub

  13. github-actions[bot]botadded label maintenance:workinglast month on GitHub

  14. MaggieAppletonhead-ref-force-pushed bd5758a27603last month on GitHub

  15. MaggieAppletonremoved label maintenance:workinglast month on GitHub

  16. MaggieAppletonmerged b28882d25334last month on GitHub

  17. MaggieAppletonclosedlast month on GitHub

  18. MaggieAppletonhead-ref-deletedlast month on GitHub