โ ๏ธ High complexity (${cx})
`; fixes.push({ title: 'Split by responsibility', steps: [ '1. List the route groups (admin, forum, auth, etc.)', '2. Create lib/api/{group}.js for each group', '3. Move routes + their helpers into each module', '4. Export a router from each, mount in server.js', '5. Verify: all routes still respond, no 404s', ], }); } else if (cx > 200) { html += `โก Moderate complexity (${cx})
`; fixes.push({ title: 'Extract helpers', steps: [ '1. Identify repeated logic blocks', '2. Move to named functions at top of file', '3. Reduce nesting: early returns instead of deep if/else', ], }); } else { html += `โ Healthy complexity (${cx})
`; } if (deps.length > 15) { html += `โก ${deps.length} dependencies โ high coupling
`; fixes.push({ title: 'Reduce coupling', steps: [ '1. List all ' + deps.length + ' imports', '2. Group by purpose (db, views, utils, etc.)', '3. Create facade modules that re-export related imports', '4. File imports 3-5 facades instead of ' + deps.length + ' individuals', ], }); } const routeCount = routes.length; if (routeCount > 50) { html += `โ ๏ธ ${routeCount} routes in one file
`; fixes.push({ title: 'Thin route handlers', steps: [ '1. Each route handler should be <20 lines', '2. Move business logic to lib/services/{domain}.js', '3. Routes become: validate input โ call service โ send response', '4. Test: each service function independently', ], }); } if (!hasTest && cx > 200) { fixes.push({ title: 'Add test coverage', steps: [ '1. Create test file mirroring source structure', '2. Test happy path first (does it work at all?)', '3. Test error paths (what if input is bad?)', '4. Test edge cases (empty, null, oversized)', '5. Target: 80% of branches covered', ], }); } // Render fix guides if (fixes.length > 0) { html += '${fix.title}
`; html += fix.steps.map(s => `${escapeHtml(s)}
` ).join(''); html += `