@saulwade/swl-ses 1.5.1 → 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 (133) hide show
  1. package/CLAUDE.md +225 -209
  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 +207 -4
  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 +94 -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 +121 -4
  37. package/habilidades/testing-python/SKILL.md +340 -340
  38. package/habilidades/web-fetcher-routing/SKILL.md +75 -75
  39. package/hooks/check-update.js +31 -3
  40. package/hooks/claudemd-bloat-detector.js +161 -161
  41. package/hooks/lib/agent-routing.js +107 -107
  42. package/hooks/lib/auto-consolidator.js +335 -335
  43. package/hooks/lib/error-classifier.js +308 -308
  44. package/hooks/lib/merkle-audit.js +96 -96
  45. package/hooks/lib/provenance-tracker.js +191 -191
  46. package/hooks/lib/rate-limit-tracker.js +253 -253
  47. package/hooks/lib/resource-quota.js +122 -122
  48. package/hooks/lib/retry-jitter.js +165 -165
  49. package/hooks/lib/security-net.js +201 -201
  50. package/hooks/lib/skill-auditor.js +588 -588
  51. package/hooks/lib/sync-status.js +228 -228
  52. package/hooks/lib/taint-tracker.js +107 -107
  53. package/hooks/lib/text-similarity.js +241 -241
  54. package/hooks/lib/toon-compressor.js +245 -245
  55. package/hooks/registro-turnos.js +209 -209
  56. package/hooks/sugerir-regenerar-inventario.js +170 -170
  57. package/hooks/validar-formato-post-subagente.js +140 -140
  58. package/hooks/validar-memoria-hook.js +218 -218
  59. package/instintos/prompt-appendices.yaml +57 -57
  60. package/manifiestos/agent-output-schemas.json +57 -57
  61. package/manifiestos/modulos.json +1324 -1321
  62. package/manifiestos/skills-lock.json +1114 -1114
  63. package/package.json +2 -2
  64. package/plantillas/auditor-veto-template.md +105 -105
  65. package/plantillas/github-workflows/README.md +47 -47
  66. package/plantillas/github-workflows/release-please.yml +44 -44
  67. package/plantillas/github-workflows/swl-ci.yml +107 -107
  68. package/plantillas/github-workflows/swl-security.yml +51 -51
  69. package/plugin.json +353 -351
  70. package/reglas/analisis-previo-tareas-grandes.md +172 -172
  71. package/reglas/arreglar-al-detectar.md +147 -147
  72. package/reglas/fragmentos-compartidos.md +152 -152
  73. package/reglas/harness-claude-code.md +213 -213
  74. package/reglas/registro-componentes-nuevos.md +192 -0
  75. package/reglas/usar-context7.md +226 -226
  76. package/schemas/diary-entry.schema.json +80 -80
  77. package/scripts/actualizar.js +110 -1
  78. package/scripts/audit-tools/audit-history.js +330 -330
  79. package/scripts/audit-tools/bundle-tracker.js +290 -290
  80. package/scripts/audit-tools/canary-monitor.js +352 -352
  81. package/scripts/audit-tools/code-profiler.js +605 -605
  82. package/scripts/audit-tools/dep-doctor.js +320 -320
  83. package/scripts/audit-tools/env-validator.js +206 -206
  84. package/scripts/audit-tools/lib/fs-walk.js +48 -48
  85. package/scripts/audit-tools/lib/output.js +23 -23
  86. package/scripts/audit-tools/migration-checker.js +392 -392
  87. package/scripts/audit-tools/pentest-scanner.js +1436 -1436
  88. package/scripts/benchmark-memoria.js +167 -167
  89. package/scripts/configurar-branch-protection.js +418 -418
  90. package/scripts/derivar-feature-list.js +489 -489
  91. package/scripts/detectar-aprendizajes-duplicados.js +151 -151
  92. package/scripts/doctor.js +27 -0
  93. package/scripts/field-report.js +199 -199
  94. package/scripts/generar-checklists-consolidados.js +273 -273
  95. package/scripts/generar-inventario.js +420 -420
  96. package/scripts/generar-matriz-lenguajes.js +271 -271
  97. package/scripts/lib/artefactos-python.js +43 -43
  98. package/scripts/lib/benchmark-metrics.js +160 -160
  99. package/scripts/lib/budget-enforcer.js +252 -252
  100. package/scripts/lib/configurar-ci.js +380 -380
  101. package/scripts/lib/contadores-inventario.js +217 -217
  102. package/scripts/lib/detectar-stack-detallado.js +307 -307
  103. package/scripts/lib/diary-entry.js +234 -234
  104. package/scripts/lib/eval-metrics-store.js +218 -218
  105. package/scripts/lib/eval-quality.js +171 -171
  106. package/scripts/lib/eval-schemas.js +144 -144
  107. package/scripts/lib/eval-self-correct.js +106 -106
  108. package/scripts/lib/eval-validator.js +185 -185
  109. package/scripts/lib/expandir-targets.js +71 -71
  110. package/scripts/lib/jaccard-similarity.js +98 -98
  111. package/scripts/lib/longmemeval-runner.js +125 -125
  112. package/scripts/lib/mcp_config.py +127 -0
  113. package/scripts/lib/npm-version.js +261 -261
  114. package/scripts/lib/paquetes-conocidos.js +50 -50
  115. package/scripts/lib/prompt-builder.js +264 -264
  116. package/scripts/lib/rrf-fusion.js +175 -175
  117. package/scripts/lib/scoring-instintos.js +277 -277
  118. package/scripts/lib/semantic-search.js +252 -252
  119. package/scripts/lib/toml-merge.js +204 -204
  120. package/scripts/lib/transformadores/codex.js +375 -375
  121. package/scripts/lib/transformadores/cursor.js +359 -359
  122. package/scripts/limpiar-artefactos-python.js +131 -131
  123. package/scripts/mcp-orchestrator.py +8 -18
  124. package/scripts/mcp-pool-manager.py +12 -23
  125. package/scripts/mcp-server/README.md +170 -170
  126. package/scripts/mcp-server/auth.js +105 -105
  127. package/scripts/mcp-server/cache.js +106 -106
  128. package/scripts/mcp-server/telemetry.js +78 -78
  129. package/scripts/migrar-csv-a-array.js +168 -168
  130. package/scripts/migrar-fase-dominio.js +201 -201
  131. package/scripts/publicar.js +511 -511
  132. package/scripts/run-eval.js +141 -141
  133. 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 };