@mocoto/mahoraga 0.14.7 → 0.14.9

package/dist/analysts/python/corrections/correction-python.js

@@ -1,13 +1,3 @@
-// @mahoraga-disable ml-deep-nesting
-// SPDX-License-Identifier: MIT
-/**
- * [CHAVE-INGLESA] Correções automáticas para Python
- *
- * Fornece correções sugeridas para problemas comuns em Python:
- * - Bare except → except Exception
- * - Mutable defaults → None pattern
- * - print → logging
- */
 import { messages } from '../../../core/messages/index.js';
 import { maskPythonComments, splitLines } from '../../../shared/helpers/index.js';
 import { criarOcorrencia } from '../../../types/index.js';

package/dist/analysts/python/corrections/index.js

@@ -1,2 +1 @@
-// SPDX-License-Identifier: MIT
 export * from './correction-python.js';

package/dist/analysts/python/detectors/detector-python-best-practices.js

@@ -1,10 +1,3 @@
-// SPDX-License-Identifier: MIT
-/**
- * [BUSCA] Detector de boas práticas Python
- *
- * Delega para o analista principal (analyst-python) e retorna
- * as ocorrências usando mensagens centralizadas.
- */
 import { collectPythonIssues } from '../analysts/analyst-python.js';
 export const detectorPythonBestPractices = {
     nome: 'detector-python-best-practices',

package/dist/analysts/python/detectors/index.js

@@ -1,2 +1 @@
-// SPDX-License-Identifier: MIT
 export * from './detector-python-best-practices.js';

package/dist/analysts/python/index.js

@@ -1,9 +1,3 @@
-// SPDX-License-Identifier: MIT
-/**
- * [BUSCA] Analistas de Python
- *
- * Heurísticas leves para Python (boas práticas e segurança).
- */
 export * from './analysts/analyst-python.js';
 export * from './corrections/index.js';
 export * from './detectors/index.js';

package/dist/analysts/python/scorers/index.js

@@ -1,2 +1 @@
-// SPDX-License-Identifier: MIT
 export * from './scorer-python.js';

package/dist/analysts/python/scorers/scorer-python.js

@@ -1,14 +1,3 @@
-// @mahoraga-disable ml-deep-nesting
-// SPDX-License-Identifier: MIT
-/**
- * [GRAFICO] Pontuador de qualidade Python
- *
- * Avalia a qualidade do código Python baseado em:
- * - Type hints coverage
- * - Security issues
- * - Error handling
- * - Code organization
- */
 import { maskPythonComments, splitLines } from '../../../shared/helpers/index.js';
 export function pontuarPython(src, _relPath) {
     let score = 0;
@@ -18,7 +7,6 @@ export function pontuarPython(src, _relPath) {
     const scan = maskPythonComments(src);
     const _linhas = splitLines(scan);
     const srcCompleto = scan;
-    // Type hints (25 pontos)
     const funcs = (srcCompleto.match(/def\s+\w+/g) ?? []).length;
     const funcsWithHints = (srcCompleto.match(/def\s+\w+\s*\([^)]*\)\s*->/g) ?? []).length;
     if (funcs > 0) {
@@ -37,7 +25,6 @@ export function pontuarPython(src, _relPath) {
     else {
         score += 25;
     }
-    // Sem eval/exec (20 pontos)
     if (!/\b(eval|exec)\s*\(/.test(srcCompleto)) {
         score += 20;
         strengths.push('Sem uso de eval/exec');
@@ -45,7 +32,6 @@ export function pontuarPython(src, _relPath) {
     else {
         issues.push('eval/exec detectado - risco de segurança');
     }
-    // Error handling (20 pontos)
     const bareExcept = (srcCompleto.match(/^\s*except\s*:/gm) ?? []).length;
     if (bareExcept === 0) {
         score += 20;
@@ -55,7 +41,6 @@ export function pontuarPython(src, _relPath) {
         score += Math.max(0, 20 - bareExcept * 5);
         issues.push(`${bareExcept} bare except(s) detectado(s)`);
     }
-    // Sem mutable defaults (15 pontos)
     const mutableDefaults = (srcCompleto.match(/def\s+\w+\s*\([^)]*=\s*(?:\[|\{)/g) ?? []).length;
     if (mutableDefaults === 0) {
         score += 15;
@@ -64,7 +49,6 @@ export function pontuarPython(src, _relPath) {
     else {
         issues.push(`${mutableDefaults} função(ões) com default mutável`);
     }
-    // Logging ao invés de print (10 pontos)
     const printCount = (srcCompleto.match(/^\s*print\s*\(/gm) ?? []).length;
     const loggingCount = (srcCompleto.match(/logging\.\w+\(/g) ?? []).length;
     if (printCount === 0 || loggingCount > printCount) {
@@ -74,7 +58,6 @@ export function pontuarPython(src, _relPath) {
     else {
         issues.push(`${printCount} print(s) - considere usar logging`);
     }
-    // Sem subprocess shell=True (10 pontos)
     if (!/subprocess.*shell\s*=\s*True/.test(srcCompleto)) {
         score += 10;
         strengths.push('Sem subprocess com shell=True');

package/dist/analysts/react/analysts/analyst-react-hooks.js

@@ -4,14 +4,13 @@ import * as t from '@babel/types';
 import { messages } from '../../../core/messages/index.js';
 import { createLineLookup } from '../../../shared/helpers/index.js';
 import { criarAnalista, criarOcorrencia } from '../../../types/index.js';
-const disableEnv = process.env.SUKUNA_DISABLE_PLUGIN_REACT_HOOKS === '1';
+const disableEnv = process.env.MAHORAGA_DISABLE_PLUGIN_REACT_HOOKS === '1';
 const REACT_18_HOOKS = [
     'useState', 'useEffect', 'useMemo', 'useCallback', 'useRef',
     'useContext', 'useReducer', 'useLayoutEffect', 'useImperativeHandle',
     'useDebugValue', 'useDeferredValue', 'useTransition',
     'useSyncExternalStore', 'useInsertionEffect', 'useId',
 ];
-// React 19+ built-in hooks
 const REACT_19_HOOKS = [
     'use', 'useOptimistic', 'useActionState', 'useFormStatus',
 ];
@@ -109,25 +108,17 @@ function extractHookCallContent(src, startIndex) {
 function collectHookIssues(src, relPath, opts = {}) {
     const ocorrencias = [];
     const lineOf = createLineLookup(src).lineAt;
-    // As verificações estruturais (useEffect/useMemo/useCallback sem deps e hook
-    // em condicional) são imprecisas quando feitas por regex/linha e geram falsos
-    // positivos. Quando a análise por AST (`parseHooksWithBabel`) está disponível,
-    // ela é a fonte autoritativa e estas verificações devem ser ignoradas.
     const skipStructural = opts.skipStructural === true;
     if (!skipStructural) {
-        // Check useEffect/useLayoutEffect for deps
         const effectMatches = [...src.matchAll(/use(Layout)?Effect\s*\(/g)];
         effectMatches.forEach(m => {
             const hookInicio = m.index;
             const fullCall = extractHookCallContent(src, hookInicio + m[0].length - 1);
-            // More robust deps detection: find the last comma followed by an array or identifier before the closing paren
             const hasDepsArg = (() => {
-                // Find the last ')' (closing paren of the full call)
                 const lastClosing = fullCall.lastIndexOf(')');
                 if (lastClosing === -1)
                     return false;
                 const beforeClosing = fullCall.slice(0, lastClosing);
-                // Find the last '[' or identifier before the closing paren
                 const arrayMatch = beforeClosing.match(/,\s*(\[[\s\S]*?\])\s*$/);
                 if (arrayMatch)
                     return true;
@@ -153,7 +144,6 @@ function collectHookIssues(src, relPath, opts = {}) {
                 }
             }
         });
-        // useMemo/useCallback deps check
         const memoMatches = [...src.matchAll(/use(Memo|Callback)\s*\(/g)];
         memoMatches.forEach(m => {
             const hookInicio = m.index;
@@ -165,43 +155,27 @@ function collectHookIssues(src, relPath, opts = {}) {
                 ocorrencias.push(warn(messages.ReactHooksMensagens.memoCallbackNoDeps, relPath, line));
             }
         });
-        // Conditional hooks — only flag when a hook call is directly inside a conditional block,
-        // not when the conditional is inside a hook callback (e.g. cleanup inside useEffect).
-        // The regex matches: if/for/while (...) { ... useXxx ... } but limits the match scope
-        // to avoid matching conditionals that appear inside hook callbacks.
         const conditionalHooks = [...src.matchAll(/(?:^|(?<=[\n;]))\s*(if|for|while)\s*\([^)]*\)\s*\{[\s\S]{0,160}?use[A-Z][A-Za-z0-9_]*/g)];
         conditionalHooks.forEach(m => {
             const line = lineOf(m.index || 0);
-            // Heuristic: skip if the 'if' keyword is preceded by a closing paren or brace at the same indentation,
-            // which suggests it's inside a callback body rather than at the component top level.
             const pre = src.slice(Math.max(0, (m.index ?? 0) - 80), m.index ?? 0);
             if (/=>\s*$|\)\s*=>\s*$|\)\s*\{[\s\S]{0,40}$/.test(pre))
                 return;
             ocorrencias.push(warn(messages.ReactHooksMensagens.hookInConditional, relPath, line));
         });
-    } // fim de if (!skipStructural)
-    // React 19: useOptimistic detected
+    }
     for (const match of src.matchAll(/\buseOptimistic\s*\(/g)) {
         const line = lineOf(match.index);
         ocorrencias.push(warn('useOptimistic (React 19) detected', relPath, line, messages.SeverityNiveis.info));
     }
-    // React 19: useActionState detected
     for (const match of src.matchAll(/\buseActionState\s*\(/g)) {
         const line = lineOf(match.index);
         ocorrencias.push(warn('useActionState (React 19) detected - verify form action integration', relPath, line, messages.SeverityNiveis.info));
     }
-    // React 19: useFormStatus detected
     for (const match of src.matchAll(/\buseFormStatus\s*\(/g)) {
         const line = lineOf(match.index);
         ocorrencias.push(warn('useFormStatus (React 19) detected - access pending form state', relPath, line, messages.SeverityNiveis.info));
     }
-    // React 19: use() hook - suspends, must be in Suspense boundary.
-    // OBS: esta detecção por regex é intencionalmente OMITIDA aqui porque
-    // casaria falsamente com comentários/strings (ex.: "terms-of-use (...)") e
-    // com identificadores como "reuse(" / "abuse(". O hook use() é detectado de
-    // forma precisa via AST em parseHooksWithBabel (que ignora comentários),
-    // tornando este regex redundante e gerador de falsos positivos.
-    // Custom hooks naming convention
     const customHooks = [...src.matchAll(/\buse[A-Z][A-Za-z0-9_]*\s*=\s*\(/g)];
     customHooks.forEach(m => {
         const hookName = m[0].replace(/\s*=\s*\(/, '');
@@ -242,7 +216,6 @@ function parseHooksWithBabel(src, relPath) {
         const isHookCallee = (callee) => {
             if (t.isIdentifier(callee)) {
                 const name = callee.name;
-                // React 19: use() is a special case (not prefixed with use but still a hook)
                 if (name === 'use') {
                     return { name };
                 }
@@ -263,20 +236,12 @@ function parseHooksWithBabel(src, relPath) {
         const isMemoLike = (name) => name === 'useMemo' || name === 'useCallback';
         const isAnyHook = (name) => /^use[A-Z0-9_]/.test(name) || name === 'use';
         const inConditionalOrLoop = (path) => {
-            // Só viola as Rules of Hooks se o hook estiver DENTRO de um
-            // if/for/while/do-while/switch/conditional no MESMO escopo de função que o
-            // contém. Condicionais em funções pais (ex.: um `if` no componente que define
-            // um hook customizado como useMusicPlayer) são válidas e NÃO devem gerar falso
-            // positivo (feedback.md seção 1.2). Callbacks aninhados (outra função) também
-            // delimitam o escopo — um condicional dentro de um callback não conta para o
-            // hook de fora.
             let current = path;
             while (current) {
                 const parent = current.parentPath;
                 if (!parent) {
                     break;
                 }
-                // Função que contém o hook: condicionais acima dela estão em escopo diferente.
                 if (parent.isFunction()) {
                     break;
                 }
@@ -361,15 +326,12 @@ function parseHooksWithBabel(src, relPath) {
                             }
                         }
                     }
-                    // React 19: use() hook should be in Suspense boundary
                     if (name === 'use') {
                         pushOnce(warn('React 19 use() hook - ensure wrapped in <Suspense> boundary', relPath, locLine));
                     }
-                    // React 19: useOptimistic should have reducer function
                     if (name === 'useOptimistic' && path.node.arguments.length < 2) {
                         pushOnce(warn('useOptimistic expects initial state and update function (React 19)', relPath, locLine));
                     }
-                    // React 19: useActionState should have form action
                     if (name === 'useActionState') {
                         pushOnce(warn('useActionState (React 19) - verify form action handler', relPath, locLine));
                     }
@@ -400,11 +362,6 @@ function parseHooksWithBabel(src, relPath) {
                                 return !depsSet.has(r);
                             });
                             if (maybeMissing.length > 1) {
-                                // Apenas useMemo/useCallback com deps faltando são "sem array de
-                                // dependências". useEffect/useLayoutEffect QUE JÁ POSSUEM um array
-                                // de deps (mesmo que incompleto) NÃO devem ser reportados como
-                                // "sem array" — ver feedback.md seção 1.1. O aviso de stale closure
-                                // para esses casos já é emitido por outro analisador.
                                 if (isMemoLike(name)) {
                                     pushOnce(warn(messages.ReactHooksMensagens.memoCallbackNoDeps, relPath, locLine));
                                 }
@@ -444,7 +401,6 @@ function parseHooksWithBabel(src, relPath) {
                     }
                 }
                 catch {
-                    // ignora
                 }
             }
         };
@@ -473,11 +429,6 @@ export const analistaReactHooks = criarAnalista({
         }
         const astMsgs = parseHooksWithBabel(src, relPath);
         if (astMsgs !== null) {
-            // O AST é a fonte autoritativa para verificações estruturais (deps de
-            // useEffect/useMemo e hooks em condicional). Quando disponível, o regex
-            // estrutural é ignorado para evitar os falsos positivos de análise por
-            // texto (ver feedback.md seção 1.1 e 1.2). As verificações informativas
-            // do regex permanecem como rede de segurança.
             const regexMsgs = collectHookIssues(src, relPath, { skipStructural: true });
             const all = [...astMsgs, ...regexMsgs];
             const seen = new Set();

package/dist/analysts/react/analysts/analyst-react.js

@@ -4,11 +4,8 @@ import * as textoTitulo from '@babel/types';
 import { messages } from '../../../core/messages/index.js';
 import { createLineLookup, maskJsComments } from '../../../shared/helpers/index.js';
 import { criarAnalista, criarOcorrencia } from '../../../types/index.js';
-const disableEnv = process.env.SUKUNA_DISABLE_PLUGIN_REACT === '1';
+const disableEnv = process.env.MAHORAGA_DISABLE_PLUGIN_REACT === '1';
 function hasJSX(src) {
-    // Evita falsos positivos com genéricos TypeScript (ex.: foo<Bar>(...)).
-    // Heurística: JSX normalmente aparece após um separador (início de arquivo/linha, whitespace,
-    // pontuação), enquanto genéricos aparecem logo após identificadores.
     const hasTagLike = /(^|[^A-Za-z0-9_$])<([A-Za-z][A-Za-z0-9]*)\b/.test(src);
     const hasFragment = /<>|<\/>|<\/\s*>|<\/\s*Fragment\s*>/i.test(src);
     return hasTagLike || hasFragment || /React\.createElement/.test(src);
@@ -27,7 +24,6 @@ function warn(message, relPath, line, nivel = messages.SeverityNiveis.warning) {
 function collectReactIssues(src, relPath) {
     const ocorrencias = [];
     const lineOf = createLineLookup(src).lineAt;
-    // <a target="_blank"> sem rel
     for (const match of src.matchAll(/<a[^>]*target=['"]?_blank['"]?[^>]*>/gi)) {
         const hasRelSafe = /rel=['"][^'"]*(noopener|noreferrer)[^'"]*['"]/i.test(match[0]);
         if (!hasRelSafe) {
@@ -35,17 +31,13 @@ function collectReactIssues(src, relPath) {
             ocorrencias.push(warn(messages.ReactMensagens.linkTargetBlank, relPath, line));
         }
     }
-    // dangerouslySetInnerHTML
     for (const match of src.matchAll(/dangerouslySetInnerHTML/gi)) {
         const line = lineOf(match.index);
-        // Skip if this is a <script type="speculationrules"> element — dangerouslySetInnerHTML
-        // is required for Speculation Rules API (no children allowed, only innerHTML).
         const lineContent = src.split('\n')[line - 1] ?? '';
         if (/speculationrules/.test(lineContent))
             continue;
         ocorrencias.push(warn(messages.ReactMensagens.dangerouslySetInnerHTML, relPath, line));
     }
-    // <img> sem alt
     for (const match of src.matchAll(/<img[^>]*>/gi)) {
         const hasAlt = /\salt=/.test(match[0]);
         const ariaHidden = /\saria-hidden=['"]true['"]/i.test(match[0]);
@@ -55,12 +47,10 @@ function collectReactIssues(src, relPath) {
             ocorrencias.push(warn(messages.ReactMensagens.imgWithoutAlt, relPath, line));
         }
     }
-    // fetch/axios com HTTP (inseguro)
     for (const match of src.matchAll(/\b(fetch|axios\.get|axios\.post|axios\.[a-z]+)\s*\(\s*['"]http:\/\//gi)) {
         const line = lineOf(match.index);
         ocorrencias.push(warn(messages.ReactMensagens.httpFetch, relPath, line));
     }
-    // Hardcoded secrets (heurística)
     for (const match of src.matchAll(/(api[key]?|token|secret|senha|password|passphrase)\s*[:=]\s*['"]([A-Za-z0-9\/_+=.-]{16,})['"]/gi)) {
         const valor = match[2] || '';
         if (valor.length < 24 || /^https?:\/\//i.test(valor)) {
@@ -69,22 +59,16 @@ function collectReactIssues(src, relPath) {
         const line = lineOf(match.index);
         ocorrencias.push(warn(messages.ReactMensagens.hardcodedCredential, relPath, line));
     }
-    // Redirecionamento direto via location.href em handlers
-    // Ignorar router.push/replace do Next.js (navegação segura via client-side routing)
     const usesNextRouter = /useRouter|router\.(push|replace)/.test(src);
     for (const match of src.matchAll(/location\.href\s*=\s*([^;]+)/gi)) {
         const assignedValor = match[1] || '';
         const line = lineOf(match.index);
-        // Ignorar se valor é string literal (não vem de input do usuário)
         const isStaticString = /^['"`][^'"\`]*['"`]$/.test(assignedValor.trim());
-        // Ignorar se é window.location.href = '/path' (navegação interna estática)
         const isInternalNavigation = /^['"`]\//.test(assignedValor.trim());
-        // Só reportar se for atribuição dinâmica (potencialmente de input do usuário)
         if (!isStaticString && !isInternalNavigation && !usesNextRouter) {
             ocorrencias.push(warn(messages.ReactMensagens.locationHrefRedirect, relPath, line));
         }
     }
-    // React 19: <form action={...}>
     for (const match of src.matchAll(/<form[^>]*>/gi)) {
         const hasAction = /\saction\s*=\s*\{/.test(match[0]);
         if (hasAction) {
@@ -92,19 +76,16 @@ function collectReactIssues(src, relPath) {
             ocorrencias.push(warn('React 19 form action detected', relPath, line, messages.SeverityNiveis.info));
         }
     }
-    // React 19: use() hook (called at top level in render)
     for (const match of src.matchAll(/\bconst\s+\w+\s*=\s*use\s*\(/g)) {
         const line = lineOf(match.index);
         if (!/use[A-Z]/.test(match[0])) {
             ocorrencias.push(warn('React 19 use() hook detected — ensure wrapped in Suspense boundary', relPath, line, messages.SeverityNiveis.info));
         }
     }
-    // 'use client' / 'use server' directives
     if (/^['"]use (client|server)['"];/m.test(src)) {
         ocorrencias.push(warn('Server Component directive detected', relPath, 1, messages.SeverityNiveis.info));
     }
     if (!/^['"]use client['"];/m.test(src) && /useState|useEffect|useContext|onClick/.test(src)) {
-        // This file uses client-side features without 'use client'
         const hasClientFeatures = /useState|useEffect|useContext|onClick/.test(src);
         if (hasClientFeatures) {
             const firstLine = src.split('\n')[0]?.trim() ?? '';
@@ -113,17 +94,14 @@ function collectReactIssues(src, relPath) {
             }
         }
     }
-    // React 19: forwardRef deprecation
     for (const match of src.matchAll(/\bforwardRef\s*\(/g)) {
         const line = lineOf(match.index);
         ocorrencias.push(warn('forwardRef is deprecated in React 19 — pass ref as a regular prop instead', relPath, line, messages.SeverityNiveis.suggestion));
     }
-    // React 19: React.FC / React.FunctionComponent / React.VoidFunctionComponent
     for (const match of src.matchAll(/\bReact\.(FC|FunctionComponent|VoidFunctionComponent)\b/g)) {
         const line = lineOf(match.index);
         ocorrencias.push(warn(`React.${match[1]} is deprecated in React 19 — type props directly: (props: Props) => JSX.Element`, relPath, line, messages.SeverityNiveis.suggestion));
     }
-    // React 19: defaultProps
     for (const match of src.matchAll(/\.defaultProps\s*=\s*\{/g)) {
         const line = lineOf(match.index);
         ocorrencias.push(warn('defaultProps is deprecated in React 19 — use default parameter values instead', relPath, line, messages.SeverityNiveis.suggestion));
@@ -173,8 +151,6 @@ function parseReactWithBabel(scan, relPath) {
     const isTs = lower.endsWith('.ts') || lower.endsWith('.tsx');
     const isJsxLike = lower.endsWith('.tsx') || lower.endsWith('.jsx');
     const isJsFamily = /\.(js|mjs|cjs|jsx)$/i.test(lower);
-    // Em .ts puro o Babel com JSX pode aumentar falsos positivos; só parseamos quando o
-    // arquivo já passou pela heurística de JSX antes.
     if (lower.endsWith('.ts') && !isJsxLike) {
         return null;
     }
@@ -183,7 +159,6 @@ function parseReactWithBabel(scan, relPath) {
         if (isTs) {
             plugins.unshift('typescript');
         }
-        // Robustez: projetos React frequentemente usam JSX em .js
         if (isJsxLike || isJsFamily) {
             plugins.unshift('jsx');
         }
@@ -232,14 +207,11 @@ function parseReactWithBabel(scan, relPath) {
                         const ariaHiddenVal = normalizeStringValue(ariaHidden?.value);
                         const roleVal = normalizeStringValue(role?.value);
                         const decorative = ariaHiddenVal === 'true' || /^(presentation|none)$/i.test(roleVal);
-                        // Considerar alt presente se há atributo alt, mesmo se valor é expressão
                         if (!alt && !decorative) {
                             pushOnce(warn(messages.ReactMensagens.imgWithoutAlt, relPath, locLine));
                         }
                     }
                     if (findAttr(attrs, 'dangerouslySetInnerHTML')) {
-                        // Skip if this is a <script type="speculationrules"> — dangerouslySetInnerHTML
-                        // is required for Speculation Rules API (no children, only innerHTML).
                         if (tag.toLowerCase() === 'script') {
                             const typeAttr = findAttr(attrs, 'type');
                             if (typeAttr && /speculationrules/i.test(normalizeStringValue(typeAttr.value)))
@@ -247,16 +219,13 @@ function parseReactWithBabel(scan, relPath) {
                         }
                         pushOnce(warn(messages.ReactMensagens.dangerouslySetInnerHTML, relPath, locLine));
                     }
-                    // Inline Styles
                     const styleAttr = findAttr(attrs, 'style');
                     if (styleAttr?.value?.includes('{')) {
-                        // Heurística: se o estilo tem mais de 2 propriedades, sugere mover para CSS/Styled
                         const numProps = (styleAttr.value.match(/:/g) ?? []).length;
                         if (numProps > 2) {
                             pushOnce(warn(messages.ReactMensagens.complexInlineStyles, relPath, locLine, messages.SeverityNiveis.warning));
                         }
                     }
-                    // Verificar handlers inline em atributos JSX (ex.: onClick={() => ...})
                     for (const arquivo of node.attributes) {
                         try {
                             if (!textoTitulo.isJSXAttribute(arquivo) || !textoTitulo.isJSXIdentifier(arquivo.name)) {
@@ -268,7 +237,6 @@ function parseReactWithBabel(scan, relPath) {
                                 if (textoTitulo.isArrowFunctionExpression(expr) || textoTitulo.isFunctionExpression(expr)) {
                                     pushOnce(warn(messages.ReactMensagens.inlineHandlerJsx, relPath, locLine));
                                 }
-                                // Detectar uso de índice como key (key={i} / key={index})
                                 if (attrNome === 'key' && textoTitulo.isJSXExpressionContainer(arquivo.value) && textoTitulo.isIdentifier(arquivo.value.expression)) {
                                     const id = arquivo.value.expression.name;
                                     if (['i', 'index', 'idx'].includes(id)) {
@@ -278,12 +246,10 @@ function parseReactWithBabel(scan, relPath) {
                             }
                         }
                         catch {
-                            // ignora
                         }
                     }
                 }
                 catch {
-                    // ignora
                 }
             },
             ClassMethod(path) {
@@ -298,7 +264,6 @@ function parseReactWithBabel(scan, relPath) {
                     }
                 }
                 catch {
-                    // ignora
                 }
             },
             CallExpression(path) {
@@ -324,7 +289,6 @@ function parseReactWithBabel(scan, relPath) {
                                 pushOnce(warn(messages.ReactMensagens.httpFetch, relPath, locLine));
                             }
                         }
-                        // Detectar Array#map com callback retornando JSX sem key
                         if (propNome === 'map' && args.length >= 1) {
                             const cb = args[0];
                             const checkCallback = (fn) => {
@@ -338,7 +302,6 @@ function parseReactWithBabel(scan, relPath) {
                                             }
                                         }
                                         else {
-                                            // JSXFragment (<>...</>) não aceita atributo key - tratar como sem key
                                             pushOnce(warn(messages.ReactMensagens.listItemNoKey, relPath, jsx.loc?.start.line ?? locLine));
                                         }
                                     };
@@ -356,7 +319,6 @@ function parseReactWithBabel(scan, relPath) {
                                     }
                                 }
                                 catch {
-                                    // ignora
                                 }
                             };
                             if (textoTitulo.isFunctionExpression(cb)) {
@@ -369,7 +331,6 @@ function parseReactWithBabel(scan, relPath) {
                     }
                 }
                 catch {
-                    // ignora
                 }
             },
             AssignmentExpression(path) {
@@ -385,7 +346,6 @@ function parseReactWithBabel(scan, relPath) {
                     const objNome = textoTitulo.isIdentifier(obj) ? obj.name : '';
                     const propNome = textoTitulo.isIdentifier(prop) ? prop.name : '';
                     if (objNome === 'location' && propNome === 'href') {
-                        // Ignorar se RHS é string literal estática ou navegação interna
                         const right = node.right;
                         let isSafe = false;
                         if (textoTitulo.isStringLiteral(right)) {
@@ -398,7 +358,6 @@ function parseReactWithBabel(scan, relPath) {
                     }
                 }
                 catch {
-                    // ignora
                 }
             },
             ObjectProperty(path) {
@@ -424,7 +383,6 @@ function parseReactWithBabel(scan, relPath) {
                     pushOnce(warn(messages.ReactMensagens.hardcodedCredential, relPath, locLine));
                 }
                 catch {
-                    // ignora
                 }
             }
         };
@@ -449,8 +407,6 @@ export const analistaReact = criarAnalista({
             return null;
         }
         const scan = maskJsComments(src);
-        // Em arquivos .ts (não TSX), seja mais conservador para evitar falsos positivos
-        // com assertions/generics usando <T>.
         if (/\.ts$/i.test(relPath) && !/\.tsx$/i.test(relPath)) {
             const hasStrongJsxEvidence = /<\/[A-Za-z][^>]*>/.test(scan) || /\/>/.test(scan) || /React\.createElement/.test(scan);
             if (!hasStrongJsxEvidence) {
@@ -465,7 +421,6 @@ export const analistaReact = criarAnalista({
         if (!astMsgs.length && !heuristicMsgs.length) {
             return null;
         }
-        // Dedupe para evitar duplicatas quando AST e regex cobrem o mesmo caso.
         const seen = new Set();
         const merged = [];
         const pushOnce = (metrica) => {

package/dist/analysts/react/corrections/correction-react.js

@@ -87,7 +87,6 @@ export function detectarCorrecoesReact(src, _relPath) {
                 confianca: 50
             });
         }
-        // React 19: forwardRef replacement
         if (/\bforwardRef\s*\(/.test(linha)) {
             correcoes.push({
                 tipo: 'react-forwardref-to-prop',
@@ -97,7 +96,6 @@ export function detectarCorrecoesReact(src, _relPath) {
                 confianca: 85
             });
         }
-        // React 19: React.FC replacement
         if (/\bReact\.(FC|FunctionComponent)\s*</.test(linha)) {
             correcoes.push({
                 tipo: 'react-fc-to-direct-type',
@@ -107,7 +105,6 @@ export function detectarCorrecoesReact(src, _relPath) {
                 confianca: 85
             });
         }
-        // React 19: defaultProps replacement
         if (/\.defaultProps\s*=/.test(linha)) {
             correcoes.push({
                 tipo: 'react-default-props-to-params',
@@ -117,7 +114,6 @@ export function detectarCorrecoesReact(src, _relPath) {
                 confianca: 90
             });
         }
-        // React 19: 'use client' directive missing
         if (/^\s*import\s+/.test(linha) && /useState|useEffect|onClick/.test(linha)) {
             correcoes.push({
                 tipo: 'react-use-client-missing',

package/dist/analysts/react/corrections/index.js

@@ -1,2 +1 @@
-// SPDX-License-Identifier: MIT
 export * from './correction-react.js';

package/dist/analysts/react/detectors/detector-react-best-practices.js

@@ -9,18 +9,6 @@ const D = messages.DetectorReactMensagens;
 function warn(tipo, mensagem, relPath, linha, sugestao, nivel = 'aviso') {
     return criarOcorrencia({ tipo, nivel, mensagem, relPath, linha, sugestao });
 }
-/**
- * Decide se o valor passado a `dangerouslySetInnerHTML` é estático/controlado
- * e, portanto, seguro o suficiente para NÃO gerar falso positivo.
- * (ver feedback.md seção 1.3 — JSON-LD, string interna, conteúdo sanitizado).
- *
- * Consideramos controlado quando o valor é:
- *  - uma string literal (estática);
- *  - resultado de `JSON.stringify(...)` (ex.: JSON-LD / structured data);
- *  - resultado de uma chamada de sanitização/escape (`sanitize`, `escape`,
- *    `encode`, `secure`, etc.);
- *  - um template literal (conteúdo estático na prática).
- */
 function isJsonStringifyCall(node) {
     if (!t.isCallExpression(node)) {
         return false;
@@ -55,7 +43,6 @@ function isControlledDangerousHtml(expr) {
         if (isJsonStringifyCall(expr)) {
             return true;
         }
-        // JSON.stringify(data).replace(...) / .trim() — o objeto da chamada é JSON.stringify
         if (t.isMemberExpression(expr.callee) && isJsonStringifyCall(expr.callee.object)) {
             return true;
         }
@@ -64,8 +51,6 @@ function isControlledDangerousHtml(expr) {
             return true;
         }
     }
-    // JSON.stringify(...).replace(...) — o objeto da MemberExpression é a chamada
-    // JSON.stringify, então o conteúdo é estruturado/controlado (ex.: JSON-LD).
     if (t.isMemberExpression(expr) && isJsonStringifyCall(expr.object)) {
         return true;
     }
@@ -118,11 +103,6 @@ export const detectorReactBestPractices = {
                 }
             }
         }
-        // Fallback conservador: quando o parse por AST falha, ainda detectamos
-        // dangerouslySetInnerHTML de forma limitada, alertando apenas quando o
-        // valor claramente vem de fonte externa/não controlada (props, params,
-        // localStorage, fetch, location, etc). Conteúdo estático/controlado é
-        // ignorado (ver feedback.md seção 1.3 - ex.: JSON-LD).
         if (!deeper) {
             linhas.forEach((linha, index) => {
                 if (/dangerouslySetInnerHTML/.test(linha) && !/speculationrules/i.test(linha)) {
@@ -175,20 +155,15 @@ function detectWithBabel(src, relPath) {
                 }
                 const locLine = path.node.loc?.start.line;
                 if (isHookName(name)) {
-                    // Only flag hooks inside conditionals in the SAME function scope.
-                    // Stop climbing at function boundaries (callbacks, event handlers, etc.)
-                    // to avoid false positives when a conditional is inside a hook callback.
                     const parent = (() => {
                         let currentPath = path;
                         while (currentPath) {
                             const found = currentPath.findParent(p => p.isIfStatement() || p.isConditionalExpression() || p.isLogicalExpression() || p.isSwitchStatement() || p.isSwitchCase() || p.isForStatement() || p.isWhileStatement() || p.isDoWhileStatement() || p.isLoop());
                             if (found)
                                 return found;
-                            // Stop at function boundaries
                             const fnBoundary = currentPath.findParent(p => p.isFunction());
                             if (!fnBoundary)
                                 return null;
-                            // Check above the function boundary
                             const aboveFn = fnBoundary.findParent(p => p.isIfStatement() || p.isConditionalExpression() || p.isLogicalExpression() || p.isSwitchStatement() || p.isSwitchCase() || p.isForStatement() || p.isWhileStatement() || p.isDoWhileStatement() || p.isLoop());
                             if (aboveFn)
                                 return aboveFn;
@@ -397,10 +372,6 @@ function detectWithBabel(src, relPath) {
                         pushOnce(warn('react-key-index', D.keyIndexUsage, relPath, locLine, D.keyIndexUsageSugestao, 'aviso'));
                     }
                 }
-                // dangerouslySetInnerHTML: só alerta quando o valor NÃO é estático/
-                // controlado (ex.: vem de props, params, fetch, localStorage). Conteúdo
-                // como JSON-LD (JSON.stringify) ou string interna é seguro e não deve
-                // gerar ruído (ver feedback.md seção 1.3).
                 if (name === 'dangerouslySetInnerHTML') {
                     const value = path.node.value;
                     let htmlExpr;

package/dist/analysts/react/detectors/index.js

@@ -1,2 +1 @@
-// SPDX-License-Identifier: MIT
 export * from './detector-react-best-practices.js';