mcli-framework 7.12.0__py3-none-any.whl → 7.12.2__py3-none-any.whl

mcli/app/lock_cmd.py

@@ -229,11 +229,15 @@ def write_state(json_file):
 @click.option(
     "--global", "-g", "is_global", is_flag=True, help="Verify global commands instead of local"
 )
-def verify_commands(is_global):
+@click.option(
+    "--code", "-c", is_flag=True, help="Also validate that workflow code is executable"
+)
+def verify_commands(is_global, code):
     """
-    Verify that custom commands match the lockfile.
+    Verify that custom commands match the lockfile and optionally validate code.
 
     By default verifies local commands (if in git repo), use --global/-g for global commands.
+    Use --code/-c to also validate that workflow code is valid and executable.
     """
     manager = get_command_manager(global_mode=is_global)
 
@@ -242,28 +246,82 @@ def verify_commands(is_global):
 
     verification = manager.verify_lockfile()
 
-    if verification["valid"]:
-        console.print("[green]All custom commands are in sync with the lockfile.[/green]")
-        return 0
-
-    console.print("[yellow]Commands are out of sync with the lockfile:[/yellow]\n")
-
-    if verification["missing"]:
-        console.print("Missing commands (in lockfile but not found):")
-        for name in verification["missing"]:
-            console.print(f"  - {name}")
-
-    if verification["extra"]:
-        console.print("\nExtra commands (not in lockfile):")
-        for name in verification["extra"]:
-            console.print(f"  - {name}")
-
-    if verification["modified"]:
-        console.print("\nModified commands:")
-        for name in verification["modified"]:
-            console.print(f"  - {name}")
+    has_issues = False
+
+    if not verification["valid"]:
+        has_issues = True
+        console.print("[yellow]Commands are out of sync with the lockfile:[/yellow]\n")
+
+        if verification["missing"]:
+            console.print("Missing commands (in lockfile but not found):")
+            for name in verification["missing"]:
+                console.print(f"  - {name}")
+
+        if verification["extra"]:
+            console.print("\nExtra commands (not in lockfile):")
+            for name in verification["extra"]:
+                console.print(f"  - {name}")
+
+        if verification["modified"]:
+            console.print("\nModified commands:")
+            for name in verification["modified"]:
+                console.print(f"  - {name}")
+
+        console.print("\n[dim]Run 'mcli lock update' to sync the lockfile[/dim]\n")
+
+    # Validate workflow code if requested
+    if code:
+        console.print("[cyan]Validating workflow code...[/cyan]\n")
+
+        commands = manager.load_all_commands()
+        invalid_workflows = []
+
+        for cmd_data in commands:
+            if cmd_data.get("group") != "workflow":
+                continue
+
+            cmd_name = cmd_data.get("name")
+            temp_group = click.Group()
+            language = cmd_data.get("language", "python")
+
+            try:
+                if language == "shell":
+                    success = manager.register_shell_command_with_click(cmd_data, temp_group)
+                else:
+                    success = manager.register_command_with_click(cmd_data, temp_group)
+
+                if not success or not temp_group.commands.get(cmd_name):
+                    invalid_workflows.append({
+                        "name": cmd_name,
+                        "reason": "Code does not define a valid Click command"
+                    })
+            except SyntaxError as e:
+                invalid_workflows.append({
+                    "name": cmd_name,
+                    "reason": f"Syntax error: {e}"
+                })
+            except Exception as e:
+                invalid_workflows.append({
+                    "name": cmd_name,
+                    "reason": f"Failed to load: {e}"
+                })
+
+        if invalid_workflows:
+            has_issues = True
+            console.print("[red]Invalid workflows found:[/red]\n")
+
+            for item in invalid_workflows:
+                console.print(f"  [red]✗[/red] {item['name']}")
+                console.print(f"    [dim]{item['reason']}[/dim]")
+
+            console.print(f"\n[yellow]Fix with:[/yellow] mcli workflow edit <workflow-name>")
+            console.print(f"[dim]Tip: Workflow code must define a Click command decorated with @click.command()[/dim]\n")
+        else:
+            console.print("[green]✓ All workflow code is valid[/green]\n")
 
-    console.print("\n[dim]Run 'mcli lock update' to sync the lockfile[/dim]")
+    if not has_issues:
+        console.print("[green]✓ All custom commands are verified[/green]")
+        return 0
 
     return 1
 

mcli/app/main.py

@@ -467,54 +467,14 @@ def create_app() -> click.Group:
 
     app = click.Group(name="mcli")
 
-    # Clean top-level commands
-    @app.command()
-    @click.option("--verbose", "-v", is_flag=True, help="Show additional system information")
-    def version(verbose: bool):
-        """Show mcli version and system information"""
-        message = get_version_info(verbose)
-        logger.info(message)
-        info(message)
-
+    # Version command moved to self group (mcli self version)
     # Add lazy-loaded command groups
     _add_lazy_commands(app)
 
     return app
 
 
