pumuki 6.3.330 → 6.3.332

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/notifications/emitAuditSummaryNotification.ts

@@ -25,6 +25,14 @@ type AuditSummaryNotificationDependencies = {
   env: NodeJS.ProcessEnv;
 };
 
+type NotificationBlockingCause = {
+  code: string;
+  message: string;
+  ruleId?: string;
+  file?: string;
+  remediation?: string;
+};
+
 const defaultDependencies: AuditSummaryNotificationDependencies = {
   readEvidence,
   emitSystemNotification,
@@ -62,6 +70,38 @@ const normalizeNotificationStage = (stage: string): PumukiNotificationStage => {
   return 'PRE_COMMIT';
 };
 
+const normalizeRuleCode = (ruleId: string): string =>
+  ruleId
+    .replace(/^skills\./u, 'SKILLS_')
+    .replace(/[.-]/gu, '_')
+    .toUpperCase();
+
+const extractNestedSkillCauseFromWrapper = (cause: NotificationBlockingCause): NotificationBlockingCause => {
+  if (cause.ruleId?.startsWith('skills.') || cause.code.startsWith('SKILLS_')) {
+    return cause;
+  }
+  const nested = cause.message.match(/\bBlocking causes:\s*1\)\s*(skills\.[a-z0-9_.-]+)\s+(.+?)$/iu);
+  if (!nested?.[1]) {
+    return cause;
+  }
+  const ruleId = nested[1];
+  const nestedMessage = nested[2]?.trim() || 'Skill violada.';
+  return {
+    ...cause,
+    code: normalizeRuleCode(ruleId),
+    ruleId,
+    message: `rule=${ruleId} message=${nestedMessage}`,
+    remediation:
+      cause.remediation && !/corrige la violaci[oó]n indicada/i.test(cause.remediation)
+        ? cause.remediation
+        : 'Corrige la violación de la skill indicada en el fichero afectado y vuelve a intentar el commit.',
+  };
+};
+
+const normalizeNotificationBlockingCause = (
+  cause: NotificationBlockingCause
+): NotificationBlockingCause => extractNestedSkillCauseFromWrapper(cause);
+
 export const shouldEmitAuditSummaryNotificationForStage = (
   stage: AuditSummaryNotificationStage,
   env: NodeJS.ProcessEnv = process.env
@@ -87,13 +127,15 @@ export const toAuditSummaryEventFromEvidence = (
       causeCode: primaryCause.code,
       causeMessage: formatEvidenceBlockingCause(primaryCause),
       remediation: primaryCause.remediation ?? 'Corrige la violación indicada y vuelve a intentar el commit.',
-      blockingCauses: blockingCauses.map((cause) => ({
-        code: cause.code,
-        ruleId: cause.ruleId,
-        file: cause.file,
-        message: formatEvidenceBlockingCause(cause),
-        remediation: cause.remediation,
-      })),
+      blockingCauses: blockingCauses.map((cause) =>
+        normalizeNotificationBlockingCause({
+          code: cause.code,
+          ruleId: cause.ruleId,
+          file: cause.file,
+          message: formatEvidenceBlockingCause(cause),
+          remediation: cause.remediation,
+        })
+      ),
     };
   }
   return {
@@ -125,12 +167,14 @@ export const toAuditSummaryEventFromAiGate = (params: {
       causeCode: primaryViolation.code,
       causeMessage: primaryViolation.message,
       remediation: 'Corrige la violación indicada y vuelve a intentar el commit.',
-      blockingCauses: params.aiGateResult.violations.map((violation) => ({
-        code: violation.code,
-        ruleId: violation.code,
-        message: violation.message,
-        remediation: 'Corrige la violación indicada y vuelve a intentar el commit.',
-      })),
+      blockingCauses: params.aiGateResult.violations.map((violation) =>
+        normalizeNotificationBlockingCause({
+          code: violation.code,
+          ruleId: violation.code.startsWith('skills.') ? violation.code : undefined,
+          message: violation.message,
+          remediation: 'Corrige la violación indicada y vuelve a intentar el commit.',
+        })
+      ),
     };
   }
   return {
@@ -233,7 +277,7 @@ export const emitGateBlockedNotification = (
         causeMessage: params.causeMessage,
       }),
       remediation: params.remediation,
-      blockingCauses: params.blockingCauses,
+      blockingCauses: params.blockingCauses?.map(normalizeNotificationBlockingCause),
     },
     repoRoot: params.repoRoot,
   });

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "pumuki",
-  "version": "6.3.330",
+  "version": "6.3.332",
   "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;