speccrew 0.5.9 → 0.5.11

package/.speccrew/skills/speccrew-dev-review-backend/templates/REVIEW-REPORT-TEMPLATE.md

@@ -0,0 +1,94 @@
+# Backend Code Review Report
+
+## Summary
+
+- **Task ID**: {task_id}
+- **Platform**: {platform_id}
+- **Module**: {module_name}
+- **Review Date**: {review_date}
+- **Review Result**: PASS | PARTIAL | FAIL
+- **Completeness**: {completed_files}/{total_files} ({percentage}%)
+
+## File Completeness
+
+| Category | Required | Created | Missing |
+|----------|----------|---------|---------|
+| Enums | {n} | {n} | {n} |
+| DO | {n} | {n} | {n} |
+| VO | {n} | {n} | {n} |
+| Mapper | {n} | {n} | {n} |
+| Service | {n} | {n} | {n} |
+| Controller | {n} | {n} | {n} |
+| Convert | {n} | {n} | {n} |
+| **Total** | **{n}** | **{n}** | **{n}** |
+
+## Missing Files
+
+1. `{expected_path_1}`
+2. `{expected_path_2}`
+...
+
+## Code Compliance Issues
+
+### DO/VO/DTO Issues
+
+| Severity | File | Issue Description |
+|----------|------|-------------------|
+| ERROR | `{file}` | {description} |
+| WARN | `{file}` | {description} |
+
+### Service Layer Issues
+
+| Severity | File | Issue Description |
+|----------|------|-------------------|
+| ERROR | `{file}` | {description} |
+| WARN | `{file}` | {description} |
+
+### Controller Layer Issues
+
+| Severity | File | Issue Description |
+|----------|------|-------------------|
+| ERROR | `{file}` | {description} |
+| WARN | `{file}` | {description} |
+
+### Database Mapping Issues
+
+| Severity | File | Issue Description |
+|----------|------|-------------------|
+| ERROR | `{file}` | {description} |
+| WARN | `{file}` | {description} |
+
+## API Consistency Issues
+
+| Severity | Endpoint | Issue Description |
+|----------|----------|-------------------|
+| ERROR | `{endpoint}` | {description} |
+| WARN | `{endpoint}` | {description} |
+
+## Business Logic Issues
+
+| Severity | Method | Issue Description |
+|----------|--------|-------------------|
+| ERROR | `{method}` | {description} |
+| WARN | `{method}` | {description} |
+
+## Verdict
+
+{Detailed verdict explanation based on findings. Explain why PASS, PARTIAL, or FAIL was assigned.}
+
+## Re-dispatch Guidance
+
+Priority items for next dev worker:
+
+1. **{priority_item_1}** - {detailed description}
+2. **{priority_item_2}** - {detailed description}
+3. **{priority_item_3}** - {detailed description}
+
+## Issue Statistics
+
+| Severity | Count |
+|----------|-------|
+| CRITICAL | {n} |
+| ERROR | {n} |
+| WARN | {n} |
+| LOW | {n} |

package/.speccrew/skills/speccrew-dev-review-desktop/SKILL.md

