buildlog 0.1.0__py3-none-any.whl → 0.2.0__py3-none-any.whl

buildlog/cli.py

@@ -9,7 +9,7 @@ from pathlib import Path
 import click
 
 from buildlog.distill import CATEGORIES, distill_all, format_output
-from buildlog.skills import generate_skills, format_skills
+from buildlog.skills import format_skills, generate_skills
 from buildlog.stats import calculate_stats, format_dashboard, format_json
 
 
@@ -27,6 +27,7 @@ def get_template_dir() -> Path | None:
 
     # 2. Check installed location (site-packages/../share/buildlog)
     import sysconfig
+
     data_dir = Path(sysconfig.get_path("data")) / "share" / "buildlog"
     if (data_dir / "copier.yml").exists():
         return data_dir
@@ -67,13 +68,16 @@ def init(no_claude_md: bool):
         try:
             subprocess.run(
                 [
-                    sys.executable, "-m", "copier", "copy",
+                    sys.executable,
+                    "-m",
+                    "copier",
+                    "copy",
                     "--trust",
                     *(["--data", "update_claude_md=false"] if no_claude_md else []),
                     str(template_dir),
-                    "."
+                    ".",
                 ],
-                check=True
+                check=True,
             )
         except subprocess.CalledProcessError:
             click.echo("Failed to initialize buildlog.", err=True)
@@ -84,13 +88,16 @@ def init(no_claude_md: bool):
         try:
             subprocess.run(
                 [
-                    sys.executable, "-m", "copier", "copy",
+                    sys.executable,
+                    "-m",
+                    "copier",
+                    "copy",
                     "--trust",
                     *(["--data", "update_claude_md=false"] if no_claude_md else []),
                     "gh:Peleke/buildlog-template",
-                    "."
+                    ".",
                 ],
-                check=True
+                check=True,
             )
         except subprocess.CalledProcessError:
             click.echo("Failed to initialize buildlog.", err=True)
@@ -102,7 +109,9 @@ def init(no_claude_md: bool):
 
 @main.command()
 @click.argument("slug")
-@click.option("--date", "-d", "entry_date", default=None, help="Date for entry (YYYY-MM-DD)")
+@click.option(
+    "--date", "-d", "entry_date", default=None, help="Date for entry (YYYY-MM-DD)"
+)
 def new(slug: str, entry_date: str | None):
     """Create a new buildlog entry.
 
@@ -121,7 +130,9 @@ def new(slug: str, entry_date: str | None):
         raise SystemExit(1)
 
     if not template_file.exists():
-        click.echo("No _TEMPLATE.md found in buildlog/. Run 'buildlog init' first.", err=True)
+        click.echo(
+            "No _TEMPLATE.md found in buildlog/. Run 'buildlog init' first.", err=True
+        )
         raise SystemExit(1)
 
     # Determine date
@@ -170,8 +181,7 @@ def list():
         raise SystemExit(1)
 
     entries = sorted(
-        buildlog_dir.glob("20??-??-??-*.md"),
-        reverse=True  # Most recent first
+        buildlog_dir.glob("20??-??-??-*.md"), reverse=True  # Most recent first
     )
 
     if not entries:
@@ -183,7 +193,9 @@ def list():
         # Extract title from first line if possible
         try:
             first_line = entry.read_text().split("\n")[0]
-            title = first_line.replace("# Build Journal: ", "").replace("# ", "").strip()
+            title = (
+                first_line.replace("# Build Journal: ", "").replace("# ", "").strip()
+            )
             if title == "[TITLE]":
                 title = "(untitled)"
         except Exception:
@@ -202,21 +214,23 @@ def update():
         click.echo("Updating from local template...")
         try:
             subprocess.run(
-                [sys.executable, "-m", "copier", "update", "--trust"],
-                check=True
+                [sys.executable, "-m", "copier", "update", "--trust"], check=True
             )
         except subprocess.CalledProcessError:
-            click.echo("Failed to update. Try running 'copier update' directly.", err=True)
+            click.echo(
+                "Failed to update. Try running 'copier update' directly.", err=True
+            )
             raise SystemExit(1)
     else:
         click.echo("Updating from GitHub...")
         try:
             subprocess.run(
-                [sys.executable, "-m", "copier", "update", "--trust"],
-                check=True
+                [sys.executable, "-m", "copier", "update", "--trust"], check=True
             )
         except subprocess.CalledProcessError:
-            click.echo("Failed to update. Try running 'copier update' directly.", err=True)
+            click.echo(
+                "Failed to update. Try running 'copier update' directly.", err=True
+            )
             raise SystemExit(1)
 
     click.echo("\n✓ buildlog updated!")
@@ -273,7 +287,7 @@ def distill(output: str | None, fmt: str, since: datetime | None, category: str
 
     # Format output
     try:
-        formatted = format_output(result, fmt)
+        formatted = format_output(result, fmt)  # type: ignore[arg-type]
     except ImportError as e:
         click.echo(str(e), err=True)
         raise SystemExit(1)
@@ -283,7 +297,9 @@ def distill(output: str | None, fmt: str, since: datetime | None, category: str
         output_path = Path(output)
         try:
             output_path.write_text(formatted, encoding="utf-8")
-            click.echo(f"Wrote {result.statistics.get('total_patterns', 0)} patterns to {output_path}")
+            click.echo(
+                f"Wrote {result.statistics.get('total_patterns', 0)} patterns to {output_path}"
+            )
         except Exception as e:
             click.echo(f"Failed to write output: {e}", err=True)
             raise SystemExit(1)
@@ -293,8 +309,15 @@ def distill(output: str | None, fmt: str, since: datetime | None, category: str
 
 @main.command()
 @click.option("--json", "output_json", is_flag=True, help="Output as JSON")
-@click.option("--detailed", is_flag=True, help="Show detailed breakdown including top sources")
-@click.option("--since", "since_date", default=None, help="Only include entries since date (YYYY-MM-DD)")
+@click.option(
+    "--detailed", is_flag=True, help="Show detailed breakdown including top sources"
+)
+@click.option(
+    "--since",
+    "since_date",
+    default=None,
+    help="Only include entries since date (YYYY-MM-DD)",
+)
 def stats(output_json: bool, detailed: bool, since_date: str | None):
     """Show buildlog statistics and analytics.
 
