@marimo-team/islands 0.24.1-dev73 → 0.24.1-dev76

package/dist/{common-D37C4Qj7.js → common-nuR9LfxI.js}

@@ -18570,19 +18570,23 @@ function Kj(e, t) {
 //#endregion
 //#region src/core/packages/toast-components.tsx
 var qj = (e, t) => {
+	let n = Array.isArray(e) ? e : [e];
 	m(t ? {
-		title: "Failed to add package",
+		title: n.length > 1 ? "Failed to add packages" : "Failed to add package",
 		description: t,
 		variant: "danger"
 	} : {
-		title: "Package added",
+		title: n.length > 1 ? "Packages added" : "Package added",
 		description: /* @__PURE__ */ (0, U.jsxs)("div", { children: [/* @__PURE__ */ (0, U.jsxs)("div", { children: [
-			"The package ",
-			/* @__PURE__ */ (0, U.jsx)(ej, {
+			n.length > 1 ? "The packages " : "The package ",
+			n.map((e, t) => /* @__PURE__ */ (0, U.jsxs)("span", { children: [t > 0 && ", ", /* @__PURE__ */ (0, U.jsx)(ej, {
 				className: "inline",
 				children: e
-			}),
-			" and its dependencies has been added to your environment."
+			})] }, e)),
+			" ",
+			"and ",
+			n.length > 1 ? "their" : "its",
+			" dependencies have been added to your environment."
 		] }), /* @__PURE__ */ (0, U.jsx)("div", {
 			className: "text-xs text-muted-foreground mt-1",
 			children: "Some Python packages may require a kernel restart to see changes."
@@ -18598,11 +18602,8 @@ function Jj() {
 		handleInstallPackages: async (e, r) => {
 			t(!0);
 			try {
-				for (let [t, r] of e.entries()) {
-					let i = await n({ package: r });
-					i.success ? qj(r) : qj(r, i.error), t < e.length - 1 && await new Promise((e) => setTimeout(e, 1e3));
-				}
-				r?.();
+				let t = await n({ package: e.join(" ") });
+				t.success ? qj(e) : qj(e, t.error), r?.();
 			} catch (e) {
 				i.error(e);
 			} finally {
@@ -31786,7 +31787,7 @@ function QG() {
 		return i.warn("Failed to get version from mount config"), null;
 	}
 }
-var $G = QG() || "0.24.1-dev73", eK = Ci($G), tK = Ci(!0);
+var $G = QG() || "0.24.1-dev76", eK = Ci($G), tK = Ci(!0);
 Ci(null);
 //#endregion
 //#region src/core/meta/code-visibility.ts

package/dist/main.js

@@ -6,7 +6,7 @@ import { n as x, r as S, t as C } from "./copy-D9h0DuU7.js";
 import { $ as w, $n as T, B as E, Bn as D, C as O, Cn as k, Ct as A, Dn as j, Dt as M, Et as N, F as P, Fn as F, Ft as I, G as L, Gn as R, Ht as ee, I as z, It as te, J as ne, K as re, Kn as ie, Kt as ae, L as oe, Ln as se, Mn as ce, Mt as le, Nn as ue, On as de, Ot as fe, P as pe, Pn as me, Qn as he, R as ge, Rn as _e, Sn as ve, St as ye, U as be, Un as xe, Ut as Se, W as Ce, Wn as we, Wt as Te, X as Ee, Xn as De, Xt as Oe, Y as ke, Yn as Ae, Z as je, _ as Me, _t as Ne, a as Pe, an as Fe, at as Ie, b as Le, bn as Re, bt as ze, ct as Be, dn as Ve, en as He, g as Ue, gn as We, hn as Ge, i as Ke, in as qe, it as Je, j as Ye, jt as Xe, k as Ze, kn as Qe, kt as $e, lt as et, mn as tt, n as nt, nn as rt, o as it, on as at, q as ot, qn as st, qt as ct, rn as lt, rt as ut, st as dt, tn as ft, v as pt, vn as mt, vt as ht, wn as gt, wt as _t, xn as vt, xt as yt, y as bt, yt as xt, z as St, zt as Ct } from "./html-to-image-B7RQgQQy.js";
 import { o as wt, s as Tt, u as Et } from "./dist-wdOJ5T7-.js";
 import { D as Dt, E as Ot, O as kt, S as At, T as jt, _ as Mt, a as Nt, c as Pt, d as Ft, f as It, g as Lt, h as Rt, i as zt, l as Bt, m as Vt, n as Ht, o as Ut, p as Wt, r as Gt, s as Kt, t as qt, u as Jt, v as Yt, w as Xt, x as Zt, y as Qt } from "./spec-BQ36ell9.js";
-import { $ as $t, $t as en, A as tn, At as nn, B as rn, Bt as an, C as on, Ct as sn, D as cn, Dt as ln, E as un, Et as dn, F as fn, Ft as pn, G as mn, Gt as hn, H as gn, Ht as _n, I as vn, It as yn, Jt as bn, K as xn, Kt as Sn, L as Cn, Lt as wn, M as Tn, Mt as En, N as Dn, Nt as On, O as kn, Ot as An, P as jn, Pt as Mn, Q as Nn, Qt as Pn, R as Fn, Rt as In, S as Ln, St as Rn, T as zn, Tt as Bn, U as Vn, Ut as Hn, V as Un, Vt as Wn, W as Gn, Wt as Kn, Xt as qn, Yt as Jn, Z as Yn, Zt as Xn, _ as Zn, an as Qn, at as $n, b as er, bt as tr, c as nr, cn as rr, ct as ir, d as ar, dn as or, dt as sr, en as cr, et as lr, f as ur, fn as dr, ft as fr, g as pr, gt as mr, h as hr, ht as gr, i as _r, it as vr, j as yr, jt as br, k as xr, kt as Sr, ln as Cr, lt as wr, m as Tr, mt as Er, n as Dr, nn as Or, nt as kr, on as Ar, ot as jr, p as Mr, pn as Nr, pt as Pr, q as Fr, qt as Ir, r as Lr, rn as Rr, rt as zr, sn as Br, st as Vr, t as Hr, tn as Ur, tt as Wr, u as Gr, un as Kr, ut as qr, v as Jr, w as Yr, wt as Xr, x as Zr, xt as Qr, y as $r, yt as ei, z as ti, zt as ni } from "./common-D37C4Qj7.js";
+import { $ as $t, $t as en, A as tn, At as nn, B as rn, Bt as an, C as on, Ct as sn, D as cn, Dt as ln, E as un, Et as dn, F as fn, Ft as pn, G as mn, Gt as hn, H as gn, Ht as _n, I as vn, It as yn, Jt as bn, K as xn, Kt as Sn, L as Cn, Lt as wn, M as Tn, Mt as En, N as Dn, Nt as On, O as kn, Ot as An, P as jn, Pt as Mn, Q as Nn, Qt as Pn, R as Fn, Rt as In, S as Ln, St as Rn, T as zn, Tt as Bn, U as Vn, Ut as Hn, V as Un, Vt as Wn, W as Gn, Wt as Kn, Xt as qn, Yt as Jn, Z as Yn, Zt as Xn, _ as Zn, an as Qn, at as $n, b as er, bt as tr, c as nr, cn as rr, ct as ir, d as ar, dn as or, dt as sr, en as cr, et as lr, f as ur, fn as dr, ft as fr, g as pr, gt as mr, h as hr, ht as gr, i as _r, it as vr, j as yr, jt as br, k as xr, kt as Sr, ln as Cr, lt as wr, m as Tr, mt as Er, n as Dr, nn as Or, nt as kr, on as Ar, ot as jr, p as Mr, pn as Nr, pt as Pr, q as Fr, qt as Ir, r as Lr, rn as Rr, rt as zr, sn as Br, st as Vr, t as Hr, tn as Ur, tt as Wr, u as Gr, un as Kr, ut as qr, v as Jr, w as Yr, wt as Xr, x as Zr, xt as Qr, y as $r, yt as ei, z as ti, zt as ni } from "./common-nuR9LfxI.js";
 import { c as ri, i as ii, n as ai, o as oi, t as si } from "./useLifecycle-hEcsyQ2P.js";
 import { t as ci } from "./check-BcJDmQdF.js";
 import { A as li, C as ui, D as di, E as fi, F as pi, I as mi, M as hi, N as gi, P as _i, R as vi, S as yi, a as bi, b as xi, c as Si, i as Ci, j as wi, l as Ti, n as Ei, o as Di, r as Oi, s as ki, t as Ai, w as ji, x as Mi } from "./select-DQ9jU_B_.js";
@@ -35256,7 +35256,7 @@ function Gq(e) {
 }
 //#endregion
 //#region src/components/editor/renderers/slides-layout/slides-layout.tsx
-var Kq = q.lazy(() => import("./reveal-component-B3r0D5zd.js")), qq = (e) => {
+var Kq = q.lazy(() => import("./reveal-component-CzLgRK9f.js")), qq = (e) => {
 	let t = (0, Y.c)(33), { layout: n, setLayout: r, cells: i, mode: a } = e, o = Nm(_t), s = a === "read" || o, c = Nm(ne) > 1, [l, u] = (0, q.useState)(null), d;
 	t[0] !== i || t[1] !== n ? (d = UU(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/{reveal-component-B3r0D5zd.js → reveal-component-CzLgRK9f.js}

@@ -4,7 +4,7 @@ import { t as o } from "./react-DMWcHin-.js";
 import { t as s } from "./compiler-runtime-CAH9RLw2.js";
 import { it as c, wt as l } from "./html-to-image-B7RQgQQy.js";
 import { u } from "./dist-wdOJ5T7-.js";
-import { J as d, X as f, Y as p, in as m, l as h, n as g, rn as _, sn as v, t as y, u as b } from "./common-D37C4Qj7.js";
+import { J as d, X as f, Y as p, in as m, l as h, n as g, rn as _, sn as v, t as y, u as b } from "./common-nuR9LfxI.js";
 import { R as x } from "./select-DQ9jU_B_.js";
 import { X as S } from "./dropdown-menu-C8N-kZzD.js";
 import { t as C } from "./jsx-runtime-CpoKNRgu.js";

package/package.json

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

package/src/core/packages/__tests__/useInstallPackage.test.tsx

@@ -0,0 +1,123 @@
+/* Copyright 2026 Marimo. All rights reserved. */
+import { act, renderHook } from "@testing-library/react";
+import { beforeEach, describe, expect, it, vi } from "vitest";
+import { useInstallPackages } from "../useInstallPackage";
+
+const toast = vi.fn();
+const addPackage = vi.fn();
+
+vi.mock("@/components/ui/use-toast", () => ({
+  toast: (...args: unknown[]) => toast(...args),
+}));
+
+vi.mock("../../network/requests", () => ({
+  useRequestClient: () => ({
+    addPackage: (...args: unknown[]) => addPackage(...args),
+  }),
+}));
+
+describe("useInstallPackages", () => {
+  beforeEach(() => {
+    toast.mockClear();
+    addPackage.mockClear();
+  });
+
+  it("batches all packages into a single install call", async () => {
+    addPackage.mockResolvedValue({ success: true, error: null });
+    const { result } = renderHook(() => useInstallPackages());
+
+    await act(async () => {
+      await result.current.handleInstallPackages(["numpy", "pandas", "scipy"]);
+    });
+
+    // A single batched install call, not one per package.
+    expect(addPackage).toHaveBeenCalledTimes(1);
+    expect(addPackage).toHaveBeenCalledWith({ package: "numpy pandas scipy" });
+  });
+
+  it("shows a single aggregate success toast for multiple packages", async () => {
+    addPackage.mockResolvedValue({ success: true, error: null });
+    const { result } = renderHook(() => useInstallPackages());
+
+    await act(async () => {
+      await result.current.handleInstallPackages(["numpy", "pandas", "scipy"]);
+    });
+
+    // Only one toast, regardless of package count — no false per-package status.
+    expect(toast).toHaveBeenCalledTimes(1);
+    expect(toast).toHaveBeenCalledWith(
+      expect.objectContaining({ title: "Packages added" }),
+    );
+  });
+
+  it("shows a single aggregate error toast when the batch fails", async () => {
+    addPackage.mockResolvedValue({ success: false, error: "boom" });
+    const { result } = renderHook(() => useInstallPackages());
+
+    await act(async () => {
+      await result.current.handleInstallPackages(["numpy", "pandas"]);
+    });
+
+    expect(toast).toHaveBeenCalledTimes(1);
+    expect(toast).toHaveBeenCalledWith(
+      expect.objectContaining({
+        title: "Failed to add packages",
+        description: "boom",
+        variant: "danger",
+      }),
+    );
+  });
+
+  it("collapses a partial (mixed) failure into one aggregate error toast", async () => {
+    // The backend resolves the batch as a single transaction and reports an
+    // aggregate outcome, so even when only some packages fail we surface a
+    // single error toast covering the whole batch — never a per-package
+    // success/failure split the response can't actually represent.
+    addPackage.mockResolvedValue({
+      success: false,
+      error: "pandas failed to install",
+    });
+    const { result } = renderHook(() => useInstallPackages());
+
+    await act(async () => {
+      await result.current.handleInstallPackages(["numpy", "pandas"]);
+    });
+
+    expect(addPackage).toHaveBeenCalledTimes(1);
+    expect(addPackage).toHaveBeenCalledWith({ package: "numpy pandas" });
+    expect(toast).toHaveBeenCalledTimes(1);
+    expect(toast).toHaveBeenCalledWith(
+      expect.objectContaining({
+        title: "Failed to add packages",
+        description: "pandas failed to install",
+        variant: "danger",
+      }),
+    );
+  });
+
+  it("uses singular wording for a single package", async () => {
+    addPackage.mockResolvedValue({ success: true, error: null });
+    const { result } = renderHook(() => useInstallPackages());
+
+    await act(async () => {
+      await result.current.handleInstallPackages(["numpy"]);
+    });
+
+    expect(toast).toHaveBeenCalledTimes(1);
+    expect(toast).toHaveBeenCalledWith(
+      expect.objectContaining({ title: "Package added" }),
+    );
+  });
+
+  it("calls onSuccess after a successful install", async () => {
+    addPackage.mockResolvedValue({ success: true, error: null });
+    const onSuccess = vi.fn();
+    const { result } = renderHook(() => useInstallPackages());
+
+    await act(async () => {
+      await result.current.handleInstallPackages(["numpy"], onSuccess);
+    });
+
+    expect(onSuccess).toHaveBeenCalledTimes(1);
+  });
+});

package/src/core/packages/toast-components.tsx

@@ -4,23 +4,34 @@ import { Kbd } from "@/components/ui/kbd";
 import { toast } from "@/components/ui/use-toast";
 
 export const showAddPackageToast = (
-  packageName: string,
+  packageName: string | string[],
   error?: string | null,
 ) => {
+  const packageNames = Array.isArray(packageName) ? packageName : [packageName];
   if (error) {
     toast({
-      title: "Failed to add package",
+      title:
+        packageNames.length > 1
+          ? "Failed to add packages"
+          : "Failed to add package",
       description: error,
       variant: "danger",
     });
   } else {
     toast({
-      title: "Package added",
+      title: packageNames.length > 1 ? "Packages added" : "Package added",
       description: (
         <div>
           <div>
-            The package <Kbd className="inline">{packageName}</Kbd> and its
-            dependencies has been added to your environment.
+            {packageNames.length > 1 ? "The packages " : "The package "}
+            {packageNames.map((name, index) => (
+              <span key={name}>
+                {index > 0 && ", "}
+                <Kbd className="inline">{name}</Kbd>
+              </span>
+            ))}{" "}
+            and {packageNames.length > 1 ? "their" : "its"} dependencies have
+            been added to your environment.
           </div>
           <div className="text-xs text-muted-foreground mt-1">
             Some Python packages may require a kernel restart to see changes.

package/src/core/packages/useInstallPackage.ts

@@ -22,17 +22,18 @@ export function useInstallPackages(): {
     setLoading(true);
 
     try {
-      for (const [idx, packageName] of packages.entries()) {
-        const response = await addPackage({ package: packageName });
-        if (response.success) {
-          showAddPackageToast(packageName);
-        } else {
-          showAddPackageToast(packageName, response.error);
-        }
-        // Wait 1s if there are more packages to install
-        if (idx < packages.length - 1) {
-          await new Promise((resolve) => setTimeout(resolve, 1000));
-        }
+      // Batch all packages into a single install call.
+      // The worker splits by space and passes the full list to micropip,
+      // which resolves and downloads in parallel internally.
+      const response = await addPackage({ package: packages.join(" ") });
+      // The backend resolves the whole list as a single transaction, so the
+      // response only carries an aggregate success/error. Report a single
+      // toast covering all requested packages rather than implying a
+      // per-package outcome we don't actually have.
+      if (response.success) {
+        showAddPackageToast(packages);
+      } else {
+        showAddPackageToast(packages, response.error);
       }
       onSuccess?.();
     } catch (error) {