@pennyfarthing/core 10.1.0 → 10.3.0

package/pennyfarthing_scripts/complexity/analyze.py

@@ -13,7 +13,7 @@ import json
 import re
 from pathlib import Path
 
-from pennyfarthing_scripts.complexity.models import FileComplexity, ComplexityResult
+from pennyfarthing_scripts.complexity.models import ComplexityResult, FileComplexity
 
 # Regex patterns for extracting numeric values from ESLint messages
 COMPLEXITY_RE = re.compile(r"complexity of (\d+)")

package/pennyfarthing_scripts/complexity/cli.py

@@ -9,9 +9,13 @@ from __future__ import annotations
 
 import asyncio
 from pathlib import Path
+from typing import TYPE_CHECKING
 
 import click
 
+if TYPE_CHECKING:
+    from pennyfarthing_scripts.complexity.models import ComplexityResult
+
 
 @click.group()
 def complexity():
@@ -39,7 +43,7 @@ def _common_options(fn):
     return fn
 
 
-def _run_analysis(target_path: str | None, exclude: tuple) -> "ComplexityResult":
+def _run_analysis(target_path: str | None, exclude: tuple) -> ComplexityResult:
     """Run analysis and return result."""
     from pennyfarthing_scripts.complexity.analyze import analyze_complexity
 

package/pennyfarthing_scripts/complexity/formatters.py

@@ -11,7 +11,7 @@ import io
 import json
 from dataclasses import asdict
 
-from pennyfarthing_scripts.complexity.models import FileComplexity, ComplexityResult
+from pennyfarthing_scripts.complexity.models import ComplexityResult, FileComplexity
 
 
 def format_file_table(files: list[FileComplexity], top_n: int = 20) -> str:

package/pennyfarthing_scripts/context.py

@@ -9,7 +9,6 @@ import os
 import time
 from dataclasses import dataclass
 from pathlib import Path
-from typing import Optional
 
 try:
     import yaml
@@ -45,8 +44,8 @@ class ContextResult:
 
     # Status
     status: str = "OK"  # OK, HIGH
-    warning: Optional[str] = None  # None, High, Critical
-    recommendation: Optional[str] = None
+    warning: str | None = None  # None, High, Critical
+    recommendation: str | None = None
 
     # Mode settings
     permission_mode: str = "manual"
@@ -56,7 +55,7 @@ class ContextResult:
     is_cyclist: bool = False
 
     # Error state
-    error: Optional[str] = None
+    error: str | None = None
 
     def to_env_vars(self) -> str:
         """Output as shell environment variables."""
@@ -111,7 +110,7 @@ class ContextResult:
         return "\n".join(lines)
 
 
-def load_config(project_dir: Optional[str] = None) -> ContextConfig:
+def load_config(project_dir: str | None = None) -> ContextConfig:
     """Load context configuration from config files.
 
     Checks .pennyfarthing/config.local.yaml first, falls back to
@@ -166,7 +165,7 @@ def _apply_config(config: ContextConfig, data: dict) -> None:
         config.relay_mode = wf.get("relay_mode", False) is True
 
 
-def get_claude_project_path(project_dir: Optional[str] = None) -> Path:
+def get_claude_project_path(project_dir: str | None = None) -> Path:
     """Get the Claude Code project path for transcripts.
 
     Claude Code stores transcripts at ~/.claude/projects/<path-with-dashes>
@@ -184,10 +183,10 @@ def get_claude_project_path(project_dir: Optional[str] = None) -> Path:
 
 def find_transcript(
     project_path: Path,
-    explicit_session: Optional[str] = None,
-    session_id_env: Optional[str] = None,
+    explicit_session: str | None = None,
+    session_id_env: str | None = None,
     stale_threshold_seconds: int = 60,
-) -> Optional[Path]:
+) -> Path | None:
     """Find the appropriate transcript file.
 
     Priority:
@@ -208,7 +207,7 @@ def find_transcript(
         return None
 
     # Helper to find most recent transcript
-    def most_recent() -> Optional[Path]:
+    def most_recent() -> Path | None:
         transcripts = sorted(
             [f for f in project_path.glob("*.jsonl") if "agent-" not in f.name],
             key=lambda f: f.stat().st_mtime,
@@ -236,7 +235,7 @@ def find_transcript(
     return most_recent()
 
 
-def parse_transcript(transcript_path: Path) -> tuple[Optional[int], Optional[int]]:
+def parse_transcript(transcript_path: Path) -> tuple[int | None, int | None]:
     """Parse transcript for first and last usage totals.
 
     Returns:
@@ -265,7 +264,7 @@ def parse_transcript(transcript_path: Path) -> tuple[Optional[int], Optional[int
     return first_total, last_total
 
 
-def detect_cyclist(project_dir: Optional[str] = None) -> bool:
+def detect_cyclist(project_dir: str | None = None) -> bool:
     """Detect if running inside Cyclist.
 
     Checks:
@@ -300,7 +299,7 @@ def detect_cyclist(project_dir: Optional[str] = None) -> bool:
                     result = s.connect_ex(("127.0.0.1", port))
                     if result == 0:
                         return True
-            except (ValueError, OSError, socket.error):
+            except (ValueError, OSError):
                 # Port file invalid or port not responding
                 continue
 
@@ -308,8 +307,8 @@ def detect_cyclist(project_dir: Optional[str] = None) -> bool:
 
 
 def check_context(
-    explicit_session: Optional[str] = None,
-    project_dir: Optional[str] = None,
+    explicit_session: str | None = None,
+    project_dir: str | None = None,
 ) -> ContextResult:
     """Check current context usage.
 

package/pennyfarthing_scripts/deadcode/analyze.py

@@ -9,10 +9,9 @@ from __future__ import annotations
 
 import asyncio
 import fnmatch
-from datetime import datetime, timezone
-from pathlib import Path
-
 import re
+from datetime import UTC, datetime
+from pathlib import Path
 
 from pennyfarthing_scripts.deadcode.models import (
     DeadCodeResult,
@@ -156,7 +155,7 @@ async def find_stale_files(
         filtered.append(fpath)
 
     # Enrich each stale file
-    now = datetime.now(timezone.utc)
+    now = datetime.now(UTC)
     stale_files = []
     for fpath in sorted(filtered):
         # Get last commit date

package/pennyfarthing_scripts/deadcode/cli.py

@@ -41,8 +41,8 @@ def _common_options(fn):
 
 def _run_analysis(repo: str | None, repo_path: str | None, days: int, exclude: tuple, branch: str):
     """Run analysis and return result."""
-    from pennyfarthing_scripts.deadcode.analyze import analyze_repo, find_stale_files
     from pennyfarthing_scripts.common.config import get_project_root
+    from pennyfarthing_scripts.deadcode.analyze import analyze_repo
 
     excludes = list(exclude) if exclude else None
 
@@ -109,8 +109,8 @@ def _exports_options(fn):
 
 def _run_exports_analysis(repo: str | None, repo_path: str | None):
     """Run unused export analysis and return result."""
-    from pennyfarthing_scripts.deadcode.analyze import find_unused_exports
     from pennyfarthing_scripts.common.config import get_project_root
+    from pennyfarthing_scripts.deadcode.analyze import find_unused_exports
 
     if repo_path:
         p = Path(repo_path).resolve()

package/pennyfarthing_scripts/dependencies/__init__.py

@@ -5,12 +5,12 @@ Wraps npm outdated and npm audit to detect outdated packages
 and security vulnerabilities.
 """
 
+from pennyfarthing_scripts.dependencies.analyze import analyze_dependencies
 from pennyfarthing_scripts.dependencies.models import (
+    DependenciesResult,
     OutdatedPackage,
     SecurityAdvisory,
-    DependenciesResult,
 )
-from pennyfarthing_scripts.dependencies.analyze import analyze_dependencies
 
 __all__ = [
     "OutdatedPackage",

package/pennyfarthing_scripts/dependencies/analyze.py

@@ -14,9 +14,9 @@ from collections import Counter
 from pathlib import Path
 
 from pennyfarthing_scripts.dependencies.models import (
+    DependenciesResult,
     OutdatedPackage,
     SecurityAdvisory,
-    DependenciesResult,
 )
 
 

package/pennyfarthing_scripts/dependencies/cli.py

@@ -9,9 +9,13 @@ from __future__ import annotations
 
 import asyncio
 from pathlib import Path
+from typing import TYPE_CHECKING
 
 import click
 
+if TYPE_CHECKING:
+    from pennyfarthing_scripts.dependencies.models import DependenciesResult
+
 
 @click.group()
 def dependencies():
@@ -24,7 +28,7 @@ def dependencies():
     pass
 
 
-def _run_analysis(target_path: str | None) -> "DependenciesResult":
+def _run_analysis(target_path: str | None) -> DependenciesResult:
     """Run analysis and return result."""
     from pennyfarthing_scripts.dependencies.analyze import analyze_dependencies
 
@@ -35,10 +39,10 @@ def _run_analysis(target_path: str | None) -> "DependenciesResult":
 def _output_result(result, fmt: str, output_file: str | None):
     """Format and output the analysis result."""
     from pennyfarthing_scripts.dependencies.formatters import (
-        format_outdated_table,
-        format_audit_table,
-        export_json,
         export_csv,
+        export_json,
+        format_audit_table,
+        format_outdated_table,
     )
 
     if fmt == "json":

package/pennyfarthing_scripts/dependencies/formatters.py

@@ -12,9 +12,9 @@ import json
 from dataclasses import asdict
 
 from pennyfarthing_scripts.dependencies.models import (
+    DependenciesResult,
     OutdatedPackage,
     SecurityAdvisory,
-    DependenciesResult,
 )
 
 

package/pennyfarthing_scripts/git/__init__.py

@@ -8,15 +8,15 @@ This package provides async git operations for multi-repo management:
 - create_branches: Create feature branches across repos in parallel
 """
 
+from pennyfarthing_scripts.git.create_branches import (
+    BranchResult,
+    create_feature_branches,
+)
 from pennyfarthing_scripts.git.status_all import (
     RepoStatus,
-    get_all_repo_status,
     format_status_brief,
     format_status_full,
-)
-from pennyfarthing_scripts.git.create_branches import (
-    BranchResult,
-    create_feature_branches,
+    get_all_repo_status,
 )
 
 __all__ = [

package/pennyfarthing_scripts/git/create_branches.py

@@ -14,10 +14,11 @@ Features:
 """
 
 import asyncio
+from collections.abc import Sequence
 from dataclasses import dataclass
 from enum import Enum
 from pathlib import Path
-from typing import Literal, Sequence
+from typing import Literal
 
 
 class BranchAction(Enum):
@@ -296,7 +297,7 @@ def format_results(results: Sequence[BranchResult], branch_name: str) -> str:
         Multi-line formatted string with results and verification
     """
     lines = []
-    lines.append(f"🌿 Creating/checking out feature branches...")
+    lines.append("🌿 Creating/checking out feature branches...")
     lines.append(f"   Branch: {branch_name}")
     lines.append("")
 

package/pennyfarthing_scripts/git/status_all.py

@@ -13,9 +13,9 @@ Features:
 """
 
 import asyncio
+from collections.abc import Sequence
 from dataclasses import dataclass
 from pathlib import Path
-from typing import Sequence
 
 
 @dataclass

package/pennyfarthing_scripts/healthscore/__init__.py

@@ -6,12 +6,12 @@ churn, TODO density, complexity, test gaps, dead code,
 deprecation debt, dependency freshness, agent context efficiency.
 """
 
+from pennyfarthing_scripts.healthscore.analyze import analyze_healthscore
 from pennyfarthing_scripts.healthscore.models import (
+    DEFAULT_WEIGHTS,
     DimensionScore,
     HealthscoreResult,
-    DEFAULT_WEIGHTS,
 )
-from pennyfarthing_scripts.healthscore.analyze import analyze_healthscore
 
 __all__ = [
     "DimensionScore",

package/pennyfarthing_scripts/healthscore/__main__.py

@@ -1,6 +1,14 @@
 """Allow running as: python -m pennyfarthing_scripts.healthscore"""
 
+import logging
+import sys
+
 from pennyfarthing_scripts.healthscore.cli import healthscore
 
 if __name__ == "__main__":
+    logging.basicConfig(
+        level=logging.INFO,
+        format="%(message)s",
+        stream=sys.stderr,
+    )
     healthscore()