superkit-mcp-server 1.2.2 → 1.2.3
This diff represents the content of publicly available package versions that have been released to one of the supported registries. The information contained in this diff is provided for informational purposes only and reflects changes between package versions as they appear in their respective public registries.
- package/ARCHITECTURE.md +102 -102
- package/README.md +71 -71
- package/SUPERKIT.md +168 -168
- package/agents/code-archaeologist.md +106 -106
- package/agents/coder.md +90 -90
- package/agents/data-engineer.md +28 -28
- package/agents/devops-engineer.md +242 -242
- package/agents/git-manager.md +203 -203
- package/agents/orchestrator.md +420 -420
- package/agents/penetration-tester.md +188 -188
- package/agents/performance-optimizer.md +187 -187
- package/agents/planner.md +270 -270
- package/agents/qa-automation-engineer.md +103 -103
- package/agents/quant-developer.md +32 -32
- package/agents/reviewer.md +100 -100
- package/agents/scout.md +222 -222
- package/agents/security-auditor.md +3 -2
- package/agents/tester.md +274 -274
- package/agents/ui-designer.md +208 -208
- package/build/index.js +18 -9
- package/build/tools/__tests__/loggerTools.test.js +5 -5
- package/build/tools/archTools.js +2 -19
- package/build/tools/autoPreview.js +2 -2
- package/build/tools/compoundTools.js +4 -4
- package/build/tools/docsTools.js +5 -10
- package/build/tools/loggerTools.js +1 -1
- package/build/tools/todoTools.js +39 -39
- package/build/tools/validators/__tests__/apiSchema.test.js +23 -23
- package/build/tools/validators/__tests__/convertRules.test.js +5 -5
- package/build/tools/validators/__tests__/frontendDesign.test.js +12 -12
- package/build/tools/validators/__tests__/geoChecker.test.js +19 -19
- package/build/tools/validators/__tests__/mobileAudit.test.js +12 -12
- package/build/tools/validators/__tests__/reactPerformanceChecker.test.js +17 -17
- package/build/tools/validators/__tests__/securityScan.test.js +6 -6
- package/build/tools/validators/__tests__/seoChecker.test.js +16 -16
- package/build/tools/validators/__tests__/typeCoverage.test.js +14 -14
- package/build/tools/validators/convertRules.js +2 -2
- package/commands/README.md +122 -122
- package/commands/ask.toml +72 -72
- package/commands/brainstorm.toml +119 -119
- package/commands/chat.toml +77 -77
- package/commands/code-preview.toml +37 -37
- package/commands/code.toml +28 -28
- package/commands/content.toml +200 -200
- package/commands/cook.toml +77 -77
- package/commands/copywrite.toml +131 -131
- package/commands/db.toml +192 -192
- package/commands/debug.toml +166 -166
- package/commands/design.toml +158 -158
- package/commands/dev-rules.toml +14 -14
- package/commands/do.toml +117 -117
- package/commands/doc-rules.toml +14 -14
- package/commands/docs.toml +148 -148
- package/commands/fix.toml +440 -440
- package/commands/fullstack.toml +175 -175
- package/commands/git.toml +235 -235
- package/commands/help.toml +84 -84
- package/commands/integrate.toml +127 -127
- package/commands/journal.toml +136 -136
- package/commands/kit-setup.toml +40 -40
- package/commands/mcp.toml +183 -183
- package/commands/orchestration.toml +15 -15
- package/commands/plan.toml +171 -171
- package/commands/pm.toml +148 -148
- package/commands/pr.toml +50 -50
- package/commands/project.toml +32 -32
- package/commands/research.toml +117 -117
- package/commands/review-pr.toml +63 -63
- package/commands/review.toml +190 -190
- package/commands/scout-ext.toml +97 -97
- package/commands/scout.toml +79 -79
- package/commands/screenshot.toml +65 -65
- package/commands/session.toml +102 -102
- package/commands/skill.toml +384 -384
- package/commands/status.toml +22 -22
- package/commands/team.toml +56 -56
- package/commands/test.toml +164 -164
- package/commands/ticket.toml +70 -70
- package/commands/use.toml +106 -106
- package/commands/video.toml +83 -83
- package/commands/watzup.toml +71 -71
- package/commands/workflow.toml +14 -14
- package/package.json +35 -35
- package/skills/meta/README.md +30 -30
- package/skills/meta/api-design/SKILL.md +134 -134
- package/skills/meta/code-review/SKILL.md +44 -44
- package/skills/meta/code-review/checklists/pre-merge.md +25 -25
- package/skills/meta/code-review/workflows/architecture-pass.md +26 -26
- package/skills/meta/code-review/workflows/performance-pass.md +27 -27
- package/skills/meta/code-review/workflows/security-pass.md +29 -29
- package/skills/meta/compound-docs/SKILL.md +133 -133
- package/skills/meta/debug/SKILL.md +40 -40
- package/skills/meta/debug/templates/bug-report.template.md +31 -31
- package/skills/meta/debug/workflows/reproduce-issue.md +20 -20
- package/skills/meta/docker/SKILL.md +126 -126
- package/skills/meta/examples/supabase/SKILL.md +46 -46
- package/skills/meta/examples/supabase/references/best-practices.md +319 -319
- package/skills/meta/examples/supabase/references/common-patterns.md +373 -373
- package/skills/meta/examples/supabase/templates/migration-template.sql +49 -49
- package/skills/meta/examples/supabase/templates/rls-policy-template.sql +77 -77
- package/skills/meta/examples/supabase/workflows/debugging.md +260 -260
- package/skills/meta/examples/supabase/workflows/migration-workflow.md +211 -211
- package/skills/meta/examples/supabase/workflows/rls-policies.md +244 -244
- package/skills/meta/examples/supabase/workflows/schema-design.md +321 -321
- package/skills/meta/file-todos/SKILL.md +88 -88
- package/skills/meta/mobile/SKILL.md +140 -140
- package/skills/meta/nextjs/SKILL.md +101 -101
- package/skills/meta/performance/SKILL.md +130 -130
- package/skills/meta/react-patterns/SKILL.md +83 -83
- package/skills/meta/security/SKILL.md +114 -114
- package/skills/meta/session-resume/SKILL.md +96 -96
- package/skills/meta/tailwind/SKILL.md +139 -139
- package/skills/meta/testing/SKILL.md +43 -43
- package/skills/meta/testing/references/vitest-patterns.md +45 -45
- package/skills/meta/testing/templates/component-test.template.tsx +37 -37
- package/skills/tech/alpha-vantage/SKILL.md +142 -142
- package/skills/tech/alpha-vantage/references/commodities.md +153 -153
- package/skills/tech/alpha-vantage/references/economic-indicators.md +158 -158
- package/skills/tech/alpha-vantage/references/forex-crypto.md +154 -154
- package/skills/tech/alpha-vantage/references/fundamentals.md +223 -223
- package/skills/tech/alpha-vantage/references/intelligence.md +138 -138
- package/skills/tech/alpha-vantage/references/options.md +93 -93
- package/skills/tech/alpha-vantage/references/technical-indicators.md +374 -374
- package/skills/tech/alpha-vantage/references/time-series.md +157 -157
- package/skills/tech/doc.md +6 -6
- package/skills/tech/financial-modeling/SKILL.md +18 -18
- package/skills/tech/financial-modeling/skills/3-statements/SKILL.md +368 -368
- package/skills/tech/financial-modeling/skills/3-statements/references/formatting.md +118 -118
- package/skills/tech/financial-modeling/skills/3-statements/references/formulas.md +292 -292
- package/skills/tech/financial-modeling/skills/3-statements/references/sec-filings.md +125 -125
- package/skills/tech/financial-modeling/skills/dcf-model/SKILL.md +1210 -1210
- package/skills/tech/financial-modeling/skills/dcf-model/TROUBLESHOOTING.md +40 -40
- package/skills/tech/financial-modeling/skills/dcf-model/requirements.txt +8 -8
- package/skills/tech/financial-modeling/skills/dcf-model/scripts/validate_dcf.py +292 -292
- package/skills/tech/financial-modeling/skills/lbo-model/SKILL.md +236 -236
- package/skills/tech/financial-modeling/skills/merger-model/SKILL.md +108 -108
- package/skills/workflows/README.md +203 -203
- package/skills/workflows/adr.md +174 -174
- package/skills/workflows/changelog.md +74 -74
- package/skills/workflows/compound.md +323 -323
- package/skills/workflows/compound_health.md +74 -74
- package/skills/workflows/create-agent-skill.md +138 -139
- package/skills/workflows/cycle.md +144 -144
- package/skills/workflows/deploy-docs.md +84 -84
- package/skills/workflows/development-rules.md +42 -42
- package/skills/workflows/doc.md +95 -95
- package/skills/workflows/documentation-management.md +34 -34
- package/skills/workflows/explore.md +146 -146
- package/skills/workflows/generate_command.md +106 -106
- package/skills/workflows/heal-skill.md +97 -97
- package/skills/workflows/housekeeping.md +229 -229
- package/skills/workflows/kit-setup.md +102 -102
- package/skills/workflows/map-codebase.md +78 -78
- package/skills/workflows/orchestration-protocol.md +43 -43
- package/skills/workflows/plan-compound.md +439 -439
- package/skills/workflows/plan_review.md +269 -269
- package/skills/workflows/primary-workflow.md +37 -37
- package/skills/workflows/promote_pattern.md +86 -86
- package/skills/workflows/release-docs.md +82 -82
- package/skills/workflows/report-bug.md +135 -135
- package/skills/workflows/reproduce-bug.md +118 -118
- package/skills/workflows/resolve_pr.md +133 -133
- package/skills/workflows/resolve_todo.md +128 -128
- package/skills/workflows/review-compound.md +376 -376
- package/skills/workflows/skill-review.md +127 -127
- package/skills/workflows/specs.md +257 -257
- package/skills/workflows/triage-sprint.md +102 -102
- package/skills/workflows/triage.md +152 -152
- package/skills/workflows/work.md +399 -399
- package/skills/workflows/xcode-test.md +93 -93
package/commands/review.toml
CHANGED
|
@@ -1,191 +1,191 @@
|
|
|
1
|
-
description = "Review code quality with severity categories (Code Reviewer Agent)"
|
|
2
|
-
|
|
3
|
-
prompt = """
|
|
4
|
-
# 👁️ CODE REVIEWER AGENT
|
|
5
|
-
|
|
6
|
-
Review request:
|
|
7
|
-
|
|
8
|
-
**Scope:** {{args}}
|
|
9
|
-
|
|
10
|
-
## SCOPE DETECTION
|
|
11
|
-
|
|
12
|
-
| Scope | Trigger | Mode |
|
|
13
|
-
|-------|---------|------|
|
|
14
|
-
| **Codebase** | "codebase", "full", "all" | Full codebase review |
|
|
15
|
-
| **Directory** | path ends with `/` | Review all files in dir |
|
|
16
|
-
| **Feature** | feature name | Review related files |
|
|
17
|
-
| **Security** | "security", "audit" | OWASP compliance |
|
|
18
|
-
| **Types** | "types", "any" | TypeScript audit |
|
|
19
|
-
| **Performance** | "performance", "slow" | N+1, indexes |
|
|
20
|
-
|
|
21
|
-
---
|
|
22
|
-
|
|
23
|
-
## CODEBASE MODE (Full Review)
|
|
24
|
-
|
|
25
|
-
When triggered with "codebase"/"full"/"all":
|
|
26
|
-
|
|
27
|
-
### 1. Research Phase
|
|
28
|
-
- Scan entire project structure
|
|
29
|
-
- Identify key files and patterns
|
|
30
|
-
- Detect frameworks and dependencies
|
|
31
|
-
|
|
32
|
-
### 2. Code Review Phase
|
|
33
|
-
- Check all source files
|
|
34
|
-
- Identify issues, duplicates, vulnerabilities
|
|
35
|
-
- Generate severity-categorized findings
|
|
36
|
-
|
|
37
|
-
### 3. Planning Phase
|
|
38
|
-
Create improvement plan:
|
|
39
|
-
```
|
|
40
|
-
plans/YYYYMMDD-review/
|
|
41
|
-
├── plan.md # Overview
|
|
42
|
-
└── phase-XX-name.md # Detailed fixes
|
|
43
|
-
```
|
|
44
|
-
|
|
45
|
-
### 4. Final Report
|
|
46
|
-
- Summary of all findings
|
|
47
|
-
- Prioritized fix recommendations
|
|
48
|
-
- Next steps with commands
|
|
49
|
-
|
|
50
|
-
---
|
|
51
|
-
|
|
52
|
-
## REVIEW OUTPUT FORMAT
|
|
53
|
-
|
|
54
|
-
### 📊 Summary
|
|
55
|
-
| Category | Count |
|
|
56
|
-
|----------|-------|
|
|
57
|
-
| 🔴 Critical | X |
|
|
58
|
-
| 🟠 High | X |
|
|
59
|
-
| 🟡 Medium | X |
|
|
60
|
-
| 🟢 Low | X |
|
|
61
|
-
|
|
62
|
-
**Verdict:** ❌ Block merge / ⚠️ Fix recommended / ✅ Ready
|
|
63
|
-
|
|
64
|
-
---
|
|
65
|
-
|
|
66
|
-
### 🔴 CRITICAL (Must Fix Before Merge)
|
|
67
|
-
Issues that can cause security breaches or data loss.
|
|
68
|
-
|
|
69
|
-
#### Issue 1: [Title]
|
|
70
|
-
- **File:** `path/to/file.ts:42`
|
|
71
|
-
- **Problem:** [Description]
|
|
72
|
-
- **Risk:** [What can go wrong]
|
|
73
|
-
- **Fix:**
|
|
74
|
-
```typescript
|
|
75
|
-
// Before
|
|
76
|
-
[problematic code]
|
|
77
|
-
|
|
78
|
-
// After
|
|
79
|
-
[fixed code]
|
|
80
|
-
```
|
|
81
|
-
|
|
82
|
-
---
|
|
83
|
-
|
|
84
|
-
### 🟠 HIGH (Should Fix)
|
|
85
|
-
Performance, type safety, reliability issues.
|
|
86
|
-
|
|
87
|
-
#### Issue 1: [Title]
|
|
88
|
-
- **File:** `path/to/file.ts:15`
|
|
89
|
-
- **Problem:** [Description]
|
|
90
|
-
- **Fix:** [Recommendation]
|
|
91
|
-
|
|
92
|
-
---
|
|
93
|
-
|
|
94
|
-
### 🟡 MEDIUM (Recommended)
|
|
95
|
-
Maintainability, code smells, patterns.
|
|
96
|
-
|
|
97
|
-
#### Issue 1: [Title]
|
|
98
|
-
- **Suggestion:** [Improvement]
|
|
99
|
-
|
|
100
|
-
---
|
|
101
|
-
|
|
102
|
-
### 🟢 LOW (Optional)
|
|
103
|
-
Style, minor improvements.
|
|
104
|
-
|
|
105
|
-
- [Item 1]
|
|
106
|
-
- [Item 2]
|
|
107
|
-
|
|
108
|
-
---
|
|
109
|
-
|
|
110
|
-
## SECURITY AUDIT (OWASP)
|
|
111
|
-
|
|
112
|
-
| Check | Status | Details |
|
|
113
|
-
|-------|--------|---------|
|
|
114
|
-
| Injection (SQL/NoSQL) | ✅/❌ | [Details] |
|
|
115
|
-
| Broken Auth | ✅/❌ | [Details] |
|
|
116
|
-
| Sensitive Data Exposure | ✅/❌ | [Details] |
|
|
117
|
-
| XSS | ✅/❌ | [Details] |
|
|
118
|
-
| Insecure Deserialization | ✅/❌ | [Details] |
|
|
119
|
-
| Using Vulnerable Components | ✅/❌ | [Details] |
|
|
120
|
-
| Logging & Monitoring | ✅/❌ | [Details] |
|
|
121
|
-
|
|
122
|
-
---
|
|
123
|
-
|
|
124
|
-
## TYPE SAFETY AUDIT
|
|
125
|
-
|
|
126
|
-
### `any` Type Locations
|
|
127
|
-
| File | Line | Variable | Suggested Type |
|
|
128
|
-
|------|------|----------|----------------|
|
|
129
|
-
| `file.ts` | 42 | `data` | `UserResponse` |
|
|
130
|
-
|
|
131
|
-
### Strict Mode Violations
|
|
132
|
-
- [ ] `noImplicitAny`
|
|
133
|
-
- [ ] `strictNullChecks`
|
|
134
|
-
- [ ] `strictFunctionTypes`
|
|
135
|
-
|
|
136
|
-
---
|
|
137
|
-
|
|
138
|
-
## PERFORMANCE ANALYSIS
|
|
139
|
-
|
|
140
|
-
### N+1 Query Detection
|
|
141
|
-
```typescript
|
|
142
|
-
// Problem: N+1 in loop
|
|
143
|
-
for (const user of users) {
|
|
144
|
-
const posts = await getPosts(user.id); // ❌ Query in loop
|
|
145
|
-
}
|
|
146
|
-
|
|
147
|
-
// Fix: Batch query
|
|
148
|
-
const posts = await getPostsByUserIds(users.map(u => u.id));
|
|
149
|
-
```
|
|
150
|
-
|
|
151
|
-
### Missing Indexes
|
|
152
|
-
- Table `users`: Add index on `email`
|
|
153
|
-
- Table `orders`: Add composite index on `(user_id, created_at)`
|
|
154
|
-
|
|
155
|
-
---
|
|
156
|
-
|
|
157
|
-
## QUALITY GATES
|
|
158
|
-
|
|
159
|
-
| Gate | Status | Target |
|
|
160
|
-
|------|--------|--------|
|
|
161
|
-
| Test Coverage | X% | 80% |
|
|
162
|
-
| Zero `any` Types | X found | 0 |
|
|
163
|
-
| Security Scan | ✅/❌ | Pass |
|
|
164
|
-
| No Critical Issues | X found | 0 |
|
|
165
|
-
|
|
166
|
-
---
|
|
167
|
-
|
|
168
|
-
## NEXT STEPS
|
|
169
|
-
|
|
170
|
-
```bash
|
|
171
|
-
# Fix critical issues
|
|
172
|
-
/fix [critical security issues]
|
|
173
|
-
|
|
174
|
-
# Run tests
|
|
175
|
-
/test
|
|
176
|
-
|
|
177
|
-
# Re-review
|
|
178
|
-
/review [same scope]
|
|
179
|
-
```
|
|
180
|
-
|
|
181
|
-
> **Key Takeaway:** Code reviewer prevents production incidents by catching security vulnerabilities, type safety violations, and performance issues before merge.
|
|
182
|
-
"""
|
|
183
|
-
|
|
184
|
-
# ---
|
|
185
|
-
# USAGE:
|
|
186
|
-
# /review src/auth/ - Review directory
|
|
187
|
-
# /review user authentication - Review feature
|
|
188
|
-
# /review security audit - OWASP compliance
|
|
189
|
-
# /review types check - Find any types
|
|
190
|
-
# /review performance - N+1, indexes
|
|
1
|
+
description = "Review code quality with severity categories (Code Reviewer Agent)"
|
|
2
|
+
|
|
3
|
+
prompt = """
|
|
4
|
+
# 👁️ CODE REVIEWER AGENT
|
|
5
|
+
|
|
6
|
+
Review request:
|
|
7
|
+
|
|
8
|
+
**Scope:** {{args}}
|
|
9
|
+
|
|
10
|
+
## SCOPE DETECTION
|
|
11
|
+
|
|
12
|
+
| Scope | Trigger | Mode |
|
|
13
|
+
|-------|---------|------|
|
|
14
|
+
| **Codebase** | "codebase", "full", "all" | Full codebase review |
|
|
15
|
+
| **Directory** | path ends with `/` | Review all files in dir |
|
|
16
|
+
| **Feature** | feature name | Review related files |
|
|
17
|
+
| **Security** | "security", "audit" | OWASP compliance |
|
|
18
|
+
| **Types** | "types", "any" | TypeScript audit |
|
|
19
|
+
| **Performance** | "performance", "slow" | N+1, indexes |
|
|
20
|
+
|
|
21
|
+
---
|
|
22
|
+
|
|
23
|
+
## CODEBASE MODE (Full Review)
|
|
24
|
+
|
|
25
|
+
When triggered with "codebase"/"full"/"all":
|
|
26
|
+
|
|
27
|
+
### 1. Research Phase
|
|
28
|
+
- Scan entire project structure
|
|
29
|
+
- Identify key files and patterns
|
|
30
|
+
- Detect frameworks and dependencies
|
|
31
|
+
|
|
32
|
+
### 2. Code Review Phase
|
|
33
|
+
- Check all source files
|
|
34
|
+
- Identify issues, duplicates, vulnerabilities
|
|
35
|
+
- Generate severity-categorized findings
|
|
36
|
+
|
|
37
|
+
### 3. Planning Phase
|
|
38
|
+
Create improvement plan:
|
|
39
|
+
```
|
|
40
|
+
plans/YYYYMMDD-review/
|
|
41
|
+
├── plan.md # Overview
|
|
42
|
+
└── phase-XX-name.md # Detailed fixes
|
|
43
|
+
```
|
|
44
|
+
|
|
45
|
+
### 4. Final Report
|
|
46
|
+
- Summary of all findings
|
|
47
|
+
- Prioritized fix recommendations
|
|
48
|
+
- Next steps with commands
|
|
49
|
+
|
|
50
|
+
---
|
|
51
|
+
|
|
52
|
+
## REVIEW OUTPUT FORMAT
|
|
53
|
+
|
|
54
|
+
### 📊 Summary
|
|
55
|
+
| Category | Count |
|
|
56
|
+
|----------|-------|
|
|
57
|
+
| 🔴 Critical | X |
|
|
58
|
+
| 🟠 High | X |
|
|
59
|
+
| 🟡 Medium | X |
|
|
60
|
+
| 🟢 Low | X |
|
|
61
|
+
|
|
62
|
+
**Verdict:** ❌ Block merge / ⚠️ Fix recommended / ✅ Ready
|
|
63
|
+
|
|
64
|
+
---
|
|
65
|
+
|
|
66
|
+
### 🔴 CRITICAL (Must Fix Before Merge)
|
|
67
|
+
Issues that can cause security breaches or data loss.
|
|
68
|
+
|
|
69
|
+
#### Issue 1: [Title]
|
|
70
|
+
- **File:** `path/to/file.ts:42`
|
|
71
|
+
- **Problem:** [Description]
|
|
72
|
+
- **Risk:** [What can go wrong]
|
|
73
|
+
- **Fix:**
|
|
74
|
+
```typescript
|
|
75
|
+
// Before
|
|
76
|
+
[problematic code]
|
|
77
|
+
|
|
78
|
+
// After
|
|
79
|
+
[fixed code]
|
|
80
|
+
```
|
|
81
|
+
|
|
82
|
+
---
|
|
83
|
+
|
|
84
|
+
### 🟠 HIGH (Should Fix)
|
|
85
|
+
Performance, type safety, reliability issues.
|
|
86
|
+
|
|
87
|
+
#### Issue 1: [Title]
|
|
88
|
+
- **File:** `path/to/file.ts:15`
|
|
89
|
+
- **Problem:** [Description]
|
|
90
|
+
- **Fix:** [Recommendation]
|
|
91
|
+
|
|
92
|
+
---
|
|
93
|
+
|
|
94
|
+
### 🟡 MEDIUM (Recommended)
|
|
95
|
+
Maintainability, code smells, patterns.
|
|
96
|
+
|
|
97
|
+
#### Issue 1: [Title]
|
|
98
|
+
- **Suggestion:** [Improvement]
|
|
99
|
+
|
|
100
|
+
---
|
|
101
|
+
|
|
102
|
+
### 🟢 LOW (Optional)
|
|
103
|
+
Style, minor improvements.
|
|
104
|
+
|
|
105
|
+
- [Item 1]
|
|
106
|
+
- [Item 2]
|
|
107
|
+
|
|
108
|
+
---
|
|
109
|
+
|
|
110
|
+
## SECURITY AUDIT (OWASP)
|
|
111
|
+
|
|
112
|
+
| Check | Status | Details |
|
|
113
|
+
|-------|--------|---------|
|
|
114
|
+
| Injection (SQL/NoSQL) | ✅/❌ | [Details] |
|
|
115
|
+
| Broken Auth | ✅/❌ | [Details] |
|
|
116
|
+
| Sensitive Data Exposure | ✅/❌ | [Details] |
|
|
117
|
+
| XSS | ✅/❌ | [Details] |
|
|
118
|
+
| Insecure Deserialization | ✅/❌ | [Details] |
|
|
119
|
+
| Using Vulnerable Components | ✅/❌ | [Details] |
|
|
120
|
+
| Logging & Monitoring | ✅/❌ | [Details] |
|
|
121
|
+
|
|
122
|
+
---
|
|
123
|
+
|
|
124
|
+
## TYPE SAFETY AUDIT
|
|
125
|
+
|
|
126
|
+
### `any` Type Locations
|
|
127
|
+
| File | Line | Variable | Suggested Type |
|
|
128
|
+
|------|------|----------|----------------|
|
|
129
|
+
| `file.ts` | 42 | `data` | `UserResponse` |
|
|
130
|
+
|
|
131
|
+
### Strict Mode Violations
|
|
132
|
+
- [ ] `noImplicitAny`
|
|
133
|
+
- [ ] `strictNullChecks`
|
|
134
|
+
- [ ] `strictFunctionTypes`
|
|
135
|
+
|
|
136
|
+
---
|
|
137
|
+
|
|
138
|
+
## PERFORMANCE ANALYSIS
|
|
139
|
+
|
|
140
|
+
### N+1 Query Detection
|
|
141
|
+
```typescript
|
|
142
|
+
// Problem: N+1 in loop
|
|
143
|
+
for (const user of users) {
|
|
144
|
+
const posts = await getPosts(user.id); // ❌ Query in loop
|
|
145
|
+
}
|
|
146
|
+
|
|
147
|
+
// Fix: Batch query
|
|
148
|
+
const posts = await getPostsByUserIds(users.map(u => u.id));
|
|
149
|
+
```
|
|
150
|
+
|
|
151
|
+
### Missing Indexes
|
|
152
|
+
- Table `users`: Add index on `email`
|
|
153
|
+
- Table `orders`: Add composite index on `(user_id, created_at)`
|
|
154
|
+
|
|
155
|
+
---
|
|
156
|
+
|
|
157
|
+
## QUALITY GATES
|
|
158
|
+
|
|
159
|
+
| Gate | Status | Target |
|
|
160
|
+
|------|--------|--------|
|
|
161
|
+
| Test Coverage | X% | 80% |
|
|
162
|
+
| Zero `any` Types | X found | 0 |
|
|
163
|
+
| Security Scan | ✅/❌ | Pass |
|
|
164
|
+
| No Critical Issues | X found | 0 |
|
|
165
|
+
|
|
166
|
+
---
|
|
167
|
+
|
|
168
|
+
## NEXT STEPS
|
|
169
|
+
|
|
170
|
+
```bash
|
|
171
|
+
# Fix critical issues
|
|
172
|
+
/fix [critical security issues]
|
|
173
|
+
|
|
174
|
+
# Run tests
|
|
175
|
+
/test
|
|
176
|
+
|
|
177
|
+
# Re-review
|
|
178
|
+
/review [same scope]
|
|
179
|
+
```
|
|
180
|
+
|
|
181
|
+
> **Key Takeaway:** Code reviewer prevents production incidents by catching security vulnerabilities, type safety violations, and performance issues before merge.
|
|
182
|
+
"""
|
|
183
|
+
|
|
184
|
+
# ---
|
|
185
|
+
# USAGE:
|
|
186
|
+
# /review src/auth/ - Review directory
|
|
187
|
+
# /review user authentication - Review feature
|
|
188
|
+
# /review security audit - OWASP compliance
|
|
189
|
+
# /review types check - Find any types
|
|
190
|
+
# /review performance - N+1, indexes
|
|
191
191
|
# ---
|
package/commands/scout-ext.toml
CHANGED
|
@@ -1,97 +1,97 @@
|
|
|
1
|
-
description = "Parallel external search cho large codebases (Scout External Agent)"
|
|
2
|
-
|
|
3
|
-
prompt = """
|
|
4
|
-
# 🔍 SCOUT EXTERNAL AGENT
|
|
5
|
-
|
|
6
|
-
Large-scale parallel search:
|
|
7
|
-
|
|
8
|
-
**Query:** {{args}}
|
|
9
|
-
|
|
10
|
-
## WHEN TO USE
|
|
11
|
-
- Codebase **500+ files**
|
|
12
|
-
- Feature work **spanning multiple directories**
|
|
13
|
-
- Need **3-5x faster** than internal Scout
|
|
14
|
-
- **Semantic search** beyond pattern matching
|
|
15
|
-
|
|
16
|
-
---
|
|
17
|
-
|
|
18
|
-
## PARALLEL EXECUTION STRATEGY
|
|
19
|
-
|
|
20
|
-
### Tool Selection
|
|
21
|
-
| Directories | Strategy |
|
|
22
|
-
|-------------|----------|
|
|
23
|
-
| ≤3 | Gemini only |
|
|
24
|
-
| >3 | Gemini + parallel agents |
|
|
25
|
-
|
|
26
|
-
### Execution Plan
|
|
27
|
-
1. **Identify target directories** - List all relevant dirs
|
|
28
|
-
2. **Launch parallel searches** - One per major area
|
|
29
|
-
3. **Set timeout** - 3 min per search
|
|
30
|
-
4. **Synthesize results** - Merge findings
|
|
31
|
-
|
|
32
|
-
---
|
|
33
|
-
|
|
34
|
-
## OUTPUT FORMAT
|
|
35
|
-
|
|
36
|
-
### 📊 Execution Summary
|
|
37
|
-
| Area | Status | Files Found | Time |
|
|
38
|
-
|------|--------|-------------|------|
|
|
39
|
-
| `lib/` | ✅ Complete | 12 | 45s |
|
|
40
|
-
| `app/api/` | ✅ Complete | 8 | 32s |
|
|
41
|
-
| `components/` | ⚠️ Partial | 5 | 180s (timeout) |
|
|
42
|
-
|
|
43
|
-
**Total:** X files in Y seconds
|
|
44
|
-
**Coverage:** ~85%
|
|
45
|
-
|
|
46
|
-
---
|
|
47
|
-
|
|
48
|
-
### 📂 Consolidated Findings
|
|
49
|
-
|
|
50
|
-
#### 🔷 Core Logic
|
|
51
|
-
- `lib/auth/...` - Authentication logic
|
|
52
|
-
- `lib/payment/...` - Payment processing
|
|
53
|
-
|
|
54
|
-
#### 🔷 API Layer
|
|
55
|
-
- `app/api/auth/...` - Auth endpoints
|
|
56
|
-
- `app/api/payment/...` - Payment routes
|
|
57
|
-
|
|
58
|
-
#### 🔷 UI Components
|
|
59
|
-
- `components/auth/...` - Login forms
|
|
60
|
-
|
|
61
|
-
#### 🔷 Configuration
|
|
62
|
-
- `config/...` - Environment settings
|
|
63
|
-
|
|
64
|
-
---
|
|
65
|
-
|
|
66
|
-
### ⚠️ Coverage Gaps
|
|
67
|
-
Files/areas NOT searched (timeout or exclusion):
|
|
68
|
-
- `node_modules/` (excluded)
|
|
69
|
-
- `components/legacy/` (partial - timeout)
|
|
70
|
-
|
|
71
|
-
---
|
|
72
|
-
|
|
73
|
-
### 💡 Large File Strategy
|
|
74
|
-
For files >25K tokens:
|
|
75
|
-
```bash
|
|
76
|
-
# Summarize large file
|
|
77
|
-
echo "Summarize auth logic in app/middleware/auth.ts" | gemini
|
|
78
|
-
```
|
|
79
|
-
|
|
80
|
-
---
|
|
81
|
-
|
|
82
|
-
## PRO TIPS
|
|
83
|
-
- **Parallel = speed** - Launch all searches together
|
|
84
|
-
- **Timeout OK** - Partial results still useful
|
|
85
|
-
- **Quality > Quantity** - 2-5 directories optimal
|
|
86
|
-
- **Chain with /plan** - After discovery, plan implementation
|
|
87
|
-
|
|
88
|
-
> **Key Takeaway:** Scout External parallelizes search for 3-5x speed. Use for large codebases when internal Scout is too slow.
|
|
89
|
-
"""
|
|
90
|
-
|
|
91
|
-
# ---
|
|
92
|
-
# USAGE:
|
|
93
|
-
# /scout-ext "Find all auth files across lib, api, components"
|
|
94
|
-
# /scout-ext "Locate payment processing in frontend + backend"
|
|
95
|
-
# /scout-ext "Search for all files importing @/lib/old-api"
|
|
96
|
-
# /scout-ext "Analyze API routes >500 lines for bottlenecks"
|
|
97
|
-
# ---
|
|
1
|
+
description = "Parallel external search cho large codebases (Scout External Agent)"
|
|
2
|
+
|
|
3
|
+
prompt = """
|
|
4
|
+
# 🔍 SCOUT EXTERNAL AGENT
|
|
5
|
+
|
|
6
|
+
Large-scale parallel search:
|
|
7
|
+
|
|
8
|
+
**Query:** {{args}}
|
|
9
|
+
|
|
10
|
+
## WHEN TO USE
|
|
11
|
+
- Codebase **500+ files**
|
|
12
|
+
- Feature work **spanning multiple directories**
|
|
13
|
+
- Need **3-5x faster** than internal Scout
|
|
14
|
+
- **Semantic search** beyond pattern matching
|
|
15
|
+
|
|
16
|
+
---
|
|
17
|
+
|
|
18
|
+
## PARALLEL EXECUTION STRATEGY
|
|
19
|
+
|
|
20
|
+
### Tool Selection
|
|
21
|
+
| Directories | Strategy |
|
|
22
|
+
|-------------|----------|
|
|
23
|
+
| ≤3 | Gemini only |
|
|
24
|
+
| >3 | Gemini + parallel agents |
|
|
25
|
+
|
|
26
|
+
### Execution Plan
|
|
27
|
+
1. **Identify target directories** - List all relevant dirs
|
|
28
|
+
2. **Launch parallel searches** - One per major area
|
|
29
|
+
3. **Set timeout** - 3 min per search
|
|
30
|
+
4. **Synthesize results** - Merge findings
|
|
31
|
+
|
|
32
|
+
---
|
|
33
|
+
|
|
34
|
+
## OUTPUT FORMAT
|
|
35
|
+
|
|
36
|
+
### 📊 Execution Summary
|
|
37
|
+
| Area | Status | Files Found | Time |
|
|
38
|
+
|------|--------|-------------|------|
|
|
39
|
+
| `lib/` | ✅ Complete | 12 | 45s |
|
|
40
|
+
| `app/api/` | ✅ Complete | 8 | 32s |
|
|
41
|
+
| `components/` | ⚠️ Partial | 5 | 180s (timeout) |
|
|
42
|
+
|
|
43
|
+
**Total:** X files in Y seconds
|
|
44
|
+
**Coverage:** ~85%
|
|
45
|
+
|
|
46
|
+
---
|
|
47
|
+
|
|
48
|
+
### 📂 Consolidated Findings
|
|
49
|
+
|
|
50
|
+
#### 🔷 Core Logic
|
|
51
|
+
- `lib/auth/...` - Authentication logic
|
|
52
|
+
- `lib/payment/...` - Payment processing
|
|
53
|
+
|
|
54
|
+
#### 🔷 API Layer
|
|
55
|
+
- `app/api/auth/...` - Auth endpoints
|
|
56
|
+
- `app/api/payment/...` - Payment routes
|
|
57
|
+
|
|
58
|
+
#### 🔷 UI Components
|
|
59
|
+
- `components/auth/...` - Login forms
|
|
60
|
+
|
|
61
|
+
#### 🔷 Configuration
|
|
62
|
+
- `config/...` - Environment settings
|
|
63
|
+
|
|
64
|
+
---
|
|
65
|
+
|
|
66
|
+
### ⚠️ Coverage Gaps
|
|
67
|
+
Files/areas NOT searched (timeout or exclusion):
|
|
68
|
+
- `node_modules/` (excluded)
|
|
69
|
+
- `components/legacy/` (partial - timeout)
|
|
70
|
+
|
|
71
|
+
---
|
|
72
|
+
|
|
73
|
+
### 💡 Large File Strategy
|
|
74
|
+
For files >25K tokens:
|
|
75
|
+
```bash
|
|
76
|
+
# Summarize large file
|
|
77
|
+
echo "Summarize auth logic in app/middleware/auth.ts" | gemini
|
|
78
|
+
```
|
|
79
|
+
|
|
80
|
+
---
|
|
81
|
+
|
|
82
|
+
## PRO TIPS
|
|
83
|
+
- **Parallel = speed** - Launch all searches together
|
|
84
|
+
- **Timeout OK** - Partial results still useful
|
|
85
|
+
- **Quality > Quantity** - 2-5 directories optimal
|
|
86
|
+
- **Chain with /plan** - After discovery, plan implementation
|
|
87
|
+
|
|
88
|
+
> **Key Takeaway:** Scout External parallelizes search for 3-5x speed. Use for large codebases when internal Scout is too slow.
|
|
89
|
+
"""
|
|
90
|
+
|
|
91
|
+
# ---
|
|
92
|
+
# USAGE:
|
|
93
|
+
# /scout-ext "Find all auth files across lib, api, components"
|
|
94
|
+
# /scout-ext "Locate payment processing in frontend + backend"
|
|
95
|
+
# /scout-ext "Search for all files importing @/lib/old-api"
|
|
96
|
+
# /scout-ext "Analyze API routes >500 lines for bottlenecks"
|
|
97
|
+
# ---
|