specsmith 0.15.3.dev572__tar.gz → 0.15.3.dev573__tar.gz

{specsmith-0.15.3.dev572/src/specsmith.egg-info → specsmith-0.15.3.dev573}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: specsmith
-Version: 0.15.3.dev572
+Version: 0.15.3.dev573
 Summary: AEE governance toolkit for AI-assisted development — session preflight gates, multi-agent dispatch, requirements↔test traceability, ESDB persistence, MCP server, and skills for Warp, Cursor, Claude Code, Copilot, Windsurf, and Aider.
 Author: Layer1Labs Silicon, Inc.
 License-Expression: MIT
@@ -40,7 +40,7 @@ Requires-Dist: pre-commit>=3.0; extra == "dev"
 Requires-Dist: types-pyyaml>=6.0; extra == "dev"
 Requires-Dist: cryptography>=42.0; extra == "dev"
 Provides-Extra: esdb
-Requires-Dist: chronomemory>=0.1.7; extra == "esdb"
+Requires-Dist: chronomemory>=0.1.8; extra == "esdb"
 Requires-Dist: cryptography>=42.0; extra == "esdb"
 Provides-Extra: docs
 Requires-Dist: mkdocs>=1.6; extra == "docs"

{specsmith-0.15.3.dev572 → specsmith-0.15.3.dev573}/pyproject.toml

@@ -4,7 +4,7 @@ build-backend = "setuptools.build_meta"
 
 [project]
 name = "specsmith"
-version = "0.15.3.dev572"
+version = "0.15.3.dev573"
 description = "AEE governance toolkit for AI-assisted development — session preflight gates, multi-agent dispatch, requirements↔test traceability, ESDB persistence, MCP server, and skills for Warp, Cursor, Claude Code, Copilot, Windsurf, and Aider."
 readme = "README.md"
 license = "MIT"
@@ -60,7 +60,7 @@ dev = [
 # Install: pip install specsmith[esdb]
 # Activate: specsmith esdb enable --key-file /path/to/your.esdb.key
 esdb = [
-    "chronomemory>=0.1.7",   # proprietary commercial — see chronomemory LICENSE
+    "chronomemory>=0.1.8",   # proprietary commercial — see chronomemory LICENSE
     "cryptography>=42.0",
 ]
 docs = [

{specsmith-0.15.3.dev572 → specsmith-0.15.3.dev573}/src/specsmith/cli.py

@@ -445,8 +445,17 @@ def _interactive_config(no_git: bool) -> ProjectConfig:
 def audit(fix: bool, project_dir: str) -> None:
     """Run drift and health checks (Section 23 + 26)."""
     from specsmith.auditor import run_audit
+    from specsmith.sync import auto_migrate_if_needed
 
     root = Path(project_dir).resolve()
+    auto_counts = auto_migrate_if_needed(root)
+    if auto_counts:
+        console.print(
+            "  [cyan]⟳[/cyan] ESDB auto-migrate: "
+            f"{auto_counts.get('requirements', 0)} requirements + "
+            f"{auto_counts.get('testcases', 0)} testcases "
+            f"({auto_counts.get('skipped', 0)} skipped)"
+        )
     console.print(f"[bold]Auditing[/bold] {root}\n")
     report = run_audit(root)
 
@@ -1489,14 +1498,20 @@ def diff(project_dir: str, html_output: str | None) -> None:
     default=False,
     help="Run the new-user onboarding checklist (REQ-127).",
 )
-def doctor(project_dir: str, onboarding: bool) -> None:
+@click.option(
+    "--json",
+    "as_json",
+    is_flag=True,
+    default=False,
+    help="Emit machine-readable health report JSON.",
+)
+def doctor(project_dir: str, onboarding: bool, as_json: bool) -> None:
     """Check if verification tools are installed locally.
 
     With --onboarding, walks through a 6-step checklist for new users that
     confirms scaffold.yml, governance files, agent provider setup, optional
     Nexus broker availability, and prints next-step commands (REQ-127).
     """
-    from specsmith.doctor import run_doctor
 
     root = Path(project_dir).resolve()
 
@@ -1572,28 +1587,108 @@ def doctor(project_dir: str, onboarding: bool) -> None:
             )
         return
 
