pumuki 6.3.159 → 6.3.161

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

@@ -564,11 +564,34 @@ import Testing
 struct LoginModernTests {
   @Test func login() async {}
 }
+`;
+  const mixedWithReexportedTesting = `
+import XCTest
+
+final class LoginTests: XCTestCase {
+  func testLegacyLogin() {}
+}
+
+struct LoginModernTests {
+  @Test func login() async {}
+}
+`;
+  const ignoredCommentsAndStrings = `
+import XCTest
+
+final class LoginTests: XCTestCase {
+  func testLegacyLogin() {
+    let text = "@Test func login() async {}"
+    // @Suite
+  }
+}
 `;
 
   assert.equal(hasSwiftMixedTestingFrameworksUsage(mixedSuite), true);
+  assert.equal(hasSwiftMixedTestingFrameworksUsage(mixedWithReexportedTesting), true);
   assert.equal(hasSwiftMixedTestingFrameworksUsage(legacyOnly), false);
   assert.equal(hasSwiftMixedTestingFrameworksUsage(modernOnly), false);
+  assert.equal(hasSwiftMixedTestingFrameworksUsage(ignoredCommentsAndStrings), false);
 });
 
 test('hasSwiftXCTestAssertionUsage detecta XCTAssert y XCTFail reales', () => {
@@ -684,26 +707,45 @@ await fulfillment(of: [expectation], timeout: 1)
   assert.equal(hasSwiftWaitForExpectationsUsage(modernWait), false);
 });
 
-test('hasSwiftLegacyExpectationDescriptionUsage detecta expectation(description:) sin flujo moderno', () => {
-  const legacyExpectation = `
+test('hasSwiftLegacyExpectationDescriptionUsage detecta expectation(description:) en tests async sin flujo moderno', () => {
+  const legacyAsyncExpectation = `
+func testLegacyAsync() async {
 let expectation = expectation(description: "Done")
 doWork { expectation.fulfill() }
-waitForExpectations(timeout: 1)
+}
 `;
-  const modernExpectation = `
+  const legacySyncExpectation = `
+func testLegacySync() {
+let expectation = expectation(description: "Done")
+doWork { expectation.fulfill() }
+}
+`;
+  const modernAsyncExpectation = `
+func testModernAsync() async {
 let expectation = expectation(description: "Done")
 doWork { expectation.fulfill() }
 await fulfillment(of: [expectation], timeout: 1)
+}
 `;
-  const confirmationOnly = `
+  const confirmationFlow = `
+func testConfirmation() async {
 await confirmation("Done") { confirm in
   await doWork { confirm() }
 }
+}
+`;
+  const commentedExpectation = `
+func testModernAsync() async {
+// let expectation = expectation(description: "Done")
+await confirmation("Done") { confirm in confirm() }
+}
 `;
 
-  assert.equal(hasSwiftLegacyExpectationDescriptionUsage(legacyExpectation), true);
-  assert.equal(hasSwiftLegacyExpectationDescriptionUsage(modernExpectation), false);
-  assert.equal(hasSwiftLegacyExpectationDescriptionUsage(confirmationOnly), false);
+  assert.equal(hasSwiftLegacyExpectationDescriptionUsage(legacyAsyncExpectation), true);
+  assert.equal(hasSwiftLegacyExpectationDescriptionUsage(legacySyncExpectation), false);
+  assert.equal(hasSwiftLegacyExpectationDescriptionUsage(modernAsyncExpectation), false);
+  assert.equal(hasSwiftLegacyExpectationDescriptionUsage(confirmationFlow), false);
+  assert.equal(hasSwiftLegacyExpectationDescriptionUsage(commentedExpectation), false);
 });
 
 test('hasSwiftNSManagedObjectBoundaryUsage detecta boundaries con NSManagedObject y excluye IDs o subclases', () => {

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

@@ -788,6 +788,10 @@ const hasSwiftTestingSuiteAttributeUsage = (source: string): boolean => {
   return hasSwiftSanitizedRegexMatch(source, /\B@(?:Test|Suite)\b/);
 };
 
+const hasSwiftTestingFrameworkMarkerUsage = (source: string): boolean => {
+  return hasSwiftTestingImportUsage(source) || hasSwiftTestingSuiteAttributeUsage(source);
+};
+
 const hasSwiftXCTestCaseSubclassUsage = (source: string): boolean => {
   return hasSwiftSanitizedRegexMatch(
     source,
@@ -869,11 +873,7 @@ export const hasSwiftModernizableXCTestSuiteUsage = (source: string): boolean =>
 };
 
 export const hasSwiftMixedTestingFrameworksUsage = (source: string): boolean => {
-  if (!hasSwiftXCTestImportUsage(source) || !hasSwiftXCTestCaseSubclassUsage(source)) {
-    return false;
-  }
-
-  return hasSwiftTestingImportUsage(source) || hasSwiftTestingSuiteAttributeUsage(source);
+  return hasSwiftXCTestCaseSubclassUsage(source) && hasSwiftTestingFrameworkMarkerUsage(source);
 };
 
 export const hasSwiftXCTestAssertionUsage = (source: string): boolean => {
@@ -923,20 +923,15 @@ export const hasSwiftWaitForExpectationsUsage = (source: string): boolean => {
 };
 
 export const hasSwiftLegacyExpectationDescriptionUsage = (source: string): boolean => {
-  const hasLegacyExpectation = collectSwiftRegexLines(
-    source,
-    /\bexpectation\s*\(\s*description\s*:/
-  ).length > 0;
-
-  if (!hasLegacyExpectation) {
-    return false;
-  }
-
-  if (hasSwiftAwaitFulfillmentUsage(source) || hasSwiftConfirmationUsage(source)) {
-    return false;
-  }
-
-  return true;
+  return collectSwiftFunctionDeclarations(source).some((declaration) => {
+    if (!/\basync\b/.test(declaration.signature)) {
+      return false;
+    }
+    if (!hasSwiftSanitizedRegexMatch(declaration.body, /\bexpectation\s*\(\s*description\s*:/)) {
+      return false;
+    }
+    return !hasSwiftAwaitFulfillmentUsage(declaration.body) && !hasSwiftConfirmationUsage(declaration.body);
+  });
 };
 
 export const hasSwiftNSManagedObjectBoundaryUsage = (source: string): boolean => {

package/package.json

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