teddy-cli 0.1.17.dev1325__tar.gz → 0.1.17.dev1327__tar.gz

{teddy_cli-0.1.17.dev1325 → teddy_cli-0.1.17.dev1327}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.5
 Name: teddy-cli
-Version: 0.1.17.dev1325
+Version: 0.1.17.dev1327
 Summary: The local-first, test-driven agentic engineering harness. Because you should own the means of development.
 Author: Raphael Atteritano
 License: AGPL-3.0-only

{teddy_cli-0.1.17.dev1325 → teddy_cli-0.1.17.dev1327}/pyproject.toml

@@ -1,6 +1,6 @@
 [project]
 name = "teddy-cli"
-version = "0.1.17.dev1325"
+version = "0.1.17.dev1327"
 description = "The local-first, test-driven agentic engineering harness. Because you should own the means of development."
 readme = "README.md"
 keywords = [

{teddy_cli-0.1.17.dev1325 → teddy_cli-0.1.17.dev1327}/src/teddy_executor/core/ports/inbound/plan_validator.py

@@ -2,10 +2,6 @@
 This module defines the inbound port for plan validation.
 """
 
-# Placeholder content
-# The Developer will implement this based on the design document.
-# See: docs/core/ports/inbound/plan_validator.md
-
 from abc import ABC, abstractmethod
 from typing import Dict, Optional, Sequence
 

{teddy_cli-0.1.17.dev1325 → teddy_cli-0.1.17.dev1327}/src/teddy_executor/core/ports/outbound/markdown_report_formatter.py

@@ -2,10 +2,6 @@
 This module defines the outbound port for markdown report formatting.
 """
 
-# Placeholder content
-# The Developer will implement this based on the design document.
-# See: docs/core/ports/outbound/markdown_report_formatter.md
-
 from abc import ABC, abstractmethod
 
 

{teddy_cli-0.1.17.dev1325 → teddy_cli-0.1.17.dev1327}/src/teddy_executor/core/services/update_checker.py

@@ -178,6 +178,8 @@ def write_update_cache(cache_path: Path, latest_version: str) -> None:
     """
     Atomically write the cache file (write to temp file, rename).
     Ensures the main thread never reads a partially written file.
+    The write is best-effort: a directory-creation or write failure is
+    logged at debug level and swallowed.
     """
     import json
     from datetime import datetime, timezone
@@ -186,9 +188,9 @@ def write_update_cache(cache_path: Path, latest_version: str) -> None:
         "latest_version": latest_version,
         "checked_at": datetime.now(timezone.utc).isoformat(),
     }
-    cache_path.parent.mkdir(parents=True, exist_ok=True)
     tmp_path = cache_path.with_suffix(".tmp")
     try:
+        cache_path.parent.mkdir(parents=True, exist_ok=True)
         tmp_path.write_text(
             json.dumps(cache_data, indent=2), encoding="utf-8", newline=""
         )
@@ -206,8 +208,15 @@ def background_check(cache_path: Path, index_url: str = PYPI_URL) -> None:
     """
     Non-blocking background version check.
     Intended to run in a daemon thread. Fetches latest version from PyPI
-    and writes it to cache. All errors are silently caught.
+    and writes it to cache. Every error is swallowed and logged at debug
+    level, so a late cache write can never fail the owning process.
     """
-    latest = fetch_latest_version(index_url)
-    if latest is not None:
-        write_update_cache(cache_path, latest)
+    import logging
+
+    try:
+        latest = fetch_latest_version(index_url)
+        if latest is not None:
+            write_update_cache(cache_path, latest)
+    except Exception as e:
+        logger = logging.getLogger(__name__)
+        logger.debug("background_check failed: %s", e)

{teddy_cli-0.1.17.dev1325 → teddy_cli-0.1.17.dev1327}/src/teddy_executor/resources/templates/makefile.md

