zrb 0.28.0__py3-none-any.whl → 1.0.0__py3-none-any.whl

zrb/task/cmd_task.py

@@ -1,428 +1,275 @@
-import asyncio
-import atexit
-import logging
 import os
-import pathlib
-import signal
-import subprocess
-import sys
-import time
-from collections.abc import Callable, Iterable
-from typing import Any, Optional, TypeVar, Union
-
-from zrb.config.config import DEFAULT_SHELL, LOGGING_LEVEL
-from zrb.helper.accessories.color import colored
-from zrb.helper.asyncio_task import stop_asyncio_sync
-from zrb.helper.log import logger
-from zrb.helper.typecheck import typechecked
-from zrb.helper.typing import JinjaTemplate
+from functools import partial
+
+from zrb.attr.type import BoolAttr, IntAttr, StrAttr
+from zrb.cmd.cmd_result import CmdResult
+from zrb.cmd.cmd_val import AnyCmdVal, CmdVal, SingleCmdVal
+from zrb.config import DEFAULT_SHELL, WARN_UNRECOMMENDED_COMMAND
+from zrb.context.any_context import AnyContext
+from zrb.env.any_env import AnyEnv
+from zrb.input.any_input import AnyInput
 from zrb.task.any_task import AnyTask
-from zrb.task.any_task_event_handler import (
-    OnFailed,
-    OnReady,
-    OnRetry,
-    OnSkipped,
-    OnStarted,
-    OnTriggered,
-    OnWaiting,
-)
-from zrb.task.base_task.base_task import BaseTask
-from zrb.task_env.env import Env
-from zrb.task_env.env_file import EnvFile
-from zrb.task_group.group import Group
-from zrb.task_input.any_input import AnyInput
-
-logger.debug(colored("Loading zrb.task.cmd_task", attrs=["dark"]))
-
-_has_stty = True
-try:
-    _original_stty = subprocess.getoutput("stty -g").rstrip()
-except Exception:
-    _has_stty = False
-
-
-def _reset_stty():
-    global _has_stty
-    if _has_stty:
-        try:
-            subprocess.run(["stty", _original_stty], check=True)
-        except Exception:
-            _has_stty = False
-
-
-def _log_error(message: Any):
-    if LOGGING_LEVEL > logging.ERROR:
-        return
-    colored_message = colored(f"{message}", color="red", attrs=["bold"])
-    logger.error(colored_message, exc_info=True)
-
-
-def _print_out_dark(message: Any):
-    message_str = f"{message}"
-    print(colored(message_str, attrs=["dark"]), file=sys.stderr)
-
-
-CmdVal = Union[
-    JinjaTemplate,
-    Iterable[JinjaTemplate],
-    Callable[..., Union[Iterable[JinjaTemplate], JinjaTemplate]],
-]
-TCmdTask = TypeVar("TCmdTask", bound="CmdTask")
-
-
-class CmdResult:
-    def __init__(self, output: str, error: str):
-        self.output = output
-        self.error = error
-
-    def __str__(self) -> str:
-        return self.output
-
+from zrb.task.base_task import BaseTask
+from zrb.util.attr import get_int_attr, get_str_attr
+from zrb.util.cmd.command import check_unrecommended_commands, run_command
+from zrb.util.cmd.remote import get_remote_cmd_script
+from zrb.xcom.xcom import Xcom
 
-class CmdGlobalState:
-    def __init__(self):
-        self.no_more_attempt: bool = False
-        self.is_killed_by_signal: bool = False
 
-
-@typechecked
 class CmdTask(BaseTask):
