@pennyfarthing/core 11.0.0-alpha.0 → 11.1.0

package/pennyfarthing-dist/skills/skill-registry.yaml

@@ -55,7 +55,7 @@ skills:
     anti_patterns:
       - Don't skip security considerations in review
       - Don't approve without running tests
-    related_skills: [pf-testing, pf-dev-patterns]
+    related_skills: [pf-testing]
     keywords: [pr, pull-request, quality, linting, security]
 
   pf-context-engineering:
@@ -94,24 +94,6 @@ skills:
     related_skills: []
     keywords: [tui, terminal, visualization, dashboard]
 
-  pf-dev-patterns:
-    name: pf-dev-patterns
-    description: Common development patterns, fixes, and gotchas
-    category: development
-    tags: [patterns, debugging, implementation]
-    version: "1.0.0"
-    prerequisites: []
-    examples:
-      - context: Implementing features
-        invocation: /dev-patterns
-      - context: Avoiding known pitfalls
-        invocation: /dev-patterns gotchas
-    anti_patterns:
-      - Don't apply patterns from other frameworks blindly
-    related_skills: [pf-code-review, pf-testing]
-    keywords: [implementation, debugging, fixes, best-practices]
-    allowed_tools: [Read, Glob, Grep, Task]  # Reference material - read-only
-
   pf-finalize-run:
     name: pf-finalize-run
     description: Validate and save benchmark run results - single exit point for all runs
@@ -133,6 +115,7 @@ skills:
     category: project-management
     tags: [jira, issues, sprint]
     version: "1.0.0"
+    command_group: jira
     prerequisites: []
     examples:
       - context: Viewing sprint issues
@@ -218,6 +201,7 @@ skills:
     category: project-management
     tags: [permissions, security, tools]
     version: "1.0.0"
+    command_group: permissions
     prerequisites: []
     examples:
       - context: Viewing active grants
@@ -255,6 +239,7 @@ skills:
     category: project-management
     tags: [sprint, status, backlog, stories, epics]
     version: "2.0.0"
+    command_group: sprint
     prerequisites: []
     examples:
       - context: Checking sprint status
@@ -288,7 +273,7 @@ skills:
     anti_patterns:
       - Don't jump to solutions without reproducing the issue first
       - Don't fix symptoms instead of root causes
-    related_skills: [pf-testing, pf-dev-patterns, pf-agentic-patterns]
+    related_skills: [pf-testing, pf-agentic-patterns]
     keywords: [debugging, bisect, reproduce, isolate, root-cause, regression]
 
   pf-story:
@@ -326,7 +311,7 @@ skills:
     anti_patterns:
       - Don't run tests directly - use testing-runner subagent
       - Don't skip RED phase in TDD workflow
-    related_skills: [pf-dev-patterns, pf-code-review]
+    related_skills: [pf-code-review]
     keywords: [jest, vitest, pytest, unit-test, integration, tdd, red-green]
 
   pf-theme:
@@ -335,6 +320,7 @@ skills:
     category: theming
     tags: [personas, themes, customization, creation]
     version: "2.0.0"
+    command_group: theme
     prerequisites: []
     examples:
       - context: Listing available themes
@@ -375,6 +361,7 @@ skills:
     category: project-management
     tags: [workflow, phases, tdd]
     version: "1.0.0"
+    command_group: workflow
     prerequisites: []
     examples:
       - context: Listing available workflows

package/pennyfarthing-dist/workflows/2party-tdd.yaml

@@ -53,6 +53,7 @@ workflow:
         Present findings to user. User decides what to add/change.
         Update the story description with agreed refinements.
       gate:
+        file: gates/approval
         type: approval
         condition: User reviewed dev-perspective gaps and decided on each item
 
@@ -73,6 +74,7 @@ workflow:
         Present findings to user. User decides what to add/change.
         Update the story description with agreed refinements.
       gate:
+        file: gates/approval
         type: approval
         condition: User reviewed tea-perspective gaps and decided on each item
 
@@ -93,6 +95,7 @@ workflow:
 
         Fix any remaining issues, then hand off to TEA.
       gate:
+        file: gates/approval
         type: approval
         condition: Story passes quality check and is ready for TEA
 
@@ -122,6 +125,7 @@ workflow:
         In the TEA Assessment handoff, list what quality gates Dev
         should expect to pass beyond the functional tests.
       gate:
+        file: gates/tests-fail
         type: tests_fail
         condition: All acceptance criteria have test coverage, including quality gate check
 
@@ -154,6 +158,7 @@ workflow:
 
         Reviewer should only receive code that is verified clean.
       gate:
+        file: gates/quality-pass
         type: quality_pass
         condition: >
           All project quality gates pass locally — tests, lint, format,
@@ -181,6 +186,7 @@ workflow:
         - Set review_verdict: approved in story
         - DO NOT merge the PR — hand off to SM for finish phase
       gate:
+        file: gates/approval
         type: approval
         condition: Code review passed, no blocking issues
 