-    report = run_doctor(root)
+    import importlib.metadata
+    import json as _json
+    import shutil
+    import subprocess
+    import sys
+
+    from specsmith.auditor import run_audit
+    from specsmith.esdb import ESDB_BACKEND, open_default_store
+    from specsmith.phase import read_phase
 
-    if not report.checks:
-        console.print("[yellow]No scaffold.yml found — cannot determine tools.[/yellow]")
-        return
+    def _tool_version(cmd: str) -> tuple[bool, str]:
+        exe = shutil.which(cmd)
+        if not exe:
+            return False, "not found"
+        try:
+            proc = subprocess.run(
+                [exe, "--version"],
+                capture_output=True,
+                text=True,
+                timeout=5,
+                check=False,
+            )
+            out = (proc.stdout or proc.stderr or "").strip().splitlines()
+            if out:
+                return True, out[0]
+            return True, "installed"
+        except (OSError, subprocess.TimeoutExpired):
+            return True, "installed"
+
+    checks: list[dict[str, str]] = []
+
+    def _add(name: str, passed: bool, detail: str, warn: bool = False) -> None:
+        status = "warn" if warn else ("pass" if passed else "fail")
+        checks.append({"name": name, "status": status, "detail": detail})
+
+    py_ok = sys.version_info >= (3, 10)
+    _add(
+        "python",
+        py_ok,
+        f"{sys.version_info.major}.{sys.version_info.minor}.{sys.version_info.micro}",
+    )
 
-    console.print(f"[bold]Doctor[/bold] — checking {len(report.checks)} tools\n")
-    for check in report.checks:
-        if check.installed:
-            ver = f" ({check.version})" if check.version else ""
-            console.print(f"  [green]✓[/green] {check.category}: {check.name}{ver}")
-        else:
-            console.print(f"  [red]✗[/red] {check.category}: {check.name} — not found")
+    git_ok, git_detail = _tool_version("git")
+    _add("git", git_ok, git_detail)
 
-    console.print()
-    if report.missing_count == 0:
-        console.print(f"[bold green]All {report.installed_count} tools available.[/bold green]")
+    _add("specsmith", True, __version__)
+
+    try:
+        chrono_ver = importlib.metadata.version("chronomemory")
+        _add("chronomemory", True, chrono_ver)
+    except importlib.metadata.PackageNotFoundError:
+        _add("chronomemory", False, "not installed")
+
+    esdb_open = False
+    chain_ok = False
+    record_count = 0
+    try:
+        with open_default_store(root, warn=False) as store:  # type: ignore[attr-defined]
+            esdb_open = True
+            record_count = int(store.record_count())
+            chain_ok = store.chain_valid() is not False
+    except Exception as exc:  # noqa: BLE001
+        _add("esdb backend", False, f"open failed: {exc}")
+    if esdb_open:
+        _add("esdb backend", True, f"{ESDB_BACKEND} open")
+        _add(
+            "esdb chain",
+            chain_ok,
+            f"{'valid' if chain_ok else 'invalid'} ({record_count} records)",
+        )
+        _add("esdb record count", True, str(record_count))
+
+    audit_report = run_audit(root)
+    _add(
+        "audit",
+        audit_report.healthy,
+        "clean" if audit_report.healthy else f"{audit_report.failed} issue(s)",
+    )
+
+    phase_key = read_phase(root)
+    _add("phase", True, phase_key)
+
+    for tool in ("ruff", "pytest", "pipx"):
+        ok, detail = _tool_version(tool)
+        _add(tool, ok, detail)
+
+    overall = "pass" if all(c["status"] != "fail" for c in checks) else "fail"
+
+    if as_json:
+        click.echo(_json.dumps({"checks": checks, "overall": overall}, indent=2))
+        return
+
+    icon = {"pass": "✅", "fail": "❌", "warn": "⚠️"}
+    console.print("specsmith doctor")
+    console.print("━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━")
+    for c in checks:
+        console.print(f"  {c['name']:<14} {icon[c['status']]}  {c['detail']}")
+    console.print("━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━")
+    if overall == "pass":
+        console.print("All checks passed.")
     else:
