@codyswann/lisa 1.22.0 → 1.23.0

package/all/copy-overwrite/CLAUDE.md

@@ -16,7 +16,8 @@ Always use project-relative paths rather than absolute paths in documentation an
 Always ignore build folders (dist, build, etc) unless specified otherwise
 Always delete and remove old code completely - no deprecation needed
 Always add `GIT_SSH_COMMAND="ssh -o ServerAliveInterval=30 -o ServerAliveCountMax=5" ` when running `git push`
-Always err on the side of creating a plan before directly executing a coding task 
+Always ignore knip configuration hints (warnings) — only fix actual unused exports/dependencies reported as errors
+Always err on the side of creating a plan before directly executing a coding task
 
 Never modify this file (CLAUDE.md) directly. To add a memory or learning, add it to .claude/rules/PROJECT_RULES.md or create a skill with /skill-creator.
 Never commit changes to an environment branch (dev, staging, main) directly. This is enforced by the pre-commit hook.

package/expo/copy-overwrite/.claude/skills/expo-env-config/SKILL.md

@@ -108,7 +108,7 @@ The Zod pattern validates `process.env` directly - it doesn't care how variables
 
 ## Testing Pattern
 
-### Jest Setup (`jest.setup.ts`)
+### Jest Setup (`jest.setup.local.ts`)
 
 ```typescript
 // Mock the env module for all tests
@@ -304,6 +304,6 @@ When adding or modifying environment variables:
 - [ ] Variable is added to the Zod schema in `src/lib/env.ts`
 - [ ] Appropriate Zod type/transform is used (url, enum, boolean transform, etc.)
 - [ ] Default value provided for optional variables
-- [ ] Jest mock updated in `jest.setup.ts`
+- [ ] Jest mock updated in `jest.setup.local.ts`
 - [ ] Variable documented in `.env.example` or `.env.development`
 - [ ] Sensitive values use EAS Secrets, not `.env` files

package/expo/copy-overwrite/.claude/skills/expo-env-config/references/validation-patterns.md

@@ -241,7 +241,7 @@ export const env = parseEnv();
 ### Complete Mock Setup
 
 ```typescript
