@marimo-team/islands 0.25.3-dev17 → 0.25.3-dev173
This diff represents the content of publicly available package versions that have been released to one of the supported registries. The information contained in this diff is provided for informational purposes only and reflects changes between package versions as they appear in their respective public registries.
- package/dist/{ConnectedDataExplorerComponent-CNMliVzw.js → ConnectedDataExplorerComponent-Le_OBfmt.js} +21 -19
- package/dist/{ErrorBoundary-jaGao4LA.js → ErrorBoundary-wT3f3iJ1.js} +19 -19
- package/dist/{abnfDiagram-VCTEODGH-D-Ez91gV.js → abnfDiagram-VCTEODGH-Tf1ghyQQ.js} +2 -2
- package/dist/any-language-editor-B92WMzoC.js +75 -0
- package/dist/{architectureDiagram-5GKGNRK7-CTpcNtO8.js → architectureDiagram-5GKGNRK7-xlnbTG0T.js} +1 -1
- package/dist/{arrays-CN1djC4k.js → arrays-R6JsImiT.js} +8 -0
- package/dist/assets/__vite-browser-external-DVeFf2e7.js +1 -0
- package/dist/assets/worker-C1pfNBsx.js +113 -0
- package/dist/{button-CvjkwdSk.js → button-BQRdqQi1.js} +199 -192
- package/dist/{capabilities-CQ5IjhzL.js → capabilities-CI4xUPsp.js} +2 -2
- package/dist/{chat-ui-DaHynk8g.js → chat-ui-CuORr6fL.js} +1142 -1141
- package/dist/{check-EDwSwP89.js → check-C9Mp9eCK.js} +1 -1
- package/dist/{chunk-IMKFNOWR-DGxi-zeo.js → chunk-IMKFNOWR-Cmd4a5jy.js} +1 -1
- package/dist/{chunk-SHT3W25Y-CV9J3rAB.js → chunk-SHT3W25Y-mSNp-WpO.js} +1 -1
- package/dist/{chunk-SVP7TREG-BL-9Fqgg.js → chunk-SVP7TREG-CWC49D8X.js} +1 -1
- package/dist/{chunk-TICWLB2K-Cu_XMjDM.js → chunk-TICWLB2K-De9FINtG.js} +1 -1
- package/dist/{chunk-YOKDWASO-DPZk8vhS.js → chunk-YOKDWASO-DKplTrBY.js} +20 -20
- package/dist/{classDiagram-ZZMXUADV-BXXV13Hq.js → classDiagram-ZZMXUADV-DweTkYdp.js} +2 -2
- package/dist/{classDiagram-v2-VYDZK3BY-B7FXJ1Cp.js → classDiagram-v2-VYDZK3BY-A6MR2TiE.js} +2 -2
- package/dist/{code-visibility-Bub8tnaA.js → code-visibility-ZReaeMaF.js} +25 -25
- package/dist/{common-CdT0nyN1.js → common-DCevHg1m.js} +6917 -6738
- package/dist/{copy-icon-Ft7t9bqz.js → copy-icon-De584pZ4.js} +3 -3
- package/dist/{cynefinDiagram-5FMLGOSQ-CH96rwba.js → cynefinDiagram-5FMLGOSQ-BSvo5le-.js} +1 -1
- package/dist/{dagre-GXQ25YYZ-BD3PmpFY.js → dagre-GXQ25YYZ-tXRQslO6.js} +1 -1
- package/dist/{data-grid-overlay-editor-Dbcss6_2.js → data-grid-overlay-editor-CTC-bJus.js} +5 -5
- package/dist/{data-grid-types-BH_r7g7l.js → data-grid-types-0PJ-9N5n.js} +1 -1
- package/dist/{diagram-S7CK7UJ4-BsSJJldB.js → diagram-S7CK7UJ4-r3t5VtwI.js} +1 -1
- package/dist/{diagram-UQ7AKVKN-muQtNZc4.js → diagram-UQ7AKVKN-qgbC8YnT.js} +1 -1
- package/dist/{diagram-VX7I27RA-CdhuZAqR.js → diagram-VX7I27RA-CiHxEwtp.js} +1 -1
- package/dist/{diagram-Z3DM3KII-BD8mXjOj.js → diagram-Z3DM3KII-D95u15Ev.js} +1 -1
- package/dist/{dialog-CEBFZNyc.js → dialog-DVRfQuhD.js} +25 -25
- package/dist/{dist-DpBpA_n1.js → dist-Lm-D-Vr6.js} +3 -3
- package/dist/{dropdown-menu-CP_qCYHC.js → dropdown-menu-DCMbbQ35.js} +1481 -1433
- package/dist/{ebnfDiagram-PWID7BFC-BIN1sweE.js → ebnfDiagram-PWID7BFC-P7e4bryK.js} +2 -2
- package/dist/{erDiagram-RLTQ6QDP-Ctp5cfoN.js → erDiagram-RLTQ6QDP-CUHZcmRj.js} +1 -1
- package/dist/{error-banner-D-WCIiMt.js → error-banner-DxJ3s8YO.js} +86 -86
- package/dist/{error-entries-Bv6bZPF3.js → error-entries-CmWMEsY_.js} +5756 -5643
- package/dist/{feedback-button-B3Wrnsf_.js → feedback-button-DwXkmewo.js} +35 -35
- package/dist/{flowDiagram-HODETNUW-3EfLrlFb.js → flowDiagram-HODETNUW-BGZ95AzI.js} +2 -2
- package/dist/{formats-27CW_UP4.js → formats-DI7HEVv7.js} +3 -3
- package/dist/{ganttDiagram-EL5Y4UJY-BQT16JSc.js → ganttDiagram-EL5Y4UJY-Dh_4cBJZ.js} +2 -2
- package/dist/{glide-data-editor-BDYKJdqK.js → glide-data-editor-BlmcP-lw.js} +370 -368
- package/dist/{glide-utils-DJyqh0EM.js → glide-utils-D98laidb.js} +16 -16
- package/dist/{globals-DGTE1S0_.js → globals-BdRitZ4p.js} +3 -3
- package/dist/{highlighted-body-KPVGNVTW-zwjq21tf.js → highlighted-body-KPVGNVTW-3Ziwfz-V.js} +2 -2
- package/dist/{html-to-image-Bf8QMGi4.js → html-to-image-bpiQh7ND.js} +639 -602
- package/dist/{infoDiagram-27XIBGKW-DipQoHxl.js → infoDiagram-27XIBGKW-BF0YX70D.js} +1 -1
- package/dist/{ishikawaDiagram-5VMMS53U-DgFMjSGz.js → ishikawaDiagram-5VMMS53U-_itsAAn-.js} +1 -1
- package/dist/{kanban-definition-UXKFOSKX-w1ACVjgl.js → kanban-definition-UXKFOSKX-VNW2Beqh.js} +1 -1
- package/dist/{label-BDKF3Afc.js → label-Qvw1E97W.js} +3 -3
- package/dist/{linear-Cnaq3Tfb.js → linear-DWERh7eI.js} +121 -73
- package/dist/{loader-BQOfjTIJ.js → loader-DOVB0Syl.js} +12 -12
- package/dist/main.js +9672 -7674
- package/dist/{mermaid-CHo49Yv-.js → mermaid-CZ6FrClS.js} +38 -38
- package/dist/mermaid-HWGCJPDP-Du3-HPQA.js +2 -0
- package/dist/{mindmap-definition-YA3MSWOX-CrT0KSfX.js → mindmap-definition-YA3MSWOX-DR4ripvd.js} +1 -1
- package/dist/{number-overlay-editor-S8UJY0i5.js → number-overlay-editor-DQG-hFHT.js} +3 -3
- package/dist/{objects-CzUgoMIF.js → objects-DCmMvZtn.js} +1 -3
- package/dist/{once-BtE-wKRL.js → once-eG8Vmvjd.js} +1 -1
- package/dist/{pegDiagram-XKGWAZYB-BqvywNJA.js → pegDiagram-XKGWAZYB-CFr3XlGl.js} +2 -2
- package/dist/{pieDiagram-E7YTZNPT-BfxJ_XCc.js → pieDiagram-E7YTZNPT-j91XYSLY.js} +1 -1
- package/dist/precisionRound-DhAzeRcm.js +49 -0
- package/dist/{process-output-BZg56CCb.js → process-output-BzloOI7F.js} +4 -4
- package/dist/{quadrantDiagram-AXDQQJYC-Ccw0xMjN.js → quadrantDiagram-AXDQQJYC-Blrm7V7_.js} +1 -1
- package/dist/{railroadDiagram-O6MQD6OU-8vFLVTPu.js → railroadDiagram-O6MQD6OU-Cd47ZV9E.js} +2 -2
- package/dist/{react-vega-BED0DjiB.js → react-vega-C4T9Rb_w.js} +12284 -12270
- package/dist/{requirementDiagram-BXWQKSXE-a_MxGvxe.js → requirementDiagram-BXWQKSXE-eJH6gLtX.js} +1 -1
- package/dist/{reveal-component-De4Pvf4V.js → reveal-component-D3r3vSTK.js} +14 -14
- package/dist/{select-BButFi-d.js → select-2KOiReNY.js} +536 -457
- package/dist/{sequenceDiagram-WJ2MYXX4-CXFMnxGh.js → sequenceDiagram-WJ2MYXX4-DdfFEIrm.js} +1 -1
- package/dist/{spec-BwarBpUJ.js → spec-h69nC3SI.js} +41 -11
- package/dist/{stateDiagram-D77RDMKH-DQeA_CwI.js → stateDiagram-D77RDMKH-DkjDvip8.js} +2 -2
- package/dist/{stateDiagram-v2-MP3YSRHH-BxxQucm1.js → stateDiagram-v2-MP3YSRHH-DBHACnOo.js} +2 -2
- package/dist/style.css +1 -1
- package/dist/{swimlanes-42K2YHIH-Do6L-cYg.js → swimlanes-42K2YHIH-ogImlOmp.js} +1 -1
- package/dist/{swimlanesDiagram-VR7AAH4N-DUzPvSVU.js → swimlanesDiagram-VR7AAH4N-D6x-zsci.js} +2 -2
- package/dist/{swiper-component-DKntyn8p.js → swiper-component-BkCm-kLg.js} +2 -2
- package/dist/{time-BieKwdYS.js → time-BaMbJs4z.js} +18 -18
- package/dist/{timeline-definition-24CTP7MA-D3c46WM5.js → timeline-definition-24CTP7MA-DQpLtsHD.js} +1 -1
- package/dist/{toDate-UtiwmnH2.js → toDate-CDzoofJo.js} +37 -36
- package/dist/{toast-B9UMEi8u.js → toast-B5t9o6s_.js} +219 -201
- package/dist/{tooltip-BO1REbqs.js → tooltip-s4zZxwKe.js} +325 -301
- package/dist/{use-restore-focus-BEM7sFWl.js → use-restore-focus-BgDTd2tc.js} +1482 -1433
- package/dist/{useAsyncData-Deh5kszT.js → useAsyncData-Bn8SxUqM.js} +15 -14
- package/dist/{useDateFormatter-D8XruNGG.js → useDateFormatter-D3Tkfzej.js} +2 -2
- package/dist/{useDeepCompareMemoize-DXwzRINI.js → useDeepCompareMemoize-CC2tnPCk.js} +1 -1
- package/dist/{useIframeCapabilities-DBETvvPl.js → useIframeCapabilities-CSdWn6NP.js} +1 -1
- package/dist/{useLifecycle-j7D9uD1o.js → useLifecycle-BJlxHqWa.js} +28 -25
- package/dist/{useTheme-CRnjkku0.js → useTheme-DMfQYmlG.js} +301 -297
- package/dist/{vega-component-BeF9dfyO.js → vega-component-CovWHi60.js} +148 -148
- package/dist/{vega-embed-container-BNHE18G9.js → vega-embed-container--LYZqguO.js} +2 -2
- package/dist/{vega-loader.browser-DNST6uVf.js → vega-loader.browser-B9Pz0EGP.js} +298 -507
- package/dist/{vennDiagram-4TSXK5OY-BSZD8JcZ.js → vennDiagram-4TSXK5OY-BD-SL_vq.js} +1 -1
- package/dist/{wardleyDiagram-VM6X3IG4-48cUWkfA.js → wardleyDiagram-VM6X3IG4-BDgJV4ah.js} +1 -1
- package/dist/{xychartDiagram-S5SC5T6Z-CswYgYeu.js → xychartDiagram-S5SC5T6Z-1hIHN6Rl.js} +2 -2
- package/package.json +16 -16
- package/src/__mocks__/requests.ts +0 -1
- package/src/__tests__/module-entry.test.ts +44 -0
- package/src/__tests__/mount.test.ts +1 -21
- package/src/__tests__/{host-imports.test.ts → replica-imports.test.ts} +12 -11
- package/src/components/ai/__tests__/ai-model-dropdown.test.tsx +63 -0
- package/src/components/ai/ai-model-dropdown.tsx +11 -6
- package/src/components/app-config/ai-config.tsx +7 -2
- package/src/components/app-config/user-config-form.tsx +35 -0
- package/src/components/chat/__tests__/message-queue.test.tsx +17 -0
- package/src/components/chat/acp/__tests__/session-tabs.test.tsx +43 -0
- package/src/components/chat/acp/__tests__/state.test.ts +2 -2
- package/src/components/chat/acp/blocks.tsx +3 -2
- package/src/components/chat/acp/session-tabs.tsx +8 -3
- package/src/components/chat/chat-components.tsx +2 -0
- package/src/components/chat/chat-panel.tsx +9 -8
- package/src/components/chat/chat-utils.ts +4 -8
- package/src/components/data-table/TableBottomBar.tsx +4 -1
- package/src/components/data-table/TableTopBar.tsx +2 -0
- package/src/components/data-table/__tests__/chart-spec-model.test.ts +195 -195
- package/src/components/data-table/__tests__/column-explorer.test.tsx +27 -1
- package/src/components/data-table/__tests__/columns.test.tsx +105 -0
- package/src/components/data-table/__tests__/context-menu-interaction.test.tsx +130 -0
- package/src/components/data-table/__tests__/data-table.test.tsx +1 -0
- package/src/components/data-table/__tests__/date-popover.test.tsx +72 -0
- package/src/components/data-table/__tests__/export-actions.test.tsx +105 -0
- package/src/components/data-table/__tests__/keyboard-selection.test.tsx +188 -0
- package/src/components/data-table/__tests__/pagination.test.tsx +28 -1
- package/src/components/data-table/__tests__/search-highlight.test.tsx +109 -0
- package/src/components/data-table/__tests__/url-detector.test.tsx +74 -0
- package/src/components/data-table/cell-selection/feature.ts +2 -2
- package/src/components/data-table/cell-selection/types.ts +2 -2
- package/src/components/data-table/charts/__tests__/chart-panel.test.tsx +74 -1
- package/src/components/data-table/charts/__tests__/spec-snapshot.test.ts +20 -20
- package/src/components/data-table/charts/__tests__/spec.test.ts +78 -52
- package/src/components/data-table/charts/__tests__/storage.test.ts +4 -1
- package/src/components/data-table/charts/chart-spec/spec.ts +68 -43
- package/src/components/data-table/charts/charts.tsx +40 -30
- package/src/components/data-table/charts/storage.ts +16 -5
- package/src/components/data-table/column-explorer-panel/column-explorer.tsx +1 -1
- package/src/components/data-table/column-summary/chart-spec-model.tsx +23 -16
- package/src/components/data-table/column-summary/column-summary.tsx +2 -4
- package/src/components/data-table/column-visibility-dropdown.tsx +8 -13
- package/src/components/data-table/columns.tsx +45 -46
- package/src/components/data-table/context-menu.tsx +36 -20
- package/src/components/data-table/data-table.tsx +17 -1
- package/src/components/data-table/date-popover.tsx +6 -4
- package/src/components/data-table/export-actions.tsx +76 -48
- package/src/components/data-table/pagination.tsx +44 -33
- package/src/components/data-table/range-focus/atoms.ts +4 -6
- package/src/components/data-table/range-focus/use-cell-range-selection.ts +65 -38
- package/src/components/data-table/range-focus/use-scroll-into-view.ts +21 -2
- package/src/components/data-table/renderers.tsx +16 -7
- package/src/components/data-table/row-viewer-panel/__tests__/row-viewer.test.tsx +24 -1
- package/src/components/data-table/row-viewer-panel/row-viewer.tsx +4 -4
- package/src/components/data-table/search-highlight.tsx +79 -0
- package/src/components/data-table/types.ts +1 -0
- package/src/components/data-table/url-detector.tsx +49 -9
- package/src/components/datasources/__tests__/column-actions.test.tsx +97 -0
- package/src/components/datasources/__tests__/column-preview.test.tsx +98 -5
- package/src/components/datasources/__tests__/datasources-interactions.test.tsx +181 -0
- package/src/components/datasources/__tests__/table-list.test.tsx +118 -0
- package/src/components/datasources/__tests__/utils.test.ts +57 -6
- package/src/components/datasources/column-preview.tsx +13 -4
- package/src/components/datasources/datasources.tsx +51 -39
- package/src/components/datasources/utils.ts +32 -23
- package/src/components/debugger/debugger-code.tsx +5 -0
- package/src/components/dependency-graph/dependency-graph-tree.tsx +8 -8
- package/src/components/dependency-graph/elements.ts +24 -21
- package/src/components/dependency-graph/panels.tsx +2 -1
- package/src/components/dependency-graph/utils/__tests__/layout.test.ts +18 -11
- package/src/components/editor/SortableCell.tsx +1 -3
- package/src/{core/attachments/__tests__/toast.test.tsx → components/editor/__tests__/agent-toast.test.tsx} +1 -1
- package/src/components/editor/__tests__/data-attributes.test.tsx +6 -2
- package/src/components/editor/__tests__/dynamic-favicon.test.tsx +111 -18
- package/src/components/editor/__tests__/package-alert.test.tsx +22 -1
- package/src/components/editor/actions/__tests__/pair-with-agent-modal.test.tsx +24 -0
- package/src/components/editor/actions/pair-with-agent-modal.tsx +18 -15
- package/src/components/editor/actions/useRestartKernel.tsx +1 -1
- package/src/components/editor/ai/add-cell-with-ai.tsx +3 -3
- package/src/components/editor/ai/ai-completion-editor.tsx +1 -0
- package/src/components/editor/alerts/__tests__/connection-notice.test.tsx +1 -1
- package/src/components/editor/alerts/connection-notice.tsx +1 -1
- package/src/components/editor/alerts/startup-logs-alert.tsx +1 -0
- package/src/components/editor/app-container.tsx +1 -1
- package/src/components/editor/boundary/__tests__/ErrorBoundary.test.tsx +1 -1
- package/src/components/editor/cell/CreateCellButton.tsx +10 -12
- package/src/components/editor/cell/DeleteButton.tsx +3 -2
- package/src/components/editor/cell/RunButton.tsx +2 -2
- package/src/components/editor/cell/StagedAICell.tsx +10 -2
- package/src/components/editor/cell/StopButton.tsx +2 -2
- package/src/components/editor/cell/__tests__/useShouldShowInterrupt.test.ts +39 -0
- package/src/components/editor/cell/code/__tests__/language-toggle.test.tsx +54 -0
- package/src/components/editor/cell/code/cell-editor.tsx +9 -16
- package/src/components/editor/cell/code/language-toggle.tsx +8 -7
- package/src/components/editor/cell/collapse.tsx +5 -3
- package/src/components/editor/cell/useShouldShowInterrupt.ts +2 -12
- package/src/components/editor/chrome/components/__tests__/feedback-button.test.tsx +1 -1
- package/src/components/editor/chrome/components/feedback-button.tsx +1 -1
- package/src/components/editor/chrome/panels/__tests__/packages-panel.test.tsx +31 -2
- package/src/components/editor/chrome/panels/__tests__/runtime-details.test.tsx +1 -1
- package/src/components/editor/chrome/panels/__tests__/sandbox-recovery.test.tsx +1 -1
- package/src/components/editor/chrome/panels/cache-panel.tsx +1 -0
- package/src/components/editor/chrome/panels/outline/__tests__/outline-keyboard.test.tsx +84 -0
- package/src/components/editor/chrome/panels/outline/floating-outline.tsx +39 -14
- package/src/components/editor/chrome/panels/outline/useActiveOutline.tsx +3 -2
- package/src/components/editor/chrome/panels/packages-panel.tsx +4 -7
- package/src/components/editor/chrome/panels/secrets-panel.tsx +1 -0
- package/src/components/editor/chrome/panels/snippets-panel.tsx +5 -1
- package/src/components/editor/chrome/wrapper/__tests__/attachment-status.test.tsx +9 -18
- package/src/components/editor/chrome/wrapper/__tests__/document-sync-status.test.tsx +92 -0
- package/src/components/editor/chrome/wrapper/__tests__/pending-ai-cells.test.tsx +58 -1
- package/src/components/editor/chrome/wrapper/footer-items/attachment-status.tsx +4 -4
- package/src/components/editor/chrome/wrapper/footer-items/backend-status.tsx +1 -1
- package/src/components/editor/chrome/wrapper/footer-items/document-sync-status.tsx +54 -0
- package/src/components/editor/chrome/wrapper/footer-items/lsp-status.tsx +1 -1
- package/src/components/editor/chrome/wrapper/footer-items/machine-stats.tsx +1 -1
- package/src/components/editor/chrome/wrapper/footer-items/runtime-settings.tsx +1 -7
- package/src/components/editor/chrome/wrapper/footer.tsx +2 -0
- package/src/components/editor/chrome/wrapper/pending-ai-cells.tsx +12 -2
- package/src/components/editor/chrome/wrapper/storage.ts +1 -1
- package/src/components/editor/connections/components.tsx +1 -1
- package/src/components/editor/controls/Controls.tsx +2 -2
- package/src/components/editor/controls/__tests__/command-palette.test.tsx +53 -0
- package/src/components/editor/controls/__tests__/keyboard-shortcuts.test.tsx +61 -0
- package/src/components/editor/controls/__tests__/notebook-menu-dropdown.test.tsx +1 -1
- package/src/components/editor/controls/command-palette.tsx +22 -10
- package/src/components/editor/controls/keyboard-shortcuts.tsx +14 -5
- package/src/components/editor/dynamic-favicon.tsx +21 -40
- package/src/components/editor/errors/__tests__/send-error-report-button.test.tsx +9 -23
- package/src/components/editor/errors/send-error-report-button.tsx +3 -6
- package/src/components/editor/file-tree/__tests__/file-explorer-interactions.test.tsx +6 -0
- package/src/components/editor/file-tree/__tests__/file-explorer-navigation.test.tsx +47 -0
- package/src/components/editor/file-tree/__tests__/use-file-search.test.tsx +91 -0
- package/src/components/editor/file-tree/file-explorer.tsx +45 -8
- package/src/components/editor/file-tree/file-name-input.tsx +1 -0
- package/src/components/editor/file-tree/file-viewer.tsx +4 -2
- package/src/components/editor/file-tree/use-file-search.ts +35 -21
- package/src/components/editor/header/__tests__/status.test.tsx +5 -3
- package/src/components/editor/header/app-header.tsx +1 -1
- package/src/components/editor/header/status.tsx +40 -20
- package/src/components/editor/links/__tests__/cell-link.test.tsx +146 -0
- package/src/components/editor/links/cell-link.tsx +14 -5
- package/src/components/editor/navigation/focus-utils.ts +4 -4
- package/src/components/editor/notebook-banner.tsx +1 -0
- package/src/components/editor/notebook-cell.tsx +12 -3
- package/src/components/editor/output/MarimoErrorOutput.tsx +2 -1
- package/src/components/editor/output/MarimoTracebackOutput.tsx +19 -17
- package/src/components/editor/output/Outputs.css +8 -0
- package/src/components/editor/output/__tests__/traceback.test.tsx +27 -1
- package/src/components/editor/output/ansi-reduce.ts +33 -20
- package/src/components/editor/output/console/ConsoleOutput.tsx +19 -25
- package/src/components/editor/package-alert.tsx +14 -7
- package/src/components/editor/renderers/grid-layout/grid-layout.tsx +12 -6
- package/src/components/editor/renderers/vertical-layout/sidebar/toggle.tsx +7 -1
- package/src/components/home/components.tsx +17 -31
- package/src/components/icons/__tests__/large-spinner.test.tsx +39 -0
- package/src/components/icons/large-spinner.tsx +3 -14
- package/src/components/icons/spinner.tsx +1 -1
- package/src/components/mcp/mcp-status-indicator.tsx +1 -1
- package/src/components/pages/__tests__/home-page.test.tsx +116 -0
- package/src/components/pages/home-page.tsx +164 -91
- package/src/components/scratchpad/scratchpad.test.tsx +95 -0
- package/src/components/scratchpad/scratchpad.tsx +16 -3
- package/src/components/slides/reveal-component.tsx +4 -2
- package/src/components/slides/slide-cell-view.tsx +1 -1
- package/src/components/slides/slide-form.tsx +1 -0
- package/src/components/tracing/tracing-keyboard.test.tsx +85 -0
- package/src/components/tracing/tracing.tsx +45 -41
- package/src/components/ui/__tests__/draggable-popover.test.tsx +24 -0
- package/src/components/ui/__tests__/range-slider.test.tsx +165 -0
- package/src/components/ui/alert-dialog.tsx +1 -1
- package/src/components/ui/draggable-popover.tsx +24 -2
- package/src/components/ui/range-slider.tsx +22 -9
- package/src/core/ai/__tests__/staged-cells.test.ts +8 -5
- package/src/core/ai/context/providers/datasource.ts +6 -2
- package/src/core/ai/staged-cells.ts +7 -2
- package/src/core/ai/tools/__tests__/edit-notebook-tool.test.ts +3 -15
- package/src/core/ai/tools/base.ts +16 -7
- package/src/core/ai/tools/edit-notebook-tool.ts +61 -48
- package/src/core/ai/tools/registry.ts +13 -8
- package/src/core/ai/tools/sample-tool.ts +6 -7
- package/src/core/cells/__tests__/__snapshots__/cells.test.ts.snap +4 -0
- package/src/core/cells/__tests__/cell.test.ts +39 -0
- package/src/core/cells/__tests__/cells.test.ts +83 -346
- package/src/core/cells/__tests__/collapseConsoleOutputs.test.ts +54 -0
- package/src/core/cells/__tests__/document.test.ts +444 -0
- package/src/core/cells/__tests__/flush-document.test.ts +40 -0
- package/src/core/cells/__tests__/notebook-outline.test.ts +108 -0
- package/src/core/cells/__tests__/session.test.ts +117 -36
- package/src/core/cells/__tests__/with-document-save.test.ts +67 -0
- package/src/core/cells/cell.ts +14 -3
- package/src/core/cells/cells.ts +65 -190
- package/src/core/cells/columns.ts +38 -0
- package/src/core/cells/document.ts +389 -0
- package/src/core/cells/pending-delete-service.ts +2 -6
- package/src/core/cells/session.ts +1 -1
- package/src/core/cells/types.ts +22 -1
- package/src/core/codemirror/__tests__/setup.test.ts +29 -29
- package/src/core/codemirror/ai/__tests__/resources.test.ts +117 -0
- package/src/core/codemirror/ai/resources.ts +1 -1
- package/src/core/codemirror/cells/__tests__/extensions.test.ts +42 -1
- package/src/core/codemirror/cells/extensions.ts +21 -3
- package/src/core/codemirror/format.ts +6 -5
- package/src/core/codemirror/keymaps/vim.ts +2 -6
- package/src/core/codemirror/language/__tests__/sql.test.ts +22 -20
- package/src/core/codemirror/language/__tests__/utils.test.ts +187 -15
- package/src/core/codemirror/language/cell-code.ts +114 -0
- package/src/core/codemirror/language/extension.ts +52 -35
- package/src/core/codemirror/language/languages/markdown.ts +8 -6
- package/src/core/codemirror/language/languages/python.ts +16 -10
- package/src/core/codemirror/language/languages/sql/sql.ts +16 -24
- package/src/core/codemirror/language/panel/panel.tsx +1 -0
- package/src/core/codemirror/language/types.ts +7 -7
- package/src/core/codemirror/language/utils.ts +9 -39
- package/src/core/codemirror/lsp/notebook-lsp.ts +6 -5
- package/src/core/codemirror/readonly/__tests__/extension.test.ts +1 -1
- package/src/core/codemirror/readonly/extension.ts +2 -2
- package/src/core/codemirror/rtc/extension.ts +24 -13
- package/src/core/codemirror/rtc/loro/awareness.ts +34 -19
- package/src/core/codemirror/rtc/loro/sync.ts +1 -0
- package/src/core/codemirror/utils.ts +1 -0
- package/src/core/config/__tests__/config-schema.test.ts +5 -3
- package/src/core/config/config-schema.ts +3 -0
- package/src/core/datasets/__tests__/data-source-discovery.test.ts +5 -5
- package/src/core/datasets/data-source-discovery.ts +4 -6
- package/src/core/edit-app.tsx +13 -9
- package/src/core/errors/error-entries.ts +3 -0
- package/src/core/export/hooks.ts +1 -1
- package/src/core/export/layout.ts +2 -2
- package/src/core/hotkeys/actions.ts +2 -4
- package/src/core/islands/__tests__/bootstrap.test.ts +73 -43
- package/src/core/islands/__tests__/bridge.test.ts +72 -25
- package/src/core/islands/bootstrap.ts +46 -53
- package/src/core/islands/bridge.ts +74 -21
- package/src/core/islands/components/__tests__/IslandControls.test.tsx +7 -5
- package/src/core/islands/worker/__tests__/controller.test.ts +62 -4
- package/src/core/islands/worker/controller.ts +4 -10
- package/src/core/islands/worker/worker.tsx +33 -33
- package/src/core/kernel/__tests__/RuntimeState.test.ts +1 -1
- package/src/core/kernel/__tests__/handlers.test.ts +7 -153
- package/src/core/kernel/handlers.ts +18 -59
- package/src/core/network/__tests__/connection.test.ts +1 -1
- package/src/core/network/connection-notice.ts +1 -1
- package/src/core/network/connection.ts +5 -3
- package/src/core/network/requests-lazy.ts +1 -2
- package/src/core/network/requests-network.ts +0 -9
- package/src/core/network/requests-static.ts +0 -1
- package/src/core/network/requests-toasting.tsx +0 -1
- package/src/core/network/types.ts +0 -5
- package/src/core/network/useConnectionNotice.ts +8 -2
- package/src/core/notebook/attachments.ts +38 -0
- package/src/core/notebook/document/__tests__/codemirror.test.ts +547 -0
- package/src/core/notebook/document/__tests__/draft.test.ts +322 -0
- package/src/core/notebook/document/__tests__/fixtures.ts +92 -0
- package/src/core/notebook/document/__tests__/hold.test.ts +88 -0
- package/src/core/notebook/document/__tests__/hooks.test.tsx +391 -0
- package/src/core/notebook/document/__tests__/lifecycle-scenarios.test.ts +61 -0
- package/src/core/notebook/document/__tests__/model.test.ts +247 -0
- package/src/core/notebook/document/__tests__/protocol-fixtures.ts +38 -0
- package/src/core/notebook/document/__tests__/recovery-scenarios.test.ts +230 -0
- package/src/core/notebook/document/__tests__/scenario-fixture.ts +43 -0
- package/src/core/notebook/document/__tests__/scenarios.test.ts +205 -0
- package/src/core/notebook/document/__tests__/state.test.ts +438 -0
- package/src/core/notebook/document/__tests__/submission.test.ts +203 -0
- package/src/core/notebook/document/__tests__/test-host.test.ts +92 -0
- package/src/core/notebook/document/__tests__/test-host.ts +247 -0
- package/src/core/notebook/document/__tests__/test-notebook.ts +149 -0
- package/src/core/notebook/document/__tests__/user.ts +61 -0
- package/src/core/notebook/document/codemirror.ts +216 -0
- package/src/core/notebook/document/draft.ts +472 -0
- package/src/core/notebook/document/hooks.tsx +118 -0
- package/src/core/notebook/document/index.ts +23 -0
- package/src/core/notebook/document/model.ts +105 -0
- package/src/core/notebook/document/select.ts +41 -0
- package/src/core/notebook/document/state.ts +374 -0
- package/src/core/notebook/document/sync.ts +418 -0
- package/src/core/notebook/index.ts +18 -0
- package/src/core/notebook/replica/AGENTS.md +11 -0
- package/src/core/notebook/replica/__tests__/attachments.test.ts +56 -0
- package/src/core/notebook/replica/__tests__/document.test.ts +126 -0
- package/src/core/notebook/replica/__tests__/fixtures.ts +79 -0
- package/src/core/notebook/replica/__tests__/replica.test.ts +550 -0
- package/src/core/notebook/replica/backend.ts +19 -0
- package/src/core/notebook/replica/connection.ts +42 -0
- package/src/core/notebook/replica/index.ts +13 -0
- package/src/core/notebook/replica/replica.ts +286 -0
- package/src/core/notebook/replica/resources/attachments.ts +22 -0
- package/src/core/notebook/replica/resources/document.ts +101 -0
- package/src/core/packages/__tests__/useSandboxController.test.tsx +1 -1
- package/src/core/packages/useSandboxController.ts +1 -1
- package/src/core/run-app.tsx +48 -51
- package/src/core/runtime/__tests__/adapter.test.ts +1 -1
- package/src/core/runtime/__tests__/createWsUrl.test.ts +77 -7
- package/src/core/runtime/__tests__/runtime.test.ts +32 -58
- package/src/core/runtime/adapter.ts +1 -1
- package/src/core/runtime/config.ts +2 -2
- package/src/core/runtime/runtime.ts +38 -42
- package/src/core/runtime/types.ts +0 -2
- package/src/core/saving/filename.ts +1 -1
- package/src/core/saving/save-component.tsx +10 -4
- package/src/core/saving/useAutoSave.ts +1 -1
- package/src/core/session/__tests__/backend.test.ts +235 -0
- package/src/core/session/__tests__/useNotebookReplica.hook.test.tsx +183 -0
- package/src/core/session/__tests__/useNotebookSession.hook.test.tsx +663 -0
- package/src/core/session/active-notebook.ts +11 -0
- package/src/core/session/backend.ts +125 -0
- package/src/core/{websocket → session}/types.ts +2 -2
- package/src/core/session/useNotebookReplica.ts +72 -0
- package/src/core/{websocket → session}/useNotebookSession.tsx +70 -68
- package/src/core/state/__tests__/jotai.test.tsx +108 -0
- package/src/core/state/jotai.ts +1 -1
- package/src/core/static/__tests__/files.test.ts +38 -10
- package/src/core/static/files.ts +2 -0
- package/src/core/{websocket → transports}/__tests__/connection-state.test.ts +1 -13
- package/src/core/transports/__tests__/server.test.ts +408 -0
- package/src/core/transports/__tests__/static.test.ts +175 -0
- package/src/core/transports/__tests__/wasm.test.ts +166 -0
- package/src/core/{websocket → transports}/connection-state.ts +9 -16
- package/src/core/transports/server.ts +152 -0
- package/src/core/transports/static.ts +84 -0
- package/src/core/transports/wasm.ts +97 -0
- package/src/core/wasm/__tests__/bridge.test.ts +213 -50
- package/src/core/wasm/__tests__/worker.test.ts +187 -0
- package/src/core/wasm/bridge.ts +75 -53
- package/src/core/wasm/rpc.ts +15 -3
- package/src/core/wasm/store.ts +3 -3
- package/src/core/wasm/worker/bootstrap.ts +55 -51
- package/src/core/wasm/worker/getMarimoWheel.ts +1 -1
- package/src/core/wasm/worker/save-worker.ts +9 -10
- package/src/core/wasm/worker/types.ts +29 -18
- package/src/core/wasm/worker/worker.ts +88 -44
- package/src/css/globals.css +10 -0
- package/src/hooks/__tests__/useAsyncData.test.tsx +148 -0
- package/src/hooks/__tests__/useDebounce.test.tsx +51 -0
- package/src/hooks/__tests__/useDeepCompareMemoize.test.tsx +39 -0
- package/src/hooks/__tests__/useDelayElapsed.test.tsx +28 -0
- package/src/hooks/__tests__/useInternalStateWithSync.test.tsx +20 -0
- package/src/hooks/__tests__/useScript.test.tsx +77 -0
- package/src/hooks/__tests__/useStateForKey.test.tsx +257 -0
- package/src/hooks/useAsyncData.ts +3 -3
- package/src/hooks/useDataSourceDiscovery.ts +6 -4
- package/src/hooks/useDebounce.ts +6 -10
- package/src/hooks/useDeepCompareMemoize.ts +1 -1
- package/src/hooks/useDelayElapsed.ts +14 -10
- package/src/hooks/useElapsedTime.ts +3 -3
- package/src/hooks/useEventListener.ts +1 -2
- package/src/hooks/useInputHistory.ts +4 -7
- package/src/hooks/useInternalStateWithSync.ts +6 -6
- package/src/hooks/useResizeObserver.ts +4 -7
- package/src/hooks/useScript.ts +67 -70
- package/src/hooks/useStateForKey.ts +93 -0
- package/src/mount.tsx +1 -5
- package/src/plugins/__tests__/plugin-object-schemas.test.ts +104 -0
- package/src/plugins/core/registerReactComponent.tsx +1 -1
- package/src/plugins/impl/CodeEditorPlugin.tsx +1 -1
- package/src/plugins/impl/DataEditorPlugin.tsx +1 -1
- package/src/plugins/impl/DataTablePlugin.tsx +69 -37
- package/src/plugins/impl/DropdownPlugin.tsx +2 -6
- package/src/plugins/impl/FileBrowserPlugin.tsx +4 -16
- package/src/plugins/impl/FormPlugin.tsx +1 -1
- package/src/plugins/impl/RangeSliderPlugin.tsx +16 -2
- package/src/plugins/impl/RefreshPlugin.tsx +1 -0
- package/src/plugins/impl/SliderPlugin.tsx +6 -8
- package/src/plugins/impl/__tests__/DataTablePlugin.test.tsx +387 -1
- package/src/plugins/impl/__tests__/FileBrowserPlugin.test.tsx +45 -0
- package/src/plugins/impl/__tests__/RangeSliderPlugin.test.tsx +109 -0
- package/src/plugins/impl/__tests__/SliderPlugin.test.tsx +24 -3
- package/src/plugins/impl/anywidget/__tests__/registry.test.ts +153 -0
- package/src/plugins/impl/anywidget/__tests__/widget-binding.test.ts +241 -2
- package/src/plugins/impl/anywidget/model.ts +3 -1
- package/src/plugins/impl/anywidget/registry.ts +18 -14
- package/src/plugins/impl/anywidget/runtime.ts +21 -9
- package/src/plugins/impl/anywidget/widget-binding.ts +160 -82
- package/src/plugins/impl/chat/__tests__/chat-prompts.test.tsx +93 -0
- package/src/plugins/impl/chat/chat-ui.tsx +14 -11
- package/src/plugins/impl/code/any-language-editor.tsx +2 -2
- package/src/plugins/impl/data-editor/glide-data-editor.tsx +8 -5
- package/src/plugins/impl/data-editor/glide-utils.ts +3 -1
- package/src/plugins/impl/data-explorer/ConnectedDataExplorerComponent.tsx +1 -1
- package/src/plugins/impl/data-explorer/components/column-summary.tsx +4 -2
- package/src/plugins/impl/data-explorer/state/reducer.ts +1 -1
- package/src/plugins/impl/data-frames/DataFramePlugin.tsx +1 -2
- package/src/plugins/impl/data-frames/__tests__/sidebar.test.tsx +160 -0
- package/src/plugins/impl/data-frames/forms/__tests__/__snapshots__/form.test.tsx.snap +66 -66
- package/src/plugins/impl/data-frames/panel.tsx +43 -16
- package/src/plugins/impl/mpl-interactive/MplInteractivePlugin.tsx +5 -3
- package/src/plugins/impl/panel/PanelPlugin.tsx +16 -17
- package/src/plugins/impl/panel/__tests__/utils.test.ts +39 -1
- package/src/plugins/impl/panel/utils.ts +17 -0
- package/src/plugins/impl/plotly/PlotlyPlugin.tsx +2 -5
- package/src/plugins/impl/plotly/selection.ts +25 -19
- package/src/plugins/impl/vega/VegaPlugin.tsx +2 -3
- package/src/plugins/impl/vega/make-selectable.ts +4 -7
- package/src/plugins/layout/RoutesPlugin.tsx +1 -1
- package/src/stories/__tests__/editor-story.test.tsx +33 -0
- package/src/stories/button.stories.tsx +1 -0
- package/src/stories/cell.stories.tsx +1 -1
- package/src/stories/data-table.stories.tsx +31 -0
- package/src/stories/editor.stories.tsx +24 -17
- package/src/stories/form-wrapper.stories.tsx +2 -2
- package/src/stories/layout/vertical/one-column.stories.tsx +1 -1
- package/src/stories/runtime-story.tsx +1 -1
- package/src/stories/select.stories.tsx +2 -2
- package/src/stories/switchable-multi-select.stories.tsx +1 -1
- package/src/utils/__tests__/arrays.test.ts +35 -0
- package/src/utils/__tests__/batch-requests.test.ts +3 -3
- package/src/utils/__tests__/{createReducer.test.ts → createReducer.test.tsx} +29 -0
- package/src/utils/__tests__/numbers.test.ts +105 -1
- package/src/utils/__tests__/objects.test.ts +21 -0
- package/src/utils/arrays.ts +11 -2
- package/src/utils/blob.ts +1 -1
- package/src/utils/createReducer.ts +53 -26
- package/src/utils/maps.ts +0 -3
- package/src/utils/numbers.ts +79 -18
- package/src/utils/objects.ts +4 -9
- package/src/utils/once.ts +19 -19
- package/src/utils/tracer.ts +9 -5
- package/src/utils/zod-utils.ts +1 -3
- package/dist/any-language-editor-DZ_YTVus.js +0 -76
- package/dist/assets/__vite-browser-external-BXYkiFsk.js +0 -1
- package/dist/assets/worker-HPY9fbhz.js +0 -114
- package/dist/mermaid-HWGCJPDP-ByCsZnYF.js +0 -2
- package/dist/precisionRound-DMIVQrt2.js +0 -97
- package/src/core/attachments/state.ts +0 -21
- package/src/core/cells/__tests__/apply-transaction.test.ts +0 -991
- package/src/core/cells/__tests__/document-changes.test.ts +0 -1140
- package/src/core/cells/__tests__/document-roundtrip.test.ts +0 -392
- package/src/core/cells/document-changes.ts +0 -944
- package/src/core/host/AGENTS.md +0 -6
- package/src/core/host/__tests__/events.test.ts +0 -46
- package/src/core/host/__tests__/notebook-client.test.ts +0 -191
- package/src/core/host/__tests__/protocol.test.ts +0 -214
- package/src/core/host/connection.ts +0 -81
- package/src/core/host/events.ts +0 -60
- package/src/core/host/notebook-client.ts +0 -226
- package/src/core/host/protocol.ts +0 -1943
- package/src/core/state/__tests__/jotai.test.ts +0 -25
- package/src/core/wasm/worker/message-buffer.ts +0 -33
- package/src/core/websocket/__tests__/createConnectionTransport.test.ts +0 -49
- package/src/core/websocket/__tests__/useNotebookClient.hook.test.tsx +0 -140
- package/src/core/websocket/__tests__/useNotebookSession.hook.test.tsx +0 -883
- package/src/core/websocket/transports/__tests__/basic.test.ts +0 -275
- package/src/core/websocket/transports/__tests__/sse.test.ts +0 -335
- package/src/core/websocket/transports/__tests__/transport.test.ts +0 -308
- package/src/core/websocket/transports/__tests__/ws.test.ts +0 -125
- package/src/core/websocket/transports/basic.ts +0 -102
- package/src/core/websocket/transports/create.ts +0 -31
- package/src/core/websocket/transports/sse.ts +0 -308
- package/src/core/websocket/transports/transport.ts +0 -50
- package/src/core/websocket/transports/ws.ts +0 -98
- package/src/core/websocket/useNotebookClient.ts +0 -80
- /package/src/{core/attachments/toast.ts → components/editor/agent-toast.ts} +0 -0
- /package/src/core/{websocket → session}/connection-utils.ts +0 -0
|
@@ -0,0 +1,391 @@
|
|
|
1
|
+
/* Copyright 2026 Marimo. All rights reserved. */
|
|
2
|
+
|
|
3
|
+
import { act, render, renderHook } from "@testing-library/react";
|
|
4
|
+
import { atom, createStore, Provider } from "jotai";
|
|
5
|
+
import type { PropsWithChildren } from "react";
|
|
6
|
+
import { afterEach, describe, expect, it, vi } from "vitest";
|
|
7
|
+
import { Deferred } from "@/utils/Deferred";
|
|
8
|
+
import type { Document } from "../../replica/resources/document";
|
|
9
|
+
import {
|
|
10
|
+
useNotebookCellCode,
|
|
11
|
+
useNotebookCellConfig,
|
|
12
|
+
useNotebookCellName,
|
|
13
|
+
useNotebookCellOrder,
|
|
14
|
+
useNotebookDocumentEdit,
|
|
15
|
+
useNotebookDocumentSync,
|
|
16
|
+
} from "@/core/notebook";
|
|
17
|
+
import { createDocumentModel, DocumentProvider } from "..";
|
|
18
|
+
import { cell, CONFIG, document, withCells } from "./fixtures";
|
|
19
|
+
import { type FakeEdit, fakeBackend } from "../../replica/__tests__/fixtures";
|
|
20
|
+
|
|
21
|
+
const lifetimes: AbortController[] = [];
|
|
22
|
+
|
|
23
|
+
afterEach(() => {
|
|
24
|
+
act(() => lifetimes.splice(0).forEach((controller) => controller.abort()));
|
|
25
|
+
});
|
|
26
|
+
|
|
27
|
+
function setup(
|
|
28
|
+
initial: Document | null = document(),
|
|
29
|
+
edit: FakeEdit = () => new Promise(() => {}),
|
|
30
|
+
) {
|
|
31
|
+
const store = createStore();
|
|
32
|
+
const source = atom(initial);
|
|
33
|
+
const detached = vi.fn();
|
|
34
|
+
source.onMount = () => detached;
|
|
35
|
+
const model = createDocumentModel({
|
|
36
|
+
store,
|
|
37
|
+
documentAtom: source,
|
|
38
|
+
backendAtom: atom(fakeBackend({ edit })),
|
|
39
|
+
});
|
|
40
|
+
const controller = new AbortController();
|
|
41
|
+
lifetimes.push(controller);
|
|
42
|
+
model.start({ signal: controller.signal });
|
|
43
|
+
const wrapper = ({ children }: PropsWithChildren) => (
|
|
44
|
+
<Provider store={store}>
|
|
45
|
+
<DocumentProvider model={model}>{children}</DocumentProvider>
|
|
46
|
+
</Provider>
|
|
47
|
+
);
|
|
48
|
+
return { store, source, model, controller, detached, wrapper };
|
|
49
|
+
}
|
|
50
|
+
|
|
51
|
+
describe("document hooks", () => {
|
|
52
|
+
it("flushes edits through the model and aborts work when the session ends", async () => {
|
|
53
|
+
const accepted = withCells(document(), { c1: cell("c1", "local") });
|
|
54
|
+
const edit = vi
|
|
55
|
+
.fn<FakeEdit>()
|
|
56
|
+
.mockResolvedValueOnce({
|
|
57
|
+
ok: true,
|
|
58
|
+
status: 200,
|
|
59
|
+
data: { document: accepted.revision, cells: {} },
|
|
60
|
+
})
|
|
61
|
+
.mockImplementation(
|
|
62
|
+
() => new Deferred<Awaited<ReturnType<FakeEdit>>>().promise,
|
|
63
|
+
);
|
|
64
|
+
const { store, source, wrapper, controller } = setup(document(), edit);
|
|
65
|
+
const { result, unmount } = renderHook(
|
|
66
|
+
() => ({
|
|
67
|
+
code: useNotebookCellCode("c1"),
|
|
68
|
+
edit: useNotebookDocumentEdit(),
|
|
69
|
+
sync: useNotebookDocumentSync(),
|
|
70
|
+
}),
|
|
71
|
+
{ wrapper, reactStrictMode: true },
|
|
72
|
+
);
|
|
73
|
+
act(() => result.current.edit((tx) => tx.setCode("c1", "local")));
|
|
74
|
+
expect(result.current.sync.status).toEqual({ status: "pending" });
|
|
75
|
+
const { flush, retry } = result.current.sync;
|
|
76
|
+
await act(async () => {
|
|
77
|
+
const flushed = flush();
|
|
78
|
+
// Allow the response to arrive before publishing its confirmation.
|
|
79
|
+
await Promise.resolve();
|
|
80
|
+
store.set(source, accepted);
|
|
81
|
+
await expect(flushed).resolves.toEqual(accepted.revision);
|
|
82
|
+
});
|
|
83
|
+
expect(result.current.sync).toEqual({
|
|
84
|
+
status: { status: "idle" },
|
|
85
|
+
flush,
|
|
86
|
+
retry,
|
|
87
|
+
});
|
|
88
|
+
expect(result.current.code).toBe("local");
|
|
89
|
+
act(() => result.current.edit((tx) => tx.setCode("c1", "next")));
|
|
90
|
+
await act(async () => {
|
|
91
|
+
const rejected = (async () => {
|
|
92
|
+
await expect(flush()).rejects.toThrow("session ended");
|
|
93
|
+
})();
|
|
94
|
+
await Promise.resolve();
|
|
95
|
+
unmount();
|
|
96
|
+
controller.abort(new Error("session ended"));
|
|
97
|
+
await rejected;
|
|
98
|
+
});
|
|
99
|
+
expect(edit).toHaveBeenCalledTimes(2);
|
|
100
|
+
expect(edit.mock.calls[1][2]?.signal?.aborted).toBe(true);
|
|
101
|
+
});
|
|
102
|
+
|
|
103
|
+
it("publishes compound edits atomically without changing host records", () => {
|
|
104
|
+
const initial = document();
|
|
105
|
+
const { store, source, wrapper } = setup(initial);
|
|
106
|
+
const seen = vi.fn();
|
|
107
|
+
const { result } = renderHook(
|
|
108
|
+
() => {
|
|
109
|
+
const value = {
|
|
110
|
+
code: useNotebookCellCode("c1"),
|
|
111
|
+
name: useNotebookCellName("c1"),
|
|
112
|
+
config: useNotebookCellConfig("c1"),
|
|
113
|
+
order: useNotebookCellOrder(),
|
|
114
|
+
};
|
|
115
|
+
seen(value);
|
|
116
|
+
return { ...value, edit: useNotebookDocumentEdit() };
|
|
117
|
+
},
|
|
118
|
+
{ wrapper },
|
|
119
|
+
);
|
|
120
|
+
const edit = result.current.edit;
|
|
121
|
+
seen.mockClear();
|
|
122
|
+
|
|
123
|
+
act(() =>
|
|
124
|
+
edit((tx) => {
|
|
125
|
+
tx.setCode("c1", "edited");
|
|
126
|
+
tx.setName("c1", "renamed");
|
|
127
|
+
tx.updateConfig("c1", { hide_code: true });
|
|
128
|
+
tx.setOrder(["c2", "c1"]);
|
|
129
|
+
}),
|
|
130
|
+
);
|
|
131
|
+
|
|
132
|
+
const expected = {
|
|
133
|
+
code: "edited",
|
|
134
|
+
name: "renamed",
|
|
135
|
+
config: { ...CONFIG, hide_code: true },
|
|
136
|
+
order: ["c2", "c1"],
|
|
137
|
+
};
|
|
138
|
+
expect(result.current).toEqual({ ...expected, edit });
|
|
139
|
+
expect(seen.mock.calls.length).toBeGreaterThan(0);
|
|
140
|
+
for (const [value] of seen.mock.calls) {
|
|
141
|
+
expect(value).toEqual(expected);
|
|
142
|
+
}
|
|
143
|
+
expect(store.get(source)).toBe(initial);
|
|
144
|
+
|
|
145
|
+
expect(() =>
|
|
146
|
+
act(() =>
|
|
147
|
+
edit((tx) => {
|
|
148
|
+
tx.setCode("c1", "rolled back");
|
|
149
|
+
tx.deleteCell("missing");
|
|
150
|
+
}),
|
|
151
|
+
),
|
|
152
|
+
).toThrow("does not exist");
|
|
153
|
+
expect(result.current).toEqual({ ...expected, edit });
|
|
154
|
+
});
|
|
155
|
+
|
|
156
|
+
it("subscribes to individual fields and keeps the edit function stable", () => {
|
|
157
|
+
const { store, source, wrapper } = setup();
|
|
158
|
+
const onCode = vi.fn();
|
|
159
|
+
const onName = vi.fn();
|
|
160
|
+
const onConfig = vi.fn();
|
|
161
|
+
const onOrder = vi.fn();
|
|
162
|
+
const onEdit = vi.fn();
|
|
163
|
+
function Code() {
|
|
164
|
+
onCode(useNotebookCellCode("c1"));
|
|
165
|
+
return null;
|
|
166
|
+
}
|
|
167
|
+
function Name() {
|
|
168
|
+
onName(useNotebookCellName("c1"));
|
|
169
|
+
return null;
|
|
170
|
+
}
|
|
171
|
+
function Config() {
|
|
172
|
+
onConfig(useNotebookCellConfig("c1"));
|
|
173
|
+
return null;
|
|
174
|
+
}
|
|
175
|
+
function Order() {
|
|
176
|
+
onOrder(useNotebookCellOrder());
|
|
177
|
+
return null;
|
|
178
|
+
}
|
|
179
|
+
function Edit() {
|
|
180
|
+
onEdit(useNotebookDocumentEdit());
|
|
181
|
+
return null;
|
|
182
|
+
}
|
|
183
|
+
render(
|
|
184
|
+
<>
|
|
185
|
+
<Code />
|
|
186
|
+
<Name />
|
|
187
|
+
<Config />
|
|
188
|
+
<Order />
|
|
189
|
+
<Edit />
|
|
190
|
+
</>,
|
|
191
|
+
{ wrapper },
|
|
192
|
+
);
|
|
193
|
+
const edit: ReturnType<typeof useNotebookDocumentEdit> =
|
|
194
|
+
onEdit.mock.calls[0][0];
|
|
195
|
+
const clear = () =>
|
|
196
|
+
[onCode, onName, onConfig, onOrder, onEdit].forEach((fn) =>
|
|
197
|
+
fn.mockClear(),
|
|
198
|
+
);
|
|
199
|
+
clear();
|
|
200
|
+
|
|
201
|
+
act(() => edit((tx) => tx.setCode("c1", "local")));
|
|
202
|
+
expect(onCode).toHaveBeenCalledWith("local");
|
|
203
|
+
expect(onName).not.toHaveBeenCalled();
|
|
204
|
+
expect(onConfig).not.toHaveBeenCalled();
|
|
205
|
+
expect(onOrder).not.toHaveBeenCalled();
|
|
206
|
+
expect(onEdit).not.toHaveBeenCalled();
|
|
207
|
+
clear();
|
|
208
|
+
|
|
209
|
+
act(() => edit((tx) => tx.setCode("c2", "other")));
|
|
210
|
+
expect(
|
|
211
|
+
[onCode, onName, onConfig, onOrder, onEdit].map((fn) => fn.mock.calls),
|
|
212
|
+
).toEqual([[], [], [], [], []]);
|
|
213
|
+
|
|
214
|
+
act(() =>
|
|
215
|
+
store.set(
|
|
216
|
+
source,
|
|
217
|
+
withCells(document(), {
|
|
218
|
+
c1: cell("c1", "remote", {
|
|
219
|
+
name: "host name",
|
|
220
|
+
config: { ...CONFIG },
|
|
221
|
+
}),
|
|
222
|
+
}),
|
|
223
|
+
),
|
|
224
|
+
);
|
|
225
|
+
expect(onName).toHaveBeenCalledWith("host name");
|
|
226
|
+
expect(onCode).not.toHaveBeenCalled();
|
|
227
|
+
expect(onConfig).not.toHaveBeenCalled();
|
|
228
|
+
expect(onOrder).not.toHaveBeenCalled();
|
|
229
|
+
expect(onEdit).not.toHaveBeenCalled();
|
|
230
|
+
clear();
|
|
231
|
+
|
|
232
|
+
act(() =>
|
|
233
|
+
edit((tx) => {
|
|
234
|
+
tx.updateConfig("c1", { disabled: true });
|
|
235
|
+
tx.setOrder(["c2", "c1"]);
|
|
236
|
+
}),
|
|
237
|
+
);
|
|
238
|
+
expect(onConfig).toHaveBeenCalledWith({ ...CONFIG, disabled: true });
|
|
239
|
+
expect(onOrder).toHaveBeenCalledWith(["c2", "c1"]);
|
|
240
|
+
expect(onCode).not.toHaveBeenCalled();
|
|
241
|
+
expect(onName).not.toHaveBeenCalled();
|
|
242
|
+
expect(onEdit).not.toHaveBeenCalled();
|
|
243
|
+
});
|
|
244
|
+
|
|
245
|
+
it("switches field subscriptions when the cell id changes", () => {
|
|
246
|
+
const { wrapper } = setup();
|
|
247
|
+
const { result, rerender } = renderHook(
|
|
248
|
+
(id) => ({
|
|
249
|
+
code: useNotebookCellCode(id),
|
|
250
|
+
edit: useNotebookDocumentEdit(),
|
|
251
|
+
}),
|
|
252
|
+
{ wrapper, initialProps: "c1" },
|
|
253
|
+
);
|
|
254
|
+
const edit = result.current.edit;
|
|
255
|
+
rerender("c2");
|
|
256
|
+
expect(result.current).toEqual({ code: "y = 2", edit });
|
|
257
|
+
act(() => edit((tx) => tx.setCode("c1", "old subscription")));
|
|
258
|
+
expect(result.current).toEqual({ code: "y = 2", edit });
|
|
259
|
+
act(() => edit((tx) => tx.setCode("c2", "new subscription")));
|
|
260
|
+
expect(result.current).toEqual({ code: "new subscription", edit });
|
|
261
|
+
});
|
|
262
|
+
|
|
263
|
+
it("distinguishes unavailable documents, hidden source, empty code, and deleted cells", () => {
|
|
264
|
+
const { store, source, wrapper } = setup(null);
|
|
265
|
+
const { result } = renderHook(
|
|
266
|
+
() => ({
|
|
267
|
+
order: useNotebookCellOrder(),
|
|
268
|
+
code: useNotebookCellCode("c1"),
|
|
269
|
+
name: useNotebookCellName("c1"),
|
|
270
|
+
config: useNotebookCellConfig("c1"),
|
|
271
|
+
edit: useNotebookDocumentEdit(),
|
|
272
|
+
}),
|
|
273
|
+
{ wrapper },
|
|
274
|
+
);
|
|
275
|
+
const edit = result.current.edit;
|
|
276
|
+
const absent = {
|
|
277
|
+
code: undefined,
|
|
278
|
+
name: undefined,
|
|
279
|
+
config: undefined,
|
|
280
|
+
edit,
|
|
281
|
+
};
|
|
282
|
+
expect(result.current).toEqual({ ...absent, order: null });
|
|
283
|
+
const hidden = withCells(document(), {
|
|
284
|
+
c1: cell("c1", "", { code: null }),
|
|
285
|
+
});
|
|
286
|
+
act(() => store.set(source, hidden));
|
|
287
|
+
expect(result.current).toEqual({
|
|
288
|
+
code: null,
|
|
289
|
+
name: "_",
|
|
290
|
+
config: CONFIG,
|
|
291
|
+
order: ["c1", "c2"],
|
|
292
|
+
edit,
|
|
293
|
+
});
|
|
294
|
+
act(() => edit((tx) => tx.setCode("c1", "")));
|
|
295
|
+
expect(result.current.code).toBe("");
|
|
296
|
+
act(() => store.set(source, null));
|
|
297
|
+
expect(result.current).toEqual({ ...absent, order: null });
|
|
298
|
+
act(() => store.set(source, hidden));
|
|
299
|
+
expect(result.current.code).toBe("");
|
|
300
|
+
act(() =>
|
|
301
|
+
edit((tx) => {
|
|
302
|
+
tx.deleteCell("c1");
|
|
303
|
+
tx.deleteCell("c2");
|
|
304
|
+
}),
|
|
305
|
+
);
|
|
306
|
+
expect(result.current).toEqual({ ...absent, order: [] });
|
|
307
|
+
});
|
|
308
|
+
|
|
309
|
+
it("preserves drafts across Strict Mode remounts until the session ends", () => {
|
|
310
|
+
const { wrapper, controller, detached } = setup();
|
|
311
|
+
const useProbe = () => ({
|
|
312
|
+
code: useNotebookCellCode("c1"),
|
|
313
|
+
edit: useNotebookDocumentEdit(),
|
|
314
|
+
});
|
|
315
|
+
const first = renderHook(useProbe, { wrapper, reactStrictMode: true });
|
|
316
|
+
const edit = first.result.current.edit;
|
|
317
|
+
act(() => edit((tx) => tx.setCode("c1", "strict mode edit")));
|
|
318
|
+
expect(first.result.current.code).toBe("strict mode edit");
|
|
319
|
+
first.unmount();
|
|
320
|
+
expect(detached).not.toHaveBeenCalled();
|
|
321
|
+
|
|
322
|
+
const second = renderHook(useProbe, { wrapper, reactStrictMode: true });
|
|
323
|
+
expect(second.result.current.code).toBe("strict mode edit");
|
|
324
|
+
act(() => edit((tx) => tx.setCode("c1", "same session")));
|
|
325
|
+
expect(second.result.current.code).toBe("same session");
|
|
326
|
+
second.unmount();
|
|
327
|
+
controller.abort();
|
|
328
|
+
expect(detached).toHaveBeenCalledOnce();
|
|
329
|
+
const callback = vi.fn();
|
|
330
|
+
edit(callback);
|
|
331
|
+
expect(callback).not.toHaveBeenCalled();
|
|
332
|
+
});
|
|
333
|
+
|
|
334
|
+
it("switches reads and edits to the supplied model and store", () => {
|
|
335
|
+
const first = setup();
|
|
336
|
+
const second = setup(
|
|
337
|
+
withCells(document("n2"), {
|
|
338
|
+
c1: cell("c1", "second source", { notebook_id: "n2" }),
|
|
339
|
+
}),
|
|
340
|
+
);
|
|
341
|
+
const seen = vi.fn();
|
|
342
|
+
function Probe() {
|
|
343
|
+
seen({
|
|
344
|
+
code: useNotebookCellCode("c1"),
|
|
345
|
+
edit: useNotebookDocumentEdit(),
|
|
346
|
+
});
|
|
347
|
+
return null;
|
|
348
|
+
}
|
|
349
|
+
const tree = ({ store, model }: typeof first) => (
|
|
350
|
+
<Provider store={store}>
|
|
351
|
+
<DocumentProvider model={model}>
|
|
352
|
+
<Probe />
|
|
353
|
+
</DocumentProvider>
|
|
354
|
+
</Provider>
|
|
355
|
+
);
|
|
356
|
+
const { rerender } = render(tree(first));
|
|
357
|
+
const oldEdit: ReturnType<typeof useNotebookDocumentEdit> =
|
|
358
|
+
seen.mock.calls.at(-1)![0].edit;
|
|
359
|
+
act(() => oldEdit((tx) => tx.setCode("c1", "first draft")));
|
|
360
|
+
rerender(tree(second));
|
|
361
|
+
act(() => first.controller.abort());
|
|
362
|
+
expect(first.detached).toHaveBeenCalledOnce();
|
|
363
|
+
const latest: {
|
|
364
|
+
code: string | null | undefined;
|
|
365
|
+
edit: ReturnType<typeof useNotebookDocumentEdit>;
|
|
366
|
+
} = seen.mock.calls.at(-1)![0];
|
|
367
|
+
expect(latest.code).toBe("second source");
|
|
368
|
+
expect(latest.edit).not.toBe(oldEdit);
|
|
369
|
+
act(() => latest.edit((tx) => tx.setCode("c1", "second draft")));
|
|
370
|
+
expect(seen.mock.calls.at(-1)![0].code).toBe("second draft");
|
|
371
|
+
const stale = vi.fn();
|
|
372
|
+
oldEdit(stale);
|
|
373
|
+
expect(stale).not.toHaveBeenCalled();
|
|
374
|
+
seen.mockClear();
|
|
375
|
+
act(() =>
|
|
376
|
+
first.store.set(
|
|
377
|
+
first.source,
|
|
378
|
+
withCells(document(), {
|
|
379
|
+
c1: cell("c1", "stale host"),
|
|
380
|
+
}),
|
|
381
|
+
),
|
|
382
|
+
);
|
|
383
|
+
expect(seen).not.toHaveBeenCalled();
|
|
384
|
+
});
|
|
385
|
+
|
|
386
|
+
it("requires a document provider", () => {
|
|
387
|
+
expect(() => renderHook(useNotebookDocumentEdit)).toThrow(
|
|
388
|
+
"DocumentProvider",
|
|
389
|
+
);
|
|
390
|
+
});
|
|
391
|
+
});
|
|
@@ -0,0 +1,61 @@
|
|
|
1
|
+
/* Copyright 2026 Marimo. All rights reserved. */
|
|
2
|
+
|
|
3
|
+
import { vi } from "vitest";
|
|
4
|
+
import { test } from "./scenario-fixture";
|
|
5
|
+
|
|
6
|
+
test("closing a session before the debounce prevents submission", async ({
|
|
7
|
+
expect,
|
|
8
|
+
session: { user, notebook, controller },
|
|
9
|
+
}) => {
|
|
10
|
+
user.changeCode("c3", "local");
|
|
11
|
+
const reason = new Error("Editing session ended");
|
|
12
|
+
controller.abort(reason);
|
|
13
|
+
await vi.advanceTimersByTimeAsync(1000);
|
|
14
|
+
expect(notebook.edit).not.toHaveBeenCalled();
|
|
15
|
+
expect(vi.getTimerCount()).toBe(0);
|
|
16
|
+
await expect(user.sync()).rejects.toThrow("not started");
|
|
17
|
+
});
|
|
18
|
+
|
|
19
|
+
test("closing a session before a queued flush starts prevents submission", async ({
|
|
20
|
+
expect,
|
|
21
|
+
session: { user, notebook, controller },
|
|
22
|
+
}) => {
|
|
23
|
+
user.changeCode("c3", "local");
|
|
24
|
+
const flushed = user.sync();
|
|
25
|
+
const reason = new Error("Editing session ended");
|
|
26
|
+
controller.abort(reason);
|
|
27
|
+
await expect(flushed).rejects.toBe(reason);
|
|
28
|
+
expect(notebook.edit).not.toHaveBeenCalled();
|
|
29
|
+
expect(vi.getTimerCount()).toBe(0);
|
|
30
|
+
});
|
|
31
|
+
|
|
32
|
+
test.for(["request", "confirmation", "retry"] as const)(
|
|
33
|
+
"closing a session during %s stops synchronization",
|
|
34
|
+
async (phase, { expect, session: { user, host, notebook, controller } }) => {
|
|
35
|
+
user.changeCode("c3", "local");
|
|
36
|
+
const flushed = user.sync();
|
|
37
|
+
const request = await host.nextRequest();
|
|
38
|
+
if (phase === "confirmation") {
|
|
39
|
+
request.process();
|
|
40
|
+
request.deliverResponse();
|
|
41
|
+
} else if (phase === "retry") {
|
|
42
|
+
request.loseResponse();
|
|
43
|
+
}
|
|
44
|
+
await vi.advanceTimersByTimeAsync(0);
|
|
45
|
+
const reason = new Error("Editing session ended");
|
|
46
|
+
controller.abort(reason);
|
|
47
|
+
await expect(flushed).rejects.toBe(reason);
|
|
48
|
+
if (phase === "request") {
|
|
49
|
+
expect(request.signal.aborted).toBe(true);
|
|
50
|
+
}
|
|
51
|
+
|
|
52
|
+
host.changeCell("c1", { code: "later" });
|
|
53
|
+
host.deliverSnapshot();
|
|
54
|
+
user.changeCode("c3", "after closing");
|
|
55
|
+
await vi.advanceTimersByTimeAsync(1000);
|
|
56
|
+
expect(notebook.edit).toHaveBeenCalledOnce();
|
|
57
|
+
expect(vi.getTimerCount()).toBe(0);
|
|
58
|
+
await expect(user.sync()).rejects.toThrow("not started");
|
|
59
|
+
await expect(user.retry()).rejects.toThrow("not started");
|
|
60
|
+
},
|
|
61
|
+
);
|
|
@@ -0,0 +1,247 @@
|
|
|
1
|
+
/* Copyright 2026 Marimo. All rights reserved. */
|
|
2
|
+
|
|
3
|
+
import type { Result } from "@marimo-team/client";
|
|
4
|
+
import type * as Protocol from "@marimo-team/client/protocol";
|
|
5
|
+
import { atom, createStore } from "jotai/vanilla";
|
|
6
|
+
import { afterEach, beforeEach, describe, expect, it, vi } from "vitest";
|
|
7
|
+
import { Deferred } from "@/utils/Deferred";
|
|
8
|
+
import type { NotebookBackend } from "../../replica/backend";
|
|
9
|
+
import type { Document } from "../../replica/resources/document";
|
|
10
|
+
import { createDocumentModel } from "../model";
|
|
11
|
+
import { type FakeEdit, fakeBackend } from "../../replica/__tests__/fixtures";
|
|
12
|
+
import { cell, document, withCells } from "./fixtures";
|
|
13
|
+
|
|
14
|
+
function response(confirmed: Document): Result<Protocol.EditResponseOutput> {
|
|
15
|
+
return {
|
|
16
|
+
ok: true,
|
|
17
|
+
status: 200,
|
|
18
|
+
data: { document: confirmed.revision, cells: {} },
|
|
19
|
+
};
|
|
20
|
+
}
|
|
21
|
+
|
|
22
|
+
// Observe rejections before advancing timers to avoid unhandled promises.
|
|
23
|
+
async function expectFailure(promise: Promise<unknown>, message: string) {
|
|
24
|
+
await expect(promise).rejects.toThrow(message);
|
|
25
|
+
}
|
|
26
|
+
|
|
27
|
+
async function expectAbort(promise: Promise<unknown>, reason: Error) {
|
|
28
|
+
await expect(promise).rejects.toBe(reason);
|
|
29
|
+
}
|
|
30
|
+
|
|
31
|
+
/** An edit the host received, with its notebook and options. */
|
|
32
|
+
interface SentEdit {
|
|
33
|
+
notebookId: Protocol.NotebookId;
|
|
34
|
+
changes: readonly Protocol.Change[];
|
|
35
|
+
idempotencyKey?: string;
|
|
36
|
+
signal?: AbortSignal;
|
|
37
|
+
}
|
|
38
|
+
|
|
39
|
+
function setup({ connected = true } = {}) {
|
|
40
|
+
const store = createStore();
|
|
41
|
+
const source = atom<Document | null>(document());
|
|
42
|
+
const unmounted = vi.fn();
|
|
43
|
+
source.onMount = () => unmounted;
|
|
44
|
+
const replies: Deferred<Result<Protocol.EditResponseOutput>>[] = [];
|
|
45
|
+
const sent = vi.fn((_edit: SentEdit) => {
|
|
46
|
+
const reply = new Deferred<Result<Protocol.EditResponseOutput>>();
|
|
47
|
+
replies.push(reply);
|
|
48
|
+
return reply.promise;
|
|
49
|
+
});
|
|
50
|
+
const backend = fakeBackend({
|
|
51
|
+
edit: (notebookId, changes, options) =>
|
|
52
|
+
sent({ notebookId, changes, ...options }),
|
|
53
|
+
});
|
|
54
|
+
const backendAtom = atom<NotebookBackend | null>(connected ? backend : null);
|
|
55
|
+
const model = createDocumentModel({
|
|
56
|
+
store,
|
|
57
|
+
documentAtom: source,
|
|
58
|
+
backendAtom,
|
|
59
|
+
});
|
|
60
|
+
const controller = new AbortController();
|
|
61
|
+
lifetimes.push(controller);
|
|
62
|
+
model.start({ signal: controller.signal });
|
|
63
|
+
return {
|
|
64
|
+
store,
|
|
65
|
+
source,
|
|
66
|
+
backend,
|
|
67
|
+
backendAtom,
|
|
68
|
+
model,
|
|
69
|
+
sent,
|
|
70
|
+
replies,
|
|
71
|
+
controller,
|
|
72
|
+
unmounted,
|
|
73
|
+
edit: (code: string) =>
|
|
74
|
+
store.set(model.editAtom, (tx) => tx.setCode("c1", code)),
|
|
75
|
+
status: () => store.get(model.syncStatusAtom),
|
|
76
|
+
value: () => store.get(model.documentAtom)?.cells.get("c1"),
|
|
77
|
+
};
|
|
78
|
+
}
|
|
79
|
+
|
|
80
|
+
const lifetimes: AbortController[] = [];
|
|
81
|
+
beforeEach(() => vi.useFakeTimers());
|
|
82
|
+
afterEach(() => {
|
|
83
|
+
lifetimes.splice(0).forEach((controller) => controller.abort());
|
|
84
|
+
vi.useRealTimers();
|
|
85
|
+
});
|
|
86
|
+
|
|
87
|
+
describe("document synchronization", () => {
|
|
88
|
+
it("subscribes to nothing until started", async () => {
|
|
89
|
+
const store = createStore();
|
|
90
|
+
const source = atom<Document | null>(document());
|
|
91
|
+
const mounted = vi.fn();
|
|
92
|
+
source.onMount = mounted;
|
|
93
|
+
const edit = vi.fn<FakeEdit>();
|
|
94
|
+
const model = createDocumentModel({
|
|
95
|
+
store,
|
|
96
|
+
documentAtom: source,
|
|
97
|
+
backendAtom: atom(fakeBackend({ edit })),
|
|
98
|
+
});
|
|
99
|
+
store.set(model.editAtom, (tx) => tx.setCode("c1", "local"));
|
|
100
|
+
await expect(model.flush()).rejects.toThrow("not started");
|
|
101
|
+
expect(store.get(model.documentAtom)).toBeNull();
|
|
102
|
+
expect(store.get(model.syncStatusAtom)).toEqual({ status: "unavailable" });
|
|
103
|
+
expect(mounted).not.toHaveBeenCalled();
|
|
104
|
+
expect(edit).not.toHaveBeenCalled();
|
|
105
|
+
expect(vi.getTimerCount()).toBe(0);
|
|
106
|
+
});
|
|
107
|
+
|
|
108
|
+
it("rejects an aborted signal before subscribing", () => {
|
|
109
|
+
const store = createStore();
|
|
110
|
+
const source = atom<Document | null>(document());
|
|
111
|
+
const mounted = vi.fn();
|
|
112
|
+
source.onMount = mounted;
|
|
113
|
+
const model = createDocumentModel({
|
|
114
|
+
store,
|
|
115
|
+
documentAtom: source,
|
|
116
|
+
backendAtom: atom(fakeBackend({ edit: vi.fn<FakeEdit>() })),
|
|
117
|
+
});
|
|
118
|
+
const reason = new Error("Session ended");
|
|
119
|
+
expect(() => model.start({ signal: AbortSignal.abort(reason) })).toThrow(
|
|
120
|
+
reason,
|
|
121
|
+
);
|
|
122
|
+
expect(mounted).not.toHaveBeenCalled();
|
|
123
|
+
});
|
|
124
|
+
|
|
125
|
+
it("starts again after a subscriber aborts during startup", async () => {
|
|
126
|
+
const store = createStore();
|
|
127
|
+
const model = createDocumentModel({
|
|
128
|
+
store,
|
|
129
|
+
documentAtom: atom(document()),
|
|
130
|
+
backendAtom: atom(null),
|
|
131
|
+
});
|
|
132
|
+
const controller = new AbortController();
|
|
133
|
+
lifetimes.push(controller);
|
|
134
|
+
const unsubscribe = store.sub(model.documentAtom, () => {
|
|
135
|
+
if (store.get(model.documentAtom)) {
|
|
136
|
+
controller.abort();
|
|
137
|
+
}
|
|
138
|
+
});
|
|
139
|
+
model.start({ signal: controller.signal });
|
|
140
|
+
unsubscribe();
|
|
141
|
+
expect(controller.signal.aborted).toBe(true);
|
|
142
|
+
expect(store.get(model.documentAtom)).toBeNull();
|
|
143
|
+
await expect(model.flush()).rejects.toThrow("not started");
|
|
144
|
+
|
|
145
|
+
const next = new AbortController();
|
|
146
|
+
lifetimes.push(next);
|
|
147
|
+
model.start({ signal: next.signal });
|
|
148
|
+
expect(store.get(model.documentAtom)?.cells.get("c1")?.code).toBe("x = 1");
|
|
149
|
+
});
|
|
150
|
+
|
|
151
|
+
it("starts again without the drafts of a stopped run", () => {
|
|
152
|
+
const { store, model, controller, edit, value } = setup();
|
|
153
|
+
expect(() => model.start({ signal: new AbortController().signal })).toThrow(
|
|
154
|
+
"already started",
|
|
155
|
+
);
|
|
156
|
+
edit("local");
|
|
157
|
+
controller.abort();
|
|
158
|
+
const next = new AbortController();
|
|
159
|
+
lifetimes.push(next);
|
|
160
|
+
model.start({ signal: next.signal });
|
|
161
|
+
expect(value()?.code).toBe("x = 1");
|
|
162
|
+
store.set(model.editAtom, (tx) => tx.setCode("c1", "restarted"));
|
|
163
|
+
expect(value()?.code).toBe("restarted");
|
|
164
|
+
});
|
|
165
|
+
|
|
166
|
+
it("fails without repeating a request when the response names another notebook", async () => {
|
|
167
|
+
const { sent, replies, edit, model, status } = setup();
|
|
168
|
+
edit("local");
|
|
169
|
+
const rejected = expectFailure(model.flush(), "another notebook");
|
|
170
|
+
await vi.advanceTimersByTimeAsync(0);
|
|
171
|
+
replies[0].resolve(response(document("n2")));
|
|
172
|
+
await rejected;
|
|
173
|
+
await vi.advanceTimersByTimeAsync(5000);
|
|
174
|
+
expect(sent).toHaveBeenCalledOnce();
|
|
175
|
+
expect(status()).toMatchObject({ status: "error" });
|
|
176
|
+
});
|
|
177
|
+
|
|
178
|
+
it("ignores a late response and source update after its lifetime ends", async () => {
|
|
179
|
+
const {
|
|
180
|
+
store,
|
|
181
|
+
source,
|
|
182
|
+
model,
|
|
183
|
+
replies,
|
|
184
|
+
sent,
|
|
185
|
+
edit,
|
|
186
|
+
controller,
|
|
187
|
+
unmounted,
|
|
188
|
+
value,
|
|
189
|
+
} = setup();
|
|
190
|
+
edit("local");
|
|
191
|
+
const reason = new Error("Editing session ended");
|
|
192
|
+
const rejected = expectAbort(model.flush(), reason);
|
|
193
|
+
await vi.advanceTimersByTimeAsync(0);
|
|
194
|
+
controller.abort(reason);
|
|
195
|
+
await rejected;
|
|
196
|
+
const accepted = withCells(document(), { c1: cell("c1", "local") });
|
|
197
|
+
replies[0].resolve(response(accepted));
|
|
198
|
+
store.set(source, accepted);
|
|
199
|
+
const staleEdit = vi.fn();
|
|
200
|
+
store.set(model.editAtom, staleEdit);
|
|
201
|
+
await vi.advanceTimersByTimeAsync(1000);
|
|
202
|
+
expect(unmounted).toHaveBeenCalledOnce();
|
|
203
|
+
expect(staleEdit).not.toHaveBeenCalled();
|
|
204
|
+
expect(value()).toBeUndefined();
|
|
205
|
+
expect(sent).toHaveBeenCalledOnce();
|
|
206
|
+
expect(vi.getTimerCount()).toBe(0);
|
|
207
|
+
});
|
|
208
|
+
|
|
209
|
+
it("aborts an in-flight edit when notebook identity changes", async () => {
|
|
210
|
+
const { store, source, sent, edit, model, status } = setup();
|
|
211
|
+
edit("local");
|
|
212
|
+
const flushed = model.flush();
|
|
213
|
+
const rejected = expectFailure(flushed, "notebook changed");
|
|
214
|
+
await vi.advanceTimersByTimeAsync(0);
|
|
215
|
+
store.set(source, document("n2"));
|
|
216
|
+
await rejected;
|
|
217
|
+
expect(sent.mock.calls[0][0].signal?.aborted).toBe(true);
|
|
218
|
+
expect(status()).toMatchObject({ status: "error", retryable: false });
|
|
219
|
+
});
|
|
220
|
+
|
|
221
|
+
it("aborts an in-flight edit when the backend changes and never sends it to the new one", async () => {
|
|
222
|
+
const { store, backendAtom, sent, edit, model } = setup();
|
|
223
|
+
edit("local");
|
|
224
|
+
const rejected = expectFailure(model.flush(), "notebook changed");
|
|
225
|
+
await vi.advanceTimersByTimeAsync(0);
|
|
226
|
+
const other = vi.fn<FakeEdit>();
|
|
227
|
+
store.set(backendAtom, fakeBackend({ edit: other }));
|
|
228
|
+
await rejected;
|
|
229
|
+
expect(sent).toHaveBeenCalledOnce();
|
|
230
|
+
expect(sent.mock.calls[0][0].signal?.aborted).toBe(true);
|
|
231
|
+
expect(other).not.toHaveBeenCalled();
|
|
232
|
+
});
|
|
233
|
+
|
|
234
|
+
it("keeps edits as drafts until a backend is connected", async () => {
|
|
235
|
+
const { store, backend, backendAtom, sent, edit, status, value } = setup({
|
|
236
|
+
connected: false,
|
|
237
|
+
});
|
|
238
|
+
edit("local");
|
|
239
|
+
await vi.advanceTimersByTimeAsync(1000);
|
|
240
|
+
expect(status()).toEqual({ status: "unavailable" });
|
|
241
|
+
expect(value()?.code).toBe("local");
|
|
242
|
+
|
|
243
|
+
store.set(backendAtom, backend);
|
|
244
|
+
await vi.advanceTimersByTimeAsync(400);
|
|
245
|
+
expect(sent).toHaveBeenCalledOnce();
|
|
246
|
+
});
|
|
247
|
+
});
|