@polymorphism-tech/morph-spec 8.37.1 → 8.40.0

package/src/lib/validators/reuse/reuse-validator.js

@@ -114,8 +114,8 @@ export function findDuplicateSymbols(declarations, existing, opts = {}) {
     const language = languageOf(decls[0].file);
 
     // Onde mais esse nome vive: no grafo (código que já existia) e nos OUTROS
-    // arquivos desta mesma leva (código novo que o grafo ainda não viu — o
-    // rebuild é assíncrono, e é exatamente o caso SimpleBuilding).
+    // arquivos desta mesma leva (código novo que o grafo ainda não viu — ele
+    // só é refeito depois do commit, e é exatamente o caso SimpleBuilding).
     const fromGraph = (existing.get(key) ?? []).filter(
       (e) => e.language === language && !declaredFiles.has(e.file),
     );
@@ -280,6 +280,8 @@ export async function validateReuse(projectPath, featureName = null, options = {
     // segue em frente. Um validador que reprovasse aqui transformaria uma
     // dependência opcional em pré-requisito.
     result._skipReason = 'sem grafo de codebase — `morph-spec graph build` habilita esta verificação';
+    // "Could not check", unlike "nothing to check": verify reports it aloud.
+    result._skipUnavailable = true;
     return result;
   }
 

package/src/lib/validators/reuse/symbol-scanner.js

@@ -1,12 +1,13 @@
 /**
  * Extração determinística de declarações de tipo a partir do TEXTO do arquivo.
  *
- * Por que não ler as declarações do grafo: o grafo é reconstruído em background
- * pelo hook post-commit do graphify, então o código que acabou de ser escrito
- * (o caso que este validador existe para pegar) ainda NÃO está nele. O grafo
- * responde "o que já existia"; o disco responde "o que esta feature acabou de
- * declarar". Inverter os dois papéis produziria um validador cego justamente na
- * janela em que ele precisa morder — ver graph-loader.js `getFreshness`.
+ * Por que não ler as declarações do grafo: o grafo só é reconstruído depois de
+ * um commit (o refresh em background de framework/hooks/shared/graph-refresh.js),
+ * então o código que acabou de ser escrito — o caso que este validador existe
+ * para pegar — ainda NÃO está nele. O grafo responde "o que já existia"; o
+ * disco responde "o que esta feature acabou de declarar". Inverter os dois
+ * papéis produziria um validador cego justamente na janela em que ele precisa
+ * morder — ver graph-loader.js `getFreshness`.
  *
  * A varredura é por linha, com regex ancorada no início da declaração. Mesma
  * limitação (e mesmo grau de tolerância) do design-system-validator: uma
@@ -16,20 +17,11 @@
  * @module reuse-symbol-scanner
  */
 
-import { languageOf } from '../../graph/sanitize.js';
+import { isTestPath, languageOf } from '../../graph/sanitize.js';
 
 /** Extensões que este scanner sabe ler. Fora daqui, o arquivo é ignorado. */
 const SCANNABLE = /\.(cs|ts|tsx|js|jsx|mjs|cjs)$/i;
 
-/**
- * Arquivo de teste, em qualquer das convenções que os dois stacks usam.
- * Duplicar um stub entre três arquivos de teste (`ResizeObserverStub` no
- * ProspectPRO) é ruído conhecido: teste é código descartável por natureza e a
- * regra de reuso não se aplica a ele. Cobre `__tests__/`, `.test.`, `.spec.`,
- * e os projetos `*.Tests/` do .NET.
- */
-const TEST_FILE = /(^|\/)__tests__\/|\.(test|spec)\.[jt]sx?$|(^|\/)(?:[Tt]ests?|[^/]*\.[Tt]ests?|[^/]*[a-z0-9_]Tests?)\//;
-
 /**
  * Diretórios que contêm arquivos com extensão de código mas que não são o
  * código-fonte do projeto. `.morph/` é o caso que apareceu de verdade: o
@@ -71,17 +63,30 @@ const WEB_DECL =
 /** Linha que é claramente comentário — não vale a pena parsear. */
 const COMMENT_LINE = /^\s*(?:\/\/|\/\*|\*|<!--)/;
 
+/**
+ * @param {string} sourceFile
+ * @returns {boolean} código do projeto numa extensão lida, fora de `.d.ts` e de
+ *   pasta que não é fonte — teste incluído
+ */
+export function isProjectSource(sourceFile) {
+  const f = String(sourceFile || '').replace(/\\/g, '/');
+  return SCANNABLE.test(f) && !DECLARATION_FILE.test(f) && !NOT_SOURCE.test(f);
+}
+
+/**
+ * @param {string} sourceFile
+ * @returns {boolean} arquivo de teste, pelo mesmo predicado do grafo
+ */
+export function isTestFile(sourceFile) {
+  return isTestPath(String(sourceFile || '').replace(/\\/g, '/'));
+}
+
 /**
  * @param {string} sourceFile
  * @returns {boolean} o arquivo entra na varredura de reuso
  */
 export function isScannableFile(sourceFile) {
-  const f = String(sourceFile || '').replace(/\\/g, '/');
-  if (!SCANNABLE.test(f)) return false;
-  if (DECLARATION_FILE.test(f)) return false;
-  if (TEST_FILE.test(f)) return false;
-  if (NOT_SOURCE.test(f)) return false;
-  return true;
+  return isProjectSource(sourceFile) && !isTestFile(sourceFile);
 }
 
 /**

package/src/lib/validators/validation-runner.js

@@ -24,7 +24,7 @@ import { aggregateScores, loadRubric } from './score-aggregator.js';
  */
 const HEAVY_VALIDATORS = new Set([
   'architecture', 'packages', 'contract-compliance',
-  'css', 'design-system', 'reuse'
+  'css', 'design-system', 'reuse', 'code-budget'
 ]);
 
 /**
@@ -89,6 +89,11 @@ export async function runValidation(projectPath, featureName, options = {}) {
   if (!options.skipReuse && mode !== 'light') {
     validatorIds.push('reuse');
   }
+  // Mesma política, pelo mesmo motivo: comentário e orçamento de teste valem
+  // em qualquer stack, e a medida é da feature, não de um checkpoint.
+  if (mode !== 'light') {
+    validatorIds.push('code-budget');
+  }
 
   if (validatorIds.length === 0) {
     // Say which of the two it is. On a pure `dotnet` project every default
@@ -135,6 +140,7 @@ export async function runValidation(projectPath, featureName, options = {}) {
   // projeto sem grafo). Um validador assim não acha nada — e "não achou nada"
   // é indistinguível de "verificou e está limpo" se ninguém contar.
   const selfSkipped = [];
+  result.skipped = selfSkipped;
 
   // Run each validator
   for (const validatorId of uniqueValidators) {
@@ -149,15 +155,20 @@ export async function runValidation(projectPath, featureName, options = {}) {
 
       if (validatorResult) {
         if (validatorResult._skipReason) {
-          selfSkipped.push(`${validatorId}: ${validatorResult._skipReason}`);
+          selfSkipped.push({
+            validator: validatorId,
+            reason: validatorResult._skipReason,
+            unavailable: validatorResult._skipUnavailable === true,
+          });
         }
         const { errors, warnings, issues } = normalizeValidatorResult(validatorId, validatorResult);
         result.errors += errors;
         result.warnings += warnings;
         result.issues.push(...issues);
 
-        // Store per-validator results for hooks
-        result.results[validatorId] = { passed: errors === 0, issues };
+        // A validator that skipped itself checked nothing: in `results` it would
+        // be scored as a clean 10 and dilute the ones that ran.
+        if (!validatorResult._skipReason) result.results[validatorId] = { passed: errors === 0, issues };
       }
     } catch (err) {
       // A validator that throws is a FAILED validation, not a silent skip.
@@ -187,7 +198,7 @@ export async function runValidation(projectPath, featureName, options = {}) {
   // `validators: pass` do `morph-spec verify` tendo verificado exatamente nada
   // — o modo de falha que este repositório já pagou caro (guardrail inerte).
   if (selfSkipped.length > 0 && selfSkipped.length === uniqueValidators.length) {
-    result._skipReason = `No validators ran — ${selfSkipped.join('; ')}`;
+    result._skipReason = `No validators ran — ${selfSkipped.map((s) => `${s.validator}: ${s.reason}`).join('; ')}`;
     result.summary = result._skipReason;
     return result;
   }
@@ -325,6 +336,11 @@ async function runSingleValidator(validatorId, projectPath, featureName, options
       return await validateReuse(projectPath, featureName, options);
     }
 
+    case 'code-budget': {
+      const { validateCodeBudget } = await import('./code-budget/code-budget-validator.js');
+      return await validateCodeBudget(projectPath, featureName, options);
+    }
+
     case 'nextjs-component': {
       const { validateNextComponentFiles } = await import('./nextjs/next-component-validator.js');
       return await validateNextComponentFiles(projectPath, options);

package/src/lib/validators/verify-runner.js

@@ -975,12 +975,15 @@ async function runValidatorsNode({ projectPath, featureName, rubricId, _runValid
 
   const durationMs = Date.now() - start;
 
+  const skipped = validationResult?.skipped ?? [];
+
   if (validationResult && validationResult._skipReason) {
     return {
       status: 'skip',
       score: null,
       action: null,
       ran: [],
+      skipped,
       reason: validationResult._skipReason,
       priorityFixes: [],
       durationMs,
@@ -995,6 +998,7 @@ async function runValidatorsNode({ projectPath, featureName, rubricId, _runValid
     score: scoreResult.score,
     action: scoreResult.action,
     ran: Object.keys(validationResult.results || {}),
+    skipped,
     reason: null,
     priorityFixes: scoreResult.priorityFixes || [],
     durationMs,
@@ -1002,6 +1006,13 @@ async function runValidatorsNode({ projectPath, featureName, rubricId, _runValid
 }
 
 function appendValidatorsFeedback(feedback, validatorsNode) {
+  // A validator that could not check (reuse without a graph) is invisible in a
+  // green score; when the whole node skipped, its own line already says why.
+  if (validatorsNode.status !== 'skip') {
+    for (const s of validatorsNode.skipped || []) {
+      if (s.unavailable) feedback.push(`[validators/${s.validator}] não rodou: ${s.reason}`);
+    }
+  }
   for (const fix of validatorsNode.priorityFixes || []) {
     const topIssue = fix.issues && fix.issues[0];
     if (!topIssue) continue;