@mocoto/mahoraga 0.14.0 → 0.14.1

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

@@ -1,3 +1,13 @@
+// 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;
@@ -7,21 +17,26 @@ export function pontuarPython(src, _relPath) {
     const scan = maskPythonComments(src);
     const _linhas = splitLines(scan);
     const srcCompleto = scan;
-    const funcs = (srcCompleto.match(/def\s+\w+/g) || []).length;
-    const funcsWithHints = (srcCompleto.match(/def\s+\w+\s*\([^)]*\)\s*->/g) || []).length;
+    // 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) {
         const ratio = funcsWithHints / funcs;
         score += Math.round(ratio * 25);
-        if (ratio > 0.8)
+        if (ratio > 0.8) {
             strengths.push('Boa cobertura de type hints');
-        else if (ratio > 0.5)
+        }
+        else if (ratio > 0.5) {
             issues.push('Type hints parciais');
-        else
+        }
+        else {
             issues.push('Poucas funções com type hints');
+        }
     }
     else {
         score += 25;
     }
+    // Sem eval/exec (20 pontos)
     if (!/\b(eval|exec)\s*\(/.test(srcCompleto)) {
         score += 20;
         strengths.push('Sem uso de eval/exec');
@@ -29,7 +44,8 @@ export function pontuarPython(src, _relPath) {
     else {
         issues.push('eval/exec detectado - risco de segurança');
     }
-    const bareExcept = (srcCompleto.match(/^\s*except\s*:/gm) || []).length;
+    // Error handling (20 pontos)
+    const bareExcept = (srcCompleto.match(/^\s*except\s*:/gm) ?? []).length;
     if (bareExcept === 0) {
         score += 20;
         strengths.push('Error handling adequado');
@@ -38,7 +54,8 @@ export function pontuarPython(src, _relPath) {
         score += Math.max(0, 20 - bareExcept * 5);
         issues.push(`${bareExcept} bare except(s) detectado(s)`);
     }
-    const mutableDefaults = (srcCompleto.match(/def\s+\w+\s*\([^)]*=\s*(?:\[|\{)/g) || []).length;
+    // Sem mutable defaults (15 pontos)
+    const mutableDefaults = (srcCompleto.match(/def\s+\w+\s*\([^)]*=\s*(?:\[|\{)/g) ?? []).length;
     if (mutableDefaults === 0) {
         score += 15;
         strengths.push('Sem argumentos default mutáveis');
@@ -46,8 +63,9 @@ export function pontuarPython(src, _relPath) {
     else {
         issues.push(`${mutableDefaults} função(ões) com default mutável`);
     }
-    const printCount = (srcCompleto.match(/^\s*print\s*\(/gm) || []).length;
-    const loggingCount = (srcCompleto.match(/logging\.\w+\(/g) || []).length;
+    // 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) {
         score += 10;
         strengths.push('Uso adequado de logging');
@@ -55,6 +73,7 @@ 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');
@@ -76,4 +95,3 @@ export const pontuadorPython = {
     descricao: 'Pontua qualidade de código Python',
     pontuar: pontuarPython
 };
-//# sourceMappingURL=scorer-python.js.map

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

@@ -4,7 +4,7 @@ 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.MAHORAGA_DISABLE_PLUGIN_REACT_HOOKS === '1';
+const disableEnv = process.env.SUKUNA_DISABLE_PLUGIN_REACT_HOOKS === '1';
 function hasHooksUsage(src) {
     return /use(State|Effect|Memo|Callback|Reducer|Ref|LayoutEffect|ImperativeHandle|Transition|DeferredValue|SyncExternalStore|InsertionEffect|Id|DebugValue)/.test(src);
 }
@@ -31,10 +31,12 @@ function extractHookCallContent(src, startIndex) {
         else if (char === ')') {
             depth--;
         }
-        if (started)
+        if (started) {
             content += char;
-        if (started && depth === 0)
+        }
+        if (started && depth === 0) {
             break;
+        }
     }
     return content;
 }
@@ -54,7 +56,7 @@ function collectHookIssues(src, relPath) {
         if (hasDepsArg && /\[\s*\]/.test(fullCall)) {
             const bodyWithRefs = fullCall.match(/^[\s\S]*?(?=,\s*\[)/);
             if (bodyWithRefs) {
-                const varRefs = bodyWithRefs[0].match(/\b([a-z]\w+)\b/g) || [];
+                const varRefs = bodyWithRefs[0].match(/\b([a-z]\w+)\b/g) ?? [];
                 const nonTrivialRefs = varRefs.filter(v => !/^(set|use)/.test(v) && v.length > 1 && !['undefined', 'null', 'true', 'false'].includes(v));
                 if (nonTrivialRefs.length > 0) {
                     const line = lineOf(hookInicio);
@@ -97,32 +99,37 @@ function parseHooksWithBabel(src, relPath) {
     const isTs = lower.endsWith('.ts') || lower.endsWith('.tsx');
     const isJsxLike = lower.endsWith('.tsx') || lower.endsWith('.jsx');
     try {
-        const plugins = ['decorators-legacy'];
-        if (isTs)
+        const plugins = ['decorators-legacy', 'classProperties', 'classPrivateProperties', 'classPrivateMethods', 'optionalChaining', 'nullishCoalescingOperator', 'topLevelAwait', 'importAttributes'];
+        if (isTs) {
             plugins.unshift('typescript');
-        if (isJsxLike)
+        }
+        if (isJsxLike) {
             plugins.unshift('jsx');
+        }
         const ast = babelParse(src, { sourceType: 'unambiguous', errorRecovery: true, plugins });
         const out = [];
         const seen = new Set();
         const pushOnce = (m) => {
             const chave = `${m.mensagem}|${m.relPath}|${m.linha ?? 0}`;
-            if (seen.has(chave))
+            if (seen.has(chave)) {
                 return;
+            }
             seen.add(chave);
             out.push(m);
         };
         const declaredRefs = new Set();
         const isHookCallee = (callee) => {
-            if (t.isIdentifier(callee))
+            if (t.isIdentifier(callee)) {
                 return { name: callee.name };
+            }
             if (t.isMemberExpression(callee)) {
                 const obj = callee.object;
                 const prop = callee.property;
                 const objNome = t.isIdentifier(obj) ? obj.name : '';
                 const propNome = t.isIdentifier(prop) ? prop.name : '';
-                if (objNome === 'React' && propNome)
+                if (objNome === 'React' && propNome) {
                     return { name: propNome };
+                }
             }
             return null;
         };
@@ -134,8 +141,9 @@ function parseHooksWithBabel(src, relPath) {
         };
         const getIdentifiersInNode = (node) => {
             const ids = new Set();
-            if (!node)
+            if (!node) {
                 return ids;
+            }
             const text = src.slice(node.start ?? 0, node.end ?? 0);
             const refMatches = text.matchAll(/\b([a-z]\w*)\b/g);
             for (const m of refMatches) {
@@ -161,10 +169,11 @@ function parseHooksWithBabel(src, relPath) {
             CallExpression(path) {
                 try {
                     const calleeInfo = isHookCallee(path.node.callee);
-                    if (!calleeInfo)
+                    if (!calleeInfo) {
                         return;
+                    }
                     const { name } = calleeInfo;
-                    const locLine = path.node.loc?.start?.line;
+                    const locLine = path.node.loc?.start.line;
                     if (isEffectLike(name) && path.node.arguments.length < 2 && path.node.arguments.length >= 1) {
                         pushOnce(warn(messages.ReactHooksMensagens.useEffectNoDeps, relPath, locLine));
                     }
@@ -179,7 +188,7 @@ function parseHooksWithBabel(src, relPath) {
                     if (isAnyHook(name) && arrowParent && fnParent) {
                         pushOnce(warn('Hook inside nested callback - move to top level', relPath, locLine));
                     }
-                    if (fnParent && fnParent.isFunction()) {
+                    if (fnParent?.isFunction()) {
                         const fnBody = fnParent.node.body;
                         if (t.isBlockStatement(fnBody)) {
                             const bodyStmts = fnBody.body;
@@ -205,18 +214,23 @@ function parseHooksWithBabel(src, relPath) {
                         if (t.isArrayExpression(deps)) {
                             const depsSet = new Set();
                             for (const el of deps.elements) {
-                                if (t.isIdentifier(el))
+                                if (t.isIdentifier(el)) {
                                     depsSet.add(el.name);
+                                }
                             }
                             const maybeMissing = [...refsInCb].filter(r => {
-                                if (/^[A-Z]/.test(r))
+                                if (/^[A-Z]/.test(r)) {
                                     return false;
-                                if (r === name || r === 'undefined' || r === 'null' || r === 'true' || r === 'false')
+                                }
+                                if (r === name || r === 'undefined' || r === 'null' || r === 'true' || r === 'false') {
                                     return false;
-                                if (/^(set|dispatch)/.test(r))
+                                }
+                                if (/^(set|dispatch)/.test(r)) {
                                     return false;
-                                if (declaredRefs.has(r) && r.startsWith('ref'))
+                                }
+                                if (declaredRefs.has(r) && r.startsWith('ref')) {
                                     return false;
+                                }
                                 return !depsSet.has(r);
                             });
                             if (maybeMissing.length > 1) {
@@ -229,10 +243,12 @@ function parseHooksWithBabel(src, relPath) {
                             }
                             if (deps.elements.length === 0 && refsInCb.size > 0) {
                                 const nonTrivialRefs = [...refsInCb].filter(r => {
-                                    if (/^[A-Z]/.test(r))
+                                    if (/^[A-Z]/.test(r)) {
                                         return false;
-                                    if (r === 'undefined' || r === 'null' || r === 'true' || r === 'false')
+                                    }
+                                    if (r === 'undefined' || r === 'null' || r === 'true' || r === 'false') {
                                         return false;
+                                    }
                                     return !/^(set|dispatch)/.test(r);
                                 });
                                 if (nonTrivialRefs.length > 0) {
@@ -260,6 +276,7 @@ function parseHooksWithBabel(src, relPath) {
                     }
                 }
                 catch {
+                    // ignora
                 }
             }
         };
@@ -276,13 +293,16 @@ export const analistaReactHooks = criarAnalista({
     descricao: 'Heurísticas de Hooks com análise de dependências (sem ESLint).',
     global: false,
     test: (relPath) => /\.(jsx|tsx|js|ts)$/i.test(relPath),
-    aplicar: async (src, relPath) => {
-        if (disableEnv)
+    aplicar: (src, relPath) => {
+        if (disableEnv) {
             return null;
-        if (relPath.includes('src/analysts/react/analysts/analyst-react-hooks.ts'))
+        }
+        if (relPath.includes('src/analysts/react/analysts/analyst-react-hooks.ts')) {
             return null;
-        if (!hasHooksUsage(src))
+        }
+        if (!hasHooksUsage(src)) {
             return null;
+        }
         const astMsgs = parseHooksWithBabel(src, relPath);
         if (astMsgs !== null) {
             const regexMsgs = collectHookIssues(src, relPath);
@@ -290,8 +310,9 @@ export const analistaReactHooks = criarAnalista({
             const seen = new Set();
             const deduped = all.filter(m => {
                 const chave = `${m.mensagem}|${m.relPath}|${m.linha ?? 0}`;
-                if (seen.has(chave))
+                if (seen.has(chave)) {
                     return false;
+                }
                 seen.add(chave);
                 return true;
             });
@@ -301,4 +322,3 @@ export const analistaReactHooks = criarAnalista({
         return msgs.length ? msgs : null;
     }
 });
-//# sourceMappingURL=analyst-react-hooks.js.map