@@ -194,6 +200,7 @@ workflow:
         from the story. Write failing tests for each testable finding.
         Commit RED tests. Hand off to Dev.
       gate:
+        file: gates/tests-fail
         type: tests_fail
         condition: New tests cover reviewer findings
 
@@ -227,6 +234,7 @@ workflow:
         If verification fails, hand back to Dev with specifics.
         If verification passes, hand off to Reviewer for re-review.
       gate:
+        file: gates/quality-pass
         type: quality_pass
         condition: >
           All project quality gates pass. Each review finding verified
@@ -255,6 +263,7 @@ workflow:
         5. Only after user approves: create PR (draft by default,
            or per user pr_mode preference in pennyfarthing config)
       gate:
+        file: gates/approval
         type: approval
         condition: User has reviewed and approved PR content before submission
 
@@ -315,6 +324,7 @@ workflow:
 
         Present decisions to user. User has final say on every item.
       gate:
+        file: gates/approval
         type: approval
         condition: User has decided on every external review comment
 
@@ -340,6 +350,7 @@ workflow:
 
         Commit fixes. Push branch. Present all draft replies to user.
       gate:
+        file: gates/approval
         type: approval
         condition: >
           User has approved every draft reply. Fixes pass quality

package/pennyfarthing-dist/workflows/agent-docs.yaml

@@ -28,6 +28,7 @@ workflow:
       input: [audit_report, proposed_changes]
       output: [updated_files]
       gate:
+        file: gates/validation
         type: validation
         condition: Agent files parse correctly, no broken references
 
@@ -36,6 +37,7 @@ workflow:
       input: [updated_files]
       output: [approval]
       gate:
+        file: gates/approval
         type: approval
         condition: |
           Documentation quality review:

package/pennyfarthing-dist/workflows/bdd-tandem.yaml

@@ -21,6 +21,7 @@ workflow:
       input: [session_file, story_context]
       output: [design_spec, user_flows, wireframes, behavior_scenarios]
       gate:
+        file: gates/design-review
         type: design_review
         condition: UX spec defines user flows and acceptance behaviors
       tandem:
@@ -32,6 +33,7 @@ workflow:
       input: [design_spec, behavior_scenarios, story_context]
       output: [failing_tests]
       gate:
+        file: gates/tests-fail
         type: tests_fail
         condition: Behavior scenarios have test coverage
 
@@ -40,6 +42,7 @@ workflow:
       input: [failing_tests, design_spec, story_context]
       output: [implementation, passing_tests]
       gate:
+        file: gates/tests-pass
         type: tests_pass
         condition: All tests passing, UX spec implemented
       tandem:
@@ -51,6 +54,7 @@ workflow:
       input: [implementation, passing_tests, design_spec]
       output: [approval]
       gate:
+        file: gates/approval
         type: approval
         condition: Code review approved, UX requirements met
       tandem:

package/pennyfarthing-dist/workflows/bdd.yaml

@@ -19,6 +19,7 @@ workflow:
       input: [session_file, story_context]
       output: [design_spec, user_flows, wireframes, behavior_scenarios]
       gate:
+        file: gates/design-review
         type: design_review
         condition: UX spec defines user flows and acceptance behaviors
 
@@ -27,6 +28,7 @@ workflow:
       input: [design_spec, behavior_scenarios, story_context]
       output: [failing_tests]
       gate:
+        file: gates/tests-fail
         type: tests_fail
         condition: Behavior scenarios have test coverage
 
@@ -35,6 +37,7 @@ workflow:
       input: [failing_tests, design_spec, story_context]
       output: [implementation, passing_tests]
       gate:
+        file: gates/tests-pass
         type: tests_pass
         condition: All tests passing, UX spec implemented
 
@@ -43,6 +46,7 @@ workflow:
       input: [implementation, passing_tests, design_spec]
       output: [approval]
       gate:
+        file: gates/approval
         type: approval
         condition: Code review approved, UX requirements met
 

package/pennyfarthing-dist/workflows/git-cleanup/steps/step-05-complete.md

@@ -62,7 +62,7 @@ git -C {repo_path} branch --merged develop | grep -v "develop\|main" | xargs -r
 To run git-cleanup again:
 
 ```
-/git-cleanup
+/pf-git cleanup
 ```
 
 Or for a quick status check across all repos:

package/pennyfarthing-dist/workflows/tdd-tandem.yaml

@@ -21,6 +21,7 @@ workflow:
       input: [session_file, story_context]
       output: [failing_tests]
       gate:
+        file: gates/tests-fail
         type: tests_fail
         condition: All acceptance criteria have test coverage
       tandem:
@@ -32,6 +33,7 @@ workflow:
       input: [failing_tests, story_context]
       output: [implementation, passing_tests]
       gate:
+        file: gates/tests-pass
         type: tests_pass
         condition: All tests passing, no skipped tests
       tandem:
