splent-cli 1.4.4__tar.gz → 1.5.0__tar.gz

{splent_cli-1.4.4/src/splent_cli.egg-info → splent_cli-1.5.0}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: splent_cli
-Version: 1.4.4
+Version: 1.5.0
 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
@@ -21,7 +21,6 @@ Requires-Dist: black==24.10.0; extra == "dev"
 Requires-Dist: coverage==7.6.10; extra == "dev"
 Requires-Dist: docker==7.1.0; extra == "dev"
 Requires-Dist: Faker==33.3.1; extra == "dev"
-Requires-Dist: flamapy==2.5.0; extra == "dev"
 Requires-Dist: flake8==7.1.1; extra == "dev"
 Requires-Dist: graphviz==0.20.3; extra == "dev"
 Requires-Dist: iniconfig==2.0.0; extra == "dev"
@@ -37,6 +36,8 @@ Requires-Dist: selenium-wire==5.1.0; extra == "dev"
 Requires-Dist: pip-tools==7.4.1; extra == "dev"
 Requires-Dist: tomli_w==1.2.0; extra == "dev"
 Requires-Dist: PyYAML==6.0.3; extra == "dev"
+Provides-Extra: uvl
+Requires-Dist: flamapy==2.5.0; extra == "uvl"
 Dynamic: license-file
 
 # SPLENT CLI

{splent_cli-1.4.4 → splent_cli-1.5.0}/pyproject.toml

@@ -4,7 +4,7 @@ build-backend = "setuptools.build_meta"
 
 [project]
 name = "splent_cli"
-version = "1.4.4"
+version = "1.5.0"
 description = "SPLENT-CLI is a CLI to be able to work on your development more easily."
 readme = "README.md"
 requires-python = ">=3.13"
