spec-runner 2.2.0__tar.gz → 2.2.2__tar.gz

{spec_runner-2.2.0/src/spec_runner.egg-info → spec_runner-2.2.2}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: spec-runner
-Version: 2.2.0
+Version: 2.2.2
 Summary: Task automation from markdown specs via Claude CLI
 Author: Andrei
 License-Expression: MIT

{spec_runner-2.2.0 → spec_runner-2.2.2}/pyproject.toml

@@ -4,7 +4,7 @@ build-backend = "setuptools.build_meta"
 
 [project]
 name = "spec-runner"
-version = "2.2.0"
+version = "2.2.2"
 description = "Task automation from markdown specs via Claude CLI"
 readme = "README.md"
 requires-python = ">=3.10"

{spec_runner-2.2.0 → spec_runner-2.2.2}/src/spec_runner/logging.py

@@ -28,13 +28,18 @@ def redact_sensitive(logger: object, method_name: object, event_dict: dict) -> d
 
 def setup_logging(
     level: str = "info",
-    json_output: bool = True,  # ignored — obs always emits JSON
+    json_output: bool = True,  # ignored — obs always emits JSON to the file sink
     log_file: Path | None = None,
-    tui_mode: bool = False,  # ignored — obs writes to file; stdout stays free
+    tui_mode: bool = False,  # True → no console sink (the TUI owns the screen)
 ) -> None:
-    """Delegate to obs.init_logging; preserved signature for back-compat."""
+    """Delegate to obs.init_logging; preserved signature for back-compat.
+
+    Outside TUI mode a compact progress sink is mirrored to stderr so plain
+    ``run``/``watch`` invocations aren't silent; the JSON file sink is always
+    written. In TUI mode stderr stays free so it doesn't corrupt the display.
+    """
     log_dir = log_file.parent if log_file else None
-    obs.init_logging("spec-runner", level=level, log_dir=log_dir)
+    obs.init_logging("spec-runner", level=level, log_dir=log_dir, console=not tui_mode)
 
 
 def get_logger(module: str) -> structlog.BoundLogger:

{spec_runner-2.2.0 → spec_runner-2.2.2}/src/spec_runner/obs.py

@@ -12,6 +12,7 @@ import logging as _stdlib_logging
 import os
 import re
 import secrets
+import sys
 import time
 from collections.abc import Iterator
 from contextlib import contextmanager
@@ -128,6 +129,33 @@ def _reshape_to_otel(project: str):
     return processor
 
 
+def _console_progress():
+    """Side-effect processor: emit a compact human line to the current stderr.
+
+    Resolves ``sys.stderr`` at call time (not at bind time) so it never writes
+    to a stream that was swapped out or closed — e.g. under pytest capture, or
+    if the host reassigns stderr mid-run. Returns ``event_dict`` unchanged so
+    the JSON file sink still receives the full OTel record; the console copy is
+    trimmed of trace/transport plumbing (``pipeline_id``, span/trace ids). Must
+    run after ``_redact`` (secrets masked) and before ``_reshape_to_otel``.
+    """
+    renderer = structlog.dev.ConsoleRenderer(colors=sys.stderr.isatty())
+
+    def processor(logger, method_name, event_dict):
+        line = {
+            k: v
+            for k, v in event_dict.items()
+            if not k.startswith("_") and k not in ("pipeline_id", "parent_span_id")
+        }
+        line["level"] = method_name
+        line.setdefault("timestamp", datetime.now().strftime("%H:%M:%S"))
+        sys.stderr.write(renderer(logger, method_name, line) + "\n")
+        sys.stderr.flush()
+        return event_dict
+
+    return processor
+
+
 def _default_log_dir() -> Path:
     env_dir = os.environ.get("ORCHESTRA_LOG_DIR")
     if env_dir:
@@ -142,6 +170,7 @@ def init_logging(
     level: str | None = None,
     log_dir: Path | None = None,
     redact_keys: list[str] | None = None,
+    console: bool = False,
 ) -> None:
     global _initialized
     _initialized = False
