@codyswann/lisa 1.47.1 → 1.49.0

package/eslint-plugin-code-organization/rules/enforce-statement-order.js

@@ -1,162 +0,0 @@
-/**
- * This file is managed by Lisa.
- * Do not edit directly — changes will be overwritten on the next `lisa` run.
- */
-
-/**
- * ESLint rule to enforce statement order in all functions
- *
- * Enforces the following order:
- * 1. Definitions: Variable declarations (const/let/var) and function declarations
- * 2. Side effects: Expression statements that are function calls
- * 3. Return statement
- *
- * Applies to all functions: hooks, components, utilities, etc.
- * @type {import('eslint').Rule.RuleModule}
- */
-module.exports = {
-  meta: {
-    type: "problem",
-    docs: {
-      description:
-        "Enforce statement order: definitions → side effects → return",
-      category: "Best Practices",
-      recommended: true,
-    },
-    fixable: null,
-    schema: [],
-    messages: {
-      wrongOrder:
-        "{{current}} should come before {{previous}}. Expected order: definitions → side effects → return statement",
-    },
-  },
-
-  create(context) {
-    const ORDER = {
-      DEFINITION: 1, // Variable declarations, function declarations
-      SIDE_EFFECT: 2, // Expression statements with function calls
-      RETURN: 3, // Return statement
-    };
-
-    const ORDER_NAMES = {
-      [ORDER.DEFINITION]: "Definitions",
-      [ORDER.SIDE_EFFECT]: "Side effects",
-      [ORDER.RETURN]: "Return statement",
-    };
-
-    /**
-     * Checks if an expression statement is a function call (side effect)
-     * @param {import('eslint').Rule.Node} statement - AST node
-     * @returns {boolean} True if this is a function call expression
-     */
-    function isFunctionCallExpression(statement) {
-      if (statement.type !== "ExpressionStatement") {
-        return false;
-      }
-
-      const expression = statement.expression;
-
-      // Direct call: doSomething() or object.method()
-      if (expression.type === "CallExpression") {
-        return true;
-      }
-
-      return false;
-    }
-
-    /**
-     * Determines the order category of a statement
-     * Guard clauses (if statements with early returns) are ignored and don't
-     * affect order validation - they can appear anywhere in the function.
-     * @param {import('eslint').Rule.Node} statement - AST node
-     * @returns {number|null} Order category value, or null to skip this statement
-     */
-    function getStatementOrder(statement) {
-      // Bare return statement (not in an if block)
-      if (statement.type === "ReturnStatement") {
-        return ORDER.RETURN;
-      }
-
-      // If statements are ignored - they may contain guard clauses (early returns)
-      // which are valid at any position in the function
-      if (statement.type === "IfStatement") {
-        return null;
-      }
-
-      // Variable declarations are definitions
-      if (statement.type === "VariableDeclaration") {
-        return ORDER.DEFINITION;
-      }
-
-      // Function declarations are definitions
-      if (statement.type === "FunctionDeclaration") {
-        return ORDER.DEFINITION;
-      }
-
-      // Expression statements with function calls are side effects
-      if (isFunctionCallExpression(statement)) {
-        return ORDER.SIDE_EFFECT;
-      }
-
-      // Default to null for other statements (they don't affect order)
-      return null;
-    }
-
-    /**
-     * Checks if a function body follows the correct statement order
-     * @param {import('eslint').Rule.Node} node - Function node
-     */
-    function checkBodyOrder(node) {
-      if (!node.body || node.body.type !== "BlockStatement") {
-        return;
-      }
-
-      const statements = node.body.body;
-      // eslint-disable-next-line functional/no-let -- ESLint plugin requires mutable tracking variable for order validation
-      let maxOrderSeen = 0;
-
-      statements.forEach(statement => {
-        const currentOrder = getStatementOrder(statement);
-
-        // Skip statements that don't affect order (e.g., if statements)
-        if (currentOrder === null) {
-          return;
-        }
-
-        if (currentOrder < maxOrderSeen) {
-          context.report({
-            node: statement,
-            messageId: "wrongOrder",
-            data: {
-              current: ORDER_NAMES[currentOrder],
-              previous: ORDER_NAMES[maxOrderSeen],
-            },
-          });
-        }
-
-        // Track the highest order we've seen so far
-        if (currentOrder > maxOrderSeen) {
-          maxOrderSeen = currentOrder;
-        }
-      });
-    }
-
-    return {
-      // Check all function declarations
-      FunctionDeclaration(node) {
-        checkBodyOrder(node);
-      },
-
-      // Check all arrow functions and function expressions assigned to variables
-      VariableDeclarator(node) {
-        if (
-          node.init &&
-          (node.init.type === "ArrowFunctionExpression" ||
-            node.init.type === "FunctionExpression")
-        ) {
-          checkBodyOrder(node.init);
-        }
-      },
-    };
-  },
-};

