pumuki 6.3.280 → 6.3.282

package/CHANGELOG.md

@@ -1,5 +1,13 @@
 # Changelog
 
+## [6.3.282] - 2026-05-19
+
+- `PUMUKI-INC-150`: PRE_WRITE evidence coverage now materializes `skills.ios.critical-test-quality` as active and evaluated for XCTest test scopes that satisfy `makeSUT()` and `trackForMemoryLeaks()`. This removes the false `EVIDENCE_PLATFORM_CRITICAL_SKILLS_RULES_MISSING` block after the brownfield XCTest spec has no quality findings.
+
+## [6.3.281] - 2026-05-19
+
+- `PUMUKI-INC-150`: `skills.ios.prefer-swift-testing` no longer blocks brownfield XCTest specs that already follow the repo quality contract with `makeSUT()` and `trackForMemoryLeaks()`. Swift Testing migration findings remain active for modernizable XCTest suites without an explicit brownfield quality pattern.
+
 ## [6.3.280] - 2026-05-18
 
 - `PUMUKI-INC-149`: PRE_WRITE validation no longer reports the generic `EVIDENCE_GATE_BLOCKED` symptom when a concrete actionable cause has already been resolved from `next_action`. JSON output, panels and blocked notifications now keep the real cause only, avoiding contradictory "blocked by stale evidence" guidance when the remediation is an atomic-slice/worktree action.

package/VERSION

@@ -1 +1 @@
-v6.3.277
+v6.3.282

package/core/facts/detectors/text/ios.test.ts

@@ -1775,6 +1775,27 @@ final class SyncTests: XCTestCase {
   assert.equal(hasSwiftLegacyXCTestImportUsage(performanceTest), false);
 });
 
