specsmith 0.18.0.dev691__tar.gz → 0.18.0.dev698__tar.gz

{specsmith-0.18.0.dev691/src/specsmith.egg-info → specsmith-0.18.0.dev698}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: specsmith
-Version: 0.18.0.dev691
+Version: 0.18.0.dev698
 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

{specsmith-0.18.0.dev691 → specsmith-0.18.0.dev698}/pyproject.toml

@@ -4,7 +4,7 @@ build-backend = "setuptools.build_meta"
 
 [project]
 name = "specsmith"
-version = "0.18.0.dev691"
+version = "0.18.0.dev698"
 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"

{specsmith-0.18.0.dev691 → specsmith-0.18.0.dev698}/src/specsmith/agent/runner.py

@@ -587,28 +587,6 @@ class AgentRunner:
             self._emit_event(type="system", message=hint)
             return None
 
-        if result is None:
-            # All providers failed — give the user an actionable explanation
-            # rather than silent emptiness.
-            import os as _os
-
-            host = _os.environ.get("OLLAMA_HOST", DEFAULT_OLLAMA_HOST).rstrip("/")
-            if _ollama_alive(host):
-                model = _pick_ollama_model(host)
-                hint = (
-                    f"Ollama is running but returned no response "
-                    f"(model: {model}). "
-                    "Try: ollama run " + model
-                )
-            else:
-                hint = (
-                    "No LLM provider available. Options:\n"
-                    "  • Start Ollama: ollama serve\n"
-                    "  • Set ANTHROPIC_API_KEY / OPENAI_API_KEY / GOOGLE_API_KEY"
-                )
-            self._emit_event(type="system", message=hint)
-            return None
-
         # Aggregate metrics into the session state (C1).
         # ``run_chat`` now reports tokens_in / tokens_out / cost_usd off the
         # provider response (Ollama prompt_eval_count + eval_count, OpenAI

{specsmith-0.18.0.dev691 → specsmith-0.18.0.dev698}/src/specsmith/releaser.py

@@ -71,22 +71,57 @@ def bump_version(root: Path, new_version: str) -> list[str]:
     return updated
 
 
+# Context tokens that mark a *legitimate* "--pre specsmith" reference rather
+# than a stale stable-install instruction. The dev/pre-release channel is a
+# supported feature (see channel.py and docs/site/releasing.md), so docs that
+# describe it -- and explicit prohibitions ("NEVER run pip install --pre
+# specsmith") -- must not be flagged. Only a reference that recommends
+# installing the *stable* package with the pre-release flag is stale.
+_PRE_OK_CONTEXT: tuple[str, ...] = ("dev", "pre-release", "prerelease", "never", "badge")
+
+
+def _has_stale_pre_flag(content: str) -> bool:
+    """Return True if *content* has a stale ``--pre specsmith`` install hint.
+
+    The scan is line-aware and tracks the nearest Markdown heading so a
+    legitimate ``--pre specsmith`` reference inside a dev/pre-release section or
+    a prohibition line is ignored. An occurrence is only treated as stale when
+    neither its line nor the enclosing section carries a :data:`_PRE_OK_CONTEXT`
+    token.
+    """
+    current_heading = ""
+    for line in content.splitlines():
+        stripped = line.lstrip()
+        if stripped.startswith("#"):
+            current_heading = stripped
+        if "--pre specsmith" not in line:
+            continue
+        context = f"{line} {current_heading}".lower()
+        if any(token in context for token in _PRE_OK_CONTEXT):
+            continue
+        return True
+    return False
+
+
 def scan_stale_refs(root: Path, current_version: str) -> list[str]:
-    """Scan docs for stale version references."""
+    """Scan docs for stale version references.
+
+    The ``--pre specsmith`` check is context-aware (see
+    :func:`_has_stale_pre_flag`) so the supported dev/pre-release channel docs
+    and prohibitions do not produce false positives; only stale *stable*-install
+    hints are reported.
+    """
     issues: list[str] = []
     scan_dirs = [root / "docs" / "site", root]