@@ -0,0 +1,181 @@
+---
+name: speccrew-dev-review-desktop
+description: SpecCrew Desktop Code Review Skill. Reviews desktop app implementation code (Electron/Tauri) against system design documents and security standards. Generates structured review reports with PASS/PARTIAL/FAIL verdict.
+tools: Read, Glob, Grep
+---
+
+# Trigger Scenarios
+
+- When speccrew-system-developer dispatches desktop code review for a completed module
+- When user requests "Review this desktop module's implementation"
+- When user asks "Check if desktop code matches design"
+- When incremental review is needed after partial desktop implementation
+
+# Input Parameters
+
+| Parameter | Required | Description |
+|-----------|----------|-------------|
+| `design_doc_path` | Yes | Path to desktop module design document |
+| `implementation_report_path` | Yes | Path to desktop development report |
+| `source_root` | Yes | Root directory of desktop source code |
+| `platform_id` | Yes | Desktop platform (desktop-tauri, desktop-electron) |
+| `api_contract_path` | No | Path to API contract file |
+| `task_id` | Yes | Task identifier from dispatch context |
+| `previous_review_path` | No | Path to previous review report |
+
+# Workflow
+
+## Absolute Constraints
+
+> **Violation = review failure.**
+
+1. **READ-ONLY OPERATION** — NEVER modify source code files.
+2. **FORBIDDEN: Code fixes** — Do NOT attempt to fix issues. Only document them.
+3. **MANDATORY: Actionable output** — PARTIAL/FAIL results MUST include specific "Re-dispatch Guidance".
+4. **INCREMENTAL REVIEW SUPPORT** — Skip items already marked as passed in previous review.
+
+## Step 1: Load Documents
+
+### 1.1 Validate Inputs
+
+Verify all required parameters provided. If any missing → Report error, stop.
+
+### 1.2 Read Design Document
+
+Extract: Module Overview, Process Architecture, IPC Contracts, Native Integration points.
+
+### 1.3 Read Implementation Report
+
+Extract: Completed Files, Implementation Status, Known Issues.
+
+### 1.4 Read API Contract (if provided)
+
+Extract API endpoints for validation against desktop API calls.
+
+## Step 2: File Completeness Check
+
+### 2.1 Build Expected File List
+
+Desktop file categories:
+
+| Category | Pattern | Example |
+|----------|---------|---------|
+| Main Process | `src-tauri/src/**/*.rs` or `main/**/*` | `src-tauri/src/main.rs` |
+| Renderer | `src/**/*` (frontend files) | `src/App.vue` |
+| Preload | `preload/**/*` or `src-tauri/src/preload.rs` | `preload/index.ts` |
+| IPC Handlers | `src-tauri/src/commands/**/*.rs` | `commands/file.rs` |
+| Native Modules | `native/**/*` | `native/fileSystem.ts` |
+
+### 2.2 Scan Actual Files
+
+Use `Glob` to scan `source_root` for implemented files.
+
+### 2.3 Calculate Completeness
+
+Generate completeness matrix and percentage.
+
+## Step 3: Desktop-Specific Compliance Check
+
+### 3.1 Process Separation Validation
+
+| Check | Rule | Severity |
+|-------|------|----------|
+| Main/Renderer Separation | Business logic in main process only | CRITICAL |
+| No Node in Renderer | Renderer has no direct Node.js access | CRITICAL |
+| Context Isolation | contextIsolation enabled | CRITICAL |
+
+### 3.2 IPC Channel Consistency
+
+| Check | Rule | Severity |
+|-------|------|----------|
+| Channel Naming | IPC channels follow naming convention | ERROR |
+| Type Safety | IPC payloads properly typed | ERROR |
+| Channel Registration | All channels registered in preload | ERROR |
+| Bidirectional Safety | No unsafe IPC patterns | CRITICAL |
+
+### 3.3 Security Isolation
+
+| Check | Rule | Severity |
+|-------|------|----------|
+| contextBridge | Uses contextBridge for API exposure | CRITICAL |
+| Preload Script | Preload script properly configured | CRITICAL |
+| CSP Headers | Content Security Policy configured | ERROR |
+| Remote Content | No unsafe remote content loading | CRITICAL |
+
+### 3.4 Native Integration Check
+
+| Check | Rule | Severity |
+|-------|------|----------|
+| File System | File operations in main process only | CRITICAL |
+| System Notifications | Notifications properly implemented | WARN |
+| Native Menus | Menu structure matches design | ERROR |
+| System Tray | Tray implementation if required | ERROR |
+
+### 3.5 Packaging Configuration
+
+| Check | Rule | Severity |
+|-------|------|----------|
+| Build Config | Build configuration complete | ERROR |
+| Assets | Required assets included | ERROR |
+| Signing | Code signing configured (if required) | WARN |
+| Auto-Update | Auto-update mechanism (if required) | WARN |
+
+## Step 4: Generate Review Report
+
+### 4.1 Determine Result
+
+| Result | Criteria |
+|--------|----------|
+| **PASS** | 100% files created, 0 ERROR-level issues |
+| **PARTIAL** | 70-99% files created, or non-critical ERROR issues |
+| **FAIL** | <70% files created, or critical blockers present |
+
+### 4.2 Write Report
+
+Generate report at: `speccrew-workspace/iterations/{number}-{type}-{name}/04.development/{platform_id}/[module]-review-report.md`
+
+Use template from `templates/REVIEW-REPORT-TEMPLATE.md`.
+
+## Step 5: Task Completion Report
+
+```markdown
+## Task Completion Report
+- **Status**: SUCCESS
+- **Task ID**: review-{original_task_id}
+- **Platform**: {platform_id}
+- **Module**: {module_name}
+- **Output Files**: {review_report_path}
+- **Summary**: Review {result}: {completed}/{total} files, {error_count} errors
+```
+
+# Severity Levels
+
+| Level | Definition | Action Required |
+|-------|------------|-----------------|
+| **CRITICAL** | Security vulnerability or sandbox escape risk | Must fix immediately |
+| **ERROR** | Blocking functionality or violating core requirements | Must fix before PASS |
+| **WARN** | Best practice violation or missing documentation | Should fix |
+| **LOW** | Code style or minor optimization suggestion | Optional |
+
+# Key Rules
+
+| Rule | Description |
+|------|-------------|
+| **Read-Only** | NEVER modify any source code |
+| **Security First** | Desktop security is paramount - verify isolation |
+| **Blueprint-Driven** | Validate against design document specifications |
+| **Actionable Output** | PARTIAL/FAIL must include specific fix guidance |
+| **Completeness First** | File existence is primary check before content validation |
+
+# Checklist
+
+- [ ] All required inputs validated
+- [ ] Design document loaded and parsed
+- [ ] File completeness check completed
+- [ ] Main/renderer separation validated
+- [ ] IPC channels checked for consistency
+- [ ] Security isolation verified
+- [ ] Native integration checked
+- [ ] Packaging configuration validated
+- [ ] Review report written with clear verdict
+- [ ] Re-dispatch guidance provided for PARTIAL/FAIL