@@ -28,7 +28,6 @@ dev = [
     "coverage==7.6.10",
     "docker==7.1.0",
     "Faker==33.3.1",
-    "flamapy==2.5.0",
     "flake8==7.1.1",
     "graphviz==0.20.3",
     "iniconfig==2.0.0",
@@ -45,6 +44,9 @@ dev = [
     "tomli_w==1.2.0",
     "PyYAML==6.0.3"
 ]
+uvl = [
+    "flamapy==2.5.0",
+]
 
 [project.scripts]
 splent = "splent_cli.__main__:main"

{splent_cli-1.4.4 → splent_cli-1.5.0}/src/splent_cli/commands/database/db_upgrade.py

@@ -59,15 +59,17 @@ def db_upgrade(feature):
     import logging
 
     logging.getLogger("alembic").setLevel(logging.WARNING)
+    logging.getLogger("alembic.runtime.migration").setLevel(logging.WARNING)
 
     for feat, mdir in dirs.items():
         try:
+            logging.getLogger("alembic.runtime.migration").setLevel(logging.WARNING)
             alembic_upgrade(directory=mdir)
             revision = MigrationManager.get_current_feature_revision(
                 feat, app.extensions["migrate"].db.engine
             )
             MigrationManager.update_feature_status(app, feat, revision)
-            click.echo(click.style(f"  ✅ {feat} → {revision or 'head'}", fg="green"))
+            click.echo(click.style(f"    {feat} -> {revision or 'head'}", fg="green"))
 
             # Advance lifecycle state to "migrated"
             info = entry_lookup.get(feat)

{splent_cli-1.4.4 → splent_cli-1.5.0}/src/splent_cli/commands/feature/feature_add.py

@@ -3,7 +3,7 @@ import tomllib
 import tomli_w
 import click
 from splent_cli.services import context
-from splent_cli.utils.feature_utils import normalize_namespace
+from splent_cli.utils.feature_utils import normalize_namespace, hot_reinstall
 from splent_cli.utils.manifest import feature_key, set_feature_state
 
 
@@ -39,11 +39,8 @@ def feature_add(full_name, env_scope):
       splent feature:add splent-io/splent_feature_admin --dev
     """
 
-    # --------------------------
-    # 0️⃣ Validate format
-    # --------------------------
     if "/" not in full_name:
-        click.echo("❌ Invalid format. Use: <namespace>/<feature_name>")
+        click.secho("  Invalid format. Use: <namespace>/<feature_name>", fg="red")
         raise SystemExit(1)
 
     namespace, feature_name = full_name.split("/", 1)
@@ -55,16 +52,17 @@ def feature_add(full_name, env_scope):
     # Editable features live at workspace root
     feature_dir = os.path.join(workspace, feature_name)
     if not os.path.exists(feature_dir):
-        click.echo(f"❌ Feature not found at workspace root: {feature_dir}")
-        click.echo(f"   Create it first with: splent feature:create {full_name}")
+        click.secho(f"  {feature_name} not found at workspace root.", fg="red")
+        click.echo(
+            click.style("  create it first: ", dim=True)
+            + f"splent feature:create {full_name}"
+        )
         raise SystemExit(1)
 
-    # --------------------------
-    # 1️⃣ Update pyproject.toml
-    # --------------------------
+    # ── Update pyproject.toml ─────────────────────────────────────────
     pyproject_path = os.path.join(workspace, product, "pyproject.toml")
     if not os.path.exists(pyproject_path):
-        click.echo("❌ pyproject.toml not found in product directory.")
+        click.secho("  pyproject.toml not found.", fg="red")
         raise SystemExit(1)
 
     with open(pyproject_path, "rb") as f:
@@ -82,19 +80,21 @@ def feature_add(full_name, env_scope):
         else (data.get("tool", {}).get("splent", {}).get(features_key, []))
     )
 
-    if full_name not in features:
-        features.append(full_name)
-        write_features_to_data(data, features, key=features_key)
-        with open(pyproject_path, "wb") as f:
-            tomli_w.dump(data, f)
-        scope_label = f" ({env_scope} only)" if env_scope else ""
-        click.echo(f"🧩 Added '{full_name}' to {features_key}{scope_label}.")
-    else:
-        click.echo(f"ℹ️ Feature '{full_name}' already present in {features_key}.")
-
-    # --------------------------
-    # 2️⃣ Create symlink
-    # --------------------------
+    short = feature_name.replace("splent_feature_", "")
+
+    if full_name in features:
+        click.echo(f"  {short} already in {features_key}.")
+        return
+
+    features.append(full_name)
+    write_features_to_data(data, features, key=features_key)
+    with open(pyproject_path, "wb") as f:
+        tomli_w.dump(data, f)
+
+    scope_label = f" ({env_scope} only)" if env_scope else ""
+    click.echo(f"  {short} added to {features_key}{scope_label}")
+
+    # ── Create symlink ────────────────────────────────────────────────
     product_features_dir = os.path.join(workspace, product, "features", org_safe)
     os.makedirs(product_features_dir, exist_ok=True)
 
@@ -105,11 +105,8 @@ def feature_add(full_name, env_scope):
     except FileExistsError:
         os.unlink(link_path)
         os.symlink(rel_target, link_path)
-    click.echo(f"🔗 Linked {link_path} → {rel_target}")
 
-    # --------------------------
-    # 3️⃣ Update manifest
-    # --------------------------
+    # ── Update manifest ───────────────────────────────────────────────
     product_path = os.path.join(workspace, product)
     key = feature_key(namespace, feature_name)
     set_feature_state(
@@ -123,4 +120,7 @@ def feature_add(full_name, env_scope):
         mode="editable",
     )
 
-    click.echo(f"✅ Feature '{full_name}' added successfully to product '{product}'.")
+    # ── Hot reinstall in web container ────────────────────────────────
+    hot_reinstall(product_path, f"/workspace/{feature_name}", feature_name)
+
+    click.secho("  done.", fg="green")

{splent_cli-1.4.4 → splent_cli-1.5.0}/src/splent_cli/commands/feature/feature_attach.py

@@ -3,6 +3,7 @@ import tomllib
 import tomli_w
 import click
 from splent_cli.services import context, compose
+from splent_cli.utils.feature_utils import hot_reinstall
 from splent_cli.utils.manifest import feature_key, set_feature_state
 
 
@@ -32,7 +33,7 @@ def feature_attach(feature_identifier, version, env_scope):
       If not, run: splent feature:clone <namespace>/<feature>@<version>
     - Updates pyproject.toml referencing feature@version.
     - Creates/updates the versioned symlink in features/<namespace>/.
-    - Updates the manifest state to 'declared'.
+    - Reinstalls the feature in the web container for hot reload.
     """
     product = context.require_app()
     ws = context.workspace()
@@ -47,22 +48,23 @@ def feature_attach(feature_identifier, version, env_scope):
     pyproject_path = os.path.join(product_path, "pyproject.toml")
 
     if not os.path.exists(pyproject_path):
-        click.echo("❌ pyproject.toml not found in product.")
+        click.secho("  pyproject.toml not found.", fg="red")
         raise SystemExit(1)
 
-    # --- 1️⃣ Verify feature exists in cache ---------------------------------
+    # ── Verify feature exists in cache ────────────────────────────────
     versioned_dir = os.path.join(cache_base, f"{feature_name}@{version}")
 
     if not os.path.exists(versioned_dir):
+        click.secho(f"  {feature_name}@{version} not found in cache.", fg="red")
         click.echo(
-            f"❌ Feature '{namespace}/{feature_name}@{version}' not found in cache.\n"
-            f"   Run first: splent feature:clone {namespace}/{feature_name}@{version}"
+            click.style("  clone it first: ", dim=True)
+            + f"splent feature:clone {namespace}/{feature_name}@{version}"
         )
         raise SystemExit(1)
 
-    click.echo(f"✅ Cache found → {versioned_dir}")
+    short = feature_name.replace("splent_feature_", "")
 
-    # --- 2️⃣ Update pyproject.toml ------------------------------------------
+    # ── Update pyproject.toml ─────────────────────────────────────────
     full_name = f"{namespace}/{feature_name}@{version}"
     bare_name = f"{namespace}/{feature_name}"
 
@@ -82,9 +84,9 @@ def feature_attach(feature_identifier, version, env_scope):
     )
 
     if full_name in features:
