falyx 0.1.16__py3-none-any.whl → 0.1.18__py3-none-any.whl

falyx/command.py

@@ -29,14 +29,16 @@ from rich.tree import Tree
 from falyx.action import Action, ActionGroup, BaseAction, ChainedAction
 from falyx.context import ExecutionContext
 from falyx.debug import register_debug_hooks
+from falyx.exceptions import FalyxError
 from falyx.execution_registry import ExecutionRegistry as er
 from falyx.hook_manager import HookManager, HookType
 from falyx.io_action import BaseIOAction
 from falyx.options_manager import OptionsManager
+from falyx.prompt_utils import should_prompt_user
 from falyx.retry import RetryPolicy
 from falyx.retry_utils import enable_retries_recursively
 from falyx.themes.colors import OneColors
-from falyx.utils import _noop, ensure_async, logger
+from falyx.utils import _noop, confirm_async, ensure_async, logger
 
 console = Console()
 
@@ -180,7 +182,10 @@ class Command(BaseModel):
             self.action.set_options_manager(self.options_manager)
 
     async def __call__(self, *args, **kwargs) -> Any:
-        """Run the action with full hook lifecycle, timing, and error handling."""
+        """
+        Run the action with full hook lifecycle, timing, error handling,
+        confirmation prompts, preview, and spinner integration.
+        """
         self._inject_options_manager()
         combined_args = args + self.args
         combined_kwargs = {**self.kwargs, **kwargs}
@@ -191,11 +196,29 @@ class Command(BaseModel):
             action=self,
         )
         self._context = context
+
+        if should_prompt_user(confirm=self.confirm, options=self.options_manager):
+            if self.preview_before_confirm:
+                await self.preview()
+            if not await confirm_async(self.confirmation_prompt):
+                logger.info(f"[Command:{self.key}] ❌ Cancelled by user.")
+                raise FalyxError(f"[Command:{self.key}] Cancelled by confirmation.")
+
         context.start_timer()
 
         try:
             await self.hooks.trigger(HookType.BEFORE, context)
-            result = await self.action(*combined_args, **combined_kwargs)
+            if self.spinner:
+                with console.status(
+                    self.spinner_message,
+                    spinner=self.spinner_type,
+                    spinner_style=self.spinner_style,
+                    **self.spinner_kwargs,
+                ):
+                    result = await self.action(*combined_args, **combined_kwargs)
+            else:
+                result = await self.action(*combined_args, **combined_kwargs)
+
             context.result = result
             await self.hooks.trigger(HookType.ON_SUCCESS, context)
             return context.result

falyx/context.py

@@ -129,7 +129,7 @@ class ExecutionContext(BaseModel):
         if self.start_wall:
             message.append(f"Start: {self.start_wall.strftime('%H:%M:%S')} | ")
 
-        if self.end_time:
+        if self.end_wall:
             message.append(f"End: {self.end_wall.strftime('%H:%M:%S')} | ")
 
         message.append(f"Duration: {summary['duration']:.3f}s | ")

falyx/falyx.py

@@ -55,13 +55,7 @@ from falyx.parsers import get_arg_parsers
 from falyx.retry import RetryPolicy
 from falyx.signals import BackSignal, QuitSignal
 from falyx.themes.colors import OneColors, get_nord_theme
-from falyx.utils import (
-    CaseInsensitiveDict,
-    chunks,
-    confirm_async,
-    get_program_invocation,
-    logger,
-)
+from falyx.utils import CaseInsensitiveDict, chunks, get_program_invocation, logger
 from falyx.version import __version__
 
 
@@ -93,9 +87,8 @@ class Falyx:
         key_bindings (KeyBindings | None): Custom Prompt Toolkit key bindings.
         include_history_command (bool): Whether to add a built-in history viewer command.
         include_help_command (bool): Whether to add a built-in help viewer command.
-        confirm_on_error (bool): Whether to prompt the user after errors.
-        never_prompt (bool): Whether to skip confirmation prompts entirely.
-        always_confirm (bool): Whether to force confirmation prompts for all actions.
+        never_prompt (bool): Seed default for `OptionsManager["never_prompt"]`
+        force_confirm (bool): Seed default for `OptionsManager["force_confirm"]`
         cli_args (Namespace | None): Parsed CLI arguments, usually from argparse.
         options (OptionsManager | None): Declarative option mappings.
         custom_table (Callable[[Falyx], Table] | Table | None): Custom menu table generator.