-    """
-    Command Task.
-    You can use this task to run shell command.
-
-    Examples:
-        >>> from zrb import runner, CmdTask, StrInput, Env
-        >>> hello = CmdTask(
-        >>>     name='hello',
-        >>>     inputs=[StrInput(name='name', default='World')],
-        >>>     envs=[Env(name='HOME_DIR', os_name='HOME')],
-        >>>     cmd=[
-        >>>         'echo Hello {{ input.name }}',
-        >>>         'echo Home directory is: $HOME_DIR',
-        >>>     ]
-        >>> )
-        >>> runner.register(hello)
-    """
-
-    _pids: list[int] = []
-    _global_state = CmdGlobalState()
-
     def __init__(
         self,
         name: str,
-        group: Optional[Group] = None,
-        inputs: Iterable[AnyInput] = [],
-        envs: Iterable[Env] = [],
-        env_files: Iterable[EnvFile] = [],
-        icon: Optional[str] = None,
-        color: Optional[str] = None,
-        description: str = "",
-        executable: Optional[str] = None,
+        color: int | None = None,
+        icon: str | None = None,
+        description: str | None = None,
+        cli_only: bool = False,
+        input: list[AnyInput | None] | AnyInput | None = None,
+        env: list[AnyEnv | None] | AnyEnv | None = None,
+        shell: StrAttr | None = None,
+        render_shell: bool = True,
+        shell_flag: StrAttr | None = None,
+        render_shell_flag: bool = True,
+        remote_host: StrAttr | None = None,
+        render_remote_host: bool = True,
+        remote_port: IntAttr | None = None,
+        render_remote_port: bool = True,
+        remote_user: StrAttr | None = None,
+        render_remote_user: bool = True,
+        remote_password: StrAttr | None = None,
+        render_remote_password: bool = True,
+        remote_ssh_key: StrAttr | None = None,
+        render_remote_ssh_key: bool = True,
         cmd: CmdVal = "",
-        cmd_path: CmdVal = "",
-        cwd: Optional[Union[JinjaTemplate, pathlib.Path]] = None,
-        should_render_cwd: bool = True,
-        upstreams: Iterable[AnyTask] = [],
-        fallbacks: Iterable[AnyTask] = [],
-        on_triggered: Optional[OnTriggered] = None,
-        on_waiting: Optional[OnWaiting] = None,
-        on_skipped: Optional[OnSkipped] = None,
-        on_started: Optional[OnStarted] = None,
-        on_ready: Optional[OnReady] = None,
-        on_retry: Optional[OnRetry] = None,
-        on_failed: Optional[OnFailed] = None,
-        checkers: Iterable[AnyTask] = [],
-        checking_interval: Union[float, int] = 0.05,
-        retry: int = 2,
-        retry_interval: Union[float, int] = 1,
+        render_cmd: bool = True,
+        cwd: str | None = None,
+        render_cwd: bool = True,
+        plain_print: bool = False,
+        warn_unrecommended_command: bool | None = None,
         max_output_line: int = 1000,
         max_error_line: int = 1000,
-        preexec_fn: Optional[Callable[[], Any]] = os.setsid,
-        should_execute: Union[bool, str, Callable[..., bool]] = True,
-        return_upstream_result: bool = False,
-        should_print_cmd_result: bool = True,
-        should_show_cmd: bool = True,
-        should_show_working_directory: bool = True,
+        execute_condition: BoolAttr = True,
+        retries: int = 2,
+        retry_period: float = 0,
+        readiness_check: list[AnyTask] | AnyTask | None = None,
+        readiness_check_delay: float = 0.5,
+        readiness_check_period: float = 5,
+        readiness_failure_threshold: int = 1,
+        readiness_timeout: int = 60,
+        monitor_readiness: bool = False,
+        upstream: list[AnyTask] | AnyTask | None = None,
+        fallback: list[AnyTask] | AnyTask | None = None,
+        successor: list[AnyTask] | AnyTask | None = None,
     ):
