mcli-framework 8.0.61__tar.gz → 8.0.62__tar.gz

{mcli_framework-8.0.61/src/mcli_framework.egg-info → mcli_framework-8.0.62}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: mcli-framework
-Version: 8.0.61
+Version: 8.0.62
 Summary: Portable workflow framework - transform any script into a versioned, schedulable command. Store in ~/.mcli/workflows/, version with lockfile, run as daemon or cron job.
 Author-email: Luis Fernandez de la Vara <luis@lefv.io>
 Maintainer-email: Luis Fernandez de la Vara <luis@lefv.io>

{mcli_framework-8.0.61 → mcli_framework-8.0.62}/pyproject.toml

@@ -1,6 +1,6 @@
 [project]
 name = "mcli-framework"
-version = "8.0.61"
+version = "8.0.62"
 description = "Portable workflow framework - transform any script into a versioned, schedulable command. Store in ~/.mcli/workflows/, version with lockfile, run as daemon or cron job."
 readme = "README.md"
  requires-python = ">=3.10"

{mcli_framework-8.0.61 → mcli_framework-8.0.62}/src/mcli/lib/constants/defaults.py

@@ -39,6 +39,7 @@ class URLs:
     GITHUB_API = "https://api.github.com"
     GITHUB_API_MCLI = "https://api.github.com/repos/gwicho38/mcli"
     GITHUB_API_ACTIONS = "https://api.github.com/repos/gwicho38/mcli/actions/runs"
+    GITHUB_ISSUES_MCLI = "https://github.com/gwicho38/mcli/issues"
 
     # PyPI
     PYPI_API = "https://pypi.org/pypi"

{mcli_framework-8.0.61 → mcli_framework-8.0.62}/src/mcli/lib/constants/messages.py

@@ -17,6 +17,7 @@ class ErrorMessages:
     )
     COMMAND_NOT_FOUND = "Command '{name}' not found"
     COMMAND_NOT_AVAILABLE = "Command {name} is not available"
+    WORKFLOW_CRASHED = "Workflow '{name}' crashed: {error}"
     GIT_COMMAND_FAILED = "Git command failed: {error}"
     FILE_NOT_FOUND = "File not found: {path}"
     DIRECTORY_NOT_FOUND = "Directory not found: {path}"
@@ -66,6 +67,9 @@ class InfoMessages:
 
     COPYING_COMMANDS = "Copying commands from {source} to {dest}..."
     NO_CHANGES_TO_COMMIT = "No changes to commit"
+    WORKFLOW_CRASH_LOCATION = "  in {location}"
+    WORKFLOW_CRASH_TRACE_HINT = "  Set MCLI_TRACE_LEVEL=1 (or MCLI_DEBUG=1) for the full traceback."
+    WORKFLOW_CRASH_REPORT = "  Report this: {url}"
     LOADING = "Loading {item}..."
     PROCESSING = "Processing {item}..."
     CONNECTING = "Connecting to {service}..."

mcli_framework-8.0.62/src/mcli/lib/workflow_runtime.py