-        console.print(
-            f"[bold red]{report.missing_count} tool(s) missing.[/bold red] "
-            f"{report.installed_count} installed."
-        )
+        console.print("One or more checks failed.")
 
 
 @main.command()
@@ -6019,10 +6114,11 @@ def sync_cmd(project_dir: str, check_only: bool, as_json: bool) -> None:
     """
     import json as _json
 
-    from specsmith.sync import run_sync
+    from specsmith.sync import auto_migrate_if_needed, run_sync
 
     root = Path(project_dir).resolve()
     result = run_sync(root, dry_run=check_only)
+    auto_counts = {} if check_only else auto_migrate_if_needed(root)
 
     if as_json:
         click.echo(
@@ -6036,6 +6132,8 @@ def sync_cmd(project_dir: str, check_only: bool, as_json: bool) -> None:
                     "tests_changed": result.tests_changed,
                     "in_sync": not result.changed,
                     "dry_run": result.dry_run,
+                    "auto_migrated": bool(auto_counts),
+                    "auto_migrate_counts": auto_counts,
                 },
                 indent=2,
             )
@@ -6049,6 +6147,13 @@ def sync_cmd(project_dir: str, check_only: bool, as_json: bool) -> None:
                 console.print(f"[green]\u2713[/green] {result.message}")
         else:
             console.print("[green]\u2713[/green] Machine state already in sync.")
+        if auto_counts:
+            console.print(
+                "  [cyan]⟳[/cyan] ESDB auto-migrate: "
+                f"{auto_counts.get('requirements', 0)} requirements + "
+                f"{auto_counts.get('testcases', 0)} testcases "
+                f"({auto_counts.get('skipped', 0)} skipped)"
+            )
 
     if check_only and result.changed:
         raise SystemExit(1)
@@ -10475,8 +10580,10 @@ def esdb_status_cmd(project_dir: str, as_json: bool) -> None:
 
     from specsmith.esdb import CHRONO_AVAILABLE, open_default_store
     from specsmith.esdb._license import check_license, resolve_license_path
+    from specsmith.sync import auto_migrate_if_needed
 
     root = Path(project_dir).resolve()
+    auto_counts = auto_migrate_if_needed(root)
 
     # Determine license status without side-effect warnings
     lic_path = resolve_license_path()
@@ -10542,6 +10649,8 @@ def esdb_status_cmd(project_dir: str, as_json: bool) -> None:
                     "chain_valid": chain_ok,
                     "counts": counts,
                     "license": license_info,
+                    "auto_migrated": bool(auto_counts),
+                    "auto_migrate_counts": auto_counts,
                 },
                 indent=2,
             )
@@ -10572,6 +10681,13 @@ def esdb_status_cmd(project_dir: str, as_json: bool) -> None:
         console.print(
             "  [dim]Use 'specsmith esdb enable --key-file <path>' to activate ChronoStore.[/dim]"
         )
+    if auto_counts:
+        console.print(
+            "  [cyan]⟳[/cyan] Auto-migrated from legacy JSON: "
+            f"{auto_counts.get('requirements', 0)} requirements + "
+            f"{auto_counts.get('testcases', 0)} testcases "
+            f"({auto_counts.get('skipped', 0)} skipped)"
+        )
 
 
 @esdb_group.command(name="enable")

{specsmith-0.15.3.dev572 → specsmith-0.15.3.dev573}/src/specsmith/sync.py

@@ -553,6 +553,65 @@ def _sync_esdb(root: Path, state_dir: Path) -> None:
         pass
 
 
+def _json_file_has_content(path: Path) -> bool:
+    """Return True when *path* exists and contains a non-empty JSON list.
+
+    Falls back to a small size heuristic for malformed JSON so we still treat
+    legacy state files with obvious content as migratable.
+    """
+    if not path.is_file():
+        return False
+    try:
+        payload = json.loads(path.read_text(encoding="utf-8"))
+        return isinstance(payload, list) and len(payload) > 0
+    except (OSError, ValueError):
+        with contextlib.suppress(OSError):
+            return path.stat().st_size > 2
+        return False
+
+
+def _should_auto_migrate(store: Any, specsmith_dir: Path) -> bool:
+    """Return True when ESDB is empty and legacy JSON sources have content.
+
+    This is used by CLI surfaces (`sync`, `audit`, `esdb status`) to
+    opportunistically bootstrap ESDB from `.specsmith/*.json` exactly once.
+    """
+    if store.record_count() > 0:
+        return False
+    candidates = (
+        specsmith_dir / "requirements.json",
+        specsmith_dir / "testcases.json",
+    )
+    return any(_json_file_has_content(path) for path in candidates)
+
+
+def auto_migrate_if_needed(root: Path) -> dict[str, int]:
+    """Best-effort ESDB bootstrap from legacy JSON when the store is empty.
+
+    Returns migration counts on success, or an empty dict when no migration was
+    needed or if any backend error occurred.
+    """
+    from specsmith.esdb import open_default_store
+
+    specsmith_dir = root / ".specsmith"
+    if not specsmith_dir.exists():
+        return {}
+    try:
+        with open_default_store(root, warn=False) as store:  # type: ignore[attr-defined]
+            if not _should_auto_migrate(store, specsmith_dir):
+                return {}
+            counts = store.migrate_from_json(specsmith_dir)
+            if isinstance(counts, dict):
+                return {
+                    "requirements": int(counts.get("requirements", 0)),
+                    "testcases": int(counts.get("testcases", 0)),
+                    "skipped": int(counts.get("skipped", 0)),
+                }
+            return {}
+    except Exception:  # noqa: BLE001 - never block caller commands
+        return {}
+
+
 def check_sync(root: Path) -> SyncResult:
     """Check whether machine state is in sync without writing anything.
 