-        BaseTask.__init__(
-            self,
+        super().__init__(
             name=name,
-            group=group,
-            inputs=inputs,
-            envs=envs,
-            env_files=env_files,
-            icon=icon,
             color=color,
+            icon=icon,
             description=description,
-            upstreams=upstreams,
-            fallbacks=fallbacks,
-            on_triggered=on_triggered,
-            on_waiting=on_waiting,
-            on_skipped=on_skipped,
-            on_started=on_started,
-            on_ready=on_ready,
-            on_retry=on_retry,
-            on_failed=on_failed,
-            checkers=checkers,
-            checking_interval=checking_interval,
-            retry=retry,
-            retry_interval=retry_interval,
-            should_execute=should_execute,
-            return_upstream_result=return_upstream_result,
+            cli_only=cli_only,
+            input=input,
+            env=env,
+            execute_condition=execute_condition,
+            retries=retries,
+            retry_period=retry_period,
+            readiness_check=readiness_check,
+            readiness_check_delay=readiness_check_delay,
+            readiness_check_period=readiness_check_period,
+            readiness_failure_threshold=readiness_failure_threshold,
+            readiness_timeout=readiness_timeout,
+            monitor_readiness=monitor_readiness,
+            upstream=upstream,
+            fallback=fallback,
+            successor=successor,
         )
-        max_output_line = max_output_line if max_output_line > 0 else 1
-        max_error_line = max_error_line if max_error_line > 0 else 1
+        self._shell = shell
+        self._render_shell = render_shell
+        self._shell_flag = shell_flag
+        self._render_shell_flag = render_shell_flag
+        self._remote_host = remote_host
+        self._render_remote_host = render_remote_host
+        self._remote_port = remote_port
+        self._render_remote_port = render_remote_port
+        self._remote_user = remote_user
+        self._render_remote_user = render_remote_user
+        self._remote_password = remote_password
+        self._render_remote_password = render_remote_password
+        self._remote_ssh_key = remote_ssh_key
+        self._render_remote_ssh_key = render_remote_ssh_key
         self._cmd = cmd
-        self._cmd_path = cmd_path
-        self.__set_cwd(cwd)
-        self._should_render_cwd = should_render_cwd
-        self._max_output_size = max_output_line
-        self._max_error_size = max_error_line
-        self._output_buffer: Iterable[str] = []
-        self._error_buffer: Iterable[str] = []
-        if executable is None and DEFAULT_SHELL != "":
-            executable = DEFAULT_SHELL
-        self._executable = executable
-        self._process: Optional[asyncio.subprocess.Process]
-        self._preexec_fn = preexec_fn
-        self._should_print_cmd_result = should_print_cmd_result
-        self._should_show_working_directory = should_show_working_directory
-        self._should_show_cmd = should_show_cmd
-
-    def copy(self) -> TCmdTask:
-        return super().copy()
-
-    def set_cwd(self, cwd: Union[str, pathlib.Path]):
-        self.__set_cwd(cwd)
-
-    def __set_cwd(self, cwd: Optional[Union[str, pathlib.Path]]):
-        if cwd is None:
-            self._cwd: Union[str, pathlib.Path] = os.getcwd()
-            return
-        self._cwd: Union[str, pathlib.Path] = os.path.abspath(cwd)
-
-    def to_function(
-        self,
-        env_prefix: str = "",
-        raise_error: bool = True,
-        is_async: bool = False,
-        show_done_info: bool = True,
-        should_clear_xcom: bool = False,
-        should_stop_looper: bool = False,
-    ) -> Callable[..., CmdResult]:
-        return super().to_function(
-            env_prefix=env_prefix,
-            raise_error=raise_error,
-            is_async=is_async,
-            show_done_info=show_done_info,
-            should_clear_xcom=should_clear_xcom,
-            should_stop_looper=should_stop_looper,
-        )
-
-    def print_result(self, result: CmdResult):
-        if not self._should_print_cmd_result or result.output == "":
-            return
-        print(result.output)
-
-    async def run(self, *args: Any, **kwargs: Any) -> CmdResult:
-        cmd = self.get_cmd_script(*args, **kwargs)
-        if self._should_show_cmd:
-            self.print_out_dark("Run script: " + self.__get_multiline_repr(cmd))
-        cwd = self._get_cwd()
-        if self._should_show_working_directory:
-            self.print_out_dark("Working directory: " + cwd)
-        self._output_buffer = []
-        self._error_buffer = []
-        process = await asyncio.create_subprocess_shell(
-            cmd,
-            cwd=cwd,
-            stdout=asyncio.subprocess.PIPE,
-            stderr=asyncio.subprocess.PIPE,
-            env=self.get_env_map(),
-            shell=True,
-            executable=self._executable,
-            close_fds=True,
-            preexec_fn=self._preexec_fn,
-            bufsize=0,
-        )
-        self._set_task_pid(process.pid)
-        self._pids.append(process.pid)
-        self._process = process
-        try:
-            signal.signal(signal.SIGINT, self.__on_kill)
-            signal.signal(signal.SIGTERM, self.__on_kill)
-        except Exception as e:
-            self.print_err(e)
-        atexit.register(self.__on_exit)
-        await self.__wait_process(process)
-        self.log_info("Process completed")
-        atexit.unregister(self.__on_exit)
-        output = "\n".join(self._output_buffer)
-        error = "\n".join(self._error_buffer)
-        # get return code
-        return_code = process.returncode
-        self.log_info(f"Exit status: {return_code}")
-        if return_code != 0 and not self._global_state.is_killed_by_signal:
-            raise Exception(f"Process {self._name} exited ({return_code}): {error}")
-        self.set_task_xcom(key="output", value=output)
-        self.set_task_xcom(key="error", value=error)
-        return CmdResult(output, error)
-
-    def _get_cwd(self) -> Union[str, pathlib.Path]:
-        if self._should_render_cwd and isinstance(self._cwd, str):
-            return self.render_str(self._cwd)
-        return self._cwd
-
-    def _should_attempt(self) -> bool:
-        if self._global_state.no_more_attempt:
-            return False
-        return super()._should_attempt()
-
-    def _is_last_attempt(self) -> bool:
-        if self._global_state.no_more_attempt:
-            return True
-        return super()._is_last_attempt()
-
-    def __on_kill(self, signum: Any, frame: Any):
-        self._global_state.no_more_attempt = True
-        self._global_state.is_killed_by_signal = True
-        _print_out_dark(f"Getting signal {signum}")
-        for pid in self._pids:
-            self.__kill_by_pid(pid)
-        stop_asyncio_sync()
-
-    def __on_exit(self):
-        self._global_state.no_more_attempt = True
-        self.__kill_by_pid(self._process.pid)
-
-    def __kill_by_pid(self, pid: int):
+        self._render_cmd = render_cmd
+        self._cwd = cwd
+        self._render_cwd = render_cwd
+        self._max_output_line = max_output_line
+        self._max_error_line = max_error_line
+        self._should_plain_print = plain_print
+        self._should_warn_unrecommended_command = warn_unrecommended_command
+
+    async def _exec_action(self, ctx: AnyContext) -> CmdResult:
+        """Turn _cmd attribute into subprocess.Popen and execute it as task's action.
+
+        Args:
+            session (AnySession): The shared session.
+
+        Returns:
+            Any: The result of the action execution.
         """
-        Kill a pid, gracefully
-        """
-        try:
-            process_ever_exists = False
-            if self.__is_process_exist(pid):
-                process_ever_exists = True
-                _print_out_dark(f"Send SIGTERM to process {pid}")
-                os.killpg(os.getpgid(pid), signal.SIGTERM)
-                time.sleep(0.3)
-            if self.__is_process_exist(pid):
-                _print_out_dark(f"Send SIGINT to process {pid}")
-                os.killpg(os.getpgid(pid), signal.SIGINT)
-                time.sleep(0.3)
-            if self.__is_process_exist(pid):
-                _print_out_dark(f"Send SIGKILL to process {pid}")
-                os.killpg(os.getpgid(pid), signal.SIGKILL)
-            if process_ever_exists:
-                _print_out_dark(f"Process {pid} is killed successfully")
-        except Exception:
-            _log_error(f"Cannot kill process {pid}")
-
-    def __is_process_exist(self, pid: int) -> bool:
-        try:
-            os.killpg(os.getpgid(pid), 0)
-            return True
-        except ProcessLookupError:
-            return False
-
-    async def __wait_process(self, process: asyncio.subprocess.Process):
-        # Create queue
-        stdout_queue = asyncio.Queue()
-        stderr_queue = asyncio.Queue()
-        # Read from streams and put into queue
-        stdout_process = asyncio.create_task(
-            self.__queue_stream(process.stdout, stdout_queue)
-        )
-        stderr_process = asyncio.create_task(
-            self.__queue_stream(process.stderr, stderr_queue)
+        cmd_script = self._get_cmd_script(ctx)
+        ctx.log_debug(f"Script: {self.__get_multiline_repr(cmd_script)}")
+        shell = self._get_shell(ctx)
+        ctx.log_debug(f"Shell: {shell}")
+        shell_flag = self._get_shell_flag(ctx)
+        cwd = self._get_cwd(ctx)
+        ctx.log_debug(f"Working directory: {cwd}")
+        env_map = self.__get_env_map(ctx)
+        ctx.log_debug(f"Environment map: {env_map}")
+        if self._get_should_warn_unrecommended_commands():
+            self._check_unrecommended_commands(ctx, shell, cmd_script)
+        ctx.log_info("Running script")
+        print_method = (
+            partial(ctx.print, plain=True) if self._should_plain_print else ctx.print
         )
-        # Handle messages in queue
-        stdout_log_process = asyncio.create_task(
-            self.__log_from_queue(
-                stdout_queue, self.print_out, self._output_buffer, self._max_output_size
-            )
+        xcom_pid_key = f"{self.name}-pid"
+        ctx.xcom[xcom_pid_key] = Xcom([])
+        cmd_result, return_code = await run_command(
+            cmd=[shell, shell_flag, cmd_script],
+            cwd=cwd,
+            env_map=env_map,
+            print_method=print_method,
+            register_pid_method=lambda pid: ctx.xcom.get(xcom_pid_key).push(pid),
+            max_output_line=self._max_output_line,
+            max_error_line=self._max_error_line,
         )
-        stderr_log_process = asyncio.create_task(
-            self.__log_from_queue(
-                stderr_queue, self.print_err, self._error_buffer, self._max_error_size
-            )
+        # Check for errors
+        if return_code > 0:
+            raise Exception(f"Process {self._name} exited ({return_code})")
+        ctx.log_info(f"Exit status: {return_code}")
+        return cmd_result
+
+    def _get_should_warn_unrecommended_commands(self):
+        if self._should_warn_unrecommended_command is None:
+            return WARN_UNRECOMMENDED_COMMAND
+        return self._should_warn_unrecommended_command
+
+    def _check_unrecommended_commands(
+        self, ctx: AnyContext, shell: str, cmd_script: str
+    ):
+        if shell.endswith("bash") or shell.endswith("zsh"):
+            unrecommended_commands = check_unrecommended_commands(cmd_script)
+            if unrecommended_commands:
+                ctx.log_warning("The script contains unrecommended commands")
+            for command, reason in unrecommended_commands.items():
+                ctx.log_warning(f"- {command}: {reason}")
+
+    def __get_env_map(self, ctx: AnyContext) -> dict[str, str]:
+        envs = {key: val for key, val in ctx.env.items()}
+        envs["_ZRB_SSH_PASSWORD"] = self._get_remote_password(ctx)
+        envs["PYTHONBUFFERED"] = "1"
+        return envs
+
+    def _get_shell(self, ctx: AnyContext) -> str:
+        return get_str_attr(
+            ctx, self._shell, DEFAULT_SHELL, auto_render=self._render_shell
         )
-        # wait process
-        await asyncio.gather(
-            process.wait(),
-            stdout_process,
-            stderr_process,
+
+    def _get_shell_flag(self, ctx: AnyContext) -> str:
+        default_shell_flags = {
+            "node": "-e",
+            "ruby": "-e",
+            "php": "-r",
+            "powershell": "/c",
+        }
+        default_shell_flag = default_shell_flags.get(self._get_shell(ctx).lower(), "-c")
+        return get_str_attr(
+            ctx,
+            self._shell_flag,
+            default_shell_flag,
+            auto_render=self._render_shell_flag,
         )
-        # stop messages in queue
-        await asyncio.gather(stdout_queue.put(None), stderr_queue.put(None))
-        # end logging
-        await asyncio.gather(stdout_log_process, stderr_log_process)
 
-    def get_cmd_script(self, *args: Any, **kwargs: Any) -> str:
-        return self._create_cmd_script(self._cmd_path, self._cmd, *args, **kwargs)
+    def _get_remote_host(self, ctx: AnyContext) -> str:
+        return get_str_attr(
+            ctx, self._remote_host, "", auto_render=self._render_remote_host
+        )
 
-    def _create_cmd_script(
-        self, cmd_path: CmdVal, cmd: CmdVal, *args: Any, **kwargs: Any
-    ) -> str:
-        if not isinstance(cmd_path, str) or cmd_path != "":
-            if callable(cmd_path):
-                return self.__get_rendered_cmd_path(cmd_path(*args, **kwargs))
-            return self.__get_rendered_cmd_path(cmd_path)
-        if callable(cmd):
-            return self.__get_rendered_cmd(cmd(*args, **kwargs))
-        return self.__get_rendered_cmd(cmd)
+    def _get_remote_port(self, ctx: AnyContext) -> int:
+        return get_int_attr(
+            ctx, self._remote_port, 22, auto_render=self._render_remote_port
+        )
 
-    def __get_rendered_cmd_path(self, cmd_path: Union[str, Iterable[str]]) -> str:
-        if isinstance(cmd_path, str):
-            return self.render_file(cmd_path)
-        return "\n".join([self.render_file(cmd_path_str) for cmd_path_str in cmd_path])
+    def _get_remote_user(self, ctx: AnyContext) -> str:
+        return get_str_attr(
+            ctx, self._remote_user, "", auto_render=self._render_remote_user
+        )
 
-    def __get_rendered_cmd(
-        self, cmd: Union[JinjaTemplate, Iterable[JinjaTemplate]]
-    ) -> str:
-        if isinstance(cmd, str):
-            return self.render_str(cmd)
-        return self.render_str("\n".join(list(cmd)))
+    def _get_remote_password(self, ctx: AnyContext) -> str:
+        return get_str_attr(
+            ctx,
+            self._remote_password,
+            "",
+            auto_render=self._render_remote_password,
+        )
 
-    async def __queue_stream(self, stream, queue: asyncio.Queue):
-        while True:
-            try:
-                line = await stream.readline()
-            except Exception as e:
-                line = str(e)
-            if not line:
-                break
-            await queue.put(line)
+    def _get_remote_ssh_key(self, ctx: AnyContext) -> str:
+        return get_str_attr(
+            ctx, self._remote_ssh_key, "", auto_render=self._render_remote_ssh_key
+        )
 
-    async def __log_from_queue(
-        self,
-        queue: asyncio.Queue,
-        print_log: Callable[[str], None],
-        buffer: Iterable[str],
-        max_size: int,
-    ):
-        while True:
-            line = await queue.get()
-            if not line:
-                break
-            line_str = line.decode("utf-8").rstrip()
-            self.__add_to_buffer(buffer, max_size, line_str)
-            _reset_stty()
-            print_log(line_str)
-            _reset_stty()
+    def _get_cwd(self, ctx: AnyContext) -> str:
+        cwd = get_str_attr(ctx, self._cwd, os.getcwd(), auto_render=self._render_cwd)
+        if cwd is None:
+            cwd = os.getcwd()
+        return os.path.abspath(cwd)
+
+    def _get_cmd_script(self, ctx: AnyContext) -> str:
+        if self._remote_host is None:
+            return self._get_local_cmd_script(ctx)
+        return self._get_remote_cmd_script(ctx)
+
+    def _get_remote_cmd_script(self, ctx: AnyContext) -> str:
+        return get_remote_cmd_script(
+            cmd_script=self._get_local_cmd_script(ctx),
+            host=self._get_remote_host(ctx),
+            port=self._get_remote_port(ctx),
+            user=self._get_remote_user(ctx),
+            password="$_ZRB_SSH_PASSWORD",
+            use_password=self._get_remote_password(ctx) != "",
+            ssh_key=self._get_remote_ssh_key(ctx),
+        )
 
-    def __add_to_buffer(self, buffer: Iterable[str], max_size: int, new_line: str):
-        if len(buffer) >= max_size:
-            buffer.pop(0)
-        buffer.append(new_line)
+    def _get_local_cmd_script(self, ctx: AnyContext) -> str:
+        return self._render_cmd_val(ctx, self._cmd)
+
+    def _render_cmd_val(self, ctx: AnyContext, cmd_val: CmdVal) -> str:
+        if isinstance(cmd_val, list):
+            cmd_val_list = [
+                self.__render_single_cmd_val(ctx, single_cmd_val)
+                for single_cmd_val in cmd_val
+            ]
+            return "\n".join(
+                [cmd_val for cmd_val in cmd_val_list if cmd_val is not None]
+            )
+        return self.__render_single_cmd_val(ctx, cmd_val)
+
+    def __render_single_cmd_val(
+        self, ctx: AnyContext, single_cmd_val: SingleCmdVal
+    ) -> str | None:
+        if callable(single_cmd_val):
+            return single_cmd_val(ctx)
+        if isinstance(single_cmd_val, str):
+            if self._render_cmd:
+                return ctx.render(single_cmd_val)
+            return single_cmd_val
+        if isinstance(single_cmd_val, AnyCmdVal):
+            return single_cmd_val.to_str(ctx)
+        return None
 
     def __get_multiline_repr(self, text: str) -> str:
-        lines_repr: Iterable[str] = []
+        lines_repr: list[str] = []
         lines = text.split("\n")
         if len(lines) == 1:
             return lines[0]
         for index, line in enumerate(lines):
             line_number_repr = str(index + 1).rjust(4, "0")
-            lines_repr.append(f"        {line_number_repr} | {line}")
+            lines_repr.append(f"   {line_number_repr} | {line}")
         return "\n" + "\n".join(lines_repr)