splent-cli 1.2.6__tar.gz → 1.2.8__tar.gz

{splent_cli-1.2.6/src/splent_cli.egg-info → splent_cli-1.2.8}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: splent_cli
-Version: 1.2.6
+Version: 1.2.8
 Summary: SPLENT-CLI is a CLI to be able to work on your development more easily.
 Author-email: DiversoLab <diversolab@us.es>
 Project-URL: Homepage, https://github.com/diverso-lab/splent_cli

{splent_cli-1.2.6 → splent_cli-1.2.8}/pyproject.toml

@@ -4,7 +4,7 @@ build-backend = "setuptools.build_meta"
 
 [project]
 name = "splent_cli"
-version = "1.2.6"
+version = "1.2.8"
 description = "SPLENT-CLI is a CLI to be able to work on your development more easily."
 readme = "README.md"
 requires-python = ">=3.13"

{splent_cli-1.2.6 → splent_cli-1.2.8}/src/splent_cli/commands/cache/cache_orphans.py

@@ -40,6 +40,7 @@ def _get_cache_entries(cache_root: Path) -> list:
 def _get_all_product_refs(workspace: Path) -> set:
     """Returns set of 'name' and 'name@version' (no namespace) from all products' pyproject.toml."""
     import tomllib
+
     refs = set()
     for product_dir in sorted(workspace.iterdir()):
         if not product_dir.is_dir() or product_dir.name.startswith("."):
@@ -57,7 +58,7 @@ def _get_all_product_refs(workspace: Path) -> set:
             for entry in feats:
                 ref = entry.split("/", 1)[1] if "/" in entry else entry
                 refs.add(ref)
-        except Exception:
+        except (OSError, tomllib.TOMLDecodeError):
             continue
     return refs
 

{splent_cli-1.2.6 → splent_cli-1.2.8}/src/splent_cli/commands/cache/cache_status.py

@@ -37,10 +37,12 @@ def _get_workspace_root_features(workspace: Path) -> dict:
         if not src.is_dir():
             continue
         for ns_dir in src.iterdir():
-            if (ns_dir.is_dir()
-                    and not ns_dir.name.startswith(("_", "."))
-                    and "." not in ns_dir.name
-                    and (ns_dir / entry.name).is_dir()):
+            if (
+                ns_dir.is_dir()
+                and not ns_dir.name.startswith(("_", "."))
+                and "." not in ns_dir.name
+                and (ns_dir / entry.name).is_dir()
+            ):
                 grouped[f"{ns_dir.name}/{entry.name}"].append("workspace")
                 break
     return grouped
@@ -75,11 +77,20 @@ def cache_status():
         for i, v in enumerate(sorted_versions):
             connector = "└──" if i == len(sorted_versions) - 1 else "├──"
             if v == "workspace":
-                click.echo(f"    {connector} " + click.style("editable (workspace root)", fg="magenta"))
+                click.echo(
+                    f"    {connector} "
+                    + click.style("editable (workspace root)", fg="magenta")
+                )
             elif v is None:
-                click.echo(f"    {connector} " + click.style("editable (cache)", fg="blue"))
+                click.echo(
+                    f"    {connector} " + click.style("editable (cache)", fg="blue")
+                )
             else:
-                click.echo(f"    {connector} " + click.style(f"@{v}", fg="green") + click.style(" (pinned)", fg="bright_black"))
+                click.echo(
+                    f"    {connector} "
+                    + click.style(f"@{v}", fg="green")
+                    + click.style(" (pinned)", fg="bright_black")
+                )
         click.echo()
 
 

{splent_cli-1.2.6 → splent_cli-1.2.8}/src/splent_cli/commands/check/check_deps.py

@@ -16,7 +16,6 @@ import re
 import click
 
 from splent_cli.services import context
-from splent_cli.utils.feature_utils import read_features_from_data
 from splent_framework.utils.pyproject_reader import PyprojectReader
 
 
@@ -24,6 +23,7 @@ from splent_framework.utils.pyproject_reader import PyprojectReader
 # UVL parser — extract dependency graph
 # ---------------------------------------------------------------------------
 
