@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.
Files changed (114) hide show
  1. package/CLAUDE.md +1 -1
  2. package/README.md +1 -1
  3. package/agentes/nemesis-auditor-swl.md +161 -161
  4. package/bin/swl-mcp-server.js +187 -187
  5. package/comandos/swl/.evolved.json +22 -22
  6. package/comandos/swl/contribuir.md +233 -233
  7. package/comandos/swl/nemesis.md +122 -122
  8. package/gateway/lib/event-channel.js +191 -191
  9. package/habilidades/backend-production-resilience/SKILL.md +288 -288
  10. package/habilidades/benchmark-memoria/SKILL.md +186 -186
  11. package/habilidades/diagrama-arquitectura/assets/template.html +276 -276
  12. package/habilidades/doubt-driven-review/SKILL.md +171 -171
  13. package/habilidades/doubt-driven-review/recursos/EXAMPLES.md +130 -130
  14. package/habilidades/eval-framework/SKILL.md +212 -212
  15. package/habilidades/feynman-auditor-swl/SKILL.md +123 -123
  16. package/habilidades/feynman-auditor-swl/recursos/preguntas-language-agnostic.md +108 -108
  17. package/habilidades/harness-claude-code/SKILL.md +299 -299
  18. package/habilidades/infra-github-actions/SKILL.md +166 -166
  19. package/habilidades/legacy-code-rescue/SKILL.md +267 -267
  20. package/habilidades/manejo-errores/.evolved.json +8 -8
  21. package/habilidades/meta-skills-estandar/recursos/convencion-examples.md +93 -93
  22. package/habilidades/meta-skills-estandar/recursos/skills-as-agents.md +163 -163
  23. package/habilidades/patrones-python/SKILL.md +229 -229
  24. package/habilidades/patrones-python/recursos/patrones-avanzados.md +469 -469
  25. package/habilidades/planear-fase/SKILL.md +319 -319
  26. package/habilidades/release-semver/.evolved.json +8 -8
  27. package/habilidades/state-inconsistency-auditor-swl/SKILL.md +166 -166
  28. package/habilidades/state-inconsistency-auditor-swl/recursos/coupled-state-patterns.md +147 -147
  29. package/habilidades/testing-python/SKILL.md +340 -340
  30. package/habilidades/web-fetcher-routing/SKILL.md +75 -75
  31. package/hooks/claudemd-bloat-detector.js +161 -161
  32. package/hooks/lib/agent-routing.js +107 -107
  33. package/hooks/lib/auto-consolidator.js +335 -335
  34. package/hooks/lib/error-classifier.js +308 -308
  35. package/hooks/lib/merkle-audit.js +96 -96
  36. package/hooks/lib/provenance-tracker.js +191 -191
  37. package/hooks/lib/rate-limit-tracker.js +253 -253
  38. package/hooks/lib/resource-quota.js +122 -122
  39. package/hooks/lib/retry-jitter.js +165 -165
  40. package/hooks/lib/security-net.js +201 -201
  41. package/hooks/lib/skill-auditor.js +588 -588
  42. package/hooks/lib/sync-status.js +228 -228
  43. package/hooks/lib/taint-tracker.js +107 -107
  44. package/hooks/lib/text-similarity.js +241 -241
  45. package/hooks/lib/toon-compressor.js +245 -245
  46. package/hooks/registro-turnos.js +209 -209
  47. package/hooks/sugerir-regenerar-inventario.js +170 -170
  48. package/hooks/validar-formato-post-subagente.js +140 -140
  49. package/hooks/validar-memoria-hook.js +218 -218
  50. package/instintos/prompt-appendices.yaml +57 -57
  51. package/manifiestos/agent-output-schemas.json +57 -57
  52. package/manifiestos/modulos.json +11 -6
  53. package/manifiestos/perfiles.json +2 -1
  54. package/manifiestos/skills-lock.json +1114 -1114
  55. package/package.json +1 -1
  56. package/plantillas/auditor-veto-template.md +105 -105
  57. package/plantillas/github-workflows/README.md +47 -47
  58. package/plantillas/github-workflows/release-please.yml +44 -44
  59. package/plantillas/github-workflows/swl-ci.yml +107 -107
  60. package/plantillas/github-workflows/swl-security.yml +51 -51
  61. package/plugin.json +9 -1
  62. package/reglas/analisis-previo-tareas-grandes.md +172 -172
  63. package/reglas/arreglar-al-detectar.md +147 -147
  64. package/reglas/fragmentos-compartidos.md +152 -152
  65. package/reglas/harness-claude-code.md +213 -213
  66. package/reglas/usar-context7.md +226 -226
  67. package/schemas/diary-entry.schema.json +80 -80
  68. package/scripts/audit-tools/audit-history.js +330 -330
  69. package/scripts/audit-tools/bundle-tracker.js +290 -290
  70. package/scripts/audit-tools/canary-monitor.js +352 -352
  71. package/scripts/audit-tools/code-profiler.js +605 -605
  72. package/scripts/audit-tools/dep-doctor.js +320 -320
  73. package/scripts/audit-tools/env-validator.js +206 -206
  74. package/scripts/audit-tools/lib/fs-walk.js +48 -48
  75. package/scripts/audit-tools/lib/output.js +23 -23
  76. package/scripts/audit-tools/migration-checker.js +392 -392
  77. package/scripts/audit-tools/pentest-scanner.js +1436 -1436
  78. package/scripts/benchmark-memoria.js +167 -167
  79. package/scripts/configurar-branch-protection.js +418 -418
  80. package/scripts/detectar-aprendizajes-duplicados.js +151 -151
  81. package/scripts/field-report.js +199 -199
  82. package/scripts/generar-checklists-consolidados.js +273 -273
  83. package/scripts/generar-inventario.js +420 -420
  84. package/scripts/generar-matriz-lenguajes.js +271 -271
  85. package/scripts/lib/artefactos-python.js +43 -43
  86. package/scripts/lib/benchmark-metrics.js +160 -160
  87. package/scripts/lib/budget-enforcer.js +252 -252
  88. package/scripts/lib/configurar-ci.js +380 -380
  89. package/scripts/lib/contadores-inventario.js +217 -217
  90. package/scripts/lib/detectar-stack-detallado.js +307 -307
  91. package/scripts/lib/diary-entry.js +234 -234
  92. package/scripts/lib/eval-metrics-store.js +218 -218
  93. package/scripts/lib/eval-quality.js +171 -171
  94. package/scripts/lib/eval-schemas.js +144 -144
  95. package/scripts/lib/eval-self-correct.js +106 -106
  96. package/scripts/lib/eval-validator.js +185 -185
  97. package/scripts/lib/jaccard-similarity.js +98 -98
  98. package/scripts/lib/longmemeval-runner.js +125 -125
  99. package/scripts/lib/manifiestos.js +42 -1
  100. package/scripts/lib/npm-version.js +261 -261
  101. package/scripts/lib/paquetes-conocidos.js +50 -50
  102. package/scripts/lib/prompt-builder.js +264 -264
  103. package/scripts/lib/rrf-fusion.js +175 -175
  104. package/scripts/lib/scoring-instintos.js +277 -277
  105. package/scripts/lib/semantic-search.js +252 -252
  106. package/scripts/limpiar-artefactos-python.js +131 -131
  107. package/scripts/mcp-server/README.md +128 -128
  108. package/scripts/mcp-server/handlers.js +206 -206
  109. package/scripts/migrar-csv-a-array.js +168 -168
  110. package/scripts/migrar-fase-dominio.js +201 -201
  111. package/scripts/publicar.js +511 -511
  112. package/scripts/run-eval.js +141 -141
  113. package/scripts/validar-manifest.js +231 -195
  114. 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 };