refactor!: Move freezing code to workspace.freezer

Author: tony

docs/api.md

@@ -127,7 +127,7 @@ If you need an internal API stabilized please [file an issue](https://github.com
 ## Workspace Freezer
 
 ```{eval-rst}
-.. automethod:: tmuxp.workspace.builder.freeze
+.. automethod:: tmuxp.workspace.freezer.freeze
 ```
 
 ```{eval-rst}

src/tmuxp/cli/freeze.py

@@ -10,7 +10,7 @@
 
 from .. import util
 from ..workspace import config
-from ..workspace.builder import freeze
+from ..workspace.freezer import freeze
 from .utils import get_config_dir, prompt, prompt_choices, prompt_yes_no
 
 if t.TYPE_CHECKING:

src/tmuxp/workspace/builder.py

@@ -494,69 +494,3 @@ def find_current_attached_session(self):
 
     def first_window_pass(self, i, session, append):
         return len(session.windows) == 1 and i == 1 and not append
-
-
-def freeze(session):
-    """
-    Freeze live tmux session and Return session config :py:obj:`dict`.
-
-    Parameters
-    ----------
-    session : :class:`libtmux.Session`
-        session object
-
-    Returns
-    -------
-    dict
-        tmuxp compatible workspace config
-    """
-    sconf = {"session_name": session["session_name"], "windows": []}
-
-    for w in session.windows:
-        wconf = {
-            "options": w.show_window_options(),
-            "window_name": w.name,
-            "layout": w.layout,
-            "panes": [],
-        }
-        if w.get("window_active", "0") == "1":
-            wconf["focus"] = "true"
-
-        # If all panes have same path, set 'start_directory' instead
-        # of using 'cd' shell commands.
-        def pane_has_same_path(p):
-            return w.panes[0].current_path == p.current_path
-
-        if all(pane_has_same_path(p) for p in w.panes):
-            wconf["start_directory"] = w.panes[0].current_path
-
-        for p in w.panes:
-            pconf = {"shell_command": []}
-
-            if "start_directory" not in wconf:
-                pconf["shell_command"].append("cd " + p.current_path)
-
-            if p.get("pane_active", "0") == "1":
-                pconf["focus"] = "true"
-
-            current_cmd = p.current_command
-
-            def filter_interpretters_and_shells():
-                return current_cmd.startswith("-") or any(
-                    current_cmd.endswith(cmd) for cmd in ["python", "ruby", "node"]
-                )
-
-            if filter_interpretters_and_shells():
-                current_cmd = None
-
-            if current_cmd:
-                pconf["shell_command"].append(current_cmd)
-            else:
-                if not len(pconf["shell_command"]):
-                    pconf = "pane"
-
-            wconf["panes"].append(pconf)
-
-        sconf["windows"].append(wconf)
-
-    return sconf

src/tmuxp/workspace/freezer.py

@@ -0,0 +1,64 @@
+def freeze(session):
+    """
+    Freeze live tmux session and Return session config :py:obj:`dict`.
+
+    Parameters
+    ----------
+    session : :class:`libtmux.Session`
+        session object
+
+    Returns
+    -------
+    dict
+        tmuxp compatible workspace config
+    """
+    sconf = {"session_name": session["session_name"], "windows": []}
+
+    for w in session.windows:
+        wconf = {
+            "options": w.show_window_options(),
+            "window_name": w.name,
+            "layout": w.layout,
+            "panes": [],
+        }
+        if w.get("window_active", "0") == "1":
+            wconf["focus"] = "true"
+
+        # If all panes have same path, set 'start_directory' instead
+        # of using 'cd' shell commands.
+        def pane_has_same_path(p):
+            return w.panes[0].current_path == p.current_path
+
+        if all(pane_has_same_path(p) for p in w.panes):
+            wconf["start_directory"] = w.panes[0].current_path
+
+        for p in w.panes:
+            pconf = {"shell_command": []}
+
+            if "start_directory" not in wconf:
+                pconf["shell_command"].append("cd " + p.current_path)
+
+            if p.get("pane_active", "0") == "1":
+                pconf["focus"] = "true"
+
+            current_cmd = p.current_command
+
+            def filter_interpretters_and_shells():
+                return current_cmd.startswith("-") or any(
+                    current_cmd.endswith(cmd) for cmd in ["python", "ruby", "node"]
+                )
+
+            if filter_interpretters_and_shells():
+                current_cmd = None
+
+            if current_cmd:
+                pconf["shell_command"].append(current_cmd)
+            else:
+                if not len(pconf["shell_command"]):
+                    pconf = "pane"
+
+            wconf["panes"].append(pconf)
+
+        sconf["windows"].append(wconf)
+
+    return sconf

tests/workspace/test_freezer.py

@@ -3,7 +3,8 @@
 
 from tmuxp.config_reader import ConfigReader
 from tmuxp.workspace import config
-from tmuxp.workspace.builder import WorkspaceBuilder, freeze
+from tmuxp.workspace.builder import WorkspaceBuilder
+from tmuxp.workspace.freezer import freeze
 
 from ..fixtures import utils as test_utils