@marimo-team/islands 0.24.3-dev70 → 0.24.3-dev72

package/dist/{ErrorBoundary-DRnJ4tjG.js → ErrorBoundary-Wa8uIU73.js}

@@ -135,7 +135,7 @@ function F(e) {
 }
 //#endregion
 //#region src/components/editor/boundary/ErrorBoundary.tsx
-var I = (0, T.lazy)(() => import("./feedback-button-CC0QIjji.js").then((e) => ({ default: e.FeedbackModal }))), L = (e) => {
+var I = (0, T.lazy)(() => import("./feedback-button-CTlzrpEN.js").then((e) => ({ default: e.FeedbackModal }))), L = (e) => {
 	let t = (0, w.c)(2), n;
 	return t[0] === e.children ? n = t[1] : (n = /* @__PURE__ */ (0, j.jsx)(O, {
 		FallbackComponent: R,

package/dist/{code-visibility-BcnKKK7F.js → code-visibility-D4b6xC1_.js}

@@ -18,7 +18,7 @@ function x() {
 		return e.warn("Failed to get version from mount config"), null;
 	}
 }
-var S = x() || "0.24.3-dev70", C = u(S), w = u(!0);
+var S = x() || "0.24.3-dev72", C = u(S), w = u(!0);
 u(null);
 //#endregion
 //#region src/core/wasm/state.ts

package/dist/{common-DeZXXH6G.js → common-DOyLCadc.js}

@@ -12,7 +12,7 @@ import { t as dt } from "./check-B7cIK6gb.js";
 import { A as ft, C as pt, D as mt, M as ht, O as gt, S as _t, _ as vt, b as yt, c as bt, d as xt, g as St, h as Ct, i as wt, m as Tt, n as Et, p as Dt, s as Ot, t as kt, u as At, v as jt, y as Mt } from "./select-C6plhmZr.js";
 import { $ as Nt, $t as Pt, A as Ft, At as It, B as Lt, Bt as Rt, Ct as zt, D as Bt, Dt as Vt, E as Ht, Et as Ut, F as Wt, Ft as Gt, G as Kt, Gt as qt, H as Jt, Ht as Yt, I as Xt, It as Zt, Jt as Qt, K as $t, Kt as en, L as tn, Lt as nn, M as rn, Mt as an, N as on, Nt as sn, O as cn, Ot as ln, P as un, Pt as dn, Q as fn, Qt as pn, R as mn, Rt as hn, S as gn, St as _n, T as vn, Tt as yn, U as bn, Ut as xn, V as Sn, Vt as Cn, W as wn, Wt as Tn, X as En, Xt as Dn, Y as On, Yt as kn, Z as An, Zt as jn, _ as Mn, _t as Nn, a as Pn, an as Fn, b as In, bt as Ln, cn as Rn, ct as zn, d as Bn, dn as Vn, dt as Hn, en as Un, et as Wn, f as Gn, fn as Kn, ft as qn, gt as Jn, h as Yn, ht as Xn, i as Zn, it as Qn, j as $n, jt as er, k as tr, kt as nr, l as rr, ln as ir, lt as ar, mt as or, n as sr, nn as cr, nt as lr, o as ur, on as dr, ot as fr, pn as pr, pt as mr, qt as hr, rn as gr, rt as _r, s as vr, sn as yr, st as br, t as xr, tn as Sr, tt as Cr, u as wr, un as Tr, ut as Er, vt as Dr, w as Or, wt as kr, x as Ar, xt as jr, y as Mr, yt as Nr, z as Pr, zt as Fr } from "./dropdown-menu-CZgjYSAh.js";
 import { _ as Ir, a as Lr, c as Rr, d as zr, i as Br, n as Vr, o as Hr, p as Ur, r as Wr, s as Gr, t as Kr } from "./toDate-WXg1yLcI.js";
-import { n as qr, t as Jr } from "./ErrorBoundary-DRnJ4tjG.js";
+import { n as qr, t as Jr } from "./ErrorBoundary-Wa8uIU73.js";
 import { t as Yr } from "./clsx-CoeGtWvn.js";
 import { t as Xr } from "./react-dom-BUgJ-ab_.js";
 import { t as Zr } from "./jsx-runtime-DgDPJV9S.js";

package/dist/{data-grid-overlay-editor-DpRB7C-a.js → data-grid-overlay-editor-BILyU65g.js}

@@ -2,7 +2,7 @@ import { a as e } from "./rolldown-runtime-Cb76FRpK.js";
 import { t } from "./react-ofJbxM4x.js";
 import { t as n } from "./react-dom-BUgJ-ab_.js";
 import { c as r, l as i, u as a } from "./data-grid-types-BucVmDDp.js";
-import { a as o, i as s, n as c, r as l } from "./glide-data-editor-vv9GN4qa.js";
+import { a as o, i as s, n as c, r as l } from "./glide-data-editor-pQiTAqma.js";
 //#region ../node_modules/.pnpm/@glideapps+glide-data-grid@6.0.4-alpha24_lodash@4.17.21_marked@15.0.12_react-dom@19.2.8_b9e6aec46d952f631ce71b0585d2a8e5/node_modules/@glideapps/glide-data-grid/dist/esm/internal/data-grid-overlay-editor/data-grid-overlay-editor-style.js
 var u = /* @__PURE__ */ e(t(), 1), d = /* @__PURE__ */ e(n(), 1), f = /*#__PURE__*/ o("div")({
 	name: "DataGridOverlayEditorStyle",

package/dist/{feedback-button-CC0QIjji.js → feedback-button-CTlzrpEN.js}

@@ -5,12 +5,12 @@ import { t as i } from "./compiler-runtime-Cj8ZWiNS.js";
 import { t as a } from "./copy-icon-CLdENY__.js";
 import { Bt as o, H as s, Ht as c, L as l, Ot as u, V as d, d as f, hn as p } from "./cells-BLjghRgH.js";
 import { b as m } from "./select-C6plhmZr.js";
-import { n as h, t as ee } from "./globals-Df9GEzVy.js";
+import { n as h, t as ee } from "./globals-dzjJ4Mbf.js";
 import { t as g } from "./jsx-runtime-DgDPJV9S.js";
 import { C as _, _ as te, i as v } from "./useTheme-DzXmYZ5V.js";
 import { a as ne, i as re, n as ie, r as ae } from "./dialog-CgNq6y9-.js";
 import { t as oe } from "./useAsyncData-BD6AStMt.js";
-import { t as se } from "./code-visibility-BcnKKK7F.js";
+import { t as se } from "./code-visibility-D4b6xC1_.js";
 import { i as ce, n as le } from "./error-entries-CJR74llu.js";
 //#region src/components/ui/skeleton.tsx
 var y = /* @__PURE__ */ e(r(), 1), b = i(), x = g(), ue = (e) => {

package/dist/{glide-data-editor-vv9GN4qa.js → glide-data-editor-pQiTAqma.js}

@@ -6,7 +6,7 @@ import { i as s, n as c, r as l } from "./copy-icon-CLdENY__.js";
 import { C as u, c as d, i as f, n as p, s as m, t as h, x as g } from "./select-C6plhmZr.js";
 import { d as _, h as v, i as y, l as b, n as x, o as S, s as C, t as w, u as T } from "./dropdown-menu-CZgjYSAh.js";
 import { c as E, d as D, l as O, n as k, o as A, s as ee, t as j, u as te } from "./glide-utils-B3rUT5PI.js";
-import { n as ne, t as re } from "./ErrorBoundary-DRnJ4tjG.js";
+import { n as ne, t as re } from "./ErrorBoundary-Wa8uIU73.js";
 import { t as M } from "./react-dom-BUgJ-ab_.js";
 import { t as N } from "./jsx-runtime-DgDPJV9S.js";
 import { t as ie } from "./label-BJ49YTvk.js";
@@ -7062,7 +7062,7 @@ function rs(e, t, n, r, i) {
 }
 //#endregion
 //#region ../node_modules/.pnpm/@glideapps+glide-data-grid@6.0.4-alpha24_lodash@4.17.21_marked@15.0.12_react-dom@19.2.8_b9e6aec46d952f631ce71b0585d2a8e5/node_modules/@glideapps/glide-data-grid/dist/esm/data-editor/data-editor.js
-var is = /* @__PURE__ */ e(_r(), 1), as = /* @__PURE__ */ e(xr(), 1), os = K.lazy(async () => await import("./data-grid-overlay-editor-DpRB7C-a.js")), ss = 0;
+var is = /* @__PURE__ */ e(_r(), 1), as = /* @__PURE__ */ e(xr(), 1), os = K.lazy(async () => await import("./data-grid-overlay-editor-BILyU65g.js")), ss = 0;
 function cs(e) {
 	return (0, is.default)((0, as.default)((0, as.default)(e).filter((e) => e.span !== void 0).map((e) => (0, Za.default)((e.span?.[0] ?? 0) + 1, (e.span?.[1] ?? 0) + 1))));
 }
@@ -9293,7 +9293,7 @@ function Vs(e, t, n, r, i) {
 }
 //#endregion
 //#region ../node_modules/.pnpm/@glideapps+glide-data-grid@6.0.4-alpha24_lodash@4.17.21_marked@15.0.12_react-dom@19.2.8_b9e6aec46d952f631ce71b0585d2a8e5/node_modules/@glideapps/glide-data-grid/dist/esm/cells/number-cell.js
-var Hs = K.lazy(async () => await import("./number-overlay-editor-DoPKikqK.js")), Us = {
+var Hs = K.lazy(async () => await import("./number-overlay-editor-Bo2AW2sM.js")), Us = {
 	getAccessibilityString: (e) => e.data?.toString() ?? "",
 	kind: W.Number,
 	needsHover: (e) => e.hoverEffect === !0,

package/dist/{globals-Df9GEzVy.js → globals-dzjJ4Mbf.js}

@@ -1,6 +1,6 @@
 import { g as e } from "./tooltip-qB8NlSqV.js";
 import { _ as t } from "./useTheme-DzXmYZ5V.js";
-import { r as n } from "./code-visibility-BcnKKK7F.js";
+import { r as n } from "./code-visibility-D4b6xC1_.js";
 var r = e("triangle-alert", [
 	["path", {
 		d: "m21.73 18-8-14a2 2 0 0 0-3.48 0l-8 14A2 2 0 0 0 4 21h16a2 2 0 0 0 1.73-3",

package/dist/main.js

@@ -7,7 +7,7 @@ import { C as T, F as E, G as D, I as O, J as k, K as A, L as j, P as M, R as N,
 import { a as se, f as ce, g as V, i as le, n as ue, o as de, p as fe, r as pe, t as me } from "./tooltip-qB8NlSqV.js";
 import { $ as he, $t as ge, At as _e, B as ve, C as ye, D as be, Dt as xe, F as Se, G as Ce, Gt as we, I as Te, J as Ee, L as De, Mt as Oe, Nt as ke, O as Ae, Ot as je, P as Me, Q as Ne, Qt as Pe, S as Fe, T as Ie, Tt as Le, U as Re, Ut as ze, W as Be, Wt as Ve, X as He, Xt as Ue, Y as We, Yt as Ge, Zt as Ke, _n as qe, _t as Je, an as Ye, bn as Xe, c as Ze, cn as Qe, ct as $e, d as et, dn as tt, dt as nt, f as rt, fn as it, ft as at, g as ot, gn as st, gt as ct, h as lt, in as ut, it as dt, jt as ft, k as pt, kt as mt, l as ht, m as gt, mn as _t, n as vt, o as yt, on as bt, p as xt, pn as St, pt as Ct, q as wt, qt as Tt, r as Et, rn as Dt, rt as Ot, u as kt, v as At, w as jt, xt as Mt, yn as Nt, zt as Pt } from "./cells-BLjghRgH.js";
 import { D as Ft, E as It, O as Lt, S as Rt, T as zt, _ as Bt, a as Vt, c as Ht, d as Ut, f as Wt, g as Gt, h as Kt, i as qt, l as Jt, m as Yt, n as Xt, o as Zt, p as Qt, r as $t, s as en, t as tn, u as nn, v as rn, w as an, x as on, y as sn } from "./spec-CAYwFeV6.js";
-import { $t as cn, A as ln, At as un, B as dn, Bt as fn, C as pn, Ct as mn, D as hn, Dt as gn, E as _n, Et as vn, F as yn, Ft as bn, G as xn, Gt as Sn, H as Cn, Ht as wn, I as Tn, It as En, J as Dn, Jt as On, K as kn, Kt as An, L as jn, Lt as Mn, M as Nn, Mt as Pn, N as Fn, Nt as In, O as Ln, Ot as Rn, P as zn, Pt as Bn, Qt as Vn, R as Hn, Rt as Un, S as Wn, St as Gn, T as Kn, Tt as qn, U as Jn, Ut as Yn, V as Xn, Vt as Zn, W as Qn, Wt as $n, X as er, Xt as tr, Y as nr, Yt as rr, Zt as ir, _ as ar, _t as or, an as sr, at as cr, b as lr, c as ur, cn as dr, ct as fr, d as pr, dn as mr, dt as hr, en as gr, et as _r, f as vr, fn as yr, ft as br, g as xr, gt as Sr, h as Cr, hn as wr, ht as Tr, in as Er, it as Dr, j as Or, jt as kr, k as Ar, kt as jr, ln as Mr, lt as Nr, m as Pr, mn as Fr, mt as Ir, nn as Lr, nt as Rr, ot as zr, p as Br, pn as Vr, pt as Hr, q as Ur, qt as Wr, r as Gr, rn as Kr, rt as qr, sn as Jr, st as Yr, t as Xr, tn as Zr, tt as Qr, u as $r, un as ei, ut as ti, v as ni, vt as ri, w as ii, wt as ai, x as oi, y as si, yt as ci, z as li, zt as ui } from "./common-DeZXXH6G.js";
+import { $t as cn, A as ln, At as un, B as dn, Bt as fn, C as pn, Ct as mn, D as hn, Dt as gn, E as _n, Et as vn, F as yn, Ft as bn, G as xn, Gt as Sn, H as Cn, Ht as wn, I as Tn, It as En, J as Dn, Jt as On, K as kn, Kt as An, L as jn, Lt as Mn, M as Nn, Mt as Pn, N as Fn, Nt as In, O as Ln, Ot as Rn, P as zn, Pt as Bn, Qt as Vn, R as Hn, Rt as Un, S as Wn, St as Gn, T as Kn, Tt as qn, U as Jn, Ut as Yn, V as Xn, Vt as Zn, W as Qn, Wt as $n, X as er, Xt as tr, Y as nr, Yt as rr, Zt as ir, _ as ar, _t as or, an as sr, at as cr, b as lr, c as ur, cn as dr, ct as fr, d as pr, dn as mr, dt as hr, en as gr, et as _r, f as vr, fn as yr, ft as br, g as xr, gt as Sr, h as Cr, hn as wr, ht as Tr, in as Er, it as Dr, j as Or, jt as kr, k as Ar, kt as jr, ln as Mr, lt as Nr, m as Pr, mn as Fr, mt as Ir, nn as Lr, nt as Rr, ot as zr, p as Br, pn as Vr, pt as Hr, q as Ur, qt as Wr, r as Gr, rn as Kr, rt as qr, sn as Jr, st as Yr, t as Xr, tn as Zr, tt as Qr, u as $r, un as ei, ut as ti, v as ni, vt as ri, w as ii, wt as ai, x as oi, y as si, yt as ci, z as li, zt as ui } from "./common-DOyLCadc.js";
 import { c as di, i as fi, n as pi, o as mi, t as hi } from "./useLifecycle-DPgiiwwU.js";
 import { t as gi } from "./check-B7cIK6gb.js";
 import { A as _i, C as vi, D as yi, E as bi, M as xi, O as Si, S as Ci, T as wi, a as Ti, b as Ei, c as Di, i as Oi, k as ki, l as Ai, n as ji, o as Mi, r as Ni, s as Pi, t as Fi, w as Ii, x as Li } from "./select-C6plhmZr.js";
@@ -15,8 +15,8 @@ import { B as Ri, C as zi, J as Bi, Mt as Vi, Nt as Hi, S as Ui, Vt as Wi, W as
 import { _ as ha, c as ga, d as _a, f as va, g as ya, h as ba, m as xa, u as Sa, v as Ca } from "./toDate-WXg1yLcI.js";
 import { a as wa, c as Ta, d as Ea, f as Da, i as Oa, l as ka, n as Aa, o as ja, r as Ma, s as Na, t as Pa } from "./process-output-BurYfV57.js";
 import { a as Fa, c as Ia, d as La, i as Ra, l as za, r as Ba, u as Va } from "./glide-utils-B3rUT5PI.js";
-import { n as Ha, t as Ua } from "./ErrorBoundary-DRnJ4tjG.js";
-import { n as Wa, t as Ga } from "./globals-Df9GEzVy.js";
+import { n as Ha, t as Ua } from "./ErrorBoundary-Wa8uIU73.js";
+import { n as Wa, t as Ga } from "./globals-dzjJ4Mbf.js";
 import { t as Ka } from "./react-dom-BUgJ-ab_.js";
 import { t as qa } from "./jsx-runtime-DgDPJV9S.js";
 import { B as Ja, C as H, Ct as Ya, D as Xa, E as Za, Et as Qa, F as U, G as $a, H as eo, I as to, L as no, M as W, N as ro, O as io, P as ao, Q as oo, R as so, S as G, St as co, Tt as lo, U as uo, W as fo, Y as po, Z as mo, _ as ho, a as go, b as _o, bt as vo, c as yo, d as bo, dt as xo, f as So, g as Co, gt as wo, h as To, i as Eo, j as K, k as Do, l as Oo, m as ko, n as Ao, o as jo, p as Mo, r as No, s as Po, u as Fo, v as Io, vt as Lo, w as Ro, wt as zo, x as Bo, xt as Vo, y as Ho, yt as Uo } from "./use-restore-focus-D5D9xEjZ.js";
@@ -40,7 +40,7 @@ import { t as Ns } from "./useAsyncData-BD6AStMt.js";
 import { i as Ps, n as Fs, r as Is } from "./formats-BR5iGb4s.js";
 import { M as Ls, N as Rs } from "./vega-loader.browser-D95WfUcj.js";
 import { a as zs, n as Bs } from "./loader-D2W4oM-3.js";
-import { c as Vs, d as Hs, i as Us, l as Ws, o as Gs, s as Ks, t as qs, u as Js } from "./code-visibility-BcnKKK7F.js";
+import { c as Vs, d as Hs, i as Us, l as Ws, o as Gs, s as Ks, t as qs, u as Js } from "./code-visibility-D4b6xC1_.js";
 import { t as Ys } from "./useDeepCompareMemoize-D_HNdyvI.js";
 import { t as Xs } from "./esm-BElI3qKH.js";
 //#region ../node_modules/.pnpm/iconify-icon@2.3.0/node_modules/iconify-icon/dist/iconify-icon.mjs
@@ -10458,7 +10458,7 @@ function RC(e, t) {
 }
 //#endregion
 //#region src/plugins/impl/DataEditorPlugin.tsx
-var zC = J.lazy(() => import("./glide-data-editor-vv9GN4qa.js").then((e) => e.t)), BC = xS("marimo-data-editor", { cssStyles: [WS] }).withData(W({
+var zC = J.lazy(() => import("./glide-data-editor-pQiTAqma.js").then((e) => e.t)), BC = xS("marimo-data-editor", { cssStyles: [WS] }).withData(W({
 	initialValue: W({ edits: G(W({
 		rowIdx: K(),
 		columnId: U(),
@@ -29922,7 +29922,7 @@ function bR(e) {
 }
 //#endregion
 //#region src/components/editor/renderers/slides-layout/slides-layout.tsx
-var xR = J.lazy(() => import("./reveal-component-ykpOHQgq.js")), SR = (e) => {
+var xR = J.lazy(() => import("./reveal-component-Dldi8B9A.js")), SR = (e) => {
 	let t = (0, X.c)(33), { layout: n, setLayout: r, cells: i, mode: a } = e, o = rs(Ce), s = a === "read" || o, c = rs(xt) > 1, [l, u] = (0, J.useState)(null), d;
 	t[0] !== i || t[1] !== n ? (d = vP(i, n), t[0] = i, t[1] = n, t[2] = d) : d = t[2];
 	let { slideCells: f, skippedIds: p, noOutputIds: m, slideTypes: h, startCellIndex: g } = d, _;

package/dist/{number-overlay-editor-DoPKikqK.js → number-overlay-editor-Bo2AW2sM.js}

@@ -1,6 +1,6 @@
 import { a as e } from "./rolldown-runtime-Cb76FRpK.js";
 import { t } from "./react-ofJbxM4x.js";
-import { a as n } from "./glide-data-editor-vv9GN4qa.js";
+import { a as n } from "./glide-data-editor-pQiTAqma.js";
 //#region ../node_modules/.pnpm/@glideapps+glide-data-grid@6.0.4-alpha24_lodash@4.17.21_marked@15.0.12_react-dom@19.2.8_b9e6aec46d952f631ce71b0585d2a8e5/node_modules/@glideapps/glide-data-grid/dist/esm/internal/data-grid-overlay-editor/private/number-overlay-editor-style.js
 var r = /* @__PURE__ */ e(t()), i = /*#__PURE__*/ n("div")({
 	name: "NumberOverlayEditorStyle",

package/dist/{reveal-component-ykpOHQgq.js → reveal-component-Dldi8B9A.js}

@@ -4,13 +4,13 @@ import { t as o } from "./react-ofJbxM4x.js";
 import { t as s } from "./compiler-runtime-Cj8ZWiNS.js";
 import { g as c, t as l } from "./tooltip-qB8NlSqV.js";
 import { G as u, w as d } from "./cells-BLjghRgH.js";
-import { $ as f, Q as p, Z as m, an as h, i as g, l as _, ln as v, n as y, on as b, r as x, t as S, u as C } from "./common-DeZXXH6G.js";
+import { $ as f, Q as p, Z as m, an as h, i as g, l as _, ln as v, n as y, on as b, r as x, t as S, u as C } from "./common-DOyLCadc.js";
 import { M as w } from "./select-C6plhmZr.js";
 import { J as T } from "./dropdown-menu-CZgjYSAh.js";
 import { t as E } from "./jsx-runtime-DgDPJV9S.js";
 import { q as ee } from "./use-restore-focus-D5D9xEjZ.js";
 import { A as D, C as O, b as k } from "./useTheme-DzXmYZ5V.js";
-import { a as A, n as te, t as ne } from "./code-visibility-BcnKKK7F.js";
+import { a as A, n as te, t as ne } from "./code-visibility-D4b6xC1_.js";
 /* empty css                */
 var j = c("sticky-note", [["path", {
 	d: "M21 9a2.4 2.4 0 0 0-.706-1.706l-3.588-3.588A2.4 2.4 0 0 0 15 3H5a2 2 0 0 0-2 2v14a2 2 0 0 0 2 2h14a2 2 0 0 0 2-2z",

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@marimo-team/islands",
-  "version": "0.24.3-dev70",
+  "version": "0.24.3-dev72",
   "main": "dist/main.js",
   "types": "dist/index.d.ts",
   "type": "module",

package/src/components/data-table/__tests__/filters.test.ts

@@ -368,11 +368,10 @@ describe("schema validation", () => {
       operator: ">=",
       value: 18,
     });
-    expect(result.success).toBe(true);
-    if (result.success) {
-      expect(result.data.type).toBe("condition");
-      expect(result.data.negate).toBe(false);
-    }
+    expect(result).toMatchObject({
+      success: true,
+      data: { type: "condition", negate: false },
+    });
   });
 
   it("FilterConditionSchema defaults type and negate", () => {
@@ -381,11 +380,10 @@ describe("schema validation", () => {
       operator: "==",
       value: 5,
     });
-    expect(result.success).toBe(true);
-    if (result.success) {
-      expect(result.data.type).toBe("condition");
-      expect(result.data.negate).toBe(false);
-    }
+    expect(result).toMatchObject({
+      success: true,
+      data: { type: "condition", negate: false },
+    });
   });
 
   it("FilterConditionSchema accepts negate=true", () => {
@@ -395,10 +393,10 @@ describe("schema validation", () => {
       value: 5,
       negate: true,
     });
-    expect(result.success).toBe(true);
-    if (result.success) {
-      expect(result.data.negate).toBe(true);
-    }
+    expect(result).toMatchObject({
+      success: true,
+      data: { negate: true },
+    });
   });
 
   it("FilterGroupSchema accepts valid group", () => {
@@ -440,12 +438,14 @@ describe("schema validation", () => {
 
   it("FilterGroupSchema defaults fields", () => {
     const result = FilterGroupSchema.safeParse({});
-    expect(result.success).toBe(true);
-    if (result.success) {
-      expect(result.data.type).toBe("group");
-      expect(result.data.operator).toBe("and");
-      expect(result.data.children).toEqual([]);
-      expect(result.data.negate).toBe(false);
-    }
+    expect(result).toMatchObject({
+      success: true,
+      data: {
+        type: "group",
+        operator: "and",
+        children: [],
+        negate: false,
+      },
+    });
   });
 });

package/src/components/data-table/charts/__tests__/spec.test.ts

@@ -169,16 +169,11 @@ describe("getAxisEncoding", () => {
 
       const expectedAggregate = (result as { aggregate?: string }).aggregate;
 
-      // For aggregations that are not valid for string data types, we should return undefined
-      if (
-        agg === NONE_VALUE ||
-        agg === BIN_AGGREGATION ||
-        !STRING_AGGREGATION_FNS.includes(agg)
-      ) {
-        expect(expectedAggregate).toBeUndefined();
-      } else if (STRING_AGGREGATION_FNS.includes(agg)) {
-        expect(expectedAggregate).toEqual(agg);
-      }
+      const isStringAggregation =
+        agg !== NONE_VALUE &&
+        agg !== BIN_AGGREGATION &&
+        STRING_AGGREGATION_FNS.includes(agg);
+      expect(expectedAggregate).toBe(isStringAggregation ? agg : undefined);
     }
   });
 

package/src/components/editor/file-tree/__tests__/file-explorer-navigation.test.tsx

@@ -278,18 +278,20 @@ describe("file browser navigation", () => {
         await new Promise(requestAnimationFrame);
       });
       dragEvent(folder, "drop");
-      if (canMove) {
-        await waitFor(() =>
-          expect(client.sendRenameFileOrFolder).toHaveBeenCalledExactlyOnceWith(
-            {
-              path: "/workspace/notes.txt",
-              newPath: "/workspace/data/notes.txt",
-            },
-          ),
+      await waitFor(() => {
+        expect(client.sendRenameFileOrFolder.mock.calls).toEqual(
+          canMove
+            ? [
+                [
+                  {
+                    path: "/workspace/notes.txt",
+                    newPath: "/workspace/data/notes.txt",
+                  },
+                ],
+              ]
+            : [],
         );
-      } else {
-        expect(client.sendRenameFileOrFolder).not.toHaveBeenCalled();
-      }
+      });
       dragEvent(source, "dragend");
     },
   );

package/src/components/editor/renderers/slides-layout/__tests__/plugin.test.ts

@@ -217,7 +217,7 @@ interface BackwardsCompatCase {
    * properties need to be listed — extra properties are ignored.
    */
   expected: {
-    deck?: unknown;
+    deck: unknown;
     cellIds: string[];
     cellEntries?: Array<[string, unknown]>;
   };
@@ -355,9 +355,7 @@ describe("SlidesLayoutPlugin backwards compatibility", () => {
       // known field, the deserialized config won't carry it and the
       // assertion below fails.
       const layout = SlidesLayoutPlugin.deserializeLayout(parsed.data, cells);
-      if (expected.deck !== undefined) {
-        expect(layout.deck).toEqual(expected.deck);
-      }
+      expect(layout.deck).toEqual(expected.deck);
       for (const [cellEntryId, expectedConfig] of expected.cellEntries ?? []) {
         expect(layout.cells.get(cellId(cellEntryId))).toMatchObject(
           expectedConfig as object,
@@ -379,9 +377,7 @@ describe("SlidesLayoutPlugin backwards compatibility", () => {
           expectedConfig as object,
         );
       }
-      if (expected.deck !== undefined) {
-        expect(redeserialized.deck).toEqual(expected.deck);
-      }
+      expect(redeserialized.deck).toEqual(expected.deck);
     },
   );
 

package/src/components/ui/__tests__/badge.test.ts

@@ -54,10 +54,12 @@ describe("badgeVariants", () => {
           ].join(" "),
         ),
       );
-      if (variant !== "outline") {
-        expect(classes).toContain("forced-colors:hover:bg-[Highlight]");
-        expect(classes).toContain("forced-colors:hover:text-[HighlightText]");
-      }
+      expect(classes.includes("forced-colors:hover:bg-[Highlight]")).toBe(
+        variant !== "outline",
+      );
+      expect(classes.includes("forced-colors:hover:text-[HighlightText]")).toBe(
+        variant !== "outline",
+      );
     },
   );
 });

package/src/core/ai/context/providers/__tests__/error.test.ts

@@ -104,10 +104,10 @@ describe("ErrorContextProvider", () => {
       ]);
 
       const items = provider.getItems();
-      expect(items[1].data.type).toBe("cell-error");
-      if (items[1].data.type === "cell-error") {
-        expect(items[1].data.error.cellName).toBe("cell-0");
-      }
+      expect(items[1].data).toMatchObject({
+        type: "cell-error",
+        error: { cellName: "cell-0" },
+      });
       expect(items[1].name).toBe("Error: cell-0");
       expect(items[1].description).toBe("Invalid syntax");
     });
@@ -150,25 +150,19 @@ describe("ErrorContextProvider", () => {
       `);
 
       // Test the info function
-      expect(completion.info).toBeDefined();
-      if (typeof completion.info === "function") {
-        const infoResult = completion.info(completion);
-        if (
-          infoResult &&
-          typeof infoResult === "object" &&
-          "dom" in infoResult
-        ) {
-          const infoElement = infoResult.dom as HTMLElement;
-          expect(infoElement.tagName).toBe("DIV");
-          expect(infoElement.textContent).toContain("Errors");
-          expect(infoElement.textContent).toContain("2 errors");
-        } else if (infoResult && "tagName" in (infoResult as any)) {
-          const infoElement = infoResult as HTMLElement;
-          expect(infoElement.tagName).toBe("DIV");
-          expect(infoElement.textContent).toContain("Errors");
-          expect(infoElement.textContent).toContain("2 errors");
-        }
+      if (typeof completion.info !== "function") {
+        throw new Error("Expected completion info to be a function");
+      }
+      const infoResult = completion.info(completion);
+      if (!infoResult || typeof infoResult !== "object") {
+        throw new Error("Expected completion info to return an element");
       }
+      const infoElement = (
+        "dom" in infoResult ? infoResult.dom : infoResult
+      ) as HTMLElement;
+      expect(infoElement.tagName).toBe("DIV");
+      expect(infoElement.textContent).toContain("Errors");
+      expect(infoElement.textContent).toContain("2 errors");
     });
 
     it("should handle single error correctly", () => {

package/src/core/cells/__tests__/pending-delete-service.test.tsx

@@ -194,9 +194,10 @@ describe("pending-delete-service", () => {
       wrapper,
     });
     expect(result.current.isPending).toBe(true);
-    if (result.current.isPending) {
-      expect(result.current.shouldConfirmDelete).toBe(false);
-      expect("confirm" in result.current).toBe(false);
+    if (!result.current.isPending) {
+      throw new Error("Expected cell deletion to be pending");
     }
+    expect(result.current.shouldConfirmDelete).toBe(false);
+    expect("confirm" in result.current).toBe(false);
   });
 });

package/src/core/codemirror/find-replace/__tests__/navigate.test.ts

@@ -500,17 +500,15 @@ describe("navigate", () => {
       const view1Positions = result.position.get(view1);
       const view2Positions = result.position.get(view2);
 
-      if (view1Positions) {
-        expect(view1Positions.size).toBeGreaterThan(0);
-        // Check that position keys are in "from:to" format
-        for (const key of view1Positions.keys()) {
-          expect(key).toMatch(/^\d+:\d+$/);
-        }
+      invariant(view1Positions, "Expected positions for the first view");
+      expect(view1Positions.size).toBeGreaterThan(0);
+      // Check that position keys are in "from:to" format
+      for (const key of view1Positions.keys()) {
+        expect(key).toMatch(/^\d+:\d+$/);
       }
 
-      if (view2Positions) {
-        expect(view2Positions.size).toBeGreaterThan(0);
-      }
+      invariant(view2Positions, "Expected positions for the second view");
+      expect(view2Positions.size).toBeGreaterThan(0);
     });
 
     it("should handle regex patterns", () => {
@@ -654,13 +652,7 @@ describe("navigate", () => {
 
       const result = findNext();
       // Should still work by starting from index 0
-      if (result) {
-        expect(result).toBeTruthy();
-        invariant(result, "findNext should return a result");
-      } else {
-        // It's ok if no match is found since "hello" might not be in the mock views
-        expect(result).toBe(false);
-      }
+      expect(result).toMatchObject({ from: 0, to: 5 });
     });
 
     it("should handle malformed regex in replace pattern", () => {

package/src/core/codemirror/language/embedded/__tests__/embedded-python.test.ts

@@ -17,14 +17,10 @@ describe("parsePython", () => {
     const pythonParser = python().language.parser;
     const config = parsePython(pythonParser, IS_ACTIVE);
 
-    expect(config.defineNodes).toBeDefined();
-    if (config.defineNodes) {
-      expect(config.defineNodes).toHaveLength(2);
-      // @ts-expect-error - we know that the defineNodes are NodeSpec
-      expect(config.defineNodes[0].name).toBe("Python");
-      // @ts-expect-error - we know that the defineNodes are NodeSpec
-      expect(config.defineNodes[1].name).toBe("PythonMark");
-    }
+    expect(config.defineNodes).toMatchObject([
+      { name: "Python" },
+      { name: "PythonMark" },
+    ]);
   });
 
   it("should not parse double curly braces", () => {

package/src/core/export/__tests__/hooks.test.ts

@@ -552,12 +552,7 @@ describe("useEnrichCellOutputs", () => {
     // Verify the exact return type structure
     expect(output).toHaveProperty(cid);
     const cellOutput = output[cid];
-    expect(cellOutput).toBeDefined();
-    expect(Array.isArray(cellOutput)).toBe(true);
-    if (cellOutput) {
-      expect(cellOutput[0]).toBe("image/png");
-      expect(cellOutput[1]).toBe(mockDataUrl);
-    }
+    expect(cellOutput).toEqual(["image/png", mockDataUrl]);
   });
 });
 

package/src/core/network/__tests__/requests-network.test.ts

@@ -69,9 +69,10 @@ describe("createNetworkRequests", () => {
     const methodNames = Object.keys(requests);
 
     for (const methodName of methodNames) {
-      if (!ACTIONS[methodName as keyof typeof ACTIONS]) {
-        expect.fail(`Method ${methodName} has no action defined`);
-      }
+      expect(
+        ACTIONS[methodName as keyof typeof ACTIONS],
+        `Method ${methodName} has no action defined`,
+      ).toBeDefined();
     }
   });
 

package/src/core/static/__tests__/files.test.ts

@@ -291,14 +291,10 @@ describe("patchVegaLoader - loader.load", () => {
     const loader = createLoader();
     const unpatch = patchVegaLoader(loader, virtualFiles);
 
-    try {
-      const content = await loader.load("file:///path/to/@file/vega-data.json");
-      expect(content).toBe('{"values": [1, 2, 3]}');
-    } catch (error) {
-      // If it falls back to original loader and fails, that's expected for file:// URLs
-      // The important thing is that the virtual file lookup was attempted
-      expect(error).toBeDefined();
-    }
+    const content = await loader
+      .load("file:///path/to/@file/vega-data.json")
+      .catch(() => undefined);
+    expect([undefined, '{"values": [1, 2, 3]}']).toContain(content);
 
     unpatch();
   });
@@ -340,15 +336,10 @@ describe("patchVegaLoader - loader.load", () => {
     }
 
     // Test file:// URL separately since it might fallback
-    try {
-      const content = await loader.load(
-        "file:///local/path/@file/pattern-test.json",
-      );
-      expect(content).toBe('{"pattern": "test"}');
-    } catch (error) {
-      // Expected if it falls back to original loader
-      expect(error).toBeDefined();
-    }
+    const fileUrlContent = await loader
+      .load("file:///local/path/@file/pattern-test.json")
+      .catch(() => undefined);
+    expect([undefined, '{"pattern": "test"}']).toContain(fileUrlContent);
 
     unpatch();
   });
@@ -640,12 +631,9 @@ describe("error handling and edge cases", () => {
     expect(window.fetch).toBe(originalFetch);
 
     // Test that the loader functions are functional (they should work normally)
-    try {
-      await loader.load("http://example.com/test.json");
-    } catch (error) {
-      // Expected to fail for non-existent URLs, but the function should be callable
-      expect(error).toBeDefined();
-    }
+    // A network failure is expected for this non-existent URL. A custom loader
+    // may also handle it, so only verify that the restored method is callable.
+    await loader.load("http://example.com/test.json").catch(() => undefined);
   });
 
   it("should handle Request objects in patchFetch", async () => {
@@ -676,13 +664,7 @@ describe("error handling and edge cases", () => {
       .mockImplementation(Functions.NOOP);
 
     // This should catch the error and fallback to original fetch
-    try {
-      await window.fetch("not-a-valid-url");
-      // If it doesn't throw, that's also fine (fallback behavior)
-    } catch (error) {
-      // Expected behavior - invalid URLs should be handled gracefully
-      expect(error).toBeDefined();
-    }
+    await window.fetch("not-a-valid-url").catch(() => undefined);
 
     unpatch();
     loggerSpy.mockRestore();

package/src/core/websocket/__tests__/useMarimoKernelConnection.test.ts

@@ -47,9 +47,10 @@ describe("classifyCloseEvent", () => {
         state: WebSocketState.CLOSED,
         code: WebSocketClosedReason.KERNEL_DISCONNECTED,
       });
-      if (decision.kind === "terminal") {
-        expect(decision.closeTransport).toBe(true);
-      }
+      expect(decision).toMatchObject({
+        kind: "terminal",
+        closeTransport: true,
+      });
     });
 
     it("MARIMO_KERNEL_STARTUP_ERROR → terminal + closeTransport", () => {
@@ -59,9 +60,10 @@ describe("classifyCloseEvent", () => {
         state: WebSocketState.CLOSED,
         code: WebSocketClosedReason.KERNEL_STARTUP_ERROR,
       });
-      if (decision.kind === "terminal") {
-        expect(decision.closeTransport).toBe(true);
-      }
+      expect(decision).toMatchObject({
+        kind: "terminal",
+        closeTransport: true,
+      });
     });
   });
 

package/src/plugins/impl/vega/__tests__/make-selectable.test.ts

@@ -986,22 +986,15 @@ describe("makeSelectable", () => {
     expect(paramNames).toContain("legend_selection_Origin");
 
     // Nested specs should NOT have params (they should be hoisted)
-    if ("hconcat" in newSpec) {
-      for (const subSpec of newSpec.hconcat) {
-        if ("vconcat" in subSpec) {
-          invariant("vconcat" in subSpec, "subSpec should have vconcat");
-          for (const innerSpec of subSpec.vconcat) {
-            expect("params" in innerSpec).toBe(false);
-            // But should have opacity encoding
-            if ("mark" in innerSpec) {
-              expect(innerSpec.encoding?.opacity).toBeDefined();
-            }
-          }
-        } else if ("mark" in subSpec) {
-          expect(subSpec.params).toBeUndefined();
-          // But should have opacity encoding
-          expect(subSpec.encoding?.opacity).toBeDefined();
-        }
+    invariant("hconcat" in newSpec, "expected a horizontal concat spec");
+    for (const subSpec of newSpec.hconcat) {
+      const innerSpecs = "vconcat" in subSpec ? subSpec.vconcat : [subSpec];
+      for (const innerSpec of innerSpecs) {
+        expect(
+          "params" in innerSpec ? innerSpec.params : undefined,
+        ).toBeUndefined();
+        invariant("mark" in innerSpec, "expected a marked nested spec");
+        expect(innerSpec.encoding?.opacity).toBeDefined();
       }
     }
     expect(newSpec).toMatchSnapshot();
@@ -1045,10 +1038,9 @@ describe("makeSelectable", () => {
     expect(topLevelParamNames).toContain("select_interval");
 
     // Nested specs should not have params
-    if ("hconcat" in newSpec) {
-      for (const subSpec of newSpec.hconcat) {
-        expect("params" in subSpec).toBe(false);
-      }
+    invariant("hconcat" in newSpec, "expected a horizontal concat spec");
+    for (const subSpec of newSpec.hconcat) {
+      expect("params" in subSpec).toBe(false);
     }
     expect(newSpec).toMatchSnapshot();
     expect(parse(newSpec)).toBeDefined();
@@ -1085,12 +1077,11 @@ describe("makeSelectable", () => {
 
     // But chart selection params should NOT be hoisted (different types)
     // Bar gets point+interval with x encoding, area gets point with color encoding
-    if ("hconcat" in newSpec) {
-      const subspecParams = newSpec.hconcat.flatMap((subSpec) =>
-        "params" in subSpec ? subSpec.params : [],
-      );
-      expect(subspecParams.length).toBeGreaterThan(0);
-    }
+    invariant("hconcat" in newSpec, "expected a horizontal concat spec");
+    const subspecParams = newSpec.hconcat.flatMap((subSpec) =>
+      "params" in subSpec ? subSpec.params : [],
+    );
+    expect(subspecParams.length).toBeGreaterThan(0);
     expect(newSpec).toMatchSnapshot();
     expect(parse(newSpec)).toBeDefined();
   });
@@ -1120,13 +1111,12 @@ describe("makeSelectable", () => {
 
     // Chart selection params should NOT be hoisted (different encodings)
     // Point chart uses x,y encodings, arc chart uses color encoding
-    if ("hconcat" in newSpec) {
-      const subspecParams = newSpec.hconcat.flatMap((subSpec) =>
-        "params" in subSpec ? subSpec.params : [],
-      );
-      // Both subspecs should have their own params
-      expect(subspecParams.length).toBeGreaterThan(0);
-    }
+    invariant("hconcat" in newSpec, "expected a horizontal concat spec");
+    const subspecParams = newSpec.hconcat.flatMap((subSpec) =>
+      "params" in subSpec ? subSpec.params : [],
+    );
+    // Both subspecs should have their own params
+    expect(subspecParams.length).toBeGreaterThan(0);
     expect(newSpec).toMatchSnapshot();
     expect(parse(newSpec)).toBeDefined();
   });
@@ -1161,13 +1151,12 @@ describe("makeSelectable", () => {
 
     // But chart-specific params may or may not be hoisted depending on if they match
     // The point chart has x,y encodings, bar has x encoding - these differ
-    if ("hconcat" in newSpec) {
-      const subspecParams = newSpec.hconcat.flatMap((subSpec) =>
-        "params" in subSpec ? subSpec.params : [],
-      );
-      // Each subspec should have its own chart selection params
-      expect(subspecParams.length).toBeGreaterThan(0);
-    }
+    invariant("hconcat" in newSpec, "expected a horizontal concat spec");
+    const subspecParams = newSpec.hconcat.flatMap((subSpec) =>
+      "params" in subSpec ? subSpec.params : [],
+    );
+    // Each subspec should have its own chart selection params
+    expect(subspecParams.length).toBeGreaterThan(0);
     expect(newSpec).toMatchSnapshot();
     expect(parse(newSpec)).toBeDefined();
   });
@@ -1331,16 +1320,21 @@ describe("pan/zoom modifier key (issue #3812)", () => {
     const newSpec = makeSelectable(spec, {});
     const mod = getPanZoomModifier();
 
-    for (const param of newSpec.params ?? []) {
-      if (!("select" in param) || typeof param.select !== "object") {
-        continue;
-      }
-      const select = param.select as { on?: string; translate?: string };
+    const eventSelections = (newSpec.params ?? [])
+      .flatMap((param) =>
+        "select" in param && typeof param.select === "object"
+          ? [param.select as { on?: string; translate?: string }]
+          : [],
+      )
+      .filter(
+        (select): select is { on: string; translate?: string } =>
+          typeof select.on === "string" && select.on.includes("event."),
+      );
+    expect(eventSelections).not.toHaveLength(0);
+    for (const select of eventSelections) {
       // The selection gating and pan/zoom must agree on the modifier, otherwise
       // one gesture triggers both.
-      if (typeof select.on === "string" && select.on.includes("event.")) {
-        expect(select.on).toContain(`event.${mod}`);
-      }
+      expect(select.on).toContain(`event.${mod}`);
     }
   });
 });

package/src/utils/__tests__/path.test.ts

@@ -213,15 +213,16 @@ describe("PathBuilder", () => {
       "should handle unicode and spaces in path operations: %s",
       (filename) => {
         const baseName = Paths.basename(filename);
-        const extension = Paths.extension(filename);
 
         // Should preserve unicode characters in basename
         expect(baseName).toContain(filename.split(".")[0]);
+      },
+    );
 
-        // Should correctly extract extension
-        if (filename.includes(".")) {
-          expect(extension).toBe(filename.split(".").pop());
-        }
+    it.each(EDGE_CASE_FILENAMES)(
+      "should extract the extension from edge-case filenames: %s",
+      (filename) => {
+        expect(Paths.extension(filename)).toBe("py");
       },
     );
   });