-        click.echo(f"ℹ️  Feature '{full_name}' already present in {features_key}.")
+        click.echo(f"  {short}@{version} already in {features_key}.")
     else:
-        # Replace bare entry (added by uvl:sync) or old versioned entry if present
+        # Replace bare entry or old versioned entry if present
         features = [
             f for f in features if f != bare_name and not f.startswith(f"{bare_name}@")
         ]
@@ -93,9 +95,9 @@ def feature_attach(feature_identifier, version, env_scope):
         with open(pyproject_path, "wb") as f:
             tomli_w.dump(data, f)
         scope_label = f" ({env_scope} only)" if env_scope else ""
-        click.echo(f"🧩 Updated {features_key} → {full_name}{scope_label}")
+        click.echo(f"  {short}@{version} attached{scope_label}")
 
-    # --- 3️⃣ Create/update symlink ------------------------------------------
+    # ── Create/update symlink ─────────────────────────────────────────
     product_features_dir = os.path.join(product_path, "features", namespace_fs)
     os.makedirs(product_features_dir, exist_ok=True)
 
@@ -105,9 +107,7 @@ def feature_attach(feature_identifier, version, env_scope):
     rel_target = os.path.relpath(versioned_dir, product_features_dir)
     os.symlink(rel_target, new_link)
 
-    click.echo(f"🔗 Linked {new_link} → {rel_target}")
-
-    # --- 4️⃣ Update manifest ------------------------------------------------
+    # ── Update manifest ───────────────────────────────────────────────
     key = feature_key(namespace_fs, feature_name, version)
     set_feature_state(
         product_path,
@@ -120,4 +120,11 @@ def feature_attach(feature_identifier, version, env_scope):
         mode="pinned",
     )
 
-    click.echo("🎯 Feature successfully attached.")
+    # ── Hot reinstall in web container ────────────────────────────────
+    # Symlink resolves to cache path — install from there
+    install_path = (
+        f"/workspace/{product}/features/{namespace_fs}/{feature_name}@{version}"
+    )
+    hot_reinstall(product_path, install_path, feature_name)
+
+    click.secho("  done.", fg="green")

{splent_cli-1.4.4 → splent_cli-1.5.0}/src/splent_cli/commands/feature/feature_detach.py

@@ -2,6 +2,7 @@ import os
 import re
 import click
 from splent_cli.services import context, compose