@@ -169,7 +169,7 @@ Attribution is deterministic: SHA → run → run-level + per-job conclusion (ne
 
 Key behaviors:
 
-1. **Automatic SHA and BASE resolution.** `SHA` defaults to current `HEAD` (or `COMMIT=<sha>`), and `BASE` resolves per workflow to the most recent ancestor of `SHA` that carries a run for that workflow (bounded by `BASE_DEPTH`, default 50).
+1. **Automatic SHA and BASE resolution.** `SHA` defaults to current `HEAD` (or `COMMIT=<sha>`), and `BASE` resolves per workflow to the most recent ancestor of `SHA` that carries a run for that workflow (bounded by `BASE_DEPTH`, default 50). An explicit `BASE=<sha>` pins the baseline verbatim and skips resolution; if the pinned commit carries no run for the workflow, the target fails safe as an unknown base.
 2. **Registration lag is tolerated by a bounded retry.** CI can fail to register a run for the pushed SHA immediately. The target polls `gh run list --commit <sha>` up to `RETRY_POLLS × RETRY_SLEEP` (default 10 × 2s ≈ 20s) before attributing. It fails safe if the run never registers.
 3. **No CI is non-blocking.** If the repository has no workflow files under `.github/workflows/` (`*.yml`/`*.yaml`), the target logs `No CI configured - skipping attribution` and exits 0.
 
@@ -180,6 +180,7 @@ RETRY_POLLS ?= 10
 RETRY_SLEEP ?= 2
 DETAIL_LINES ?= 50
 BASE_DEPTH ?= 50
+BASE ?=
 
 ci-watch:
 	@SHA=$$(git rev-parse $${COMMIT:-HEAD}); \
@@ -223,8 +224,8 @@ ci-watch:
 	    echo "$$WF: BLOCKING (run=$$NEWCONC) — unknown conclusion — fail-safe block"; \
 	    STATUS=1; continue; \
 	  fi; \
-	  BASE=""; CUR="$$SHA"; DEPTH=0; \
-	  while [ $$DEPTH -lt $(BASE_DEPTH) ]; do \
+	  BASE="$(BASE)"; CUR="$$SHA"; DEPTH=0; \
+	  while [ -z "$$BASE" ] && [ $$DEPTH -lt $(BASE_DEPTH) ]; do \
 	    CUR=$$(git rev-parse "$$CUR~1" 2>/dev/null) || break; \
 	    [ -n "$$CUR" ] || break; \
 	    if [ -n "$$(gh run list --commit "$$CUR" --workflow "$$WF" --json databaseId --jq '.[0].databaseId')" ]; then \
@@ -237,6 +238,10 @@ ci-watch:
 	    STATUS=1; continue; \
 	  fi; \
 	  OLD=$$(gh run list --commit "$$BASE" --workflow "$$WF" --json databaseId --jq '.[0].databaseId'); \
+	  if [ -z "$$OLD" ]; then \
+	    echo "$$WF: BLOCKING failure ($$NEWFAIL) — pinned base has no run (unknown base) — YOUR regression"; \
+	    STATUS=1; continue; \
+	  fi; \
 	  OLDJOBS=$$(gh run view "$$OLD" --json jobs --jq '.jobs | length'); \
 	  if [ "$$OLDJOBS" = "0" ]; then \
 	    echo "$$WF: BLOCKING failure ($$NEWFAIL) — base run is job-less (invalid workflow, not green) — YOUR regression"; \
@@ -257,6 +262,7 @@ ci-watch:
 **Usage:**
 - `make ci-watch` (watches CI for current `HEAD`)
 - `make ci-watch COMMIT=<sha>` (watches CI for a specific commit)
+- `make ci-watch BASE=<sha>` (pins the comparison baseline to a commit)
 
 ## Remote Probing Protocol (RPP)
 

{teddy_cli-0.1.17.dev1325 → teddy_cli-0.1.17.dev1327}/src/teddy_executor/resources/templates/pre-commit.md

@@ -77,7 +77,7 @@ repos:
 
 **Tip:** Pin exact `rev` tags so every teammate runs identical checks.
 
-**Line endings:** The whitespace hooks (`trailing-whitespace`, `end-of-file-fixer`) fix trailing spaces and the missing final newline. They do NOT normalize LF versus CRLF. Add a committed `.gitattributes` with the rule `* text=auto eol=lf`. Git then stores and checks out text as LF on every platform. This rule applies to every clone and to unstaged files, so it is more robust than a hook. TeDDy does NOT create this file. Create it yourself in the project root.
+**Line endings:** The whitespace hooks (`trailing-whitespace`, `end-of-file-fixer`) fix trailing spaces and the missing final newline. They do NOT normalize LF versus CRLF. Add a committed `.gitattributes` with the rule `* text=auto eol=lf`. Git then stores and checks out text as LF on every platform. This rule applies to every clone and to unstaged files, so it is more robust than a hook.
 
 ## 3. Install the hooks