+test('hasSwiftLegacyXCTestImportUsage excluye specs XCTest brownfield con calidad makeSUT y memory tracking', () => {
+  const brownfieldSpec = `
+import XCTest
+
+final class BuyerOnboardingStringsTests: XCTestCase {
+  func testTitle() {
+    let sut = makeSUT()
+    XCTAssertEqual(sut.title, "Comprar cerca")
+  }
+
+  private func makeSUT() -> BuyerOnboardingStrings {
+    let sut = BuyerOnboardingStrings()
+    trackForMemoryLeaks(sut)
+    return sut
+  }
+}
+`;
+
+  assert.equal(hasSwiftLegacyXCTestImportUsage(brownfieldSpec), false);
+});
+
 test('hasSwiftLegacySwiftUiObservableWrapperUsage detecta @StateObject/@ObservedObject legacy', () => {
   const legacyWrapper = `
 @StateObject private var viewModel = LegacyViewModel()
@@ -2001,6 +2022,28 @@ final class LoginUITests: XCTestCase {
   assert.deepEqual(collectSwiftModernizableXCTestSuiteLines(mixedSuite), []);
 });
 
+test('hasSwiftModernizableXCTestSuiteUsage no bloquea specs XCTest brownfield con makeSUT y trackForMemoryLeaks', () => {
+  const brownfieldSpec = `
+import XCTest
+
+final class BuyerOnboardingStringsTests: XCTestCase {
+  func testTitle() {
+    let sut = makeSUT()
+    XCTAssertEqual(sut.title, "Comprar cerca")
+  }
+
+  private func makeSUT() -> BuyerOnboardingStrings {
+    let sut = BuyerOnboardingStrings()
+    trackForMemoryLeaks(sut)
+    return sut
+  }
+}
+`;
+
+  assert.equal(hasSwiftModernizableXCTestSuiteUsage(brownfieldSpec), false);
+  assert.deepEqual(collectSwiftModernizableXCTestSuiteLines(brownfieldSpec), []);
+});
+
 test('hasSwiftMixedTestingFrameworksUsage detecta mezcla XCTestCase y Testing/@Test', () => {
   const mixedSuite = `
 import XCTest

package/core/facts/detectors/text/ios.ts

@@ -1606,6 +1606,11 @@ const hasSwiftLegacyXCTestMethodUsage = (source: string): boolean => {
     .length > 0;
 };
 
+const hasSwiftBrownfieldXCTestQualityPattern = (source: string): boolean => {
+  const sanitized = sanitizeSwiftSourceForMultilineRegex(source);
+  return /\bmakeSUT\s*\(/.test(sanitized) && /\btrackForMemoryLeaks\s*\(/.test(sanitized);
+};
+
 export const hasSwiftLegacyXCTestImportUsage = (source: string): boolean => {
   if (!hasSwiftXCTestImportUsage(source)) {
     return false;
@@ -1615,6 +1620,10 @@ export const hasSwiftLegacyXCTestImportUsage = (source: string): boolean => {
     return false;
   }
 
+  if (hasSwiftBrownfieldXCTestQualityPattern(source)) {
+    return false;
+  }
+
   return true;
 };
 

package/integrations/git/runPlatformGate.ts

@@ -485,6 +485,21 @@ const hasMakeSUTPattern = (content: string): boolean => /\bmakeSUT\s*\(/.test(co
 const hasTrackForMemoryLeaksPattern = (content: string): boolean =>
   /\btrackForMemoryLeaks\s*\(/.test(content);
 
+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));
+
+const appendUniqueRuleId = (
+  ruleIds: ReadonlyArray<string>,
+  ruleId: string
+): readonly string[] => {
+  if (ruleIds.includes(ruleId)) {
+    return [...ruleIds];
+  }
+  return [...ruleIds, ruleId].sort((left, right) => left.localeCompare(right));
+};
+
 const toIosTestsQualityBlockingFinding = (params: {
   stage: 'PRE_COMMIT' | 'PRE_PUSH' | 'CI';
   facts: ReadonlyArray<Fact>;
@@ -1187,14 +1202,22 @@ export async function runPlatformGate(params: {
     }
   }
   const degradedModeBlocks = params.policyTrace?.degraded?.action === 'block';
+  const coverageActiveRuleIds =
+    coverage && hasIosTestQualityScope(facts)
+      ? appendUniqueRuleId(coverage.activeRuleIds, IOS_CRITICAL_TEST_QUALITY_RULE_ID)
+      : coverage?.activeRuleIds;
+  const coverageEvaluatedRuleIds =
+    coverage && hasIosTestQualityScope(facts)
+      ? appendUniqueRuleId(coverage.evaluatedRuleIds, IOS_CRITICAL_TEST_QUALITY_RULE_ID)
+      : coverage?.evaluatedRuleIds;
   const rulesCoverage = coverage
     ? {
       stage: params.policy.stage,
       contract: skillsRuleSet.registryCoverage?.contract ?? 'AUTO_RUNTIME_RULES_FOR_STAGE',
       scope_note:
         'rules_coverage reports AUTO runtime rules applicable to this stage; it does not claim full DECLARATIVE registry execution.',
-      active_rule_ids: [...coverage.activeRuleIds],
-      evaluated_rule_ids: [...coverage.evaluatedRuleIds],
+      active_rule_ids: [...(coverageActiveRuleIds ?? coverage.activeRuleIds)],
+      evaluated_rule_ids: [...(coverageEvaluatedRuleIds ?? coverage.evaluatedRuleIds)],
       matched_rule_ids: [...coverage.matchedRuleIds],
       unevaluated_rule_ids: [...coverage.unevaluatedRuleIds],
       ...(skillsRuleSet.registryCoverage
@@ -1214,8 +1237,8 @@ export async function runPlatformGate(params: {
         }
         : {}),
       counts: {
-        active: coverage.activeRuleIds.length,
-        evaluated: coverage.evaluatedRuleIds.length,
+        active: (coverageActiveRuleIds ?? coverage.activeRuleIds).length,
+        evaluated: (coverageEvaluatedRuleIds ?? coverage.evaluatedRuleIds).length,
         matched: coverage.matchedRuleIds.length,
         unevaluated: coverage.unevaluatedRuleIds.length,
         ...(skillsRuleSet.registryCoverage
@@ -1235,9 +1258,14 @@ export async function runPlatformGate(params: {
           : {}),
       },
       coverage_ratio:
-        coverage.activeRuleIds.length === 0
+        (coverageActiveRuleIds ?? coverage.activeRuleIds).length === 0
           ? 1
-          : Number((coverage.evaluatedRuleIds.length / coverage.activeRuleIds.length).toFixed(6)),
+          : Number(
+              (
+                (coverageEvaluatedRuleIds ?? coverage.evaluatedRuleIds).length /
+                (coverageActiveRuleIds ?? coverage.activeRuleIds).length
+              ).toFixed(6)
+            ),
     }
     : createEmptySnapshotRulesCoverage(params.policy.stage);
   const brownfieldHotspotFindings = dependencies.evaluateBrownfieldHotspotFindings({

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "pumuki",
-  "version": "6.3.280",
+  "version": "6.3.282",
   "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": {