pumuki 6.3.188 → 6.3.190

package/VERSION

@@ -1 +1 @@
-v6.3.188
+v6.3.190

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

@@ -10,8 +10,10 @@ import {
   hasKotlinDispatcherMainBoundaryLeakUsage,
   hasKotlinGlobalScopeUsage,
   hasKotlinHardcodedBackgroundDispatcherUsage,
+  hasKotlinJUnit4Usage,
   hasKotlinLiveDataStateExposureUsage,
   hasKotlinLifecycleScopeUsage,
+  hasKotlinProductionMockUsage,
   hasKotlinSharedPreferencesUsage,
   hasKotlinWithContextUsage,
   hasKotlinManualCoroutineScopeInViewModelUsage,
@@ -275,6 +277,81 @@ class PreferencesStore {
   assert.equal(hasKotlinSharedPreferencesUsage(source), false);
 });
 
+test('hasKotlinJUnit4Usage detecta imports y anotaciones JUnit4 en tests Kotlin Android', () => {
+  const importSource = `
+import org.junit.Test
+import org.junit.Assert
+
+class OrdersViewModelTest {
+  @Test
+  fun loadsOrders() {
+    Assert.assertEquals(1, 1)
+  }
+}
+`;
+  const runnerSource = `
+import org.junit.runner.RunWith
+
+@RunWith(AndroidJUnit4::class)
+class OrdersInstrumentedTest
+`;
+
+  assert.equal(hasKotlinJUnit4Usage(importSource), true);
+  assert.equal(hasKotlinJUnit4Usage(runnerSource), true);
+});
+
+test('hasKotlinJUnit4Usage ignora JUnit5, comentarios y strings', () => {
+  const source = `
+import org.junit.jupiter.api.Test
+// import org.junit.Test
+val sample = "Assert.assertEquals(1, 1)"
+
+class OrdersViewModelTest {
+  @Test
+  fun loadsOrders() {
+    assertEquals(1, 1)
+  }
+}
+`;
+  assert.equal(hasKotlinJUnit4Usage(source), false);
+});
+
+test('hasKotlinProductionMockUsage detecta mocks y spies en Kotlin Android production', () => {
+  const mockkSource = `
+class OrdersRepositoryFactory {
+  fun create(): OrdersRepository = mockk(relaxed = true)
+}
+`;
+  const mockitoSource = `
+class OrdersRepositoryFactory {
+  private val api = Mockito.mock(OrdersApi::class.java)
+}
+`;
+  const annotationSource = `
+class OrdersRepositoryFactory {
+  @Spy
+  lateinit var repository: OrdersRepository
+}
+`;
+
+  assert.equal(hasKotlinProductionMockUsage(mockkSource), true);
+  assert.equal(hasKotlinProductionMockUsage(mockitoSource), true);
+  assert.equal(hasKotlinProductionMockUsage(annotationSource), true);
+});
+
+test('hasKotlinProductionMockUsage ignora imports, comentarios, strings y nombres parciales', () => {
+  const source = `
+import io.mockk.mockk
+import org.mockito.Mockito
+// val api = mockk<OrdersApi>()
+val sample = "Mockito.mock(OrdersApi::class.java)"
+class OrdersRepositoryFactory {
+  fun create() = mockkitoFactory()
+}
+`;
+  assert.equal(hasKotlinProductionMockUsage(source), false);
+});
+
 test('hasKotlinSupervisorScopeUsage detecta supervisorScope con parentesis y llaves', () => {
   const parenthesesSource = `
 class SyncOrdersUseCase {

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

@@ -271,6 +271,24 @@ export const hasKotlinSharedPreferencesUsage = (source: string): boolean => {
   return collectKotlinRegexLines(source, /\b(?:SharedPreferences|PreferenceManager\s*\.|getSharedPreferences\s*\()/).length > 0;
 };
 
+export const hasKotlinJUnit4Usage = (source: string): boolean => {
+  return source.split(/\r?\n/).some((line) => {
+    const sanitized = stripKotlinLineForSemanticScan(line);
+    return (
+      /\bimport\s+org\.junit\.(?:Test|Before|After|BeforeClass|AfterClass|Assert|Rule|ClassRule|runner\.RunWith|rules\.)\b/.test(sanitized) ||
+      /@(?:RunWith|Rule|ClassRule)\b/.test(sanitized) ||
+      /\bAssert\s*\./.test(sanitized)
+    );
+  });
+};
+
+export const hasKotlinProductionMockUsage = (source: string): boolean => {
+  return collectKotlinRegexLines(
+    source,
+    /(?:\b(?:mockk|spyk|mockkObject|mockkStatic|mockkConstructor|mockito|mockitoSession|mockitoRule)\s*(?:<[^>\n]+>\s*)?\(|\bMockito\s*\.\s*(?:mock|spy)\s*\(|@(?:Mock|Spy|MockK|RelaxedMockK)\b)/
+  ).length > 0;
+};
+
 export const hasKotlinSupervisorScopeUsage = (source: string): boolean => {
   return collectKotlinRegexLines(source, /\bsupervisorScope\s*(?:<[^>\n]+>\s*)?(?:\(|\{)/).length > 0;
 };

package/core/facts/extractHeuristicFacts.ts

@@ -161,6 +161,10 @@ const isKotlinTestPath = (path: string): boolean => {
   );
 };
 
+const isAndroidKotlinTestPath = (path: string): boolean => {
+  return isAndroidKotlinPath(path) && isKotlinTestPath(path);
+};
+
 const isExcludedProjectScanPath = (path: string): boolean => {
   const normalized = path.replace(/\\/g, '/').toLowerCase();
   return (
@@ -654,6 +658,8 @@ const textDetectorRegistry: ReadonlyArray<TextDetectorRegistryEntry> = [
   { platform: 'android', pathCheck: isAndroidKotlinPath, excludePaths: [isKotlinTestPath], detect: TextAndroid.hasKotlinRunBlockingUsage, ruleId: 'heuristics.android.run-blocking.ast', code: 'HEURISTICS_ANDROID_RUN_BLOCKING_AST', message: 'AST heuristic detected runBlocking usage in production Kotlin code.' },
   { platform: 'android', pathCheck: isAndroidLocalPropertiesPath, excludePaths: [], detect: detectsTrackedFilePresence, ruleId: 'heuristics.android.security.local-properties-tracked.ast', code: 'HEURISTICS_ANDROID_SECURITY_LOCAL_PROPERTIES_TRACKED_AST', message: 'AST heuristic detected tracked Android local.properties file.' },
   { platform: 'android', pathCheck: isAndroidKotlinPath, excludePaths: [isKotlinTestPath], detect: TextAndroid.hasKotlinSharedPreferencesUsage, ruleId: 'heuristics.android.persistence.shared-preferences-usage.ast', code: 'HEURISTICS_ANDROID_PERSISTENCE_SHARED_PREFERENCES_USAGE_AST', message: 'AST heuristic detected SharedPreferences usage in Android production Kotlin code.' },
+  { platform: 'android', pathCheck: isAndroidKotlinPath, excludePaths: [isKotlinTestPath], detect: TextAndroid.hasKotlinProductionMockUsage, ruleId: 'heuristics.android.testing.production-mock-usage.ast', code: 'HEURISTICS_ANDROID_TESTING_PRODUCTION_MOCK_USAGE_AST', message: 'AST heuristic detected mock or spy usage in Android production Kotlin code.' },
+  { platform: 'android', pathCheck: isAndroidKotlinTestPath, excludePaths: [], detect: TextAndroid.hasKotlinJUnit4Usage, ruleId: 'heuristics.android.testing.junit4-usage.ast', code: 'HEURISTICS_ANDROID_TESTING_JUNIT4_USAGE_AST', message: 'AST heuristic detected JUnit4 usage in Android Kotlin tests where JUnit5 is preferred.' },
   { platform: 'android', pathCheck: isAndroidPresentationPath, excludePaths: [isKotlinTestPath], detect: TextAndroid.hasKotlinLiveDataStateExposureUsage, ruleId: 'heuristics.android.flow.livedata-state-exposure.ast', code: 'HEURISTICS_ANDROID_FLOW_LIVEDATA_STATE_EXPOSURE_AST', message: 'AST heuristic detected LiveData state exposure in Android presentation code where StateFlow or SharedFlow should be preferred.' },
   { platform: 'android', pathCheck: isAndroidPresentationPath, excludePaths: [isKotlinTestPath], detect: TextAndroid.hasKotlinManualCoroutineScopeInViewModelUsage, ruleId: 'heuristics.android.coroutines.manual-scope-in-viewmodel.ast', code: 'HEURISTICS_ANDROID_COROUTINES_MANUAL_SCOPE_IN_VIEWMODEL_AST', message: 'AST heuristic detected manual CoroutineScope inside an Android ViewModel where viewModelScope should be preferred.' },
   { platform: 'android', pathCheck: isAndroidNonPresentationKotlinPath, excludePaths: [isKotlinTestPath], detect: TextAndroid.hasKotlinDispatcherMainBoundaryLeakUsage, ruleId: 'heuristics.android.coroutines.dispatchers-main-boundary-leak.ast', code: 'HEURISTICS_ANDROID_COROUTINES_DISPATCHERS_MAIN_BOUNDARY_LEAK_AST', message: 'AST heuristic detected Dispatchers.Main outside Android presentation code.' },

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

@@ -3,7 +3,7 @@ import test from 'node:test';
 import { androidRules } from './android';
 
 test('androidRules define reglas heurísticas locked para plataforma android', () => {
-  assert.equal(androidRules.length, 18);
+  assert.equal(androidRules.length, 20);
 
   const ids = androidRules.map((rule) => rule.id);
   assert.deepEqual(ids, [
@@ -13,6 +13,8 @@ test('androidRules define reglas heurísticas locked para plataforma android', (
     'heuristics.android.flow.livedata-state-exposure.ast',
     'heuristics.android.security.local-properties-tracked.ast',
     'heuristics.android.persistence.shared-preferences-usage.ast',
+    'heuristics.android.testing.junit4-usage.ast',
+    'heuristics.android.testing.production-mock-usage.ast',
     'heuristics.android.coroutines.manual-scope-in-viewmodel.ast',
     'heuristics.android.coroutines.dispatchers-main-boundary-leak.ast',
     'heuristics.android.coroutines.hardcoded-background-dispatcher.ast',
@@ -62,6 +64,14 @@ test('androidRules define reglas heurísticas locked para plataforma android', (
     byId.get('heuristics.android.persistence.shared-preferences-usage.ast')?.then.code,
     'HEURISTICS_ANDROID_PERSISTENCE_SHARED_PREFERENCES_USAGE_AST'
   );
+  assert.equal(
+    byId.get('heuristics.android.testing.junit4-usage.ast')?.then.code,
+    'HEURISTICS_ANDROID_TESTING_JUNIT4_USAGE_AST'
+  );
+  assert.equal(
+    byId.get('heuristics.android.testing.production-mock-usage.ast')?.then.code,
+    'HEURISTICS_ANDROID_TESTING_PRODUCTION_MOCK_USAGE_AST'
+  );
   assert.equal(
     byId.get('heuristics.android.coroutines.manual-scope-in-viewmodel.ast')?.then.code,
     'HEURISTICS_ANDROID_COROUTINES_MANUAL_SCOPE_IN_VIEWMODEL_AST'

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

@@ -115,6 +115,46 @@ export const androidRules: RuleSet = [
       code: 'HEURISTICS_ANDROID_PERSISTENCE_SHARED_PREFERENCES_USAGE_AST',
     },
   },
+  {
+    id: 'heuristics.android.testing.junit4-usage.ast',
+    description:
+      'Detects JUnit4 usage in Android Kotlin tests where JUnit5 is preferred.',
+    severity: 'WARN',
+    platform: 'android',
+    locked: true,
+    when: {
+      kind: 'Heuristic',
+      where: {
+        ruleId: 'heuristics.android.testing.junit4-usage.ast',
+      },
+    },
+    then: {
+      kind: 'Finding',
+      message:
+        'AST heuristic detected JUnit4 usage in Android Kotlin tests.',
+      code: 'HEURISTICS_ANDROID_TESTING_JUNIT4_USAGE_AST',
+    },
+  },
+  {
+    id: 'heuristics.android.testing.production-mock-usage.ast',
+    description:
+      'Detects mock or spy usage in Android production Kotlin code.',
+    severity: 'WARN',
+    platform: 'android',
+    locked: true,
+    when: {
+      kind: 'Heuristic',
+      where: {
+        ruleId: 'heuristics.android.testing.production-mock-usage.ast',
+      },
+    },
+    then: {
+      kind: 'Finding',
+      message:
+        'AST heuristic detected mock or spy usage in Android production Kotlin code.',
+      code: 'HEURISTICS_ANDROID_TESTING_PRODUCTION_MOCK_USAGE_AST',
+    },
+  },
   {
     id: 'heuristics.android.coroutines.manual-scope-in-viewmodel.ast',
     description:

package/docs/codex-skills/android-enterprise-rules.md

@@ -146,6 +146,14 @@ app/
 ✅ Este baseline no obliga a que todo ViewModel use Flow; solo detecta una alternativa legacy concreta cuando aparece en código de presentación Android.
 ✅ `Flow`, `collectAsState`, `stateIn`, Room observable queries y SharedFlow events quedan fuera hasta que tengan detectores propios y regresiones dirigidas.
 
+### Enforcement AST inicial de testing Android
+✅ `skills.android.guideline.android.junit5-framework-de-testing-preferido-sobre-junit4` debe mapear a señal ejecutable de uso JUnit4 en tests Kotlin Android.
+✅ Este baseline detecta imports/anotaciones JUnit4 (`org.junit.Test`, `org.junit.Assert`, `@RunWith`, reglas JUnit4) bajo `apps/android/**/test/**` y `apps/android/**/androidTest/**`.
+✅ JUnit5/Jupiter queda preservado como caso limpio; migración completa de runner, Gradle y APIs de assertions queda fuera hasta tener detectores propios.
+✅ `skills.android.guideline.android.en-produccio-n-ni-un-mocks-ni-un-spies-todo-real-de-apis-y-persistenci` debe mapear a señal ejecutable de mocks/spies en Kotlin Android production.
+✅ Este baseline detecta usos explícitos de MockK/Mockito y anotaciones `@Mock`/`@Spy` fuera de `test` y `androidTest`; los dobles en tests siguen permitidos.
+✅ La regla no prohíbe clases fake de test doubles en suites de test ni analiza wiring Gradle todavía.
+
 ### Dependency Injection (Hilt):
 ✅ **Hilt** - DI framework (NO manual factories)
 ✅ **@HiltAndroidApp** - Application class

package/integrations/config/skillsDetectorRegistry.ts

@@ -255,6 +255,14 @@ const registryByRuleId: Record<string, SkillsDetectorBinding> = {
     'android.persistence.shared-preferences',
     ['heuristics.android.persistence.shared-preferences-usage.ast']
   ),
+  'skills.android.guideline.android.junit5-framework-de-testing-preferido-sobre-junit4': heuristicDetector(
+    'android.testing.junit4-usage',
+    ['heuristics.android.testing.junit4-usage.ast']
+  ),
+  'skills.android.guideline.android.en-produccio-n-ni-un-mocks-ni-un-spies-todo-real-de-apis-y-persistenci': heuristicDetector(
+    'android.testing.production-mock-usage',
+    ['heuristics.android.testing.production-mock-usage.ast']
+  ),
   'skills.android.guideline.android.stateflow-estado-mutable-observable': heuristicDetector(
     'android.flow.state-exposure',
     ['heuristics.android.flow.livedata-state-exposure.ast']

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "pumuki",
-  "version": "6.3.188",
+  "version": "6.3.190",
   "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-12T21:13:45.516Z",
+  "generatedAt": "2026-05-12T21:28:07.507Z",
   "bundles": [
     {
       "name": "android-guidelines",