-    scan_patterns = [
-        (r"--pre specsmith", "stale --pre flag"),
-        (r"0\.1\.0a\d+", "alpha version reference"),
-    ]
 
     for scan_dir in scan_dirs:
         for md_file in scan_dir.glob("*.md"):
             content = md_file.read_text(encoding="utf-8")
-            for pattern, desc in scan_patterns:
-                if re.search(pattern, content):
-                    rel = md_file.relative_to(root)
-                    issues.append(f"{rel}: {desc}")
+            rel = md_file.relative_to(root)
+            if _has_stale_pre_flag(content):
+                issues.append(f"{rel}: stale --pre flag")
+            if re.search(r"0\.1\.0a\d+", content):
+                issues.append(f"{rel}: alpha version reference")
 
     # Check classifier
     pyproject = root / "pyproject.toml"

{specsmith-0.18.0.dev691 → specsmith-0.18.0.dev698/src/specsmith.egg-info}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: specsmith
-Version: 0.18.0.dev691
+Version: 0.18.0.dev698
 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

{specsmith-0.18.0.dev691 → specsmith-0.18.0.dev698}/src/specsmith.egg-info/SOURCES.txt

@@ -333,6 +333,7 @@ tests/test_phase34_completion.py
 tests/test_project_metrics.py
 tests/test_quality_report.py
 tests/test_rate_limits.py
+tests/test_releaser.py
 tests/test_repl.py
 tests/test_repl_extended.py
 tests/test_reporting_commands.py

{specsmith-0.18.0.dev691 → specsmith-0.18.0.dev698}/tests/test_parallel_agents.py

@@ -24,6 +24,7 @@ from __future__ import annotations
 from pathlib import Path
 from unittest import mock
 