@@ -123,9 +116,8 @@ class Falyx:
         key_bindings: KeyBindings | None = None,
         include_history_command: bool = True,
         include_help_command: bool = True,
-        confirm_on_error: bool = True,
         never_prompt: bool = False,
-        always_confirm: bool = False,
+        force_confirm: bool = False,
         cli_args: Namespace | None = None,
         options: OptionsManager | None = None,
         render_menu: Callable[["Falyx"], None] | None = None,
@@ -150,16 +142,15 @@ class Falyx:
         self.last_run_command: Command | None = None
         self.key_bindings: KeyBindings = key_bindings or KeyBindings()
         self.bottom_bar: BottomBar | str | Callable[[], None] = bottom_bar
-        self.confirm_on_error: bool = confirm_on_error
         self._never_prompt: bool = never_prompt
-        self._always_confirm: bool = always_confirm
+        self._force_confirm: bool = force_confirm
         self.cli_args: Namespace | None = cli_args
         self.render_menu: Callable[["Falyx"], None] | None = render_menu
         self.custom_table: Callable[["Falyx"], Table] | Table | None = custom_table
-        self.set_options(cli_args, options)
+        self.validate_options(cli_args, options)
         self._session: PromptSession | None = None
 
-    def set_options(
+    def validate_options(
         self,
         cli_args: Namespace | None,
         options: OptionsManager | None = None,
@@ -175,8 +166,6 @@ class Falyx:
         assert isinstance(
             cli_args, Namespace
         ), "CLI arguments must be a Namespace object."
-        if options is None:
-            self.options.from_namespace(cli_args, "cli_args")
 
         if not isinstance(self.options, OptionsManager):
             raise FalyxError("Options must be an instance of OptionsManager.")
@@ -705,33 +694,8 @@ class Falyx:
                 self.console.print(
                     f"[{OneColors.LIGHT_YELLOW}]⚠️ Unknown command '{choice}'[/]"
                 )
-        logger.warning(f"⚠️ Command '{choice}' not found.")
         return None
 
-    async def _should_run_action(self, selected_command: Command) -> bool:
-        if self._never_prompt:
-            return True
-
-        if self.cli_args and getattr(self.cli_args, "skip_confirm", False):
-            return True
-
-        if (
-            self._always_confirm
-            or selected_command.confirm
-            or self.cli_args
-            and getattr(self.cli_args, "force_confirm", False)
-        ):
-            if selected_command.preview_before_confirm:
-                await selected_command.preview()
-            confirm_answer = await confirm_async(selected_command.confirmation_prompt)
-
-            if confirm_answer:
-                logger.info(f"[{selected_command.description}]🔐 confirmed.")
-            else:
-                logger.info(f"[{selected_command.description}]❌ cancelled.")
-            return confirm_answer
-        return True
-
     def _create_context(self, selected_command: Command) -> ExecutionContext:
         """Creates a context dictionary for the selected command."""
         return ExecutionContext(
@@ -741,16 +705,6 @@ class Falyx:
             action=selected_command,
         )
 
-    async def _run_action_with_spinner(self, command: Command) -> Any:
-        """Runs the action of the selected command with a spinner."""
-        with self.console.status(
-            command.spinner_message,
-            spinner=command.spinner_type,
-            spinner_style=command.spinner_style,
-            **command.spinner_kwargs,
-        ):
-            return await command()
-
     async def _handle_action_error(
         self, selected_command: Command, error: Exception
     ) -> None:
@@ -784,19 +738,12 @@ class Falyx:
             logger.info(f"🔙 Back selected: exiting {self.get_title()}")
             return False
 
-        if not await self._should_run_action(selected_command):
-            logger.info(f"{selected_command.description} cancelled.")
-            return True
-
         context = self._create_context(selected_command)
         context.start_timer()
         try:
             await self.hooks.trigger(HookType.BEFORE, context)
 
-            if selected_command.spinner:
-                result = await self._run_action_with_spinner(selected_command)
-            else:
-                result = await selected_command()
+            result = await selected_command()
             context.result = result
             await self.hooks.trigger(HookType.ON_SUCCESS, context)
         except Exception as error:
@@ -824,22 +771,11 @@ class Falyx:
             selected_command.description,
         )
 
-        if not await self._should_run_action(selected_command):
-            logger.info("[run_key] ❌ Cancelled: %s", selected_command.description)
-            raise FalyxError(
-                f"[run_key] '{selected_command.description}' "
-                "cancelled by confirmation."
-            )
-
         context = self._create_context(selected_command)
         context.start_timer()
         try:
             await self.hooks.trigger(HookType.BEFORE, context)
-
-            if selected_command.spinner:
-                result = await self._run_action_with_spinner(selected_command)
-            else:
-                result = await selected_command()
+            result = await selected_command()
             context.result = result
 
             await self.hooks.trigger(HookType.ON_SUCCESS, context)
@@ -939,6 +875,13 @@ class Falyx:
         """Run Falyx CLI with structured subcommands."""
         if not self.cli_args:
             self.cli_args = get_arg_parsers().root.parse_args()
+        self.options.from_namespace(self.cli_args, "cli_args")
+
+        if not self.options.get("never_prompt"):
+            self.options.set("never_prompt", self._never_prompt)
+
+        if not self.options.get("force_confirm"):
+            self.options.set("force_confirm", self._force_confirm)
 
         if self.cli_args.verbose:
             logging.getLogger("falyx").setLevel(logging.DEBUG)
@@ -947,9 +890,6 @@ class Falyx:
             logger.debug("✅ Enabling global debug hooks for all commands")
             self.register_all_with_debug_hooks()
 
-        if self.cli_args.never_prompt:
-            self._never_prompt = True
-
         if self.cli_args.command == "list":
             await self._show_help()
             sys.exit(0)

falyx/http_action.py

@@ -97,30 +97,36 @@ class HTTPAction(Action):
         )
 
     async def _request(self, *args, **kwargs) -> dict[str, Any]:
