feat: Add support for root user to connect to --user services

Removed the dedicated screen from the previous commit and instead opted
for a quick local test before switching. This removes the need for the
user to configure anything and it should just work out-of-the-box and
still educate the user.

Fixes #30

Author: kainctl

docs/customization.md

@@ -285,9 +285,3 @@ to lower this value.
 
 To view the entire output, open the output in the pager.
 
-### Auto Accept Connection to `root --user` Bus
-
-{{ config_block(20) }}
-
-Usually this is _not_ what you should do, but I am supporting it for now.
-

src/isd_tui/isd.py

@@ -295,49 +295,6 @@ def render_keybinding(inp_str: str) -> str:
     return "+".join(key_tokens)
 
 
-class RootUserBusModal(ModalScreen[bool]):
-    """
-    Present a simple screen to inform the user that they
-    are about to access the `root --user` service bus,
-    which usually leads to a crash.
-
-    Returns their selection.
-    """
-
-    BINDINGS = [
-        Binding("escape", "close", "Close", show=True),
-    ]
-    AUTO_FOCUS = "#no"
-
-    def __init__(
-        self,
-        **kwargs,
-    ):
-        super().__init__(**kwargs)
-
-    def compose(self) -> ComposeResult:
-        with VerticalGroup():
-            yield Markdown(
-                "# About to access **root --user** bus\n"
-                + "Usually the `root` user does _not_ have a dedicated `--user` bus. "
-                + "The application will crash if you try to connect to a non-existing bus. "
-                + "Are you sure you want to connect to it?\n\n"
-                + "You can update the settings to auto-accept this question if this is what you want."
-            )
-            with HorizontalGroup():
-                yield Button("No", id="no", variant="primary")
-                yield Button("Yes", id="yes", variant="warning")
-        yield Footer()
-
-    def action_close(self) -> None:
-        self.dismiss(False)
-
-    def on_button_pressed(self, event: Button.Pressed) -> None:
-        if event.button.id == "no":
-            self.dismiss(False)
-        self.dismiss(True)
-
-
 class SystemctlActionScreen(ModalScreen[Optional[str]]):
     """
     Present a screen with the configured systemctl actions.
@@ -923,15 +880,6 @@ class Settings(BaseSettings):
             Note: The output is not trimmed when a pager or editor is opened!"""),
     )
 
-    auto_accept_connection_to_root_user_bus: bool = Field(
-        default=False,
-        description=dedent("""\
-            By default, the user is asked if they actually want to connect
-            to the `root --user` bus, as this usually crashes the running program.
-            If this option is `True`, the question is skipped.
-        """),
-    )
-
     # https://github.com/tinted-theming/home?tab=readme-ov-file
     @classmethod
     def settings_customise_sources(
@@ -2183,20 +2131,7 @@ def action_copy_unit_path(self) -> None:
         self.app.copy_to_clipboard(path)
         self.notify(f"Copied '{path}' to the clipboard.")
 
-    async def wants_root_user_bus(self) -> bool:
-        """
-        Ask the user if they know what they actually want to connect to the
-        user-bus of the `root` user.
-        """
-        prev_focus = self.focused
-        self.set_focus(None)
-        do_switch = await self.app.push_screen_wait(RootUserBusModal())
-        self.set_focus(prev_focus)
-        return do_switch
-
-    # `work` is required to push the screen for `wants_root_user_bus`.
-    @work
-    async def action_toggle_mode(self) -> None:
+    def action_toggle_mode(self) -> None:
         """
         Toggle the current `bus`.
         Try to protect the user from accidentally crashing the program by trying
@@ -2205,12 +2140,22 @@ async def action_toggle_mode(self) -> None:
 
         - <https://github.com/isd-project/isd/issues/30>
         """
-        if (
-            is_root()
-            and self.mode == "system"
-            and not self.settings.auto_accept_connection_to_root_user_bus
-        ):
-            if not (await self.wants_root_user_bus()):
+        if is_root() and self.mode == "system":
+            # Test if `root` user can actually connect to a `--user` bus.
+            proc = subprocess.run(
+                systemctl_args_builder("is-system-running", mode="user", units=[]),
+                capture_output=True,
+                text=True,
+            )
+            if proc.returncode != 0:
+                self.notify(
+                    "Could not connect to `root` users `--user` bus.\n"
+                    + "Usually, this does not work, as the `root` user does not have any `user` services.\n"
+                    + "The connection was tested via `systemctl --user is-system-running` as `root` user with the full error message below:\n\n"
+                    + proc.stderr,
+                    severity="error",
+                    timeout=60,
+                )
                 return
 
         self.mode = "system" if self.mode == "user" else "user"