{specsmith-0.15.3.dev572 → specsmith-0.15.3.dev573/src/specsmith.egg-info}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: specsmith
-Version: 0.15.3.dev572
+Version: 0.15.3.dev573
 Summary: AEE governance toolkit for AI-assisted development — session preflight gates, multi-agent dispatch, requirements↔test traceability, ESDB persistence, MCP server, and skills for Warp, Cursor, Claude Code, Copilot, Windsurf, and Aider.
 Author: Layer1Labs Silicon, Inc.
 License-Expression: MIT
@@ -40,7 +40,7 @@ Requires-Dist: pre-commit>=3.0; extra == "dev"
 Requires-Dist: types-pyyaml>=6.0; extra == "dev"
 Requires-Dist: cryptography>=42.0; extra == "dev"
 Provides-Extra: esdb
-Requires-Dist: chronomemory>=0.1.7; extra == "esdb"
+Requires-Dist: chronomemory>=0.1.8; extra == "esdb"
 Requires-Dist: cryptography>=42.0; extra == "esdb"
 Provides-Extra: docs
 Requires-Dist: mkdocs>=1.6; extra == "docs"

{specsmith-0.15.3.dev572 → specsmith-0.15.3.dev573}/src/specsmith.egg-info/SOURCES.txt

@@ -259,6 +259,7 @@ tests/test_chat_diff_decision.py
 tests/test_chat_runner_openai_compat.py
 tests/test_chat_stdin_protocol.py
 tests/test_cli.py
+tests/test_cli_regressions.py
 tests/test_cli_workflows_history_drive.py
 tests/test_compliance.py
 tests/test_compliance_governance.py

{specsmith-0.15.3.dev572 → specsmith-0.15.3.dev573}/src/specsmith.egg-info/requires.txt

@@ -36,7 +36,7 @@ mkdocs>=1.6
 mkdocs-material>=9.5
 
 [esdb]
-chronomemory>=0.1.7
+chronomemory>=0.1.8
 cryptography>=42.0
 
 [gemini]

{specsmith-0.15.3.dev572 → specsmith-0.15.3.dev573}/tests/test_auditor.py

@@ -8,7 +8,7 @@ from pathlib import Path
 
 import pytest
 
-from specsmith.auditor import run_audit
+from specsmith.auditor import check_industrial_artifacts, run_audit
 
 
 @pytest.fixture
@@ -89,6 +89,23 @@ class TestAuditDetectsIssues:
         assert "REQ-CLI-002" in coverage[0].message
 
 
+def test_industrial_artifacts_normalizes_windows_declared_paths(tmp_path: Path) -> None:
+    (tmp_path / "scaffold.yml").write_text(
+        "name: test\n"
+        "type: cli-python\n"
+        "industrial_artifacts:\n"
+        "  canopen_eds:\n"
+        "    - path: hardware\\device.eds\n",
+        encoding="utf-8",
+    )
+    (tmp_path / "hardware").mkdir()
+    (tmp_path / "hardware" / "device.eds").write_text("EDS CONTENT", encoding="utf-8")
+
+    results = check_industrial_artifacts(tmp_path)
+    assert results
+    assert any(r.name == "industrial-artifacts" and r.passed for r in results)
+
+
 # ---------------------------------------------------------------------------
 # Regression: issue #183 — letter-suffix TEST IDs in req trace
 # ---------------------------------------------------------------------------

specsmith-0.15.3.dev573/tests/test_cli_regressions.py

@@ -0,0 +1,88 @@
+# SPDX-License-Identifier: MIT
+# Copyright (c) 2026 Layer1Labs Silicon, Inc. All rights reserved.
+"""Targeted CLI regressions for #249, #254, and #212."""
+
+from __future__ import annotations
+
+import json
+from pathlib import Path
+
+import yaml
+from click.testing import CliRunner
+
+from specsmith.cli import main
+
+
+def _invoke(args: list[str]):
+    runner = CliRunner()
+    return runner.invoke(
+        main,
+        args,
+        env={
+            "SPECSMITH_NO_AUTO_UPDATE": "1",
+            "SPECSMITH_PYPI_CHECKED": "1",
+            "SPECSMITH_ALLOW_NON_PIPX": "1",
+        },
+    )
+
+
+def test_esdb_enable_same_path_skips_copy2(monkeypatch, tmp_path: Path) -> None:
+    """#249: esdb enable must not call copy2 when src and dst resolve equal."""
+
+    class _Status:
+        valid = True
+        customer = "Test Customer"
+        expires_at = "2099-12-31T00:00:00Z"
+        reason = ""
+
+    home = tmp_path / "home"
+    key_dest = home / ".specsmith" / "esdb.key"
+    key_dest.parent.mkdir(parents=True)
+    key_dest.write_text("{}", encoding="utf-8")
+
+    monkeypatch.setattr("specsmith.esdb._license.verify_license_file", lambda _: _Status())
+    monkeypatch.setattr(Path, "home", classmethod(lambda cls: home))
+    monkeypatch.setattr(
+        "shutil.copy2", lambda *_a, **_k: (_ for _ in ()).throw(AssertionError("copy2"))
+    )
+
+    result = _invoke(["esdb", "enable", "--key-file", str(key_dest), "--json"])
+    assert result.exit_code == 0, result.output
+    payload = json.loads(result.output)
+    assert payload["ok"] is True
+
+
+def test_phase_next_skips_suppressed_phases(tmp_path: Path) -> None:
+    """#254: phase next should skip phases listed in suppressed_phases."""
+    scaffold = {
+        "name": "phase-regression",
+        "type": "cli-python",
+        "language": "python",
+        "aee_phase": "requirements",
+        "suppressed_phases": ["test_spec"],
+    }
+    (tmp_path / "scaffold.yml").write_text(yaml.safe_dump(scaffold), encoding="utf-8")
+
+    result = _invoke(["phase", "next", "--force", "--project-dir", str(tmp_path)])
+    assert result.exit_code == 0, result.output
+
+    updated = yaml.safe_load((tmp_path / "scaffold.yml").read_text(encoding="utf-8"))
+    assert updated["aee_phase"] == "implementation"
+    assert "Skipped suppressed phase" in result.output
+
+
+def test_doctor_json_outputs_overall_key(tmp_path: Path) -> None:
+    """#212: doctor --json should return valid JSON with an overall status key."""
+    result = _invoke(["doctor", "--project-dir", str(tmp_path), "--json"])
+    assert result.exit_code == 0, result.output
+    payload = json.loads(result.output)
+    assert "checks" in payload
+    assert "overall" in payload
+    assert payload["overall"] in ("pass", "fail")
+
+
+def test_doctor_human_exits_zero(tmp_path: Path) -> None:
+    """#212: doctor human output should exit 0 in normal environments."""
+    result = _invoke(["doctor", "--project-dir", str(tmp_path)])
+    assert result.exit_code == 0, result.output
+    assert "specsmith doctor" in result.output

{specsmith-0.15.3.dev572 → specsmith-0.15.3.dev573}/tests/test_esdb_enforcement.py

@@ -12,7 +12,7 @@ from click.testing import CliRunner
 
 from specsmith.cli import main
 from specsmith.esdb.sqlite_store import SqliteRecord, SqliteStore
-from specsmith.sync import run_sync
+from specsmith.sync import _should_auto_migrate, auto_migrate_if_needed, run_sync
 
 
 def _run(args: list[str]):  # type: ignore[return]  # noqa: ANN201
@@ -79,3 +79,52 @@ def test_save_uses_sqlite_store_backup(tmp_path: Path, monkeypatch: pytest.Monke
     backup_path = Path(backup_step["path"])
     assert backup_path.is_file()
     assert backup_path.suffix == ".sqlite3"
+
+
+def test_should_auto_migrate_true_when_store_empty(tmp_path: Path) -> None:
+    state = tmp_path / ".specsmith"
+    state.mkdir()
+    (state / "requirements.json").write_text(
+        json.dumps([{"id": "REQ-001", "title": "R"}]), encoding="utf-8"
+    )
+    (state / "testcases.json").write_text("[]", encoding="utf-8")
+
+    with SqliteStore(tmp_path) as store:
+        assert _should_auto_migrate(store, state) is True
+
+
+def test_should_auto_migrate_false_when_store_non_empty(tmp_path: Path) -> None:
+    state = tmp_path / ".specsmith"
+    state.mkdir()
+    (state / "requirements.json").write_text(
+        json.dumps([{"id": "REQ-001", "title": "R"}]), encoding="utf-8"
+    )
+    (state / "testcases.json").write_text("[]", encoding="utf-8")
+
+    with SqliteStore(tmp_path) as store:
+        store.upsert(SqliteRecord(id="REQ-000", kind="requirement", label="already-there"))
+        assert _should_auto_migrate(store, state) is False
+
+
+def test_auto_migrate_if_needed_populates_sqlite_from_legacy_json(
+    tmp_path: Path, monkeypatch: pytest.MonkeyPatch
+) -> None:
+    monkeypatch.setenv("SPECSMITH_ESDB_BACKEND", "sqlite")
+    state = tmp_path / ".specsmith"
+    state.mkdir()
+    (state / "requirements.json").write_text(
+        json.dumps([{"id": "REQ-101", "title": "Auto migrated req"}]), encoding="utf-8"
+    )
+    (state / "testcases.json").write_text(
+        json.dumps(
+            [{"id": "TEST-101", "title": "Auto migrated test", "requirement_id": "REQ-101"}]
+        ),
+        encoding="utf-8",
+    )
+
+    counts = auto_migrate_if_needed(tmp_path)
+    assert counts.get("requirements") == 1
+    assert counts.get("testcases") == 1
+
+    with SqliteStore(tmp_path) as store:
+        assert store.record_count() >= 2