@mocoto/mahoraga 0.14.0 → 0.14.1

package/dist/analysts/detectors/detector-structure.js

@@ -1,9 +1,13 @@
-import { config, getAllExpectedDirNames, getDirVariantPattern, getNameConventions, isInsideSrc } from '../../core/config/index.js';
-import { messages } from '../../core/messages/index.js';
 import { detectarContextoProjeto } from '../../shared/index.js';
 import micromatch from 'micromatch';
+import { config, getAllExpectedDirNames, getDirVariantPattern, getNameConventions, isInsideSrc } from '../../core/config/index.js';
+import { messages } from '../../core/messages/index.js';
 import { grafoDependencias } from './detector-dependencies.js';
 export const sinaisDetectados = {};
+/**
+ * Analisa a estrutura do projeto e detecta padrões como monorepo, fullstack, mistura de src/packages, etc.
+ * Retorna ocorrências para cada sinal relevante encontrado.
+ */
 export const detectorEstrutura = {
     nome: 'detector-estrutura',
     global: true,
@@ -11,21 +15,27 @@ export const detectorEstrutura = {
         return true;
     },
     aplicar(_src, _relPath, _ast, _fullPath, contexto) {
-        if (!contexto)
+        if (!contexto) {
             return [];
+        }
         const caminhos = contexto.arquivos.map(arquivo => arquivo.relPath);
+        // Normaliza separadores para evitar falsos negativos em Windows (\\ vs /)
         const caminhosNorm = caminhos.map(caminho => caminho ? caminho.replace(/\\/g, '/') : '');
+        // Conjuntos para buscas O(1) e deduplicação implícita
         const setCaminhos = new Set(caminhos);
-        const arquivoPrincipal = caminhos?.find(caminho => /package\.json$/.test(caminho)) ?? caminhos?.[0] ?? '';
-        const conteudoPrincipal = contexto.arquivos?.find(arquivo => arquivo.relPath === arquivoPrincipal)?.content || '';
+        // Detecta contexto geral do projeto para análise inteligente
+        const arquivoPrincipal = caminhos.find(caminho => /package\.json$/.test(caminho)) ?? caminhos[0] ?? '';
+        const conteudoPrincipal = contexto.arquivos.find(arquivo => arquivo.relPath === arquivoPrincipal)?.content ?? '';
         const contextoProjeto = detectarContextoProjeto({
             arquivo: arquivoPrincipal,
             conteudo: conteudoPrincipal,
             relPath: arquivoPrincipal
         });
+        // Se é um projeto pequeno ou de teste, reduz a agressividade das verificações
         const isProjetoPequeno = caminhos.length < 20;
         const isProjetoTeste = contextoProjeto.isTest || caminhos.every(caminho => /(test|spec|mock)/i.test(caminho));
         const sinais = {
+            // Next.js: considerar tanto pages/ quanto app/
             temPages: caminhosNorm.some(caminho => caminho.includes('pages/') || caminho.includes('app/')),
             temApi: caminhosNorm.some(caminho => caminho.includes('api/')),
             temControllers: caminhosNorm.some(caminho => caminho.includes('controllers/')),
@@ -40,6 +50,7 @@ export const detectorEstrutura = {
         const ehMonorepo = !!sinais.temPackages;
         Object.assign(sinaisDetectados, sinais);
         const ocorrencias = [];
+        // Estrutura monorepo
         if (ehMonorepo) {
             ocorrencias.push({
                 tipo: 'estrutura-monorepo',
@@ -49,6 +60,7 @@ export const detectorEstrutura = {
                 relPath: '[raiz do projeto]',
                 linha: 0
             });
+            // Monorepo sem packages
             if (!caminhos.some(caminho => caminho.includes('packages/'))) {
                 ocorrencias.push({
                     tipo: 'estrutura-monorepo-incompleto',
@@ -60,6 +72,7 @@ export const detectorEstrutura = {
                 });
             }
         }
+        // Estrutura fullstack
         if (ehFullstack) {
             ocorrencias.push({
                 tipo: 'estrutura-fullstack',
@@ -71,37 +84,45 @@ export const detectorEstrutura = {
             });
         }
         else if (sinais.temPages && !sinais.temApi && !isProjetoPequeno) {
+            // Só reporta para projetos maiores que realmente deveriam ter API
             ocorrencias.push({
                 tipo: 'estrutura-incompleta',
                 nivel: 'info',
+                // Reduzido de 'aviso' para 'info'
                 mensagem: messages.DetectorEstruturaMensagens.pagesSemApi,
                 origem: 'detector-estrutura',
                 relPath: '[raiz do projeto]',
                 linha: 0
             });
         }
+        // Mistura de padrões: src/ e packages/ juntos - só reporta se não for projeto experimental
         if (sinais.temSrc && sinais.temPackages && !contextoProjeto.isTest) {
             ocorrencias.push({
                 tipo: 'estrutura-mista',
                 nivel: 'info',
+                // Reduzido de 'aviso' para 'info'
                 mensagem: messages.DetectorEstruturaMensagens.estruturaMista,
                 origem: 'detector-estrutura',
                 relPath: '[raiz do projeto]',
                 linha: 0
             });
         }
+        // Muitos arquivos na raiz (considera apenas nível imediato sem subpastas)
         const arquivosRaiz = caminhosNorm.filter(caminho => !caminho.includes('/') && caminho.trim() !== '');
-        const LIMITE_RAIZ = Number(config.ESTRUTURA_ARQUIVOS_RAIZ_MAX || 15);
+        const LIMITE_RAIZ = (config.ESTRUTURA_ARQUIVOS_RAIZ_MAX || 15); // Aumentado de 10 para 15
+        // Só reporta se não for projeto pequeno ou de teste
         if (arquivosRaiz.length > LIMITE_RAIZ && !isProjetoPequeno && !isProjetoTeste) {
             ocorrencias.push({
                 tipo: 'estrutura-suspeita',
                 nivel: 'info',
+                // Reduzido de 'aviso' para 'info'
                 mensagem: messages.DetectorEstruturaMensagens.muitosArquivosRaiz,
                 origem: 'detector-estrutura',
                 relPath: '[raiz do projeto]',
                 linha: 0
             });
         }
+        // Sinais de backend
         if (sinais.temControllers || sinais.temPrisma || sinais.temApi) {
             ocorrencias.push({
                 tipo: 'estrutura-backend',
@@ -112,6 +133,7 @@ export const detectorEstrutura = {
                 linha: 0
             });
         }
+        // Sinais de frontend
         if (sinais.temComponents || sinais.temPages) {
             ocorrencias.push({
                 tipo: 'estrutura-frontend',
@@ -122,6 +144,7 @@ export const detectorEstrutura = {
                 linha: 0
             });
         }
+        // Arquivos de configuração conhecidos - agrupa em uma única ocorrência para reduzir ruído
         const arquivosConfiguracao = ['package.json', 'tsconfig.json', 'turbo.json', 'pnpm-workspace.yaml'];
         const detectados = arquivosConfiguracao.filter(cfg => setCaminhos.has(cfg));
         if (detectados.length > 0) {
@@ -134,16 +157,19 @@ export const detectorEstrutura = {
                 linha: 1
             });
         }
+        // Múltiplos entrypoints
+        // Lista de entrypoints potencialmente grande em repositórios com dependências; filtra ignorados
         const entrypointsAll = caminhosNorm.filter(caminho => /(^|[\\/])(cli|index|main)\.(ts|js)$/.test(caminho));
-        const ignoredDyn = config.INCLUDE_EXCLUDE_RULES && config.INCLUDE_EXCLUDE_RULES.globalExcludeGlob || [];
+        const ignoredDyn = config.INCLUDE_EXCLUDE_RULES.globalExcludeGlob ?? [];
         const entrypoints = entrypointsAll.filter(caminho => !micromatch.isMatch(caminho, ignoredDyn || [])).sort();
         if (entrypoints.length > 1) {
             const MAX_LIST = 20;
             if (entrypoints.length > MAX_LIST) {
+                // Muitos entrypoints: agrega por diretório para reduzir ruído sem perder o sinal
                 const dirCounts = new Map();
                 for (const caminho of entrypoints) {
                     const dir = caminho.includes('/') ? caminho.slice(0, caminho.lastIndexOf('/')) : '.';
-                    dirCounts.set(dir, (dirCounts.get(dir) || 0) + 1);
+                    dirCounts.set(dir, (dirCounts.get(dir) ?? 0) + 1);
                 }
                 const ordenado = [...dirCounts.entries()].sort((arquivo, base) => base[1] - arquivo[1] || (arquivo[0] < base[0] ? -1 : arquivo[0] > base[0] ? 1 : 0));
                 const MAX_DIRS = 10;
@@ -166,6 +192,7 @@ export const detectorEstrutura = {
                 });
             }
             else {
+                // Quantidade moderada: lista direta mantendo ordenação estável
                 const preview = entrypoints.slice(0, MAX_LIST);
                 const resto = entrypoints.length - preview.length;
                 ocorrencias.push({
@@ -178,17 +205,22 @@ export const detectorEstrutura = {
                 });
             }
         }
+        // Ausência de src/ em projetos grandes (verifica realmente se diretório src existe fisicamente)
+        // Só reporta para projetos que realmente deveriam ter src/ baseado no contexto
         const deveriaTerSrc = !contextoProjeto.isTest && !contextoProjeto.isConfiguracao && (contextoProjeto.isLibrary || contextoProjeto.frameworks.length > 0);
         if (!sinais.temSrc && caminhosNorm.length > 50 && deveriaTerSrc) {
+            // Aumentado de 30 para 50
             ocorrencias.push({
                 tipo: 'estrutura-sem-src',
                 nivel: 'info',
+                // Reduzido de 'aviso' para 'info'
                 mensagem: messages.DetectorEstruturaMensagens.projetoGrandeSemSrc,
                 origem: 'detector-estrutura',
                 relPath: '[raiz do projeto]',
                 linha: 0
             });
         }
+        // Verifica convenções de nomes de diretórios
         const expectedDirs = getAllExpectedDirNames();
         if (expectedDirs.length > 0) {
             const topLevelDirs = new Set(caminhosNorm.filter(caminho => caminho.includes('/') && !caminho.startsWith('node_modules/') && !caminho.startsWith('dist/')).map(caminho => caminho.split('/')[0]));
@@ -215,4 +247,3 @@ export const detectorEstrutura = {
         return Array.isArray(ocorrencias) ? ocorrencias : [];
     }
 };
-//# sourceMappingURL=detector-structure.js.map

package/dist/analysts/detectors/index.js

@@ -1,3 +1,7 @@
+// SPDX-License-Identifier: MIT
+/**
+ * @fileoverview Detectores core do projeto (genéricos, não ligados a tech específica)
+ */
 export * from './detector-architecture.js';
 export * from './detector-bugs-ml.js';
 export * from './detector-code-fragile.js';
@@ -9,4 +13,3 @@ export * from './detector-phantoms.js';
 export * from './detector-recommendation-architecture.js';
 export * from './detector-security.js';
 export * from './detector-structure.js';
-//# sourceMappingURL=index.js.map

package/dist/analysts/github-actions/analysts/analyst-github-actions-security-hardening.js

@@ -1,5 +1,8 @@
 import { YAMLMap } from 'yaml';
 import { registrarDetectorGithubActions } from './analyst-github-actions.js';
+/**
+ * Plugin de exemplo para demonstrar o sistema de extensibilidade v0.5.0
+ */
 registrarDetectorGithubActions({
     nome: 'security-hardening',
     descricao: 'Regras adicionais de segurança para workflows',
@@ -7,9 +10,11 @@ registrarDetectorGithubActions({
     testar: (workflow) => {
         const problemas = [];
         const wf = workflow;
-        if (!wf || !wf.jobs)
+        if (!wf?.jobs) {
             return problemas;
+        }
         for (const [jobId, job] of Object.entries(wf.jobs)) {
+            // Regra: Todo job deve ter permissões explícitas
             if (!job.permissions && !wf.permissions) {
                 problemas.push({
                     tipo: 'security-hardening',
@@ -24,8 +29,9 @@ registrarDetectorGithubActions({
     },
     corrigir: (doc) => {
         const jobs = doc.get('jobs');
-        if (!jobs || !(jobs instanceof YAMLMap))
+        if (!jobs || !(jobs instanceof YAMLMap)) {
             return;
+        }
         jobs.items.forEach((pair) => {
             const job = pair.value;
             if (job instanceof YAMLMap && !job.has('permissions') && !doc.has('permissions')) {
@@ -34,4 +40,3 @@ registrarDetectorGithubActions({
         });
     }
 });
-//# sourceMappingURL=analyst-github-actions-security-hardening.js.map

package/dist/analysts/github-actions/analysts/analyst-github-actions.js

@@ -1,5 +1,6 @@
-import { messages } from '../../../core/messages/index.js';
+// SPDX-License-Identifier: MIT
 import { parseDocument } from 'yaml';
+import { messages } from '../../../core/messages/index.js';
 import { detectorDependenciasVulneraveis } from '../detectors/detector-dependencies-vulnerable.js';
 import { detectarProblemasAcessibilidade } from '../detectors/detector-workflow-accessibility.js';
 import { detectarProblemasCompliance } from '../detectors/detector-workflow-compliance.js';
@@ -8,6 +9,7 @@ import { detectorWorkflowSecurity } from '../detectors/detector-workflow-securit
 import { detectarJobsPassivos, detectorWorkflowEstrutura } from '../detectors/detector-workflow-structure.js';
 import { detectorWorkflowTriggerInseguro } from '../detectors/detector-workflow-trigger-unsafe.js';
 import { isOrgVerificada } from '../detectors/org-verified.js';
+/** Flag global temporária (resetada a cada execução) para indicar se o workflow precisa de histórico completo do git */
 let _workflowNeedsFullHistory = false;
 const detectoresRegistrados = [];
 export function registrarDetectorGithubActions(detector) {
@@ -26,8 +28,8 @@ export const analistaGithubActions = {
         try {
             const doc = parseDocument(conteudo);
             const workflow = doc.toJS();
-            const problemas = await executarDetectoresNativos(workflow, conteudo);
-            const caminhos = contexto?.arquivos.map(arquivo => arquivo.relPath) || [];
+            const problemas = executarDetectoresNativos(workflow, conteudo);
+            const caminhos = contexto?.arquivos.map(arquivo => arquivo.relPath) ?? [];
             const plugResults = await Promise.all(detectoresRegistrados.map(diretorio => Promise.resolve(diretorio.testar(workflow, {
                 conteudo,
                 caminhos
@@ -44,14 +46,15 @@ export const analistaGithubActions = {
                     tipo: `GITHUB_ACTIONS_${caminho.tipo.toUpperCase().replace(/-/g, '_')}`,
                     mensagem: caminho.descricao,
                     relPath,
-                    linha: caminho.linha || 1,
+                    linha: caminho.linha ?? 1,
                     coluna: 1,
-                    sugestao: caminho.sugestao || '',
+                    sugestao: caminho.sugestao ?? '',
                     nivel: caminho.severidade === 'critica' || caminho.severidade === 'alta' ? 'erro' : caminho.severidade === 'media' ? 'aviso' : 'info'
                 });
             }
         }
         catch {
+            // Basic regex fallback if YAML fails
         }
         return ocorrencias;
     }
@@ -61,11 +64,13 @@ export const analistaGithubActionsGlobal = {
     categoria: 'workflows',
     descricao: 'Governança Global do GitHub',
     test: () => false,
-    async aplicar(_c, _p, _a, _f, ctx) {
-        if (!ctx)
+    aplicar(_c, _p, _a, _f, ctx) {
+        if (!ctx) {
             return [];
+        }
         const ores = [];
         const paths = ctx.arquivos.map(arquivo => arquivo.relPath);
+        // CODEOWNERS
         if (!paths.some(caminho => /CODEOWNERS/i.test(caminho))) {
             ores.push({
                 tipo: 'GITHUB_ACTIONS_CODEOWNERS_MISSING',
@@ -76,8 +81,9 @@ export const analistaGithubActionsGlobal = {
                 nivel: 'info'
             });
         }
-        const readme = ctx.arquivos?.find(arquivo => /README\.md/i.test(arquivo.relPath ?? ''));
-        const content = readme ? readme.content || '' : '';
+        // README
+        const readme = ctx.arquivos.find(arquivo => /README\.md/i.test(arquivo.relPath ?? ''));
+        const content = readme ? readme.content ?? '' : '';
         if (readme && !/Code of Conduct|Código de Conduta/i.test(content)) {
             ores.push({
                 tipo: 'GITHUB_ACTIONS_DOC_GOVERNANCE_MISSING',
@@ -88,6 +94,7 @@ export const analistaGithubActionsGlobal = {
                 nivel: 'info'
             });
         }
+        // Stale Bot
         if (!paths.some(caminho => /stale/i.test(caminho))) {
             ores.push({
                 tipo: 'GITHUB_ACTIONS_STALE_BOT_MISSING',
@@ -98,6 +105,7 @@ export const analistaGithubActionsGlobal = {
                 nivel: 'info'
             });
         }
+        // Release
         if (!paths.some(caminho => /release|deploy/i.test(caminho))) {
             ores.push({
                 tipo: 'GITHUB_ACTIONS_RELEASE_AUTOMATION_MISSING',
@@ -108,6 +116,7 @@ export const analistaGithubActionsGlobal = {
                 nivel: 'info'
             });
         }
+        // Issue Templates
         if (!paths.some(caminho => /\.github\/ISSUE_TEMPLATE/i.test(caminho))) {
             ores.push({
                 tipo: 'GITHUB_ACTIONS_ISSUE_TEMPLATES_MISSING',
@@ -118,6 +127,7 @@ export const analistaGithubActionsGlobal = {
                 nivel: 'info'
             });
         }
+        // PR Template
         if (!paths.some(caminho => /pull_request_template/i.test(caminho))) {
             ores.push({
                 tipo: 'GITHUB_ACTIONS_PR_TEMPLATE_MISSING',
@@ -128,6 +138,7 @@ export const analistaGithubActionsGlobal = {
                 nivel: 'info'
             });
         }
+        // Semantic Versioning & Changelog
         const hasSemanticRelease = paths.some(caminho => /semantic-release|semrel/i.test(caminho));
         const hasChangelog = paths.some(caminho => /changelog/i.test(caminho));
         if (!hasSemanticRelease) {
@@ -150,6 +161,7 @@ export const analistaGithubActionsGlobal = {
                 nivel: 'info'
             });
         }
+        // License Headers
         const sourceFiles = ctx.arquivos.filter(arquivo => /\.(ts|js)$/.test(arquivo.relPath));
         const missingLicense = sourceFiles.filter(arquivo => !/SPDX-License-Identifier/i.test(arquivo.content || ''));
         if (missingLicense.length > sourceFiles.length / 2) {
@@ -165,33 +177,43 @@ export const analistaGithubActionsGlobal = {
         return ores;
     }
 };
+/**
+ * Verifica se o workflow contém comandos git que exigem histórico completo.
+ */
 function workflowRequerHistoricoCompleto(wf, raw) {
+    // Verificar no raw por comandos git conhecidos que precisam de histórico completo
     const gitCommandsQuePrecisamDeHistorico = /\b(git\s+(rev-parse|tag|describe|log|shortlog|diff|bisect|blame)|git\s+checkout\s+-b|git\s+fetch\s+--unshallow|git\s+clone\s+--depth)/gi;
     if (gitCommandsQuePrecisamDeHistorico.test(raw)) {
         return true;
     }
-    const jobs = wf?.jobs ? Object.values(wf.jobs) : [];
-    const allSteps = jobs.flatMap(job => job?.steps || []);
+    // Verificar nas ações steps por comandos que indicam necessidade de histórico
+    const jobs = wf.jobs ? Object.values(wf.jobs) : [];
+    const allSteps = jobs.flatMap(job => job.steps ?? []);
     for (const step of allSteps) {
-        if (step?.run && gitCommandsQuePrecisamDeHistorico.test(step.run)) {
+        if (step.run && gitCommandsQuePrecisamDeHistorico.test(step.run)) {
             return true;
         }
     }
     return false;
 }
-async function executarDetectoresNativos(wf, raw) {
+function executarDetectoresNativos(wf, raw) {
     const probs = [];
-    if (!wf)
+    if (!wf) {
         return probs;
+    }
     _workflowNeedsFullHistory = workflowRequerHistoricoCompleto(wf, raw);
+    // Handle various snippet styles for tests
     const candidates = Array.isArray(wf) ? wf : [wf];
     for (const item of candidates) {
         if (typeof item === 'object' && item !== null) {
             analisarEstrutura(item, probs);
         }
     }
+    // Compliance checks
     probs.push(...detectarProblemasCompliance(wf, raw));
+    // Accessibility checks
     probs.push(...detectarProblemasAcessibilidade(wf));
+    // Passive jobs check
     if (typeof wf === 'object' && 'jobs' in wf) {
         const wfJobs = wf.jobs;
         if (wfJobs) {
@@ -206,6 +228,7 @@ async function executarDetectoresNativos(wf, raw) {
             }
         }
     }
+    // Final check for raw string patterns (like sudo)
     if (/sudo\s+/.test(raw)) {
         probs.push({
             tipo: 'uso-sudo',
@@ -223,6 +246,7 @@ async function executarDetectoresNativos(wf, raw) {
     return probs;
 }
 function analisarEstrutura(wf, probs) {
+    // Workflow sem nome
     if (!wf.name) {
         probs.push({
             tipo: 'workflow-sem-nome',
@@ -230,6 +254,7 @@ function analisarEstrutura(wf, probs) {
             severidade: 'baixa'
         });
     }
+    // Container (Root level for snippets)
     if (wf.container && typeof wf.container === 'string') {
         probs.push({
             tipo: 'container-sem-user',
@@ -237,6 +262,7 @@ function analisarEstrutura(wf, probs) {
             severidade: 'media'
         });
     }
+    // Strategy
     if (wf.strategy?.matrix && wf.strategy['fail-fast'] === undefined) {
         probs.push({
             tipo: 'matrix-sem-fail-fast',
@@ -244,6 +270,7 @@ function analisarEstrutura(wf, probs) {
             severidade: 'baixa'
         });
     }
+    // Detect Secrets in generic maps (for environment snippets in tests)
     for (const [chave, valor] of Object.entries(wf)) {
         if (/(KEY|TOKEN|SECRET|PASSWORD)/i.test(chave) && valor && typeof valor !== 'object' && !String(valor).includes('${{')) {
             probs.push({
@@ -253,24 +280,26 @@ function analisarEstrutura(wf, probs) {
             });
         }
     }
+    // Jobs
     if (wf.jobs) {
         const jobEntries = Object.entries(wf.jobs);
         for (const [id, job] of jobEntries) {
-            if (job?.strategy?.matrix && job.strategy['fail-fast'] === undefined) {
+            if (job.strategy?.matrix && job.strategy['fail-fast'] === undefined) {
                 probs.push({
                     tipo: 'matrix-sem-fail-fast',
                     descricao: `Job ${id} sem fail-fast`,
                     severidade: 'baixa'
                 });
             }
-            if (job?.container && typeof job.container === 'string') {
+            if (job.container && typeof job.container === 'string') {
                 probs.push({
                     tipo: 'container-sem-user',
                     descricao: 'Container rodando como root',
                     severidade: 'media'
                 });
             }
-            if (/^(job\d+|step\d+|build\d*|test\d*|j\d+)$/i.test(id) && !job?.name) {
+            // --- v0.4.5: Job sem nome descritivo ---
+            if (/^(job\d+|step\d+|build\d*|test\d*|j\d+)$/i.test(id) && !job.name) {
                 probs.push({
                     tipo: 'job-sem-nome',
                     descricao: `Job '${id}' usa ID genérico sem campo 'name'`,
@@ -278,10 +307,13 @@ function analisarEstrutura(wf, probs) {
                     sugestao: 'Usar nomes semânticos como build-backend, test-integration'
                 });
             }
-            if (job?.steps) {
-                for (const string of job.steps)
+            if (job.steps) {
+                for (const string of job.steps) {
                     analisarStep(string, probs);
+                }
+                // --- v0.4.5: Steps sem nome ---
                 analisarStepsSemNome(job.steps, probs);
+                // --- v0.7.0: Jobs muito longos ---
                 if (job.steps.length > 15) {
                     probs.push({
                         tipo: 'job-longo',
@@ -292,21 +324,30 @@ function analisarEstrutura(wf, probs) {
                 }
             }
         }
+        // --- v0.4.5: Build sem parallelismo ---
         analisarParalelismo(jobEntries, probs);
+        // --- v0.4.5: Download desnecessário de artifacts ---
         analisarDownloadArtifacts(jobEntries, probs);
+        // --- v0.4.5: Múltiplas instalações de dependências ---
         analisarMultiplasInstalacoesDeps(jobEntries, probs);
     }
+    // Se forem passos diretos na raiz (snippet)
     if (wf.steps && Array.isArray(wf.steps)) {
-        for (const string of wf.steps)
+        for (const string of wf.steps) {
             analisarStep(string, probs);
+        }
         analisarStepsSemNome(wf.steps, probs);
     }
-    if (wf.uses || wf.run)
+    // Direct steps (if snippet is a list)
+    if (wf.uses || wf.run) {
         analisarStep(wf, probs);
+    }
 }
 function analisarStep(string, probs) {
-    if (!string)
+    if (!string) {
         return;
+    }
+    // Pinning - only for non-verified orgs
     if (string.uses && /@v\d+/.test(string.uses)) {
         const actionRef = string.uses.split('@')[0];
         if (!isOrgVerificada(actionRef)) {
@@ -317,6 +358,7 @@ function analisarStep(string, probs) {
             });
         }
     }
+    // Upload
     if (string.uses?.includes('upload-artifact') && string.with?.path?.includes('.env')) {
         probs.push({
             tipo: 'upload-sensivel',
@@ -324,6 +366,7 @@ function analisarStep(string, probs) {
             severidade: 'critica'
         });
     }
+    // Env
     if (string.env) {
         for (const [chave, valor] of Object.entries(string.env)) {
             if (/(KEY|TOKEN|SECRET|PASSWORD)/i.test(chave) && valor && typeof valor !== 'object' && !String(valor).includes('${{')) {
@@ -335,6 +378,7 @@ function analisarStep(string, probs) {
             }
         }
     }
+    // Injection
     if (string.run && /\$\{\{\s*github\.event\./.test(string.run)) {
         probs.push({
             tipo: 'script-injection',
@@ -342,6 +386,7 @@ function analisarStep(string, probs) {
             severidade: 'alta'
         });
     }
+    // --- v0.4.5: Docker build sem layer caching ---
     if (string.uses && /docker\/build-push-action/.test(string.uses)) {
         const hasCacheFrom = string.with?.['cache-from'] !== undefined;
         const hasCacheTo = string.with?.['cache-to'] !== undefined;
@@ -354,9 +399,11 @@ function analisarStep(string, probs) {
             });
         }
     }
+    // --- v0.4.5: Checkout com fetch-depth: 0 desnecessário ---
     if (string.uses && /actions\/checkout/.test(string.uses) && string.with) {
         const fd = string.with['fetch-depth'];
         if (fd === 0 || fd === '0') {
+            // Não emitir se já foi identificado que o workflow precisa de histórico completo
             if (!_workflowNeedsFullHistory) {
                 probs.push({
                     tipo: 'fetch-depth-desnecessario',
@@ -398,11 +445,17 @@ const ACTIONS_AUTO_EXPLICATIVAS = [
     'softprops/action-gh-release',
     'cypress-io/github-action',
 ];
+/**
+ * v0.4.5: Detecta steps sem campo 'name'
+ * Não reporta para actions bem conhecidas e auto-explicativas
+ */
 function analisarStepsSemNome(steps, probs) {
     for (const string of steps) {
-        if (!string || string.name)
+        if (!string || string.name) {
             continue;
+        }
         if (string.uses || string.run) {
+            // Pular actions auto-explicativas (não precisam de name)
             if (string.uses && ACTIONS_AUTO_EXPLICATIVAS.some(action => string.uses?.startsWith(action))) {
                 continue;
             }
@@ -416,19 +469,29 @@ function analisarStepsSemNome(steps, probs) {
         }
     }
 }
+/**
+ * v0.4.5: Detecta jobs que poderiam rodar em paralelo mas têm 'needs' desnecessário
+ * Heurística: se todos os jobs têm 'needs' formando uma cadeia linear, sugere paralelismo
+ * Exceção: workflows de release/deploy são intrinsecamente sequenciais
+ */
 function analisarParalelismo(jobEntries, probs) {
-    if (jobEntries.length < 3)
+    if (jobEntries.length < 3) {
         return;
-    const workflowName = jobEntries[0]?.[1]?.name?.toLowerCase() || '';
-    const jobNames = jobEntries.map(([, j]) => j?.name?.toLowerCase() || '').join(' ');
+    }
+    // Check if this is a release/deploy workflow (sequential by nature)
+    const workflowName = jobEntries[0]?.[1]?.name?.toLowerCase() ?? '';
+    const jobNames = jobEntries.map(([, j]) => j.name?.toLowerCase() ?? '').join(' ');
     const isReleaseWorkflow = /release|deploy|publish|ship/i.test(workflowName) || /release|deploy|publish|ship/i.test(jobNames);
-    if (isReleaseWorkflow)
+    if (isReleaseWorkflow) {
         return;
+    }
     let totalWithNeeds = 0;
     for (const [, job] of jobEntries) {
-        if (job?.needs)
+        if (job.needs) {
             totalWithNeeds++;
+        }
     }
+    // Se todos os jobs (exceto o primeiro) dependem do anterior em cadeia linear
     if (totalWithNeeds === jobEntries.length - 1) {
         probs.push({
             tipo: 'build-sem-parallelismo',
@@ -438,14 +501,19 @@ function analisarParalelismo(jobEntries, probs) {
         });
     }
 }
+/**
+ * v0.4.5: Detecta múltiplos download-artifact em jobs diferentes
+ */
 function analisarDownloadArtifacts(jobEntries, probs) {
     let downloadCount = 0;
     for (const [, job] of jobEntries) {
-        if (!job?.steps)
+        if (!job.steps) {
             continue;
+        }
         for (const string of job.steps) {
-            if (string.uses && /download-artifact/.test(string.uses))
+            if (string.uses && /download-artifact/.test(string.uses)) {
                 downloadCount++;
+            }
         }
     }
     if (downloadCount >= 3) {
@@ -457,13 +525,17 @@ function analisarDownloadArtifacts(jobEntries, probs) {
         });
     }
 }
+/**
+ * v0.4.5: Detecta npm ci/npm install em múltiplos jobs sem cache compartilhado
+ */
 function analisarMultiplasInstalacoesDeps(jobEntries, probs) {
     let installCount = 0;
     let hasCacheStep = false;
     let hasSetupNodeWithCache = false;
     for (const [, job] of jobEntries) {
-        if (!job?.steps)
+        if (!job.steps) {
             continue;
+        }
         let jobHasInstall = false;
         for (const string of job.steps) {
             if (string.run && /\b(npm\s+(ci|install)|yarn\s+install|pnpm\s+install)\b/.test(string.run)) {
@@ -476,8 +548,9 @@ function analisarMultiplasInstalacoesDeps(jobEntries, probs) {
                 hasSetupNodeWithCache = true;
             }
         }
-        if (jobHasInstall)
+        if (jobHasInstall) {
             installCount++;
+        }
     }
     if (installCount >= 2 && !hasCacheStep && !hasSetupNodeWithCache) {
         probs.push({
@@ -488,4 +561,3 @@ function analisarMultiplasInstalacoesDeps(jobEntries, probs) {
         });
     }
 }
-//# sourceMappingURL=analyst-github-actions.js.map