-// jest.setup.ts
+// jest.setup.local.ts
 const mockEnv = {
   EXPO_PUBLIC_API_URL: "https://test.example.com",
   EXPO_PUBLIC_APP_ENV: "development" as const,

package/expo/copy-overwrite/jest.expo.ts

@@ -15,12 +15,11 @@
  * Instead, this config manually replicates the preset's resolution,
  * transform, and haste settings without any preset setupFiles.
  *
- * `setupFiles` is intentionally empty because `jest-expo/src/preset/setup.js`
- * requires `__DEV__` to be defined before it runs, and `mergeConfigs`
- * concatenates arrays with base entries first — making it impossible for
- * project-local setupFiles to prepend a `__DEV__` definition. Projects
- * should add their own setupFiles in `jest.config.local.ts` with the
- * correct ordering (define globals first, then load jest-expo setup).
+ * `setupFiles` and `setupFilesAfterEnv` are configured here to wire up
+ * the base jest setup files (`jest.setup.pre.js` and `jest.setup.ts`).
+ * These base files import project-local overrides (`jest.setup.pre.local.js`
+ * and `jest.setup.local.ts`) which are create-only templates that projects
+ * can customize without Lisa overwriting them.
  *
  * Coverage collection is scoped to standard Expo source directories
  * rather than a catch-all glob, preventing config files, scripts, and
@@ -67,9 +66,7 @@ interface ExpoJestOptions {
  * @returns Jest config object with jsdom environment, babel-jest transform, and React Native resolver
  * @remarks Avoids `jest-expo` preset to prevent jsdom + `react-native/jest/setup.js`
  * incompatibility. Manually configures haste, resolver, and transform to match the
- * preset's behavior without the problematic window redefinition. `setupFiles` is
- * empty — projects must provide their own in `jest.config.local.ts` with correct
- * ordering (define `__DEV__` before loading `jest-expo/src/preset/setup.js`).
+ * preset's behavior without the problematic window redefinition.
  */
 export const getExpoJestConfig = ({
   thresholds = defaultThresholds,
@@ -80,7 +77,8 @@ export const getExpoJestConfig = ({
     platforms: ["android", "ios", "native"],
   },
   resolver: "react-native/jest/resolver.js",
-  setupFiles: [],
+  setupFiles: ["<rootDir>/jest.setup.pre.js"],
+  setupFilesAfterEnv: ["<rootDir>/jest.setup.ts"],
   transform: {
     "\\.[jt]sx?$": [
       "babel-jest",
@@ -114,7 +112,6 @@ export const getExpoJestConfig = ({
     "stores/**/*.{ts,tsx}",
     "types/**/*.{ts,tsx}",
     "utils/**/*.{ts,tsx}",
-    "!**/*.d.ts",
     ...defaultCoverageExclusions,
   ],
   coverageThreshold: thresholds,

package/expo/{create-only → copy-overwrite}/jest.setup.pre.js

@@ -1,10 +1,18 @@
 /**
- * Jest Pre-Setup File
+ * This file is managed by Lisa.
+ * Do not edit directly — changes will be overwritten on the next `lisa` run.
+ */
+
+/**
+ * Jest Pre-Setup File - Base Configuration
  *
  * Runs before Jest loads any other modules to set up global flags needed
  * by React Native. This file must be JavaScript (not TypeScript) and must
  * not import any modules that depend on the global flags being set.
  *
+ * Project-specific pre-framework globals belong in `jest.setup.pre.local.js`
+ * (create-only).
+ *
  * @remarks
  * - Listed in `setupFiles` (runs before test framework loads)
  * - Sets `__DEV__`, `IS_REACT_ACT_ENVIRONMENT`, and other RN globals
@@ -104,3 +112,6 @@ global.window = {
     removeListener: jest.fn(),
   })),
 };
+
+// Project-local pre-framework globals (create-only — Lisa never overwrites this)
+require("./jest.setup.pre.local");

package/expo/{create-only → copy-overwrite}/jest.setup.ts

@@ -1,10 +1,17 @@
 /**
- * Jest Setup File
+ * This file is managed by Lisa.
+ * Do not edit directly — changes will be overwritten on the next `lisa` run.
+ */
+
+/**
+ * Jest Setup File - Base Configuration
  *
  * Configures the testing environment after Jest loads modules.
  * Sets up React Native Testing Library matchers, global mocks,
  * and manual cleanup for React 19 compatibility.
  *
+ * Project-specific mocks belong in `jest.setup.local.ts` (create-only).
+ *
  * @remarks
  * - Listed in `setupFilesAfterEnv` (runs after test framework loads)
  * - Basic globals like `__DEV__` are set earlier in `jest.setup.pre.js`
@@ -16,21 +23,14 @@
  * @module jest.setup
  */
 
+// Project-local mocks and setup (create-only — Lisa never overwrites this)
+import "./jest.setup.local";
+
 // Import from pure to avoid auto-cleanup (we set RNTL_SKIP_AUTO_CLEANUP in pre-setup)
 // Then extend Jest matchers with React Native Testing Library matchers
 import { cleanup } from "@testing-library/react-native/pure";
 import "@testing-library/react-native/build/matchers/extend-expect";
 
-// Mock environment variables module for type-safe env access in tests
-// Tests can override specific values via jest.spyOn or jest.doMock
-// Replace the mock below with your project's actual env module and values
-jest.mock("@/lib/env", () => ({
-  env: {
-    EXPO_PUBLIC_ENV: "dev",
-    // Add your project-specific environment variables here
-  },
-}));
-
 // Extend global type for the unhandledRejection handler set in jest.setup.pre.js
 declare global {
   let __unhandledRejectionHandler:
@@ -38,8 +38,8 @@ declare global {
     | undefined;
 }
 
-// Manual cleanup after each test (replacing RTLRN's auto cleanup)
-// This handles React 19's cleanup which can throw null
+// Manual cleanup and mock reset after each test
+// Replaces RTLRN's auto cleanup to handle React 19's null-throw during teardown
 afterEach(async () => {
   try {
     await cleanup();
@@ -49,6 +49,7 @@ afterEach(async () => {
       throw error;
     }
   }
+  jest.clearAllMocks();
 });
 
 // Mock ResizeObserver for tests that use it
@@ -90,25 +91,6 @@ jest.mock("expo-router", () => ({
   },
 }));
 
-// Mock @react-native-firebase/analytics
-jest.mock("@react-native-firebase/analytics", () => {
-  const logEvent = jest.fn();
-  const getAnalytics = jest.fn(() => ({}));
-  return { logEvent, getAnalytics };
-});
-
-// Mock firebase/analytics
-jest.mock("firebase/analytics", () => {
-  const logEvent = jest.fn();
-  const getAnalytics = jest.fn(() => ({}));
-  return { logEvent, getAnalytics };
-});
-
-// Clear mocks after each test
-afterEach(() => {
-  jest.clearAllMocks();
-});
-
 // Remove the unhandledRejection handler after all tests to allow Jest to exit cleanly
 // The handler is registered in jest.setup.pre.js and stored on global
 afterAll(() => {

package/expo/copy-overwrite/tsconfig.expo.json

@@ -14,5 +14,5 @@
     "moduleSuffixes": [".ios", ".android", ".native", ".web", ""]
   },
   "include": ["**/*.ts", "**/*.tsx", "nativewind-env.d.ts"],
-  "exclude": ["node_modules", "dist", "web-build"]
+  "exclude": ["node_modules", "dist", "web-build", "components/ui"]
 }

package/expo/create-only/jest.config.local.ts

@@ -5,20 +5,15 @@
  * meaning Lisa will create it but never overwrite your customizations.
  *
  * The Expo stack's `jest.expo.ts` provides haste, resolver, transform,
- * and base coverage settings. This file should only contain settings
- * that are project-specific or need to override the base config.
+ * setupFiles, and base coverage settings. This file should only contain
+ * settings that are project-specific or need to override the base config.
  *
- * @remarks `setupFiles` must define `__DEV__` and other React Native
- * globals before any RN modules load — `jest.setup.pre.js` handles this.
  * @see https://jestjs.io/docs/configuration
  * @module jest.config.local
  */
 import type { Config } from "jest";
 
 const config: Config = {
-  setupFiles: ["<rootDir>/jest.setup.pre.js"],
-  setupFilesAfterEnv: ["<rootDir>/jest.setup.ts"],
-
   // Path aliases matching tsconfig.expo.json paths
   moduleNameMapper: {
     "^@/(.*)$": "<rootDir>/$1",

package/expo/create-only/jest.setup.local.ts

@@ -0,0 +1,13 @@
+/**
+ * Jest Setup - Project-Local Customizations
+ *
+ * Add project-specific jest.mock() calls, afterEach hooks, and other
+ * test setup here. This file is create-only — Lisa will never overwrite it.
+ *
+ * Common additions:
+ * - jest.mock("@/lib/env", ...) for environment variable mocking
+ * - jest.mock("@sentry/react-native", ...) for Sentry
+ * - jest.mock("nativewind", ...) for NativeWind/styling
+ * - jest.mock("react-native-reanimated", ...) for animations
+ * - Additional expo-* module mocks specific to your dependencies
+ */

package/expo/create-only/jest.setup.pre.local.js

@@ -0,0 +1,12 @@
+/**
+ * Jest Pre-Setup - Project-Local Customizations
+ *
+ * Add project-specific pre-framework globals here. This file runs before
+ * Jest loads any test modules. This file is create-only — Lisa will never
+ * overwrite it.
+ *
+ * Common additions:
+ * - Additional global polyfills (setImmediate, ErrorUtils, Web Streams)
+ * - Expo-specific polyfills (expo-modules-core global polyfill)
+ * - Custom __ExpoImportMetaRegistry setup
+ */

package/package.json

@@ -88,7 +88,7 @@
     "@isaacs/brace-expansion": "^5.0.1"
   },
   "name": "@codyswann/lisa",
-  "version": "1.22.0",
+  "version": "1.23.0",
   "description": "Claude Code governance framework that applies guardrails, guidance, and automated enforcement to projects",
   "main": "dist/index.js",
   "bin": {