pumuki 6.3.331 → 6.3.333

package/integrations/evidence/blockingCauses.ts

@@ -14,6 +14,13 @@ export type EvidenceBlockingCause = {
 const isBlockingSeverity = (severity: string): boolean =>
   severity.toUpperCase() === 'CRITICAL' || severity.toUpperCase() === 'ERROR';
 
+const isNonBlockingSwiftTestingMigrationWarning = (params: {
+  ruleId?: string;
+  severity: string;
+}): boolean =>
+  params.ruleId === 'skills.ios.prefer-swift-testing'
+  && params.severity.toUpperCase() === 'WARN';
+
 const toFindingBlockingCause = (finding: SnapshotFinding): EvidenceBlockingCause => ({
   ruleId: finding.ruleId,
   code: finding.code,
@@ -56,7 +63,14 @@ export const extractEvidenceBlockingCauses = (
 
   for (const finding of evidence.snapshot.findings) {
     const explicitlyBlocking = finding.blocking === true;
-    if (finding.blocking === false || (!explicitlyBlocking && !isBlockingSeverity(finding.severity))) {
+    if (
+      finding.blocking === false
+      || isNonBlockingSwiftTestingMigrationWarning({
+        ruleId: finding.ruleId,
+        severity: finding.severity,
+      })
+      || (!explicitlyBlocking && !isBlockingSeverity(finding.severity))
+    ) {
       continue;
     }
     const cause = toFindingBlockingCause(finding);
@@ -69,9 +83,14 @@ export const extractEvidenceBlockingCauses = (
 
   for (const violation of evidence.ai_gate.violations) {
     const explicitlyBlocking = violation.blocking === true;
+    const severity = resolveViolationSeverity(violation);
     if (
       violation.blocking === false ||
-      (!explicitlyBlocking && !isBlockingSeverity(resolveViolationSeverity(violation)))
+      isNonBlockingSwiftTestingMigrationWarning({
+        ruleId: violation.ruleId,
+        severity,
+      }) ||
+      (!explicitlyBlocking && !isBlockingSeverity(severity))
     ) {
       continue;
     }

package/integrations/gate/evaluateAiGate.ts

@@ -486,6 +486,28 @@ const resolvePreWriteCriticalSkillsRules = (params: {
   return ruleIds;
 };
 
+const hasRuleCoverage = (
+  coverage: NonNullable<Extract<EvidenceReadResult, { kind: 'valid' }>['evidence']['snapshot']['rules_coverage']>,
+  ruleId: string
+): boolean =>
+  coverage.active_rule_ids.includes(ruleId) && coverage.evaluated_rule_ids.includes(ruleId);
+
+const hasEquivalentCriticalRuleCoverage = (
+  coverage: NonNullable<Extract<EvidenceReadResult, { kind: 'valid' }>['evidence']['snapshot']['rules_coverage']>,
+  ruleId: string
+): boolean => {
+  if (ruleId !== 'skills.ios.critical-test-quality') {
+    return false;
+  }
+  return hasRuleCoverage(coverage, 'skills.ios.prefer-swift-testing');
+};
+
+const hasRequiredCriticalRuleCoverage = (
+  coverage: NonNullable<Extract<EvidenceReadResult, { kind: 'valid' }>['evidence']['snapshot']['rules_coverage']>,
+  ruleId: string
+): boolean =>
+  hasRuleCoverage(coverage, ruleId) || hasEquivalentCriticalRuleCoverage(coverage, ruleId);
+
 const isPlatformPath = (platform: PreWriteSkillsPlatform, filePath: string): boolean => {
   const normalized = normalizeChangedPath(filePath).toLowerCase();
   if (platform === 'ios') {
@@ -757,9 +779,7 @@ const collectPreWritePlatformSkillsViolations = (params: {
       continue;
     }
     const missingCriticalRuleIds = requiredCriticalRuleIds.filter((ruleId) => {
-      const hasActive = params.coverage.active_rule_ids.includes(ruleId);
-      const hasEvaluated = params.coverage.evaluated_rule_ids.includes(ruleId);
-      return !hasActive || !hasEvaluated;
+      return !hasRequiredCriticalRuleCoverage(params.coverage, ruleId);
     });
     if (missingCriticalRuleIds.length === 0) {
       continue;
@@ -1051,9 +1071,7 @@ const toSkillsContractAssessment = (params: {
     );
     const missingCriticalRuleIds = coverage
       ? requiredCriticalRuleIds.filter((ruleId) => {
-          const hasActive = coverage.active_rule_ids.includes(ruleId);
-          const hasEvaluated = coverage.evaluated_rule_ids.includes(ruleId);
-          return !hasActive || !hasEvaluated;
+          return !hasRequiredCriticalRuleCoverage(coverage, ruleId);
         })
       : [...requiredCriticalRuleIds];
     const transversalCriticalCovered =

package/integrations/git/runPlatformGate.ts

@@ -641,7 +641,7 @@ const hasTrackForMemoryLeaksPattern = (content: string): boolean =>
 const IOS_CRITICAL_TEST_QUALITY_RULE_ID = 'skills.ios.critical-test-quality';
 
 const hasIosTestQualityScope = (facts: ReadonlyArray<Fact>): boolean =>
-  collectIosTestFileContents(facts).some((testFile) => isXCTestSource(testFile.content));
+  collectIosTestFileContents(facts).length > 0;
 
 const appendUniqueRuleId = (
   ruleIds: ReadonlyArray<string>,

package/integrations/tdd/enforcement.ts

@@ -3,6 +3,7 @@ import { isAbsolute, resolve } from 'node:path';
 import type { Fact } from '../../core/facts/Fact';
 import type { Finding } from '../../core/gate/Finding';
 import { readTddBddEvidence } from './contract';
+import type { TddSlice } from './contract';
 import { classifyTddBddScope } from './scope';
 import type { TddBddSnapshot } from './types';
 import { resolveActiveTddBddWaiver } from './waiver';
@@ -78,6 +79,14 @@ const isTimelineOrdered = (timestamps: ReadonlyArray<string | undefined>): boole
   return true;
 };
 
+const selectEnforcedEvidenceSlices = (slices: ReadonlyArray<TddSlice>): ReadonlyArray<TddSlice> => {
+  if (slices.length <= 1) {
+    return slices;
+  }
+  const latestSlice = slices[slices.length - 1];
+  return latestSlice ? [latestSlice] : [];
+};
+
 export const enforceTddBddPolicy = (params: {
   facts: ReadonlyArray<Fact>;
   repoRoot: string;
@@ -222,7 +231,9 @@ export const enforceTddBddPolicy = (params: {
     verify: 'missing',
   };
 
-  if (evidenceRead.evidence.slices.length === 0) {
+  const enforcedSlices = selectEnforcedEvidenceSlices(evidenceRead.evidence.slices);
+
+  if (enforcedSlices.length === 0) {
     sliceFindings.push(
       buildFinding({
         ruleId: 'generic_tdd_vertical_required',
@@ -233,7 +244,7 @@ export const enforceTddBddPolicy = (params: {
     );
   }
 
-  for (const slice of evidenceRead.evidence.slices) {
+  for (const slice of enforcedSlices) {
     const sliceErrorsBefore = sliceFindings.length;
 
     if (seenSliceIds.has(slice.id)) {
@@ -355,7 +366,7 @@ export const enforceTddBddPolicy = (params: {
     }
   }
 
-  const invalidSlices = Math.max(0, evidenceRead.evidence.slices.length - validSlices);
+  const invalidSlices = Math.max(0, enforcedSlices.length - validSlices);
   const hasBlockingFindings = sliceFindings.some(
     (finding) => finding.severity === 'ERROR' || finding.severity === 'CRITICAL'
   );
@@ -369,7 +380,7 @@ export const enforceTddBddPolicy = (params: {
         ...baseSnapshot.evidence,
         state: 'valid',
         version: evidenceRead.evidence.version,
-        slices_total: evidenceRead.evidence.slices.length,
+        slices_total: enforcedSlices.length,
         slices_valid: validSlices,
         slices_invalid: invalidSlices,
         integrity_ok: evidenceRead.integrity.valid,

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "pumuki",
-  "version": "6.3.331",
+  "version": "6.3.333",
   "description": "Enterprise-grade AST Intelligence System with multi-platform support (iOS, Android, Backend, Frontend) and Feature-First + DDD + Clean Architecture enforcement. Includes dynamic violations API for intelligent querying.",
   "main": "index.js",
   "bin": {

package/scripts/framework-menu-system-notifications-macos-dialog-payload.ts

@@ -248,6 +248,7 @@ const formatPlatformName = (platform: string | null): string =>
 const extractSkillsContractPlatform = (message: string): string | null =>
   message.match(/\bplatforms?=([a-z0-9_,.-]+)/i)?.[1]?.split(',')[0]?.trim()
   ?? message.match(/\b(ios|android|frontend|backend)\(missing_critical_rule_ids=/i)?.[1]?.trim()
+  ?? message.match(/\bskills\.(ios|android|frontend|backend)\./i)?.[1]?.trim()
   ?? message.match(/\bfor\s+(ios|android|frontend|backend)\s*:/i)?.[1]?.trim()
   ?? null;
 
@@ -261,6 +262,9 @@ const extractSkillsContractRules = (message: string): string | null => {
 const SKILLS_CONTRACT_REMEDIATION =
   'Actualiza o reconcilia el contrato de skills de Pumuki y vuelve a intentar el commit. Si ya estás en la última versión, es un bug del motor de Pumuki y debe corregirse allí antes de cerrar el commit.';
 
+const SKILLS_CONTRACT_DIALOG_REMEDIATION =
+  'Corrige el contrato de skills indicado arriba y vuelve a intentar el commit.';
+
 const WORKTREE_HYGIENE_REMEDIATION =
   'Reduce el worktree pendiente a un slice atómico: stagea solo la tarea activa o guarda el resto en stash nombrado, y reejecuta PRE_WRITE.';
 
@@ -409,7 +413,7 @@ export const buildBlockedDialogPayload = (params: {
     : hasOnlyBddScenarioMissingCauses(params.event.blockingCauses)
     ? 'Crea el fichero .feature esperado para la slice activa o corrige la referencia de tarea. Después vuelve a intentar el commit.'
     : hasOnlySkillsContractCauses(params.event.blockingCauses)
-    ? SKILLS_CONTRACT_REMEDIATION
+    ? SKILLS_CONTRACT_DIALOG_REMEDIATION
     : params.event.blockingCauses && params.event.blockingCauses.length > 0
     ? 'Corrige las violaciones listadas y vuelve a intentar el commit.'
     : resolveBlockedRemediation(params.event, causeCode);

package/scripts/framework-menu-system-notifications-payloads-blocked.ts

@@ -67,6 +67,7 @@ const formatPlatformName = (platform: string | null): string =>
 const extractSkillsContractPlatform = (message: string): string | null =>
   message.match(/\bplatforms?=([a-z0-9_,.-]+)/i)?.[1]?.split(',')[0]?.trim()
   ?? message.match(/\b(ios|android|frontend|backend)\(missing_critical_rule_ids=/i)?.[1]?.trim()
+  ?? message.match(/\bskills\.(ios|android|frontend|backend)\./i)?.[1]?.trim()
   ?? message.match(/\bfor\s+(ios|android|frontend|backend)\s*:/i)?.[1]?.trim()
   ?? null;