-@lru_cache()
-def get_version_info(verbose: bool = False) -> str:
-    """Get version info, cached to prevent multiple calls."""
-    try:
-        # Try mcli-framework first (PyPI package name), then mcli (local dev)
-        mcli_version = None
-        meta = None
-
-        for pkg_name in ["mcli-framework", "mcli"]:
-            try:
-                mcli_version = version(pkg_name)
-                meta = metadata(pkg_name)
-                break
-            except Exception:
-                continue
-
-        if mcli_version is None:
-            return "Could not determine version: Package metadata not found"
-
-        info = [f"mcli version {mcli_version}"]
-
-        if verbose:
-            info.extend(
-                [
-                    f"\nPython: {sys.version.split()[0]}",
-                    f"Platform: {platform.platform()}",
-                    f"Description: {meta.get('Summary', 'Not available')}",
-                    f"Author: {meta.get('Author', 'Not available')}",
-                ]
-            )
-        return "\n".join(info)
-    except Exception as e:
-        return f"Could not determine version: {e}"
+# get_version_info moved to self_cmd.py
 
 
 def main():

mcli/self/self_cmd.py

@@ -8,9 +8,13 @@ import importlib
 import inspect
 import json
 import os
+import platform
 import re
+import sys
 import time
 from datetime import datetime
+from functools import lru_cache
+from importlib.metadata import metadata, version
 from pathlib import Path
 from typing import Any, Dict, List, Optional
 
@@ -48,6 +52,43 @@ console = Console()
 
 LOCKFILE_PATH = Path.home() / ".local" / "mcli" / "command_lock.json"
 
+
+# Version info utility
+@lru_cache()
+def get_version_info(verbose: bool = False) -> str:
+    """Get version info, cached to prevent multiple calls."""
+    try:
+        # Try mcli-framework first (PyPI package name), then mcli (local dev)
+        mcli_version = None
+        meta = None
+
+        for pkg_name in ["mcli-framework", "mcli"]:
+            try:
+                mcli_version = version(pkg_name)
+                meta = metadata(pkg_name)
+                break
+            except Exception:
+                continue
+
+        if mcli_version is None:
+            return "Could not determine version: Package metadata not found"
+
+        info = [f"mcli version {mcli_version}"]
+
+        if verbose:
+            info.extend(
+                [
+                    f"\nPython: {sys.version.split()[0]}",
+                    f"Platform: {platform.platform()}",
+                    f"Description: {meta.get('Summary', 'Not available')}",
+                    f"Author: {meta.get('Author', 'Not available')}",
+                ]
+            )
+        return "\n".join(info)
+    except Exception as e:
+        return f"Error getting version info: {e}"
+
+
 # Utility functions for command state lockfile
 
 
@@ -649,6 +690,17 @@ def plugin_update(plugin_name):
     return 0
 
 
+@self_app.command("version")
+@click.option("--verbose", "-v", is_flag=True, help="Show additional system information")
+def version_cmd(verbose: bool):
+    """Show mcli version and system information."""
+    from mcli.lib.ui.styling import info
+
+    message = get_version_info(verbose)
+    logger.info(message)
+    info(message)
+
+
 @self_app.command("hello")
 @click.argument("name", default="World")
 def hello(name: str):

{mcli_framework-7.12.0.dist-info → mcli_framework-7.12.2.dist-info}/METADATA

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: mcli-framework
-Version: 7.12.0
+Version: 7.12.2
 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-7.12.0.dist-info → mcli_framework-7.12.2.dist-info}/RECORD

@@ -6,8 +6,8 @@ mcli/app/__init__.py,sha256=D4RiKk2gOEXwanbe_jXyNSb5zdgNi47kahtskMnEwjY,489
 mcli/app/commands_cmd.py,sha256=RAwkNQIy2XFUQxn56M6rIKas7Qb9BhIFWB4iOj4zTFA,50255
 mcli/app/completion_helpers.py,sha256=e62C6w2N-XoD66GYYHgtvKKoD3kYMuIeBBGzVKbuL04,7497
 mcli/app/init_cmd.py,sha256=Pueg96Gi10VJB20xdEwfw_tDj4v5A5f_dEzKj_130UU,12451
-mcli/app/lock_cmd.py,sha256=dlMk94Lw7FtKDZC92H4GSOBKJva2sKWExIafIr1ftcQ,10146
-mcli/app/main.py,sha256=wFfvILpaP9JmGvRaLqHoLYIAiUGhvPBzh2L149rn9zo,20561
+mcli/app/lock_cmd.py,sha256=dmuX0peOCWt8sQtyakBVVQ6PTkkPnyiuW2x9QRACLXs,12458
+mcli/app/main.py,sha256=QX_NMi3TqsPbH0_1mCpA1we3jyGca0LpZAv4ljFf9lo,19210
 mcli/app/model_cmd.py,sha256=LQQD8FaebFoaJGK3u_kt19wZ3HJyo_ecwSMYyC2xIp8,2497
 mcli/app/store_cmd.py,sha256=Pr2S0G-TEC1UuGSIzyNnJBxPEc1lH42Cs8WCUsZHJ8I,14316
 mcli/app/model/__init__.py,sha256=47DEQpj8HBSa-_TImW-5JCeuQeRkm5NMpJWZG3hSuFU,0
@@ -204,7 +204,7 @@ mcli/self/completion_cmd.py,sha256=JXJ70p_yyfC8KlC5QifkQWLNeS2hgUFmNw8xdiix5O8,7
 mcli/self/logs_cmd.py,sha256=SCzZ4VZs6p42hksun_w4WN33xIZgmq7RjdWX8P2WcT4,15056
 mcli/self/migrate_cmd.py,sha256=DalSn-pvfez4KdYYaYlYi1hqqtZM6bgS--pNZzfVsE8,14665
 mcli/self/redis_cmd.py,sha256=Cl0LQ3Mqt27gLeb542_xw6bJBbIE-CBmWyMmaUTSk8c,9426
-mcli/self/self_cmd.py,sha256=m0GhY8u2o5Xo0ws9QaTO_UbdwloOtNQin0vrQEY0o44,38017
+mcli/self/self_cmd.py,sha256=F8_W82ZoMYa3amdsweQA-EbW4f9zOBDK8elvG0JadPo,39606
 mcli/self/store_cmd.py,sha256=O6arjRr4qWQKh1QyVWtzyXq5R7yZEBL87FSI59Db7IY,13320
 mcli/self/test_cmd.py,sha256=WjzgoH1WFa79wc8A7O6UMuJfookLfgciUNcCMbKHAQQ,21
 mcli/self/visual_cmd.py,sha256=jXighahHxeM9HANQ2Brk6nKFgi2ZuQBOBH7PE5xhebk,9428
@@ -271,9 +271,9 @@ mcli/workflow/sync/test_cmd.py,sha256=neVgs9zEnKSxlvzDpFkuCGucqnzjrShm2OvJtHibsl
 mcli/workflow/videos/__init__.py,sha256=aV3DEoO7qdKJY4odWKoQbOKDQq4ludTeCLnZcupOFIM,25
 mcli/workflow/wakatime/__init__.py,sha256=wKG8cVIHVtMPhNRFGFtX43bRnocHqOMMkFMkmW-M6pU,2626
 mcli/workflow/wakatime/wakatime.py,sha256=sEjsUKa3-XyE8Ni6sAb_D3GAY5jDcA30KknW9YTbLTA,142
-mcli_framework-7.12.0.dist-info/licenses/LICENSE,sha256=sahwAMfrJv2-V66HNPTp7A9UmMjxtyejwTZZoWQvEcI,1075
-mcli_framework-7.12.0.dist-info/METADATA,sha256=OohrTi2RR9BjdVJQlLsX7skMsOYiC5Gfd5EGQVCmmcY,18635
-mcli_framework-7.12.0.dist-info/WHEEL,sha256=_zCd3N1l69ArxyTb8rzEoP9TpbYXkqRFSNOD5OuxnTs,91
-mcli_framework-7.12.0.dist-info/entry_points.txt,sha256=dYrZbDIm-KUPsl1wfv600Kx_8sMy89phMkCihbDRgP8,261
-mcli_framework-7.12.0.dist-info/top_level.txt,sha256=_bnO8J2EUkliWivey_1le0UrnocFKmyVMQjbQ8iVXjc,5
-mcli_framework-7.12.0.dist-info/RECORD,,
+mcli_framework-7.12.2.dist-info/licenses/LICENSE,sha256=sahwAMfrJv2-V66HNPTp7A9UmMjxtyejwTZZoWQvEcI,1075
+mcli_framework-7.12.2.dist-info/METADATA,sha256=6KK20iC7JN8RWTAY4V78hvRBVPAYrbs4-kHKEY9mZOg,18635
+mcli_framework-7.12.2.dist-info/WHEEL,sha256=_zCd3N1l69ArxyTb8rzEoP9TpbYXkqRFSNOD5OuxnTs,91
+mcli_framework-7.12.2.dist-info/entry_points.txt,sha256=dYrZbDIm-KUPsl1wfv600Kx_8sMy89phMkCihbDRgP8,261
+mcli_framework-7.12.2.dist-info/top_level.txt,sha256=_bnO8J2EUkliWivey_1le0UrnocFKmyVMQjbQ8iVXjc,5
+mcli_framework-7.12.2.dist-info/RECORD,,