+
 def _parse_uvl_deps(uvl_path: str) -> tuple[dict[str, str], dict[str, set[str]]]:
     """Parse UVL file and return (package_map, allowed_deps).
 
@@ -82,7 +82,10 @@ def _parse_uvl_deps(uvl_path: str) -> tuple[dict[str, str], dict[str, set[str]]]
 # Source code scanner — find actual imports of other features
 # ---------------------------------------------------------------------------
 
-def _scan_feature_imports(feature_path: str, feature_name: str, all_packages: set[str]) -> set[str]:
+
+def _scan_feature_imports(
+    feature_path: str, feature_name: str, all_packages: set[str]
+) -> set[str]:
     """Scan all .py files in a feature and return set of other feature packages imported."""
     imported: set[str] = set()
 
@@ -105,11 +108,13 @@ def _scan_feature_imports(feature_path: str, feature_name: str, all_packages: se
             try:
                 with open(filepath, "r", encoding="utf-8") as fh:
                     content = fh.read()
-            except Exception:
+            except (OSError, PermissionError):
                 continue
 
             # Find imports: from splent_io.splent_feature_X... or import splent_io.splent_feature_X
-            for match in re.findall(r"(?:from|import)\s+splent_io\.(splent_feature_\w+)", content):
+            for match in re.findall(
+                r"(?:from|import)\s+splent_io\.(splent_feature_\w+)", content
+            ):
                 if match != feature_name and match in all_packages:
                     imported.add(match)
 
@@ -120,7 +125,10 @@ def _scan_feature_imports(feature_path: str, feature_name: str, all_packages: se
 # Feature path resolver
 # ---------------------------------------------------------------------------
 
-def _resolve_feature_paths(workspace: str, product: str, features: list[str]) -> dict[str, str]:
+
+def _resolve_feature_paths(
+    workspace: str, product: str, features: list[str]
+) -> dict[str, str]:
     """Return {package_name: feature_path} for each declared feature."""
     result = {}
     features_dir = os.path.join(workspace, product, "features")
@@ -144,6 +152,7 @@ def _resolve_feature_paths(workspace: str, product: str, features: list[str]) ->
 # Command
 # ---------------------------------------------------------------------------
 
+
 @click.command(
     "check:deps",
     short_help="Validate that feature imports respect UVL dependency constraints.",
@@ -202,17 +211,21 @@ def check_deps():
         fpath = feature_paths.get(pkg_name)
 
         if not fpath:
-            click.echo(click.style(f"  {short}", bold=True) +
-                       click.style("  (not in cache, skipped)", fg="bright_black"))
+            click.echo(
+                click.style(f"  {short}", bold=True)
+                + click.style("  (not in cache, skipped)", fg="bright_black")
+            )
             continue
 
         actual_imports = _scan_feature_imports(fpath, pkg_name, all_packages)
         allowed = allowed_deps.get(pkg_name, set())
 
         if not actual_imports:
-            click.echo(click.style("  [✔] ", fg="green") +
-                       click.style(f"{short}", bold=True) +
-                       " — no cross-feature imports")
+            click.echo(
+                click.style("  [✔] ", fg="green")
+                + click.style(f"{short}", bold=True)
+                + " — no cross-feature imports"
+            )
             ok += 1
             continue
 
@@ -221,24 +234,38 @@ def check_deps():
             imp_short = pkg_to_short.get(imp, imp)
 
             if imp in allowed:
-                click.echo(click.style("  [✔] ", fg="green") +
-                           click.style(f"{short}", bold=True) +
-                           f" imports {imp_short}" +
-                           click.style(f"  (allowed: {short} => {imp_short})", fg="bright_black"))
+                click.echo(
+                    click.style("  [✔] ", fg="green")
+                    + click.style(f"{short}", bold=True)
+                    + f" imports {imp_short}"
+                    + click.style(
+                        f"  (allowed: {short} => {imp_short})", fg="bright_black"
+                    )
+                )
                 ok += 1
             else:
                 # Check if the reverse is declared (inverted dependency)
                 reverse_allowed = allowed_deps.get(imp, set())
                 if pkg_name in reverse_allowed:
-                    click.echo(click.style("  [✖] ", fg="red") +
-                               click.style(f"{short}", bold=True) +
-                               f" imports {imp_short}" +
-                               click.style(f"  INVERTED — UVL says {imp_short} => {short}, not the reverse", fg="red"))
+                    click.echo(
+                        click.style("  [✖] ", fg="red")
+                        + click.style(f"{short}", bold=True)
+                        + f" imports {imp_short}"
+                        + click.style(
+                            f"  INVERTED — UVL says {imp_short} => {short}, not the reverse",
+                            fg="red",
+                        )
+                    )
                 else:
-                    click.echo(click.style("  [✖] ", fg="red") +
-                               click.style(f"{short}", bold=True) +
-                               f" imports {imp_short}" +
-                               click.style(f"  UNDECLARED — no UVL constraint between {short} and {imp_short}", fg="red"))
+                    click.echo(
+                        click.style("  [✖] ", fg="red")
+                        + click.style(f"{short}", bold=True)
+                        + f" imports {imp_short}"
+                        + click.style(
+                            f"  UNDECLARED — no UVL constraint between {short} and {imp_short}",
+                            fg="red",
+                        )
+                    )
                 violations += 1
                 has_violation = True
 
@@ -247,10 +274,16 @@ def check_deps():
 
     click.echo()
     if violations:
-        click.secho(f"  {violations} violation(s) found. Fix the code or update the UVL.", fg="red")
+        click.secho(
+            f"  {violations} violation(s) found. Fix the code or update the UVL.",
+            fg="red",
+        )
         raise SystemExit(1)
     else:
-        click.secho(f"  ✅ All cross-feature imports are consistent with UVL ({ok} checks passed).", fg="green")
+        click.secho(
+            f"  ✅ All cross-feature imports are consistent with UVL ({ok} checks passed).",
+            fg="green",
+        )
     click.echo()
 
 

{splent_cli-1.2.6 → splent_cli-1.2.8}/src/splent_cli/commands/check/check_env.py

@@ -1,6 +1,7 @@
 """
 check:env — Validate workspace environment variables and tool versions.
 """
+
 import os
 import sys
 
@@ -15,21 +16,26 @@ def _pkg_version(name: str) -> str | None:
         return None
 
 
-@click.command("check:env", short_help="Validate workspace environment and tool versions.")
+@click.command(
+    "check:env", short_help="Validate workspace environment and tool versions."
+)
 def check_env():
     """Check Python version, SPLENT env vars, CLI/framework compatibility."""
     ok = fail = warn = 0
 
     def _ok(msg):
-        nonlocal ok; ok += 1
+        nonlocal ok
+        ok += 1
         click.echo(click.style("  [✔] ", fg="green") + msg)
 
     def _fail(msg):
-        nonlocal fail; fail += 1
+        nonlocal fail
+        fail += 1
         click.echo(click.style("  [✖] ", fg="red") + msg)
 
     def _warn(msg):
-        nonlocal warn; warn += 1
+        nonlocal warn
+        warn += 1
         click.echo(click.style("  [⚠] ", fg="yellow") + msg)
 
     click.echo()
@@ -78,7 +84,9 @@ def check_env():
         else:
             _fail(f"CLI {cli_v} / Framework {fw_v} — major version mismatch")
     else:
-        _fail(f"CLI={'?' if not cli_v else cli_v} / Framework={'?' if not fw_v else fw_v}")
+        _fail(
+            f"CLI={'?' if not cli_v else cli_v} / Framework={'?' if not fw_v else fw_v}"
+        )
 
     # Credentials
     if os.getenv("GITHUB_TOKEN"):

{splent_cli-1.2.6 → splent_cli-1.2.8}/src/splent_cli/commands/check/check_features.py

@@ -1,6 +1,7 @@
 """
 check:features — Validate feature cache, symlinks, pip install, and git state.
 """
+
 import os
 import subprocess
 import importlib.metadata
@@ -20,7 +21,9 @@ def _pkg_installed(name: str) -> bool:
         return False
 
 
-@click.command("check:features", short_help="Validate feature cache, symlinks, and install state.")
+@click.command(
+    "check:features", short_help="Validate feature cache, symlinks, and install state."
+)
 def check_features():
     """Check every declared feature: cache entry, symlink, pip install, git state."""
     workspace = str(context.workspace())
@@ -31,15 +34,18 @@ def check_features():
     ok = fail = warn = 0
 
     def _ok(msg):
-        nonlocal ok; ok += 1
+        nonlocal ok
+        ok += 1
         click.echo(click.style("  [✔] ", fg="green") + msg)
 
     def _fail(msg):
-        nonlocal fail; fail += 1
+        nonlocal fail
+        fail += 1
         click.echo(click.style("  [✖] ", fg="red") + msg)
 
     def _warn(msg):
-        nonlocal warn; warn += 1
+        nonlocal warn
+        warn += 1
         click.echo(click.style("  [⚠] ", fg="yellow") + msg)
 
     click.echo()
@@ -102,7 +108,7 @@ def check_features():
             if os.path.exists(link_path):
                 target = os.readlink(link_path)
                 if os.path.isabs(target):
-                    _warn(f"Symlink uses absolute path (should be relative)")
+                    _warn("Symlink uses absolute path (should be relative)")
                 else:
                     _ok("Symlink OK (relative)")
             else:
@@ -123,7 +129,9 @@ def check_features():
             try:
                 r = subprocess.run(
                     ["git", "-C", feature_dir, "status", "--porcelain"],
-                    capture_output=True, text=True, timeout=5,
+                    capture_output=True,
+                    text=True,
+                    timeout=5,
                 )
                 if r.returncode != 0:
                     _warn("Not a git repo")
@@ -131,7 +139,7 @@ def check_features():
                     _warn("Uncommitted changes")
                 else:
                     _ok("Git clean")
-            except (subprocess.TimeoutExpired, FileNotFoundError):
+            except (subprocess.TimeoutExpired, FileNotFoundError, OSError):
                 _warn("Could not check git status")
 
         click.echo()

{splent_cli-1.2.6 → splent_cli-1.2.8}/src/splent_cli/commands/check/check_pyproject.py

@@ -1,6 +1,7 @@
 """
 check:pyproject — Validate the active product's pyproject.toml.
 """
+
 import os
 import re
 
@@ -13,6 +14,7 @@ from splent_cli.utils.feature_utils import read_features_from_data
 
 def _find_missing_pkgs(deps: list) -> list:
     import importlib.metadata
+
     missing = []
     for dep in deps:
         pkg = re.split(r"[=<>!~\[]", dep)[0].strip()
@@ -24,7 +26,9 @@ def _find_missing_pkgs(deps: list) -> list:
     return missing
 
 
-@click.command("check:pyproject", short_help="Validate pyproject.toml and dependencies.")
+@click.command(
+    "check:pyproject", short_help="Validate pyproject.toml and dependencies."
+)
 def check_pyproject():
     """Parse pyproject.toml, check dependencies, and validate feature declarations."""
     workspace = str(context.workspace())
@@ -34,15 +38,18 @@ def check_pyproject():
     ok = fail = warn = 0
 
     def _ok(msg):
-        nonlocal ok; ok += 1
+        nonlocal ok
+        ok += 1
         click.echo(click.style("  [✔] ", fg="green") + msg)
 
     def _fail(msg):
-        nonlocal fail; fail += 1
+        nonlocal fail
+        fail += 1
         click.echo(click.style("  [✖] ", fg="red") + msg)
 
     def _warn(msg):
-        nonlocal warn; warn += 1
+        nonlocal warn
+        warn += 1
         click.echo(click.style("  [⚠] ", fg="yellow") + msg)
 
     click.echo()

{splent_cli-1.2.6 → splent_cli-1.2.8}/src/splent_cli/commands/coverage.py

@@ -36,5 +36,8 @@ def coverage(module_name, html):
 
     try:
         subprocess.run(coverage_cmd, check=True)
-    except subprocess.CalledProcessError as e:
-        click.echo(click.style(f"Error running coverage: {e}", fg="red"))
+    except subprocess.CalledProcessError:
+        click.echo(
+            click.style("❌ Coverage run failed (tests may be failing).", fg="red")
+        )
+        raise SystemExit(1)

{splent_cli-1.2.6 → splent_cli-1.2.8}/src/splent_cli/commands/database/db_console.py

@@ -16,18 +16,23 @@ def db_console():
     mariadb_database = os.getenv("MARIADB_DATABASE")
 
     missing = [
-        name for name, val in {
+        name
+        for name, val in {
             "MARIADB_HOSTNAME": mariadb_hostname,
             "MARIADB_USER": mariadb_user,
             "MARIADB_PASSWORD": mariadb_password,
             "MARIADB_DATABASE": mariadb_database,
-        }.items() if not val
+        }.items()
+        if not val
     ]
     if missing:
         click.secho("❌ Missing required environment variables:", fg="red")
         for var in missing:
             click.secho(f"   - {var}", fg="red")
-        click.secho("\n   Make sure the product .env is loaded: splent product:env --merge --dev", fg="yellow")
+        click.secho(
+            "\n   Make sure the product .env is loaded: splent product:env --merge --dev",
+            fg="yellow",
+        )
         raise SystemExit(1)
 
     try:

{splent_cli-1.2.6 → splent_cli-1.2.8}/src/splent_cli/commands/database/db_dump.py

@@ -18,6 +18,20 @@ def db_dump(filename):
     mariadb_password = os.getenv("MARIADB_PASSWORD")
     mariadb_database = os.getenv("MARIADB_DATABASE")
 
+    missing = [
+        k
+        for k, v in {
+            "MARIADB_HOSTNAME": mariadb_hostname,
+            "MARIADB_USER": mariadb_user,
+            "MARIADB_PASSWORD": mariadb_password,
+            "MARIADB_DATABASE": mariadb_database,
+        }.items()
+        if not v
+    ]
+    if missing:
+        click.secho(f"❌ Missing env vars: {', '.join(missing)}", fg="red")
+        raise SystemExit(1)
+
     # Generate default filename if not provided
     if not filename:
         timestamp = datetime.now().strftime("%Y%m%d_%H%M%S")
@@ -28,23 +42,29 @@ def db_dump(filename):
             filename += ".sql"
 
     try:
+        env = {**os.environ, "MYSQL_PWD": mariadb_password or ""}
         with open(filename, "wb") as out:
             subprocess.run(
                 [
                     "mysqldump",
                     f"-h{mariadb_hostname}",
                     f"-u{mariadb_user}",
-                    f"-p{mariadb_password}",
                     mariadb_database,
                 ],
                 stdout=out,
                 check=True,
+                env=env,
             )
         click.echo(
             click.style(f"Database dump created successfully: {filename}", fg="green")
         )
     except subprocess.CalledProcessError as e:
         click.echo(click.style(f"Error creating database dump: {e}", fg="red"))
+        if os.path.exists(filename):
+            os.remove(filename)
+            click.echo(
+                click.style(f"Partial file removed: {filename}", fg="yellow")
+            )
 
 
 cli_command = db_dump

{splent_cli-1.2.6 → splent_cli-1.2.8}/src/splent_cli/commands/database/db_migrate.py

@@ -5,7 +5,11 @@ from flask import current_app
 from flask_migrate import migrate as alembic_migrate, upgrade as alembic_upgrade
 
 from splent_cli.utils.decorators import requires_db
-from splent_cli.utils.lifecycle import advance_state, resolve_feature_key_from_entry, require_editable
+from splent_cli.utils.lifecycle import (
+    advance_state,
+    resolve_feature_key_from_entry,
+    require_editable,
+)
 from splent_framework.managers.migration_manager import MigrationManager
 from splent_framework.utils.feature_utils import get_features_from_pyproject
 from splent_framework.utils.path_utils import PathUtils
@@ -25,16 +29,17 @@ def _is_empty_migration(path: str) -> bool:
         content = f.read()
     # Strip upgrade() and downgrade() bodies — if both are just pass, it's empty
     import re
+
     upgrades = re.findall(r"def upgrade\(\).*?(?=\ndef |\Z)", content, re.DOTALL)
     downgrades = re.findall(r"def downgrade\(\).*?(?=\ndef |\Z)", content, re.DOTALL)
     for body in upgrades + downgrades:
         # Remove comments, docstrings, and whitespace — if only 'pass' remains, it's empty
         lines = [
-            l.strip()
-            for l in body.splitlines()
-            if l.strip()
-            and not l.strip().startswith("#")
-            and not l.strip().startswith("def ")
+            line.strip()
+            for line in body.splitlines()
+            if line.strip()
+            and not line.strip().startswith("#")
+            and not line.strip().startswith("def ")
         ]
         if any(line != "pass" for line in lines):
             return False
@@ -97,14 +102,18 @@ def db_migrate(feature):
 
         # Suppress Alembic's verbose output during generation
         import logging
+
         alembic_logger = logging.getLogger("alembic")
         prev_level = alembic_logger.level
         alembic_logger.setLevel(logging.WARNING)
 
         try:
             alembic_migrate(directory=mdir, message=feat)
-        except Exception:
-            pass
+        except Exception as e:
+            if os.getenv("SPLENT_DEBUG"):
+                click.secho(
+                    f"  ⚠️  {feat}: migration generation skipped ({e})", fg="yellow"
+                )
         finally:
             alembic_logger.setLevel(prev_level)
 
@@ -114,14 +123,20 @@ def db_migrate(feature):
         if after > before:
             versions_dir = os.path.join(mdir, "versions")
             newest = max(
-                (os.path.join(versions_dir, f) for f in os.listdir(versions_dir) if f.endswith(".py")),
+                (
+                    os.path.join(versions_dir, f)
+                    for f in os.listdir(versions_dir)
+                    if f.endswith(".py")
+                ),
                 key=os.path.getmtime,
             )
             if _is_empty_migration(newest):
                 os.remove(newest)
                 click.echo(click.style(f"  ✔ {feat}: up to date", fg="green"))
             else:
-                click.echo(click.style(f"  📝 {feat}: new migration generated", fg="cyan"))
+                click.echo(
+                    click.style(f"  📝 {feat}: new migration generated", fg="cyan")
+                )
         else:
             click.echo(click.style(f"  ✔ {feat}: up to date", fg="green"))
 
@@ -139,8 +154,13 @@ def db_migrate(feature):
             if info:
                 key, ns, name, version = info
                 advance_state(
-                    product_path, product_name, key,
-                    to="migrated", namespace=ns, name=name, version=version,
+                    product_path,
+                    product_name,
+                    key,
+                    to="migrated",
+                    namespace=ns,
+                    name=name,
+                    version=version,
                 )
         except Exception as e:
             click.echo(click.style(f"  ❌ {feat}: {e}", fg="red"))

{splent_cli-1.2.6 → splent_cli-1.2.8}/src/splent_cli/commands/database/db_reset.py

@@ -92,7 +92,11 @@ def db_reset(yes):
     if not dirs:
         click.echo(click.style("⚠️  No feature migrations found.", fg="yellow"))
     else:
-        click.echo(click.style(f"⬆️  Applying migrations for {len(dirs)} features...", fg="cyan"))
+        click.echo(
+            click.style(
+                f"⬆️  Applying migrations for {len(dirs)} features...", fg="cyan"
+            )
+        )
         for feat, mdir in dirs.items():
             try:
                 alembic_upgrade(directory=mdir)
@@ -109,8 +113,13 @@ def db_reset(yes):
                 if info:
                     key, ns, name, version = info
                     advance_state(
-                        product_path, product_name, key,
-                        to="migrated", namespace=ns, name=name, version=version,
+                        product_path,
+                        product_name,
+                        key,
+                        to="migrated",
+                        namespace=ns,
+                        name=name,
+                        version=version,
                     )
             except Exception as e:
                 click.echo(click.style(f"  ❌ {feat}: {e}", fg="red"))

{splent_cli-1.2.6 → splent_cli-1.2.8}/src/splent_cli/commands/database/db_restore.py

@@ -57,11 +57,13 @@ def db_restore(filename, yes):
             raise SystemExit(0)
 
     try:
+        env = {**os.environ, "MYSQL_PWD": password or ""}
         with open(filename, "rb") as sql_file:
             subprocess.run(
-                ["mysql", f"-h{host}", f"-u{user}", f"-p{password}", database],
+                ["mysql", f"-h{host}", f"-u{user}", database],
                 stdin=sql_file,
                 check=True,
+                env=env,
             )
         click.secho(f"✅ Database restored from: {filename}", fg="green")
     except subprocess.CalledProcessError as e:

{splent_cli-1.2.6 → splent_cli-1.2.8}/src/splent_cli/commands/database/db_rollback.py

@@ -49,8 +49,13 @@ def db_rollback(feature, steps):
             if name == feature:
                 target = "installed" if revision is None else "migrated"
                 advance_state(
-                    product_path, product_name, key,
-                    to=target, namespace=ns, name=name, version=version,
+                    product_path,
+                    product_name,
+                    key,
+                    to=target,
+                    namespace=ns,
+                    name=name,
+                    version=version,
                 )
                 break
     except Exception as e: