@pennyfarthing/core 11.3.8 → 11.5.0-alpha.0

package/pennyfarthing-dist/pf/git/hooks_installer.py

@@ -10,9 +10,10 @@ Usage via CLI:
 
 from __future__ import annotations
 
+import os
 from pathlib import Path
 
-from pf.common.config import get_project_root
+from pf.common.config import get_dist_root, get_project_root
 
 DISPATCHER_MARKER = "pennyfarthing-dispatcher"
 PF_PREFIX = "10"
@@ -48,9 +49,9 @@ def install_git_hooks(project_root: Path | None = None) -> int:
     if project_root is None:
         project_root = get_project_root()
 
-    pf_dist = project_root / "pennyfarthing-dist"
-    if not pf_dist.is_dir():
-        print("Error: This script requires pennyfarthing-dist/ at the project root")
+    pf_dist = get_dist_root(project_root=project_root)
+    if pf_dist is None or not pf_dist.is_dir():
+        print("Error: pennyfarthing-dist not found")
         print("       End-user projects should use: pennyfarthing init")
         return 1
 
@@ -127,8 +128,8 @@ def install_git_hooks(project_root: Path | None = None) -> int:
             print(f"  NEW  {dest_name} dispatcher")
 
         # Symlink pennyfarthing hook into .d/
-        # Relative path from .git/hooks/{hook}.d/ to pennyfarthing-dist/scripts/hooks/
-        relative_path = Path("../../../pennyfarthing-dist/scripts/hooks") / source_file
+        # Compute relative path from .git/hooks/{hook}.d/ to the hooks source
+        relative_path = Path(os.path.relpath(hooks_source / source_file, d_dir))
 
         if pf_hook_path.is_symlink():
             current_target = pf_hook_path.readlink()

package/pennyfarthing-dist/pf/handoff/gate_file.py

@@ -14,6 +14,8 @@ from __future__ import annotations
 
 from pathlib import Path
 
+from pf.common.config import get_dist_root
+
 
 def resolve_gate_file(
     gate_ref: str,
@@ -44,8 +46,10 @@ def resolve_gate_file(
     # Resolution order: local first, built-in fallback
     search_paths = [
         project_root / ".pennyfarthing" / "gates" / f"{name}.md",
-        project_root / "pennyfarthing-dist" / "gates" / f"{name}.md",
     ]
+    dist_root = get_dist_root(project_root=project_root)
+    if dist_root:
+        search_paths.append(dist_root / "gates" / f"{name}.md")
 
     for candidate in search_paths:
         if candidate.is_file():

package/pennyfarthing-dist/pf/hooks/statusline.py

@@ -256,6 +256,17 @@ def _get_character_display(project_root: str, agent_name: str) -> tuple[str, str
         return "", None
 
     theme_file = Path(project_root) / ".pennyfarthing" / "personas" / "themes" / f"{theme}.yaml"
+    if not theme_file.is_file():
+        # Fallback: check pennyfarthing-dist via get_dist_root (npm context)
+        try:
+            from pf.common.config import get_dist_root
+            dist_root = get_dist_root(project_root=Path(project_root))
+            if dist_root:
+                candidate = dist_root / "personas" / "themes" / f"{theme}.yaml"
+                if candidate.is_file():
+                    theme_file = candidate
+        except Exception:
+            pass
     if not theme_file.is_file():
         # Capitalize theme name as fallback
         return theme[0].upper() + theme[1:] if theme else "", str(theme_file)

package/pennyfarthing-dist/pf/jira/__init__.py

@@ -45,18 +45,15 @@ from pf.jira.client import (
     # Classes
     JiraClient,
     # Functions
-    add_comment,
     check_dependencies,
     extract_jira_key,
     get_client,
-    get_issue,
     get_jira_field,
     get_story_points,
     is_jira_cli_available,
     map_github_to_jira,
     map_jira_to_status,
     map_status_to_jira,
-    update_issue_status,
 )
 
 __all__ = [
@@ -69,18 +66,15 @@ __all__ = [
     # Classes
     "JiraClient",
     # Functions
-    "add_comment",
     "check_dependencies",
     "extract_jira_key",
     "get_client",
-    "get_issue",
     "get_jira_field",
     "get_story_points",
     "is_jira_cli_available",
     "map_github_to_jira",
     "map_jira_to_status",
     "map_status_to_jira",
-    "update_issue_status",
     # Submodules
     "bidirectional",
     "claim",

package/pennyfarthing-dist/pf/jira/claim.py

@@ -19,10 +19,8 @@ import sys
 from typing import Any
 
 from pf.jira.client import (
-    get_issue,
+    get_client,
     get_jira_field,
-    is_jira_cli_available,
-    update_issue_status,
 )
 
 
@@ -57,14 +55,7 @@ def check_availability(issue_key: str) -> dict[str, Any]:
     Returns:
         Dict with available status and details
     """
-    if not is_jira_cli_available():
-        return {
-            "available": False,
-            "error": "Jira CLI not installed",
-            "exit_code": 3,
-        }
-
-    issue = get_issue(issue_key)
+    issue = get_client().get_issue_sync(issue_key)
     if not issue:
         return {
             "available": False,
@@ -102,15 +93,6 @@ def claim_story(issue_key: str) -> dict[str, Any]:
     Returns:
         Dict with success status and details
     """
-    import subprocess
-
-    if not is_jira_cli_available():
-        return {
-            "success": False,
-            "error": "Jira CLI not installed",
-            "exit_code": 3,
-        }
-
     # Check availability first
     availability = check_availability(issue_key)
     if not availability["available"]:
@@ -120,39 +102,27 @@ def claim_story(issue_key: str) -> dict[str, Any]:
             "exit_code": 1,
         }
 
+    from pf.jira.client import get_current_user_email
+
     actions = []
     errors = []
 
-    # Get current user
-    result = subprocess.run(
-        ["jira", "me"],
-        capture_output=True,
-        text=True,
-    )
-    if result.returncode != 0:
-        return {
-            "success": False,
-            "error": "Could not get current Jira user",
-            "exit_code": 3,
-        }
-    current_user = result.stdout.strip()
+    current_user = get_current_user_email()
+    client = get_client()
 
-    # Assign to self
-    result = subprocess.run(
-        ["jira", "issue", "assign", issue_key, current_user, "--project", "MSSCI"],
-        capture_output=True,
-        text=True,
-    )
-    if result.returncode == 0:
+    # Assign to self via REST API
+    assign_result = client.assign_issue_sync(issue_key, current_user)
+    if assign_result.get("success"):
         actions.append(f"Assigned to {current_user}")
     else:
-        errors.append(f"Failed to assign: {result.stderr}")
+        errors.append(f"Failed to assign: {assign_result.get('error', 'unknown')}")
 
-    # Move to In Progress
-    if update_issue_status(issue_key, "In Progress"):
+    # Move to In Progress via REST API
+    transition_result = client.transition_sync(issue_key, "In Progress")
+    if transition_result.get("success"):
         actions.append("Moved to In Progress")
     else:
-        errors.append("Failed to move to In Progress")
+        errors.append(f"Failed to move to In Progress: {transition_result.get('error', 'unknown')}")
 
     if errors:
         return {