+from splent_cli.utils.feature_utils import hot_uninstall
 from splent_cli.utils.manifest import (
     feature_key,
     remove_feature,
@@ -39,39 +40,37 @@ def feature_detach(feature_identifier, version, force):
 
     product_path = str(ws / product)
     pyproject_path = os.path.join(product_path, "pyproject.toml")
+    short = feature_name.replace("splent_feature_", "")
 
     if not os.path.exists(pyproject_path):
-        click.echo("❌ pyproject.toml not found in product.")
+        click.secho("  pyproject.toml not found.", fg="red")
         raise SystemExit(1)
 
     if not force:
-        # --- Guard: dependency check ----------------------------------------
+        # Guard: dependency check
         dependents = get_dependents(product_path, feature_name)
         if dependents:
             click.secho(
-                f"❌ Cannot detach '{feature_name}': the following installed features depend on it:\n"
-                + "".join(f"   • {d}\n" for d in dependents)
-                + "   Remove those features first, or use --force to bypass.",
+                f"  Cannot detach '{short}': the following features depend on it:\n"
+                + "".join(f"    - {d}\n" for d in dependents)
+                + "  Remove those first, or use --force.",
                 fg="red",
             )
             raise SystemExit(1)
 
-        # --- Guard: migration state -----------------------------------------
+        # Guard: migration state
         key = feature_key(namespace_fs, feature_name, version)
         state = get_feature_state(product_path, key)
         if state in ("migrated", "active"):
             click.secho(
-                f"❌ Feature '{feature_name}' has migrations applied (state: {state}).\n"
-                f"   Roll them back first:\n"
-                f"   splent db:rollback {feature_name} --steps 999\n"
-                f"   Or use --force to skip this check.",
+                f"  {short} has migrations applied (state: {state}).\n"
+                f"  Roll them back first: splent db:rollback {feature_name} --steps 999\n"
+                f"  Or use --force.",
                 fg="red",
             )
             raise SystemExit(1)
 
-    # --- 1️⃣ Remove versioned reference from pyproject ----------------------
-    click.echo(f"🧹 Removing {feature_name}@{version} from pyproject.toml...")
-
+    # ── Remove versioned reference from pyproject ─────────────────────
     with open(pyproject_path, "r", encoding="utf-8") as f:
         content = f.read()
 
@@ -81,22 +80,20 @@ def feature_detach(feature_identifier, version, force):
     with open(pyproject_path, "w", encoding="utf-8") as f:
         f.write(new_content)
 
-    click.echo("🧩 pyproject.toml cleaned.")
+    click.echo(f"  {short}@{version} removed from pyproject.toml")
 
-    # --- 2️⃣ Remove symlink --------------------------------------------------
+    # ── Remove symlink ────────────────────────────────────────────────
     product_features_dir = os.path.join(product_path, "features", namespace_fs)
     link_path = os.path.join(product_features_dir, f"{feature_name}@{version}")
 
     if os.path.islink(link_path):
         os.unlink(link_path)
-        click.echo(f"🔗 Removed symlink: {link_path}")
-    else:
-        click.echo(
-            f"⚠️ No symlink found for {feature_name}@{version} in {namespace_fs}/"
-        )
 
-    # --- 3️⃣ Update manifest ------------------------------------------------
+    # ── Update manifest ───────────────────────────────────────────────
     key = feature_key(namespace_fs, feature_name, version)
     remove_feature(str(ws / product), product, key)
 
-    click.echo("🎯 Feature successfully detached.")
+    # ── Hot uninstall from web container ──────────────────────────────
+    hot_uninstall(product_path, feature_name)
+
+    click.secho("  done.", fg="green")

{splent_cli-1.4.4 → splent_cli-1.5.0}/src/splent_cli/commands/feature/feature_edit.py

@@ -8,62 +8,6 @@ from splent_cli.utils.feature_utils import read_features_from_data
 from splent_cli.utils.cache_utils import make_feature_writable
 
 
-# =====================================================================
-# GITHUB WRITE ACCESS CHECK
-# =====================================================================
-def _has_write_access(ns_git: str, name: str) -> tuple[bool, str]:
-    """Check if the authenticated user has push access to ns_git/name.
-
-    Returns (has_access, reason_if_denied).
-    """
-    token = os.getenv("GITHUB_TOKEN")
-    github_user = os.getenv("GITHUB_USER", "(not set)")
-    if not token:
-        return False, (
-            f"GitHub user (env): {github_user}\n"
-            f"     Repo owner:      {ns_git}\n"
-            f"     GITHUB_TOKEN not set"
-        )
-
-    headers = {
-        "Accept": "application/vnd.github+json",
-        "Authorization": f"token {token}",
-        "User-Agent": "splent-cli",
-    }
-    try:
-        resp = requests.get(
-            f"https://api.github.com/repos/{ns_git}/{name}",
-            headers=headers,
-            timeout=5,
-        )
-        if resp.status_code == 404:
-            return False, (
-                f"GitHub user (env): {github_user}\n"
-                f"     Repo owner:      {ns_git}\n"
-                f"     Repo {ns_git}/{name} not found (404)"
-            )
-        if resp.status_code == 200:
-            perms = resp.json().get("permissions", {})
-            if perms.get("push", False):
-                return True, ""
-            return False, (
-                f"GitHub user (env): {github_user}\n"
-                f"     Repo owner:      {ns_git}\n"
-                f"     Push access: {perms.get('push', False)}"
-            )
-        return False, (
-            f"GitHub user (env): {github_user}\n"
-            f"     Repo owner:      {ns_git}\n"
-            f"     GitHub API returned {resp.status_code}"
-        )
-    except requests.RequestException as e:
-        return False, (
-            f"GitHub user (env): {github_user}\n"
-            f"     Repo owner:      {ns_git}\n"
-            f"     API error: {e}"
-        )
-
-
 # =====================================================================
 # CACHE PATH RESOLVER
 # =====================================================================
@@ -140,39 +84,12 @@ def replace_pyproject_reference(pyproject_path: str, name: str, version: str):
 
 
 # =====================================================================
-# HOT REINSTALL — pip install + Flask reload in the web container
+# HOT REINSTALL — delegated to shared utility
 # =====================================================================
 def _hot_reinstall(workspace: str, product_path: str, editable_path: str, name: str):
-    """Reinstall the feature via pip in the product's web container and trigger Flask reload."""
-    product = os.path.basename(product_path)
-    env = os.getenv("SPLENT_ENV", "dev")
-    docker_dir = os.path.join(product_path, "docker")
+    from splent_cli.utils.feature_utils import hot_reinstall
 
-    compose_file = compose.resolve_file(product_path, env)
-    if not compose_file:
-        return  # no docker-compose file — nothing to do
-
-    pname = compose.project_name(product, env)
-    container_id = compose.find_main_container(pname, compose_file, docker_dir)
-    if not container_id:
-        return  # container not running — nothing to do
-
-    # 1. pip install -e from the new path
-    click.echo(f"     🔄 Reinstalling {name} in web container...")
-    pip_cmd = (
-        f"pip install --no-cache-dir --root-user-action=ignore -q -e /workspace/{name}"
-    )
-    subprocess.run(
-        ["docker", "exec", container_id, "bash", "-c", pip_cmd],
-        capture_output=True,
-    )
-
-    # 2. Touch the app's __init__.py to trigger watchmedo auto-restart
-    init_py = f"/workspace/{product}/src/{product}/__init__.py"
-    subprocess.run(
-        ["docker", "exec", container_id, "bash", "-c", f"touch {init_py}"],
-        capture_output=True,
-    )
+    hot_reinstall(product_path, f"/workspace/{name}", name)
 
 
 # =====================================================================
@@ -205,7 +122,7 @@ def _compile_assets(workspace: str, product_path: str, name: str):
     if not container_id:
         return
 
-    click.echo("     📦 Compiling assets...")
+    click.echo(click.style("    compiling assets...", dim=True))
     product_root = f"/workspace/{product}"
     cmd = f"cd {shlex.quote(product_root)} && npx webpack --config {shlex.quote(webpack_file)} --mode development"
     result = subprocess.run(
@@ -213,10 +130,40 @@ def _compile_assets(workspace: str, product_path: str, name: str):
         capture_output=True,
         text=True,
     )
-    if result.returncode == 0:
-        click.echo("     ✔  Assets compiled.")
-    else:
-        click.secho("     ⚠  Asset compilation failed.", fg="yellow")
+    if result.returncode != 0:
+        click.secho("    asset compilation failed.", fg="yellow")
+
+
+# =====================================================================
+# WRITE ACCESS WARNING
+# =====================================================================
+def _warn_no_push_access(ns_git: str, name: str):
+    """Print a warning if the user cannot push to the repo. Non-blocking."""
+    token = os.getenv("GITHUB_TOKEN")
+    if not token:
+        return  # no token = can't check, don't nag
+
+    headers = {
+        "Accept": "application/vnd.github+json",
+        "Authorization": f"token {token}",
+        "User-Agent": "splent-cli",
+    }
+    try:
+        resp = requests.get(
+            f"https://api.github.com/repos/{ns_git}/{name}",
+            headers=headers,
+            timeout=5,
+        )
+        if resp.status_code == 200:
+            perms = resp.json().get("permissions", {})
+            if not perms.get("push", False):
+                click.secho(
+                    f"    note: no push access to {ns_git}/{name}\n"
+                    f"    use 'splent feature:fork' to work on your own copy",
+                    fg="yellow",
+                )
+    except requests.RequestException:
+        pass  # network error — don't block
 
 
 # =====================================================================
@@ -239,38 +186,26 @@ def _edit_one(
         name, version = rest, None
 
     if not version:
-        click.echo(f"  ℹ️  {match} — already editable, skipping.")
+        click.echo(click.style(f"  {name}", dim=True) + " already editable")
         return True
 
-    # Guard: require write access to the GitHub repo
-    if not force:
-        has_access, reason = _has_write_access(ns_git, name)
-        if not has_access:
-            click.secho(
-                f"  ❌ No write access to {ns_git}/{name}.\n"
-                f"     {reason}\n"
-                f"     To work on your own copy, use: splent feature:fork {ns_git}/{name}\n"
-                f"     Or use --force to bypass this check.",
-                fg="red",
-            )
-            return False
-
-    click.echo(f"  🧩 {ns_git}/{name}@{version}")
+    short = name.replace("splent_feature_", "")
+    click.echo(f"  {short} ({version}) -> editable")
 
     versioned_path, editable_path = get_feature_paths(workspace, ns_fs, name, version)
 
     if not os.path.exists(versioned_path):
-        click.secho(f"  ❌ Versioned cache not found: {versioned_path}", fg="red")
+        click.secho(f"    cached version not found: {versioned_path}", fg="red")
         return False
 
     if not os.path.exists(editable_path):
         import shutil
 
-        click.echo(f"     📦 Creating editable copy → {editable_path}")
+        click.echo(click.style("    copying to workspace root...", dim=True))
         result = subprocess.run(["cp", "-r", versioned_path, editable_path])
         if result.returncode != 0:
             shutil.rmtree(editable_path, ignore_errors=True)
-            click.secho("     ❌ Failed to copy feature to editable path.", fg="red")
+            click.secho("    failed to copy feature.", fg="red")
             return False
 
     # Always ensure editable copy is writable (cache files are read-only)
@@ -302,7 +237,10 @@ def _edit_one(
     # Compile webpack assets via the product's web container
     _compile_assets(workspace, product_path, name)
 
-    click.secho("     ✔  ready for editing.", fg="green")
+    # Non-blocking warning if user can't push
+    _warn_no_push_access(ns_git, name)
+
+    click.secho("    ready.", fg="green")
     return True
 
 
@@ -329,7 +267,7 @@ def feature_edit(feature_name, edit_all, force):
     pyproject_path = os.path.join(product_path, "pyproject.toml")
 
     if not os.path.exists(pyproject_path):
-        click.echo("❌ pyproject.toml not found.")
+        click.secho("  pyproject.toml not found.", fg="red")
         raise SystemExit(1)
 
     if not feature_name and not edit_all:
@@ -353,7 +291,7 @@ def feature_edit(feature_name, edit_all, force):
             None,
         )
         if not match:
-            click.echo(f"❌ Feature {feature_name} not found in pyproject.")
+            click.secho(f"  {feature_name} not found in pyproject.", fg="red")
             raise SystemExit(1)
 
         click.echo()
@@ -367,17 +305,20 @@ def feature_edit(feature_name, edit_all, force):
 
     click.echo()
     if already_editable:
-        click.secho(
-            f"  ℹ️  {len(already_editable)} feature(s) already editable — skipping.",
-            fg="bright_black",
+        click.echo(
+            click.style(
+                f"  {len(already_editable)} feature(s) already editable, skipping.",
+                dim=True,
+            )
         )
 
     if not versioned:
-        click.secho("  ✅ All features are already editable.", fg="green")
+        click.secho("  All features are already editable.", fg="green")
         click.echo()
         return
 
-    click.secho(f"  Converting {len(versioned)} feature(s) to editable:\n", fg="cyan")
+    click.echo(f"  Converting {len(versioned)} feature(s) to editable:")
+    click.echo()
     ok = 0
     for match in versioned:
         if _edit_one(workspace, product_path, pyproject_path, match, force=force):
@@ -385,7 +326,7 @@ def feature_edit(feature_name, edit_all, force):
         click.echo()
 
     click.secho(
-        f"  {ok}/{len(versioned)} feature(s) converted.",
+        f"  {ok}/{len(versioned)} converted.",
         fg="green" if ok == len(versioned) else "yellow",
     )
     click.echo()

{splent_cli-1.4.4 → splent_cli-1.5.0}/src/splent_cli/commands/feature/feature_release.py

@@ -405,6 +405,12 @@ def feature_release(feature_ref, version, attach):
         post_pypi_hook=_post_pypi,
     )
 
+    if not attach:
+        product = context.require_app()
+        attach = click.confirm(
+            f"\n  Attach {feature_name}@{tag} to {product}?", default=True
+        )
+
     if attach:
         click.echo("  attach   linking to product...")
         ctx = click.get_current_context()