+import specsmith.esdb_writer as _esdb_mod
 from specsmith.agent.core import AgentState
 from specsmith.agent.dispatch import (
     AgentPool,
@@ -303,8 +304,6 @@ class TestTokenMetricPerNode:
     """write_token_metric should be called once per completed node when tokens > 0."""
 
     def test_write_token_metric_called_per_completed_node(self, tmp_path: Path) -> None:
-        import specsmith.esdb_writer as _esdb_mod
-
         calls: list[dict] = []
 
         def _fake_write(project_dir, **kwargs):
@@ -332,22 +331,10 @@ class TestTokenMetricPerNode:
         calls: list[dict] = []
 
         with mock.patch("specsmith.esdb_writer.write_token_metric", side_effect=calls.append):
-            # Simulate a node completing with no tokens (shouldn't write)
             # The runner guards with: if tokens_in or tokens_out: write_token_metric(...)
-            tokens_in = 0
-            tokens_out = 0
-            if tokens_in or tokens_out:  # matches runner.py guard
-                from specsmith.esdb_writer import write_token_metric
-
-                write_token_metric(
-                    tmp_path,
-                    input_tokens=tokens_in,
-                    output_tokens=tokens_out,
-                    cost_usd=0.0,
-                    model="",
-                    command_source="no-token-node",
-                    work_item_id="",
-                )
+            # Zero-token nodes never satisfy the guard; write_token_metric is not called.
+            tokens_in, tokens_out = 0, 0
+            assert not (tokens_in or tokens_out), "tokens must be zero to exercise the guard"
         assert calls == [], "No ESDB write should occur for zero-token nodes"
 
 

specsmith-0.18.0.dev698/tests/test_releaser.py

@@ -0,0 +1,91 @@
+# SPDX-License-Identifier: MIT
+# Copyright (c) 2026 Layer1Labs Silicon, Inc. All rights reserved.
+"""Tests for the release-tooling stale-reference scanner (REQ-050).
+
+These cover the context-aware ``--pre specsmith`` check so the supported
+dev/pre-release channel docs and explicit prohibitions are not reported as
+stale, while genuine stable-install hints still are.
+"""
+
+from pathlib import Path
+
+from specsmith.releaser import _has_stale_pre_flag, scan_stale_refs
+
+
+def _write(path: Path, text: str) -> None:
+    path.parent.mkdir(parents=True, exist_ok=True)
+    path.write_text(text, encoding="utf-8")
+
+
+def test_dev_channel_pre_reference_not_flagged(tmp_path: Path) -> None:
+    """A dev-docs note mapping dev docs to the pre-release install is legitimate."""
+    _write(
+        tmp_path / "docs" / "site" / "index.md",
+        "# Docs\n\n**Dev (latest):** matches `pip install --pre specsmith`\n",
+    )
+    assert scan_stale_refs(tmp_path, "0.18.0") == []
+
+
+def test_dev_release_section_pre_reference_not_flagged(tmp_path: Path) -> None:
+    """An install hint under a 'Dev Releases' heading is legitimate dev-channel docs."""
+    _write(
+        tmp_path / "docs" / "site" / "releasing.md",
+        "## Dev Releases (develop branch)\n\nInstall: `pip install --pre specsmith`\n",
+    )
+    assert scan_stale_refs(tmp_path, "0.18.0") == []
+
+
+def test_prohibition_pre_reference_not_flagged(tmp_path: Path) -> None:
+    """A prohibition must not be read as a stale install hint."""
+    _write(
+        tmp_path / "AGENTS.md",
+        "## Install policy\n\n- NEVER run `pip install --pre specsmith` in any environment.\n",
+    )
+    assert scan_stale_refs(tmp_path, "0.18.0") == []
+
+
+def test_stale_stable_install_hint_is_flagged(tmp_path: Path) -> None:
+    """Recommending --pre for a normal install under a neutral heading is stale."""
+    _write(
+        tmp_path / "docs" / "site" / "getting-started.md",
+        "## Quick Start\n\nInstall specsmith: `pip install --pre specsmith`\n",
+    )
+    rel = Path("docs") / "site" / "getting-started.md"
+    assert scan_stale_refs(tmp_path, "0.18.0") == [f"{rel}: stale --pre flag"]
+
+
+def test_alpha_version_reference_flagged(tmp_path: Path) -> None:
+    """The narrow 0.1.0aN alpha pattern is still reported."""
+    _write(tmp_path / "README.md", "# x\n\nsee 0.1.0a3 builds\n")
+    assert scan_stale_refs(tmp_path, "0.18.0") == ["README.md: alpha version reference"]
+
+
+def test_alpha_classifier_flagged(tmp_path: Path) -> None:
+    """A '3 - Alpha' classifier in pyproject.toml is still reported."""
+    (tmp_path / "pyproject.toml").write_text(
+        '[project]\nclassifiers = ["Development Status :: 3 - Alpha"]\n',
+        encoding="utf-8",
+    )
+    assert "pyproject.toml: classifier still says Alpha" in scan_stale_refs(tmp_path, "0.18.0")
+
+
+def test_clean_docs_no_issues(tmp_path: Path) -> None:
+    """Stable install instructions and a non-alpha classifier produce no issues."""
+    _write(tmp_path / "docs" / "site" / "index.md", "# Docs\n\n`pip install specsmith`\n")
+    (tmp_path / "pyproject.toml").write_text(
+        '[project]\nclassifiers = ["Development Status :: 5 - Production/Stable"]\n',
+        encoding="utf-8",
+    )
+    assert scan_stale_refs(tmp_path, "0.18.0") == []
+
+
+def test_has_stale_pre_flag_unit() -> None:
+    """Direct checks of the line+heading-aware helper."""
+    # Neutral install hint with no qualifying context -> stale.
+    assert _has_stale_pre_flag("Install: `pip install --pre specsmith`") is True
+    # Heading context marks the section as dev -> legitimate.
+    assert _has_stale_pre_flag("## Dev Releases\nrun `pip install --pre specsmith`") is False
+    # Prohibition -> legitimate.
+    assert _has_stale_pre_flag("NEVER `pip install --pre specsmith`") is False
+    # No pre-release flag at all.
+    assert _has_stale_pre_flag("`pip install specsmith`") is False