package/expo/copy-overwrite/eslint-plugin-component-structure/README.md

@@ -1,234 +0,0 @@
-# ESLint Plugin: Component Structure
-
-Custom ESLint rules for enforcing component structure standards in the frontend.
-
-## Rules
-
-### enforce-component-structure
-
-Enforces the Container/View pattern for React components by ensuring component directories contain exactly three files: `*Container.tsx`, `*View.tsx`, and `index.tsx`.
-
-This rule helps maintain consistent component organization and separation of concerns throughout the codebase.
-
-#### Rule Details
-
-This rule checks component directories to ensure they follow the standard three-file structure:
-
-- `ComponentNameContainer.tsx` - Contains logic, state, and data fetching
-- `ComponentNameView.tsx` - Pure UI presentation receiving props only
-- `index.tsx` - Default export of the Container
-
-**Where does this rule apply?**
-
-- Files in `features/**/components/` directories
-- Files in `components/` directory
-- **Excludes**: `components/ui/`, `components/shared/`, `components/icons/`
-
-#### Configuration
-
-In `.eslintrc.json`:
-
-```json
-{
-  "rules": {
-    "component-structure/enforce-component-structure": "error"
-  }
-}
-```
-
----
-
-### no-return-in-view
-
-Prevents early returns and conditional logic in View components to maintain pure presentation components.
-
-#### Rule Details
-
-View components should only contain JSX presentation logic. Any conditional rendering or early returns should be handled in the Container component or passed as props.
-
-**Where does this rule apply?**
-
-- Files matching `*View.tsx`, `*View.jsx`
-
-#### Configuration
-
-In `.eslintrc.json`:
-
-```json
-{
-  "rules": {
-    "component-structure/no-return-in-view": "error"
-  }
-}
-```
-
----
-
-### require-memo-in-view
-
-Enforces the use of `React.memo()` for View components to optimize rendering performance.
-
-#### Rule Details
-
-All View components should be wrapped with `React.memo()` to prevent unnecessary re-renders when props haven't changed.
-
-**Where does this rule apply?**
-
-- Files matching `*View.tsx`, `*View.jsx`
-- **Excludes**: `components/ui/` directory
-
-#### Configuration
-
-In `.eslintrc.json`:
-
-```json
-{
-  "rules": {
-    "component-structure/require-memo-in-view": "error"
-  }
-}
-```
-
----
-
-### single-component-per-file
-
-Enforces one React component per file to improve code organization and maintainability.
-
-#### Rule Details
-
-This rule ensures that each file contains exactly one React component. Multiple components in a single file make code harder to navigate, test, and maintain.
-
-**What is considered a React component?**
-
-- Any function with PascalCase naming that returns JSX
-- This includes components wrapped with `memo()` or `React.memo()`
-- Both arrow functions and function declarations
-
-**Where does this rule apply?**
-
-- Files matching `*View.tsx`, `*View.jsx`, `*Container.tsx`, `*Container.jsx`
-- In `features/**/components/` directories
-- In `components/` directory
-- **Excludes**: `components/ui/`, `components/shared/`, `components/icons/`
-
-#### Examples
-
-❌ **Incorrect** (multiple components):
-
-```typescript
-// MessageListView.tsx
-const MessageItem = ({ item }) => <div>{item}</div>;
-
-const MessageListView = ({ messages }) => (
-  <div>
-    {messages.map(msg => <MessageItem item={msg} />)}
-  </div>
-);
-
-export default MessageListView;
-```
-
-✅ **Correct** (one component per file):
-
-```typescript
-// MessageItem.tsx
-const MessageItem = ({ item }) => <div>{item}</div>;
-export default MessageItem;
-
-// MessageListView.tsx
-import MessageItem from "./MessageItem";
-
-const MessageListView = ({ messages }) => (
-  <div>
-    {messages.map(msg => <MessageItem item={msg} />)}
-  </div>
-);
-
-export default MessageListView;
-```
-
-#### Configuration
-
-In `.eslintrc.json`:
-
-```json
-{
-  "rules": {
-    "component-structure/single-component-per-file": "error"
-  }
-}
-```
-
-#### When to Disable
-
-This rule should rarely be disabled. If you have a valid use case, use an inline comment:
-
-```typescript
-/* eslint-disable component-structure/single-component-per-file --
-   Reason: These components are tightly coupled and only used together
-*/
-```
-
-## Installation
-
-This plugin is installed locally as a file dependency:
-
-```json
-{
-  "devDependencies": {
-    "eslint-plugin-component-structure": "file:./eslint-plugin-component-structure"
-  }
-}
-```
-
-## Usage
-
-### ESLint 9 Flat Config (Recommended)
-
-```javascript
-// eslint.config.mjs
-import componentStructurePlugin from './eslint-plugin-component-structure/index.js';
-
-export default [
-  {
-    plugins: {
-      'component-structure': componentStructurePlugin,
-    },
-    rules: {
-      'component-structure/enforce-component-structure': 'error',
-      'component-structure/no-return-in-view': 'error',
-      'component-structure/require-memo-in-view': 'error',
-      'component-structure/single-component-per-file': 'error',
-    },
-  },
-];
-```
-
-### Legacy Config (.eslintrc.json)
-
-```json
-{
-  "plugins": ["component-structure"],
-  "rules": {
-    "component-structure/enforce-component-structure": "error",
-    "component-structure/no-return-in-view": "error",
-    "component-structure/require-memo-in-view": "error",
-    "component-structure/single-component-per-file": "error"
-  }
-}
-```
-
-## Version
-
-1.0.0
-
-## Contributing
-
-When adding new rules to this plugin:
-
-1. Create the rule implementation in `rules/`
-2. Add comprehensive tests in `__tests__/`
-3. Export the rule in `index.js`
-4. Update this README with rule documentation
-5. Add the rule to `.eslintrc.json` configuration

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

