@saulwade/swl-ses 1.4.1 → 1.4.2
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/CLAUDE.md +1 -1
- package/README.md +1 -1
- package/agentes/nemesis-auditor-swl.md +161 -161
- package/bin/swl-mcp-server.js +187 -187
- package/comandos/swl/.evolved.json +22 -22
- package/comandos/swl/contribuir.md +233 -233
- package/comandos/swl/nemesis.md +122 -122
- package/gateway/lib/event-channel.js +191 -191
- package/habilidades/backend-production-resilience/SKILL.md +288 -288
- package/habilidades/benchmark-memoria/SKILL.md +186 -186
- package/habilidades/diagrama-arquitectura/assets/template.html +276 -276
- package/habilidades/doubt-driven-review/SKILL.md +171 -171
- package/habilidades/doubt-driven-review/recursos/EXAMPLES.md +130 -130
- package/habilidades/eval-framework/SKILL.md +212 -212
- package/habilidades/feynman-auditor-swl/SKILL.md +123 -123
- package/habilidades/feynman-auditor-swl/recursos/preguntas-language-agnostic.md +108 -108
- package/habilidades/harness-claude-code/SKILL.md +299 -299
- package/habilidades/infra-github-actions/SKILL.md +166 -166
- package/habilidades/legacy-code-rescue/SKILL.md +267 -267
- package/habilidades/manejo-errores/.evolved.json +8 -8
- package/habilidades/meta-skills-estandar/recursos/convencion-examples.md +93 -93
- package/habilidades/meta-skills-estandar/recursos/skills-as-agents.md +163 -163
- package/habilidades/patrones-python/SKILL.md +229 -229
- package/habilidades/patrones-python/recursos/patrones-avanzados.md +469 -469
- package/habilidades/planear-fase/SKILL.md +319 -319
- package/habilidades/release-semver/.evolved.json +8 -8
- package/habilidades/state-inconsistency-auditor-swl/SKILL.md +166 -166
- package/habilidades/state-inconsistency-auditor-swl/recursos/coupled-state-patterns.md +147 -147
- package/habilidades/testing-python/SKILL.md +340 -340
- package/habilidades/web-fetcher-routing/SKILL.md +75 -75
- package/hooks/claudemd-bloat-detector.js +161 -161
- package/hooks/lib/agent-routing.js +107 -107
- package/hooks/lib/auto-consolidator.js +335 -335
- package/hooks/lib/error-classifier.js +308 -308
- package/hooks/lib/merkle-audit.js +96 -96
- package/hooks/lib/provenance-tracker.js +191 -191
- package/hooks/lib/rate-limit-tracker.js +253 -253
- package/hooks/lib/resource-quota.js +122 -122
- package/hooks/lib/retry-jitter.js +165 -165
- package/hooks/lib/security-net.js +201 -201
- package/hooks/lib/skill-auditor.js +588 -588
- package/hooks/lib/sync-status.js +228 -228
- package/hooks/lib/taint-tracker.js +107 -107
- package/hooks/lib/text-similarity.js +241 -241
- package/hooks/lib/toon-compressor.js +245 -245
- package/hooks/registro-turnos.js +209 -209
- package/hooks/sugerir-regenerar-inventario.js +170 -170
- package/hooks/validar-formato-post-subagente.js +140 -140
- package/hooks/validar-memoria-hook.js +218 -218
- package/instintos/prompt-appendices.yaml +57 -57
- package/manifiestos/agent-output-schemas.json +57 -57
- package/manifiestos/modulos.json +11 -6
- package/manifiestos/perfiles.json +2 -1
- package/manifiestos/skills-lock.json +1114 -1114
- package/package.json +1 -1
- package/plantillas/auditor-veto-template.md +105 -105
- package/plantillas/github-workflows/README.md +47 -47
- package/plantillas/github-workflows/release-please.yml +44 -44
- package/plantillas/github-workflows/swl-ci.yml +107 -107
- package/plantillas/github-workflows/swl-security.yml +51 -51
- package/plugin.json +9 -1
- package/reglas/analisis-previo-tareas-grandes.md +172 -172
- package/reglas/arreglar-al-detectar.md +147 -147
- package/reglas/fragmentos-compartidos.md +152 -152
- package/reglas/harness-claude-code.md +213 -213
- package/reglas/usar-context7.md +226 -226
- package/schemas/diary-entry.schema.json +80 -80
- package/scripts/audit-tools/audit-history.js +330 -330
- package/scripts/audit-tools/bundle-tracker.js +290 -290
- package/scripts/audit-tools/canary-monitor.js +352 -352
- package/scripts/audit-tools/code-profiler.js +605 -605
- package/scripts/audit-tools/dep-doctor.js +320 -320
- package/scripts/audit-tools/env-validator.js +206 -206
- package/scripts/audit-tools/lib/fs-walk.js +48 -48
- package/scripts/audit-tools/lib/output.js +23 -23
- package/scripts/audit-tools/migration-checker.js +392 -392
- package/scripts/audit-tools/pentest-scanner.js +1436 -1436
- package/scripts/benchmark-memoria.js +167 -167
- package/scripts/configurar-branch-protection.js +418 -418
- package/scripts/detectar-aprendizajes-duplicados.js +151 -151
- package/scripts/field-report.js +199 -199
- package/scripts/generar-checklists-consolidados.js +273 -273
- package/scripts/generar-inventario.js +420 -420
- package/scripts/generar-matriz-lenguajes.js +271 -271
- package/scripts/lib/artefactos-python.js +43 -43
- package/scripts/lib/benchmark-metrics.js +160 -160
- package/scripts/lib/budget-enforcer.js +252 -252
- package/scripts/lib/configurar-ci.js +380 -380
- package/scripts/lib/contadores-inventario.js +217 -217
- package/scripts/lib/detectar-stack-detallado.js +307 -307
- package/scripts/lib/diary-entry.js +234 -234
- package/scripts/lib/eval-metrics-store.js +218 -218
- package/scripts/lib/eval-quality.js +171 -171
- package/scripts/lib/eval-schemas.js +144 -144
- package/scripts/lib/eval-self-correct.js +106 -106
- package/scripts/lib/eval-validator.js +185 -185
- package/scripts/lib/jaccard-similarity.js +98 -98
- package/scripts/lib/longmemeval-runner.js +125 -125
- package/scripts/lib/manifiestos.js +42 -1
- package/scripts/lib/npm-version.js +261 -261
- package/scripts/lib/paquetes-conocidos.js +50 -50
- package/scripts/lib/prompt-builder.js +264 -264
- package/scripts/lib/rrf-fusion.js +175 -175
- package/scripts/lib/scoring-instintos.js +277 -277
- package/scripts/lib/semantic-search.js +252 -252
- package/scripts/limpiar-artefactos-python.js +131 -131
- package/scripts/mcp-server/README.md +128 -128
- package/scripts/mcp-server/handlers.js +206 -206
- package/scripts/migrar-csv-a-array.js +168 -168
- package/scripts/migrar-fase-dominio.js +201 -201
- package/scripts/publicar.js +511 -511
- package/scripts/run-eval.js +141 -141
- package/scripts/validar-manifest.js +231 -195
- package/scripts/validar-userland-vacio.js +110 -110
|
@@ -1,605 +1,605 @@
|
|
|
1
|
-
// Adaptado de temp/ultraship-main/tools/code-profiler.mjs bajo MIT License
|
|
2
|
-
// Fuente: Houseofmvps/ultraship (https://github.com/Houseofmvps/ultraship)
|
|
3
|
-
'use strict';
|
|
4
|
-
|
|
5
|
-
const { readFileSync, existsSync } = require('fs');
|
|
6
|
-
const { relative, extname } = require('path');
|
|
7
|
-
const { walkCode } = require('./lib/fs-walk');
|
|
8
|
-
const { outputJSON, outputError } = require('./lib/output');
|
|
9
|
-
|
|
10
|
-
// Patrones de I/O síncrona que BUSCAMOS en el código analizado (no los usamos aquí)
|
|
11
|
-
const SYNC_PATTERNS = [
|
|
12
|
-
{ name: 'readFileSync', fix: 'readFile (async)' },
|
|
13
|
-
{ name: 'writeFileSync', fix: 'writeFile (async)' },
|
|
14
|
-
{ name: 'readdirSync', fix: 'readdir (async)' },
|
|
15
|
-
{ name: 'statSync', fix: 'stat (async)' },
|
|
16
|
-
{ name: 'existsSync', fix: 'access (async)' },
|
|
17
|
-
{ name: 'copyFileSync', fix: 'copyFile (async)' },
|
|
18
|
-
{ name: 'mkdirSync', fix: 'mkdir (async)' },
|
|
19
|
-
];
|
|
20
|
-
|
|
21
|
-
const SHELL_SYNC_NAMES = ['execSync', 'execFileSync', 'spawnSync'];
|
|
22
|
-
|
|
23
|
-
// Patrones de I/O síncrona en Python dentro de handlers async
|
|
24
|
-
const PYTHON_SYNC_IO = [
|
|
25
|
-
{ name: 'open(', fix: 'aiofiles.open() o executor' },
|
|
26
|
-
{ name: 'subprocess.run(', fix: 'asyncio.create_subprocess_exec()' },
|
|
27
|
-
{ name: 'subprocess.call(', fix: 'asyncio.create_subprocess_exec()' },
|
|
28
|
-
{ name: 'subprocess.check_output(', fix: 'asyncio.create_subprocess_exec()' },
|
|
29
|
-
{ name: 'os.path.exists(', fix: 'asyncio.to_thread(os.path.exists, ...)' },
|
|
30
|
-
{ name: 'os.listdir(', fix: 'asyncio.to_thread(os.listdir, ...)' },
|
|
31
|
-
{ name: 'os.makedirs(', fix: 'asyncio.to_thread(os.makedirs, ...)' },
|
|
32
|
-
];
|
|
33
|
-
|
|
34
|
-
/**
|
|
35
|
-
* Determina si el archivo es un route handler (JS/TS o Python).
|
|
36
|
-
* @param {string} content
|
|
37
|
-
* @param {string} filePath
|
|
38
|
-
* @returns {boolean}
|
|
39
|
-
*/
|
|
40
|
-
function isRouteHandler(content, filePath) {
|
|
41
|
-
const rel = filePath.toLowerCase();
|
|
42
|
-
const isPython = rel.endsWith('.py');
|
|
43
|
-
|
|
44
|
-
if (!isPython) {
|
|
45
|
-
if (rel.includes('/route') || rel.includes('/api/') || rel.includes('/handler') || rel.includes('/controller')) return true;
|
|
46
|
-
if (content.includes('app.get(') || content.includes('app.post(') || content.includes('app.put(') || content.includes('app.delete(')) return true;
|
|
47
|
-
if (content.includes('router.get(') || content.includes('router.post(')) return true;
|
|
48
|
-
if (content.includes('.onRequest(') || content.includes('Hono(')) return true;
|
|
49
|
-
if (content.includes('export default') && (content.includes('GET') || content.includes('POST')) && rel.includes('route')) return true;
|
|
50
|
-
} else {
|
|
51
|
-
// Python: FastAPI, Flask, Django views
|
|
52
|
-
if (content.includes('@app.get(') || content.includes('@app.post(') ||
|
|
53
|
-
content.includes('@app.put(') || content.includes('@app.delete(') ||
|
|
54
|
-
content.includes('@app.patch(') || content.includes('@router.get(') ||
|
|
55
|
-
content.includes('@router.post(')) return true;
|
|
56
|
-
if (rel.includes('/views') || rel.includes('/routes') || rel.includes('/handlers') || rel.includes('/api/')) return true;
|
|
57
|
-
// Django class-based views
|
|
58
|
-
if (content.includes('class') && (content.includes('APIView') || content.includes('ViewSet') || content.includes('GenericView'))) return true;
|
|
59
|
-
}
|
|
60
|
-
return false;
|
|
61
|
-
}
|
|
62
|
-
|
|
63
|
-
/**
|
|
64
|
-
* Determina si el archivo es non-production (seed, test, migration).
|
|
65
|
-
* @param {string} filePath
|
|
66
|
-
* @returns {boolean}
|
|
67
|
-
*/
|
|
68
|
-
function isNonProductionFile(filePath) {
|
|
69
|
-
const rel = filePath.toLowerCase();
|
|
70
|
-
return (
|
|
71
|
-
rel.includes('seed') || rel.includes('fixture') ||
|
|
72
|
-
rel.includes('test') || rel.includes('spec') || rel.includes('__test') ||
|
|
73
|
-
rel.includes('migration') || rel.includes('migrate') ||
|
|
74
|
-
rel.includes('/scripts/') || rel.includes('/demo')
|
|
75
|
-
);
|
|
76
|
-
}
|
|
77
|
-
|
|
78
|
-
/**
|
|
79
|
-
* Verifica si una línea está dentro de un contexto seed/demo/test
|
|
80
|
-
* (buscando hacia atrás la definición de función más cercana).
|
|
81
|
-
* @param {string[]} lines
|
|
82
|
-
* @param {number} lineIndex
|
|
83
|
-
* @returns {boolean}
|
|
84
|
-
*/
|
|
85
|
-
function isInSeedContext(lines, lineIndex) {
|
|
86
|
-
const SEED_KEYWORDS = /seed|demo|populate|fixture|mock|fake|sample|generate.*data|test.*data/i;
|
|
87
|
-
const ROUTE_DEF = /\.(get|post|put|delete|patch|all)\s*\(/;
|
|
88
|
-
const FUNC_DEF = /(?:function|const|let|var)\s+\w+.*(?:=>|\{)|(?:async\s+function)/;
|
|
89
|
-
|
|
90
|
-
for (let i = lineIndex; i >= Math.max(0, lineIndex - 500); i--) {
|
|
91
|
-
const line = lines[i];
|
|
92
|
-
if (ROUTE_DEF.test(line)) {
|
|
93
|
-
for (let j = Math.max(0, i - 2); j <= Math.min(lines.length - 1, i + 2); j++) {
|
|
94
|
-
if (SEED_KEYWORDS.test(lines[j])) return true;
|
|
95
|
-
}
|
|
96
|
-
return false;
|
|
97
|
-
}
|
|
98
|
-
if (SEED_KEYWORDS.test(line) && FUNC_DEF.test(line)) return true;
|
|
99
|
-
if (/^\s*\/\/.*(?:seed|demo|populate|test data)/i.test(line)) return true;
|
|
100
|
-
}
|
|
101
|
-
return false;
|
|
102
|
-
}
|
|
103
|
-
|
|
104
|
-
/**
|
|
105
|
-
* Detecta si una línea Python está dentro de una función async.
|
|
106
|
-
* Escanea hacia atrás buscando `async def`.
|
|
107
|
-
* @param {string[]} lines
|
|
108
|
-
* @param {number} lineIndex
|
|
109
|
-
* @returns {boolean}
|
|
110
|
-
*/
|
|
111
|
-
function isInsidePythonAsyncDef(lines, lineIndex) {
|
|
112
|
-
// Obtener nivel de indentación de la línea actual
|
|
113
|
-
const currentLine = lines[lineIndex];
|
|
114
|
-
const currentIndent = currentLine.match(/^(\s*)/)[1].length;
|
|
115
|
-
|
|
116
|
-
for (let i = lineIndex - 1; i >= Math.max(0, lineIndex - 100); i--) {
|
|
117
|
-
const line = lines[i];
|
|
118
|
-
if (!line.trim()) continue;
|
|
119
|
-
const indent = line.match(/^(\s*)/)[1].length;
|
|
120
|
-
// Buscamos una definición de función con menor indentación
|
|
121
|
-
if (indent < currentIndent && /^\s*async\s+def\s+/.test(line)) return true;
|
|
122
|
-
if (indent < currentIndent && /^\s*def\s+/.test(line)) return false;
|
|
123
|
-
if (indent === 0 && /^(class|def|async def)/.test(line)) return false;
|
|
124
|
-
}
|
|
125
|
-
return false;
|
|
126
|
-
}
|
|
127
|
-
|
|
128
|
-
/**
|
|
129
|
-
* Analiza un archivo y devuelve los hallazgos de rendimiento.
|
|
130
|
-
* @param {string} filePath
|
|
131
|
-
* @param {string} relPath
|
|
132
|
-
* @param {string} content
|
|
133
|
-
* @returns {object[]}
|
|
134
|
-
*/
|
|
135
|
-
function analyzeFile(filePath, relPath, content) {
|
|
136
|
-
const findings = [];
|
|
137
|
-
const lines = content.split('\n');
|
|
138
|
-
const isHandler = isRouteHandler(content, relPath);
|
|
139
|
-
const isNonProd = isNonProductionFile(relPath);
|
|
140
|
-
const isPython = relPath.toLowerCase().endsWith('.py');
|
|
141
|
-
|
|
142
|
-
if (isPython) {
|
|
143
|
-
return analyzeFilePython(filePath, relPath, content, lines, isHandler, isNonProd);
|
|
144
|
-
}
|
|
145
|
-
|
|
146
|
-
// === N+1 Query Detection (JS/TS) ===
|
|
147
|
-
const blockLoopPatterns = [
|
|
148
|
-
/\bfor\s*\(/,
|
|
149
|
-
/\bfor\s+.*\bof\b/,
|
|
150
|
-
/\bwhile\s*\(/,
|
|
151
|
-
];
|
|
152
|
-
const callbackLoopPattern = /\.(forEach|map)\s*\(/;
|
|
153
|
-
const queryPatterns = [
|
|
154
|
-
/\.findOne\s*\(/, /\.findFirst\s*\(/, /\.findUnique\s*\(/,
|
|
155
|
-
/\.find\s*\(/, /\.findMany\s*\(/,
|
|
156
|
-
/\.query\s*\(/, /\.execute\s*\(/,
|
|
157
|
-
/await\s+db\./, /await\s+prisma\./,
|
|
158
|
-
];
|
|
159
|
-
|
|
160
|
-
const blockLoopStack = [];
|
|
161
|
-
const callbackLoopStack = [];
|
|
162
|
-
let braceDepth = 0;
|
|
163
|
-
let parenDepth = 0;
|
|
164
|
-
|
|
165
|
-
for (let i = 0; i < lines.length; i++) {
|
|
166
|
-
const line = lines[i];
|
|
167
|
-
const trimmed = line.trim();
|
|
168
|
-
if (trimmed.startsWith('//') || trimmed.startsWith('*') || trimmed.startsWith('/*')) continue;
|
|
169
|
-
|
|
170
|
-
const isBlockLoop = blockLoopPatterns.some(p => p.test(line));
|
|
171
|
-
const callbackMatch = callbackLoopPattern.test(line);
|
|
172
|
-
|
|
173
|
-
if (isBlockLoop) blockLoopStack.push(braceDepth);
|
|
174
|
-
if (callbackMatch) callbackLoopStack.push(parenDepth);
|
|
175
|
-
|
|
176
|
-
for (const ch of line) {
|
|
177
|
-
if (ch === '{') braceDepth++;
|
|
178
|
-
if (ch === '(') parenDepth++;
|
|
179
|
-
if (ch === '}') {
|
|
180
|
-
braceDepth--;
|
|
181
|
-
while (blockLoopStack.length > 0 && braceDepth <= blockLoopStack[blockLoopStack.length - 1]) {
|
|
182
|
-
blockLoopStack.pop();
|
|
183
|
-
}
|
|
184
|
-
}
|
|
185
|
-
if (ch === ')') {
|
|
186
|
-
parenDepth--;
|
|
187
|
-
while (callbackLoopStack.length > 0 && parenDepth <= callbackLoopStack[callbackLoopStack.length - 1]) {
|
|
188
|
-
callbackLoopStack.pop();
|
|
189
|
-
}
|
|
190
|
-
}
|
|
191
|
-
}
|
|
192
|
-
|
|
193
|
-
const insideBlockLoop = blockLoopStack.length > 0;
|
|
194
|
-
const insideCallbackLoop = callbackLoopStack.length > 0;
|
|
195
|
-
const sameLineCallback = callbackMatch && !insideCallbackLoop;
|
|
196
|
-
const insideAnyLoop = insideBlockLoop || insideCallbackLoop || sameLineCallback;
|
|
197
|
-
|
|
198
|
-
if (insideAnyLoop && queryPatterns.some(p => p.test(line))) {
|
|
199
|
-
const inSeedCtx = isNonProd || isInSeedContext(lines, i);
|
|
200
|
-
const severity = inSeedCtx ? 'low' : 'high';
|
|
201
|
-
const suffix = inSeedCtx ? ' (en contexto seed/test — prioridad baja)' : '';
|
|
202
|
-
findings.push({
|
|
203
|
-
file: relPath, line: i + 1, severity, category: 'n+1',
|
|
204
|
-
message: `Consulta de BD dentro de bucle — patrón N+1 detectado. Usar consulta batch (findMany/cláusula IN)${suffix}`,
|
|
205
|
-
code: trimmed.slice(0, 120),
|
|
206
|
-
});
|
|
207
|
-
}
|
|
208
|
-
}
|
|
209
|
-
|
|
210
|
-
// === Sync I/O en Request Handlers (JS/TS) ===
|
|
211
|
-
if (isHandler) {
|
|
212
|
-
for (let i = 0; i < lines.length; i++) {
|
|
213
|
-
const line = lines[i];
|
|
214
|
-
const trimmed = line.trim();
|
|
215
|
-
if (trimmed.startsWith('//') || trimmed.startsWith('*')) continue;
|
|
216
|
-
|
|
217
|
-
for (const sp of SYNC_PATTERNS) {
|
|
218
|
-
if (line.includes(sp.name)) {
|
|
219
|
-
findings.push({
|
|
220
|
-
file: relPath, line: i + 1, severity: 'high', category: 'sync-io',
|
|
221
|
-
message: `I/O síncrona (${sp.name}) en request handler bloquea el event loop — usar ${sp.fix}`,
|
|
222
|
-
code: trimmed.slice(0, 120),
|
|
223
|
-
});
|
|
224
|
-
}
|
|
225
|
-
}
|
|
226
|
-
|
|
227
|
-
for (const name of SHELL_SYNC_NAMES) {
|
|
228
|
-
if (line.includes(name)) {
|
|
229
|
-
findings.push({
|
|
230
|
-
file: relPath, line: i + 1, severity: 'high', category: 'sync-io',
|
|
231
|
-
message: `Ejecución de shell síncrona (${name}) en request handler — usar alternativa async`,
|
|
232
|
-
code: trimmed.slice(0, 120),
|
|
233
|
-
});
|
|
234
|
-
}
|
|
235
|
-
}
|
|
236
|
-
}
|
|
237
|
-
}
|
|
238
|
-
|
|
239
|
-
// === Operaciones sin límite (JS/TS) ===
|
|
240
|
-
for (let i = 0; i < lines.length; i++) {
|
|
241
|
-
const line = lines[i];
|
|
242
|
-
const trimmed = line.trim();
|
|
243
|
-
if (trimmed.startsWith('//') || trimmed.startsWith('*')) continue;
|
|
244
|
-
|
|
245
|
-
if (/\.findMany\s*\(\s*\{/.test(line)) {
|
|
246
|
-
const startIdx = content.indexOf(line);
|
|
247
|
-
const chunk = startIdx >= 0 ? content.slice(startIdx, startIdx + 300) : '';
|
|
248
|
-
if (!chunk.includes('take:') && !chunk.includes('limit')) {
|
|
249
|
-
findings.push({
|
|
250
|
-
file: relPath, line: i + 1, severity: 'high', category: 'unbounded',
|
|
251
|
-
message: 'findMany sin limit/take — puede retornar la tabla completa en memoria',
|
|
252
|
-
code: trimmed.slice(0, 120),
|
|
253
|
-
});
|
|
254
|
-
}
|
|
255
|
-
}
|
|
256
|
-
|
|
257
|
-
if (/SELECT\s+\*/i.test(line) && !/LIMIT/i.test(line)) {
|
|
258
|
-
const selectMatch = line.match(/SELECT\s+\*/i);
|
|
259
|
-
const selectIdx = selectMatch ? line.indexOf(selectMatch[0]) : -1;
|
|
260
|
-
const beforeSelect = selectIdx >= 0 ? line.slice(0, selectIdx) : '';
|
|
261
|
-
const isInJsxComment = /\{\/\*/.test(beforeSelect) && /\*\/\}/.test(line.slice(selectIdx));
|
|
262
|
-
const isInBlockComment = /\/\*/.test(beforeSelect) && !/\*\//.test(beforeSelect.slice(beforeSelect.lastIndexOf('/*')));
|
|
263
|
-
const isAfterLineComment = /\/\//.test(beforeSelect);
|
|
264
|
-
const isInComment = isInJsxComment || isInBlockComment || isAfterLineComment;
|
|
265
|
-
const isInSqlContext = /['"`].*SELECT\s+\*/i.test(line) || /SELECT\s+\*.*['"`]/i.test(line);
|
|
266
|
-
if (!isInComment && isInSqlContext) {
|
|
267
|
-
findings.push({
|
|
268
|
-
file: relPath, line: i + 1, severity: 'high', category: 'unbounded',
|
|
269
|
-
message: 'SELECT * sin LIMIT — puede retornar la tabla completa',
|
|
270
|
-
code: trimmed.slice(0, 120),
|
|
271
|
-
});
|
|
272
|
-
}
|
|
273
|
-
}
|
|
274
|
-
}
|
|
275
|
-
|
|
276
|
-
// === Índices de BD faltantes (esquemas) ===
|
|
277
|
-
if (relPath.includes('schema') || relPath.includes('migration')) {
|
|
278
|
-
const fkPattern = /\.references\s*\(\s*\(\)\s*=>\s*(\w+)\.(\w+)\)/g;
|
|
279
|
-
let fkMatch;
|
|
280
|
-
while ((fkMatch = fkPattern.exec(content)) !== null) {
|
|
281
|
-
const lineNum = content.slice(0, fkMatch.index).split('\n').length;
|
|
282
|
-
const nearby = content.slice(Math.max(0, fkMatch.index - 500), fkMatch.index + 500);
|
|
283
|
-
if (!nearby.includes('index(') && !nearby.includes('.index(')) {
|
|
284
|
-
findings.push({
|
|
285
|
-
file: relPath, line: lineNum, severity: 'medium', category: 'missing-index',
|
|
286
|
-
message: `Clave foránea a ${fkMatch[1]}.${fkMatch[2]} sin índice — los JOINs serán lentos a escala`,
|
|
287
|
-
code: fkMatch[0].slice(0, 120),
|
|
288
|
-
});
|
|
289
|
-
}
|
|
290
|
-
}
|
|
291
|
-
|
|
292
|
-
if (content.includes('@relation')) {
|
|
293
|
-
const relPattern = /@relation.*fields:\s*\[(\w+)\]/g;
|
|
294
|
-
let relMatch;
|
|
295
|
-
while ((relMatch = relPattern.exec(content)) !== null) {
|
|
296
|
-
const field = relMatch[1];
|
|
297
|
-
if (!content.includes(`@@index([${field}]`) && !content.includes('@unique')) {
|
|
298
|
-
const lineNum = content.slice(0, relMatch.index).split('\n').length;
|
|
299
|
-
findings.push({
|
|
300
|
-
file: relPath, line: lineNum, severity: 'medium', category: 'missing-index',
|
|
301
|
-
message: `Campo de relación "${field}" sin @@index — las búsquedas serán lentas a escala`,
|
|
302
|
-
code: relMatch[0].slice(0, 120),
|
|
303
|
-
});
|
|
304
|
-
}
|
|
305
|
-
}
|
|
306
|
-
}
|
|
307
|
-
}
|
|
308
|
-
|
|
309
|
-
// === Memory Leaks (JS/TS) ===
|
|
310
|
-
const SERVER_PATTERNS = /app\.get\(|app\.listen\(|app\.post\(|Hono\(|createServer|express\(|router\.(get|post|put|delete)\(|\.onRequest\(/;
|
|
311
|
-
const isOneShotScript = (
|
|
312
|
-
relPath.includes('/scripts/') ||
|
|
313
|
-
relPath.includes('cli.') ||
|
|
314
|
-
relPath.includes('/bin/') ||
|
|
315
|
-
Boolean(relPath.match(/(?:^|\/)cli[./]/))
|
|
316
|
-
) || !SERVER_PATTERNS.test(content);
|
|
317
|
-
|
|
318
|
-
let braceDepthML = 0;
|
|
319
|
-
for (let i = 0; i < lines.length; i++) {
|
|
320
|
-
const line = lines[i];
|
|
321
|
-
const trimmed = line.trim();
|
|
322
|
-
if (trimmed.startsWith('//') || trimmed.startsWith('*')) continue;
|
|
323
|
-
|
|
324
|
-
for (const ch of line) {
|
|
325
|
-
if (ch === '{') braceDepthML++;
|
|
326
|
-
if (ch === '}') braceDepthML--;
|
|
327
|
-
}
|
|
328
|
-
|
|
329
|
-
if (/^(const|let|var)\s+\w+\s*=\s*\[\s*\]/.test(trimmed) && braceDepthML === 0) {
|
|
330
|
-
const varName = trimmed.match(/^(?:const|let|var)\s+(\w+)/)?.[1];
|
|
331
|
-
if (varName && content.includes(`${varName}.push(`)) {
|
|
332
|
-
findings.push({
|
|
333
|
-
file: relPath, line: i + 1, severity: isOneShotScript ? 'low' : 'medium', category: 'memory-leak',
|
|
334
|
-
message: `Array con scope de módulo "${varName}" con .push() — crece sin límite${isOneShotScript ? ' (script de una vez)' : ', memory leak en servidores long-running'}`,
|
|
335
|
-
code: trimmed.slice(0, 120),
|
|
336
|
-
});
|
|
337
|
-
}
|
|
338
|
-
}
|
|
339
|
-
|
|
340
|
-
if (isHandler && (/\.addEventListener\s*\(/.test(line) || /\.on\s*\(\s*['"]/.test(line))) {
|
|
341
|
-
const isReqBodyParsing = /\b(req|request|res|response)\s*\.\s*on\s*\(\s*['"](data|end|error|close)['"]/i.test(line);
|
|
342
|
-
if (!isReqBodyParsing) {
|
|
343
|
-
findings.push({
|
|
344
|
-
file: relPath, line: i + 1, severity: 'medium', category: 'memory-leak',
|
|
345
|
-
message: 'Event listener en request handler — puede acumularse sin limpieza',
|
|
346
|
-
code: trimmed.slice(0, 120),
|
|
347
|
-
});
|
|
348
|
-
}
|
|
349
|
-
}
|
|
350
|
-
}
|
|
351
|
-
|
|
352
|
-
// === Error Handling en Handlers (JS/TS) ===
|
|
353
|
-
if (isHandler) {
|
|
354
|
-
for (let i = 0; i < lines.length; i++) {
|
|
355
|
-
const line = lines[i];
|
|
356
|
-
const trimmed = line.trim();
|
|
357
|
-
if (trimmed.startsWith('//') || trimmed.startsWith('*')) continue;
|
|
358
|
-
|
|
359
|
-
if (/JSON\.parse\s*\(/.test(line)) {
|
|
360
|
-
let hasTry = false;
|
|
361
|
-
for (let j = Math.max(0, i - 5); j < i; j++) {
|
|
362
|
-
if (lines[j].includes('try')) hasTry = true;
|
|
363
|
-
}
|
|
364
|
-
if (!hasTry) {
|
|
365
|
-
findings.push({
|
|
366
|
-
file: relPath, line: i + 1, severity: 'medium', category: 'error-handling',
|
|
367
|
-
message: 'JSON.parse sin try/catch en handler — input malformado crashea el handler',
|
|
368
|
-
code: trimmed.slice(0, 120),
|
|
369
|
-
});
|
|
370
|
-
}
|
|
371
|
-
}
|
|
372
|
-
|
|
373
|
-
if (/new RegExp\s*\(/.test(line)) {
|
|
374
|
-
findings.push({
|
|
375
|
-
file: relPath, line: i + 1, severity: 'medium', category: 'redos',
|
|
376
|
-
message: 'RegExp dinámico en handler — input controlado por usuario puede causar ReDoS',
|
|
377
|
-
code: trimmed.slice(0, 120),
|
|
378
|
-
});
|
|
379
|
-
}
|
|
380
|
-
}
|
|
381
|
-
}
|
|
382
|
-
|
|
383
|
-
// === Sequential Await (JS/TS) ===
|
|
384
|
-
if (!isNonProd && !relPath.includes('/scripts/') && !relPath.includes('cleanup') && !relPath.includes('migrate')) {
|
|
385
|
-
for (let i = 0; i < lines.length - 1; i++) {
|
|
386
|
-
const line = lines[i].trim();
|
|
387
|
-
const nextLine = (lines[i + 1] || '').trim();
|
|
388
|
-
if (line.startsWith('//') || line.startsWith('*')) continue;
|
|
389
|
-
|
|
390
|
-
if (/^(const|let|var)\s+\w+\s*=\s*await\b/.test(line) && /^(const|let|var)\s+\w+\s*=\s*await\b/.test(nextLine)) {
|
|
391
|
-
const firstVar = line.match(/^(?:const|let|var)\s+(\w+)/)?.[1];
|
|
392
|
-
if (firstVar && !nextLine.includes(firstVar)) {
|
|
393
|
-
findings.push({
|
|
394
|
-
file: relPath, line: i + 1, severity: 'low', category: 'sequential-await',
|
|
395
|
-
message: 'Awaits secuenciales que podrían ejecutarse en paralelo — considerar Promise.all()',
|
|
396
|
-
code: `${line.slice(0, 60)} | ${nextLine.slice(0, 60)}`,
|
|
397
|
-
});
|
|
398
|
-
}
|
|
399
|
-
}
|
|
400
|
-
}
|
|
401
|
-
}
|
|
402
|
-
|
|
403
|
-
return findings;
|
|
404
|
-
}
|
|
405
|
-
|
|
406
|
-
/**
|
|
407
|
-
* Analiza archivos Python para detectar:
|
|
408
|
-
* - N+1 con SQLAlchemy/Django ORM
|
|
409
|
-
* - I/O síncrona en handlers async
|
|
410
|
-
* - Consultas sin límite (.all() sin .limit())
|
|
411
|
-
* @param {string} filePath
|
|
412
|
-
* @param {string} relPath
|
|
413
|
-
* @param {string} content
|
|
414
|
-
* @param {string[]} lines
|
|
415
|
-
* @param {boolean} isHandler
|
|
416
|
-
* @param {boolean} isNonProd
|
|
417
|
-
* @returns {object[]}
|
|
418
|
-
*/
|
|
419
|
-
function analyzeFilePython(filePath, relPath, content, lines, isHandler, isNonProd) {
|
|
420
|
-
const findings = [];
|
|
421
|
-
|
|
422
|
-
// Patrones de consulta SQLAlchemy/Django ORM
|
|
423
|
-
const sqlalchemyQueryPatterns = [
|
|
424
|
-
/session\.query\s*\(/,
|
|
425
|
-
/\.query\s*\(/,
|
|
426
|
-
/db\.query\s*\(/,
|
|
427
|
-
/\.\bfilter\s*\(/,
|
|
428
|
-
/\.\bfirst\s*\(\s*\)/,
|
|
429
|
-
/\.\bone\s*\(\s*\)/,
|
|
430
|
-
/\.\bget\s*\(/,
|
|
431
|
-
];
|
|
432
|
-
const djangoQueryPatterns = [
|
|
433
|
-
/\bObjects\.filter\s*\(/,
|
|
434
|
-
/\bObjects\.get\s*\(/,
|
|
435
|
-
/\bObjects\.exclude\s*\(/,
|
|
436
|
-
/\.objects\.\w+\s*\(/,
|
|
437
|
-
];
|
|
438
|
-
|
|
439
|
-
// Loop patterns en Python
|
|
440
|
-
const pythonLoopPattern = /^\s*for\s+\w+.*\bin\b/;
|
|
441
|
-
const pythonWhilePattern = /^\s*while\s+/;
|
|
442
|
-
|
|
443
|
-
// Seguimiento de profundidad de indentación para loops
|
|
444
|
-
let loopIndentStack = []; // cada entry: nivel de indentación del loop
|
|
445
|
-
|
|
446
|
-
for (let i = 0; i < lines.length; i++) {
|
|
447
|
-
const line = lines[i];
|
|
448
|
-
const trimmed = line.trim();
|
|
449
|
-
if (trimmed.startsWith('#')) continue;
|
|
450
|
-
|
|
451
|
-
const currentIndent = line.match(/^(\s*)/)[1].length;
|
|
452
|
-
|
|
453
|
-
// Sacar loops que ya cerraron (por indentación)
|
|
454
|
-
loopIndentStack = loopIndentStack.filter(indent => indent < currentIndent);
|
|
455
|
-
|
|
456
|
-
const isLoopLine = pythonLoopPattern.test(line) || pythonWhilePattern.test(line);
|
|
457
|
-
if (isLoopLine) {
|
|
458
|
-
loopIndentStack.push(currentIndent);
|
|
459
|
-
}
|
|
460
|
-
|
|
461
|
-
const insideLoop = loopIndentStack.length > 0;
|
|
462
|
-
|
|
463
|
-
// N+1 con SQLAlchemy
|
|
464
|
-
if (insideLoop && !isNonProd) {
|
|
465
|
-
const hasQuery = sqlalchemyQueryPatterns.some(p => p.test(line)) ||
|
|
466
|
-
djangoQueryPatterns.some(p => p.test(trimmed.toLowerCase() === trimmed ? line : line));
|
|
467
|
-
if (hasQuery) {
|
|
468
|
-
findings.push({
|
|
469
|
-
file: relPath, line: i + 1, severity: 'high', category: 'n+1',
|
|
470
|
-
message: 'Consulta ORM dentro de bucle Python — patrón N+1 detectado. Usar carga eager (joinedload/selectinload) o consultas batch',
|
|
471
|
-
code: trimmed.slice(0, 120),
|
|
472
|
-
});
|
|
473
|
-
}
|
|
474
|
-
}
|
|
475
|
-
|
|
476
|
-
// I/O síncrona en handlers async
|
|
477
|
-
if (isHandler || /^\s*async\s+def\s+/.test(line)) {
|
|
478
|
-
// Verificar que estamos dentro de una función async
|
|
479
|
-
const isAsync = isInsidePythonAsyncDef(lines, i);
|
|
480
|
-
if (isAsync) {
|
|
481
|
-
for (const sp of PYTHON_SYNC_IO) {
|
|
482
|
-
if (line.includes(sp.name)) {
|
|
483
|
-
findings.push({
|
|
484
|
-
file: relPath, line: i + 1, severity: 'high', category: 'sync-io',
|
|
485
|
-
message: `I/O síncrona (${sp.name}) en handler async Python bloquea el event loop — usar ${sp.fix}`,
|
|
486
|
-
code: trimmed.slice(0, 120),
|
|
487
|
-
});
|
|
488
|
-
}
|
|
489
|
-
}
|
|
490
|
-
}
|
|
491
|
-
}
|
|
492
|
-
|
|
493
|
-
// Consultas sin límite: .all() sin .limit() previo
|
|
494
|
-
if (/\.all\s*\(\s*\)/.test(line)) {
|
|
495
|
-
// Buscar si hay .limit() en la misma línea o en las 5 líneas previas
|
|
496
|
-
let hasLimit = /\.limit\s*\(/.test(line);
|
|
497
|
-
if (!hasLimit) {
|
|
498
|
-
for (let j = Math.max(0, i - 5); j < i; j++) {
|
|
499
|
-
if (/\.limit\s*\(/.test(lines[j])) { hasLimit = true; break; }
|
|
500
|
-
}
|
|
501
|
-
}
|
|
502
|
-
if (!hasLimit && !isNonProd) {
|
|
503
|
-
findings.push({
|
|
504
|
-
file: relPath, line: i + 1, severity: 'high', category: 'unbounded',
|
|
505
|
-
message: '.all() sin .limit() previo — puede retornar la tabla completa en memoria',
|
|
506
|
-
code: trimmed.slice(0, 120),
|
|
507
|
-
});
|
|
508
|
-
}
|
|
509
|
-
}
|
|
510
|
-
|
|
511
|
-
// SELECT * sin LIMIT en strings Python
|
|
512
|
-
if (/SELECT\s+\*/i.test(line) && !/LIMIT/i.test(line)) {
|
|
513
|
-
const isInComment = trimmed.startsWith('#');
|
|
514
|
-
const isInSqlContext = /['""].*SELECT\s+\*/i.test(line) || /SELECT\s+\*.*['""]/.test(line);
|
|
515
|
-
if (!isInComment && isInSqlContext) {
|
|
516
|
-
findings.push({
|
|
517
|
-
file: relPath, line: i + 1, severity: 'high', category: 'unbounded',
|
|
518
|
-
message: 'SELECT * sin LIMIT en Python — puede retornar la tabla completa',
|
|
519
|
-
code: trimmed.slice(0, 120),
|
|
520
|
-
});
|
|
521
|
-
}
|
|
522
|
-
}
|
|
523
|
-
}
|
|
524
|
-
|
|
525
|
-
return findings;
|
|
526
|
-
}
|
|
527
|
-
|
|
528
|
-
function main() {
|
|
529
|
-
const dir = process.argv[2];
|
|
530
|
-
if (!dir) {
|
|
531
|
-
outputError('Uso: node code-profiler.js <directorio-proyecto>');
|
|
532
|
-
process.exit(0);
|
|
533
|
-
}
|
|
534
|
-
|
|
535
|
-
if (!existsSync(dir)) {
|
|
536
|
-
outputError(`Ruta no encontrada: ${dir}`);
|
|
537
|
-
process.exit(0);
|
|
538
|
-
}
|
|
539
|
-
|
|
540
|
-
const codeFiles = walkCode(dir, { extensions: ['.ts', '.tsx', '.js', '.jsx', '.mjs', '.cjs', '.py'] });
|
|
541
|
-
if (codeFiles.length === 0) {
|
|
542
|
-
outputJSON({ success: true, message: 'No se encontraron archivos TypeScript/JavaScript/Python', findings: [] });
|
|
543
|
-
process.exit(0);
|
|
544
|
-
}
|
|
545
|
-
|
|
546
|
-
const allFindings = [];
|
|
547
|
-
let filesAnalyzed = 0;
|
|
548
|
-
let handlersFound = 0;
|
|
549
|
-
|
|
550
|
-
for (const file of codeFiles) {
|
|
551
|
-
let content;
|
|
552
|
-
try {
|
|
553
|
-
content = readFileSync(file, 'utf8');
|
|
554
|
-
} catch {
|
|
555
|
-
continue;
|
|
556
|
-
}
|
|
557
|
-
const relPath = relative(dir, file).replace(/\\/g, '/');
|
|
558
|
-
if (isRouteHandler(content, relPath)) handlersFound++;
|
|
559
|
-
allFindings.push(...analyzeFile(file, relPath, content));
|
|
560
|
-
filesAnalyzed++;
|
|
561
|
-
}
|
|
562
|
-
|
|
563
|
-
const byCategory = {};
|
|
564
|
-
for (const f of allFindings) {
|
|
565
|
-
if (!byCategory[f.category]) byCategory[f.category] = [];
|
|
566
|
-
byCategory[f.category].push(f);
|
|
567
|
-
}
|
|
568
|
-
|
|
569
|
-
const categorySummary = {};
|
|
570
|
-
for (const [cat, items] of Object.entries(byCategory)) {
|
|
571
|
-
categorySummary[cat] = {
|
|
572
|
-
count: items.length,
|
|
573
|
-
critical: items.filter(i => i.severity === 'critical').length,
|
|
574
|
-
high: items.filter(i => i.severity === 'high').length,
|
|
575
|
-
medium: items.filter(i => i.severity === 'medium').length,
|
|
576
|
-
low: items.filter(i => i.severity === 'low').length,
|
|
577
|
-
};
|
|
578
|
-
}
|
|
579
|
-
|
|
580
|
-
let score = 100;
|
|
581
|
-
const seenCatFile = new Set();
|
|
582
|
-
for (const f of allFindings) {
|
|
583
|
-
const key = `${f.category}:${f.file}`;
|
|
584
|
-
const firstInFile = !seenCatFile.has(key);
|
|
585
|
-
seenCatFile.add(key);
|
|
586
|
-
const mult = firstInFile ? 1 : 0.5;
|
|
587
|
-
if (f.severity === 'high') score -= 5 * mult;
|
|
588
|
-
else if (f.severity === 'medium') score -= 2 * mult;
|
|
589
|
-
else if (f.severity === 'low') score -= 0.5 * mult;
|
|
590
|
-
}
|
|
591
|
-
|
|
592
|
-
outputJSON({
|
|
593
|
-
success: true,
|
|
594
|
-
files_analyzed: filesAnalyzed,
|
|
595
|
-
handlers_found: handlersFound,
|
|
596
|
-
total_findings: allFindings.length,
|
|
597
|
-
performance_score: Math.max(0, Math.round(score)),
|
|
598
|
-
categories: categorySummary,
|
|
599
|
-
findings: allFindings,
|
|
600
|
-
});
|
|
601
|
-
}
|
|
602
|
-
|
|
603
|
-
main();
|
|
604
|
-
|
|
605
|
-
module.exports = { analyzeFile, isRouteHandler, isNonProductionFile, isInSeedContext };
|
|
1
|
+
// Adaptado de temp/ultraship-main/tools/code-profiler.mjs bajo MIT License
|
|
2
|
+
// Fuente: Houseofmvps/ultraship (https://github.com/Houseofmvps/ultraship)
|
|
3
|
+
'use strict';
|
|
4
|
+
|
|
5
|
+
const { readFileSync, existsSync } = require('fs');
|
|
6
|
+
const { relative, extname } = require('path');
|
|
7
|
+
const { walkCode } = require('./lib/fs-walk');
|
|
8
|
+
const { outputJSON, outputError } = require('./lib/output');
|
|
9
|
+
|
|
10
|
+
// Patrones de I/O síncrona que BUSCAMOS en el código analizado (no los usamos aquí)
|
|
11
|
+
const SYNC_PATTERNS = [
|
|
12
|
+
{ name: 'readFileSync', fix: 'readFile (async)' },
|
|
13
|
+
{ name: 'writeFileSync', fix: 'writeFile (async)' },
|
|
14
|
+
{ name: 'readdirSync', fix: 'readdir (async)' },
|
|
15
|
+
{ name: 'statSync', fix: 'stat (async)' },
|
|
16
|
+
{ name: 'existsSync', fix: 'access (async)' },
|
|
17
|
+
{ name: 'copyFileSync', fix: 'copyFile (async)' },
|
|
18
|
+
{ name: 'mkdirSync', fix: 'mkdir (async)' },
|
|
19
|
+
];
|
|
20
|
+
|
|
21
|
+
const SHELL_SYNC_NAMES = ['execSync', 'execFileSync', 'spawnSync'];
|
|
22
|
+
|
|
23
|
+
// Patrones de I/O síncrona en Python dentro de handlers async
|
|
24
|
+
const PYTHON_SYNC_IO = [
|
|
25
|
+
{ name: 'open(', fix: 'aiofiles.open() o executor' },
|
|
26
|
+
{ name: 'subprocess.run(', fix: 'asyncio.create_subprocess_exec()' },
|
|
27
|
+
{ name: 'subprocess.call(', fix: 'asyncio.create_subprocess_exec()' },
|
|
28
|
+
{ name: 'subprocess.check_output(', fix: 'asyncio.create_subprocess_exec()' },
|
|
29
|
+
{ name: 'os.path.exists(', fix: 'asyncio.to_thread(os.path.exists, ...)' },
|
|
30
|
+
{ name: 'os.listdir(', fix: 'asyncio.to_thread(os.listdir, ...)' },
|
|
31
|
+
{ name: 'os.makedirs(', fix: 'asyncio.to_thread(os.makedirs, ...)' },
|
|
32
|
+
];
|
|
33
|
+
|
|
34
|
+
/**
|
|
35
|
+
* Determina si el archivo es un route handler (JS/TS o Python).
|
|
36
|
+
* @param {string} content
|
|
37
|
+
* @param {string} filePath
|
|
38
|
+
* @returns {boolean}
|
|
39
|
+
*/
|
|
40
|
+
function isRouteHandler(content, filePath) {
|
|
41
|
+
const rel = filePath.toLowerCase();
|
|
42
|
+
const isPython = rel.endsWith('.py');
|
|
43
|
+
|
|
44
|
+
if (!isPython) {
|
|
45
|
+
if (rel.includes('/route') || rel.includes('/api/') || rel.includes('/handler') || rel.includes('/controller')) return true;
|
|
46
|
+
if (content.includes('app.get(') || content.includes('app.post(') || content.includes('app.put(') || content.includes('app.delete(')) return true;
|
|
47
|
+
if (content.includes('router.get(') || content.includes('router.post(')) return true;
|
|
48
|
+
if (content.includes('.onRequest(') || content.includes('Hono(')) return true;
|
|
49
|
+
if (content.includes('export default') && (content.includes('GET') || content.includes('POST')) && rel.includes('route')) return true;
|
|
50
|
+
} else {
|
|
51
|
+
// Python: FastAPI, Flask, Django views
|
|
52
|
+
if (content.includes('@app.get(') || content.includes('@app.post(') ||
|
|
53
|
+
content.includes('@app.put(') || content.includes('@app.delete(') ||
|
|
54
|
+
content.includes('@app.patch(') || content.includes('@router.get(') ||
|
|
55
|
+
content.includes('@router.post(')) return true;
|
|
56
|
+
if (rel.includes('/views') || rel.includes('/routes') || rel.includes('/handlers') || rel.includes('/api/')) return true;
|
|
57
|
+
// Django class-based views
|
|
58
|
+
if (content.includes('class') && (content.includes('APIView') || content.includes('ViewSet') || content.includes('GenericView'))) return true;
|
|
59
|
+
}
|
|
60
|
+
return false;
|
|
61
|
+
}
|
|
62
|
+
|
|
63
|
+
/**
|
|
64
|
+
* Determina si el archivo es non-production (seed, test, migration).
|
|
65
|
+
* @param {string} filePath
|
|
66
|
+
* @returns {boolean}
|
|
67
|
+
*/
|
|
68
|
+
function isNonProductionFile(filePath) {
|
|
69
|
+
const rel = filePath.toLowerCase();
|
|
70
|
+
return (
|
|
71
|
+
rel.includes('seed') || rel.includes('fixture') ||
|
|
72
|
+
rel.includes('test') || rel.includes('spec') || rel.includes('__test') ||
|
|
73
|
+
rel.includes('migration') || rel.includes('migrate') ||
|
|
74
|
+
rel.includes('/scripts/') || rel.includes('/demo')
|
|
75
|
+
);
|
|
76
|
+
}
|
|
77
|
+
|
|
78
|
+
/**
|
|
79
|
+
* Verifica si una línea está dentro de un contexto seed/demo/test
|
|
80
|
+
* (buscando hacia atrás la definición de función más cercana).
|
|
81
|
+
* @param {string[]} lines
|
|
82
|
+
* @param {number} lineIndex
|
|
83
|
+
* @returns {boolean}
|
|
84
|
+
*/
|
|
85
|
+
function isInSeedContext(lines, lineIndex) {
|
|
86
|
+
const SEED_KEYWORDS = /seed|demo|populate|fixture|mock|fake|sample|generate.*data|test.*data/i;
|
|
87
|
+
const ROUTE_DEF = /\.(get|post|put|delete|patch|all)\s*\(/;
|
|
88
|
+
const FUNC_DEF = /(?:function|const|let|var)\s+\w+.*(?:=>|\{)|(?:async\s+function)/;
|
|
89
|
+
|
|
90
|
+
for (let i = lineIndex; i >= Math.max(0, lineIndex - 500); i--) {
|
|
91
|
+
const line = lines[i];
|
|
92
|
+
if (ROUTE_DEF.test(line)) {
|
|
93
|
+
for (let j = Math.max(0, i - 2); j <= Math.min(lines.length - 1, i + 2); j++) {
|
|
94
|
+
if (SEED_KEYWORDS.test(lines[j])) return true;
|
|
95
|
+
}
|
|
96
|
+
return false;
|
|
97
|
+
}
|
|
98
|
+
if (SEED_KEYWORDS.test(line) && FUNC_DEF.test(line)) return true;
|
|
99
|
+
if (/^\s*\/\/.*(?:seed|demo|populate|test data)/i.test(line)) return true;
|
|
100
|
+
}
|
|
101
|
+
return false;
|
|
102
|
+
}
|
|
103
|
+
|
|
104
|
+
/**
|
|
105
|
+
* Detecta si una línea Python está dentro de una función async.
|
|
106
|
+
* Escanea hacia atrás buscando `async def`.
|
|
107
|
+
* @param {string[]} lines
|
|
108
|
+
* @param {number} lineIndex
|
|
109
|
+
* @returns {boolean}
|
|
110
|
+
*/
|
|
111
|
+
function isInsidePythonAsyncDef(lines, lineIndex) {
|
|
112
|
+
// Obtener nivel de indentación de la línea actual
|
|
113
|
+
const currentLine = lines[lineIndex];
|
|
114
|
+
const currentIndent = currentLine.match(/^(\s*)/)[1].length;
|
|
115
|
+
|
|
116
|
+
for (let i = lineIndex - 1; i >= Math.max(0, lineIndex - 100); i--) {
|
|
117
|
+
const line = lines[i];
|
|
118
|
+
if (!line.trim()) continue;
|
|
119
|
+
const indent = line.match(/^(\s*)/)[1].length;
|
|
120
|
+
// Buscamos una definición de función con menor indentación
|
|
121
|
+
if (indent < currentIndent && /^\s*async\s+def\s+/.test(line)) return true;
|
|
122
|
+
if (indent < currentIndent && /^\s*def\s+/.test(line)) return false;
|
|
123
|
+
if (indent === 0 && /^(class|def|async def)/.test(line)) return false;
|
|
124
|
+
}
|
|
125
|
+
return false;
|
|
126
|
+
}
|
|
127
|
+
|
|
128
|
+
/**
|
|
129
|
+
* Analiza un archivo y devuelve los hallazgos de rendimiento.
|
|
130
|
+
* @param {string} filePath
|
|
131
|
+
* @param {string} relPath
|
|
132
|
+
* @param {string} content
|
|
133
|
+
* @returns {object[]}
|
|
134
|
+
*/
|
|
135
|
+
function analyzeFile(filePath, relPath, content) {
|
|
136
|
+
const findings = [];
|
|
137
|
+
const lines = content.split('\n');
|
|
138
|
+
const isHandler = isRouteHandler(content, relPath);
|
|
139
|
+
const isNonProd = isNonProductionFile(relPath);
|
|
140
|
+
const isPython = relPath.toLowerCase().endsWith('.py');
|
|
141
|
+
|
|
142
|
+
if (isPython) {
|
|
143
|
+
return analyzeFilePython(filePath, relPath, content, lines, isHandler, isNonProd);
|
|
144
|
+
}
|
|
145
|
+
|
|
146
|
+
// === N+1 Query Detection (JS/TS) ===
|
|
147
|
+
const blockLoopPatterns = [
|
|
148
|
+
/\bfor\s*\(/,
|
|
149
|
+
/\bfor\s+.*\bof\b/,
|
|
150
|
+
/\bwhile\s*\(/,
|
|
151
|
+
];
|
|
152
|
+
const callbackLoopPattern = /\.(forEach|map)\s*\(/;
|
|
153
|
+
const queryPatterns = [
|
|
154
|
+
/\.findOne\s*\(/, /\.findFirst\s*\(/, /\.findUnique\s*\(/,
|
|
155
|
+
/\.find\s*\(/, /\.findMany\s*\(/,
|
|
156
|
+
/\.query\s*\(/, /\.execute\s*\(/,
|
|
157
|
+
/await\s+db\./, /await\s+prisma\./,
|
|
158
|
+
];
|
|
159
|
+
|
|
160
|
+
const blockLoopStack = [];
|
|
161
|
+
const callbackLoopStack = [];
|
|
162
|
+
let braceDepth = 0;
|
|
163
|
+
let parenDepth = 0;
|
|
164
|
+
|
|
165
|
+
for (let i = 0; i < lines.length; i++) {
|
|
166
|
+
const line = lines[i];
|
|
167
|
+
const trimmed = line.trim();
|
|
168
|
+
if (trimmed.startsWith('//') || trimmed.startsWith('*') || trimmed.startsWith('/*')) continue;
|
|
169
|
+
|
|
170
|
+
const isBlockLoop = blockLoopPatterns.some(p => p.test(line));
|
|
171
|
+
const callbackMatch = callbackLoopPattern.test(line);
|
|
172
|
+
|
|
173
|
+
if (isBlockLoop) blockLoopStack.push(braceDepth);
|
|
174
|
+
if (callbackMatch) callbackLoopStack.push(parenDepth);
|
|
175
|
+
|
|
176
|
+
for (const ch of line) {
|
|
177
|
+
if (ch === '{') braceDepth++;
|
|
178
|
+
if (ch === '(') parenDepth++;
|
|
179
|
+
if (ch === '}') {
|
|
180
|
+
braceDepth--;
|
|
181
|
+
while (blockLoopStack.length > 0 && braceDepth <= blockLoopStack[blockLoopStack.length - 1]) {
|
|
182
|
+
blockLoopStack.pop();
|
|
183
|
+
}
|
|
184
|
+
}
|
|
185
|
+
if (ch === ')') {
|
|
186
|
+
parenDepth--;
|
|
187
|
+
while (callbackLoopStack.length > 0 && parenDepth <= callbackLoopStack[callbackLoopStack.length - 1]) {
|
|
188
|
+
callbackLoopStack.pop();
|
|
189
|
+
}
|
|
190
|
+
}
|
|
191
|
+
}
|
|
192
|
+
|
|
193
|
+
const insideBlockLoop = blockLoopStack.length > 0;
|
|
194
|
+
const insideCallbackLoop = callbackLoopStack.length > 0;
|
|
195
|
+
const sameLineCallback = callbackMatch && !insideCallbackLoop;
|
|
196
|
+
const insideAnyLoop = insideBlockLoop || insideCallbackLoop || sameLineCallback;
|
|
197
|
+
|
|
198
|
+
if (insideAnyLoop && queryPatterns.some(p => p.test(line))) {
|
|
199
|
+
const inSeedCtx = isNonProd || isInSeedContext(lines, i);
|
|
200
|
+
const severity = inSeedCtx ? 'low' : 'high';
|
|
201
|
+
const suffix = inSeedCtx ? ' (en contexto seed/test — prioridad baja)' : '';
|
|
202
|
+
findings.push({
|
|
203
|
+
file: relPath, line: i + 1, severity, category: 'n+1',
|
|
204
|
+
message: `Consulta de BD dentro de bucle — patrón N+1 detectado. Usar consulta batch (findMany/cláusula IN)${suffix}`,
|
|
205
|
+
code: trimmed.slice(0, 120),
|
|
206
|
+
});
|
|
207
|
+
}
|
|
208
|
+
}
|
|
209
|
+
|
|
210
|
+
// === Sync I/O en Request Handlers (JS/TS) ===
|
|
211
|
+
if (isHandler) {
|
|
212
|
+
for (let i = 0; i < lines.length; i++) {
|
|
213
|
+
const line = lines[i];
|
|
214
|
+
const trimmed = line.trim();
|
|
215
|
+
if (trimmed.startsWith('//') || trimmed.startsWith('*')) continue;
|
|
216
|
+
|
|
217
|
+
for (const sp of SYNC_PATTERNS) {
|
|
218
|
+
if (line.includes(sp.name)) {
|
|
219
|
+
findings.push({
|
|
220
|
+
file: relPath, line: i + 1, severity: 'high', category: 'sync-io',
|
|
221
|
+
message: `I/O síncrona (${sp.name}) en request handler bloquea el event loop — usar ${sp.fix}`,
|
|
222
|
+
code: trimmed.slice(0, 120),
|
|
223
|
+
});
|
|
224
|
+
}
|
|
225
|
+
}
|
|
226
|
+
|
|
227
|
+
for (const name of SHELL_SYNC_NAMES) {
|
|
228
|
+
if (line.includes(name)) {
|
|
229
|
+
findings.push({
|
|
230
|
+
file: relPath, line: i + 1, severity: 'high', category: 'sync-io',
|
|
231
|
+
message: `Ejecución de shell síncrona (${name}) en request handler — usar alternativa async`,
|
|
232
|
+
code: trimmed.slice(0, 120),
|
|
233
|
+
});
|
|
234
|
+
}
|
|
235
|
+
}
|
|
236
|
+
}
|
|
237
|
+
}
|
|
238
|
+
|
|
239
|
+
// === Operaciones sin límite (JS/TS) ===
|
|
240
|
+
for (let i = 0; i < lines.length; i++) {
|
|
241
|
+
const line = lines[i];
|
|
242
|
+
const trimmed = line.trim();
|
|
243
|
+
if (trimmed.startsWith('//') || trimmed.startsWith('*')) continue;
|
|
244
|
+
|
|
245
|
+
if (/\.findMany\s*\(\s*\{/.test(line)) {
|
|
246
|
+
const startIdx = content.indexOf(line);
|
|
247
|
+
const chunk = startIdx >= 0 ? content.slice(startIdx, startIdx + 300) : '';
|
|
248
|
+
if (!chunk.includes('take:') && !chunk.includes('limit')) {
|
|
249
|
+
findings.push({
|
|
250
|
+
file: relPath, line: i + 1, severity: 'high', category: 'unbounded',
|
|
251
|
+
message: 'findMany sin limit/take — puede retornar la tabla completa en memoria',
|
|
252
|
+
code: trimmed.slice(0, 120),
|
|
253
|
+
});
|
|
254
|
+
}
|
|
255
|
+
}
|
|
256
|
+
|
|
257
|
+
if (/SELECT\s+\*/i.test(line) && !/LIMIT/i.test(line)) {
|
|
258
|
+
const selectMatch = line.match(/SELECT\s+\*/i);
|
|
259
|
+
const selectIdx = selectMatch ? line.indexOf(selectMatch[0]) : -1;
|
|
260
|
+
const beforeSelect = selectIdx >= 0 ? line.slice(0, selectIdx) : '';
|
|
261
|
+
const isInJsxComment = /\{\/\*/.test(beforeSelect) && /\*\/\}/.test(line.slice(selectIdx));
|
|
262
|
+
const isInBlockComment = /\/\*/.test(beforeSelect) && !/\*\//.test(beforeSelect.slice(beforeSelect.lastIndexOf('/*')));
|
|
263
|
+
const isAfterLineComment = /\/\//.test(beforeSelect);
|
|
264
|
+
const isInComment = isInJsxComment || isInBlockComment || isAfterLineComment;
|
|
265
|
+
const isInSqlContext = /['"`].*SELECT\s+\*/i.test(line) || /SELECT\s+\*.*['"`]/i.test(line);
|
|
266
|
+
if (!isInComment && isInSqlContext) {
|
|
267
|
+
findings.push({
|
|
268
|
+
file: relPath, line: i + 1, severity: 'high', category: 'unbounded',
|
|
269
|
+
message: 'SELECT * sin LIMIT — puede retornar la tabla completa',
|
|
270
|
+
code: trimmed.slice(0, 120),
|
|
271
|
+
});
|
|
272
|
+
}
|
|
273
|
+
}
|
|
274
|
+
}
|
|
275
|
+
|
|
276
|
+
// === Índices de BD faltantes (esquemas) ===
|
|
277
|
+
if (relPath.includes('schema') || relPath.includes('migration')) {
|
|
278
|
+
const fkPattern = /\.references\s*\(\s*\(\)\s*=>\s*(\w+)\.(\w+)\)/g;
|
|
279
|
+
let fkMatch;
|
|
280
|
+
while ((fkMatch = fkPattern.exec(content)) !== null) {
|
|
281
|
+
const lineNum = content.slice(0, fkMatch.index).split('\n').length;
|
|
282
|
+
const nearby = content.slice(Math.max(0, fkMatch.index - 500), fkMatch.index + 500);
|
|
283
|
+
if (!nearby.includes('index(') && !nearby.includes('.index(')) {
|
|
284
|
+
findings.push({
|
|
285
|
+
file: relPath, line: lineNum, severity: 'medium', category: 'missing-index',
|
|
286
|
+
message: `Clave foránea a ${fkMatch[1]}.${fkMatch[2]} sin índice — los JOINs serán lentos a escala`,
|
|
287
|
+
code: fkMatch[0].slice(0, 120),
|
|
288
|
+
});
|
|
289
|
+
}
|
|
290
|
+
}
|
|
291
|
+
|
|
292
|
+
if (content.includes('@relation')) {
|
|
293
|
+
const relPattern = /@relation.*fields:\s*\[(\w+)\]/g;
|
|
294
|
+
let relMatch;
|
|
295
|
+
while ((relMatch = relPattern.exec(content)) !== null) {
|
|
296
|
+
const field = relMatch[1];
|
|
297
|
+
if (!content.includes(`@@index([${field}]`) && !content.includes('@unique')) {
|
|
298
|
+
const lineNum = content.slice(0, relMatch.index).split('\n').length;
|
|
299
|
+
findings.push({
|
|
300
|
+
file: relPath, line: lineNum, severity: 'medium', category: 'missing-index',
|
|
301
|
+
message: `Campo de relación "${field}" sin @@index — las búsquedas serán lentas a escala`,
|
|
302
|
+
code: relMatch[0].slice(0, 120),
|
|
303
|
+
});
|
|
304
|
+
}
|
|
305
|
+
}
|
|
306
|
+
}
|
|
307
|
+
}
|
|
308
|
+
|
|
309
|
+
// === Memory Leaks (JS/TS) ===
|
|
310
|
+
const SERVER_PATTERNS = /app\.get\(|app\.listen\(|app\.post\(|Hono\(|createServer|express\(|router\.(get|post|put|delete)\(|\.onRequest\(/;
|
|
311
|
+
const isOneShotScript = (
|
|
312
|
+
relPath.includes('/scripts/') ||
|
|
313
|
+
relPath.includes('cli.') ||
|
|
314
|
+
relPath.includes('/bin/') ||
|
|
315
|
+
Boolean(relPath.match(/(?:^|\/)cli[./]/))
|
|
316
|
+
) || !SERVER_PATTERNS.test(content);
|
|
317
|
+
|
|
318
|
+
let braceDepthML = 0;
|
|
319
|
+
for (let i = 0; i < lines.length; i++) {
|
|
320
|
+
const line = lines[i];
|
|
321
|
+
const trimmed = line.trim();
|
|
322
|
+
if (trimmed.startsWith('//') || trimmed.startsWith('*')) continue;
|
|
323
|
+
|
|
324
|
+
for (const ch of line) {
|
|
325
|
+
if (ch === '{') braceDepthML++;
|
|
326
|
+
if (ch === '}') braceDepthML--;
|
|
327
|
+
}
|
|
328
|
+
|
|
329
|
+
if (/^(const|let|var)\s+\w+\s*=\s*\[\s*\]/.test(trimmed) && braceDepthML === 0) {
|
|
330
|
+
const varName = trimmed.match(/^(?:const|let|var)\s+(\w+)/)?.[1];
|
|
331
|
+
if (varName && content.includes(`${varName}.push(`)) {
|
|
332
|
+
findings.push({
|
|
333
|
+
file: relPath, line: i + 1, severity: isOneShotScript ? 'low' : 'medium', category: 'memory-leak',
|
|
334
|
+
message: `Array con scope de módulo "${varName}" con .push() — crece sin límite${isOneShotScript ? ' (script de una vez)' : ', memory leak en servidores long-running'}`,
|
|
335
|
+
code: trimmed.slice(0, 120),
|
|
336
|
+
});
|
|
337
|
+
}
|
|
338
|
+
}
|
|
339
|
+
|
|
340
|
+
if (isHandler && (/\.addEventListener\s*\(/.test(line) || /\.on\s*\(\s*['"]/.test(line))) {
|
|
341
|
+
const isReqBodyParsing = /\b(req|request|res|response)\s*\.\s*on\s*\(\s*['"](data|end|error|close)['"]/i.test(line);
|
|
342
|
+
if (!isReqBodyParsing) {
|
|
343
|
+
findings.push({
|
|
344
|
+
file: relPath, line: i + 1, severity: 'medium', category: 'memory-leak',
|
|
345
|
+
message: 'Event listener en request handler — puede acumularse sin limpieza',
|
|
346
|
+
code: trimmed.slice(0, 120),
|
|
347
|
+
});
|
|
348
|
+
}
|
|
349
|
+
}
|
|
350
|
+
}
|
|
351
|
+
|
|
352
|
+
// === Error Handling en Handlers (JS/TS) ===
|
|
353
|
+
if (isHandler) {
|
|
354
|
+
for (let i = 0; i < lines.length; i++) {
|
|
355
|
+
const line = lines[i];
|
|
356
|
+
const trimmed = line.trim();
|
|
357
|
+
if (trimmed.startsWith('//') || trimmed.startsWith('*')) continue;
|
|
358
|
+
|
|
359
|
+
if (/JSON\.parse\s*\(/.test(line)) {
|
|
360
|
+
let hasTry = false;
|
|
361
|
+
for (let j = Math.max(0, i - 5); j < i; j++) {
|
|
362
|
+
if (lines[j].includes('try')) hasTry = true;
|
|
363
|
+
}
|
|
364
|
+
if (!hasTry) {
|
|
365
|
+
findings.push({
|
|
366
|
+
file: relPath, line: i + 1, severity: 'medium', category: 'error-handling',
|
|
367
|
+
message: 'JSON.parse sin try/catch en handler — input malformado crashea el handler',
|
|
368
|
+
code: trimmed.slice(0, 120),
|
|
369
|
+
});
|
|
370
|
+
}
|
|
371
|
+
}
|
|
372
|
+
|
|
373
|
+
if (/new RegExp\s*\(/.test(line)) {
|
|
374
|
+
findings.push({
|
|
375
|
+
file: relPath, line: i + 1, severity: 'medium', category: 'redos',
|
|
376
|
+
message: 'RegExp dinámico en handler — input controlado por usuario puede causar ReDoS',
|
|
377
|
+
code: trimmed.slice(0, 120),
|
|
378
|
+
});
|
|
379
|
+
}
|
|
380
|
+
}
|
|
381
|
+
}
|
|
382
|
+
|
|
383
|
+
// === Sequential Await (JS/TS) ===
|
|
384
|
+
if (!isNonProd && !relPath.includes('/scripts/') && !relPath.includes('cleanup') && !relPath.includes('migrate')) {
|
|
385
|
+
for (let i = 0; i < lines.length - 1; i++) {
|
|
386
|
+
const line = lines[i].trim();
|
|
387
|
+
const nextLine = (lines[i + 1] || '').trim();
|
|
388
|
+
if (line.startsWith('//') || line.startsWith('*')) continue;
|
|
389
|
+
|
|
390
|
+
if (/^(const|let|var)\s+\w+\s*=\s*await\b/.test(line) && /^(const|let|var)\s+\w+\s*=\s*await\b/.test(nextLine)) {
|
|
391
|
+
const firstVar = line.match(/^(?:const|let|var)\s+(\w+)/)?.[1];
|
|
392
|
+
if (firstVar && !nextLine.includes(firstVar)) {
|
|
393
|
+
findings.push({
|
|
394
|
+
file: relPath, line: i + 1, severity: 'low', category: 'sequential-await',
|
|
395
|
+
message: 'Awaits secuenciales que podrían ejecutarse en paralelo — considerar Promise.all()',
|
|
396
|
+
code: `${line.slice(0, 60)} | ${nextLine.slice(0, 60)}`,
|
|
397
|
+
});
|
|
398
|
+
}
|
|
399
|
+
}
|
|
400
|
+
}
|
|
401
|
+
}
|
|
402
|
+
|
|
403
|
+
return findings;
|
|
404
|
+
}
|
|
405
|
+
|
|
406
|
+
/**
|
|
407
|
+
* Analiza archivos Python para detectar:
|
|
408
|
+
* - N+1 con SQLAlchemy/Django ORM
|
|
409
|
+
* - I/O síncrona en handlers async
|
|
410
|
+
* - Consultas sin límite (.all() sin .limit())
|
|
411
|
+
* @param {string} filePath
|
|
412
|
+
* @param {string} relPath
|
|
413
|
+
* @param {string} content
|
|
414
|
+
* @param {string[]} lines
|
|
415
|
+
* @param {boolean} isHandler
|
|
416
|
+
* @param {boolean} isNonProd
|
|
417
|
+
* @returns {object[]}
|
|
418
|
+
*/
|
|
419
|
+
function analyzeFilePython(filePath, relPath, content, lines, isHandler, isNonProd) {
|
|
420
|
+
const findings = [];
|
|
421
|
+
|
|
422
|
+
// Patrones de consulta SQLAlchemy/Django ORM
|
|
423
|
+
const sqlalchemyQueryPatterns = [
|
|
424
|
+
/session\.query\s*\(/,
|
|
425
|
+
/\.query\s*\(/,
|
|
426
|
+
/db\.query\s*\(/,
|
|
427
|
+
/\.\bfilter\s*\(/,
|
|
428
|
+
/\.\bfirst\s*\(\s*\)/,
|
|
429
|
+
/\.\bone\s*\(\s*\)/,
|
|
430
|
+
/\.\bget\s*\(/,
|
|
431
|
+
];
|
|
432
|
+
const djangoQueryPatterns = [
|
|
433
|
+
/\bObjects\.filter\s*\(/,
|
|
434
|
+
/\bObjects\.get\s*\(/,
|
|
435
|
+
/\bObjects\.exclude\s*\(/,
|
|
436
|
+
/\.objects\.\w+\s*\(/,
|
|
437
|
+
];
|
|
438
|
+
|
|
439
|
+
// Loop patterns en Python
|
|
440
|
+
const pythonLoopPattern = /^\s*for\s+\w+.*\bin\b/;
|
|
441
|
+
const pythonWhilePattern = /^\s*while\s+/;
|
|
442
|
+
|
|
443
|
+
// Seguimiento de profundidad de indentación para loops
|
|
444
|
+
let loopIndentStack = []; // cada entry: nivel de indentación del loop
|
|
445
|
+
|
|
446
|
+
for (let i = 0; i < lines.length; i++) {
|
|
447
|
+
const line = lines[i];
|
|
448
|
+
const trimmed = line.trim();
|
|
449
|
+
if (trimmed.startsWith('#')) continue;
|
|
450
|
+
|
|
451
|
+
const currentIndent = line.match(/^(\s*)/)[1].length;
|
|
452
|
+
|
|
453
|
+
// Sacar loops que ya cerraron (por indentación)
|
|
454
|
+
loopIndentStack = loopIndentStack.filter(indent => indent < currentIndent);
|
|
455
|
+
|
|
456
|
+
const isLoopLine = pythonLoopPattern.test(line) || pythonWhilePattern.test(line);
|
|
457
|
+
if (isLoopLine) {
|
|
458
|
+
loopIndentStack.push(currentIndent);
|
|
459
|
+
}
|
|
460
|
+
|
|
461
|
+
const insideLoop = loopIndentStack.length > 0;
|
|
462
|
+
|
|
463
|
+
// N+1 con SQLAlchemy
|
|
464
|
+
if (insideLoop && !isNonProd) {
|
|
465
|
+
const hasQuery = sqlalchemyQueryPatterns.some(p => p.test(line)) ||
|
|
466
|
+
djangoQueryPatterns.some(p => p.test(trimmed.toLowerCase() === trimmed ? line : line));
|
|
467
|
+
if (hasQuery) {
|
|
468
|
+
findings.push({
|
|
469
|
+
file: relPath, line: i + 1, severity: 'high', category: 'n+1',
|
|
470
|
+
message: 'Consulta ORM dentro de bucle Python — patrón N+1 detectado. Usar carga eager (joinedload/selectinload) o consultas batch',
|
|
471
|
+
code: trimmed.slice(0, 120),
|
|
472
|
+
});
|
|
473
|
+
}
|
|
474
|
+
}
|
|
475
|
+
|
|
476
|
+
// I/O síncrona en handlers async
|
|
477
|
+
if (isHandler || /^\s*async\s+def\s+/.test(line)) {
|
|
478
|
+
// Verificar que estamos dentro de una función async
|
|
479
|
+
const isAsync = isInsidePythonAsyncDef(lines, i);
|
|
480
|
+
if (isAsync) {
|
|
481
|
+
for (const sp of PYTHON_SYNC_IO) {
|
|
482
|
+
if (line.includes(sp.name)) {
|
|
483
|
+
findings.push({
|
|
484
|
+
file: relPath, line: i + 1, severity: 'high', category: 'sync-io',
|
|
485
|
+
message: `I/O síncrona (${sp.name}) en handler async Python bloquea el event loop — usar ${sp.fix}`,
|
|
486
|
+
code: trimmed.slice(0, 120),
|
|
487
|
+
});
|
|
488
|
+
}
|
|
489
|
+
}
|
|
490
|
+
}
|
|
491
|
+
}
|
|
492
|
+
|
|
493
|
+
// Consultas sin límite: .all() sin .limit() previo
|
|
494
|
+
if (/\.all\s*\(\s*\)/.test(line)) {
|
|
495
|
+
// Buscar si hay .limit() en la misma línea o en las 5 líneas previas
|
|
496
|
+
let hasLimit = /\.limit\s*\(/.test(line);
|
|
497
|
+
if (!hasLimit) {
|
|
498
|
+
for (let j = Math.max(0, i - 5); j < i; j++) {
|
|
499
|
+
if (/\.limit\s*\(/.test(lines[j])) { hasLimit = true; break; }
|
|
500
|
+
}
|
|
501
|
+
}
|
|
502
|
+
if (!hasLimit && !isNonProd) {
|
|
503
|
+
findings.push({
|
|
504
|
+
file: relPath, line: i + 1, severity: 'high', category: 'unbounded',
|
|
505
|
+
message: '.all() sin .limit() previo — puede retornar la tabla completa en memoria',
|
|
506
|
+
code: trimmed.slice(0, 120),
|
|
507
|
+
});
|
|
508
|
+
}
|
|
509
|
+
}
|
|
510
|
+
|
|
511
|
+
// SELECT * sin LIMIT en strings Python
|
|
512
|
+
if (/SELECT\s+\*/i.test(line) && !/LIMIT/i.test(line)) {
|
|
513
|
+
const isInComment = trimmed.startsWith('#');
|
|
514
|
+
const isInSqlContext = /['""].*SELECT\s+\*/i.test(line) || /SELECT\s+\*.*['""]/.test(line);
|
|
515
|
+
if (!isInComment && isInSqlContext) {
|
|
516
|
+
findings.push({
|
|
517
|
+
file: relPath, line: i + 1, severity: 'high', category: 'unbounded',
|
|
518
|
+
message: 'SELECT * sin LIMIT en Python — puede retornar la tabla completa',
|
|
519
|
+
code: trimmed.slice(0, 120),
|
|
520
|
+
});
|
|
521
|
+
}
|
|
522
|
+
}
|
|
523
|
+
}
|
|
524
|
+
|
|
525
|
+
return findings;
|
|
526
|
+
}
|
|
527
|
+
|
|
528
|
+
function main() {
|
|
529
|
+
const dir = process.argv[2];
|
|
530
|
+
if (!dir) {
|
|
531
|
+
outputError('Uso: node code-profiler.js <directorio-proyecto>');
|
|
532
|
+
process.exit(0);
|
|
533
|
+
}
|
|
534
|
+
|
|
535
|
+
if (!existsSync(dir)) {
|
|
536
|
+
outputError(`Ruta no encontrada: ${dir}`);
|
|
537
|
+
process.exit(0);
|
|
538
|
+
}
|
|
539
|
+
|
|
540
|
+
const codeFiles = walkCode(dir, { extensions: ['.ts', '.tsx', '.js', '.jsx', '.mjs', '.cjs', '.py'] });
|
|
541
|
+
if (codeFiles.length === 0) {
|
|
542
|
+
outputJSON({ success: true, message: 'No se encontraron archivos TypeScript/JavaScript/Python', findings: [] });
|
|
543
|
+
process.exit(0);
|
|
544
|
+
}
|
|
545
|
+
|
|
546
|
+
const allFindings = [];
|
|
547
|
+
let filesAnalyzed = 0;
|
|
548
|
+
let handlersFound = 0;
|
|
549
|
+
|
|
550
|
+
for (const file of codeFiles) {
|
|
551
|
+
let content;
|
|
552
|
+
try {
|
|
553
|
+
content = readFileSync(file, 'utf8');
|
|
554
|
+
} catch {
|
|
555
|
+
continue;
|
|
556
|
+
}
|
|
557
|
+
const relPath = relative(dir, file).replace(/\\/g, '/');
|
|
558
|
+
if (isRouteHandler(content, relPath)) handlersFound++;
|
|
559
|
+
allFindings.push(...analyzeFile(file, relPath, content));
|
|
560
|
+
filesAnalyzed++;
|
|
561
|
+
}
|
|
562
|
+
|
|
563
|
+
const byCategory = {};
|
|
564
|
+
for (const f of allFindings) {
|
|
565
|
+
if (!byCategory[f.category]) byCategory[f.category] = [];
|
|
566
|
+
byCategory[f.category].push(f);
|
|
567
|
+
}
|
|
568
|
+
|
|
569
|
+
const categorySummary = {};
|
|
570
|
+
for (const [cat, items] of Object.entries(byCategory)) {
|
|
571
|
+
categorySummary[cat] = {
|
|
572
|
+
count: items.length,
|
|
573
|
+
critical: items.filter(i => i.severity === 'critical').length,
|
|
574
|
+
high: items.filter(i => i.severity === 'high').length,
|
|
575
|
+
medium: items.filter(i => i.severity === 'medium').length,
|
|
576
|
+
low: items.filter(i => i.severity === 'low').length,
|
|
577
|
+
};
|
|
578
|
+
}
|
|
579
|
+
|
|
580
|
+
let score = 100;
|
|
581
|
+
const seenCatFile = new Set();
|
|
582
|
+
for (const f of allFindings) {
|
|
583
|
+
const key = `${f.category}:${f.file}`;
|
|
584
|
+
const firstInFile = !seenCatFile.has(key);
|
|
585
|
+
seenCatFile.add(key);
|
|
586
|
+
const mult = firstInFile ? 1 : 0.5;
|
|
587
|
+
if (f.severity === 'high') score -= 5 * mult;
|
|
588
|
+
else if (f.severity === 'medium') score -= 2 * mult;
|
|
589
|
+
else if (f.severity === 'low') score -= 0.5 * mult;
|
|
590
|
+
}
|
|
591
|
+
|
|
592
|
+
outputJSON({
|
|
593
|
+
success: true,
|
|
594
|
+
files_analyzed: filesAnalyzed,
|
|
595
|
+
handlers_found: handlersFound,
|
|
596
|
+
total_findings: allFindings.length,
|
|
597
|
+
performance_score: Math.max(0, Math.round(score)),
|
|
598
|
+
categories: categorySummary,
|
|
599
|
+
findings: allFindings,
|
|
600
|
+
});
|
|
601
|
+
}
|
|
602
|
+
|
|
603
|
+
main();
|
|
604
|
+
|
|
605
|
+
module.exports = { analyzeFile, isRouteHandler, isNonProductionFile, isInSeedContext };
|