@@ -409,7 +432,7 @@ def skills(
 
     # Format output
     try:
-        formatted = format_skills(skill_set, fmt)
+        formatted = format_skills(skill_set, fmt)  # type: ignore[arg-type]
     except ImportError as e:
         click.echo(str(e), err=True)
         raise SystemExit(1)

buildlog/core/operations.py

@@ -59,7 +59,7 @@ class PromoteResult:
     """IDs of skills that were promoted."""
 
     target: str
-    """Target format (claude_md or settings_json)."""
+    """Target format (claude_md, settings_json, or skill)."""
 
     rules_added: int
     """Number of rules added."""
@@ -192,7 +192,7 @@ def status(
     confidence_order = {"low": 0, "medium": 1, "high": 2}
     min_level = confidence_order[min_confidence]
 
-    filtered: dict[str, list[dict]] = {}
+    filtered: dict[str, list[dict]] = {}  # type: ignore[type-arg]
     by_confidence = {"high": 0, "medium": 0, "low": 0}
     promotable: list[str] = []
 
@@ -215,7 +215,7 @@ def status(
                 category_skills.append(skill.to_dict())
 
         if category_skills:
-            filtered[category] = category_skills
+            filtered[category] = category_skills  # type: ignore[assignment]
 
     # Calculate actual total (sum of by_confidence, which excludes rejected)
     actual_total = sum(by_confidence.values())
@@ -232,7 +232,7 @@ def status(
 def promote(
     buildlog_dir: Path,
     skill_ids: list[str],
-    target: Literal["claude_md", "settings_json"] = "claude_md",
+    target: Literal["claude_md", "settings_json", "skill"] = "claude_md",
     target_path: Path | None = None,
 ) -> PromoteResult:
     """Promote skills to agent rules.
@@ -240,7 +240,7 @@ def promote(
     Args:
         buildlog_dir: Path to buildlog directory.
         skill_ids: List of skill IDs to promote.
-        target: Where to write rules ("claude_md" or "settings_json").
+        target: Where to write rules ("claude_md", "settings_json", or "skill").
         target_path: Optional custom path for the target file.
 
     Returns:
@@ -277,13 +277,8 @@ def promote(
     # Set up tracking path in buildlog directory
     tracking_path = _get_promoted_path(buildlog_dir)
 
-    # Get renderer with custom paths
-    if target == "claude_md":
-        from buildlog.render.claude_md import ClaudeMdRenderer
-        renderer = ClaudeMdRenderer(path=target_path, tracking_path=tracking_path)
-    else:
-        from buildlog.render.settings_json import SettingsJsonRenderer
-        renderer = SettingsJsonRenderer(path=target_path, tracking_path=tracking_path)
+    # Get renderer using the registry pattern
+    renderer = get_renderer(target, path=target_path, tracking_path=tracking_path)
 
     message = renderer.render(found_skills)
 
@@ -372,16 +367,17 @@ def diff(
     promoted_ids = _load_json_set(_get_promoted_path(buildlog_dir), "skill_ids")
 
     # Find unpromoted, unrejected skills
-    pending: dict[str, list[dict]] = {}
+    pending: dict[str, list[dict]] = {}  # type: ignore[type-arg]
     total_pending = 0
 
     for category, skill_list in skill_set.skills.items():
         pending_skills = [
-            s.to_dict() for s in skill_list
+            s.to_dict()
+            for s in skill_list
             if s.id not in rejected_ids and s.id not in promoted_ids
         ]
         if pending_skills:
-            pending[category] = pending_skills
+            pending[category] = pending_skills  # type: ignore[assignment]
             total_pending += len(pending_skills)
 
     return DiffResult(

buildlog/distill.py

@@ -17,7 +17,7 @@ import logging
 import re
 from collections.abc import Iterator
 from dataclasses import dataclass, field
-from datetime import UTC, date, datetime
+from datetime import date, datetime, timezone
 from pathlib import Path
 from typing import Final, Literal, TypedDict
 
@@ -81,7 +81,7 @@ class DistillResult:
     extracted_at: str
     entry_count: int
     patterns: dict[str, list[PatternDict]] = field(default_factory=dict)
-    statistics: StatisticsDict = field(default_factory=dict)
+    statistics: StatisticsDict = field(default_factory=dict)  # type: ignore[assignment]
 
     def to_dict(self) -> DistillResultDict:
         """Convert to dictionary for JSON/YAML serialization."""
@@ -333,7 +333,7 @@ def distill_all(
     statistics = _compute_statistics(patterns, by_month)
 
     return DistillResult(
-        extracted_at=datetime.now(UTC).isoformat().replace("+00:00", "Z"),
+        extracted_at=datetime.now(timezone.utc).isoformat().replace("+00:00", "Z"),
         entry_count=entry_count,
         patterns=patterns,
         statistics=statistics,

buildlog/embeddings.py

@@ -46,20 +46,112 @@ Embedding = list[float]
 BackendName = Literal["token", "sentence-transformers", "openai"]
 
 # Stop words to filter in token-based approach
-STOP_WORDS: Final[frozenset[str]] = frozenset({
-    "a", "an", "the", "is", "are", "was", "were", "be", "been", "being",
-    "have", "has", "had", "do", "does", "did", "will", "would", "could",
-    "should", "may", "might", "must", "shall", "can", "need", "dare",
-    "ought", "used", "to", "of", "in", "for", "on", "with", "at", "by",
-    "from", "as", "into", "through", "during", "before", "after", "above",
-    "below", "between", "under", "again", "further", "then", "once",
-    "here", "there", "when", "where", "why", "how", "all", "each", "few",
-    "more", "most", "other", "some", "such", "no", "nor", "not", "only",
-    "own", "same", "so", "than", "too", "very", "just", "also", "now",
-    "always", "never", "often", "still", "already", "ever",
-    "it", "its", "this", "that", "these", "those", "i", "you", "he",
-    "she", "we", "they", "what", "which", "who", "whom", "whose",
-})
+STOP_WORDS: Final[frozenset[str]] = frozenset(
+    {
+        "a",
+        "an",
+        "the",
+        "is",
+        "are",
+        "was",
+        "were",
+        "be",
+        "been",
+        "being",
+        "have",
+        "has",
+        "had",
+        "do",
+        "does",
+        "did",
+        "will",
+        "would",
+        "could",
+        "should",
+        "may",
+        "might",
+        "must",
+        "shall",
+        "can",
+        "need",
+        "dare",
+        "ought",
+        "used",
+        "to",
+        "of",
+        "in",
+        "for",
+        "on",
+        "with",
+        "at",
+        "by",
+        "from",
+        "as",
+        "into",
+        "through",
+        "during",
+        "before",
+        "after",
+        "above",
+        "below",
+        "between",
+        "under",
+        "again",
+        "further",
+        "then",
+        "once",
+        "here",
+        "there",
+        "when",
+        "where",
+        "why",
+        "how",
+        "all",
+        "each",
+        "few",
+        "more",
+        "most",
+        "other",
+        "some",
+        "such",
+        "no",
+        "nor",
+        "not",
+        "only",
+        "own",
+        "same",
+        "so",
+        "than",
+        "too",
+        "very",
+        "just",
+        "also",
+        "now",
+        "always",
+        "never",
+        "often",
+        "still",
+        "already",
+        "ever",
+        "it",
+        "its",
+        "this",
+        "that",
+        "these",
+        "those",
+        "i",
+        "you",
+        "he",
+        "she",
+        "we",
+        "they",
+        "what",
+        "which",
+        "who",
+        "whom",
+        "whose",
+    }
+)
 
 # Common synonyms for normalization
 SYNONYMS: Final[dict[str, str]] = {
@@ -249,7 +341,7 @@ class SentenceTransformerBackend(EmbeddingBackend):
                     "Install with: pip install buildlog[embeddings]"
                 ) from e
 
-            self._model = SentenceTransformer(self._model_name)
+            self._model = SentenceTransformer(self._model_name)  # type: ignore[assignment]
 
         return self._model
 
@@ -298,7 +390,7 @@ class OpenAIBackend(EmbeddingBackend):
                     "Install with: pip install openai"
                 ) from e
 
-            self._client = openai.OpenAI()
+            self._client = openai.OpenAI()  # type: ignore[assignment]
 
         return self._client
 

buildlog/mcp/tools.py

@@ -39,17 +39,17 @@ def buildlog_status(
 
 def buildlog_promote(
     skill_ids: list[str],
-    target: Literal["claude_md", "settings_json"] = "claude_md",
+    target: Literal["claude_md", "settings_json", "skill"] = "claude_md",
     buildlog_dir: str = "buildlog",
 ) -> dict:
     """Promote skills to your agent's rules.
 
-    Writes selected skills to CLAUDE.md or .claude/settings.json
-    so your AI agent will follow these patterns.
+    Writes selected skills to CLAUDE.md, .claude/settings.json, or
+    .claude/skills/buildlog-learned/SKILL.md (Anthropic Agent Skills format).
 
     Args:
         skill_ids: List of skill IDs to promote (e.g., ["arch-b0fcb62a1e"])
-        target: Where to write rules ("claude_md" or "settings_json")
+        target: Where to write rules ("claude_md", "settings_json", or "skill")
         buildlog_dir: Path to buildlog directory
 
     Returns:

buildlog/render/__init__.py

@@ -3,39 +3,62 @@
 from __future__ import annotations
 
 from pathlib import Path
-from typing import Literal
+from typing import TYPE_CHECKING, Literal
 
 from buildlog.render.base import RenderTarget
 from buildlog.render.claude_md import ClaudeMdRenderer
 from buildlog.render.settings_json import SettingsJsonRenderer
+from buildlog.render.skill import SkillRenderer
+
+if TYPE_CHECKING:
+    from typing import Any
 
 __all__ = [
     "RenderTarget",
     "ClaudeMdRenderer",
     "SettingsJsonRenderer",
+    "SkillRenderer",
     "get_renderer",
+    "RENDERERS",
 ]
 
+# Registry of available renderers
+# Using RenderTarget Protocol allows easy extension without modifying types
+RENDERERS: dict[str, type[RenderTarget]] = {
+    "claude_md": ClaudeMdRenderer,
+    "settings_json": SettingsJsonRenderer,
+    "skill": SkillRenderer,
+}
+
 
 def get_renderer(
-    target: Literal["claude_md", "settings_json"],
+    target: Literal["claude_md", "settings_json", "skill"],
     path: Path | None = None,
-) -> ClaudeMdRenderer | SettingsJsonRenderer:
+    **kwargs: Any,
+) -> RenderTarget:
     """Get renderer for target.
 
     Args:
-        target: Target format - "claude_md" or "settings_json".
+        target: Target format - "claude_md", "settings_json", or "skill".
         path: Optional custom path for the target file.
+        **kwargs: Additional arguments passed to the renderer constructor.
+            Common kwargs (accepted by all renderers):
+                - tracking_path: Path to promoted.json for tracking promoted IDs.
+            Skill-specific kwargs:
+                - skill_name: Name of the skill directory (default: "buildlog-learned").
+                  Must not contain path separators.
 
     Returns:
-        Renderer instance.
+        Renderer instance implementing RenderTarget protocol.
 
     Raises:
         ValueError: If target is not recognized.
     """
-    if target == "claude_md":
-        return ClaudeMdRenderer(path=path)
-    elif target == "settings_json":
-        return SettingsJsonRenderer(path=path)
-    else:
-        raise ValueError(f"Unknown render target: {target}. Must be 'claude_md' or 'settings_json'")
+    if target not in RENDERERS:
+        available = ", ".join(f"'{k}'" for k in RENDERERS.keys())
+        raise ValueError(
+            f"Unknown render target: '{target}'. Must be one of: {available}"
+        )
+
+    renderer_cls = RENDERERS[target]
+    return renderer_cls(path=path, **kwargs)  # type: ignore[call-arg]

buildlog/render/claude_md.py

@@ -2,11 +2,11 @@
 
 from __future__ import annotations
 
-import json
 from datetime import datetime
 from pathlib import Path
 from typing import TYPE_CHECKING
 
+from buildlog.render.tracking import track_promoted
 from buildlog.skills import _to_imperative
 
 if TYPE_CHECKING:
@@ -79,28 +79,7 @@ class ClaudeMdRenderer:
         else:
             self.path.write_text(content)
 
-        # Track promoted skill IDs
-        self._track_promoted(skills)
+        # Track promoted skill IDs using shared utility
+        track_promoted(skills, self.tracking_path)
 
         return f"Appended {len(skills)} rules to {self.path}"
-
-    def _track_promoted(self, skills: list[Skill]) -> None:
-        """Track which skills have been promoted."""
-        self.tracking_path.parent.mkdir(parents=True, exist_ok=True)
-
-        # Load existing tracking data (handle corrupt JSON)
-        tracking = {"skill_ids": [], "promoted_at": {}}
-        if self.tracking_path.exists():
-            try:
-                tracking = json.loads(self.tracking_path.read_text())
-            except json.JSONDecodeError:
-                pass  # Start fresh if corrupted
-
-        # Add new skill IDs
-        now = datetime.now().isoformat()
-        for skill in skills:
-            if skill.id not in tracking["skill_ids"]:
-                tracking["skill_ids"].append(skill.id)
-                tracking["promoted_at"][skill.id] = now
-
-        self.tracking_path.write_text(json.dumps(tracking, indent=2))

buildlog/render/settings_json.py

@@ -7,6 +7,7 @@ from datetime import datetime
 from pathlib import Path
 from typing import TYPE_CHECKING
 
+from buildlog.render.tracking import track_promoted
 from buildlog.skills import _to_imperative
 
 if TYPE_CHECKING:
@@ -69,28 +70,7 @@ class SettingsJsonRenderer:
         # Write back
         self.path.write_text(json.dumps(settings, indent=2))
 
-        # Track promoted skill IDs
-        self._track_promoted(skills)
+        # Track promoted skill IDs using shared utility
+        track_promoted(skills, self.tracking_path)
 
         return f"Added {added} rules to {self.path} ({len(skills) - added} duplicates skipped)"
-
-    def _track_promoted(self, skills: list[Skill]) -> None:
-        """Track which skills have been promoted."""
-        self.tracking_path.parent.mkdir(parents=True, exist_ok=True)
-
-        # Load existing tracking data (handle corrupt JSON)
-        tracking = {"skill_ids": [], "promoted_at": {}}
-        if self.tracking_path.exists():
-            try:
-                tracking = json.loads(self.tracking_path.read_text())
-            except json.JSONDecodeError:
-                pass  # Start fresh if corrupted
-
-        # Add new skill IDs
-        now = datetime.now().isoformat()
-        for skill in skills:
-            if skill.id not in tracking["skill_ids"]:
-                tracking["skill_ids"].append(skill.id)
-                tracking["promoted_at"][skill.id] = now
-
-        self.tracking_path.write_text(json.dumps(tracking, indent=2))