Claude-skill-registry fpf-review
install
source · Clone the upstream repo
git clone https://github.com/majiayu000/claude-skill-registry
Claude Code · Install into ~/.claude/skills/
T=$(mktemp -d) && git clone --depth=1 https://github.com/majiayu000/claude-skill-registry "$T" && mkdir -p ~/.claude/skills && cp -r "$T/skills/data/fpf-review" ~/.claude/skills/majiayu000-claude-skill-registry-fpf-review && rm -rf "$T"
manifest:
skills/data/fpf-review/SKILL.mdsource content
Table of Contents
- Philosophy
- Quick Start
- The Three Perspectives
- Workflow
- Output Format
- Configuration
- Guardrails
- Integration Points
FPF Architecture Review
Conduct architecture reviews using the FPF (Functional Programming Framework) methodology, evaluating codebases through three complementary perspectives.
Philosophy
Architecture reviews should be systematic and multi-dimensional. FPF provides three lenses:
- Functional: What the system does (capabilities, behaviors)
- Practical: How well it works (performance, usability)
- Foundation: What it's built on (principles, patterns)
This lighter skill-based approach replaces heavyweight tooling with focused analysis.
Quick Start
# Full FPF review /fpf-review # Specific perspective /fpf-review --perspective functional /fpf-review --perspective practical /fpf-review --perspective foundation # Focused scope /fpf-review --scope plugins/sanctum
The Three Perspectives
1. Functional Perspective
Question: What does this system do?
Evaluates:
- Feature completeness
- Capability coverage
- Behavior correctness
- Integration points
Outputs:
- Feature inventory
- Capability gaps
- Behavior anomalies
2. Practical Perspective
Question: How well does this system work?
Evaluates:
- Performance characteristics
- Usability patterns
- Operational concerns
- Scalability considerations
Outputs:
- Performance assessment
- Usability issues
- Operational recommendations
3. Foundation Perspective
Question: What is this system built on?
Evaluates:
- Architectural patterns
- Design principles
- Code quality
- Technical debt
Outputs:
- Pattern analysis
- Principle adherence
- Debt inventory
Workflow
Phase 1: Discovery (fpf-review:discovery-complete
)
fpf-review:discovery-complete- Scan codebase structure - Identify components, modules, layers
- Map dependencies - Internal and external relationships
- Identify entry points - Public APIs, commands, interfaces
Phase 2: Functional Analysis (fpf-review:functional-complete
)
fpf-review:functional-complete- Inventory features - What capabilities exist
- Trace behaviors - How features work end-to-end
- Identify gaps - Missing or incomplete functionality
Phase 3: Practical Analysis (fpf-review:practical-complete
)
fpf-review:practical-complete- Assess performance - Latency, throughput, resource usage
- Evaluate usability - Developer experience, API design
- Check operations - Logging, monitoring, error handling
Phase 4: Foundation Analysis (fpf-review:foundation-complete
)
fpf-review:foundation-complete- Pattern recognition - What patterns are used
- Principle check - SOLID, DRY, KISS adherence
- Debt assessment - Technical debt inventory
Phase 5: Synthesis (fpf-review:synthesis-complete
)
fpf-review:synthesis-complete- Cross-reference findings - Connect issues across perspectives
- Prioritize recommendations - Based on impact and effort
- Generate report - Structured findings and actions
Output Format
FPF Review Report
# FPF Architecture Review: [Project/Component] **Date:** [DATE] **Scope:** [what was reviewed] **Reviewer:** Claude Code with FPF skill ## Executive Summary [2-3 sentence overview of findings] ## Functional Perspective ### Features Inventory | Feature | Status | Notes | |---------|--------|-------| | [Feature 1] | Complete | - | | [Feature 2] | Partial | Missing edge case handling | ### Capability Gaps 1. [Gap 1] - [Impact] 2. [Gap 2] - [Impact] ## Practical Perspective ### Performance Assessment | Metric | Current | Target | Status | |--------|---------|--------|--------| | [Metric 1] | [value] | [target] | PASS/FAIL | ### Usability Issues 1. [Issue 1] - [Severity] 2. [Issue 2] - [Severity] ## Foundation Perspective ### Pattern Analysis | Pattern | Usage | Assessment | |---------|-------|------------| | [Pattern 1] | [where used] | Appropriate/Problematic | ### Technical Debt | Item | Severity | Effort | Priority | |------|----------|--------|----------| | [Debt 1] | High | Medium | P1 | ## Recommendations ### High Priority 1. **[Recommendation 1]** - Impact: [what improves] - Effort: [estimate] - Rationale: [why] ### Medium Priority 2. **[Recommendation 2]** ... ## Action Items - [ ] [Action 1] - Owner: TBD - [ ] [Action 2] - Owner: TBD
Configuration
# .fpf-review.yaml scope: include: - src/ - plugins/ exclude: - tests/ - docs/ - node_modules/ perspectives: functional: enabled: true depth: "full" # full, summary practical: enabled: true depth: "full" foundation: enabled: true depth: "full" output: format: "markdown" create_issues: false # Create GitHub issues for findings severity_threshold: "medium" # Report medium+ severity
Guardrails
- Scope boundaries - Stay within configured scope
- Evidence-based - Every finding needs supporting evidence
- Actionable output - Recommendations must be actionable
- Balanced perspectives - Don't over-index on one perspective
Required TodoWrite Items
fpf-review:discovery-completefpf-review:functional-completefpf-review:practical-completefpf-review:foundation-completefpf-review:synthesis-complete
Integration Points
: Detailed code-level reviewpensive:code-reviewer
: Review methodology patternsimbue:review-core
: Feature-specific analysisimbue:feature-review
References
- FPF Framework - Original methodology
- quint-code - Heavy implementation (this skill is lighter)
Status: Skeleton implementation. Requires:
- Detailed analysis algorithms
- Pattern recognition logic
- Report generation templates