@@ -0,0 +1,100 @@
+"""Runtime helpers for executing user workflow scripts safely.
+
+When a dynamically-loaded workflow script raises an unhandled exception at
+*invoke* time (i.e. it loads fine but crashes while running), Click/Python
+would dump a raw traceback at the user. That tells them little about which
+script failed or what to do next.
+
+``wrap_command_invoke`` wraps a loaded command so such crashes are rendered as
+a concise, actionable message — which workflow crashed, the error, the script
+location, and how to report it. The full traceback stays one env var away
+(``MCLI_TRACE_LEVEL`` / ``MCLI_DEBUG``) for debugging.
+
+Click usage errors, ``Abort`` and ``SystemExit`` are deliberately *not*
+swallowed — those are normal control flow, not script crashes.
+"""
+
+from __future__ import annotations
+
+import os
+import sys
+import traceback
+from pathlib import Path
+
+import click
+from rich.console import Console
+
+from mcli.lib.constants.defaults import URLs
+from mcli.lib.constants.env import EnvVars
+from mcli.lib.constants.messages import ErrorMessages, InfoMessages
+
+# Module-level so tests can redirect it to a buffer.
+_console = Console(stderr=True)
+
+
+def _wants_traceback() -> bool:
+    """Whether to print the full traceback (debug/trace modes)."""
+    trace = os.environ.get(EnvVars.MCLI_TRACE_LEVEL, "").strip()
+    if trace and trace != "0":
+        return True
+    debug = os.environ.get(EnvVars.MCLI_DEBUG, "").strip().lower()
+    return debug in ("1", "true", "yes", "on")
+
+
+def _deepest_script_line(script_path: Path, exc: BaseException) -> int | None:
+    """Return the deepest line number within the user's script, if any."""
+    target = str(script_path)
+    line: int | None = None
+    for frame, lineno in traceback.walk_tb(exc.__traceback__):
+        if frame.f_code.co_filename == target:
+            line = lineno
+    return line
+
+
+def render_script_error(name: str, script_path: str | Path, exc: BaseException) -> None:
+    """Print a concise, actionable crash summary for a workflow script."""
+    script_path = Path(script_path)
+    line = _deepest_script_line(script_path, exc)
+    location = str(script_path) if line is None else f"{script_path}:{line}"
+
+    summary = ErrorMessages.WORKFLOW_CRASHED.format(name=name, error=f"{type(exc).__name__}: {exc}")
+    _console.print(f"[bold red]✗[/bold red] {summary}", soft_wrap=True, highlight=False)
+    _console.print(
+        InfoMessages.WORKFLOW_CRASH_LOCATION.format(location=location),
+        soft_wrap=True,
+        highlight=False,
+    )
+    _console.print(InfoMessages.WORKFLOW_CRASH_TRACE_HINT, soft_wrap=True, highlight=False)
+    _console.print(
+        InfoMessages.WORKFLOW_CRASH_REPORT.format(url=URLs.GITHUB_ISSUES_MCLI),
+        soft_wrap=True,
+        highlight=False,
+    )
+
+    if _wants_traceback():
+        traceback.print_exception(type(exc), exc, exc.__traceback__, file=sys.stderr)
+
+
+def wrap_command_invoke(command: click.BaseCommand, script_path: str | Path):
+    """Wrap a loaded command so runtime crashes render a friendly message.
+
+    Returns the same command (mutated in place). ``None`` passes through so
+    callers don't need to guard.
+    """
+    if command is None:
+        return command
+
+    path = Path(script_path)
+    original_invoke = command.invoke
+
+    def safe_invoke(ctx):
+        try:
+            return original_invoke(ctx)
+        except (click.ClickException, click.Abort, SystemExit, KeyboardInterrupt):
+            raise
+        except Exception as exc:  # noqa: BLE001 - intentional catch-all for user scripts
+            render_script_error(command.name or path.stem, path, exc)
+            raise SystemExit(1) from exc
+
+    command.invoke = safe_invoke  # type: ignore[method-assign]
+    return command

{mcli_framework-8.0.61 → mcli_framework-8.0.62}/src/mcli/workflow/workflow.py

@@ -140,6 +140,7 @@ class ScopedWorkflowsGroup(click.Group):
         from mcli.lib.constants.messages import WarningMessages
         from mcli.lib.logger.logger import get_logger
         from mcli.lib.script_loader import LANGUAGE_TO_SUFFIX, ScriptLoader, parse_command_name
+        from mcli.lib.workflow_runtime import wrap_command_invoke
 
         logger = get_logger()
 
@@ -171,7 +172,7 @@ class ScopedWorkflowsGroup(click.Group):
                     command = loader.load_command(matches[0])
                     if command:
                         logger.debug(f"Loaded native script command: {cmd_name}")
-                        return command
+                        return wrap_command_invoke(command, matches[0])
                 except Exception as e:
                     logger.debug(f"Failed to load script '{cmd_name}': {e}")
             elif len(matches) > 1 and not lang_filter:
@@ -185,7 +186,7 @@ class ScopedWorkflowsGroup(click.Group):
                     command = loader.load_command(matches[0])
                     if command:
                         logger.debug(f"Loaded first match for ambiguous command: {base_name}")
-                        return command
+                        return wrap_command_invoke(command, matches[0])
                 except Exception as e:
                     logger.debug(f"Failed to load script '{base_name}': {e}")
 

{mcli_framework-8.0.61 → mcli_framework-8.0.62/src/mcli_framework.egg-info}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: mcli-framework
-Version: 8.0.61
+Version: 8.0.62
 Summary: Portable workflow framework - transform any script into a versioned, schedulable command. Store in ~/.mcli/workflows/, version with lockfile, run as daemon or cron job.
 Author-email: Luis Fernandez de la Vara <luis@lefv.io>
 Maintainer-email: Luis Fernandez de la Vara <luis@lefv.io>

{mcli_framework-8.0.61 → mcli_framework-8.0.62}/src/mcli_framework.egg-info/SOURCES.txt

@@ -52,6 +52,7 @@ src/mcli/lib/script_watcher.py
 src/mcli/lib/sync_key_store.py
 src/mcli/lib/types.py
 src/mcli/lib/workflow_models.py
+src/mcli/lib/workflow_runtime.py
 src/mcli/lib/workspace_registry.py
 src/mcli/lib/api/__init__.py
 src/mcli/lib/api/api.py