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