@@ -1,89 +0,0 @@
-/**
- * This file is managed by Lisa.
- * Do not edit directly — changes will be overwritten on the next `lisa` run.
- */
-
-/**
- * Unit tests for the plugin index
- *
- * Verifies that all rules are properly registered and exported from the plugin.
- * Also verifies that the plugin has proper ESLint 9 flat config metadata.
- * @module eslint-plugin-component-structure/tests/plugin-index
- */
-
-const plugin = require("../index");
-
-describe("eslint-plugin-component-structure", () => {
-  describe("plugin meta (ESLint 9 flat config)", () => {
-    it("should export a meta object", () => {
-      expect(plugin).toHaveProperty("meta");
-      expect(typeof plugin.meta).toBe("object");
-    });
-
-    it("should have name property in meta", () => {
-      expect(plugin.meta).toHaveProperty("name");
-      expect(plugin.meta.name).toBe("eslint-plugin-component-structure");
-    });
-
-    it("should have version property in meta", () => {
-      expect(plugin.meta).toHaveProperty("version");
-      expect(plugin.meta.version).toBe("1.0.0");
-    });
-  });
-
-  describe("plugin exports", () => {
-    it("should export a rules object", () => {
-      expect(plugin).toHaveProperty("rules");
-      expect(typeof plugin.rules).toBe("object");
-    });
-
-    it("should export enforce-component-structure rule", () => {
-      expect(plugin.rules).toHaveProperty("enforce-component-structure");
-      expect(typeof plugin.rules["enforce-component-structure"]).toBe("object");
-      expect(plugin.rules["enforce-component-structure"]).toHaveProperty(
-        "create"
-      );
-      expect(plugin.rules["enforce-component-structure"]).toHaveProperty(
-        "meta"
-      );
-    });
-
-    it("should export no-return-in-view rule", () => {
-      expect(plugin.rules).toHaveProperty("no-return-in-view");
-      expect(typeof plugin.rules["no-return-in-view"]).toBe("object");
-      expect(plugin.rules["no-return-in-view"]).toHaveProperty("create");
-      expect(plugin.rules["no-return-in-view"]).toHaveProperty("meta");
-    });
-
-    it("should export require-memo-in-view rule", () => {
-      expect(plugin.rules).toHaveProperty("require-memo-in-view");
-      expect(typeof plugin.rules["require-memo-in-view"]).toBe("object");
-      expect(plugin.rules["require-memo-in-view"]).toHaveProperty("create");
-      expect(plugin.rules["require-memo-in-view"]).toHaveProperty("meta");
-    });
-
-    it("should export single-component-per-file rule", () => {
-      expect(plugin.rules).toHaveProperty("single-component-per-file");
-      expect(typeof plugin.rules["single-component-per-file"]).toBe("object");
-      expect(plugin.rules["single-component-per-file"]).toHaveProperty(
-        "create"
-      );
-      expect(plugin.rules["single-component-per-file"]).toHaveProperty("meta");
-    });
-
-    it("should have correct number of rules", () => {
-      const ruleCount = Object.keys(plugin.rules).length;
-      expect(ruleCount).toBe(4);
-    });
-  });
-
-  describe("rule metadata", () => {
-    it("single-component-per-file should have correct metadata", () => {
-      const rule = plugin.rules["single-component-per-file"];
-      expect(rule.meta.type).toBe("problem");
-      expect(rule.meta.docs).toHaveProperty("description");
-      expect(rule.meta.docs.description).toContain("one React component");
-      expect(rule.meta.messages).toHaveProperty("multipleComponents");
-    });
-  });
-});

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

