pumuki 6.3.244 → 6.3.246

package/CHANGELOG.md

@@ -6,6 +6,18 @@ This project follows [Semantic Versioning](https://semver.org/spec/v2.0.0.html).
 
 ## [Unreleased]
 
+## [6.3.246] - 2026-05-14
+
+### Added
+
+- **SwiftUI view-identity parity:** `skills.ios.guideline.ios-swiftui-expert.prefer-modifiers-over-conditional-views-for-state-changes-maintains-vi` now maps to a conservative AUTO heuristic for `if/else` branches that rebuild the same SwiftUI View type for state-only visual changes, preserving legitimate conditional composition.
+
+## [6.3.245] - 2026-05-14
+
+### Added
+
+- **SwiftUI props fan-out parity:** `skills.ios.guideline.ios-swiftui-expert.pass-only-needed-values-to-views-avoid-large-config-or-context-objects` now maps to a scoped AUTO heuristic that detects SwiftUI Views storing broad `Config`/`Configuration`/`Context` properties, preserving narrow value props as the clean baseline.
+
 ## [6.3.244] - 2026-05-14
 
 ### Added

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

@@ -73,6 +73,8 @@ import {
   hasSwiftJSONSerializationUsage,
   hasSwiftExplicitColorStaticMemberUsage,
   hasSwiftClosureBasedViewBuilderContentUsage,
+  hasSwiftLargeConfigContextViewPropertyUsage,
+  hasSwiftUiConditionalSameViewIdentityUsage,
   hasSwiftRedundantReactiveStateAssignmentUsage,
   hasSwiftInlineForEachTransformUsage,
   hasSwiftStringFormatUsage,
@@ -1304,6 +1306,52 @@ let ignored = "let content: () -> Content"
   assert.equal(hasSwiftClosureBasedViewBuilderContentUsage(safe), false);
 });
 
