moops 0.13.2__tar.gz → 0.13.3__tar.gz

{moops-0.13.2 → moops-0.13.3}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: moops
-Version: 0.13.2
+Version: 0.13.3
 Summary: Write Marimo notebooks that also work as CLI scripts, with unified UI controls
 Keywords: marimo,notebooks,cli,testing
 Author: Yair Chuchem

{moops-0.13.2 → moops-0.13.3}/pyproject.toml

@@ -4,7 +4,7 @@ build-backend = "uv_build"
 
 [project]
 name = "moops"
-version = "0.13.2"
+version = "0.13.3"
 description = "Write Marimo notebooks that also work as CLI scripts, with unified UI controls"
 license = "MIT"
 readme = "README.md"

{moops-0.13.2 → moops-0.13.3}/src/moops/_options.py

@@ -425,6 +425,24 @@ def _missing_path_error(value: str) -> ParseError | None:
     return None
 
 
+def _file_browser_paths(value: typing.Any) -> list[str]:
+    if not value:
+        return []
+    if isinstance(value, str):
+        return [value]
+    if isinstance(value, pathlib.Path):
+        return [str(value)]
+    if isinstance(value, typing.Sequence):
+        items = typing.cast(typing.Sequence[object], value)
+        return [
+            str(item.path)
+            if isinstance(item, _ui_workarounds.FileBrowserFileInfo)
+            else str(item)
+            for item in items
+        ]
+    return [str(value)]
+
+
 @dataclasses.dataclass
 class FileControl(TextControl):
     def create_marimo_element(
@@ -474,6 +492,14 @@ class FileControl(TextControl):
                 continue
             return result
 
+    def format_value(self, value: typing.Any) -> list[str]:
+        paths = _file_browser_paths(value)
+        return super().format_value(paths[0] if paths else "")
+
+    def format_query_value(self, value: typing.Any) -> str | None:
+        paths = _file_browser_paths(value)
+        return super().format_query_value(paths[0] if paths else "")
+
 
 @dataclasses.dataclass
 class MultiFileControl(ValueControl):
@@ -554,13 +580,13 @@ class MultiFileControl(ValueControl):
         return [line]
 
     def format_value(self, value: typing.Any) -> list[str]:
-        values = list(value)
+        values = _file_browser_paths(value)
         if values == self.default:
             return []
         return [option_value_token(self.option, v) for v in values]
 
     def format_query_value(self, value: typing.Any) -> str | None:
-        values = list(value)
+        values = _file_browser_paths(value)
         return None if values == self.default else json.dumps(values)
 
     def prompt_interactive(self, effective_default: typing.Any = _UNSET) -> list[str]:

{moops-0.13.2 → moops-0.13.3}/src/moops/_text_wrap.py

@@ -44,14 +44,16 @@ def wrap_help_line(line: str, width: int = 88) -> list[str]:
     return result
 
 
-def wrap_command(name: str, groups: list[str], width: int = 72) -> str:
+def wrap_command(
+    name: str, groups: list[str], width: int = 72, *, quote_name: bool = True
+) -> str:
     """Render a script command, wrapping long lines with shell continuations.
 
     Short commands stay on one line. When the single-line form exceeds ``width``,
     each option group goes on its own line joined by `` \\`` continuations, which
     remains valid copy-pasteable shell.
     """
-    quoted_name = shlex.quote(name)
+    quoted_name = shlex.quote(name) if quote_name else name
     single_line = " ".join([quoted_name, *groups])
     if not groups or len(single_line) <= width:
         return single_line

{moops-0.13.2 → moops-0.13.3}/src/moops/interface.py

@@ -436,19 +436,22 @@ class Interface:
         if not self.notebook_name:
             return mo.md("Input bundle with no notebook name")
         notebook_name = html.escape(self.notebook_name)
-        href = html.escape(self._standalone_url(), quote=True)
-        link = mo.md(
-            f'<a href="{href}" target="_blank" rel="noopener">'
-            f"An embedded instance of `{notebook_name}`</a>"
-        )
+        label = f"An embedded instance of `{notebook_name}`"
+        if _running_in_edit_mode():
+            href = html.escape(self._standalone_url(), quote=True)
+            heading: mo.Html = mo.md(
+                f'<a href="{href}" target="_blank" rel="noopener">{label}</a>'
+            )
+        else:
+            heading = mo.md(label)
         if self.embedded_extra_overrides != frozenset():
             # None: embedded outside moops.embed, so overrides are unknown.
             # Non-empty: the parent injected defs beyond args, so no CLI
             # command can reproduce this setup.
-            return link
+            return heading
         command = _wrap_command(self.notebook_file, self._standalone_arg_groups())
         return mo.vstack(
-            [link, mo.md(f"To run as a standalone script:\n```\n{command}\n```")]
+            [heading, mo.md(f"To run as a standalone script:\n```\n{command}\n```")]
         )
 
     def _standalone_url(self) -> str:
@@ -633,7 +636,9 @@ def attached_interface(ctrl: typing.Any) -> Interface | None:
 
 def _wrap_command(command: str, groups: list[str]) -> str:
     if _should_use_uv_run(command):
-        return _text_wrap.wrap_command("uv", [f"run {shlex.quote(command)}", *groups])
+        return _text_wrap.wrap_command(
+            f"uv run {shlex.quote(command)}", groups, quote_name=False
+        )
     return _text_wrap.wrap_command(command, groups)
 
 
@@ -716,3 +721,25 @@ def _collect_usage_placeholders(
     for ctrl in iface.controls:
         if sub_iface := attached_interface(ctrl):
             _collect_usage_placeholders(sub_iface, result)
+
+
+def _running_in_edit_mode() -> bool:
+    """Return True only when running inside a marimo *edit* session.
+
+    ``mo.running_in_notebook()`` is True for both ``marimo edit`` and
+    ``marimo run``, but links to ``/?file=...`` only work in edit sessions.
+    """
+    if not mo.running_in_notebook():
+        return False
+    try:
+        from marimo._runtime.context import get_context
+        from marimo._runtime.context.kernel_context import KernelRuntimeContext
+        from marimo._session.model import SessionMode
+
+        ctx = get_context()
+        return (
+            isinstance(ctx, KernelRuntimeContext)
+            and ctx.session_mode == SessionMode.EDIT
+        )
+    except Exception:
+        return False