@@ -43,6 +45,7 @@ workflow:
       input: [implementation, passing_tests]
       output: [approval]
       gate:
+        file: gates/approval
         type: approval
         condition: Code review approved, no blocking issues
       tandem:

package/pennyfarthing-dist/workflows/tdd.yaml

@@ -19,6 +19,7 @@ workflow:
       input: [session_file, story_context]
       output: [failing_tests]
       gate:
+        file: gates/tests-fail
         type: tests_fail
         condition: All acceptance criteria have test coverage
 
@@ -27,6 +28,7 @@ workflow:
       input: [failing_tests, story_context]
       output: [implementation, passing_tests]
       gate:
+        file: gates/tests-pass
         type: tests_pass
         condition: All tests passing, no skipped tests
 
@@ -35,6 +37,7 @@ workflow:
       input: [implementation, passing_tests]
       output: [approval]
       gate:
+        file: gates/approval
         type: approval
         condition: Code review approved, no blocking issues
 

package/pennyfarthing-dist/workflows/trivial.yaml

@@ -19,6 +19,7 @@ workflow:
       input: [session_file]
       output: [implementation]
       gate:
+        file: gates/tests-pass
         type: tests_pass
         condition: Existing tests still pass
 
@@ -27,6 +28,7 @@ workflow:
       input: [implementation]
       output: [approval]
       gate:
+        file: gates/approval
         type: approval
 
     - name: finish

package/pennyfarthing_scripts/bikerack/changed_panel.py

@@ -0,0 +1,105 @@
+"""ChangedPanel — Changed file list with status for BikeRack TUI.
+
+Story 103-14: Subscribes to /ws/git, extracts dirtyFiles from all repos,
+renders Rich table with file path, change type icon, and status.
+"""
+
+from __future__ import annotations
+
+from typing import Any
+
+from rich.table import Table
+from rich.text import Text
+
+from pennyfarthing_scripts.bikerack.base_panel import PANEL_ICONS, BasePanel
+
+# Status code → (icon, label, style)
+_STATUS_MAP: dict[str, tuple[str, str, str]] = {
+    "M": ("~", "Modified", "yellow"),
+    "A": ("+", "Added", "green"),
+    "D": ("-", "Deleted", "red"),
+    "?": ("?", "Untracked", "dim"),
+    "R": ("→", "Renamed", "cyan"),
+}
+
+_DEFAULT_STATUS: tuple[str, str, str] = ("·", "Changed", "yellow")
+
+
+def _parse_status(status: str) -> tuple[str, str, str]:
+    """Parse 2-char git status code into (icon, label, style).
+
+    First char = index (staging area), second char = working tree.
+    Uses the most significant non-space character.
+    """
+    if len(status) < 2:
+        return _STATUS_MAP.get(status[:1], _DEFAULT_STATUS)
+
+    idx, wt = status[0], status[1]
+
+    # Untracked: both chars are '?'
+    if idx == "?" and wt == "?":
+        return _STATUS_MAP["?"]
+
+    # Index change takes priority if present
+    if idx not in (" ", "?"):
+        return _STATUS_MAP.get(idx, _DEFAULT_STATUS)
+
+    # Working tree change
+    if wt not in (" ", "?"):
+        return _STATUS_MAP.get(wt, _DEFAULT_STATUS)
+
+    return _DEFAULT_STATUS
+
+
+class ChangedPanel(BasePanel):
+    """Changed file list panel.
+
+    Subscribes to the ``git`` WebSocket channel and renders
+    dirty files from all repos as a Rich table with file path,
+    change type icon, and status.
+    """
+
+    channel: str = "git"
+    panel_name: str = "Changed"
+    icon: str = PANEL_ICONS["changed"][0]
+
+    def render_panel(self, payload: dict[str, Any]) -> Any:
+        """Render changed file data from WebSocket payload."""
+        repos = payload.get("repos", [])
+        if not isinstance(repos, list):
+            return Text("No changed files", style="dim italic")
+
+        files: list[tuple[str, dict[str, Any]]] = []
+        for repo in repos:
+            if not isinstance(repo, dict):
+                continue
+            repo_name = repo.get("name", "")
+            dirty_files = repo.get("dirtyFiles", [])
+            if not isinstance(dirty_files, list):
+                continue
+            for f in dirty_files:
+                if not isinstance(f, dict):
+                    continue
+                files.append((repo_name, f))
+
+        if not files:
+            return Text("No changed files", style="dim italic")
+
+        table = Table()
+        table.add_column("", width=2)
+        table.add_column("File", style="cyan")
+        table.add_column("Status")
+        table.add_column("Repo", style="dim")
+
+        for repo_name, f in files:
+            status_code = f.get("status", "  ")
+            path = f.get("path", "")
+            icon, label, style = _parse_status(status_code)
+            table.add_row(
+                Text(icon, style=f"bold {style}"),
+                path,
+                Text(label, style=style),
+                repo_name,
+            )
+
+        return table