+test('hasSwiftLargeConfigContextViewPropertyUsage detecta config/context grandes en SwiftUI View', () => {
+  const source = `
+struct CheckoutSummaryView: View {
+  let config: CheckoutSummaryConfiguration
+  let title: String
+
+  var body: some View {
+    VStack {
+      Text(config.title)
+    }
+  }
+}
+
+struct StoreMapView: View {
+  var context: StoreMapContext
+
+  var body: some View {
+    Text(context.title)
+  }
+}
+`;
+  const safe = `
+struct CheckoutSummaryView: View {
+  let title: String
+  let subtitle: String
+
+  var body: some View {
+    VStack {
+      Text(title)
+      Text(subtitle)
+    }
+  }
+}
+
+final class CheckoutPresenter {
+  let context: CheckoutContext
+}
+
+let ignored = "struct IgnoredView: View { let config: AppConfig var body: some View { Text(config.title) } }"
+// struct IgnoredView: View { let config: AppConfig var body: some View { Text(config.title) } }
+`;
+
+  assert.equal(hasSwiftLargeConfigContextViewPropertyUsage(source), true);
+  assert.equal(hasSwiftLargeConfigContextViewPropertyUsage(safe), false);
+});
+
 test('hasSwiftRedundantReactiveStateAssignmentUsage detecta asignaciones reactivas redundantes y preserva guard de cambio', () => {
   const source = `
 struct SearchView: View {
@@ -2024,3 +2072,43 @@ final class PumukiLspIosCanaryPremiumDiscount: PumukiLspIosCanaryDiscountApplyin
   assert.match(match.impact, /sustitución|precondiciones|regresiones/i);
   assert.match(match.expected_fix, /contrato base|adaptador|estrategia/i);
 });
+
+test('hasSwiftUiConditionalSameViewIdentityUsage detecta ramas que reconstruyen la misma View por estado visual', () => {
+  const source = `
+struct StatusBadge: View {
+  let isSelected: Bool
+
+  var body: some View {
+    if isSelected {
+      Text("Active")
+        .foregroundStyle(.green)
+    } else {
+      Text("Active")
+        .foregroundStyle(.secondary)
+    }
+  }
+}
+`;
+  const safe = `
+struct StatusBadge: View {
+  let isSelected: Bool
+
+  var body: some View {
+    Text("Active")
+      .foregroundStyle(isSelected ? .green : .secondary)
+
+    if isSelected {
+      SuccessBadge()
+    } else {
+      EmptyView()
+    }
+
+    let sample = "if isSelected { Text(\"Active\") } else { Text(\"Active\") }"
+    // if isSelected { Text("Active") } else { Text("Active") }
+  }
+}
+`;
+
+  assert.equal(hasSwiftUiConditionalSameViewIdentityUsage(source), true);
+  assert.equal(hasSwiftUiConditionalSameViewIdentityUsage(safe), false);
+});

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

@@ -962,6 +962,63 @@ export const hasSwiftClosureBasedViewBuilderContentUsage = (source: string): boo
   );
 };
 
+export const hasSwiftLargeConfigContextViewPropertyUsage = (source: string): boolean => {
+  const sanitized = sanitizeSwiftSourceForMultilineRegex(source);
+  const swiftUIViewPattern =
+    /\bstruct\s+[A-Za-z_][A-Za-z0-9_]*\s*:\s*View\s*\{[\s\S]{0,2200}?\bvar\s+body\s*:\s*some\s+View\s*\{/g;
+
+  for (const viewMatch of sanitized.matchAll(swiftUIViewPattern)) {
+    const viewSegment = viewMatch[0] ?? '';
+    if (
+      /\b(?:let|var)\s+(?:config|configuration|context)\s*:\s*[A-Za-z_][A-Za-z0-9_]*(?:Config|Configuration|Context)\b/.test(
+        viewSegment
+      )
+    ) {
+      return true;
+    }
+  }
+
+  return false;
+};
+
+const swiftIdentitySensitiveViewConstructors = [
+  'Text',
+  'Image',
+  'Button',
+  'Label',
+  'HStack',
+  'VStack',
+  'ZStack',
+  'Group',
+  'RoundedRectangle',
+  'Circle',
+  'Capsule',
+  'Rectangle',
+] as const;
+
+export const hasSwiftUiConditionalSameViewIdentityUsage = (source: string): boolean => {
+  const sanitized = sanitizeSwiftSourceForMultilineRegex(source);
+  const swiftUIViewBodyPattern =
+    /\bstruct\s+[A-Za-z_][A-Za-z0-9_]*\s*:\s*View\s*\{[\s\S]{0,2200}?\bvar\s+body\s*:\s*some\s+View\s*\{/m;
+
+  if (!swiftUIViewBodyPattern.test(sanitized)) {
+    return false;
+  }
+
+  for (const constructor of swiftIdentitySensitiveViewConstructors) {
+    const escapedConstructor = escapeRegex(constructor);
+    const sameViewBranchPattern = new RegExp(
+      `\\bif\\s+[^{}]+\\{\\s*${escapedConstructor}\\s*(?:\\(|\\{|\\.)[\\s\\S]{0,600}?\\}\\s*else\\s*\\{\\s*${escapedConstructor}\\s*(?:\\(|\\{|\\.)`,
+      'm'
+    );
+    if (sameViewBranchPattern.test(sanitized)) {
+      return true;
+    }
+  }
+
+  return false;
+};
+
 export const hasSwiftRedundantReactiveStateAssignmentUsage = (source: string): boolean => {
   const sanitized = sanitizeSwiftSourceForMultilineRegex(source);
   const reactiveAssignmentPattern =

package/core/facts/extractHeuristicFacts.ts

@@ -689,6 +689,8 @@ const textDetectorRegistry: ReadonlyArray<TextDetectorRegistryEntry> = [
   { platform: 'ios', pathCheck: isIOSPresentationPath, excludePaths: [isSwiftTestPath], detect: TextIOS.hasSwiftFontWeightBoldUsage, ruleId: 'heuristics.ios.font-weight-bold.ast', code: 'HEURISTICS_IOS_FONT_WEIGHT_BOLD_AST', message: 'AST heuristic detected fontWeight(.bold) usage where bold() may be preferred.' },
   { 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.hasSwiftLargeConfigContextViewPropertyUsage, ruleId: 'heuristics.ios.swiftui.large-config-context-prop.ast', code: 'HEURISTICS_IOS_SWIFTUI_LARGE_CONFIG_CONTEXT_PROP_AST', message: 'AST heuristic detected a SwiftUI View storing a broad config/context object; pass only needed values to reduce update fan-out.' },
+  { platform: 'ios', pathCheck: isIOSPresentationPath, excludePaths: [isSwiftTestPath], detect: TextIOS.hasSwiftUiConditionalSameViewIdentityUsage, ruleId: 'heuristics.ios.swiftui.conditional-same-view-identity.ast', code: 'HEURISTICS_IOS_SWIFTUI_CONDITIONAL_SAME_VIEW_IDENTITY_AST', message: 'AST heuristic detected conditional branches rebuilding the same SwiftUI View type; prefer conditional modifiers or values to preserve view identity.' },
   { 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.' },

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

@@ -3,7 +3,7 @@ import test from 'node:test';
 import { iosRules } from './ios';
 
 test('iosRules define reglas heurísticas locked para plataforma ios', () => {
-  assert.equal(iosRules.length, 83);
+  assert.equal(iosRules.length, 85);
 
   const ids = iosRules.map((rule) => rule.id);
   assert.deepEqual(ids, [
@@ -61,6 +61,8 @@ test('iosRules define reglas heurísticas locked para plataforma ios', () => {
     'heuristics.ios.font-weight-bold.ast',
     'heuristics.ios.swiftui.explicit-color-static-member.ast',
     'heuristics.ios.swiftui.closure-based-viewbuilder-content.ast',
+    'heuristics.ios.swiftui.large-config-context-prop.ast',
+    'heuristics.ios.swiftui.conditional-same-view-identity.ast',
     'heuristics.ios.swiftui.redundant-reactive-state-assignment.ast',
     'heuristics.ios.swiftui.non-lazy-scroll-foreach.ast',
     'heuristics.ios.swiftui.body-object-creation.ast',
@@ -205,6 +207,14 @@ test('iosRules define reglas heurísticas locked para plataforma ios', () => {
     byId.get('heuristics.ios.font-weight-bold.ast')?.then.code,
     'HEURISTICS_IOS_FONT_WEIGHT_BOLD_AST'
   );
+  assert.equal(
+    byId.get('heuristics.ios.swiftui.large-config-context-prop.ast')?.then.code,
+    'HEURISTICS_IOS_SWIFTUI_LARGE_CONFIG_CONTEXT_PROP_AST'
+  );
+  assert.equal(
+    byId.get('heuristics.ios.swiftui.conditional-same-view-identity.ast')?.then.code,
+    'HEURISTICS_IOS_SWIFTUI_CONDITIONAL_SAME_VIEW_IDENTITY_AST'
+  );
   assert.equal(
     byId.get('heuristics.ios.uiscreen-main-bounds.ast')?.then.code,
     'HEURISTICS_IOS_UISCREEN_MAIN_BOUNDS_AST'

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

@@ -989,6 +989,45 @@ export const iosRules: RuleSet = [
       code: 'HEURISTICS_IOS_SWIFTUI_CLOSURE_BASED_VIEWBUILDER_CONTENT_AST',
     },
   },
+  {
+    id: 'heuristics.ios.swiftui.large-config-context-prop.ast',
+    description: 'Detects broad Config/Context properties stored directly on SwiftUI Views.',
+    severity: 'WARN',
+    platform: 'ios',
+    locked: true,
+    when: {
+      kind: 'Heuristic',
+      where: {
+        ruleId: 'heuristics.ios.swiftui.large-config-context-prop.ast',
+      },
+    },
+    then: {
+      kind: 'Finding',
+      message:
+        'AST heuristic detected a SwiftUI View storing a broad config/context object; pass only needed values to reduce update fan-out.',
+      code: 'HEURISTICS_IOS_SWIFTUI_LARGE_CONFIG_CONTEXT_PROP_AST',
+    },
+  },
+  {
+    id: 'heuristics.ios.swiftui.conditional-same-view-identity.ast',
+    description:
+      'Detects SwiftUI if/else branches that rebuild the same View type for state-only visual changes.',
+    severity: 'WARN',
+    platform: 'ios',
+    locked: true,
+    when: {
+      kind: 'Heuristic',
+      where: {
+        ruleId: 'heuristics.ios.swiftui.conditional-same-view-identity.ast',
+      },
+    },
+    then: {
+      kind: 'Finding',
+      message:
+        'AST heuristic detected conditional branches rebuilding the same SwiftUI View type; prefer conditional modifiers or values to preserve view identity.',
+      code: 'HEURISTICS_IOS_SWIFTUI_CONDITIONAL_SAME_VIEW_IDENTITY_AST',
+    },
+  },
   {
     id: 'heuristics.ios.swiftui.redundant-reactive-state-assignment.ast',
     description: 'Detects onChange/onReceive state assignments without a value-change guard.',

package/integrations/config/skillsDetectorRegistry.ts

@@ -280,6 +280,14 @@ const registryByRuleId: Record<string, SkillsDetectorBinding> = {
     heuristicDetector('ios.swiftui.closure-based-viewbuilder-content', [
       'heuristics.ios.swiftui.closure-based-viewbuilder-content.ast',
     ]),
+  'skills.ios.guideline.ios-swiftui-expert.pass-only-needed-values-to-views-avoid-large-config-or-context-objects':
+    heuristicDetector('ios.swiftui.large-config-context-prop', [
+      'heuristics.ios.swiftui.large-config-context-prop.ast',
+    ]),
+  'skills.ios.guideline.ios-swiftui-expert.prefer-modifiers-over-conditional-views-for-state-changes-maintains-vi':
+    heuristicDetector('ios.swiftui.conditional-same-view-identity', [
+      'heuristics.ios.swiftui.conditional-same-view-identity.ast',
+    ]),
   'skills.ios.guideline.ios-swiftui-expert.avoid-redundant-state-updates-in-onreceive-onchange-scroll-handlers':
     heuristicDetector('ios.swiftui.redundant-reactive-state-assignment', [
       'heuristics.ios.swiftui.redundant-reactive-state-assignment.ast',

package/integrations/config/skillsMarkdownRules.ts

@@ -324,6 +324,12 @@ const normalizeKnownRuleTarget = (
     ) {
       return 'skills.ios.guideline.ios-swiftui-expert.use-relative-layout-over-hard-coded-constants';
     }
+    if (
+      (includes('prefer modifiers') || includes('modifiers over conditional views')) &&
+      (includes('conditional views') || includes('view identity') || includes('maintains view identity'))
+    ) {
+      return 'skills.ios.guideline.ios-swiftui-expert.prefer-modifiers-over-conditional-views-for-state-changes-maintains-vi';
+    }
     if (
       (includes('foreach') && includes('indices')) ||
       includes('stable identity for foreach') ||
@@ -393,6 +399,15 @@ const normalizeKnownRuleTarget = (
     ) {
       return 'skills.ios.guideline.ios-swiftui-expert.prefer-viewbuilder-let-content-content-over-closure-based-content-prop';
     }
+    if (
+      includes('pass only needed values') ||
+      includes('large config') ||
+      includes('large context') ||
+      includes('config or context objects') ||
+      includes('config/context objects')
+    ) {
+      return 'skills.ios.guideline.ios-swiftui-expert.pass-only-needed-values-to-views-avoid-large-config-or-context-objects';
+    }
     if (
       includes('redundant state updates') ||
       (includes('onreceive') && includes('onchange') && includes('state updates')) ||

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "pumuki",
-  "version": "6.3.244",
+  "version": "6.3.246",
   "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-14T07:29:57.101Z",
+  "generatedAt": "2026-05-14T07:54:39.127Z",
   "bundles": [
     {
       "name": "android-guidelines",
@@ -8620,7 +8620,7 @@
       "name": "ios-swiftui-expert-guidelines",
       "version": "1.0.0",
       "source": "file:vendor/skills/swiftui-expert-skill/SKILL.md",
-      "hash": "95d13f770dcf3a0ecb957e4cabcfd09c9d9b39a15421c8599c6fe5470d1c3f95",
+      "hash": "3d5c272144d49aae587877dd2f8578409e26c0a6846ec43e267bc735a02e6cb0",
       "rules": [
         {
           "id": "skills.ios.guideline.ios-swiftui-expert.action-handlers-should-reference-methods-not-contain-inline-logic",
@@ -8715,7 +8715,7 @@
           "sourcePath": "vendor/skills/swiftui-expert-skill/SKILL.md",
           "confidence": "MEDIUM",
           "locked": true,
-          "evaluationMode": "DECLARATIVE",
+          "evaluationMode": "AUTO",
           "origin": "core"
         },
         {
@@ -8727,7 +8727,7 @@
           "sourcePath": "vendor/skills/swiftui-expert-skill/SKILL.md",
           "confidence": "MEDIUM",
           "locked": true,
-          "evaluationMode": "DECLARATIVE",
+          "evaluationMode": "AUTO",
           "origin": "core"
         },
         {