pumuki 6.3.281 → 6.3.283

package/CHANGELOG.md

@@ -1,5 +1,13 @@
 # Changelog
 
+## [6.3.283] - 2026-05-19
+
+- `PUMUKI-INC-150`: brownfield XCTest specs that satisfy `makeSUT()` and `trackForMemoryLeaks()` are also exempt from Swift Testing migration assertion findings (`skills.ios.no-xctassert` / `skills.ios.no-xctunwrap`). These rules remain active for modern Swift Testing suites and XCTest specs without an explicit brownfield quality contract.
+
+## [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.

package/VERSION

@@ -1 +1 @@
-v6.3.281
+v6.3.283

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

@@ -2030,6 +2030,8 @@ final class BuyerOnboardingStringsTests: XCTestCase {
   func testTitle() {
     let sut = makeSUT()
     XCTAssertEqual(sut.title, "Comprar cerca")
+    let required = try XCTUnwrap(sut.title)
+    XCTAssertFalse(required.isEmpty)
   }
 
   private func makeSUT() -> BuyerOnboardingStrings {
@@ -2041,7 +2043,11 @@ final class BuyerOnboardingStringsTests: XCTestCase {
 `;
 
   assert.equal(hasSwiftModernizableXCTestSuiteUsage(brownfieldSpec), false);
+  assert.equal(hasSwiftXCTestAssertionUsage(brownfieldSpec), false);
+  assert.equal(hasSwiftXCTUnwrapUsage(brownfieldSpec), false);
   assert.deepEqual(collectSwiftModernizableXCTestSuiteLines(brownfieldSpec), []);
+  assert.deepEqual(collectSwiftXCTestAssertionLines(brownfieldSpec), []);
+  assert.deepEqual(collectSwiftXCTUnwrapLines(brownfieldSpec), []);
 });
 
 test('hasSwiftMixedTestingFrameworksUsage detecta mezcla XCTestCase y Testing/@Test', () => {

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

@@ -1697,6 +1697,10 @@ export const collectSwiftQuickNimbleLines = (source: string): readonly number[]
 };
 
 export const hasSwiftXCTestAssertionUsage = (source: string): boolean => {
+  if (hasSwiftBrownfieldXCTestQualityPattern(source)) {
+    return false;
+  }
+
   return (
     collectSwiftRegexLines(source, /\bXCTAssert[A-Za-z0-9_]*\s*\(/).length > 0 ||
     collectSwiftRegexLines(source, /\bXCTFail\s*\(/).length > 0
@@ -1715,10 +1719,18 @@ export const collectSwiftXCTestAssertionLines = (source: string): readonly numbe
 };
 
 export const hasSwiftXCTUnwrapUsage = (source: string): boolean => {
+  if (hasSwiftBrownfieldXCTestQualityPattern(source)) {
+    return false;
+  }
+
   return collectSwiftRegexLines(source, /\bXCTUnwrap\s*\(/).length > 0;
 };
 
 export const collectSwiftXCTUnwrapLines = (source: string): readonly number[] => {
+  if (!hasSwiftXCTUnwrapUsage(source)) {
+    return [];
+  }
+
   return sortedUniqueLines(collectSwiftRegexLines(source, /\bXCTUnwrap\s*\(/));
 };
 

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.281",
+  "version": "6.3.283",
   "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": {