Table of Contents
-
Philosophy
-
Quick Start
-
The Three Perspectives
-
Workflow
-
Output Format
-
Configuration
-
Guardrails
Run verification after review: make lint && make test to confirm no regressions.
- 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
- Functional Perspective
Question: What does this system do?
Evaluates:
-
Feature completeness
-
Capability coverage
-
Behavior correctness
-
Integration points
Outputs:
-
Feature inventory
-
Capability gaps
-
Behavior anomalies
- 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
- 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 )
-
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 )
-
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 )
-
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 )
-
Pattern recognition - What patterns are used
-
Principle check - SOLID, DRY, KISS adherence
-
Debt assessment - Technical debt inventory
Phase 5: Synthesis (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
- [Gap 1] - [Impact]
- [Gap 2] - [Impact]
Practical Perspective
Performance Assessment
| Metric | Current | Target | Status |
|---|---|---|---|
| [Metric 1] | [value] | [target] | PASS/FAIL |
Usability Issues
- [Issue 1] - [Severity]
- [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
- [Recommendation 1]
- Impact: [what improves]
- Effort: [estimate]
- Rationale: [why]
Medium Priority
- [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-complete
-
fpf-review:functional-complete
-
fpf-review:practical-complete
-
fpf-review:foundation-complete
-
fpf-review:synthesis-complete
Integration Points
-
pensive:code-reviewer : Detailed code-level review
-
imbue:review-core : Review methodology patterns
-
imbue:feature-review : Feature-specific analysis
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