couchloop-eq-mcp 1.0.4 → 1.1.0
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/README.md +24 -11
- package/dist/developer/analyzers/bloat-detector.d.ts +89 -0
- package/dist/developer/analyzers/bloat-detector.d.ts.map +1 -0
- package/dist/developer/analyzers/bloat-detector.js +483 -0
- package/dist/developer/analyzers/bloat-detector.js.map +1 -0
- package/dist/developer/backup/auto-backup.d.ts +96 -0
- package/dist/developer/backup/auto-backup.d.ts.map +1 -0
- package/dist/developer/backup/auto-backup.js +346 -0
- package/dist/developer/backup/auto-backup.js.map +1 -0
- package/dist/developer/blockers/package-blocker.d.ts +33 -0
- package/dist/developer/blockers/package-blocker.d.ts.map +1 -0
- package/dist/developer/blockers/package-blocker.js +224 -0
- package/dist/developer/blockers/package-blocker.js.map +1 -0
- package/dist/developer/evaluators/ai-error-preventer.d.ts +54 -0
- package/dist/developer/evaluators/ai-error-preventer.d.ts.map +1 -0
- package/dist/developer/evaluators/ai-error-preventer.js +270 -0
- package/dist/developer/evaluators/ai-error-preventer.js.map +1 -0
- package/dist/developer/evaluators/build-context-detector.d.ts +44 -0
- package/dist/developer/evaluators/build-context-detector.d.ts.map +1 -0
- package/dist/developer/evaluators/build-context-detector.js +258 -0
- package/dist/developer/evaluators/build-context-detector.js.map +1 -0
- package/dist/developer/evaluators/package-evaluator.d.ts +37 -0
- package/dist/developer/evaluators/package-evaluator.d.ts.map +1 -0
- package/dist/developer/evaluators/package-evaluator.js +278 -0
- package/dist/developer/evaluators/package-evaluator.js.map +1 -0
- package/dist/developer/guards/file-guardian.d.ts +79 -0
- package/dist/developer/guards/file-guardian.d.ts.map +1 -0
- package/dist/developer/guards/file-guardian.js +309 -0
- package/dist/developer/guards/file-guardian.js.map +1 -0
- package/dist/developer/managers/context-manager.d.ts +61 -0
- package/dist/developer/managers/context-manager.d.ts.map +1 -0
- package/dist/developer/managers/context-manager.js +302 -0
- package/dist/developer/managers/context-manager.js.map +1 -0
- package/dist/developer/metrics/complexity-calculator.d.ts +52 -0
- package/dist/developer/metrics/complexity-calculator.d.ts.map +1 -0
- package/dist/developer/metrics/complexity-calculator.js +259 -0
- package/dist/developer/metrics/complexity-calculator.js.map +1 -0
- package/dist/developer/reports/review-summary.d.ts +49 -0
- package/dist/developer/reports/review-summary.d.ts.map +1 -0
- package/dist/developer/reports/review-summary.js +249 -0
- package/dist/developer/reports/review-summary.js.map +1 -0
- package/dist/developer/scanners/review-assistant.d.ts +41 -0
- package/dist/developer/scanners/review-assistant.d.ts.map +1 -0
- package/dist/developer/scanners/review-assistant.js +374 -0
- package/dist/developer/scanners/review-assistant.js.map +1 -0
- package/dist/developer/scanners/secret-scanner.d.ts +66 -0
- package/dist/developer/scanners/secret-scanner.d.ts.map +1 -0
- package/dist/developer/scanners/secret-scanner.js +287 -0
- package/dist/developer/scanners/secret-scanner.js.map +1 -0
- package/dist/developer/scanners/sql-injection-detector.d.ts +54 -0
- package/dist/developer/scanners/sql-injection-detector.d.ts.map +1 -0
- package/dist/developer/scanners/sql-injection-detector.js +174 -0
- package/dist/developer/scanners/sql-injection-detector.js.map +1 -0
- package/dist/developer/scanners/xss-detector.d.ts +60 -0
- package/dist/developer/scanners/xss-detector.d.ts.map +1 -0
- package/dist/developer/scanners/xss-detector.js +229 -0
- package/dist/developer/scanners/xss-detector.js.map +1 -0
- package/dist/developer/types/ai-errors.d.ts +34 -0
- package/dist/developer/types/ai-errors.d.ts.map +1 -0
- package/dist/developer/types/ai-errors.js +271 -0
- package/dist/developer/types/ai-errors.js.map +1 -0
- package/dist/developer/types/package.d.ts +32 -0
- package/dist/developer/types/package.d.ts.map +1 -0
- package/dist/developer/types/package.js +5 -0
- package/dist/developer/types/package.js.map +1 -0
- package/dist/developer/updaters/dependency-updater.d.ts +102 -0
- package/dist/developer/updaters/dependency-updater.d.ts.map +1 -0
- package/dist/developer/updaters/dependency-updater.js +472 -0
- package/dist/developer/updaters/dependency-updater.js.map +1 -0
- package/dist/developer/validators/cargo.d.ts +14 -0
- package/dist/developer/validators/cargo.d.ts.map +1 -0
- package/dist/developer/validators/cargo.js +132 -0
- package/dist/developer/validators/cargo.js.map +1 -0
- package/dist/developer/validators/gem.d.ts +14 -0
- package/dist/developer/validators/gem.d.ts.map +1 -0
- package/dist/developer/validators/gem.js +85 -0
- package/dist/developer/validators/gem.js.map +1 -0
- package/dist/developer/validators/go.d.ts +14 -0
- package/dist/developer/validators/go.d.ts.map +1 -0
- package/dist/developer/validators/go.js +138 -0
- package/dist/developer/validators/go.js.map +1 -0
- package/dist/developer/validators/maven.d.ts +14 -0
- package/dist/developer/validators/maven.d.ts.map +1 -0
- package/dist/developer/validators/maven.js +99 -0
- package/dist/developer/validators/maven.js.map +1 -0
- package/dist/developer/validators/npm.d.ts +14 -0
- package/dist/developer/validators/npm.d.ts.map +1 -0
- package/dist/developer/validators/npm.js +96 -0
- package/dist/developer/validators/npm.js.map +1 -0
- package/dist/developer/validators/nuget.d.ts +15 -0
- package/dist/developer/validators/nuget.d.ts.map +1 -0
- package/dist/developer/validators/nuget.js +107 -0
- package/dist/developer/validators/nuget.js.map +1 -0
- package/dist/developer/validators/pypi.d.ts +14 -0
- package/dist/developer/validators/pypi.d.ts.map +1 -0
- package/dist/developer/validators/pypi.js +118 -0
- package/dist/developer/validators/pypi.js.map +1 -0
- package/dist/developer/validators/registry-manager.d.ts +37 -0
- package/dist/developer/validators/registry-manager.d.ts.map +1 -0
- package/dist/developer/validators/registry-manager.js +89 -0
- package/dist/developer/validators/registry-manager.js.map +1 -0
- package/dist/developer/validators/version-checker.d.ts +145 -0
- package/dist/developer/validators/version-checker.d.ts.map +1 -0
- package/dist/developer/validators/version-checker.js +529 -0
- package/dist/developer/validators/version-checker.js.map +1 -0
- package/dist/server/index.js.map +1 -1
- package/dist/server/middleware/auth.d.ts +7 -9
- package/dist/server/middleware/auth.d.ts.map +1 -1
- package/dist/server/middleware/auth.js.map +1 -1
- package/dist/tools/check-versions.d.ts +100 -0
- package/dist/tools/check-versions.d.ts.map +1 -0
- package/dist/tools/check-versions.js +328 -0
- package/dist/tools/check-versions.js.map +1 -0
- package/dist/tools/detect-code-smell.d.ts +9 -0
- package/dist/tools/detect-code-smell.d.ts.map +1 -0
- package/dist/tools/detect-code-smell.js +231 -0
- package/dist/tools/detect-code-smell.js.map +1 -0
- package/dist/tools/index.d.ts +471 -0
- package/dist/tools/index.d.ts.map +1 -1
- package/dist/tools/index.js +178 -0
- package/dist/tools/index.js.map +1 -1
- package/dist/tools/journey.js +1 -1
- package/dist/tools/journey.js.map +1 -1
- package/dist/tools/pre-review-code.d.ts +71 -0
- package/dist/tools/pre-review-code.d.ts.map +1 -0
- package/dist/tools/pre-review-code.js +159 -0
- package/dist/tools/pre-review-code.js.map +1 -0
- package/dist/tools/preserve-context.d.ts +27 -0
- package/dist/tools/preserve-context.d.ts.map +1 -0
- package/dist/tools/preserve-context.js +98 -0
- package/dist/tools/preserve-context.js.map +1 -0
- package/dist/tools/protect-files.d.ts +224 -0
- package/dist/tools/protect-files.d.ts.map +1 -0
- package/dist/tools/protect-files.js +286 -0
- package/dist/tools/protect-files.js.map +1 -0
- package/dist/tools/scan-security.d.ts +38 -0
- package/dist/tools/scan-security.d.ts.map +1 -0
- package/dist/tools/scan-security.js +237 -0
- package/dist/tools/scan-security.js.map +1 -0
- package/dist/tools/validate_packages.d.ts +8 -0
- package/dist/tools/validate_packages.d.ts.map +1 -0
- package/dist/tools/validate_packages.js +159 -0
- package/dist/tools/validate_packages.js.map +1 -0
- package/dist/types/auth.d.ts.map +1 -1
- package/dist/types/auth.js +1 -2
- package/dist/types/auth.js.map +1 -1
- package/dist/types/context.d.ts +46 -0
- package/dist/types/context.d.ts.map +1 -0
- package/dist/types/context.js +17 -0
- package/dist/types/context.js.map +1 -0
- package/dist/types/file-protection.d.ts +50 -0
- package/dist/types/file-protection.d.ts.map +1 -0
- package/dist/types/file-protection.js +9 -0
- package/dist/types/file-protection.js.map +1 -0
- package/dist/utils/errorHandler.d.ts.map +1 -1
- package/dist/utils/errorHandler.js +2 -1
- package/dist/utils/errorHandler.js.map +1 -1
- package/package.json +30 -4
- package/dist/db/migrate.d.ts +0 -4
- package/dist/db/migrate.d.ts.map +0 -1
- package/dist/db/migrate.js +0 -34
- package/dist/db/migrate.js.map +0 -1
- package/dist/db/migrations/schema.d.ts +0 -1074
- package/dist/db/migrations/schema.d.ts.map +0 -1
- package/dist/db/migrations/schema.js +0 -160
- package/dist/db/migrations/schema.js.map +0 -1
- package/dist/db/schema.d.ts +0 -1576
- package/dist/db/schema.d.ts.map +0 -1
- package/dist/db/schema.js +0 -204
- package/dist/db/schema.js.map +0 -1
- package/dist/db/seed.d.ts +0 -4
- package/dist/db/seed.d.ts.map +0 -1
- package/dist/db/seed.js +0 -57
- package/dist/db/seed.js.map +0 -1
- package/dist/db/seedOAuth.d.ts +0 -4
- package/dist/db/seedOAuth.d.ts.map +0 -1
- package/dist/db/seedOAuth.js +0 -76
- package/dist/db/seedOAuth.js.map +0 -1
- package/dist/governance/config.d.ts +0 -66
- package/dist/governance/config.d.ts.map +0 -1
- package/dist/governance/config.js +0 -238
- package/dist/governance/config.js.map +0 -1
- package/dist/governance/detectors/hallucination.d.ts +0 -61
- package/dist/governance/detectors/hallucination.d.ts.map +0 -1
- package/dist/governance/detectors/hallucination.js +0 -338
- package/dist/governance/detectors/hallucination.js.map +0 -1
- package/dist/governance/detectors/inconsistency.d.ts +0 -99
- package/dist/governance/detectors/inconsistency.d.ts.map +0 -1
- package/dist/governance/detectors/inconsistency.js +0 -548
- package/dist/governance/detectors/inconsistency.js.map +0 -1
- package/dist/governance/detectors/toneDrift.d.ts +0 -63
- package/dist/governance/detectors/toneDrift.d.ts.map +0 -1
- package/dist/governance/detectors/toneDrift.js +0 -421
- package/dist/governance/detectors/toneDrift.js.map +0 -1
- package/dist/governance/detectors/unsafeReasoning.d.ts +0 -54
- package/dist/governance/detectors/unsafeReasoning.d.ts.map +0 -1
- package/dist/governance/detectors/unsafeReasoning.js +0 -473
- package/dist/governance/detectors/unsafeReasoning.js.map +0 -1
- package/dist/governance/evaluationEngine.d.ts +0 -112
- package/dist/governance/evaluationEngine.d.ts.map +0 -1
- package/dist/governance/evaluationEngine.js +0 -265
- package/dist/governance/evaluationEngine.js.map +0 -1
- package/dist/governance/intervention.d.ts +0 -81
- package/dist/governance/intervention.d.ts.map +0 -1
- package/dist/governance/intervention.js +0 -405
- package/dist/governance/intervention.js.map +0 -1
- package/dist/server/oauth/anomalyDetection.d.ts +0 -146
- package/dist/server/oauth/anomalyDetection.d.ts.map +0 -1
- package/dist/server/oauth/anomalyDetection.js +0 -405
- package/dist/server/oauth/anomalyDetection.js.map +0 -1
- package/dist/server/oauth/authServer.d.ts +0 -61
- package/dist/server/oauth/authServer.d.ts.map +0 -1
- package/dist/server/oauth/authServer.js +0 -283
- package/dist/server/oauth/authServer.js.map +0 -1
- package/dist/server/oauth/dpop.d.ts +0 -135
- package/dist/server/oauth/dpop.d.ts.map +0 -1
- package/dist/server/oauth/dpop.js +0 -338
- package/dist/server/oauth/dpop.js.map +0 -1
- package/dist/server/oauth/gdpr/consent.d.ts +0 -173
- package/dist/server/oauth/gdpr/consent.d.ts.map +0 -1
- package/dist/server/oauth/gdpr/consent.js +0 -388
- package/dist/server/oauth/gdpr/consent.js.map +0 -1
- package/dist/server/oauth/gdpr/dataPortability.d.ts +0 -214
- package/dist/server/oauth/gdpr/dataPortability.d.ts.map +0 -1
- package/dist/server/oauth/gdpr/dataPortability.js +0 -486
- package/dist/server/oauth/gdpr/dataPortability.js.map +0 -1
- package/dist/server/oauth/gdpr/index.d.ts +0 -103
- package/dist/server/oauth/gdpr/index.d.ts.map +0 -1
- package/dist/server/oauth/gdpr/index.js +0 -273
- package/dist/server/oauth/gdpr/index.js.map +0 -1
- package/dist/server/oauth/gdpr/rightToErasure.d.ts +0 -184
- package/dist/server/oauth/gdpr/rightToErasure.d.ts.map +0 -1
- package/dist/server/oauth/gdpr/rightToErasure.js +0 -527
- package/dist/server/oauth/gdpr/rightToErasure.js.map +0 -1
- package/dist/server/oauth/monitoring/securityMonitor.d.ts +0 -218
- package/dist/server/oauth/monitoring/securityMonitor.d.ts.map +0 -1
- package/dist/server/oauth/monitoring/securityMonitor.js +0 -615
- package/dist/server/oauth/monitoring/securityMonitor.js.map +0 -1
- package/dist/server/oauth/pkce.d.ts +0 -61
- package/dist/server/oauth/pkce.d.ts.map +0 -1
- package/dist/server/oauth/pkce.js +0 -157
- package/dist/server/oauth/pkce.js.map +0 -1
- package/dist/server/oauth/providers/base.d.ts +0 -147
- package/dist/server/oauth/providers/base.d.ts.map +0 -1
- package/dist/server/oauth/providers/base.js +0 -312
- package/dist/server/oauth/providers/base.js.map +0 -1
- package/dist/server/oauth/providers/github.d.ts +0 -55
- package/dist/server/oauth/providers/github.d.ts.map +0 -1
- package/dist/server/oauth/providers/github.js +0 -225
- package/dist/server/oauth/providers/github.js.map +0 -1
- package/dist/server/oauth/providers/google.d.ts +0 -49
- package/dist/server/oauth/providers/google.d.ts.map +0 -1
- package/dist/server/oauth/providers/google.js +0 -153
- package/dist/server/oauth/providers/google.js.map +0 -1
- package/dist/server/oauth/providers/index.d.ts +0 -9
- package/dist/server/oauth/providers/index.d.ts.map +0 -1
- package/dist/server/oauth/providers/index.js +0 -24
- package/dist/server/oauth/providers/index.js.map +0 -1
- package/dist/server/oauth/refreshTokenRotation.d.ts +0 -114
- package/dist/server/oauth/refreshTokenRotation.d.ts.map +0 -1
- package/dist/server/oauth/refreshTokenRotation.js +0 -344
- package/dist/server/oauth/refreshTokenRotation.js.map +0 -1
- package/dist/server/oauth/security.d.ts +0 -101
- package/dist/server/oauth/security.d.ts.map +0 -1
- package/dist/server/oauth/security.js +0 -268
- package/dist/server/oauth/security.js.map +0 -1
- package/dist/server/oauth/tokenEncryption.d.ts +0 -80
- package/dist/server/oauth/tokenEncryption.d.ts.map +0 -1
- package/dist/server/oauth/tokenEncryption.js +0 -218
- package/dist/server/oauth/tokenEncryption.js.map +0 -1
- package/dist/tools/sendMessage-complex-backup.d.ts +0 -6
- package/dist/tools/sendMessage-complex-backup.d.ts.map +0 -1
- package/dist/tools/sendMessage-complex-backup.js +0 -545
- package/dist/tools/sendMessage-complex-backup.js.map +0 -1
- package/dist/tools/sendMessage-revised.d.ts +0 -11
- package/dist/tools/sendMessage-revised.d.ts.map +0 -1
- package/dist/tools/sendMessage-revised.js +0 -429
- package/dist/tools/sendMessage-revised.js.map +0 -1
- package/dist/tools/sendMessage-truly-simple.d.ts +0 -8
- package/dist/tools/sendMessage-truly-simple.d.ts.map +0 -1
- package/dist/tools/sendMessage-truly-simple.js +0 -299
- package/dist/tools/sendMessage-truly-simple.js.map +0 -1
|
@@ -0,0 +1,374 @@
|
|
|
1
|
+
/**
|
|
2
|
+
* Review Assistant Scanner
|
|
3
|
+
* Scans code for common issues to reduce human review burden
|
|
4
|
+
* Detects: console logs, commented code, TODOs/FIXMEs, error handling, hardcoded values, type definitions, unreachable code
|
|
5
|
+
*/
|
|
6
|
+
export class ReviewAssistant {
|
|
7
|
+
lines;
|
|
8
|
+
language;
|
|
9
|
+
constructor(code, language = 'typescript') {
|
|
10
|
+
this.language = language.toLowerCase();
|
|
11
|
+
this.lines = code.split('\n');
|
|
12
|
+
}
|
|
13
|
+
scan() {
|
|
14
|
+
const issues = [];
|
|
15
|
+
issues.push(...this.findConsoleLogs());
|
|
16
|
+
issues.push(...this.findCommentedCode());
|
|
17
|
+
issues.push(...this.findTodos());
|
|
18
|
+
issues.push(...this.findMissingErrorHandling());
|
|
19
|
+
issues.push(...this.findHardcodedValues());
|
|
20
|
+
issues.push(...this.findMissingTypes());
|
|
21
|
+
issues.push(...this.findUnreachableCode());
|
|
22
|
+
issues.push(...this.findNestedComplexity());
|
|
23
|
+
return this.compileScanResult(issues);
|
|
24
|
+
}
|
|
25
|
+
findConsoleLogs() {
|
|
26
|
+
const issues = [];
|
|
27
|
+
const consolePattern = /\bconsole\.(log|debug|info|warn|error)\s*\(/gi;
|
|
28
|
+
this.lines.forEach((line, index) => {
|
|
29
|
+
if (line.trim().startsWith('//'))
|
|
30
|
+
return; // Skip comments
|
|
31
|
+
let match;
|
|
32
|
+
while ((match = consolePattern.exec(line)) !== null) {
|
|
33
|
+
issues.push({
|
|
34
|
+
line: index + 1,
|
|
35
|
+
column: match.index + 1,
|
|
36
|
+
type: 'console_log',
|
|
37
|
+
severity: 'low',
|
|
38
|
+
message: `Remove debug statement: console.${match[1]}()`,
|
|
39
|
+
code: line.trim(),
|
|
40
|
+
suggestion: `Remove this line before merging to production`
|
|
41
|
+
});
|
|
42
|
+
}
|
|
43
|
+
});
|
|
44
|
+
return issues;
|
|
45
|
+
}
|
|
46
|
+
findCommentedCode() {
|
|
47
|
+
const issues = [];
|
|
48
|
+
this.lines.forEach((line, index) => {
|
|
49
|
+
const trimmed = line.trim();
|
|
50
|
+
// Skip single-line comments that are actual documentation
|
|
51
|
+
if (trimmed.startsWith('//') && !trimmed.startsWith('// '))
|
|
52
|
+
return;
|
|
53
|
+
// Check for commented out code patterns
|
|
54
|
+
const commentedCodePatterns = [
|
|
55
|
+
/^\/\/\s*(const|let|var|function|if|for|while|return|async|await|import|export)\b/,
|
|
56
|
+
/^\/\/\s*\w+\s*[\.\[\(\{]/,
|
|
57
|
+
/^\/\/\s*}\s*$/,
|
|
58
|
+
/^\/\/\s*;/
|
|
59
|
+
];
|
|
60
|
+
for (const pattern of commentedCodePatterns) {
|
|
61
|
+
if (pattern.test(trimmed)) {
|
|
62
|
+
issues.push({
|
|
63
|
+
line: index + 1,
|
|
64
|
+
column: 1,
|
|
65
|
+
type: 'commented_code',
|
|
66
|
+
severity: 'low',
|
|
67
|
+
message: 'Remove commented out code',
|
|
68
|
+
code: line.trim(),
|
|
69
|
+
suggestion: 'Use version control to recover old code if needed'
|
|
70
|
+
});
|
|
71
|
+
break;
|
|
72
|
+
}
|
|
73
|
+
}
|
|
74
|
+
// Multi-line commented code
|
|
75
|
+
if (trimmed.startsWith('/*') && !trimmed.startsWith('/**')) {
|
|
76
|
+
let commentContent = trimmed.slice(2);
|
|
77
|
+
if (commentContent.includes('{') || commentContent.includes(';') || /\w+\s*=/i.test(commentContent)) {
|
|
78
|
+
issues.push({
|
|
79
|
+
line: index + 1,
|
|
80
|
+
column: 1,
|
|
81
|
+
type: 'commented_code',
|
|
82
|
+
severity: 'low',
|
|
83
|
+
message: 'Remove commented out code block',
|
|
84
|
+
code: line.trim(),
|
|
85
|
+
suggestion: 'Use version control instead'
|
|
86
|
+
});
|
|
87
|
+
}
|
|
88
|
+
}
|
|
89
|
+
});
|
|
90
|
+
return issues;
|
|
91
|
+
}
|
|
92
|
+
findTodos() {
|
|
93
|
+
const issues = [];
|
|
94
|
+
const todoPattern = /\b(TODO|todo)\b[:\s]*(.*?)(?=\n|$)/gi;
|
|
95
|
+
const fixmePattern = /\b(FIXME|fixme)\b[:\s]*(.*?)(?=\n|$)/gi;
|
|
96
|
+
this.lines.forEach((line, index) => {
|
|
97
|
+
let match;
|
|
98
|
+
while ((match = todoPattern.exec(line)) !== null) {
|
|
99
|
+
issues.push({
|
|
100
|
+
line: index + 1,
|
|
101
|
+
column: match.index + 1,
|
|
102
|
+
type: 'todo',
|
|
103
|
+
severity: 'medium',
|
|
104
|
+
message: `TODO: ${match[2]?.trim() || 'Item left incomplete'}`,
|
|
105
|
+
code: line.trim(),
|
|
106
|
+
suggestion: 'Complete this task before merge or create an issue'
|
|
107
|
+
});
|
|
108
|
+
}
|
|
109
|
+
while ((match = fixmePattern.exec(line)) !== null) {
|
|
110
|
+
issues.push({
|
|
111
|
+
line: index + 1,
|
|
112
|
+
column: match.index + 1,
|
|
113
|
+
type: 'fixme',
|
|
114
|
+
severity: 'high',
|
|
115
|
+
message: `FIXME: ${match[2]?.trim() || 'Issue needs to be fixed'}`,
|
|
116
|
+
code: line.trim(),
|
|
117
|
+
suggestion: 'This must be fixed before merge'
|
|
118
|
+
});
|
|
119
|
+
}
|
|
120
|
+
});
|
|
121
|
+
return issues;
|
|
122
|
+
}
|
|
123
|
+
findMissingErrorHandling() {
|
|
124
|
+
const issues = [];
|
|
125
|
+
this.lines.forEach((line, index) => {
|
|
126
|
+
// Check for fetch without error handling
|
|
127
|
+
if (/fetch\s*\([^)]*\)\s*\.then\s*\(/.test(line) && !this.hasErrorHandling(index)) {
|
|
128
|
+
issues.push({
|
|
129
|
+
line: index + 1,
|
|
130
|
+
column: 1,
|
|
131
|
+
type: 'missing_error_handling',
|
|
132
|
+
severity: 'high',
|
|
133
|
+
message: 'Missing error handling on fetch call',
|
|
134
|
+
code: line.trim(),
|
|
135
|
+
suggestion: 'Add .catch() handler or use try/catch with await'
|
|
136
|
+
});
|
|
137
|
+
}
|
|
138
|
+
// Check for Promise without catch
|
|
139
|
+
if (/\.then\s*\(/.test(line) && !this.hasErrorHandling(index)) {
|
|
140
|
+
const hasCatch = this.checkMultilinePromiseChain(index);
|
|
141
|
+
if (!hasCatch) {
|
|
142
|
+
issues.push({
|
|
143
|
+
line: index + 1,
|
|
144
|
+
column: 1,
|
|
145
|
+
type: 'missing_error_handling',
|
|
146
|
+
severity: 'medium',
|
|
147
|
+
message: 'Promise chain may be missing .catch() error handler',
|
|
148
|
+
code: line.trim(),
|
|
149
|
+
suggestion: 'Add .catch() or ensure error handling in parent context'
|
|
150
|
+
});
|
|
151
|
+
}
|
|
152
|
+
}
|
|
153
|
+
// Check for try block without catch
|
|
154
|
+
if (/^\s*try\s*\{/.test(line)) {
|
|
155
|
+
const hasCatch = this.findMatchingCatch(index);
|
|
156
|
+
if (!hasCatch) {
|
|
157
|
+
issues.push({
|
|
158
|
+
line: index + 1,
|
|
159
|
+
column: 1,
|
|
160
|
+
type: 'missing_error_handling',
|
|
161
|
+
severity: 'high',
|
|
162
|
+
message: 'Try block without catch handler',
|
|
163
|
+
code: line.trim(),
|
|
164
|
+
suggestion: 'Add catch block or finally block for cleanup'
|
|
165
|
+
});
|
|
166
|
+
}
|
|
167
|
+
}
|
|
168
|
+
});
|
|
169
|
+
return issues;
|
|
170
|
+
}
|
|
171
|
+
findHardcodedValues() {
|
|
172
|
+
const issues = [];
|
|
173
|
+
const patterns = [
|
|
174
|
+
{ regex: /['"]sk-[a-zA-Z0-9]+['"]/, type: 'API key' },
|
|
175
|
+
{ regex: /['"]pk_[a-zA-Z0-9]+['"]/, type: 'API key' },
|
|
176
|
+
{ regex: /auth[_-]?token\s*[:=]\s*['"][^'"]+['"]/, type: 'Auth token' },
|
|
177
|
+
{ regex: /password\s*[:=]\s*['"][^'"]*['"]/, type: 'Password' },
|
|
178
|
+
{ regex: /secret\s*[:=]\s*['"][^'"]+['"]/, type: 'Secret' },
|
|
179
|
+
{ regex: /(['"]https?:\/\/[^'"]+['"])\s*;/, type: 'Hardcoded URL' },
|
|
180
|
+
{ regex: /api[_-]?key\s*[:=]\s*['"][^'"]+['"]/, type: 'API key' }
|
|
181
|
+
];
|
|
182
|
+
this.lines.forEach((line, index) => {
|
|
183
|
+
if (line.trim().startsWith('//'))
|
|
184
|
+
return;
|
|
185
|
+
for (const pattern of patterns) {
|
|
186
|
+
if (pattern.regex.test(line)) {
|
|
187
|
+
issues.push({
|
|
188
|
+
line: index + 1,
|
|
189
|
+
column: 1,
|
|
190
|
+
type: 'hardcoded_value',
|
|
191
|
+
severity: 'high',
|
|
192
|
+
message: `Hardcoded ${pattern.type} detected`,
|
|
193
|
+
code: line.trim(),
|
|
194
|
+
suggestion: `Use environment variable instead (e.g., process.env.${pattern.type.replace(/[^A-Z0-9]/gi, '_').toUpperCase()})`
|
|
195
|
+
});
|
|
196
|
+
}
|
|
197
|
+
}
|
|
198
|
+
});
|
|
199
|
+
return issues;
|
|
200
|
+
}
|
|
201
|
+
findMissingTypes() {
|
|
202
|
+
const issues = [];
|
|
203
|
+
if (!['typescript', 'ts'].includes(this.language)) {
|
|
204
|
+
return issues;
|
|
205
|
+
}
|
|
206
|
+
this.lines.forEach((line, index) => {
|
|
207
|
+
// Check for function parameters without types
|
|
208
|
+
const funcParamPattern = /function\s+\w+\s*\(\s*(\w+)\s*[,\)]/;
|
|
209
|
+
const arrowFuncPattern = /(\w+)\s*=>\s*/;
|
|
210
|
+
if (funcParamPattern.test(line) || arrowFuncPattern.test(line)) {
|
|
211
|
+
const match = line.match(/\(([^)]*)\)/);
|
|
212
|
+
if (match && match[1]) {
|
|
213
|
+
const params = match[1].split(',').map(p => p.trim());
|
|
214
|
+
for (const param of params) {
|
|
215
|
+
if (param && !param.includes(':') && !param.includes('?') && param !== '...rest') {
|
|
216
|
+
issues.push({
|
|
217
|
+
line: index + 1,
|
|
218
|
+
column: 1,
|
|
219
|
+
type: 'missing_types',
|
|
220
|
+
severity: 'medium',
|
|
221
|
+
message: `Parameter "${param}" missing type annotation`,
|
|
222
|
+
code: line.trim(),
|
|
223
|
+
suggestion: `Add type: (${param}: ParameterType) or enable implicit any`
|
|
224
|
+
});
|
|
225
|
+
break;
|
|
226
|
+
}
|
|
227
|
+
}
|
|
228
|
+
}
|
|
229
|
+
}
|
|
230
|
+
// Check for variables without types
|
|
231
|
+
if (/^\s*(const|let|var)\s+\w+\s*=/.test(line) && !line.includes(':')) {
|
|
232
|
+
const match = line.match(/^\s*(const|let|var)\s+(\w+)\s*=/);
|
|
233
|
+
if (match && !this.isObviouslyTyped(line)) {
|
|
234
|
+
issues.push({
|
|
235
|
+
line: index + 1,
|
|
236
|
+
column: 1,
|
|
237
|
+
type: 'missing_types',
|
|
238
|
+
severity: 'low',
|
|
239
|
+
message: `Variable "${match[2]}" could have explicit type annotation`,
|
|
240
|
+
code: line.trim(),
|
|
241
|
+
suggestion: `Add type: const ${match[2]}: Type = ...`
|
|
242
|
+
});
|
|
243
|
+
}
|
|
244
|
+
}
|
|
245
|
+
});
|
|
246
|
+
return issues;
|
|
247
|
+
}
|
|
248
|
+
findUnreachableCode() {
|
|
249
|
+
const issues = [];
|
|
250
|
+
this.lines.forEach((line, index) => {
|
|
251
|
+
if (line.trim() === 'return;' || line.trim().startsWith('return ')) {
|
|
252
|
+
// Check if there's code after return in same block
|
|
253
|
+
for (let i = index + 1; i < this.lines.length && i < index + 5; i++) {
|
|
254
|
+
const nextLine = this.lines[i]?.trim();
|
|
255
|
+
if (!nextLine || nextLine.startsWith('//'))
|
|
256
|
+
continue;
|
|
257
|
+
if (nextLine.startsWith('}'))
|
|
258
|
+
break;
|
|
259
|
+
if (!nextLine.startsWith('case') && !nextLine.startsWith('default:')) {
|
|
260
|
+
issues.push({
|
|
261
|
+
line: i + 1,
|
|
262
|
+
column: 1,
|
|
263
|
+
type: 'unreachable_code',
|
|
264
|
+
severity: 'medium',
|
|
265
|
+
message: 'Code unreachable due to return statement above',
|
|
266
|
+
code: nextLine,
|
|
267
|
+
suggestion: 'Remove this code or reorganize logic'
|
|
268
|
+
});
|
|
269
|
+
break;
|
|
270
|
+
}
|
|
271
|
+
}
|
|
272
|
+
}
|
|
273
|
+
});
|
|
274
|
+
return issues;
|
|
275
|
+
}
|
|
276
|
+
findNestedComplexity() {
|
|
277
|
+
const issues = [];
|
|
278
|
+
this.lines.forEach((line, index) => {
|
|
279
|
+
let currentDepth = this.getCurrentBraceDepth(index);
|
|
280
|
+
if (currentDepth >= 4) {
|
|
281
|
+
if (/^\s*(if|for|while|switch)\s*/.test(line)) {
|
|
282
|
+
issues.push({
|
|
283
|
+
line: index + 1,
|
|
284
|
+
column: 1,
|
|
285
|
+
type: 'nested_complexity',
|
|
286
|
+
severity: 'medium',
|
|
287
|
+
message: `High nesting depth (${currentDepth} levels) - consider refactoring`,
|
|
288
|
+
code: line.trim(),
|
|
289
|
+
suggestion: 'Extract to separate function or simplify control flow'
|
|
290
|
+
});
|
|
291
|
+
}
|
|
292
|
+
}
|
|
293
|
+
});
|
|
294
|
+
return issues;
|
|
295
|
+
}
|
|
296
|
+
hasErrorHandling(lineIndex) {
|
|
297
|
+
const nextLines = this.lines.slice(lineIndex, Math.min(lineIndex + 5, this.lines.length));
|
|
298
|
+
return nextLines.some(line => /\.catch\s*\(|\.finally\s*\(/.test(line));
|
|
299
|
+
}
|
|
300
|
+
checkMultilinePromiseChain(lineIndex) {
|
|
301
|
+
for (let i = lineIndex; i < Math.min(lineIndex + 10, this.lines.length); i++) {
|
|
302
|
+
const line = this.lines[i];
|
|
303
|
+
if (!line)
|
|
304
|
+
continue;
|
|
305
|
+
if (/\.catch\s*\(/.test(line))
|
|
306
|
+
return true;
|
|
307
|
+
if (/^\s*[}\);]/.test(line) && i > lineIndex)
|
|
308
|
+
break;
|
|
309
|
+
}
|
|
310
|
+
return false;
|
|
311
|
+
}
|
|
312
|
+
findMatchingCatch(tryLineIndex) {
|
|
313
|
+
let braceCount = 0;
|
|
314
|
+
let foundOpenBrace = false;
|
|
315
|
+
for (let i = tryLineIndex; i < Math.min(tryLineIndex + 50, this.lines.length); i++) {
|
|
316
|
+
const line = this.lines[i];
|
|
317
|
+
if (!line)
|
|
318
|
+
continue;
|
|
319
|
+
if (!foundOpenBrace && line.includes('{')) {
|
|
320
|
+
foundOpenBrace = true;
|
|
321
|
+
braceCount += (line.match(/\{/g) || []).length;
|
|
322
|
+
braceCount -= (line.match(/\}/g) || []).length;
|
|
323
|
+
}
|
|
324
|
+
else if (foundOpenBrace) {
|
|
325
|
+
braceCount += (line.match(/\{/g) || []).length;
|
|
326
|
+
braceCount -= (line.match(/\}/g) || []).length;
|
|
327
|
+
if (braceCount <= 0 && /^\s*\}\s*catch\s*\(/.test(line)) {
|
|
328
|
+
return true;
|
|
329
|
+
}
|
|
330
|
+
if (braceCount <= 0)
|
|
331
|
+
break;
|
|
332
|
+
}
|
|
333
|
+
}
|
|
334
|
+
return false;
|
|
335
|
+
}
|
|
336
|
+
getCurrentBraceDepth(lineIndex) {
|
|
337
|
+
let depth = 0;
|
|
338
|
+
for (let i = 0; i <= lineIndex; i++) {
|
|
339
|
+
const line = this.lines[i];
|
|
340
|
+
if (!line)
|
|
341
|
+
continue;
|
|
342
|
+
depth += (line.match(/\{/g) || []).length;
|
|
343
|
+
depth -= (line.match(/\}/g) || []).length;
|
|
344
|
+
}
|
|
345
|
+
return Math.max(0, depth);
|
|
346
|
+
}
|
|
347
|
+
isObviouslyTyped(line) {
|
|
348
|
+
return /= (true|false|\d+|['"]|null|undefined|\[|\{)/.test(line);
|
|
349
|
+
}
|
|
350
|
+
compileScanResult(issues) {
|
|
351
|
+
const issuesByType = {};
|
|
352
|
+
const issuesBySeverity = {
|
|
353
|
+
low: 0,
|
|
354
|
+
medium: 0,
|
|
355
|
+
high: 0
|
|
356
|
+
};
|
|
357
|
+
issues.forEach(issue => {
|
|
358
|
+
issuesByType[issue.type] = (issuesByType[issue.type] || 0) + 1;
|
|
359
|
+
if (issue.severity in issuesBySeverity) {
|
|
360
|
+
const severity = issuesBySeverity[issue.severity];
|
|
361
|
+
if (typeof severity === 'number') {
|
|
362
|
+
issuesBySeverity[issue.severity] = severity + 1;
|
|
363
|
+
}
|
|
364
|
+
}
|
|
365
|
+
});
|
|
366
|
+
return {
|
|
367
|
+
issues,
|
|
368
|
+
totalIssues: issues.length,
|
|
369
|
+
issuesByType,
|
|
370
|
+
issuesBySeverity
|
|
371
|
+
};
|
|
372
|
+
}
|
|
373
|
+
}
|
|
374
|
+
//# sourceMappingURL=review-assistant.js.map
|
|
@@ -0,0 +1 @@
|
|
|
1
|
+
{"version":3,"file":"review-assistant.js","sourceRoot":"","sources":["../../../src/developer/scanners/review-assistant.ts"],"names":[],"mappings":"AAAA;;;;GAIG;AAmBH,MAAM,OAAO,eAAe;IAClB,KAAK,CAAW;IAChB,QAAQ,CAAS;IAEzB,YAAY,IAAY,EAAE,WAAmB,YAAY;QACvD,IAAI,CAAC,QAAQ,GAAG,QAAQ,CAAC,WAAW,EAAE,CAAC;QACvC,IAAI,CAAC,KAAK,GAAG,IAAI,CAAC,KAAK,CAAC,IAAI,CAAC,CAAC;IAChC,CAAC;IAED,IAAI;QACF,MAAM,MAAM,GAAgB,EAAE,CAAC;QAE/B,MAAM,CAAC,IAAI,CAAC,GAAG,IAAI,CAAC,eAAe,EAAE,CAAC,CAAC;QACvC,MAAM,CAAC,IAAI,CAAC,GAAG,IAAI,CAAC,iBAAiB,EAAE,CAAC,CAAC;QACzC,MAAM,CAAC,IAAI,CAAC,GAAG,IAAI,CAAC,SAAS,EAAE,CAAC,CAAC;QACjC,MAAM,CAAC,IAAI,CAAC,GAAG,IAAI,CAAC,wBAAwB,EAAE,CAAC,CAAC;QAChD,MAAM,CAAC,IAAI,CAAC,GAAG,IAAI,CAAC,mBAAmB,EAAE,CAAC,CAAC;QAC3C,MAAM,CAAC,IAAI,CAAC,GAAG,IAAI,CAAC,gBAAgB,EAAE,CAAC,CAAC;QACxC,MAAM,CAAC,IAAI,CAAC,GAAG,IAAI,CAAC,mBAAmB,EAAE,CAAC,CAAC;QAC3C,MAAM,CAAC,IAAI,CAAC,GAAG,IAAI,CAAC,oBAAoB,EAAE,CAAC,CAAC;QAE5C,OAAO,IAAI,CAAC,iBAAiB,CAAC,MAAM,CAAC,CAAC;IACxC,CAAC;IAEO,eAAe;QACrB,MAAM,MAAM,GAAgB,EAAE,CAAC;QAC/B,MAAM,cAAc,GAAG,+CAA+C,CAAC;QAEvE,IAAI,CAAC,KAAK,CAAC,OAAO,CAAC,CAAC,IAAI,EAAE,KAAK,EAAE,EAAE;YACjC,IAAI,IAAI,CAAC,IAAI,EAAE,CAAC,UAAU,CAAC,IAAI,CAAC;gBAAE,OAAO,CAAC,gBAAgB;YAE1D,IAAI,KAAK,CAAC;YACV,OAAO,CAAC,KAAK,GAAG,cAAc,CAAC,IAAI,CAAC,IAAI,CAAC,CAAC,KAAK,IAAI,EAAE,CAAC;gBACpD,MAAM,CAAC,IAAI,CAAC;oBACV,IAAI,EAAE,KAAK,GAAG,CAAC;oBACf,MAAM,EAAE,KAAK,CAAC,KAAK,GAAG,CAAC;oBACvB,IAAI,EAAE,aAAa;oBACnB,QAAQ,EAAE,KAAK;oBACf,OAAO,EAAE,mCAAmC,KAAK,CAAC,CAAC,CAAC,IAAI;oBACxD,IAAI,EAAE,IAAI,CAAC,IAAI,EAAE;oBACjB,UAAU,EAAE,+CAA+C;iBAC5D,CAAC,CAAC;YACL,CAAC;QACH,CAAC,CAAC,CAAC;QAEH,OAAO,MAAM,CAAC;IAChB,CAAC;IAEO,iBAAiB;QACvB,MAAM,MAAM,GAAgB,EAAE,CAAC;QAE/B,IAAI,CAAC,KAAK,CAAC,OAAO,CAAC,CAAC,IAAI,EAAE,KAAK,EAAE,EAAE;YACjC,MAAM,OAAO,GAAG,IAAI,CAAC,IAAI,EAAE,CAAC;YAE5B,0DAA0D;YAC1D,IAAI,OAAO,CAAC,UAAU,CAAC,IAAI,CAAC,IAAI,CAAC,OAAO,CAAC,UAAU,CAAC,KAAK,CAAC;gBAAE,OAAO;YAEnE,wCAAwC;YACxC,MAAM,qBAAqB,GAAG;gBAC5B,kFAAkF;gBAClF,0BAA0B;gBAC1B,eAAe;gBACf,WAAW;aACZ,CAAC;YAEF,KAAK,MAAM,OAAO,IAAI,qBAAqB,EAAE,CAAC;gBAC5C,IAAI,OAAO,CAAC,IAAI,CAAC,OAAO,CAAC,EAAE,CAAC;oBAC1B,MAAM,CAAC,IAAI,CAAC;wBACV,IAAI,EAAE,KAAK,GAAG,CAAC;wBACf,MAAM,EAAE,CAAC;wBACT,IAAI,EAAE,gBAAgB;wBACtB,QAAQ,EAAE,KAAK;wBACf,OAAO,EAAE,2BAA2B;wBACpC,IAAI,EAAE,IAAI,CAAC,IAAI,EAAE;wBACjB,UAAU,EAAE,mDAAmD;qBAChE,CAAC,CAAC;oBACH,MAAM;gBACR,CAAC;YACH,CAAC;YAED,4BAA4B;YAC5B,IAAI,OAAO,CAAC,UAAU,CAAC,IAAI,CAAC,IAAI,CAAC,OAAO,CAAC,UAAU,CAAC,KAAK,CAAC,EAAE,CAAC;gBAC3D,IAAI,cAAc,GAAG,OAAO,CAAC,KAAK,CAAC,CAAC,CAAC,CAAC;gBACtC,IAAI,cAAc,CAAC,QAAQ,CAAC,GAAG,CAAC,IAAI,cAAc,CAAC,QAAQ,CAAC,GAAG,CAAC,IAAI,UAAU,CAAC,IAAI,CAAC,cAAc,CAAC,EAAE,CAAC;oBACpG,MAAM,CAAC,IAAI,CAAC;wBACV,IAAI,EAAE,KAAK,GAAG,CAAC;wBACf,MAAM,EAAE,CAAC;wBACT,IAAI,EAAE,gBAAgB;wBACtB,QAAQ,EAAE,KAAK;wBACf,OAAO,EAAE,iCAAiC;wBAC1C,IAAI,EAAE,IAAI,CAAC,IAAI,EAAE;wBACjB,UAAU,EAAE,6BAA6B;qBAC1C,CAAC,CAAC;gBACL,CAAC;YACH,CAAC;QACH,CAAC,CAAC,CAAC;QAEH,OAAO,MAAM,CAAC;IAChB,CAAC;IAEO,SAAS;QACf,MAAM,MAAM,GAAgB,EAAE,CAAC;QAC/B,MAAM,WAAW,GAAG,sCAAsC,CAAC;QAC3D,MAAM,YAAY,GAAG,wCAAwC,CAAC;QAE9D,IAAI,CAAC,KAAK,CAAC,OAAO,CAAC,CAAC,IAAI,EAAE,KAAK,EAAE,EAAE;YACjC,IAAI,KAAK,CAAC;YAEV,OAAO,CAAC,KAAK,GAAG,WAAW,CAAC,IAAI,CAAC,IAAI,CAAC,CAAC,KAAK,IAAI,EAAE,CAAC;gBACjD,MAAM,CAAC,IAAI,CAAC;oBACV,IAAI,EAAE,KAAK,GAAG,CAAC;oBACf,MAAM,EAAE,KAAK,CAAC,KAAK,GAAG,CAAC;oBACvB,IAAI,EAAE,MAAM;oBACZ,QAAQ,EAAE,QAAQ;oBAClB,OAAO,EAAE,SAAS,KAAK,CAAC,CAAC,CAAC,EAAE,IAAI,EAAE,IAAI,sBAAsB,EAAE;oBAC9D,IAAI,EAAE,IAAI,CAAC,IAAI,EAAE;oBACjB,UAAU,EAAE,oDAAoD;iBACjE,CAAC,CAAC;YACL,CAAC;YAED,OAAO,CAAC,KAAK,GAAG,YAAY,CAAC,IAAI,CAAC,IAAI,CAAC,CAAC,KAAK,IAAI,EAAE,CAAC;gBAClD,MAAM,CAAC,IAAI,CAAC;oBACV,IAAI,EAAE,KAAK,GAAG,CAAC;oBACf,MAAM,EAAE,KAAK,CAAC,KAAK,GAAG,CAAC;oBACvB,IAAI,EAAE,OAAO;oBACb,QAAQ,EAAE,MAAM;oBAChB,OAAO,EAAE,UAAU,KAAK,CAAC,CAAC,CAAC,EAAE,IAAI,EAAE,IAAI,yBAAyB,EAAE;oBAClE,IAAI,EAAE,IAAI,CAAC,IAAI,EAAE;oBACjB,UAAU,EAAE,iCAAiC;iBAC9C,CAAC,CAAC;YACL,CAAC;QACH,CAAC,CAAC,CAAC;QAEH,OAAO,MAAM,CAAC;IAChB,CAAC;IAEO,wBAAwB;QAC9B,MAAM,MAAM,GAAgB,EAAE,CAAC;QAE/B,IAAI,CAAC,KAAK,CAAC,OAAO,CAAC,CAAC,IAAI,EAAE,KAAK,EAAE,EAAE;YACjC,yCAAyC;YACzC,IAAI,iCAAiC,CAAC,IAAI,CAAC,IAAI,CAAC,IAAI,CAAC,IAAI,CAAC,gBAAgB,CAAC,KAAK,CAAC,EAAE,CAAC;gBAClF,MAAM,CAAC,IAAI,CAAC;oBACV,IAAI,EAAE,KAAK,GAAG,CAAC;oBACf,MAAM,EAAE,CAAC;oBACT,IAAI,EAAE,wBAAwB;oBAC9B,QAAQ,EAAE,MAAM;oBAChB,OAAO,EAAE,sCAAsC;oBAC/C,IAAI,EAAE,IAAI,CAAC,IAAI,EAAE;oBACjB,UAAU,EAAE,kDAAkD;iBAC/D,CAAC,CAAC;YACL,CAAC;YAED,kCAAkC;YAClC,IAAI,aAAa,CAAC,IAAI,CAAC,IAAI,CAAC,IAAI,CAAC,IAAI,CAAC,gBAAgB,CAAC,KAAK,CAAC,EAAE,CAAC;gBAC9D,MAAM,QAAQ,GAAG,IAAI,CAAC,0BAA0B,CAAC,KAAK,CAAC,CAAC;gBACxD,IAAI,CAAC,QAAQ,EAAE,CAAC;oBACd,MAAM,CAAC,IAAI,CAAC;wBACV,IAAI,EAAE,KAAK,GAAG,CAAC;wBACf,MAAM,EAAE,CAAC;wBACT,IAAI,EAAE,wBAAwB;wBAC9B,QAAQ,EAAE,QAAQ;wBAClB,OAAO,EAAE,qDAAqD;wBAC9D,IAAI,EAAE,IAAI,CAAC,IAAI,EAAE;wBACjB,UAAU,EAAE,yDAAyD;qBACtE,CAAC,CAAC;gBACL,CAAC;YACH,CAAC;YAED,oCAAoC;YACpC,IAAI,cAAc,CAAC,IAAI,CAAC,IAAI,CAAC,EAAE,CAAC;gBAC9B,MAAM,QAAQ,GAAG,IAAI,CAAC,iBAAiB,CAAC,KAAK,CAAC,CAAC;gBAC/C,IAAI,CAAC,QAAQ,EAAE,CAAC;oBACd,MAAM,CAAC,IAAI,CAAC;wBACV,IAAI,EAAE,KAAK,GAAG,CAAC;wBACf,MAAM,EAAE,CAAC;wBACT,IAAI,EAAE,wBAAwB;wBAC9B,QAAQ,EAAE,MAAM;wBAChB,OAAO,EAAE,iCAAiC;wBAC1C,IAAI,EAAE,IAAI,CAAC,IAAI,EAAE;wBACjB,UAAU,EAAE,8CAA8C;qBAC3D,CAAC,CAAC;gBACL,CAAC;YACH,CAAC;QACH,CAAC,CAAC,CAAC;QAEH,OAAO,MAAM,CAAC;IAChB,CAAC;IAEO,mBAAmB;QACzB,MAAM,MAAM,GAAgB,EAAE,CAAC;QAE/B,MAAM,QAAQ,GAAG;YACf,EAAE,KAAK,EAAE,yBAAyB,EAAE,IAAI,EAAE,SAAS,EAAE;YACrD,EAAE,KAAK,EAAE,yBAAyB,EAAE,IAAI,EAAE,SAAS,EAAE;YACrD,EAAE,KAAK,EAAE,wCAAwC,EAAE,IAAI,EAAE,YAAY,EAAE;YACvE,EAAE,KAAK,EAAE,kCAAkC,EAAE,IAAI,EAAE,UAAU,EAAE;YAC/D,EAAE,KAAK,EAAE,gCAAgC,EAAE,IAAI,EAAE,QAAQ,EAAE;YAC3D,EAAE,KAAK,EAAE,iCAAiC,EAAE,IAAI,EAAE,eAAe,EAAE;YACnE,EAAE,KAAK,EAAE,qCAAqC,EAAE,IAAI,EAAE,SAAS,EAAE;SAClE,CAAC;QAEF,IAAI,CAAC,KAAK,CAAC,OAAO,CAAC,CAAC,IAAI,EAAE,KAAK,EAAE,EAAE;YACjC,IAAI,IAAI,CAAC,IAAI,EAAE,CAAC,UAAU,CAAC,IAAI,CAAC;gBAAE,OAAO;YAEzC,KAAK,MAAM,OAAO,IAAI,QAAQ,EAAE,CAAC;gBAC/B,IAAI,OAAO,CAAC,KAAK,CAAC,IAAI,CAAC,IAAI,CAAC,EAAE,CAAC;oBAC7B,MAAM,CAAC,IAAI,CAAC;wBACV,IAAI,EAAE,KAAK,GAAG,CAAC;wBACf,MAAM,EAAE,CAAC;wBACT,IAAI,EAAE,iBAAiB;wBACvB,QAAQ,EAAE,MAAM;wBAChB,OAAO,EAAE,aAAa,OAAO,CAAC,IAAI,WAAW;wBAC7C,IAAI,EAAE,IAAI,CAAC,IAAI,EAAE;wBACjB,UAAU,EAAE,uDAAuD,OAAO,CAAC,IAAI,CAAC,OAAO,CAAC,aAAa,EAAE,GAAG,CAAC,CAAC,WAAW,EAAE,GAAG;qBAC7H,CAAC,CAAC;gBACL,CAAC;YACH,CAAC;QACH,CAAC,CAAC,CAAC;QAEH,OAAO,MAAM,CAAC;IAChB,CAAC;IAEO,gBAAgB;QACtB,MAAM,MAAM,GAAgB,EAAE,CAAC;QAE/B,IAAI,CAAC,CAAC,YAAY,EAAE,IAAI,CAAC,CAAC,QAAQ,CAAC,IAAI,CAAC,QAAQ,CAAC,EAAE,CAAC;YAClD,OAAO,MAAM,CAAC;QAChB,CAAC;QAED,IAAI,CAAC,KAAK,CAAC,OAAO,CAAC,CAAC,IAAI,EAAE,KAAK,EAAE,EAAE;YACjC,8CAA8C;YAC9C,MAAM,gBAAgB,GAAG,qCAAqC,CAAC;YAC/D,MAAM,gBAAgB,GAAG,eAAe,CAAC;YAEzC,IAAI,gBAAgB,CAAC,IAAI,CAAC,IAAI,CAAC,IAAI,gBAAgB,CAAC,IAAI,CAAC,IAAI,CAAC,EAAE,CAAC;gBAC/D,MAAM,KAAK,GAAG,IAAI,CAAC,KAAK,CAAC,aAAa,CAAC,CAAC;gBACxC,IAAI,KAAK,IAAI,KAAK,CAAC,CAAC,CAAC,EAAE,CAAC;oBACtB,MAAM,MAAM,GAAG,KAAK,CAAC,CAAC,CAAC,CAAC,KAAK,CAAC,GAAG,CAAC,CAAC,GAAG,CAAC,CAAC,CAAC,EAAE,CAAC,CAAC,CAAC,IAAI,EAAE,CAAC,CAAC;oBACtD,KAAK,MAAM,KAAK,IAAI,MAAM,EAAE,CAAC;wBAC3B,IAAI,KAAK,IAAI,CAAC,KAAK,CAAC,QAAQ,CAAC,GAAG,CAAC,IAAI,CAAC,KAAK,CAAC,QAAQ,CAAC,GAAG,CAAC,IAAI,KAAK,KAAK,SAAS,EAAE,CAAC;4BACjF,MAAM,CAAC,IAAI,CAAC;gCACV,IAAI,EAAE,KAAK,GAAG,CAAC;gCACf,MAAM,EAAE,CAAC;gCACT,IAAI,EAAE,eAAe;gCACrB,QAAQ,EAAE,QAAQ;gCAClB,OAAO,EAAE,cAAc,KAAK,2BAA2B;gCACvD,IAAI,EAAE,IAAI,CAAC,IAAI,EAAE;gCACjB,UAAU,EAAE,cAAc,KAAK,yCAAyC;6BACzE,CAAC,CAAC;4BACH,MAAM;wBACR,CAAC;oBACH,CAAC;gBACH,CAAC;YACH,CAAC;YAED,oCAAoC;YACpC,IAAI,+BAA+B,CAAC,IAAI,CAAC,IAAI,CAAC,IAAI,CAAC,IAAI,CAAC,QAAQ,CAAC,GAAG,CAAC,EAAE,CAAC;gBACtE,MAAM,KAAK,GAAG,IAAI,CAAC,KAAK,CAAC,iCAAiC,CAAC,CAAC;gBAC5D,IAAI,KAAK,IAAI,CAAC,IAAI,CAAC,gBAAgB,CAAC,IAAI,CAAC,EAAE,CAAC;oBAC1C,MAAM,CAAC,IAAI,CAAC;wBACV,IAAI,EAAE,KAAK,GAAG,CAAC;wBACf,MAAM,EAAE,CAAC;wBACT,IAAI,EAAE,eAAe;wBACrB,QAAQ,EAAE,KAAK;wBACf,OAAO,EAAE,aAAa,KAAK,CAAC,CAAC,CAAC,uCAAuC;wBACrE,IAAI,EAAE,IAAI,CAAC,IAAI,EAAE;wBACjB,UAAU,EAAE,mBAAmB,KAAK,CAAC,CAAC,CAAC,cAAc;qBACtD,CAAC,CAAC;gBACL,CAAC;YACH,CAAC;QACH,CAAC,CAAC,CAAC;QAEH,OAAO,MAAM,CAAC;IAChB,CAAC;IAEO,mBAAmB;QACzB,MAAM,MAAM,GAAgB,EAAE,CAAC;QAE/B,IAAI,CAAC,KAAK,CAAC,OAAO,CAAC,CAAC,IAAI,EAAE,KAAK,EAAE,EAAE;YACjC,IAAI,IAAI,CAAC,IAAI,EAAE,KAAK,SAAS,IAAI,IAAI,CAAC,IAAI,EAAE,CAAC,UAAU,CAAC,SAAS,CAAC,EAAE,CAAC;gBACnE,mDAAmD;gBACnD,KAAK,IAAI,CAAC,GAAG,KAAK,GAAG,CAAC,EAAE,CAAC,GAAG,IAAI,CAAC,KAAK,CAAC,MAAM,IAAI,CAAC,GAAG,KAAK,GAAG,CAAC,EAAE,CAAC,EAAE,EAAE,CAAC;oBACpE,MAAM,QAAQ,GAAG,IAAI,CAAC,KAAK,CAAC,CAAC,CAAC,EAAE,IAAI,EAAE,CAAC;oBACvC,IAAI,CAAC,QAAQ,IAAI,QAAQ,CAAC,UAAU,CAAC,IAAI,CAAC;wBAAE,SAAS;oBACrD,IAAI,QAAQ,CAAC,UAAU,CAAC,GAAG,CAAC;wBAAE,MAAM;oBACpC,IAAI,CAAC,QAAQ,CAAC,UAAU,CAAC,MAAM,CAAC,IAAI,CAAC,QAAQ,CAAC,UAAU,CAAC,UAAU,CAAC,EAAE,CAAC;wBACrE,MAAM,CAAC,IAAI,CAAC;4BACV,IAAI,EAAE,CAAC,GAAG,CAAC;4BACX,MAAM,EAAE,CAAC;4BACT,IAAI,EAAE,kBAAkB;4BACxB,QAAQ,EAAE,QAAQ;4BAClB,OAAO,EAAE,gDAAgD;4BACzD,IAAI,EAAE,QAAQ;4BACd,UAAU,EAAE,sCAAsC;yBACnD,CAAC,CAAC;wBACH,MAAM;oBACR,CAAC;gBACH,CAAC;YACH,CAAC;QACH,CAAC,CAAC,CAAC;QAEH,OAAO,MAAM,CAAC;IAChB,CAAC;IAEO,oBAAoB;QAC1B,MAAM,MAAM,GAAgB,EAAE,CAAC;QAE/B,IAAI,CAAC,KAAK,CAAC,OAAO,CAAC,CAAC,IAAI,EAAE,KAAK,EAAE,EAAE;YACjC,IAAI,YAAY,GAAG,IAAI,CAAC,oBAAoB,CAAC,KAAK,CAAC,CAAC;YAEpD,IAAI,YAAY,IAAI,CAAC,EAAE,CAAC;gBACtB,IAAI,8BAA8B,CAAC,IAAI,CAAC,IAAI,CAAC,EAAE,CAAC;oBAC9C,MAAM,CAAC,IAAI,CAAC;wBACV,IAAI,EAAE,KAAK,GAAG,CAAC;wBACf,MAAM,EAAE,CAAC;wBACT,IAAI,EAAE,mBAAmB;wBACzB,QAAQ,EAAE,QAAQ;wBAClB,OAAO,EAAE,uBAAuB,YAAY,iCAAiC;wBAC7E,IAAI,EAAE,IAAI,CAAC,IAAI,EAAE;wBACjB,UAAU,EAAE,uDAAuD;qBACpE,CAAC,CAAC;gBACL,CAAC;YACH,CAAC;QACH,CAAC,CAAC,CAAC;QAEH,OAAO,MAAM,CAAC;IAChB,CAAC;IAEO,gBAAgB,CAAC,SAAiB;QACxC,MAAM,SAAS,GAAG,IAAI,CAAC,KAAK,CAAC,KAAK,CAAC,SAAS,EAAE,IAAI,CAAC,GAAG,CAAC,SAAS,GAAG,CAAC,EAAE,IAAI,CAAC,KAAK,CAAC,MAAM,CAAC,CAAC,CAAC;QAC1F,OAAO,SAAS,CAAC,IAAI,CAAC,IAAI,CAAC,EAAE,CAAC,6BAA6B,CAAC,IAAI,CAAC,IAAI,CAAC,CAAC,CAAC;IAC1E,CAAC;IAEO,0BAA0B,CAAC,SAAiB;QAClD,KAAK,IAAI,CAAC,GAAG,SAAS,EAAE,CAAC,GAAG,IAAI,CAAC,GAAG,CAAC,SAAS,GAAG,EAAE,EAAE,IAAI,CAAC,KAAK,CAAC,MAAM,CAAC,EAAE,CAAC,EAAE,EAAE,CAAC;YAC7E,MAAM,IAAI,GAAG,IAAI,CAAC,KAAK,CAAC,CAAC,CAAC,CAAC;YAC3B,IAAI,CAAC,IAAI;gBAAE,SAAS;YACpB,IAAI,cAAc,CAAC,IAAI,CAAC,IAAI,CAAC;gBAAE,OAAO,IAAI,CAAC;YAC3C,IAAI,YAAY,CAAC,IAAI,CAAC,IAAI,CAAC,IAAI,CAAC,GAAG,SAAS;gBAAE,MAAM;QACtD,CAAC;QACD,OAAO,KAAK,CAAC;IACf,CAAC;IAEO,iBAAiB,CAAC,YAAoB;QAC5C,IAAI,UAAU,GAAG,CAAC,CAAC;QACnB,IAAI,cAAc,GAAG,KAAK,CAAC;QAE3B,KAAK,IAAI,CAAC,GAAG,YAAY,EAAE,CAAC,GAAG,IAAI,CAAC,GAAG,CAAC,YAAY,GAAG,EAAE,EAAE,IAAI,CAAC,KAAK,CAAC,MAAM,CAAC,EAAE,CAAC,EAAE,EAAE,CAAC;YACnF,MAAM,IAAI,GAAG,IAAI,CAAC,KAAK,CAAC,CAAC,CAAC,CAAC;YAC3B,IAAI,CAAC,IAAI;gBAAE,SAAS;YAEpB,IAAI,CAAC,cAAc,IAAI,IAAI,CAAC,QAAQ,CAAC,GAAG,CAAC,EAAE,CAAC;gBAC1C,cAAc,GAAG,IAAI,CAAC;gBACtB,UAAU,IAAI,CAAC,IAAI,CAAC,KAAK,CAAC,KAAK,CAAC,IAAI,EAAE,CAAC,CAAC,MAAM,CAAC;gBAC/C,UAAU,IAAI,CAAC,IAAI,CAAC,KAAK,CAAC,KAAK,CAAC,IAAI,EAAE,CAAC,CAAC,MAAM,CAAC;YACjD,CAAC;iBAAM,IAAI,cAAc,EAAE,CAAC;gBAC1B,UAAU,IAAI,CAAC,IAAI,CAAC,KAAK,CAAC,KAAK,CAAC,IAAI,EAAE,CAAC,CAAC,MAAM,CAAC;gBAC/C,UAAU,IAAI,CAAC,IAAI,CAAC,KAAK,CAAC,KAAK,CAAC,IAAI,EAAE,CAAC,CAAC,MAAM,CAAC;gBAE/C,IAAI,UAAU,IAAI,CAAC,IAAI,qBAAqB,CAAC,IAAI,CAAC,IAAI,CAAC,EAAE,CAAC;oBACxD,OAAO,IAAI,CAAC;gBACd,CAAC;gBAED,IAAI,UAAU,IAAI,CAAC;oBAAE,MAAM;YAC7B,CAAC;QACH,CAAC;QAED,OAAO,KAAK,CAAC;IACf,CAAC;IAEO,oBAAoB,CAAC,SAAiB;QAC5C,IAAI,KAAK,GAAG,CAAC,CAAC;QACd,KAAK,IAAI,CAAC,GAAG,CAAC,EAAE,CAAC,IAAI,SAAS,EAAE,CAAC,EAAE,EAAE,CAAC;YACpC,MAAM,IAAI,GAAG,IAAI,CAAC,KAAK,CAAC,CAAC,CAAC,CAAC;YAC3B,IAAI,CAAC,IAAI;gBAAE,SAAS;YACpB,KAAK,IAAI,CAAC,IAAI,CAAC,KAAK,CAAC,KAAK,CAAC,IAAI,EAAE,CAAC,CAAC,MAAM,CAAC;YAC1C,KAAK,IAAI,CAAC,IAAI,CAAC,KAAK,CAAC,KAAK,CAAC,IAAI,EAAE,CAAC,CAAC,MAAM,CAAC;QAC5C,CAAC;QACD,OAAO,IAAI,CAAC,GAAG,CAAC,CAAC,EAAE,KAAK,CAAC,CAAC;IAC5B,CAAC;IAEO,gBAAgB,CAAC,IAAY;QACnC,OAAO,8CAA8C,CAAC,IAAI,CAAC,IAAI,CAAC,CAAC;IACnE,CAAC;IAEO,iBAAiB,CAAC,MAAmB;QAC3C,MAAM,YAAY,GAA2B,EAAE,CAAC;QAChD,MAAM,gBAAgB,GAA2B;YAC/C,GAAG,EAAE,CAAC;YACN,MAAM,EAAE,CAAC;YACT,IAAI,EAAE,CAAC;SACR,CAAC;QAEF,MAAM,CAAC,OAAO,CAAC,KAAK,CAAC,EAAE;YACrB,YAAY,CAAC,KAAK,CAAC,IAAI,CAAC,GAAG,CAAC,YAAY,CAAC,KAAK,CAAC,IAAI,CAAC,IAAI,CAAC,CAAC,GAAG,CAAC,CAAC;YAC/D,IAAI,KAAK,CAAC,QAAQ,IAAI,gBAAgB,EAAE,CAAC;gBACvC,MAAM,QAAQ,GAAG,gBAAgB,CAAC,KAAK,CAAC,QAAQ,CAAC,CAAC;gBAClD,IAAI,OAAO,QAAQ,KAAK,QAAQ,EAAE,CAAC;oBACjC,gBAAgB,CAAC,KAAK,CAAC,QAAQ,CAAC,GAAG,QAAQ,GAAG,CAAC,CAAC;gBAClD,CAAC;YACH,CAAC;QACH,CAAC,CAAC,CAAC;QAEH,OAAO;YACL,MAAM;YACN,WAAW,EAAE,MAAM,CAAC,MAAM;YAC1B,YAAY;YACZ,gBAAgB;SACjB,CAAC;IACJ,CAAC;CACF"}
|
|
@@ -0,0 +1,66 @@
|
|
|
1
|
+
/**
|
|
2
|
+
* Secret Scanner
|
|
3
|
+
* Detects hardcoded secrets in code including:
|
|
4
|
+
* - API keys (AWS, OpenAI, Stripe, etc.)
|
|
5
|
+
* - Passwords
|
|
6
|
+
* - Private keys
|
|
7
|
+
* - Connection strings with credentials
|
|
8
|
+
* - Tokens and tokens
|
|
9
|
+
*/
|
|
10
|
+
export interface SecretVulnerability {
|
|
11
|
+
type: 'HARDCODED_API_KEY' | 'HARDCODED_PASSWORD' | 'PRIVATE_KEY' | 'CONNECTION_STRING' | 'JWT_TOKEN' | 'GENERIC_SECRET';
|
|
12
|
+
severity: 'CRITICAL' | 'HIGH';
|
|
13
|
+
line: number;
|
|
14
|
+
column: number;
|
|
15
|
+
code: string;
|
|
16
|
+
issue: string;
|
|
17
|
+
cwe: string;
|
|
18
|
+
fix: string;
|
|
19
|
+
secretType?: string;
|
|
20
|
+
secretPreview?: string;
|
|
21
|
+
}
|
|
22
|
+
export declare class SecretScanner {
|
|
23
|
+
private vulnerabilities;
|
|
24
|
+
private readonly patterns;
|
|
25
|
+
/**
|
|
26
|
+
* Scan code for hardcoded secrets
|
|
27
|
+
*/
|
|
28
|
+
scan(code: string): SecretVulnerability[];
|
|
29
|
+
/**
|
|
30
|
+
* Check if line should be ignored (comments, examples, etc.)
|
|
31
|
+
*/
|
|
32
|
+
private isSafeIgnore;
|
|
33
|
+
/**
|
|
34
|
+
* Check for AWS keys
|
|
35
|
+
*/
|
|
36
|
+
private checkAwsKeys;
|
|
37
|
+
/**
|
|
38
|
+
* Check for API keys (OpenAI, Stripe, GitHub, etc.)
|
|
39
|
+
*/
|
|
40
|
+
private checkApiKeys;
|
|
41
|
+
/**
|
|
42
|
+
* Check for connection strings with embedded credentials
|
|
43
|
+
*/
|
|
44
|
+
private checkConnectionStrings;
|
|
45
|
+
/**
|
|
46
|
+
* Check for private keys in code
|
|
47
|
+
*/
|
|
48
|
+
private checkPrivateKeys;
|
|
49
|
+
/**
|
|
50
|
+
* Check for hardcoded passwords
|
|
51
|
+
*/
|
|
52
|
+
private checkPasswords;
|
|
53
|
+
/**
|
|
54
|
+
* Check for JWT tokens
|
|
55
|
+
*/
|
|
56
|
+
private checkJwtTokens;
|
|
57
|
+
/**
|
|
58
|
+
* Redact secret for safe display
|
|
59
|
+
*/
|
|
60
|
+
private redact;
|
|
61
|
+
/**
|
|
62
|
+
* Check if position is inside a comment
|
|
63
|
+
*/
|
|
64
|
+
private isCommentOrString;
|
|
65
|
+
}
|
|
66
|
+
//# sourceMappingURL=secret-scanner.d.ts.map
|
|
@@ -0,0 +1 @@
|
|
|
1
|
+
{"version":3,"file":"secret-scanner.d.ts","sourceRoot":"","sources":["../../../src/developer/scanners/secret-scanner.ts"],"names":[],"mappings":"AAAA;;;;;;;;GAQG;AAEH,MAAM,WAAW,mBAAmB;IAClC,IAAI,EAAE,mBAAmB,GAAG,oBAAoB,GAAG,aAAa,GAAG,mBAAmB,GAAG,WAAW,GAAG,gBAAgB,CAAC;IACxH,QAAQ,EAAE,UAAU,GAAG,MAAM,CAAC;IAC9B,IAAI,EAAE,MAAM,CAAC;IACb,MAAM,EAAE,MAAM,CAAC;IACf,IAAI,EAAE,MAAM,CAAC;IACb,KAAK,EAAE,MAAM,CAAC;IACd,GAAG,EAAE,MAAM,CAAC;IACZ,GAAG,EAAE,MAAM,CAAC;IACZ,UAAU,CAAC,EAAE,MAAM,CAAC;IACpB,aAAa,CAAC,EAAE,MAAM,CAAC;CACxB;AAED,qBAAa,aAAa;IACxB,OAAO,CAAC,eAAe,CAA6B;IAGpD,OAAO,CAAC,QAAQ,CAAC,QAAQ,CA4BvB;IAEF;;OAEG;IACH,IAAI,CAAC,IAAI,EAAE,MAAM,GAAG,mBAAmB,EAAE;IAqBzC;;OAEG;IACH,OAAO,CAAC,YAAY;IAyBpB;;OAEG;IACH,OAAO,CAAC,YAAY;IA4CpB;;OAEG;IACH,OAAO,CAAC,YAAY;IAkCpB;;OAEG;IACH,OAAO,CAAC,sBAAsB;IAgC9B;;OAEG;IACH,OAAO,CAAC,gBAAgB;IAoBxB;;OAEG;IACH,OAAO,CAAC,cAAc;IA8BtB;;OAEG;IACH,OAAO,CAAC,cAAc;IA6BtB;;OAEG;IACH,OAAO,CAAC,MAAM;IAKd;;OAEG;IACH,OAAO,CAAC,iBAAiB;CAW1B"}
|