@@ -176,13 +205,19 @@ def init_logging(
     level_name = (level or os.environ.get("ORCHESTRA_LOG_LEVEL") or "info").lower()
     min_level = {"debug": 10, "info": 20, "warning": 30, "error": 40}.get(level_name, 20)
 
+    processors: list[Any] = [
+        structlog.contextvars.merge_contextvars,
+        _redact(all_redact),
+    ]
+    if console:
+        processors.append(_console_progress())
+    processors += [
+        _reshape_to_otel(project),
+        structlog.processors.JSONRenderer(sort_keys=False),
+    ]
+
     structlog.configure(
-        processors=[
-            structlog.contextvars.merge_contextvars,
-            _redact(all_redact),
-            _reshape_to_otel(project),
-            structlog.processors.JSONRenderer(sort_keys=False),
-        ],
+        processors=processors,
         wrapper_class=structlog.make_filtering_bound_logger(min_level),
         logger_factory=structlog.WriteLoggerFactory(file=output_path.open("a")),
         cache_logger_on_first_use=True,

{spec_runner-2.2.0 → spec_runner-2.2.2}/src/spec_runner/task.py

@@ -18,7 +18,7 @@ CHECKLIST_ITEM = re.compile(r"^- \[([ x])\] (.+)$")
 TRACES_TO = re.compile(r"\*\*Traces to:\*\* (.+)")
 DEPENDS_ON = re.compile(r"\*\*Depends on:\*\* (.+)")
 BLOCKS = re.compile(r"\*\*Blocks:\*\* (.+)")
-ESTIMATE = re.compile(r"Est: (\d+(?:-\d+)?[dh])")
+ESTIMATE = re.compile(r"Est: (\d+(?:\.\d+)?(?:[-–]\d+(?:\.\d+)?)?[dh])")
 
 STATUS_EMOJI = {"todo": "⬜", "in_progress": "🔄", "done": "✅", "blocked": "⏸️"}
 

{spec_runner-2.2.0 → spec_runner-2.2.2/src/spec_runner.egg-info}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: spec-runner
-Version: 2.2.0
+Version: 2.2.2
 Summary: Task automation from markdown specs via Claude CLI
 Author: Andrei
 License-Expression: MIT

{spec_runner-2.2.0 → spec_runner-2.2.2}/src/spec_runner.egg-info/SOURCES.txt

@@ -78,6 +78,7 @@ tests/test_report.py
 tests/test_runner.py
 tests/test_spec_prefix.py
 tests/test_state.py
+tests/test_task.py
 tests/test_task_diff.py
 tests/test_tui.py
 tests/test_validate.py

{spec_runner-2.2.0 → spec_runner-2.2.2}/tests/test_logging.py

@@ -21,6 +21,31 @@ class TestSetupLogging:
         setup_logging(tui_mode=True, log_file=log_file)
 
 
+class TestConsoleProgress:
+    """Console progress sink (non-TUI mode mirrors a compact line to stderr)."""
+
+    def test_non_tui_mirrors_progress_to_stderr(self, tmp_path, capsys):
+        setup_logging(tui_mode=False, log_file=tmp_path / "run.log")
+        get_logger("console_on_test").info("ping_console", task_id="TASK-099")
+        err = capsys.readouterr().err
+        assert "ping_console" in err
+        assert "TASK-099" in err
+
+    def test_tui_mode_keeps_stderr_quiet(self, tmp_path, capsys):
+        setup_logging(tui_mode=True, log_file=tmp_path / "tui.log")
+        get_logger("console_off_test").info("quiet_console", task_id="TASK-100")
+        err = capsys.readouterr().err
+        assert "quiet_console" not in err
+
+    def test_console_line_omits_trace_plumbing(self, tmp_path, capsys):
+        setup_logging(tui_mode=False, log_file=tmp_path / "run.log")
+        get_logger("console_trace_test").info("trace_check")
+        err = capsys.readouterr().err
+        assert "trace_check" in err
+        assert "pipeline_id" not in err
+        assert "_trace_id" not in err
+
+
 class TestGetLogger:
     """Tests for get_logger."""
 

{spec_runner-2.2.0 → spec_runner-2.2.2}/tests/test_obs_contract.py

@@ -12,7 +12,14 @@ import pytest
 _SPEC_RUNNER_ROOT = Path(__file__).resolve().parents[1]
 _UMBRELLA = _SPEC_RUNNER_ROOT.parent  # all_ai_orchestrators/
 _CONTRACT = _UMBRELLA / "Maestro" / "_cowork_output" / "observability-contract"
-_SCHEMA = json.loads((_CONTRACT / "log-schema.json").read_text())
+_SCHEMA_PATH = _CONTRACT / "log-schema.json"
+if not _SCHEMA_PATH.exists():
+    pytest.skip(
+        "observability-contract unavailable (external cowork workspace, "
+        "not checked out in standalone CI)",
+        allow_module_level=True,
+    )
+_SCHEMA = json.loads(_SCHEMA_PATH.read_text())
 
 
 @pytest.fixture

spec_runner-2.2.2/tests/test_task.py

@@ -0,0 +1,29 @@
+"""Regression tests for task parsing (spec/tasks.md)."""
+
+from pathlib import Path
+
+from spec_runner.task import parse_tasks
+
+
+def _single_task(tmp_path: Path, estimate_line: str) -> str:
+    """Parse a one-task file and return the parsed estimate."""
+    f = tmp_path / "tasks.md"
+    f.write_text(f"### TASK-001: Bootstrap\n{estimate_line}\n")
+    (task,) = parse_tasks(f)
+    return task.estimate
+
+
+class TestEstimateParsing:
+    def test_integer_estimate_parsed(self, tmp_path: Path) -> None:
+        assert _single_task(tmp_path, "P0 | todo | Est: 2d") == "2d"
+
+    def test_decimal_estimate_parsed(self, tmp_path: Path) -> None:
+        """Decimal estimates (e.g. 1.5d) must parse, not read as missing."""
+        assert _single_task(tmp_path, "P0 | todo | Est: 1.5d") == "1.5d"
+
+    def test_endash_range_parsed(self, tmp_path: Path) -> None:
+        """En-dash ranges (1–1.5d, U+2013) must parse, not read as missing."""
+        assert _single_task(tmp_path, "P0 | todo | Est: 1–1.5d") == "1–1.5d"
+
+    def test_ascii_hyphen_range_still_parsed(self, tmp_path: Path) -> None:
+        assert _single_task(tmp_path, "P0 | todo | Est: 1-2d") == "1-2d"