-        assert self.shared_context is not None, "SharedContext is not set"
-        context: SharedContext = self.shared_context
-
-        session = context.get("http_session")
-        if session is None:
+        # TODO: Add check for HOOK registration
+        if self.shared_context:
+            context: SharedContext = self.shared_context
+            session = context.get("http_session")
+            if session is None:
+                session = aiohttp.ClientSession()
+                context.set("http_session", session)
+                context.set("_session_should_close", True)
+        else:
             session = aiohttp.ClientSession()
-            context.set("http_session", session)
-            context.set("_session_should_close", True)
-
-        async with session.request(
-            self.method,
-            self.url,
-            headers=self.headers,
-            params=self.params,
-            json=self.json,
-            data=self.data,
-        ) as response:
-            body = await response.text()
-            return {
-                "status": response.status,
-                "url": str(response.url),
-                "headers": dict(response.headers),
-                "body": body,
-            }
+
+        try:
+            async with session.request(
+                self.method,
+                self.url,
+                headers=self.headers,
+                params=self.params,
+                json=self.json,
+                data=self.data,
+            ) as response:
+                body = await response.text()
+                return {
+                    "status": response.status,
+                    "url": str(response.url),
+                    "headers": dict(response.headers),
+                    "body": body,
+                }
+        finally:
+            if not self.shared_context:
+                await session.close()
 
     async def preview(self, parent: Tree | None = None):
         label = [

falyx/prompt_utils.py

@@ -0,0 +1,19 @@
+from falyx.options_manager import OptionsManager
+
+
+def should_prompt_user(
+    *,
+    confirm: bool,
+    options: OptionsManager,
+    namespace: str = "cli_args",
+):
+    """Determine whether to prompt the user for confirmation based on command and global options."""
+    never_prompt = options.get("never_prompt", False, namespace)
+    always_confirm = options.get("always_confirm", False, namespace)
+    force_confirm = options.get("force_confirm", False, namespace)
+    skip_confirm = options.get("skip_confirm", False, namespace)
+
+    if never_prompt or skip_confirm:
+        return False
+
+    return confirm or always_confirm or force_confirm

falyx/selection.py

@@ -145,7 +145,7 @@ def render_selection_indexed_table(
         chunks(range(len(selections)), columns), chunks(selections, columns)
     ):
         row = [
-            formatter(index, selection) if formatter else f"{index}: {selection}"
+            formatter(index, selection) if formatter else f"[{index}] {selection}"
             for index, selection in zip(indexes, chunk)
         ]
         table.add_row(*row)

falyx/selection_action.py

@@ -87,17 +87,23 @@ class SelectionAction(BaseAction):
         if isinstance(self.selections, dict):
             if maybe_result in self.selections:
                 effective_default = maybe_result
+            elif self.inject_last_result:
+                logger.warning(
+                    "[%s] Injected last result '%s' not found in selections",
+                    self.name,
+                    maybe_result,
+                )
         elif isinstance(self.selections, list):
             if maybe_result.isdigit() and int(maybe_result) in range(
                 len(self.selections)
             ):
                 effective_default = maybe_result
-        elif self.inject_last_result:
-            logger.warning(
-                "[%s] Injected last result '%s' not found in selections",
-                self.name,
-                maybe_result,
-            )
+            elif self.inject_last_result:
+                logger.warning(
+                    "[%s] Injected last result '%s' not found in selections",
+                    self.name,
+                    maybe_result,
+                )
 
         if self.never_prompt and not effective_default:
             raise ValueError(

falyx/version.py

@@ -1 +1 @@
-__version__ = "0.1.16"
+__version__ = "0.1.18"

{falyx-0.1.16.dist-info → falyx-0.1.18.dist-info}/METADATA

@@ -1,6 +1,6 @@
 Metadata-Version: 2.3
 Name: falyx
-Version: 0.1.16
+Version: 0.1.18
 Summary: Reliable and introspectable async CLI action framework.
 License: MIT
 Author: Roland Thomas Jr

{falyx-0.1.16.dist-info → falyx-0.1.18.dist-info}/RECORD

@@ -1,37 +1,39 @@
+falyx/.coverage,sha256=DNx1Ew1vSvuIcKko7httsyL62erJxVQ6CKtuJKxRVj4,53248
 falyx/.pytyped,sha256=47DEQpj8HBSa-_TImW-5JCeuQeRkm5NMpJWZG3hSuFU,0
 falyx/__init__.py,sha256=dYRamQJlT1Zoy5Uu1uG4NCV05Xk98nN1LAQrSR1CT2A,643
 falyx/__main__.py,sha256=pXxXLlDot33dc4mR11Njpr4M_xbSTdKEqKWMS2aUqfk,2195
 falyx/action.py,sha256=7WuiJyRlhXqnGxfkXWIcAppzsIZ38OyZ9SKs-4hdhAg,31170
 falyx/bottom_bar.py,sha256=83KSElBU7sFJqUhKyfef6gYfPnNDM8V7bph60cY5ARQ,7384
-falyx/command.py,sha256=kS4VvK4C7c3AO4fldt4QTS6yzKQAyWtvgWGNoKBoxNA,10939
+falyx/command.py,sha256=8Db_A3WDxmFNuZUb4_PO0UlpnfZFAUUEFkgC7B8v1Jk,11891
 falyx/config.py,sha256=czt_EjOCT0lzWcoRE2F-oS2k1AVBFdiNuTcQXHlUVD0,4534
-falyx/context.py,sha256=M8hpYpTlY5Lg2QxgiTcp2RIajemav8mVmGr11VKM07g,10070
+falyx/context.py,sha256=Dm7HV-eigU-aTv5ERah6Ow9fIRdrOsB1G6ETPIu42Gw,10070
 falyx/debug.py,sha256=-jbTti29UC5zP9qQlWs3TbkOQR2f3zKSuNluh-r56wY,1551
 falyx/exceptions.py,sha256=YVbhPp2BNvZoO_xqeGSRKHVQ2rdLOLf1HCjH4JTj9w8,776
 falyx/execution_registry.py,sha256=xB2SJuEoDxxfwUmKXLAZQSrVoNPXwnVML98sTgwBqRI,2869
-falyx/falyx.py,sha256=tDjs78Y0Ifv8YgAR8ABKRW_Q50I8jkuWdt3-Rta4BNY,40672
+falyx/falyx.py,sha256=NdQdaQcG3V4D9lgMOrgH_8_w3T-MqUiGNeyBZXKpTbQ,38522
 falyx/hook_manager.py,sha256=E9Vk4bdoUTeXPQ_BQEvY2Jt-jUAusc40LI8JDy3NLUw,2381
 falyx/hooks.py,sha256=9zXk62DsJLJrmwTdyeNy5s-rVRvl8feuYRrfMmz6cVQ,2802
-falyx/http_action.py,sha256=PZhzoAsOWCT1K1IGYwagh_NP7MTsSqkkAXm9NnI2MJE,5459
+falyx/http_action.py,sha256=thZXnypT7ZUshBg4c9F-c8Pk7-yCjvHmMwHR9cqg_Xw,5715
 falyx/init.py,sha256=-xly0VkOo0lTDaAZyRGEPD8YSP0ySxJtxFs79faZhO4,1799
 falyx/io_action.py,sha256=MsvKj63VmABqdtDURgfGOpiudpTPOL0N_hlmTbYVtcA,10662
 falyx/menu_action.py,sha256=sHKcLYHCl3yzyVjL6cu97NtGVJhPZO4oXkQ4Lb5z3C8,8060
 falyx/options_manager.py,sha256=yYpn-moYN-bRYgMLccmi_de4mUzhTT7cv_bR2FFWZ8c,2798
 falyx/parsers.py,sha256=Ki0rn2wryPDmMI9WUNBLQ5J5Y64BNten0POMZM8wPKU,5189
+falyx/prompt_utils.py,sha256=QgPJpGZfXJlX9SoZNYPnHcUKQ54fZ-Xz5ahNpTZkjcU,647
 falyx/retry.py,sha256=GncBUiDDfDHUvLsWsWQw2Nq2XYL0TR0Fne3iXPzvQ48,3551
 falyx/retry_utils.py,sha256=SN5apcsg71IG2-KylysqdJd-PkPBLoCVwsgrSTF9wrQ,666
 falyx/select_files_action.py,sha256=dJYAnwTBLHIWs-4h1sN2pWBBYykoTO717FA4InrJgeA,2409
-falyx/selection.py,sha256=0uSs6Qz5rZBrQACTco4tLVbR3_VvyYU9Dkp4PAg7pVk,9683
-falyx/selection_action.py,sha256=oSRSs3RpxBhd9s_lVIRnCfRqGt-FBLMNwSHf0x1ucqs,7888
+falyx/selection.py,sha256=JGb8KeU6SH9qep9AIgBclRfYSds2tqucgyHcJ5bdZcw,9684
+falyx/selection_action.py,sha256=G1QjyTrZKtAupOobqwa2sG-7zjSN71HgSlJLiZrpp_8,8147
 falyx/signal_action.py,sha256=wfhW9miSUj9MUoc1WOyk4tU9CtYKAXusHxQdBPYLoyQ,829
 falyx/signals.py,sha256=tlUbz3x6z3rYlUggan_Ntoy4bU5RbOd8UfR4cNcV6kQ,694
 falyx/tagged_table.py,sha256=sn2kosRRpcpeMB8vKk47c9yjpffSz_9FXH_e6kw15mA,1019
 falyx/themes/colors.py,sha256=4aaeAHJetmeNInI0Zytg4E3YqKfPFelpf04vtjSvsS8,19776
 falyx/utils.py,sha256=b1GQ3ooz4Io3zPE7MsoDm7j42AioTG-ZcWH-N2TRpbI,7710
 falyx/validators.py,sha256=NMxqCk8Fr8HQGVDYpg8B_JRk5SKR41E_G9gj1YfQnxg,1316
-falyx/version.py,sha256=yF88-8vL8keLe6gCTumymw0UoMkWkSrJnzLru4zBCLQ,23
-falyx-0.1.16.dist-info/LICENSE,sha256=B0yqgaHuSdhN7T3OBmgQSiDTy8HqT5Oe_dLypRe4Ra4,1073
-falyx-0.1.16.dist-info/METADATA,sha256=eWtPMP6LJCKeMZtZwHc-M5DRLdYzrVq0TGhCD1kQUwQ,5484
-falyx-0.1.16.dist-info/WHEEL,sha256=fGIA9gx4Qxk2KDKeNJCbOEwSrmLtjWCwzBz351GyrPQ,88
-falyx-0.1.16.dist-info/entry_points.txt,sha256=j8owOSl2j1Ss8DtGMnKfgehKaolqnIPhVFHaUBLUnMs,45
-falyx-0.1.16.dist-info/RECORD,,
+falyx/version.py,sha256=6BiuMUkhwQp6bzUZSF8np8F1NwCltEtK0sPBF__tepU,23
+falyx-0.1.18.dist-info/LICENSE,sha256=B0yqgaHuSdhN7T3OBmgQSiDTy8HqT5Oe_dLypRe4Ra4,1073
+falyx-0.1.18.dist-info/METADATA,sha256=tq_ZdtoqSKsn3klMc-ykqVkm2GxiVC96OJOvLKy028s,5484
+falyx-0.1.18.dist-info/WHEEL,sha256=fGIA9gx4Qxk2KDKeNJCbOEwSrmLtjWCwzBz351GyrPQ,88
+falyx-0.1.18.dist-info/entry_points.txt,sha256=j8owOSl2j1Ss8DtGMnKfgehKaolqnIPhVFHaUBLUnMs,45
+falyx-0.1.18.dist-info/RECORD,,