run less frequently, only on manager creation

marimo-team · Jan 14, 2025 · 0820428 · 0820428
1 parent cb81134
commit 0820428
Showing 1 changed file with 6 additions and 5 deletions.
diff --git a/marimo/_runtime/runtime.py b/marimo/_runtime/runtime.py
@@ -549,11 +549,12 @@ def _update_runtime_from_user_config(self, config: MarimoConfig) -> None:
         ):
             self.package_manager = create_package_manager(package_manager)
 
-        if self._should_update_script_metadata():
-            # All marimo notebooks depend on the marimo package; if the
-            # notebook already has an optional dependency group with marimo,
-            # such as marimo[sql], this is a NOOP.
-            self._update_script_metadata(["marimo"])
+            if self._should_update_script_metadata():
+                # All marimo notebooks depend on the marimo package; if the
+                # notebook already has marimo as a dependency, or an optional
+                # dependency group with marimo, such as marimo[sql], this is a
+                # NOOP.
+                self._update_script_metadata(["marimo"])
 
         if (
             autoreload_mode == "lazy" or autoreload_mode == "autorun"