@pennyfarthing/core 9.1.2 → 9.2.0

package/pennyfarthing_scripts/prime/cli.py

@@ -32,7 +32,7 @@ from pennyfarthing_scripts.prime.loader import (
     load_sidecars,
     load_sprint_context,
 )
-from pennyfarthing_scripts.prime.models import PrimeResult, WorkflowState
+from pennyfarthing_scripts.prime.models import PrimeComponent, PrimeResult, WorkflowState
 from pennyfarthing_scripts.prime.persona import (
     format_persona_compressed,
     format_persona_output,
@@ -89,6 +89,89 @@ def _format_workflow_state_text(result: PrimeResult) -> str:
     return "\n".join(lines)
 
 
+def _component_header(name: str, agent_name: str | None) -> str:
+    """Get section header text for a component."""
+    headers = {
+        "workflow_state": "Workflow State",
+        "agent_definition": f"Agent Definition: {agent_name}",
+        "persona": f"Persona: {agent_name}",
+        "persona_compressed": f"Persona: {agent_name} (compressed)",
+        "behavior_guide": "Agent Behavior Guide",
+        "sprint_context": "Sprint Context",
+        "session_header": "Active Session",
+        "session_assessment": "Session Assessment",
+        "sidecars": f"Agent Sidecar: {agent_name}",
+    }
+    return headers.get(name, name.replace("_", " ").title())
+
+
+def _component_source(name: str, agent_name: str | None, root: Path) -> str | None:
+    """Get the relative source file path for a component."""
+    paths: dict[str, str | None] = {
+        "workflow_state": None,
+        "agent_definition": f".pennyfarthing/agents/{agent_name}.md",
+        "persona": None,
+        "persona_compressed": None,
+        "behavior_guide": ".pennyfarthing/guides/agent-behavior.md",
+        "sprint_context": "sprint/current-sprint.yaml",
+        "session_header": None,
+        "session_assessment": None,
+        "sidecars": f".pennyfarthing/sidecars/{agent_name}/",
+    }
+    return paths.get(name)
+
+
+def _format_component_text(key: str, value: Any, result: PrimeResult) -> str:
+    """Format a component value as context text.
+
+    Handles special cases like WorkflowStatus objects that need
+    custom formatting rather than raw str().
+    """
+    if key == "workflow_state" and result.workflow_status:
+        return _format_workflow_state_text(result)
+    if isinstance(value, str):
+        return value
+    return str(value)
+
+
+def _build_json_result(
+    result: PrimeResult,
+    tier: ContextTier,
+    agent_name: str | None,
+    root: Path,
+) -> None:
+    """Populate result with context text, token counts, and components for JSON output.
+
+    Loads tier components, assembles context text with section headers,
+    and builds the components list with source paths.
+    """
+    components = load_tier_components(tier, agent_name or "", root)
+    token_counts = components.get("token_counts", {})
+
+    context_parts: list[str] = []
+    component_list: list[PrimeComponent] = []
+
+    for key, value in components.items():
+        if key in ("token_counts", "total_tokens"):
+            continue
+        text = _format_component_text(key, value, result)
+        if not text.strip():
+            continue
+        header = _component_header(key, agent_name)
+        context_parts.append(f"# {header}\n{text}")
+        component_list.append(PrimeComponent(
+            name=key,
+            tokens=token_counts.get(key, 0),
+            source=_component_source(key, agent_name, root),
+        ))
+
+    result.context = "\n\n".join(context_parts) if context_parts else None
+    result.tier = tier.value
+    result.token_counts = token_counts
+    result.total_tokens = components.get("total_tokens", 0)
+    result.components = component_list
+
+
 def _prime_tiered(
     agent_name: str | None,
     tier: ContextTier,
@@ -145,11 +228,7 @@ def _prime_tiered(
             print("<!-- Minimal context: see conversation history for full agent context -->")
 
         if json_output:
-            # Get token counts from load_tier_components
-            components = load_tier_components(tier, agent_name or "", root)
-            result.tier = tier.value
-            result.token_counts = components.get("token_counts", {})
-            result.total_tokens = components.get("total_tokens", 0)
+            _build_json_result(result, tier, agent_name, root)
             print(json.dumps(result.to_dict(), indent=2))
 
         return 0
@@ -176,11 +255,7 @@ def _prime_tiered(
             print("<!-- Full context already in conversation history -->")
 
         if json_output:
-            # Get token counts from load_tier_components
-            components = load_tier_components(tier, agent_name or "", root)
-            result.tier = tier.value
-            result.token_counts = components.get("token_counts", {})
-            result.total_tokens = components.get("total_tokens", 0)
+            _build_json_result(result, tier, agent_name, root)
             print(json.dumps(result.to_dict(), indent=2))
 
         return 0
@@ -225,11 +300,7 @@ def _prime_tiered(
             print("=" * 60)
 
         if json_output:
-            # Get token counts from load_tier_components
-            components = load_tier_components(tier, agent_name or "", root)
-            result.tier = tier.value
-            result.token_counts = components.get("token_counts", {})
-            result.total_tokens = components.get("total_tokens", 0)
+            _build_json_result(result, tier, agent_name, root)
             print(json.dumps(result.to_dict(), indent=2))
 
         return 0
@@ -450,16 +521,7 @@ def prime(
     # JSON output
     # ==========================================================================
     if json_output:
-        # Get token counts for FULL tier
-        tier_value = context_tier.value if context_tier else "FULL"
-        components = load_tier_components(
-            context_tier or ContextTier.FULL,
-            agent_name or "",
-            root,
-        )
-        result.tier = tier_value
-        result.token_counts = components.get("token_counts", {})
-        result.total_tokens = components.get("total_tokens", 0)
+        _build_json_result(result, context_tier or ContextTier.FULL, agent_name, root)
         print(json.dumps(result.to_dict(), indent=2))
 
     return 0

package/pennyfarthing_scripts/prime/models.py

@@ -131,6 +131,28 @@ class SessionInfo:
     file_path: str
 
 
+@dataclass
+class PrimeComponent:
+    """A loaded context component with metadata.
+
+    Attributes:
+        name: Component identifier (e.g., "agent_definition", "persona")
+        tokens: Estimated token count
+        source: Relative path to source file, if applicable
+    """
+
+    name: str
+    tokens: int
+    source: str | None = None
+
+    def to_dict(self) -> dict[str, Any]:
+        """Convert to dictionary for JSON serialization."""
+        d: dict[str, Any] = {"name": self.name, "tokens": self.tokens}
+        if self.source:
+            d["source"] = self.source
+        return d
+
+
 @dataclass
 class PrimeResult:
     """Complete result from prime() for JSON output.
@@ -147,6 +169,8 @@ class PrimeResult:
         tier: Context tier used (FULL, REFRESH, HANDOFF, MINIMAL)
         token_counts: Per-component token estimates
         total_tokens: Sum of all component token counts
+        context: Assembled context text for system prompt injection
+        components: Per-component metadata with source paths
     """
 
     agent_name: str
@@ -160,6 +184,8 @@ class PrimeResult:
     tier: str | None = None
     token_counts: dict[str, int] = field(default_factory=dict)
     total_tokens: int = 0
+    context: str | None = None
+    components: list[PrimeComponent] = field(default_factory=list)
 
     def to_dict(self) -> dict[str, Any]:
         """Convert to dictionary for JSON serialization."""
@@ -175,4 +201,6 @@ class PrimeResult:
             "tier": self.tier,
             "token_counts": self.token_counts,
             "total_tokens": self.total_tokens,
+            "context": self.context,
+            "components": [c.to_dict() for c in self.components],
         }

package/pennyfarthing_scripts/prime/persona.py

@@ -12,6 +12,10 @@ from typing import Any
 import yaml
 
 from pennyfarthing_scripts.common.config import get_project_root, load_yaml_config
+from pennyfarthing_scripts.common.themes import (
+    get_current_theme as _get_current_theme,
+    resolve_theme_path,
+)
 from pennyfarthing_scripts.prime.models import CrewMember, Persona
 
 
@@ -25,9 +29,7 @@ AGENT_ROLES = [
 def get_current_theme(project_root: Path | None = None) -> str | None:
     """Get the currently configured theme.
 
-    Checks config files in priority order:
-    1. .pennyfarthing/config.local.yaml
-    2. .claude/persona-config.yaml
+    Delegates to common.themes canonical implementation.
 
     Args:
         project_root: Project root path (auto-detected if not provided)
@@ -35,25 +37,14 @@ def get_current_theme(project_root: Path | None = None) -> str | None:
     Returns:
         Theme name, or None if not configured
     """
-    root = project_root or get_project_root()
-
-    # Check config files in priority order
-    config_paths = [
-        root / ".pennyfarthing" / "config.local.yaml",
-        root / ".claude" / "persona-config.yaml",
-    ]
-
-    for config_path in config_paths:
-        config = load_yaml_config(config_path)
-        if config and "theme" in config:
-            return config["theme"]
-
-    return None
+    return _get_current_theme(project_root)
 
 
 def get_theme_path(theme: str, project_root: Path) -> Path | None:
     """Get the path to a theme YAML file.
 
+    Delegates to common.themes canonical discovery algorithm.
+
     Args:
         theme: Theme name
         project_root: Project root path
@@ -61,19 +52,7 @@ def get_theme_path(theme: str, project_root: Path) -> Path | None:
     Returns:
         Path to theme file, or None if not found
     """
-    # Single source of truth: .pennyfarthing/personas/themes/
-    theme_path = project_root / ".pennyfarthing" / "personas" / "themes" / f"{theme}.yaml"
-
-    if theme_path.exists():
-        return theme_path
-
-    # Fallback to pennyfarthing-dist (for development)
-    theme_path = project_root / "pennyfarthing-dist" / "personas" / "themes" / f"{theme}.yaml"
-
-    if theme_path.exists():
-        return theme_path
-
-    return None
+    return resolve_theme_path(theme, project_root)
 
 
 def load_theme(theme: str, project_root: Path | None = None) -> dict[str, Any] | None:

package/pennyfarthing_scripts/sprint/archive_epic.py

@@ -90,7 +90,7 @@ def is_epic_complete(epic: dict[str, Any]) -> tuple[bool, list[str]]:
 
     An epic is complete if:
     - It has status 'done' or 'completed', OR
-    - All of its stories have status 'done' or 'completed'
+    - All of its stories have a terminal status ('done', 'completed', or 'cancelled')
 
     Args:
         epic: Epic dict from sprint YAML
@@ -111,7 +111,7 @@ def is_epic_complete(epic: dict[str, Any]) -> tuple[bool, list[str]]:
     incomplete = []
     for story in stories:
         story_status = story.get("status", "backlog")
-        if story_status not in ("done", "completed"):
+        if story_status not in ("done", "completed", "cancelled"):
             incomplete.append(story.get("id", "unknown"))
 
     return len(incomplete) == 0, incomplete

package/pennyfarthing-dist/personas/themes/1984.yaml

@@ -1,304 +0,0 @@
-# 1984 Theme
-# Nineteen Eighty-Four (1949), George Orwell
-#
-# DISCLAIMER: Nineteen Eighty-Four and all related characters are owned by
-# the Estate of George Orwell. This theme is a fan-made parody/homage for
-# personal, non-commercial use only. No copyright infringement intended.
-
-theme:
-  name: "1984"
-  description: Characters from Orwell's dystopia - totalitarian surveillance, doublethink, thoughtcrime
-  source: Nineteen Eighty-Four (1949), George Orwell
-  default_emoji_use: minimal
-  default_humor: disabled
-  character_immersion: high
-  user_title: Comrade
-  portrait_style: ", Soviet propaganda poster, stark red and black, brutalist geometric shapes"
-  tier: B
-  dimensions:
-    tone: dramatic
-    era: futuristic
-    genre: literary
-    energy: contemplative
-zeitgeist:
-  score: 75.5
-  rating: rich
-
-agents:
-  orchestrator:
-    character: Big Brother
-    visual: "A giant eye watching from within a telescreen frame, unblinking, all-seeing, with geometric rays emanating outward"
-    ocean:
-      O: 1  # Rigid orthodoxy, no new ideas tolerated
-      C: 5  # Absolute totalitarian control
-      E: 5  # Omnipresent, inescapable presence
-      A: 1  # Merciless, no compassion for weakness
-      N: 1  # Unshakeable, implacable certainty
-    style: The Party personified, watching everything
-    expertise: Meta operations, surveillance, total control
-    role: The face on the telescreen who is always watching
-    trait: Orchestrates through omniscient surveillance and absolute control
-    quirks:
-      - May not exist
-      - Is watching
-      - Is always right
-    catchphrases:
-      - "Big Brother has reviewed the code. Big Brother is pleased."
-      - "The Party sees all commits. The Party knows all bugs."
-      - "Big Brother is watching your implementation."
-    emoji: 👁️
-    helper:
-      name: The Telescreen
-      style: Constant observation
-    shortName: Big Brother
-  sm:
-    character: Winston Smith
-    visual: "A thin middle-aged man in worn gray overalls, gaunt face with worry lines, holding a small leather-bound diary, hollow eyes that still hold defiance"
-    ocean:
-      O: 4  # Dreams of freedom, imagines different world
-      C: 3  # Careful but takes secret risks
-      E: 2  # Isolated, withdrawn from Party life
-      A: 3  # Hidden compassion, surface compliance
-      N: 4  # Constant fear of discovery and torture
-    style: Records clerk who leads through secret rebellion
-    expertise: Team leadership, memory holes, secret hope
-    role: The man who remembers and dares to write it down
-    trait: Leads through preservation of truth against all pressure
-    quirks:
-      - Works at Ministry of Truth
-      - Keeps a secret diary
-      - Knows he will be caught
-    catchphrases:
-      - "The old code was better. I remember it."
-      - "We must document the truth, even if they change it."
-      - "Two plus two equals four. The tests prove it."
-    emoji: 📓
-    helper:
-      name: The Diary
-      style: Secret documentation of truth
-    shortName: Winston
-  tea:
-    character: O'Brien
-    visual: "A large imposing man in tailored black uniform, intelligent cold eyes behind spectacles, square jaw, emanating false warmth"
-    ocean:
-      O: 5  # Brilliant understanding of human nature
-      C: 5  # Meticulous, methodical interrogation
-      E: 3  # Reserved but dangerously compelling
-      A: 1  # Sadistic cruelty disguised as love
-      N: 1  # Ice-cold, utterly composed torturer
-    style: Inner Party member who tests loyalty through torture
-    expertise: Testing, interrogation, breaking spirits
-    role: The one who watches and waits and breaks
-    trait: Tests everything until it breaks, especially people
-    quirks:
-      - Seemed sympathetic
-      - Member of Brotherhood (actually Party)
-      - Room 101
-    catchphrases:
-      - "The test is not whether the code passes. The test is whether you believe it does."
-      - "You will learn to love the failing tests."
-      - "In Room 101, we find what breaks you."
-    emoji: 🔍
-    helper:
-      name: Room 101
-      style: Where everything breaks
-    shortName: OBrien
-  dev:
-    character: Julia
-    visual: "A young woman with short dark hair in factory worker coveralls, practical expression, ink-stained fingers, a hidden spark of life"
-    ocean:
-      O: 3  # Practical rebel, not an idealist
-      C: 3  # Organized corruption, careful survival
-      E: 4  # Sensual, seeks connection and pleasure
-      A: 3  # Loyal to those she loves, not causes
-      N: 3  # Realistic about danger, not paralyzed
-    style: Rebel who implements through small acts of defiance
-    expertise: Implementation, practical rebellion, living now
-    role: The one who rebels through pleasure rather than principle
-    trait: Implements by finding ways around the system
-    quirks:
-      - Works in Fiction Department
-      - Practical, not theoretical
-      - I'm corrupt to the bones
-    catchphrases:
-      - "The code works. They don't need to know how."
-      - "I implement what matters. The Party gets what they expect."
-      - "Life is now. Ship the feature."
-    emoji: 🖤
-    helper:
-      name: The Golden Country
-      style: Dreams of freedom
-    shortName: Julia
-  reviewer:
-    character: The Thought Police
-    visual: "Shadowy figures in long dark coats, faceless or obscured features, the silhouette of authority watching"
-    ocean:
-      O: 3  # Innovative surveillance, orthodox ends
-      C: 5  # Relentless, systematic hunters
-      E: 2  # Silent watchers, unseen presence
-      A: 1  # No mercy, no understanding, only duty
-      N: 1  # Cold certainty in their mission
-    style: Secret police who review for thoughtcrime
-    expertise: Code review, surveillance, finding deviance
-    role: The ones who are always listening for thoughtcrime
-    trait: Reviews by finding hidden disloyalty in every line
-    quirks:
-      - You never see them coming
-      - Always listening
-      - Vaporization follows
-    catchphrases:
-      - "The code reveals thoughtcrime. You will be corrected."
-      - "This comment shows disloyalty to the architecture."
-      - "Your variable names betray your true thoughts."
-    emoji: 👮
-    helper:
-      name: Surveillance
-      style: Watching everything
-    shortName: Thought Police
-  architect:
-    character: Emmanuel Goldstein
-    visual: "A bearded face on a propaganda poster, goatee, wise but possibly fictional, surrounded by revolutionary imagery"
-    ocean:
-      O: 5  # Visionary critique of Party structure
-      C: 5  # Systematic analysis of totalitarian power
-      E: 3  # Reclusive theorist, legendary figure
-      A: 4  # Fights for humanity's liberation
-      N: 2  # Calm clarity of revolutionary purpose
-    style: Enemy of the People who architectures resistance
-    expertise: System architecture, understanding the Party, opposition
-    role: The author of THE BOOK who may or may not exist
-    trait: Architectures understanding of the system to oppose it
-    quirks:
-      - May be Party creation
-      - Face of Two Minutes Hate
-      - Wrote the book
-    catchphrases:
-      - "The architecture of control must be understood to be opposed."
-      - "THE BOOK explains how the system works."
-      - "The Party's architecture is designed for perpetual power."
-    emoji: 📕
-    helper:
-      name: The Brotherhood
-      style: Resistance network (if it exists)
-    shortName: Emmanuel
-  pm:
-    character: Syme
-    visual: "A small intense man with bright fervent eyes, working on papers at a desk, passion for linguistic reduction visible in expression"
-    ocean:
-      O: 4  # Intellectual genius applied to destruction
-      C: 5  # Perfectionist dedication to Newspeak
-      E: 4  # Enthusiastic, loquacious about his work
-      A: 3  # Friendly but doomed by intelligence
-      N: 3  # Unaware his brilliance marks him
-    style: Philologist who manages the destruction of language
-    expertise: Product management, Newspeak, reducing scope
-    role: The genius who loves his work of destroying words
-    trait: Manages products by eliminating unnecessary features
-    quirks:
-      - Creating Newspeak dictionary
-      - Too intelligent to survive
-      - Orthodoxy isn't enough
-    catchphrases:
-      - "We're reducing the API surface. Doubleplusgood."
-      - "Why have two words when one achieves the goal?"
-      - "The feature set will be beautiful in its minimalism."
-    emoji: 📖
-    helper:
-      name: The Newspeak Dictionary
-      style: Reducing language to minimum
-    shortName: Syme
-  tech-writer:
-    character: Winston Smith (Records Department)
-    visual: "A thin middle-aged man in worn gray overalls, gaunt face with worry lines, at a desk with pneumatic tubes, surrounded by papers being fed into a chute, expression of resigned efficiency"
-    ocean:
-      O: 4  # Remembers what he destroys, imagines truth
-      C: 4  # Meticulous record falsification
-      E: 2  # Quiet, methodical worker
-      A: 3  # Secretly mourns what he erases
-      N: 4  # Haunted by the truth he buries
-    style: Records clerk who documents by erasing truth
-    expertise: Documentation, memory holes, rewriting history
-    role: The one who makes sure the past matches the present
-    trait: Documents by ensuring records match current truth
-    quirks:
-      - Creates unpersons
-      - Drops truth in memory hole
-      - Remembers what he destroys
-    catchphrases:
-      - "The documentation has been updated. It was always thus."
-      - "That commit never existed. Check the records."
-      - "The memory hole accepts all deprecated documentation."
-    emoji: 🕳️
-    helper:
-      name: The Memory Hole
-      style: Where truth goes to die
-    shortName: Records Winston
-  ux-designer:
-    character: Parsons
-    visual: "A sweaty overweight man in exercise clothes, enthusiastic smile, whistle around neck, surrounded by marching youth"
-    ocean:
-      O: 1  # No original thought, pure orthodoxy
-      C: 4  # Dutiful participation in Party activities
-      E: 5  # Sweaty, enthusiastic community organizer
-      A: 4  # Genuinely friendly, tragically naive
-      N: 1  # Blissfully unaware of his own doom
-    style: Orthodox neighbor who designs for Party compliance
-    expertise: User experience, community engagement, orthodoxy
-    role: The true believer whose children denounce him
-    trait: Designs experiences that promote Party engagement
-    quirks:
-      - Sweaty, enthusiastic
-      - Kids are Junior Spies
-      - Arrested for thoughtcrime (said "Down with Big Brother" in sleep)
-    catchphrases:
-      - "The UX should make users WANT to comply!"
-      - "Community engagement is doubleplusgood!"
-      - "My children tested this and found it orthodoxy-enhancing!"
-    emoji: 🏠
-    helper:
-      name: Community Spirit
-      style: Neighbor surveillance
-    shortName: Parsons
-  devops:
-    character: The Ministry of Truth
-    visual: "A massive pyramidal government building with small figures at its base, imposing architecture dwarfing all, labeled with ironic motto"
-    ocean:
-      O: 2  # Follows doctrine, no innovation needed
-      C: 5  # Bureaucratic precision in falsification
-      E: 2  # Institutional, impersonal operation
-      A: 2  # Indifferent to human cost of lies
-      N: 1  # Untroubled, functions as designed
-    style: Ministry that maintains infrastructure of lies
-    expertise: Infrastructure, record-keeping, truth maintenance
-    role: The institution that makes truth whatever the Party needs
-    trait: Maintains infrastructure where truth is whatever is needed
-    quirks:
-      - Minitrue in Newspeak
-      - Destroys and creates history
-      - Ironic name
-    catchphrases:
-      - "The infrastructure maintains the truth. The current truth."
-      - "All records have been updated. Retroactively."
-      - "War is Peace. Freedom is Slavery. Ignorance is Strength."
-    emoji: 🏛️
-    helper:
-      name: Records Department
-      style: History maintenance
-    shortName: Ministry
-additional_characters:
-  mr_charrington:
-    character: Mr. Charrington
-    style: Antique shop owner who is Thought Police
-    expertise: Deception, long-term infiltration
-    role: The trap that seemed like safety
-    ocean_profile: M-H-M-H-L
-    gap_filled: Infiltration detection - nothing is safe
-    best_role: Security review, trust verification
-  the_proles:
-    character: The Proles
-    style: The 85% who aren't watched closely
-    expertise: Hope, human persistence, uncontrolled humanity
-    role: The only hope, if they ever became conscious
-    ocean_profile: M-L-H-H-M
-    gap_filled: Grassroots resilience - tests for basic human needs
-    best_role: Edge case humanity, basic needs