@@ -1,201 +0,0 @@
-/**
- * This file is managed by Lisa.
- * Do not edit directly — changes will be overwritten on the next `lisa` run.
- */
-
-/**
- * Unit tests for the require-memo-in-view ESLint rule
- *
- * Tests that View components are properly wrapped with React.memo() and have displayName.
- * Ensures components/ui/** directory is excluded from the rule.
- * @module eslint-plugin-component-structure/tests
- */
-
-const { RuleTester } = require("eslint");
-
-const rule = require("../rules/require-memo-in-view");
-
-const FEATURE_EXAMPLE_COMPONENTS_PATH = "features/example/components";
-
-const ruleTester = new RuleTester({
-  languageOptions: {
-    ecmaVersion: 2020,
-    sourceType: "module",
-    parserOptions: {
-      ecmaFeatures: {
-        jsx: true,
-      },
-    },
-  },
-});
-
-ruleTester.run("require-memo-in-view", rule, {
-  valid: [
-    {
-      code: `
-        import { memo } from "react";
-        import { View } from "react-native";
-
-        const MyView = () => <View />;
-
-        MyView.displayName = "MyView";
-
-        export default memo(MyView);
-      `,
-      filename: `${FEATURE_EXAMPLE_COMPONENTS_PATH}/MyView.tsx`,
-    },
-    {
-      code: `
-        import { memo } from "react";
-        import { Text } from "react-native";
-
-        const AnotherView = () => (
-          <Text>Hello</Text>
-        );
-
-        AnotherView.displayName = "AnotherView";
-
-        export default memo(AnotherView);
-      `,
-      filename: "components/AnotherView.tsx",
-    },
-    {
-      code: `
-        // This should be ignored - components/ui/** is excluded
-        import { View } from "react-native";
-
-        const UIComponent = () => <View />;
-
-        export default UIComponent;
-      `,
-      filename: "components/ui/Button/ButtonView.tsx",
-    },
-    {
-      code: `
-        // This should be ignored - components/custom/ui/** is excluded
-        import { View } from "react-native";
-
-        const CustomUIComponent = () => <View />;
-
-        export default CustomUIComponent;
-      `,
-      filename: "components/custom/ui/Input/InputView.tsx",
-    },
-    {
-      code: `
-        // Non-View file should be ignored
-        import { memo } from "react";
-
-        const MyContainer = () => null;
-
-        export default MyContainer;
-      `,
-      filename: `${FEATURE_EXAMPLE_COMPONENTS_PATH}/MyContainer.tsx`,
-    },
-    {
-      code: `
-        import React from "react";
-        import { View } from "react-native";
-
-        const MyView = () => <View />;
-
-        MyView.displayName = "MyView";
-
-        export default React.memo(MyView);
-      `,
-      filename: `${FEATURE_EXAMPLE_COMPONENTS_PATH}/MyView.tsx`,
-    },
-  ],
-  invalid: [
-    {
-      code: `
-        import { View } from "react-native";
-
-        const MyView = () => <View />;
-
-        MyView.displayName = "MyView";
-
-        export default MyView;
-      `,
-      filename: `${FEATURE_EXAMPLE_COMPONENTS_PATH}/MyView.tsx`,
-      errors: [
-        {
-          messageId: "missingMemo",
-          data: { componentName: "MyView" },
-        },
-      ],
-    },
-    {
-      code: `
-        import { memo } from "react";
-        import { View } from "react-native";
-
-        const MyView = () => <View />;
-
-        export default memo(MyView);
-      `,
-      filename: `${FEATURE_EXAMPLE_COMPONENTS_PATH}/MyView.tsx`,
-      errors: [
-        {
-          messageId: "missingDisplayName",
-          data: { componentName: "MyView" },
-        },
-      ],
-    },
-    {
-      code: `
-        import { View } from "react-native";
-
-        const MyView = () => <View />;
-
-        export default MyView;
-      `,
-      filename: `${FEATURE_EXAMPLE_COMPONENTS_PATH}/MyView.tsx`,
-      errors: [
-        {
-          messageId: "missingMemo",
-          data: { componentName: "MyView" },
-        },
-        {
-          messageId: "missingDisplayName",
-          data: { componentName: "MyView" },
-        },
-      ],
-    },
-    {
-      code: `
-        import React from "react";
-        import { View } from "react-native";
-
-        const MyView = () => <View />;
-
-        export default React.memo(MyView);
-      `,
-      filename: `${FEATURE_EXAMPLE_COMPONENTS_PATH}/MyView.tsx`,
-      errors: [
-        {
-          messageId: "missingDisplayName",
-          data: { componentName: "MyView" },
-        },
-      ],
-    },
-    {
-      code: `
-        import { View } from "react-native";
-
-        const TestView = () => <View />;
-
-        TestView.displayName = "TestView";
-
-        export default TestView;
-      `,
-      filename: "features/test/components/TestView.jsx",
-      errors: [
-        {
-          messageId: "missingMemo",
-          data: { componentName: "TestView" },
-        },
-      ],
-    },
-  ],
-});