sqlseed-cli 0.2.4__tar.gz → 0.2.5__tar.gz

{sqlseed_cli-0.2.4 → sqlseed_cli-0.2.5}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.5
 Name: sqlseed-cli
-Version: 0.2.4
+Version: 0.2.5
 Summary: Command-line interface for sqlseed test data generation, preview and configuration replay
 Project-URL: Documentation, https://sunbos.github.io/sqlseed/guide/
 Project-URL: Homepage, https://github.com/sunbos/sqlseed
@@ -18,7 +18,7 @@ Classifier: Programming Language :: Python :: 3.13
 Requires-Python: >=3.10
 Requires-Dist: click>=8.0
 Requires-Dist: rich>=13.0
-Requires-Dist: sqlseed<0.3,>=0.2.4.dev0
+Requires-Dist: sqlseed<0.3,>=0.2.5.dev0
 Description-Content-Type: text/markdown
 
 # sqlseed-cli
@@ -29,13 +29,15 @@ configuration. This package provides the `sqlseed` command.
 
 ## Installation
 
-For the 0.2.4 release, use a Python 3.10+ virtual environment:
+These instructions target version 0.2.5. Check [Releases](https://github.com/sunbos/sqlseed/releases)
+for publication status; use the source installation below to test an unpublished candidate.
+Use a Python 3.10+ virtual environment:
 
 ```bash
-python -m pip install "sqlseed-cli==0.2.4"
+python -m pip install "sqlseed-cli==0.2.5"
 ```
 
-Core 0.2.3 does not provide the URL API required by this CLI.
+Core 0.2.4 and older lack the shared diagnostic interfaces required by this version.
 For development, install Core and the required local plugins together from the
 repository root:
 
@@ -77,7 +79,7 @@ sqlseed replay <cache_dir>/snapshots/YYYY-MM-DD_HHMMSS_ffffff_users.yaml
 For PostgreSQL, install the Core `postgres` extra and use `--url`:
 
 ```bash
-python -m pip install "sqlseed[postgres]==0.2.4" "sqlseed-cli==0.2.4"
+python -m pip install "sqlseed[postgres]==0.2.5" "sqlseed-cli==0.2.5"
 sqlseed fill --url "postgresql+psycopg://user:pass@host/db" -t users -n 1000 --provider faker --no-ai
 sqlseed inspect --url "postgresql+psycopg://user:pass@host/db"
 ```
@@ -105,8 +107,11 @@ The optional AI plugin registers `ai-suggest`, `ai-analyze`, and `auto-heal` thr
 
 ## Requirements
 
+These metadata requirements apply to version 0.2.5 and its source candidates.
+Use local Core and plugins together when developing from source.
+
 - Python `>=3.10`
-- `sqlseed>=0.2.4.dev0,<0.3`
+- `sqlseed>=0.2.5.dev0,<0.3`
 - `click>=8.0`
 - `rich>=13.0`
 

{sqlseed_cli-0.2.4 → sqlseed_cli-0.2.5}/README.md

@@ -6,13 +6,15 @@ configuration. This package provides the `sqlseed` command.
 
 ## Installation
 
-For the 0.2.4 release, use a Python 3.10+ virtual environment:
+These instructions target version 0.2.5. Check [Releases](https://github.com/sunbos/sqlseed/releases)
+for publication status; use the source installation below to test an unpublished candidate.
+Use a Python 3.10+ virtual environment:
 
 ```bash
-python -m pip install "sqlseed-cli==0.2.4"
+python -m pip install "sqlseed-cli==0.2.5"
 ```
 
-Core 0.2.3 does not provide the URL API required by this CLI.
+Core 0.2.4 and older lack the shared diagnostic interfaces required by this version.
 For development, install Core and the required local plugins together from the
 repository root:
 
@@ -54,7 +56,7 @@ sqlseed replay <cache_dir>/snapshots/YYYY-MM-DD_HHMMSS_ffffff_users.yaml
 For PostgreSQL, install the Core `postgres` extra and use `--url`:
 
 ```bash
-python -m pip install "sqlseed[postgres]==0.2.4" "sqlseed-cli==0.2.4"
+python -m pip install "sqlseed[postgres]==0.2.5" "sqlseed-cli==0.2.5"
 sqlseed fill --url "postgresql+psycopg://user:pass@host/db" -t users -n 1000 --provider faker --no-ai
 sqlseed inspect --url "postgresql+psycopg://user:pass@host/db"
 ```
@@ -82,8 +84,11 @@ The optional AI plugin registers `ai-suggest`, `ai-analyze`, and `auto-heal` thr
 
 ## Requirements
 
+These metadata requirements apply to version 0.2.5 and its source candidates.
+Use local Core and plugins together when developing from source.
+
 - Python `>=3.10`
-- `sqlseed>=0.2.4.dev0,<0.3`
+- `sqlseed>=0.2.5.dev0,<0.3`
 - `click>=8.0`
 - `rich>=13.0`
 

{sqlseed_cli-0.2.4 → sqlseed_cli-0.2.5}/pyproject.toml

@@ -23,7 +23,7 @@ classifiers = [
     "Programming Language :: Python :: 3.13",
 ]
 dependencies = [
-    "sqlseed>=0.2.4.dev0,<0.3",
+    "sqlseed>=0.2.5.dev0,<0.3",
     "click>=8.0",
     "rich>=13.0",
 ]

{sqlseed_cli-0.2.4 → sqlseed_cli-0.2.5}/src/sqlseed_cli/main.py

@@ -9,7 +9,6 @@ AI-related commands (e.g. ai-suggest) are discovered via the
 from __future__ import annotations
 
 import os
-import re
 from dataclasses import dataclass
 from pathlib import Path
 from typing import Any
@@ -24,26 +23,25 @@ from sqlseed import fill_from_config
 from sqlseed import preview as api_preview
 from sqlseed._utils.logger import configure_logging, get_logger
 from sqlseed._utils.paths import get_cache_dir
+from sqlseed._utils.redaction import redact_url_credentials
 from sqlseed._version import __version__
 from sqlseed.config.loader import generate_template, load_config, save_config
 from sqlseed.config.models import GeneratorConfig, ProviderType, TableConfig
 from sqlseed.config.snapshot import SnapshotManager
 from sqlseed.core.orchestrator import DataOrchestrator
+from sqlseed.generators import ConfigurationError
 
 CONFLICTING_DATABASE_OPTIONS = "Cannot specify both positional db_path and --url. Use one or the other."
 
 logger = get_logger(__name__)
 
-# Redact the username and password in database URLs before displaying errors.
-_CREDENTIAL_PATTERN = re.compile(r"://[^:@/\s]+:[^@/\s]+@")
-
 
 def _redact_credentials(text: str) -> str:
     """Redact credentials in URLs to prevent leaking secrets in error messages.
 
     Replaces the ``user:pass@`` segment of a URL with ``***:***@``.
     """
-    return _CREDENTIAL_PATTERN.sub("://***:***@", text)
+    return redact_url_credentials(text)
 
 
 @click.group()
@@ -273,7 +271,10 @@ def fill(**kwargs: Any) -> None:
         config_path=config_path,
         transform_path=transform_path,
     )
-    _execute_fill(options)
+    try:
+        _execute_fill(options)
+    except ConfigurationError as exc:
+        raise click.UsageError(_redact_credentials(str(exc))) from None
 
 
 def _execute_config_fill(options: FillOptions, config_path: str) -> None:
@@ -315,7 +316,12 @@ def _execute_fill(options: FillOptions) -> None:
     if not (fill_db_path or fill_url):
         raise click.UsageError("db_path or --url is required when not using --config")
 
-    logger.debug("Starting fill", target=fill_url or fill_db_path, table=options.table, count=effective_count)
+    logger.debug(
+        "Starting fill",
+        target=redact_url_credentials(fill_url or fill_db_path or ""),
+        table=options.table,
+        count=effective_count,
+    )
 
     try:
         result = api_fill(
@@ -333,12 +339,12 @@ def _execute_fill(options: FillOptions) -> None:
             skip_ai=options.flags.no_ai,
         )
     except ValueError as exc:
-        logger.debug("Fill failed with ValueError", error=str(exc))
+        logger.debug("Fill failed with ValueError", error=_redact_credentials(str(exc)))
         raise click.UsageError(_redact_credentials(str(exc))) from exc
     click.echo(str(result))
     if result.errors:
         for err in result.errors:
-            click.echo(f"  Warning: {err}", err=True)
+            click.echo(f"  Warning: {_redact_credentials(err)}", err=True)
 
     if options.flags.snapshot:
         _save_snapshot_cmd(
@@ -407,8 +413,8 @@ def preview(
             locale=locale,
             seed=seed,
         )
-    except (ValueError, RuntimeError, OSError) as exc:
-        logger.debug("Preview failed", error=str(exc))
+    except (ValueError, RuntimeError, OSError, ConfigurationError) as exc:
+        logger.debug("Preview failed", error=_redact_credentials(str(exc)))
         raise click.UsageError(_redact_credentials(str(exc))) from exc
 
     if not rows:
@@ -506,7 +512,7 @@ def inspect(db_path: str | None, table: str | None, show_mapping: bool, db_url:
             for tbl in tables:
                 _inspect_table(orch, tbl, show_mapping, console)
     except (ValueError, RuntimeError, OSError) as exc:
-        logger.debug("Inspect failed", error=str(exc))
+        logger.debug("Inspect failed", error=_redact_credentials(str(exc)))
         raise click.UsageError(_redact_credentials(str(exc))) from exc
 
 
@@ -544,7 +550,7 @@ def init(config_path: str, db: str | None, db_url: str | None) -> None:
         config = generate_template(db_path=effective_db, url=db_url)
         save_config(config, config_path)
     except (ValueError, RuntimeError, OSError) as exc:
-        logger.debug("Init failed", error=str(exc))
+        logger.debug("Init failed", error=_redact_credentials(str(exc)))
         raise click.UsageError(_redact_credentials(str(exc))) from exc
     click.echo(f"Configuration template saved to: {config_path}")
 
@@ -593,20 +599,23 @@ def replay(snapshot_path: str) -> None:
             table_config = tc
             break
 
-    with DataOrchestrator.from_config(config) as orch:
-        result = orch.fill_table(
-            table_name=table_name,
-            count=count,
-            seed=seed,
-            batch_size=table_config.batch_size if table_config else 5000,
-            clear_before=table_config.clear_before if table_config else False,
-            column_configs=table_config.columns if table_config else None,
-            transform=table_config.transform if table_config else None,
-        )
+    try:
+        with DataOrchestrator.from_config(config) as orch:
+            result = orch.fill_table(
+                table_name=table_name,
+                count=count,
+                seed=seed,
+                batch_size=table_config.batch_size if table_config else 5000,
+                clear_before=table_config.clear_before if table_config else False,
+                column_configs=table_config.columns if table_config else None,
+                transform=table_config.transform if table_config else None,
+            )
+    except (ValueError, RuntimeError, OSError, ConfigurationError) as exc:
+        raise click.UsageError(_redact_credentials(str(exc))) from None
     click.echo(str(result))
     if result.errors:
         for err in result.errors:
-            click.echo(f"  Warning: {err}", err=True)
+            click.echo(f"  Warning: {_redact_credentials(err)}", err=True)
         raise SystemExit(1)
 
 

{sqlseed_cli-0.2.4 → sqlseed_cli-0.2.5}/tests/test_cli_ai_commands.py

@@ -822,14 +822,16 @@ class TestReportAIFailure:
         _, kwargs = mock_echo.call_args
         assert kwargs.get("err") is True
 
-    def test_message_includes_suggestions(self) -> None:
-        """The failure message includes model/timeout suggestions."""
+    def test_message_preserves_the_failure_without_unrelated_service_advice(self) -> None:
+        """The summary does not replace a specific error with stale advice."""
         with patch("click.echo") as mock_echo, pytest.raises(SystemExit):
             _report_ai_failure()
         message = str(mock_echo.call_args[0][0])
-        assert "deepseek" in message.lower()
-        assert "openai" in message.lower()
-        assert "timeout" in message.lower()
+        assert "No suggestions received" in message
+        assert "No output file was written" in message
+        assert "deepseek" not in message.lower()
+        assert "openai" not in message.lower()
+        assert "timeout" not in message.lower()
 
 
 # ---------------------------------------------------------------------------

{sqlseed_cli-0.2.4 → sqlseed_cli-0.2.5}/tests/test_config_execution_safety.py

@@ -2,6 +2,9 @@
 
 from __future__ import annotations
 
+import os
+import subprocess
+import sys
 from typing import TYPE_CHECKING
 
 import pytest
@@ -17,6 +20,74 @@ if TYPE_CHECKING:
     from pathlib import Path
 
 
+def test_cli_invalid_url_diagnostics_hide_userinfo_and_query_password() -> None:
+    target = "sqlite://audit-user:synthetic-userinfo@host/db?password=synthetic-query&timeout=3"
+    result = CliRunner().invoke(cli, ["preview", "--url", target, "--table", "items"])
+    assert result.exit_code != 0
+    assert "Invalid database URL" in result.output
+    assert "synthetic-userinfo" not in result.output
+    assert "synthetic-query" not in result.output
+    assert "host/db?password=***&timeout=3" in result.output
+
+
+def test_debug_fill_does_not_log_raw_connection_credentials() -> None:
+    result = subprocess.run(
+        [
+            sys.executable,
+            "-c",
+            "from sqlseed_cli.main import main; main()",
+            "fill",
+            "--url",
+            "sqlite://audit-user:synthetic-userinfo@host/db?password=first-secret second-secret&timeout=3",
+            "--table",
+            "items",
+            "--count",
+            "1",
+            "--provider",
+            "base",
+        ],
+        env={**os.environ, "SQLSEED_LOG_LEVEL": "DEBUG"},
+        capture_output=True,
+        text=True,
+        timeout=30,
+        check=False,
+    )
+    assert result.returncode == 2
+    assert "Starting fill" in result.stderr
+    assert "Invalid database URL" in result.stderr
+    assert all(
+        secret not in result.stderr + result.stdout
+        for secret in ("synthetic-userinfo", "first-secret", "second-secret")
+    )
+
+
+def test_invalid_date_config_is_an_actionable_cli_error_without_appending(tmp_path: Path) -> None:
+    database = tmp_path / "invalid-date.db"
+    with sqlite_connection(database) as conn:
+        conn.execute("CREATE TABLE events (created_at DATE NOT NULL)")
+        conn.execute("INSERT INTO events VALUES ('2020-01-01')")
+    document = {
+        "db_path": str(database),
+        "provider": "base",
+        "tables": [
+            {
+                "name": "events",
+                "count": 5,
+                "columns": [{"name": "created_at", "generator": "date", "params": {"end_date": "now"}}],
+            }
+        ],
+    }
+    config = tmp_path / "invalid.yaml"
+    config.write_text(yaml.safe_dump(document), encoding="utf-8")
+    result = CliRunner().invoke(cli, ["fill", "--config", str(config), "--no-ai"])
+    assert result.exit_code == 2, result.output
+    assert "created_at" in result.output
+    assert "YYYY-MM-DD" in result.output
+    assert "Traceback" not in result.output
+    with sqlite_connection(database) as conn:
+        assert conn.execute("SELECT * FROM events").fetchall() == [("2020-01-01",)]
+
+
 @pytest.mark.parametrize("target_kind", ["path", "url"])
 def test_config_rejects_explicit_target_without_changing_either_database(tmp_path: Path, target_kind: str) -> None:
     configured_db = tmp_path / "configured.db"