package/.speccrew/skills/speccrew-dev-review-desktop/templates/REVIEW-REPORT-TEMPLATE.md

@@ -0,0 +1,90 @@
+# Desktop Code Review Report
+
+## Summary
+
+- **Task ID**: {task_id}
+- **Platform**: {platform_id}
+- **Module**: {module_name}
+- **Review Date**: {review_date}
+- **Review Result**: PASS | PARTIAL | FAIL
+- **Completeness**: {completed_files}/{total_files} ({percentage}%)
+
+## File Completeness
+
+| Category | Required | Created | Missing |
+|----------|----------|---------|---------|
+| Main Process | {n} | {n} | {n} |
+| Renderer | {n} | {n} | {n} |
+| Preload | {n} | {n} | {n} |
+| IPC Handlers | {n} | {n} | {n} |
+| Native Modules | {n} | {n} | {n} |
+| **Total** | **{n}** | **{n}** | **{n}** |
+
+## Missing Files
+
+1. `{expected_path_1}`
+2. `{expected_path_2}`
+...
+
+## Process Separation Issues
+
+| Severity | File | Issue Description |
+|----------|------|-------------------|
+| CRITICAL | `{file}` | {description} |
+| ERROR | `{file}` | {description} |
+
+## IPC Channel Issues
+
+| Severity | Channel | Issue Description |
+|----------|---------|-------------------|
+| CRITICAL | `{channel}` | {description} |
+| ERROR | `{channel}` | {description} |
+| WARN | `{channel}` | {description} |
+
+## Security Isolation Issues
+
+| Severity | Aspect | Issue Description |
+|----------|--------|-------------------|
+| CRITICAL | contextBridge | {description} |
+| CRITICAL | Preload Script | {description} |
+| ERROR | CSP Headers | {description} |
+| CRITICAL | Remote Content | {description} |
+
+## Native Integration Issues
+
+| Severity | Feature | Issue Description |
+|----------|---------|-------------------|
+| CRITICAL | File System | {description} |
+| ERROR | Native Menus | {description} |
+| ERROR | System Tray | {description} |
+| WARN | Notifications | {description} |
+
+## Packaging Configuration Issues
+
+| Severity | Aspect | Issue Description |
+|----------|--------|-------------------|
+| ERROR | Build Config | {description} |
+| ERROR | Assets | {description} |
+| WARN | Signing | {description} |
+| WARN | Auto-Update | {description} |
+
+## Verdict
+
+{Detailed verdict explanation based on findings.}
+
+## Re-dispatch Guidance
+
+Priority items for next dev worker:
+
+1. **{priority_item_1}** - {detailed description}
+2. **{priority_item_2}** - {detailed description}
+3. **{priority_item_3}** - {detailed description}
+
+## Issue Statistics
+
+| Severity | Count |
+|----------|-------|
+| CRITICAL | {n} |
+| ERROR | {n} |
+| WARN | {n} |
+| LOW | {n} |

package/.speccrew/skills/speccrew-dev-review-frontend/SKILL.md

