Add new no-empty-array-object-default-in-useOnyx rule

Author: VickyStash

eslint-plugin-expensify/no-empty-array-object-default-in-useOnyx.js

@@ -0,0 +1,62 @@
+/**
+ * @typedef {import('eslint').Rule.RuleModule} RuleModule
+ */
+
+/** @type {RuleModule} */
+module.exports = {
+    name: 'no-empty-array-object-default-in-useOnyx',
+    meta: {
+        type: 'problem',
+        docs: {
+            description: 'Disallow empty array or object as default values in useOnyx destructuring assignments. Use getEmptyArray() or getEmptyObject() instead.',
+            recommended: 'error',
+        },
+        schema: [],
+        messages: {
+            noEmptyArrayDefault: 'Avoid using empty array [] as default value in useOnyx. Use getEmptyArray<T>() instead.',
+            noEmptyObjectDefault: 'Avoid using empty object {} as default value in useOnyx. Use getEmptyObject<T>() instead.',
+        },
+    },
+    create(context) {
+        return {
+            VariableDeclarator(node) {
+                if (
+                    !node.init
+                    || node.init.type !== 'CallExpression'
+                    || node.init.callee.name !== 'useOnyx'
+                ) {
+                    return;
+                }
+
+                // Check if the variable declaration is an array pattern (destructuring)
+                if (node.id.type !== 'ArrayPattern') {
+                    return;
+                }
+
+                node.id.elements.forEach((element) => {
+                    if (!element || element.type !== 'AssignmentPattern') {
+                        return;
+                    }
+
+                    const defaultValue = element.right;
+
+                    // Check for empty array default
+                    if (defaultValue.type === 'ArrayExpression' && defaultValue.elements.length === 0) {
+                        context.report({
+                            node: defaultValue,
+                            messageId: 'noEmptyArrayDefault',
+                        });
+                    }
+
+                    // Check for empty object default
+                    if (defaultValue.type === 'ObjectExpression' && defaultValue.properties.length === 0) {
+                        context.report({
+                            node: defaultValue,
+                            messageId: 'noEmptyObjectDefault',
+                        });
+                    }
+                });
+            },
+        };
+    },
+};

eslint-plugin-expensify/tests/no-empty-array-object-default-in-useOnyx.test.js

@@ -0,0 +1,64 @@
+const RuleTester = require('eslint').RuleTester;
+const rule = require('../no-empty-array-object-default-in-useOnyx');
+
+const ruleTester = new RuleTester({
+    parserOptions: {
+        ecmaVersion: 2020,
+        sourceType: 'module',
+    },
+});
+
+ruleTester.run('no-empty-array-object-default-in-useOnyx', rule, {
+    valid: [
+        {
+            code: 'const [data = getEmptyObject()] = useOnyx(ONYXKEYS.DATA);',
+        },
+        {
+            code: 'const [data = getEmptyArray()] = useOnyx(ONYXKEYS.DATA);',
+        },
+        {
+            code: 'const [data] = useOnyx(ONYXKEYS.DATA);',
+        },
+        {
+            code: 'const [data = null] = useOnyx(ONYXKEYS.DATA);',
+        },
+        {
+            code: 'const [data = undefined] = useOnyx(ONYXKEYS.DATA);',
+        },
+        {
+            code: 'const [data = false] = useOnyx(ONYXKEYS.DATA);',
+        },
+        {
+            code: 'const [data = ""] = useOnyx(ONYXKEYS.DATA);',
+        },
+        {
+            code: 'const [data = 0] = useOnyx(ONYXKEYS.DATA);',
+        },
+        {
+            code: 'const [data = someVariable] = useOnyx(ONYXKEYS.DATA);',
+        },
+        {
+            code: 'const [data = {key: "value"}] = useOnyx(ONYXKEYS.DATA);',
+        },
+        {
+            code: 'const [data = ["item"]] = useOnyx(ONYXKEYS.DATA);',
+        },
+        {
+            code: 'const data = useOnyx(ONYXKEYS.DATA);',
+        },
+    ],
+    invalid: [
+        {
+            code: 'const [data = {}] = useOnyx(ONYXKEYS.DATA);',
+            errors: [{
+                messageId: 'noEmptyObjectDefault',
+            }],
+        },
+        {
+            code: 'const [data = []] = useOnyx(ONYXKEYS.DATA);',
+            errors: [{
+                messageId: 'noEmptyArrayDefault',
+            }],
+        },
+    ],
+});

rules/expensify.js

@@ -31,6 +31,7 @@ module.exports = {
         }],
         'rulesdir/prefer-shouldUseNarrowLayout-instead-of-isSmallScreenWidth': 'warn',
         'rulesdir/no-use-state-initializer-functions': 'error',
+        'rulesdir/no-empty-array-object-default-in-useOnyx': 'error',
     },
 };