@codyswann/lisa 1.0.0 → 1.0.5

package/expo/copy-overwrite/eslint-plugin-component-structure/__tests__/single-component-per-file.test.js

@@ -1,3 +1,8 @@
+/**
+ * This file is managed by Lisa.
+ * Do not edit directly — changes will be overwritten on the next `lisa` run.
+ */
+
 /**
  * Unit tests for the single-component-per-file ESLint rule
  *

package/expo/copy-overwrite/eslint-plugin-component-structure/index.js

@@ -1,3 +1,8 @@
+/**
+ * This file is managed by Lisa.
+ * Do not edit directly — changes will be overwritten on the next `lisa` run.
+ */
+
 /**
  * ESLint plugin for component structure standards
  *

package/expo/copy-overwrite/eslint-plugin-component-structure/rules/enforce-component-structure.js

@@ -1,3 +1,8 @@
+/**
+ * This file is managed by Lisa.
+ * Do not edit directly — changes will be overwritten on the next `lisa` run.
+ */
+
 const fs = require("fs");
 const path = require("path");
 

package/expo/copy-overwrite/eslint-plugin-component-structure/rules/no-return-in-view.js

@@ -1,3 +1,8 @@
+/**
+ * This file is managed by Lisa.
+ * Do not edit directly — changes will be overwritten on the next `lisa` run.
+ */
+
 module.exports = {
   meta: {
     type: "problem",
@@ -11,7 +16,7 @@ module.exports = {
     schema: [],
     messages: {
       noReturnInView:
-        "View components should use arrow function shorthand: () => (...) instead of () => { return (...) }",
+        "View components should use arrow function shorthand: () => (...) instead of () => { return (...) }. Hoist any definitions outside of the arrow function body or into the corresponding Container.",
     },
   },
 

package/expo/copy-overwrite/eslint-plugin-component-structure/rules/require-memo-in-view.js

@@ -1,3 +1,8 @@
+/**
+ * This file is managed by Lisa.
+ * Do not edit directly — changes will be overwritten on the next `lisa` run.
+ */
+
 /**
  * ESLint rule to enforce React.memo usage with displayName in View components
  *

package/expo/copy-overwrite/eslint-plugin-component-structure/rules/single-component-per-file.js

@@ -1,3 +1,8 @@
+/**
+ * This file is managed by Lisa.
+ * Do not edit directly — changes will be overwritten on the next `lisa` run.
+ */
+
 /**
  * ESLint rule to enforce exactly one React component per file
  *

package/expo/copy-overwrite/eslint-plugin-ui-standards/README.md

@@ -146,73 +146,6 @@ const MyComponent = () => (
 
 ---
 
-### no-inline-styles
-
-Prevents inline style objects in favor of className-based styling.
-
-#### Rule Details
-
-This rule prohibits inline `style` props with object literals. All styling should use className with Tailwind/NativeWind classes.
-
-**Why this rule exists:**
-- Inline styles bypass the design system
-- Harder to maintain consistency
-- NativeWind provides better performance
-- Enables theme support
-
-#### Examples
-
-**Incorrect:**
-
-```tsx
-const MyView = () => (
-  <View style={{ padding: 16, backgroundColor: 'white' }}>
-    <Text style={{ fontSize: 18, fontWeight: 'bold' }}>Title</Text>
-  </View>
-);
-```
-
-**Incorrect** (StyleSheet is also discouraged):
-
-```tsx
-const styles = StyleSheet.create({
-  container: { padding: 16 }
-});
-
-const MyView = () => (
-  <View style={styles.container}>
-    <Text>Content</Text>
-  </View>
-);
-```
-
-**Correct:**
-
-```tsx
-const MyView = () => (
-  <View className="p-4 bg-white">
-    <Text className="text-lg font-bold">Title</Text>
-  </View>
-);
-```
-
-#### Configuration
-
-```javascript
-// eslint.config.mjs
-{
-  rules: {
-    'ui-standards/no-inline-styles': 'error'
-  }
-}
-```
-
-**Allowed exceptions:**
-- Dynamic styles computed at runtime (with eslint-disable comment)
-- Animation styles (Reanimated)
-
----
-
 ## Installation
 
 This plugin is installed locally as a file dependency:
@@ -239,7 +172,6 @@ export default [
     rules: {
       'ui-standards/no-classname-outside-ui': 'error',
       'ui-standards/no-direct-rn-imports': 'error',
-      'ui-standards/no-inline-styles': 'error',
     },
   },
 ];

package/expo/copy-overwrite/eslint-plugin-ui-standards/index.js

@@ -1,3 +1,8 @@
+/**
+ * This file is managed by Lisa.
+ * Do not edit directly — changes will be overwritten on the next `lisa` run.
+ */
+
 /**
  * ESLint plugin for UI standards
  *
@@ -7,12 +12,10 @@
  * Rules:
  * - no-classname-outside-ui: Disallows className prop outside UI components
  * - no-direct-rn-imports: Disallows direct React Native imports
- * - no-inline-styles: Disallows inline style objects
  * @module eslint-plugin-ui-standards
  */
 const noClassnameOutsideUi = require("./rules/no-classname-outside-ui");
 const noDirectRnImports = require("./rules/no-direct-rn-imports");
-const noInlineStyles = require("./rules/no-inline-styles");
 
 const plugin = {
   meta: {
@@ -22,7 +25,6 @@ const plugin = {
   rules: {
     "no-classname-outside-ui": noClassnameOutsideUi,
     "no-direct-rn-imports": noDirectRnImports,
-    "no-inline-styles": noInlineStyles,
   },
 };
 

package/expo/copy-overwrite/eslint-plugin-ui-standards/rules/no-classname-outside-ui.js

@@ -1,3 +1,8 @@
+/**
+ * This file is managed by Lisa.
+ * Do not edit directly — changes will be overwritten on the next `lisa` run.
+ */
+
 module.exports = {
   meta: {
     type: "problem",

package/expo/copy-overwrite/eslint-plugin-ui-standards/rules/no-direct-rn-imports.js

@@ -1,3 +1,8 @@
+/**
+ * This file is managed by Lisa.
+ * Do not edit directly — changes will be overwritten on the next `lisa` run.
+ */
+
 module.exports = {
   meta: {
     type: "problem",

package/expo/copy-overwrite/eslint.config.ts

@@ -0,0 +1,53 @@
+/**
+ * This file is managed by Lisa.
+ * Do not edit directly — changes will be overwritten on the next `lisa` run.
+ */
+
+/**
+ * ESLint 9 Flat Config - Main Entry Point (Expo)
+ *
+ * This file imports the Expo-specific configuration and project-local customizations.
+ * Do not modify this file directly - use eslint.config.local.ts for project-specific rules.
+ *
+ * Inheritance chain:
+ *   eslint.config.ts (this file)
+ *   └── eslint.expo.ts
+ *       └── eslint.typescript.ts
+ *           └── eslint.base.ts
+ *
+ * @see https://eslint.org/docs/latest/use/configure/configuration-files-new
+ * @module eslint.config
+ */
+import path from "path";
+import { fileURLToPath } from "url";
+
+import {
+  defaultIgnores,
+  defaultThresholds,
+  getExpoConfig,
+} from "./eslint.expo";
+
+// Project-specific configuration loaded from JSON files
+import ignoreConfig from "./eslint.ignore.config.json" with { type: "json" };
+import thresholdsConfig from "./eslint.thresholds.json" with { type: "json" };
+
+// Project-local customizations (create-only - safe to modify)
+import localConfig from "./eslint.config.local";
+
+const __filename = fileURLToPath(import.meta.url);
+const __dirname = path.dirname(__filename);
+
+const ignorePatterns = ignoreConfig.ignores || defaultIgnores;
+const thresholds = { ...defaultThresholds, ...thresholdsConfig };
+
+export default [
+  // Stack-specific configuration (Expo)
+  ...getExpoConfig({
+    tsconfigRootDir: __dirname,
+    ignorePatterns,
+    thresholds,
+  }),
+
+  // Project-local customizations
+  ...localConfig,
+];

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

@@ -0,0 +1,330 @@
+/**
+ * This file is managed by Lisa.
+ * Do not edit directly — changes will be overwritten on the next `lisa` run.
+ */
+
+/* eslint-disable max-lines-per-function -- config file needs a lot of lines */
+/**
+ * ESLint 9 Flat Config - Expo Stack
+ *
+ * This configuration extends TypeScript config for Expo/React Native projects.
+ * It adds Expo-specific plugins and rules for React, Tailwind, accessibility,
+ * and component structure.
+ *
+ * Inheritance chain:
+ *   eslint.expo.ts (this file)
+ *   └── eslint.typescript.ts
+ *       └── eslint.base.ts
+ * @see https://eslint.org/docs/latest/use/configure/configuration-files-new
+ * @module eslint.expo
+ */
+import jsxA11y from "eslint-plugin-jsx-a11y";
+import react from "eslint-plugin-react";
+import reactCompiler from "eslint-plugin-react-compiler";
+// @ts-expect-error -- eslint-plugin-react-perf lacks type declarations
+import reactPerf from "eslint-plugin-react-perf";
+import tailwind from "eslint-plugin-tailwindcss";
+import { createRequire } from "module";
+
+// Import TypeScript config and utilities
+import {
+  codeOrganization,
+  defaultIgnores,
+  defaultThresholds,
+  getBaseConfigs,
+  getBaseLanguageOptions,
+  getJsFilesOverride,
+  getSharedFilesOverride,
+  getSharedRules,
+  getTestFilesOverride,
+  getTsFilesOverride,
+  getTsTestFilesOverride,
+} from "./eslint.typescript";
+
+// Re-export for downstream configs
+export {
+  defaultIgnores,
+  defaultThresholds,
+  getBaseConfigs,
+  getBaseLanguageOptions,
+  getSharedRules,
+};
+
+// Custom plugins (CommonJS - use createRequire)
+const require = createRequire(import.meta.url);
+const componentStructure = require("./eslint-plugin-component-structure/index.js");
+const uiStandards = require("./eslint-plugin-ui-standards/index.js");
+const expoConfig = require("eslint-config-expo/flat");
+
+/**
+ * Creates the Expo ESLint configuration.
+ * @param {object} options - Configuration options
+ * @param {string} options.tsconfigRootDir - Root directory for tsconfig.json
+ * @param {string[]} [options.ignorePatterns] - Patterns to ignore
+ * @param {object} [options.thresholds] - Threshold overrides
+ * @returns {Array} ESLint flat config array
+ */
+export function getExpoConfig({
+  tsconfigRootDir,
+  ignorePatterns = defaultIgnores,
+  thresholds = defaultThresholds,
+}: {
+  tsconfigRootDir: string;
+  ignorePatterns?: string[];
+  thresholds?: typeof defaultThresholds;
+}) {
+  return [
+    // Global ignores
+    {
+      ignores: ignorePatterns,
+    },
+
+    // Base configurations from shared module
+    ...getBaseConfigs(),
+
+    // Expo-specific configurations
+    ...expoConfig,
+
+    // React configurations
+    react.configs.flat.recommended,
+    react.configs.flat["jsx-runtime"],
+
+    // Accessibility
+    jsxA11y.flatConfigs.strict,
+
+    // Tailwind CSS
+    ...tailwind.configs["flat/recommended"],
+
+    // Performance
+    reactPerf.configs.flat.recommended,
+
+    // Base configuration for all files
+    {
+      languageOptions: {
+        ...getBaseLanguageOptions(),
+        parserOptions: {
+          ...getBaseLanguageOptions().parserOptions,
+          ecmaFeatures: { jsx: true },
+        },
+      },
+      plugins: {
+        "component-structure": componentStructure,
+        "code-organization": codeOrganization,
+        "react-compiler": reactCompiler,
+        "ui-standards": uiStandards,
+      },
+      rules: {
+        // Shared rules from base
+        ...getSharedRules(thresholds),
+
+        // Code organization
+        "code-organization/enforce-statement-order": "error",
+
+        // UI standards (all off by default)
+        "ui-standards/no-classname-outside-ui": "off",
+        "ui-standards/no-direct-rn-imports": "off",
+
+        // Tailwind
+        "tailwindcss/classnames-order": [
+          "error",
+          {
+            callees: ["tva", "classnames", "clsx", "ctl", "cva", "tv"],
+          },
+        ],
+
+        // Import rules
+        // Slow rules - disabled by default, run via `lint:slow` script
+        // @see eslint.slow.config.ts
+        "import/no-cycle": "off",
+        "import/namespace": "off",
+        // End Slow rules - disabled by default, run via `lint:slow` script
+        "import/no-unresolved": "off", // Disabled: doesn't understand React Native platform extensions (.native.tsx, .web.tsx)
+        "import/prefer-default-export": "off",
+        "import/no-duplicates": "error",
+
+        // React performance
+        "react-perf/jsx-no-new-object-as-prop": "error",
+        "react-perf/jsx-no-new-array-as-prop": "error",
+        "react-perf/jsx-no-new-function-as-prop": "error",
+
+        // React rules
+        "react/jsx-no-constructed-context-values": "error",
+        "react/jsx-uses-react": "off",
+        "react/react-in-jsx-scope": "off",
+        "react-hooks/exhaustive-deps": "error",
+        "react-hooks/rules-of-hooks": "error",
+        "react-hooks/set-state-in-effect": "error",
+        "react-hooks/preserve-manual-memoization": "error",
+        "react-hooks/refs": "error",
+        "react-hooks/immutability": "error",
+        // Slow rules - disabled by default, run via `lint:slow` script
+        // @see eslint.slow.config.ts
+        "react-compiler/react-compiler": "off",
+        "react-hooks/static-components": "off",
+        // End Slow rules - disabled by default, run via `lint:slow` script
+        // Environment variables - enforce validated env module usage
+        // @see .claude/skills/expo-env-config/SKILL.md
+        "no-restricted-syntax": [
+          "error",
+          {
+            selector:
+              "MemberExpression[object.name='process'][property.name='env']",
+            message:
+              "Direct process.env access is forbidden. Import { env } from '@/lib/env' instead for type-safe, validated environment variables.",
+          },
+        ],
+
+        // Console warnings for Expo
+        "no-console": "warn",
+
+        // Functional programming - classes not allowed in Expo (functional components only)
+        "functional/no-classes": "error",
+
+        // JSDoc rules - Expo has relaxed requirements for classes/methods
+        "jsdoc/require-jsdoc": [
+          "error",
+          {
+            require: {
+              FunctionDeclaration: true,
+              MethodDefinition: false,
+              ClassDeclaration: false,
+              ArrowFunctionExpression: false,
+              FunctionExpression: false,
+            },
+            contexts: [
+              "TSInterfaceDeclaration",
+              "TSTypeAliasDeclaration",
+              "VariableDeclaration[declarations.0.init.type='ArrowFunctionExpression']:has([id.name=/^[A-Z]/])",
+            ],
+          },
+        ],
+      },
+    },
+
+    // JavaScript files override
+    getJsFilesOverride(),
+
+    // Shared hooks and components
+    getSharedFilesOverride(),
+
+    // UI components
+    {
+      files: [
+        "components/ui/**/*.tsx",
+        "components/ui/**/*.jsx",
+        "components/custom/ui/**/*.tsx",
+        "components/custom/ui/**/*.jsx",
+      ],
+      rules: {
+        "ui-standards/no-classname-outside-ui": "off",
+      },
+    },
+
+    // Test files and Jest setup (with JSX patterns)
+    getTestFilesOverride([
+      "**/*.test.jsx",
+      "**/*.test.tsx",
+      "**/*spec.jsx",
+      "**/*spec.tsx",
+    ]),
+
+    // TypeScript files - enable type-checked linting (includes TSX)
+    {
+      ...getTsFilesOverride(["**/*.ts", "**/*.tsx"], tsconfigRootDir),
+      rules: {
+        ...getTsFilesOverride(["**/*.ts", "**/*.tsx"], tsconfigRootDir).rules,
+        "react/prop-types": "off",
+      },
+    },
+
+    // TSX files - disable require-returns for components
+    {
+      files: ["**/*.tsx"],
+      rules: {
+        "jsdoc/require-returns": "off",
+      },
+    },
+
+    // TypeScript test files - disable immutable-data (must come after TypeScript config)
+    getTsTestFilesOverride([
+      "**/*.test.ts",
+      "**/*.test.tsx",
+      "**/*spec.ts",
+      "**/*spec.tsx",
+    ]),
+
+    // Component structure rules
+    {
+      files: [
+        "features/**/components/**/*.ts",
+        "features/**/components/**/*.tsx",
+        "features/**/components/**/*.jsx",
+        "features/**/screens/**/*.ts",
+        "features/**/screens/**/*.tsx",
+        "features/**/screens/**/*.jsx",
+        "components/**/*.ts",
+        "components/**/*.tsx",
+        "components/**/*.jsx",
+      ],
+      ignores: [
+        "components/ui/**",
+        "components/shared/**",
+        "components/icons/**",
+      ],
+      rules: {
+        "component-structure/enforce-component-structure": "error",
+        "component-structure/single-component-per-file": "error",
+      },
+    },
+
+    // View component rules
+    {
+      files: ["**/*View.tsx", "**/*View.jsx"],
+      rules: {
+        "component-structure/no-return-in-view": "error",
+      },
+    },
+
+    // View memo requirement (excluding UI components)
+    {
+      files: ["**/*View.tsx", "**/*View.jsx"],
+      ignores: ["components/ui/**"],
+      rules: {
+        "component-structure/require-memo-in-view": "error",
+      },
+    },
+
+    // Environment validation module - allowed to access process.env
+    // This is the ONLY file that should directly read environment variables
+    // @see .claude/skills/expo-env-config/SKILL.md
+    {
+      files: ["src/lib/env.ts", "lib/env.ts", "lib/__tests__/env.test.ts"],
+      rules: {
+        "no-restricted-syntax": "off",
+      },
+    },
+
+    // Test files that mock process.env for environment variable testing
+    // These need direct access to process.env to set up test fixtures
+    {
+      files: [
+        "lib/apollo/sentryLink.test.ts",
+        "lib/build/info.test.ts",
+        "lib/sentry/config.test.ts",
+      ],
+      rules: {
+        "no-restricted-syntax": "off",
+      },
+    },
+
+    // Configuration files - allowed to use process.env directly
+    {
+      files: ["**/*config.*", "lighthouserc.js", "codegen.ts"],
+      rules: {
+        "no-restricted-syntax": "off",
+      },
+    },
+  ];
+}
+
+/* eslint-enable max-lines-per-function -- config file needs a lot of lines */

package/expo/copy-overwrite/eslint.slow.config.ts

@@ -0,0 +1,86 @@
+/**
+ * This file is managed by Lisa.
+ * Do not edit directly — changes will be overwritten on the next `lisa` run.
+ */
+
+/**
+ * ESLint 9 Flat Config - Slow Rules Only (Expo)
+ *
+ * This configuration runs ONLY slow linting rules that are disabled in the
+ * main eslint.config.ts for performance. Run this periodically via `lint:slow`
+ * rather than on every lint pass.
+ *
+ * Rules included:
+ * - import/namespace - Type checks all namespace imports (slow)
+ * - import/no-cycle - Detects circular dependencies (very slow)
+ * - react-compiler/react-compiler - React Compiler compatibility checks (slow)
+ * - react-hooks/static-components - Static component optimization checks (slow)
+ *
+ * @see https://github.com/import-js/eslint-plugin-import
+ * @see https://react.dev/learn/react-compiler
+ * @module eslint.slow.config
+ */
+import importPlugin from "eslint-plugin-import";
+import reactCompiler from "eslint-plugin-react-compiler";
+import reactHooks from "eslint-plugin-react-hooks";
+import sonarjsPlugin from "eslint-plugin-sonarjs";
+import tseslint from "typescript-eslint";
+
+import ignoreConfig from "./eslint.ignore.config.json" with { type: "json" };
+
+const ignorePatterns = ignoreConfig.ignores || [];
+
+// Get the TypeScript flat config from the import plugin
+const importTypescriptConfig = importPlugin.flatConfigs.typescript;
+
+export default [
+  // Use same ignores as main config, plus ignore all non-TS files
+  // This prevents errors from inline eslint directives in JS files
+  // that reference rules not loaded in this minimal config
+  {
+    ignores: [...ignorePatterns, "**/*.js", "**/*.mjs", "**/*.cjs"],
+  },
+
+  // TypeScript/TSX files - slow import and React rules only
+  {
+    files: ["**/*.ts", "**/*.tsx"],
+    languageOptions: {
+      parser: tseslint.parser,
+      parserOptions: {
+        project: "tsconfig.eslint.json",
+        ecmaFeatures: { jsx: true },
+      },
+    },
+    plugins: {
+      ...importTypescriptConfig?.plugins,
+      sonarjs: sonarjsPlugin,
+      "react-compiler": reactCompiler,
+      "react-hooks": reactHooks,
+    },
+    settings: {
+      ...(importTypescriptConfig?.settings ?? {}),
+      "import/resolver": {
+        ...((importTypescriptConfig?.settings?.["import/resolver"] as Record<
+          string,
+          unknown
+        >) ?? {}),
+        typescript: true,
+      },
+    },
+    rules: {
+      // ONLY slow rules - everything else runs in the main config
+
+      // Import rules (slow)
+      "import/namespace": "error",
+      "import/no-cycle": "error",
+
+      "sonarjs/deprecation": "error",
+
+      // React Compiler compatibility (slow)
+      "react-compiler/react-compiler": "error",
+
+      // Static component optimization (slow)
+      "react-hooks/static-components": "error",
+    },
+  },
+];