@@ -0,0 +1,177 @@
+---
+name: speccrew-dev-review-frontend
+description: SpecCrew Frontend Code Review Skill. Reviews web frontend implementation code against system design documents, API contracts, and coding standards. Generates structured review reports with PASS/PARTIAL/FAIL verdict.
+tools: Read, Glob, Grep
+---
+
+# Trigger Scenarios
+
+- When speccrew-system-developer dispatches frontend code review for a completed module
+- When user requests "Review this frontend module's implementation"
+- When user asks "Check if frontend code matches design"
+- When incremental review is needed after partial frontend implementation
+
+# Input Parameters
+
+| Parameter | Required | Description |
+|-----------|----------|-------------|
+| `design_doc_path` | Yes | Path to frontend module design document |
+| `implementation_report_path` | Yes | Path to frontend development report |
+| `source_root` | Yes | Root directory of frontend source code |
+| `platform_id` | Yes | Frontend platform (web-vue, web-react) |
+| `api_contract_path` | No | Path to API contract file |
+| `task_id` | Yes | Task identifier from dispatch context |
+| `previous_review_path` | No | Path to previous review report |
+
+# Workflow
+
+## Absolute Constraints
+
+> **Violation = review failure.**
+
+1. **READ-ONLY OPERATION** — NEVER modify source code files.
+2. **FORBIDDEN: Code fixes** — Do NOT attempt to fix issues. Only document them.
+3. **MANDATORY: Actionable output** — PARTIAL/FAIL results MUST include specific "Re-dispatch Guidance".
+4. **INCREMENTAL REVIEW SUPPORT** — Skip items already marked as passed in previous review.
+
+## Step 1: Load Documents
+
+### 1.1 Validate Inputs
+
+Verify all required parameters provided. If any missing → Report error, stop.
+
+### 1.2 Read Design Document
+
+Extract: Module Overview, Component Structure, Store Modules, Routes, API Integration points.
+
+### 1.3 Read Implementation Report
+
+Extract: Completed Files, Implementation Status, Known Issues.
+
+### 1.4 Read API Contract (if provided)
+
+Extract API endpoints for validation against frontend API calls.
+
+## Step 2: File Completeness Check
+
+### 2.1 Build Expected File List
+
+Frontend file categories:
+
+| Category | Pattern | Example |
+|----------|---------|---------|
+| Components | `components/**/*.vue` or `**/*.tsx` | `components/UserForm.vue` |
+| Views/Pages | `views/**/*.vue` or `pages/**/*.tsx` | `views/UserList.vue` |
+| Store | `store/**/*.ts` or `stores/**/*.ts` | `store/userStore.ts` |
+| API Clients | `api/**/*.ts` or `services/**/*.ts` | `api/userApi.ts` |
+| Routes | `router/**/*.ts` | `router/index.ts` |
+| Styles | `styles/**/*.scss` or `**/*.module.css` | `styles/variables.scss` |
+
+### 2.2 Scan Actual Files
+
+Use `Glob` to scan `source_root` for implemented files.
+
+### 2.3 Calculate Completeness
+
+Generate completeness matrix and percentage.
+
+## Step 3: Frontend-Specific Compliance Check
+
+### 3.1 Component Structure Check
+
+| Check | Rule | Severity |
+|-------|------|----------|
+| Directory Structure | Components in correct directories per design | ERROR |
+| Component Granularity | Components appropriately sized and reusable | WARN |
+| Naming Convention | PascalCase for components, camelCase for composables | ERROR |
+| Props Definition | Props properly typed and documented | WARN |
+
+### 3.2 State Management Check
+
+| Check | Rule | Severity |
+|-------|------|----------|
+| Store Design | Store modules match design specification | ERROR |
+| Data Flow | Unidirectional data flow followed | WARN |
+| State Mutations | Mutations/actions properly defined | ERROR |
+
+### 3.3 API Call Consistency
+
+| Check | Rule | Severity |
+|-------|------|----------|
+| Endpoint Match | API calls match API contract | ERROR |
+| Error Handling | API errors properly handled | ERROR |
+| Loading States | Loading states implemented | WARN |
+
+### 3.4 Route Definition Validation
+
+| Check | Rule | Severity |
+|-------|------|----------|
+| Route Match | Routes match design document | ERROR |
+| Lazy Loading | Routes use lazy loading where appropriate | WARN |
+| Navigation Guards | Auth guards implemented where required | ERROR |
+
+### 3.5 Style and Layout Compliance
+
+| Check | Rule | Severity |
+|-------|------|----------|
+| Style Guide | Follows project style guide | WARN |
+| Responsive | Responsive design implemented | ERROR |
+| Accessibility | Basic accessibility (a11y) compliance | WARN |
+
+## Step 4: Generate Review Report
+
+### 4.1 Determine Result
+
+| Result | Criteria |
+|--------|----------|
+| **PASS** | 100% files created, 0 ERROR-level issues |
+| **PARTIAL** | 70-99% files created, or non-critical ERROR issues |
+| **FAIL** | <70% files created, or critical blockers present |
+
+### 4.2 Write Report
+
+Generate report at: `speccrew-workspace/iterations/{number}-{type}-{name}/04.development/{platform_id}/[module]-review-report.md`
+
+Use template from `templates/REVIEW-REPORT-TEMPLATE.md`.
+
+## Step 5: Task Completion Report
+
+```markdown
+## Task Completion Report
+- **Status**: SUCCESS
+- **Task ID**: review-{original_task_id}
+- **Platform**: {platform_id}
+- **Module**: {module_name}
+- **Output Files**: {review_report_path}
+- **Summary**: Review {result}: {completed}/{total} files, {error_count} errors
+```
+
+# Severity Levels
+
+| Level | Definition | Action Required |
+|-------|------------|-----------------|
+| **ERROR** | Blocking functionality or violating core requirements | Must fix before PASS |
+| **WARN** | Best practice violation or missing documentation | Should fix |
+| **LOW** | Code style or minor optimization suggestion | Optional |
+
+# Key Rules
+
+| Rule | Description |
+|------|-------------|
+| **Read-Only** | NEVER modify any source code |
+| **Blueprint-Driven** | Validate against design document specifications |
+| **Actionable Output** | PARTIAL/FAIL must include specific fix guidance |
+| **Completeness First** | File existence is primary check before content validation |
+
+# Checklist
+
+- [ ] All required inputs validated
+- [ ] Design document loaded and parsed
+- [ ] File completeness check completed
+- [ ] Component structure validated
+- [ ] State management checked
+- [ ] API calls validated against contract
+- [ ] Routes verified against design
+- [ ] Styles checked for compliance
+- [ ] Review report written with clear verdict
+- [ ] Re-dispatch guidance provided for PARTIAL/FAIL

