Split "realize" phase into "configure" and "execute"

pasch/modules/echo.py

@@ -10,5 +10,5 @@ class Echo(Module):
     def add(self, arg: str) -> None:
         self.args.append(arg)
 
-    def realize(self) -> None:
+    def execute(self) -> None:
         run_execute("echo", *self.args)

pasch/modules/files.py

@@ -123,7 +123,7 @@ class Files(Module):
         path = self._read_path(path)
         self._files[path_to_str(path)] = content.to_bytes()
 
-    def realize(self) -> None:
+    def execute(self) -> None:
         for path, content in sorted(self._files.items()):
             self._write_file(self._read_path(path), content)
 

pasch/modules/pacman.py

@@ -26,7 +26,7 @@ class Pacman(Module):
     def exclude(self, group: str, *packages: str) -> None:
         self.excluded.setdefault(group, set()).update(packages)
 
-    def realize(self) -> None:
+    def execute(self) -> None:
         groups = self._get_groups()
 
         installed = self._get_explicitly_installed_packages()

pasch/orchestrator.py

@@ -2,23 +2,24 @@ from __future__ import annotations
 
 import getpass
 import socket
-from abc import ABC, abstractmethod
 from typing import Callable, Concatenate
 
-from rich import print
 from rich.console import Console
 from rich.markup import escape
 from xdg_base_dirs import xdg_state_home
 
 
-class Module(ABC):
+class Module:
     def __init__(self, orchestrator: Orchestrator) -> None:
         self.o = orchestrator
         self.o.register(self)
         self.c = self.o.c
 
-    @abstractmethod
+    def configure(self) -> None:
-    def realize(self) -> None: ...
+        pass
+
+    def execute(self) -> None:
+        pass
 
 
 def _snake_to_camel(s: str) -> str:
@@ -34,7 +35,7 @@ def module[**P](
         self.args = args
         self.kwargs = kwargs
 
-    def realize(self) -> None:
+    def configure(self) -> None:
         # pyrefly: ignore
         return func(self.o, *self.args, **self.kwargs)
 
@@ -42,10 +43,13 @@ def module[**P](
     return type(
         _snake_to_camel(func.__name__),
         (Module,),
-        {"__init__": __init__, "realize": realize},
+        {"__init__": __init__, "configure": configure},
     )
 
 
+# TODO @module_gen for generator-based modules
+
+
 class Orchestrator:
     def __init__(self, name: str = "pasch", dry_run: bool = False) -> None:
         self.name = name
@@ -67,6 +71,15 @@ class Orchestrator:
 
     def realize(self) -> None:
         self._frozen = True
+
+        self.c.print()
+        self.c.print("[bold bright_cyan]# Configure")
         for module in reversed(self._modules):
-            print(f"[bold bright_magenta]\\[{escape(type(module).__name__)}]")
+            self.c.print(f"[bold bright_magenta]\\[{escape(type(module).__name__)}]")
-            module.realize()
+            module.configure()
+
+        self.c.print()
+        self.c.print("[bold bright_cyan]# Execute")
+        for module in self._modules:
+            self.c.print(f"[bold bright_magenta]\\[{escape(type(module).__name__)}]")
+            module.execute()