pumuki 6.3.223 → 6.3.225

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

@@ -35,6 +35,8 @@ import {
   hasSwiftMixedTestingFrameworksUsage,
   hasSwiftLegacyXCTestImportUsage,
   hasSwiftModernizableXCTestSuiteUsage,
+  hasSwiftNonLazyScrollForEachUsage,
+  hasSwiftViewBodyObjectCreationUsage,
   hasSwiftAssumeIsolatedUsage,
   hasSwiftCoreDataLayerLeakUsage,
   hasSwiftSwiftDataLayerLeakUsage,
@@ -123,6 +125,72 @@ test('hasSwiftAnyViewUsage ignora comentarios, strings y coincidencias parciales
   assert.equal(hasSwiftAnyViewUsage(source), false);
 });
 
+test('hasSwiftNonLazyScrollForEachUsage detecta ScrollView con stack no lazy y preserva LazyVStack', () => {
+  const source = `
+struct FeedView: View {
+  let items: [Item]
+
+  var body: some View {
+    ScrollView {
+      VStack(spacing: 12) {
+        ForEach(items) { item in
+          FeedRow(item: item)
+        }
+      }
+    }
+  }
+}
+`;
+  const safe = `
+struct FeedView: View {
+  let items: [Item]
+
+  var body: some View {
+    ScrollView {
+      LazyVStack(spacing: 12) {
+        ForEach(items) { item in
+          FeedRow(item: item)
+        }
+      }
+    }
+    let sample = "ScrollView { VStack { ForEach(items) } }"
+    // ScrollView { VStack { ForEach(items) } }
+  }
+}
+`;
+
+  assert.equal(hasSwiftNonLazyScrollForEachUsage(source), true);
+  assert.equal(hasSwiftNonLazyScrollForEachUsage(safe), false);
+});
+
+test('hasSwiftViewBodyObjectCreationUsage detecta formatter creado en body y preserva dependencia externa', () => {
+  const source = `
+struct PriceView: View {
+  let amount: Decimal
+
+  var body: some View {
+    let formatter = NumberFormatter()
+    Text(formatter.string(from: amount as NSDecimalNumber) ?? "")
+  }
+}
+`;
+  const safe = `
+struct PriceView: View {
+  let formatter: NumberFormatter
+  let amount: Decimal
+
+  var body: some View {
+    Text(formatter.string(from: amount as NSDecimalNumber) ?? "")
+    let sample = "var body: some View { NumberFormatter() }"
+    // var body: some View { NumberFormatter() }
+  }
+}
+`;
+
+  assert.equal(hasSwiftViewBodyObjectCreationUsage(source), true);
+  assert.equal(hasSwiftViewBodyObjectCreationUsage(safe), false);
+});
+
 test('hasSwiftForceTryUsage detecta try! y descarta try?', () => {
   const positive = `
 func load() {

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

@@ -380,6 +380,22 @@ export const hasSwiftAnyViewUsage = (source: string): boolean => {
   });
 };
 
+export const hasSwiftNonLazyScrollForEachUsage = (source: string): boolean => {
+  const swiftSource = sanitizeSwiftSourceForMultilineRegex(source);
+  const nonLazyScrollableCollectionPattern =
+    /\bScrollView\s*(?:\([^)]*\))?\s*\{[\s\S]{0,2000}\b(?:VStack|HStack)\s*(?:\([^)]*\))?\s*\{[\s\S]{0,1200}\bForEach\s*\(/;
+
+  return nonLazyScrollableCollectionPattern.test(swiftSource);
+};
+
+export const hasSwiftViewBodyObjectCreationUsage = (source: string): boolean => {
+  const swiftSource = sanitizeSwiftSourceForMultilineRegex(source);
+  const viewBodyObjectCreationPattern =
+    /\bvar\s+body\s*:\s*some\s+View\s*\{[\s\S]{0,2400}\b(?:DateFormatter|NumberFormatter|RelativeDateTimeFormatter|ISO8601DateFormatter|ByteCountFormatter|MeasurementFormatter|DateComponentsFormatter)\s*\(/;
+
+  return viewBodyObjectCreationPattern.test(swiftSource);
+};
+
 export const hasSwiftDispatchQueueUsage = (source: string): boolean => {
   return scanCodeLikeSource(source, ({ source: swiftSource, index, current }) => {
     if (current !== 'D' || !hasIdentifierAt(swiftSource, index, 'DispatchQueue')) {

package/core/facts/extractHeuristicFacts.ts

@@ -685,6 +685,8 @@ const textDetectorRegistry: ReadonlyArray<TextDetectorRegistryEntry> = [
   { platform: 'ios', pathCheck: isIOSPresentationPath, excludePaths: [isSwiftTestPath], detect: TextIOS.hasSwiftExplicitColorStaticMemberUsage, ruleId: 'heuristics.ios.swiftui.explicit-color-static-member.ast', code: 'HEURISTICS_IOS_SWIFTUI_EXPLICIT_COLOR_STATIC_MEMBER_AST', message: 'AST heuristic detected Color.* static member usage where SwiftUI static member lookup may be preferred.' },
   { platform: 'ios', pathCheck: isIOSPresentationPath, excludePaths: [isSwiftTestPath], detect: TextIOS.hasSwiftClosureBasedViewBuilderContentUsage, ruleId: 'heuristics.ios.swiftui.closure-based-viewbuilder-content.ast', code: 'HEURISTICS_IOS_SWIFTUI_CLOSURE_BASED_VIEWBUILDER_CONTENT_AST', message: 'AST heuristic detected closure-based content storage; @ViewBuilder let content: Content remains the preferred SwiftUI container baseline.' },
   { platform: 'ios', pathCheck: isIOSPresentationPath, excludePaths: [isSwiftTestPath], detect: TextIOS.hasSwiftRedundantReactiveStateAssignmentUsage, ruleId: 'heuristics.ios.swiftui.redundant-reactive-state-assignment.ast', code: 'HEURISTICS_IOS_SWIFTUI_REDUNDANT_REACTIVE_STATE_ASSIGNMENT_AST', message: 'AST heuristic detected reactive state assignment without a value-change guard; check for value changes before assigning state in hot paths.' },
+  { platform: 'ios', pathCheck: isIOSPresentationPath, excludePaths: [isSwiftTestPath], detect: TextIOS.hasSwiftNonLazyScrollForEachUsage, ruleId: 'heuristics.ios.swiftui.non-lazy-scroll-foreach.ast', code: 'HEURISTICS_IOS_SWIFTUI_NON_LAZY_SCROLL_FOREACH_AST', message: 'AST heuristic detected ScrollView with a non-lazy stack feeding ForEach; LazyVStack/LazyHStack remain the preferred baseline for large scrollable collections.' },
+  { platform: 'ios', pathCheck: isIOSPresentationPath, excludePaths: [isSwiftTestPath], detect: TextIOS.hasSwiftViewBodyObjectCreationUsage, ruleId: 'heuristics.ios.swiftui.body-object-creation.ast', code: 'HEURISTICS_IOS_SWIFTUI_BODY_OBJECT_CREATION_AST', message: 'AST heuristic detected formatter object creation inside SwiftUI body; keep body simple and move expensive objects out of render paths.' },
   { platform: 'ios', pathCheck: isIOSSwiftPath, excludePaths: [isSwiftTestPath], detect: TextIOS.hasSwiftNavigationViewUsage, ruleId: 'heuristics.ios.navigation-view.ast', code: 'HEURISTICS_IOS_NAVIGATION_VIEW_AST', message: 'AST heuristic detected NavigationView usage.' },
   { platform: 'ios', pathCheck: isIOSSwiftPath, excludePaths: [isSwiftTestPath], detect: TextIOS.hasSwiftForegroundColorUsage, ruleId: 'heuristics.ios.foreground-color.ast', code: 'HEURISTICS_IOS_FOREGROUND_COLOR_AST', message: 'AST heuristic detected foregroundColor usage.' },
   { platform: 'ios', pathCheck: isIOSSwiftPath, excludePaths: [isSwiftTestPath], detect: TextIOS.hasSwiftCornerRadiusUsage, ruleId: 'heuristics.ios.corner-radius.ast', code: 'HEURISTICS_IOS_CORNER_RADIUS_AST', message: 'AST heuristic detected cornerRadius usage.' },

package/core/rules/presets/heuristics/ios.ts

@@ -911,6 +911,44 @@ export const iosRules: RuleSet = [
       code: 'HEURISTICS_IOS_SWIFTUI_REDUNDANT_REACTIVE_STATE_ASSIGNMENT_AST',
     },
   },
+  {
+    id: 'heuristics.ios.swiftui.non-lazy-scroll-foreach.ast',
+    description: 'Detects ScrollView content backed by non-lazy stacks and ForEach.',
+    severity: 'WARN',
+    platform: 'ios',
+    locked: true,
+    when: {
+      kind: 'Heuristic',
+      where: {
+        ruleId: 'heuristics.ios.swiftui.non-lazy-scroll-foreach.ast',
+      },
+    },
+    then: {
+      kind: 'Finding',
+      message:
+        'AST heuristic detected ScrollView with a non-lazy stack feeding ForEach; LazyVStack/LazyHStack remain the preferred baseline for large scrollable collections.',
+      code: 'HEURISTICS_IOS_SWIFTUI_NON_LAZY_SCROLL_FOREACH_AST',
+    },
+  },
+  {
+    id: 'heuristics.ios.swiftui.body-object-creation.ast',
+    description: 'Detects expensive formatter object creation inside SwiftUI body.',
+    severity: 'WARN',
+    platform: 'ios',
+    locked: true,
+    when: {
+      kind: 'Heuristic',
+      where: {
+        ruleId: 'heuristics.ios.swiftui.body-object-creation.ast',
+      },
+    },
+    then: {
+      kind: 'Finding',
+      message:
+        'AST heuristic detected formatter object creation inside SwiftUI body; keep body simple and move expensive objects out of render paths.',
+      code: 'HEURISTICS_IOS_SWIFTUI_BODY_OBJECT_CREATION_AST',
+    },
+  },
   {
     id: 'heuristics.ios.navigation-view.ast',
     description: 'Detects NavigationView usage in iOS production code.',

package/integrations/config/skillsDetectorRegistry.ts

@@ -254,6 +254,14 @@ const registryByRuleId: Record<string, SkillsDetectorBinding> = {
     heuristicDetector('ios.swiftui.redundant-reactive-state-assignment', [
       'heuristics.ios.swiftui.redundant-reactive-state-assignment.ast',
     ]),
+  'skills.ios.guideline.ios-swiftui-expert.use-lazyvstack-lazyhstack-for-large-lists':
+    heuristicDetector('ios.swiftui.non-lazy-scroll-foreach', [
+      'heuristics.ios.swiftui.non-lazy-scroll-foreach.ast',
+    ]),
+  'skills.ios.guideline.ios-swiftui-expert.no-object-creation-in-body':
+    heuristicDetector('ios.swiftui.body-object-creation', [
+      'heuristics.ios.swiftui.body-object-creation.ast',
+    ]),
   'skills.ios.no-scrollview-shows-indicators': heuristicDetector(
     'ios.scrollview-shows-indicators',
     ['heuristics.ios.scrollview-shows-indicators.ast']

package/integrations/config/skillsMarkdownRules.ts

@@ -360,6 +360,20 @@ const normalizeKnownRuleTarget = (
     ) {
       return 'skills.ios.guideline.ios-swiftui-expert.avoid-redundant-state-updates-in-onreceive-onchange-scroll-handlers';
     }
+    if (
+      (includes('lazyvstack') && includes('lazyhstack') && includes('large lists')) ||
+      (includes('lazyvstack') && includes('foreach') && includes('scrollview')) ||
+      (includes('lazyhstack') && includes('foreach') && includes('scrollview'))
+    ) {
+      return 'skills.ios.guideline.ios-swiftui-expert.use-lazyvstack-lazyhstack-for-large-lists';
+    }
+    if (
+      includes('no object creation in body') ||
+      (includes('object creation') && includes('body')) ||
+      (includes('body kept simple') && includes('pure'))
+    ) {
+      return 'skills.ios.guideline.ios-swiftui-expert.no-object-creation-in-body';
+    }
     if (
       includes('scrollindicators hidden') ||
       includes('scroll indicators hidden') ||

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "pumuki",
-  "version": "6.3.223",
+  "version": "6.3.225",
   "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/skills.lock.json

@@ -1,7 +1,7 @@
 {
   "version": "1.0",
   "compilerVersion": "1.0.0",
-  "generatedAt": "2026-05-13T16:08:30.839Z",
+  "generatedAt": "2026-05-13T16:19:49.576Z",
   "bundles": [
     {
       "name": "android-guidelines",
@@ -8632,7 +8632,7 @@
       "name": "ios-swiftui-expert-guidelines",
       "version": "1.0.0",
       "source": "file:vendor/skills/swiftui-expert-skill/SKILL.md",
-      "hash": "6570119e6399677bda8b010b801f6397859f7a066eeecf1a33fcfe9ca97348fe",
+      "hash": "2acbd66e86e9e809f0c4bdb5d215618096f5773e359085e3a4dc707b83c1ebb1",
       "rules": [
         {
           "id": "skills.ios.guideline.ios-swiftui-expert.always-mark-state-and-stateobject-as-private-makes-dependencies-clear",
@@ -8787,7 +8787,7 @@
           "sourcePath": "vendor/skills/swiftui-expert-skill/SKILL.md",
           "confidence": "MEDIUM",
           "locked": true,
-          "evaluationMode": "DECLARATIVE",
+          "evaluationMode": "AUTO",
           "origin": "core"
         },
         {