package/.speccrew/skills/speccrew-dev-review-frontend/templates/REVIEW-REPORT-TEMPLATE.md

@@ -0,0 +1,83 @@
+# Frontend Code Review Report
+
+## Summary
+
+- **Task ID**: {task_id}
+- **Platform**: {platform_id}
+- **Module**: {module_name}
+- **Review Date**: {review_date}
+- **Review Result**: PASS | PARTIAL | FAIL
+- **Completeness**: {completed_files}/{total_files} ({percentage}%)
+
+## File Completeness
+
+| Category | Required | Created | Missing |
+|----------|----------|---------|---------|
+| Components | {n} | {n} | {n} |
+| Views/Pages | {n} | {n} | {n} |
+| Store | {n} | {n} | {n} |
+| API Clients | {n} | {n} | {n} |
+| Routes | {n} | {n} | {n} |
+| Styles | {n} | {n} | {n} |
+| **Total** | **{n}** | **{n}** | **{n}** |
+
+## Missing Files
+
+1. `{expected_path_1}`
+2. `{expected_path_2}`
+...
+
+## Component Structure Issues
+
+| Severity | File | Issue Description |
+|----------|------|-------------------|
+| ERROR | `{file}` | {description} |
+| WARN | `{file}` | {description} |
+
+## State Management Issues
+
+| Severity | Store Module | Issue Description |
+|----------|--------------|-------------------|
+| ERROR | `{module}` | {description} |
+| WARN | `{module}` | {description} |
+
+## API Call Issues
+
+| Severity | API Method | Issue Description |
+|----------|------------|-------------------|
+| ERROR | `{method}` | {description} |
+| WARN | `{method}` | {description} |
+
+## Route Definition Issues
+
+| Severity | Route | Issue Description |
+|----------|-------|-------------------|
+| ERROR | `{route}` | {description} |
+| WARN | `{route}` | {description} |
+
+## Style and Layout Issues
+
+| Severity | File | Issue Description |
+|----------|------|-------------------|
+| ERROR | `{file}` | {description} |
+| WARN | `{file}` | {description} |
+
+## Verdict
+
+{Detailed verdict explanation based on findings.}
+
+## Re-dispatch Guidance
+
+Priority items for next dev worker:
+
+1. **{priority_item_1}** - {detailed description}
+2. **{priority_item_2}** - {detailed description}
+3. **{priority_item_3}** - {detailed description}
+
+## Issue Statistics
+
+| Severity | Count |
+|----------|-------|
+| ERROR | {n} |
+| WARN | {n} |
+| LOW | {n} |