githubnext/chopinPublic

Let's compose rich plans together

Add saved OpenUI comparison sections to documents #414

ClosedMaggieAppletonclosed 8 commits intomainfromrefs/pull/414/head
  1. MA
    MaggieAppleton

    ## What changed

    - Add one saved `openui-options` composition to ordinary documents: image or text previews, a comparison table, expandable details, and a per-reader category filter. - Validate a bounded OpenUI declaration graph before rendering. The canonical source stays editable and survives malformed input; a valid edit restores the preview. - Load the pinned OpenUI runtime only for documents with this fence. - Refine reading in the document: compact media-and-copy rows where space allows, denser comparisons, visible narrow-table scroll guidance, and keyboard filter navigation. The authoring syntax and saved data stay the same.

    ## Preview

    These are browser captures of saved documents. The pandas example compares current output with a proposed plain default; the pagination example is synthetic and uses a comparison-first order.

    ### Pandas at document width

    | Before | After | | --- | --- | | ![Pandas comparison before: large stacked media cards](https://github.com/user-attachments/assets/b67a66f3-78c1-4e43-b5bf-e6105ec60a67) | ![Pandas comparison after: compact media-and-copy rows and denser table](https://github.com/user-attachments/assets/91dddac7-ab9e-4e61-9dfb-8b8f01288317) |

    ### Pagination at narrow width

    | Before | After | | --- | --- | | ![Narrow pagination comparison before: horizontally clipped table without cue](https://github.com/user-attachments/assets/2de3a0be-9e58-4bab-a278-61145e2cb77e) | ![Narrow pagination comparison after: scroll cue, pinned option names, and compact details](https://github.com/user-attachments/assets/7d109c1a-6505-46d4-a844-1b32e9b8f615) |

    ## Verification

    - `bun run types`, `bun run ci`, and `bun run build` passed at the design milestone. - Focused code, OpenUI schema, and source-offset tests passed (49 tests). - Chromium composition suite passed (4/4): two saved content sets, keyboard and read-only filters, horizontal lanes, source repair, reload, and diagram coexistence. Screenshots require decoded image pixels.

    The two test documents were manually authored. A separate Planner check will assess live model-authored source against the documented grammar; this PR does not claim that result.

  2. MADocument bounded OpenUI composition contract884ad7f
  3. MARender bounded OpenUI comparison sections in documents40d53d8
  4. MAAlign comparison media privacy and responsive styles633a166

  5. MaggieAppletonreferenced this in Route Planner visual authoring through Jev #415last month on GitHub

  6. MAWait for OpenUI images in opt-in screenshots33c9660
  7. MAMerge pinned main into OpenUI composition branch7920e9c
  8. MAMerge current main into OpenUI composition branch584e50f
  9. MARefine comparison composition for document readingaac75fa
  10. MAUse shared focus styling for OpenUI options6681562

  11. MaggieAppletonready-for-reviewlast month on GitHub

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

  13. github-actions[bot]bot(edited)

    <!-- chopin:pr-maintenance:v1 --> **PR maintenance: working**

    rebase in progress.

    Inspected head: 66815629938c6ffb89693de6bd44a337c145563d

    Base head: 7c690cbcfdf23f1e1925f493bd4e10949afcb76d

    [PR readiness runs](https://github.com/githubnext/chopin/actions/workflows/pr-readiness.yml) · [CI run](https://github.com/githubnext/chopin/actions/runs/37914650272) <!-- chopin:pr-maintenance-auth:eyJwYXlsb2FkIjp7InNjaGVtYVZlcnNpb24iOjEsInJlcG9zaXRvcnkiOiJnaXRodWJuZXh0L2Nob3BpbiIsInJldmlzaW9uIjowLCJwcnMiOnsiNDE0Ijp7ImJvZHkiOiI8IS0tIGNob3Bpbjpwci1tYWludGVuYW5jZTp2MSAtLT5cbioqUFIgbWFpbnRlbmFuY2U6IHdvcmtpbmcqKlxuXG5yZWJhc2UgaW4gcHJvZ3Jlc3MuXG5cbkluc3BlY3RlZCBoZWFkOiA2NjgxNTYyOTkzOGM2ZmZiODk2OTNkZTZiZDQ0YTMzN2MxNDU1NjNkXG5cbkJhc2UgaGVhZDogN2M2OTBjYmNmZGYyM2YxZTE5MjVmNDkzYmQ0ZTEwOTQ5YWZjYjc2ZFxuXG5bUFIgcmVhZGluZXNzIHJ1bnNdKGh0dHBzOi8vZ2l0aHViLmNvbS9naXRodWJuZXh0L2Nob3Bpbi9hY3Rpb25zL3dvcmtmbG93cy9wci1yZWFkaW5lc3MueW1sKSDCtyBbQ0kgcnVuXShodHRwczovL2dpdGh1Yi5jb20vZ2l0aHVibmV4dC9jaG9waW4vYWN0aW9ucy9ydW5zLzM3OTE0NjUwMjcyKSIsImJsb2NrZXJJZCI6bnVsbH19fSwic2lnbmF0dXJlIjoiMGEzMmQyZWIxN2RkOTFmMTAzN2RmMjYzZGFmY2Q1YjYxOWViNGJkYjZjNTU3M2ViZjQ0YTU1MWJjY2NkY2YyYiJ9 -->

  14. MaggieAppletonconvert-to-draftlast month on GitHub

  15. MaggieAppletonready-for-reviewlast month on GitHub

  16. MA
    MaggieAppleton
    Author

    Closing after design review. Chopin already lets the Planner select from its allowed MDX components in a collaborative document. This PR adds a separate OpenUI authoring language for a comparison interface that can be composed from native MDX elements; we are not proceeding with this OpenUI implementation.

  17. MaggieAppletonclosedlast month on GitHub