@futdevpro/dynamo-eslint 1.14.6 → 1.14.8

package/build/plugin/rules/require-test-description-prefix.spec.js

@@ -0,0 +1,371 @@
+"use strict";
+Object.defineProperty(exports, "__esModule", { value: true });
+const tslib_1 = require("tslib");
+const require_test_description_prefix_1 = tslib_1.__importDefault(require("./require-test-description-prefix"));
+describe('| require-test-description-prefix', () => {
+    it('| should be a valid ESLint rule', () => {
+        expect(require_test_description_prefix_1.default.meta.type).toBe('suggestion');
+        expect(require_test_description_prefix_1.default.meta.docs.description).toBe('Require "| " prefix in test descriptions');
+        expect(require_test_description_prefix_1.default.meta.fixable).toBe('code');
+    });
+    it('| should not report when describe has correct prefix', () => {
+        let reportCalled = false;
+        const mockContext = {
+            report: (options) => {
+                reportCalled = true;
+            },
+            options: [{ prefix: '| ' }],
+        };
+        const rule = require_test_description_prefix_1.default.create(mockContext);
+        const mockNode = {
+            type: 'CallExpression',
+            callee: { name: 'describe' },
+            arguments: [
+                { type: 'Literal', value: '| my test suite' },
+            ],
+        };
+        rule.CallExpression(mockNode);
+        expect(reportCalled).toBe(false);
+    });
+    it('| should not report when it has correct prefix', () => {
+        let reportCalled = false;
+        const mockContext = {
+            report: (options) => {
+                reportCalled = true;
+            },
+            options: [{ prefix: '| ' }],
+        };
+        const rule = require_test_description_prefix_1.default.create(mockContext);
+        const mockNode = {
+            type: 'CallExpression',
+            callee: { name: 'it' },
+            arguments: [
+                { type: 'Literal', value: '| should do something' },
+            ],
+        };
+        rule.CallExpression(mockNode);
+        expect(reportCalled).toBe(false);
+    });
+    it('| should not report when test has correct prefix', () => {
+        let reportCalled = false;
+        const mockContext = {
+            report: (options) => {
+                reportCalled = true;
+            },
+            options: [{ prefix: '| ' }],
+        };
+        const rule = require_test_description_prefix_1.default.create(mockContext);
+        const mockNode = {
+            type: 'CallExpression',
+            callee: { name: 'test' },
+            arguments: [
+                { type: 'Literal', value: '| validates input' },
+            ],
+        };
+        rule.CallExpression(mockNode);
+        expect(reportCalled).toBe(false);
+    });
+    it('| should report when describe is missing prefix', () => {
+        let reportCalled = false;
+        const mockContext = {
+            report: (options) => {
+                reportCalled = true;
+                expect(options.messageId).toBe('missingPrefix');
+                expect(options.data.prefix).toBe('| ');
+                expect(options.fix).toBeDefined();
+            },
+            options: [{ prefix: '| ' }],
+        };
+        const rule = require_test_description_prefix_1.default.create(mockContext);
+        const mockNode = {
+            type: 'CallExpression',
+            callee: { name: 'describe' },
+            arguments: [
+                { type: 'Literal', value: 'my test suite' },
+            ],
+        };
+        rule.CallExpression(mockNode);
+        expect(reportCalled).toBe(true);
+    });
+    it('| should report when it is missing prefix', () => {
+        let reportCalled = false;
+        const mockContext = {
+            report: (options) => {
+                reportCalled = true;
+                expect(options.messageId).toBe('missingPrefix');
+                expect(options.data.prefix).toBe('| ');
+            },
+            options: [{ prefix: '| ' }],
+        };
+        const rule = require_test_description_prefix_1.default.create(mockContext);
+        const mockNode = {
+            type: 'CallExpression',
+            callee: { name: 'it' },
+            arguments: [
+                { type: 'Literal', value: 'should do something' },
+            ],
+        };
+        rule.CallExpression(mockNode);
+        expect(reportCalled).toBe(true);
+    });
+    it('| should report when describe has wrong prefix', () => {
+        let reportCalled = false;
+        const mockContext = {
+            report: (options) => {
+                reportCalled = true;
+                expect(options.messageId).toBe('wrongPrefix');
+                expect(options.data.prefix).toBe('| ');
+                expect(options.data.found).toBe('|');
+            },
+            options: [{ prefix: '| ' }],
+        };
+        const rule = require_test_description_prefix_1.default.create(mockContext);
+        const mockNode = {
+            type: 'CallExpression',
+            callee: { name: 'describe' },
+            arguments: [
+                { type: 'Literal', value: '|my test suite' },
+            ],
+        };
+        rule.CallExpression(mockNode);
+        expect(reportCalled).toBe(true);
+    });
+    it('| should report when it has only pipe without space', () => {
+        let reportCalled = false;
+        const mockContext = {
+            report: (options) => {
+                reportCalled = true;
+                expect(options.messageId).toBe('wrongPrefix');
+                expect(options.data.found).toBe('|');
+            },
+            options: [{ prefix: '| ' }],
+        };
+        const rule = require_test_description_prefix_1.default.create(mockContext);
+        const mockNode = {
+            type: 'CallExpression',
+            callee: { name: 'it' },
+            arguments: [
+                { type: 'Literal', value: '|should do something' },
+            ],
+        };
+        rule.CallExpression(mockNode);
+        expect(reportCalled).toBe(true);
+    });
+    it('| should not report on non-test functions', () => {
+        let reportCalled = false;
+        const mockContext = {
+            report: (options) => {
+                reportCalled = true;
+            },
+            options: [{ prefix: '| ' }],
+        };
+        const rule = require_test_description_prefix_1.default.create(mockContext);
+        const mockNode = {
+            type: 'CallExpression',
+            callee: { name: 'console' },
+            arguments: [
+                { type: 'Literal', value: 'log message' },
+            ],
+        };
+        rule.CallExpression(mockNode);
+        expect(reportCalled).toBe(false);
+    });
+    it('| should not report when no arguments provided', () => {
+        let reportCalled = false;
+        const mockContext = {
+            report: (options) => {
+                reportCalled = true;
+            },
+            options: [{ prefix: '| ' }],
+        };
+        const rule = require_test_description_prefix_1.default.create(mockContext);
+        const mockNode = {
+            type: 'CallExpression',
+            callee: { name: 'describe' },
+            arguments: [],
+        };
+        rule.CallExpression(mockNode);
+        expect(reportCalled).toBe(false);
+    });
+    it('| should not report when first argument is not a string', () => {
+        let reportCalled = false;
+        const mockContext = {
+            report: (options) => {
+                reportCalled = true;
+            },
+            options: [{ prefix: '| ' }],
+        };
+        const rule = require_test_description_prefix_1.default.create(mockContext);
+        const mockNode = {
+            type: 'CallExpression',
+            callee: { name: 'describe' },
+            arguments: [
+                { type: 'Identifier', name: 'myVariable' },
+            ],
+        };
+        rule.CallExpression(mockNode);
+        expect(reportCalled).toBe(false);
+    });
+    it('| should handle beforeEach and afterEach', () => {
+        let reportCalled = false;
+        const mockContext = {
+            report: (options) => {
+                reportCalled = true;
+                expect(options.messageId).toBe('missingPrefix');
+            },
+            options: [{ prefix: '| ' }],
+        };
+        const rule = require_test_description_prefix_1.default.create(mockContext);
+        const mockNode = {
+            type: 'CallExpression',
+            callee: { name: 'beforeEach' },
+            arguments: [
+                { type: 'Literal', value: 'setup test data' },
+            ],
+        };
+        rule.CallExpression(mockNode);
+        expect(reportCalled).toBe(true);
+    });
+    it('| should handle beforeAll and afterAll', () => {
+        let reportCalled = false;
+        const mockContext = {
+            report: (options) => {
+                reportCalled = true;
+                expect(options.messageId).toBe('missingPrefix');
+            },
+            options: [{ prefix: '| ' }],
+        };
+        const rule = require_test_description_prefix_1.default.create(mockContext);
+        const mockNode = {
+            type: 'CallExpression',
+            callee: { name: 'beforeAll' },
+            arguments: [
+                { type: 'Literal', value: 'setup suite' },
+            ],
+        };
+        rule.CallExpression(mockNode);
+        expect(reportCalled).toBe(true);
+    });
+    it('| should handle custom prefix configuration', () => {
+        let reportCalled = false;
+        const mockContext = {
+            report: (options) => {
+                reportCalled = true;
+                expect(options.data.prefix).toBe('>>> ');
+            },
+            options: [{ prefix: '>>> ' }],
+        };
+        const rule = require_test_description_prefix_1.default.create(mockContext);
+        const mockNode = {
+            type: 'CallExpression',
+            callee: { name: 'describe' },
+            arguments: [
+                { type: 'Literal', value: 'my test suite' },
+            ],
+        };
+        rule.CallExpression(mockNode);
+        expect(reportCalled).toBe(true);
+    });
+    it('| should handle custom test functions configuration', () => {
+        let reportCalled = false;
+        const mockContext = {
+            report: (options) => {
+                reportCalled = true;
+            },
+            options: [{ testFunctions: ['myTest'] }],
+        };
+        const rule = require_test_description_prefix_1.default.create(mockContext);
+        const mockNode = {
+            type: 'CallExpression',
+            callee: { name: 'myTest' },
+            arguments: [
+                { type: 'Literal', value: 'my test description' },
+            ],
+        };
+        rule.CallExpression(mockNode);
+        expect(reportCalled).toBe(true);
+    });
+    it('| should provide auto-fix functionality', () => {
+        const mockContext = {
+            report: (options) => {
+                expect(options.fix).toBeDefined();
+                expect(typeof options.fix).toBe('function');
+                // Test the fix function
+                const fixResult = options.fix({
+                    replaceText: (node, text) => {
+                        expect(text).toBe('\'| my test suite\'');
+                        return [];
+                    },
+                });
+                expect(Array.isArray(fixResult)).toBe(true);
+            },
+            options: [{ prefix: '| ' }],
+        };
+        const rule = require_test_description_prefix_1.default.create(mockContext);
+        const mockNode = {
+            type: 'CallExpression',
+            callee: { name: 'describe' },
+            arguments: [
+                { type: 'Literal', value: 'my test suite' },
+            ],
+        };
+        rule.CallExpression(mockNode);
+    });
+    it('| should handle empty string descriptions', () => {
+        let reportCalled = false;
+        const mockContext = {
+            report: (options) => {
+                reportCalled = true;
+                expect(options.messageId).toBe('missingPrefix');
+            },
+            options: [{ prefix: '| ' }],
+        };
+        const rule = require_test_description_prefix_1.default.create(mockContext);
+        const mockNode = {
+            type: 'CallExpression',
+            callee: { name: 'describe' },
+            arguments: [
+                { type: 'Literal', value: '' },
+            ],
+        };
+        rule.CallExpression(mockNode);
+        expect(reportCalled).toBe(true);
+    });
+    it('| should handle errors gracefully', () => {
+        const mockContext = {
+            report: (options) => {
+                // Should not throw
+            },
+            options: [{ prefix: '| ' }],
+        };
+        const rule = require_test_description_prefix_1.default.create(mockContext);
+        // Test with malformed nodes
+        const malformedNode = {
+            type: 'CallExpression',
+            // Missing required properties
+        };
+        expect(() => {
+            rule.CallExpression(malformedNode);
+        }).not.toThrow();
+    });
+    it('| should handle missing callee gracefully', () => {
+        let reportCalled = false;
+        const mockContext = {
+            report: (options) => {
+                reportCalled = true;
+            },
+            options: [{ prefix: '| ' }],
+        };
+        const rule = require_test_description_prefix_1.default.create(mockContext);
+        const mockNode = {
+            type: 'CallExpression',
+            // Missing callee
+            arguments: [
+                { type: 'Literal', value: 'my test suite' },
+            ],
+        };
+        rule.CallExpression(mockNode);
+        expect(reportCalled).toBe(false);
+    });
+});
+//# sourceMappingURL=require-test-description-prefix.spec.js.map

package/build/plugin/rules/require-test-description-prefix.spec.js.map

@@ -0,0 +1 @@
+{"version":3,"file":"require-test-description-prefix.spec.js","sourceRoot":"","sources":["../../../src/plugin/rules/require-test-description-prefix.spec.ts"],"names":[],"mappings":";;;AAAA,gHAAsE;AAEtE,QAAQ,CAAC,mCAAmC,EAAE,GAAG,EAAE;IACjD,EAAE,CAAC,iCAAiC,EAAE,GAAG,EAAE;QACzC,MAAM,CAAC,yCAAqB,CAAC,IAAI,CAAC,IAAI,CAAC,CAAC,IAAI,CAAC,YAAY,CAAC,CAAC;QAC3D,MAAM,CAAC,yCAAqB,CAAC,IAAI,CAAC,IAAI,CAAC,WAAW,CAAC,CAAC,IAAI,CAAC,0CAA0C,CAAC,CAAC;QACrG,MAAM,CAAC,yCAAqB,CAAC,IAAI,CAAC,OAAO,CAAC,CAAC,IAAI,CAAC,MAAM,CAAC,CAAC;IAC1D,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,sDAAsD,EAAE,GAAG,EAAE;QAC9D,IAAI,YAAY,GAAG,KAAK,CAAC;QACzB,MAAM,WAAW,GAAG;YAClB,MAAM,EAAE,CAAC,OAAY,EAAE,EAAE;gBACvB,YAAY,GAAG,IAAI,CAAC;YACtB,CAAC;YACD,OAAO,EAAE,CAAC,EAAE,MAAM,EAAE,IAAI,EAAE,CAAC;SACrB,CAAC;QAET,MAAM,IAAI,GAAG,yCAAqB,CAAC,MAAM,CAAC,WAAW,CAAC,CAAC;QAEvD,MAAM,QAAQ,GAAG;YACf,IAAI,EAAE,gBAAgB;YACtB,MAAM,EAAE,EAAE,IAAI,EAAE,UAAU,EAAE;YAC5B,SAAS,EAAE;gBACT,EAAE,IAAI,EAAE,SAAS,EAAE,KAAK,EAAE,iBAAiB,EAAE;aAC9C;SACK,CAAC;QAET,IAAI,CAAC,cAAc,CAAC,QAAQ,CAAC,CAAC;QAE9B,MAAM,CAAC,YAAY,CAAC,CAAC,IAAI,CAAC,KAAK,CAAC,CAAC;IACnC,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,gDAAgD,EAAE,GAAG,EAAE;QACxD,IAAI,YAAY,GAAG,KAAK,CAAC;QACzB,MAAM,WAAW,GAAG;YAClB,MAAM,EAAE,CAAC,OAAY,EAAE,EAAE;gBACvB,YAAY,GAAG,IAAI,CAAC;YACtB,CAAC;YACD,OAAO,EAAE,CAAC,EAAE,MAAM,EAAE,IAAI,EAAE,CAAC;SACrB,CAAC;QAET,MAAM,IAAI,GAAG,yCAAqB,CAAC,MAAM,CAAC,WAAW,CAAC,CAAC;QAEvD,MAAM,QAAQ,GAAG;YACf,IAAI,EAAE,gBAAgB;YACtB,MAAM,EAAE,EAAE,IAAI,EAAE,IAAI,EAAE;YACtB,SAAS,EAAE;gBACT,EAAE,IAAI,EAAE,SAAS,EAAE,KAAK,EAAE,uBAAuB,EAAE;aACpD;SACK,CAAC;QAET,IAAI,CAAC,cAAc,CAAC,QAAQ,CAAC,CAAC;QAE9B,MAAM,CAAC,YAAY,CAAC,CAAC,IAAI,CAAC,KAAK,CAAC,CAAC;IACnC,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,kDAAkD,EAAE,GAAG,EAAE;QAC1D,IAAI,YAAY,GAAG,KAAK,CAAC;QACzB,MAAM,WAAW,GAAG;YAClB,MAAM,EAAE,CAAC,OAAY,EAAE,EAAE;gBACvB,YAAY,GAAG,IAAI,CAAC;YACtB,CAAC;YACD,OAAO,EAAE,CAAC,EAAE,MAAM,EAAE,IAAI,EAAE,CAAC;SACrB,CAAC;QAET,MAAM,IAAI,GAAG,yCAAqB,CAAC,MAAM,CAAC,WAAW,CAAC,CAAC;QAEvD,MAAM,QAAQ,GAAG;YACf,IAAI,EAAE,gBAAgB;YACtB,MAAM,EAAE,EAAE,IAAI,EAAE,MAAM,EAAE;YACxB,SAAS,EAAE;gBACT,EAAE,IAAI,EAAE,SAAS,EAAE,KAAK,EAAE,mBAAmB,EAAE;aAChD;SACK,CAAC;QAET,IAAI,CAAC,cAAc,CAAC,QAAQ,CAAC,CAAC;QAE9B,MAAM,CAAC,YAAY,CAAC,CAAC,IAAI,CAAC,KAAK,CAAC,CAAC;IACnC,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,iDAAiD,EAAE,GAAG,EAAE;QACzD,IAAI,YAAY,GAAG,KAAK,CAAC;QACzB,MAAM,WAAW,GAAG;YAClB,MAAM,EAAE,CAAC,OAAY,EAAE,EAAE;gBACvB,YAAY,GAAG,IAAI,CAAC;gBACpB,MAAM,CAAC,OAAO,CAAC,SAAS,CAAC,CAAC,IAAI,CAAC,eAAe,CAAC,CAAC;gBAChD,MAAM,CAAC,OAAO,CAAC,IAAI,CAAC,MAAM,CAAC,CAAC,IAAI,CAAC,IAAI,CAAC,CAAC;gBACvC,MAAM,CAAC,OAAO,CAAC,GAAG,CAAC,CAAC,WAAW,EAAE,CAAC;YACpC,CAAC;YACD,OAAO,EAAE,CAAC,EAAE,MAAM,EAAE,IAAI,EAAE,CAAC;SACrB,CAAC;QAET,MAAM,IAAI,GAAG,yCAAqB,CAAC,MAAM,CAAC,WAAW,CAAC,CAAC;QAEvD,MAAM,QAAQ,GAAG;YACf,IAAI,EAAE,gBAAgB;YACtB,MAAM,EAAE,EAAE,IAAI,EAAE,UAAU,EAAE;YAC5B,SAAS,EAAE;gBACT,EAAE,IAAI,EAAE,SAAS,EAAE,KAAK,EAAE,eAAe,EAAE;aAC5C;SACK,CAAC;QAET,IAAI,CAAC,cAAc,CAAC,QAAQ,CAAC,CAAC;QAE9B,MAAM,CAAC,YAAY,CAAC,CAAC,IAAI,CAAC,IAAI,CAAC,CAAC;IAClC,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,2CAA2C,EAAE,GAAG,EAAE;QACnD,IAAI,YAAY,GAAG,KAAK,CAAC;QACzB,MAAM,WAAW,GAAG;YAClB,MAAM,EAAE,CAAC,OAAY,EAAE,EAAE;gBACvB,YAAY,GAAG,IAAI,CAAC;gBACpB,MAAM,CAAC,OAAO,CAAC,SAAS,CAAC,CAAC,IAAI,CAAC,eAAe,CAAC,CAAC;gBAChD,MAAM,CAAC,OAAO,CAAC,IAAI,CAAC,MAAM,CAAC,CAAC,IAAI,CAAC,IAAI,CAAC,CAAC;YACzC,CAAC;YACD,OAAO,EAAE,CAAC,EAAE,MAAM,EAAE,IAAI,EAAE,CAAC;SACrB,CAAC;QAET,MAAM,IAAI,GAAG,yCAAqB,CAAC,MAAM,CAAC,WAAW,CAAC,CAAC;QAEvD,MAAM,QAAQ,GAAG;YACf,IAAI,EAAE,gBAAgB;YACtB,MAAM,EAAE,EAAE,IAAI,EAAE,IAAI,EAAE;YACtB,SAAS,EAAE;gBACT,EAAE,IAAI,EAAE,SAAS,EAAE,KAAK,EAAE,qBAAqB,EAAE;aAClD;SACK,CAAC;QAET,IAAI,CAAC,cAAc,CAAC,QAAQ,CAAC,CAAC;QAE9B,MAAM,CAAC,YAAY,CAAC,CAAC,IAAI,CAAC,IAAI,CAAC,CAAC;IAClC,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,gDAAgD,EAAE,GAAG,EAAE;QACxD,IAAI,YAAY,GAAG,KAAK,CAAC;QACzB,MAAM,WAAW,GAAG;YAClB,MAAM,EAAE,CAAC,OAAY,EAAE,EAAE;gBACvB,YAAY,GAAG,IAAI,CAAC;gBACpB,MAAM,CAAC,OAAO,CAAC,SAAS,CAAC,CAAC,IAAI,CAAC,aAAa,CAAC,CAAC;gBAC9C,MAAM,CAAC,OAAO,CAAC,IAAI,CAAC,MAAM,CAAC,CAAC,IAAI,CAAC,IAAI,CAAC,CAAC;gBACvC,MAAM,CAAC,OAAO,CAAC,IAAI,CAAC,KAAK,CAAC,CAAC,IAAI,CAAC,GAAG,CAAC,CAAC;YACvC,CAAC;YACD,OAAO,EAAE,CAAC,EAAE,MAAM,EAAE,IAAI,EAAE,CAAC;SACrB,CAAC;QAET,MAAM,IAAI,GAAG,yCAAqB,CAAC,MAAM,CAAC,WAAW,CAAC,CAAC;QAEvD,MAAM,QAAQ,GAAG;YACf,IAAI,EAAE,gBAAgB;YACtB,MAAM,EAAE,EAAE,IAAI,EAAE,UAAU,EAAE;YAC5B,SAAS,EAAE;gBACT,EAAE,IAAI,EAAE,SAAS,EAAE,KAAK,EAAE,gBAAgB,EAAE;aAC7C;SACK,CAAC;QAET,IAAI,CAAC,cAAc,CAAC,QAAQ,CAAC,CAAC;QAE9B,MAAM,CAAC,YAAY,CAAC,CAAC,IAAI,CAAC,IAAI,CAAC,CAAC;IAClC,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,qDAAqD,EAAE,GAAG,EAAE;QAC7D,IAAI,YAAY,GAAG,KAAK,CAAC;QACzB,MAAM,WAAW,GAAG;YAClB,MAAM,EAAE,CAAC,OAAY,EAAE,EAAE;gBACvB,YAAY,GAAG,IAAI,CAAC;gBACpB,MAAM,CAAC,OAAO,CAAC,SAAS,CAAC,CAAC,IAAI,CAAC,aAAa,CAAC,CAAC;gBAC9C,MAAM,CAAC,OAAO,CAAC,IAAI,CAAC,KAAK,CAAC,CAAC,IAAI,CAAC,GAAG,CAAC,CAAC;YACvC,CAAC;YACD,OAAO,EAAE,CAAC,EAAE,MAAM,EAAE,IAAI,EAAE,CAAC;SACrB,CAAC;QAET,MAAM,IAAI,GAAG,yCAAqB,CAAC,MAAM,CAAC,WAAW,CAAC,CAAC;QAEvD,MAAM,QAAQ,GAAG;YACf,IAAI,EAAE,gBAAgB;YACtB,MAAM,EAAE,EAAE,IAAI,EAAE,IAAI,EAAE;YACtB,SAAS,EAAE;gBACT,EAAE,IAAI,EAAE,SAAS,EAAE,KAAK,EAAE,sBAAsB,EAAE;aACnD;SACK,CAAC;QAET,IAAI,CAAC,cAAc,CAAC,QAAQ,CAAC,CAAC;QAE9B,MAAM,CAAC,YAAY,CAAC,CAAC,IAAI,CAAC,IAAI,CAAC,CAAC;IAClC,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,2CAA2C,EAAE,GAAG,EAAE;QACnD,IAAI,YAAY,GAAG,KAAK,CAAC;QACzB,MAAM,WAAW,GAAG;YAClB,MAAM,EAAE,CAAC,OAAY,EAAE,EAAE;gBACvB,YAAY,GAAG,IAAI,CAAC;YACtB,CAAC;YACD,OAAO,EAAE,CAAC,EAAE,MAAM,EAAE,IAAI,EAAE,CAAC;SACrB,CAAC;QAET,MAAM,IAAI,GAAG,yCAAqB,CAAC,MAAM,CAAC,WAAW,CAAC,CAAC;QAEvD,MAAM,QAAQ,GAAG;YACf,IAAI,EAAE,gBAAgB;YACtB,MAAM,EAAE,EAAE,IAAI,EAAE,SAAS,EAAE;YAC3B,SAAS,EAAE;gBACT,EAAE,IAAI,EAAE,SAAS,EAAE,KAAK,EAAE,aAAa,EAAE;aAC1C;SACK,CAAC;QAET,IAAI,CAAC,cAAc,CAAC,QAAQ,CAAC,CAAC;QAE9B,MAAM,CAAC,YAAY,CAAC,CAAC,IAAI,CAAC,KAAK,CAAC,CAAC;IACnC,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,gDAAgD,EAAE,GAAG,EAAE;QACxD,IAAI,YAAY,GAAG,KAAK,CAAC;QACzB,MAAM,WAAW,GAAG;YAClB,MAAM,EAAE,CAAC,OAAY,EAAE,EAAE;gBACvB,YAAY,GAAG,IAAI,CAAC;YACtB,CAAC;YACD,OAAO,EAAE,CAAC,EAAE,MAAM,EAAE,IAAI,EAAE,CAAC;SACrB,CAAC;QAET,MAAM,IAAI,GAAG,yCAAqB,CAAC,MAAM,CAAC,WAAW,CAAC,CAAC;QAEvD,MAAM,QAAQ,GAAG;YACf,IAAI,EAAE,gBAAgB;YACtB,MAAM,EAAE,EAAE,IAAI,EAAE,UAAU,EAAE;YAC5B,SAAS,EAAE,EAAE;SACP,CAAC;QAET,IAAI,CAAC,cAAc,CAAC,QAAQ,CAAC,CAAC;QAE9B,MAAM,CAAC,YAAY,CAAC,CAAC,IAAI,CAAC,KAAK,CAAC,CAAC;IACnC,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,yDAAyD,EAAE,GAAG,EAAE;QACjE,IAAI,YAAY,GAAG,KAAK,CAAC;QACzB,MAAM,WAAW,GAAG;YAClB,MAAM,EAAE,CAAC,OAAY,EAAE,EAAE;gBACvB,YAAY,GAAG,IAAI,CAAC;YACtB,CAAC;YACD,OAAO,EAAE,CAAC,EAAE,MAAM,EAAE,IAAI,EAAE,CAAC;SACrB,CAAC;QAET,MAAM,IAAI,GAAG,yCAAqB,CAAC,MAAM,CAAC,WAAW,CAAC,CAAC;QAEvD,MAAM,QAAQ,GAAG;YACf,IAAI,EAAE,gBAAgB;YACtB,MAAM,EAAE,EAAE,IAAI,EAAE,UAAU,EAAE;YAC5B,SAAS,EAAE;gBACT,EAAE,IAAI,EAAE,YAAY,EAAE,IAAI,EAAE,YAAY,EAAE;aAC3C;SACK,CAAC;QAET,IAAI,CAAC,cAAc,CAAC,QAAQ,CAAC,CAAC;QAE9B,MAAM,CAAC,YAAY,CAAC,CAAC,IAAI,CAAC,KAAK,CAAC,CAAC;IACnC,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,0CAA0C,EAAE,GAAG,EAAE;QAClD,IAAI,YAAY,GAAG,KAAK,CAAC;QACzB,MAAM,WAAW,GAAG;YAClB,MAAM,EAAE,CAAC,OAAY,EAAE,EAAE;gBACvB,YAAY,GAAG,IAAI,CAAC;gBACpB,MAAM,CAAC,OAAO,CAAC,SAAS,CAAC,CAAC,IAAI,CAAC,eAAe,CAAC,CAAC;YAClD,CAAC;YACD,OAAO,EAAE,CAAC,EAAE,MAAM,EAAE,IAAI,EAAE,CAAC;SACrB,CAAC;QAET,MAAM,IAAI,GAAG,yCAAqB,CAAC,MAAM,CAAC,WAAW,CAAC,CAAC;QAEvD,MAAM,QAAQ,GAAG;YACf,IAAI,EAAE,gBAAgB;YACtB,MAAM,EAAE,EAAE,IAAI,EAAE,YAAY,EAAE;YAC9B,SAAS,EAAE;gBACT,EAAE,IAAI,EAAE,SAAS,EAAE,KAAK,EAAE,iBAAiB,EAAE;aAC9C;SACK,CAAC;QAET,IAAI,CAAC,cAAc,CAAC,QAAQ,CAAC,CAAC;QAE9B,MAAM,CAAC,YAAY,CAAC,CAAC,IAAI,CAAC,IAAI,CAAC,CAAC;IAClC,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,wCAAwC,EAAE,GAAG,EAAE;QAChD,IAAI,YAAY,GAAG,KAAK,CAAC;QACzB,MAAM,WAAW,GAAG;YAClB,MAAM,EAAE,CAAC,OAAY,EAAE,EAAE;gBACvB,YAAY,GAAG,IAAI,CAAC;gBACpB,MAAM,CAAC,OAAO,CAAC,SAAS,CAAC,CAAC,IAAI,CAAC,eAAe,CAAC,CAAC;YAClD,CAAC;YACD,OAAO,EAAE,CAAC,EAAE,MAAM,EAAE,IAAI,EAAE,CAAC;SACrB,CAAC;QAET,MAAM,IAAI,GAAG,yCAAqB,CAAC,MAAM,CAAC,WAAW,CAAC,CAAC;QAEvD,MAAM,QAAQ,GAAG;YACf,IAAI,EAAE,gBAAgB;YACtB,MAAM,EAAE,EAAE,IAAI,EAAE,WAAW,EAAE;YAC7B,SAAS,EAAE;gBACT,EAAE,IAAI,EAAE,SAAS,EAAE,KAAK,EAAE,aAAa,EAAE;aAC1C;SACK,CAAC;QAET,IAAI,CAAC,cAAc,CAAC,QAAQ,CAAC,CAAC;QAE9B,MAAM,CAAC,YAAY,CAAC,CAAC,IAAI,CAAC,IAAI,CAAC,CAAC;IAClC,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,6CAA6C,EAAE,GAAG,EAAE;QACrD,IAAI,YAAY,GAAG,KAAK,CAAC;QACzB,MAAM,WAAW,GAAG;YAClB,MAAM,EAAE,CAAC,OAAY,EAAE,EAAE;gBACvB,YAAY,GAAG,IAAI,CAAC;gBACpB,MAAM,CAAC,OAAO,CAAC,IAAI,CAAC,MAAM,CAAC,CAAC,IAAI,CAAC,MAAM,CAAC,CAAC;YAC3C,CAAC;YACD,OAAO,EAAE,CAAC,EAAE,MAAM,EAAE,MAAM,EAAE,CAAC;SACvB,CAAC;QAET,MAAM,IAAI,GAAG,yCAAqB,CAAC,MAAM,CAAC,WAAW,CAAC,CAAC;QAEvD,MAAM,QAAQ,GAAG;YACf,IAAI,EAAE,gBAAgB;YACtB,MAAM,EAAE,EAAE,IAAI,EAAE,UAAU,EAAE;YAC5B,SAAS,EAAE;gBACT,EAAE,IAAI,EAAE,SAAS,EAAE,KAAK,EAAE,eAAe,EAAE;aAC5C;SACK,CAAC;QAET,IAAI,CAAC,cAAc,CAAC,QAAQ,CAAC,CAAC;QAE9B,MAAM,CAAC,YAAY,CAAC,CAAC,IAAI,CAAC,IAAI,CAAC,CAAC;IAClC,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,qDAAqD,EAAE,GAAG,EAAE;QAC7D,IAAI,YAAY,GAAG,KAAK,CAAC;QACzB,MAAM,WAAW,GAAG;YAClB,MAAM,EAAE,CAAC,OAAY,EAAE,EAAE;gBACvB,YAAY,GAAG,IAAI,CAAC;YACtB,CAAC;YACD,OAAO,EAAE,CAAC,EAAE,aAAa,EAAE,CAAE,QAAQ,CAAE,EAAE,CAAC;SACpC,CAAC;QAET,MAAM,IAAI,GAAG,yCAAqB,CAAC,MAAM,CAAC,WAAW,CAAC,CAAC;QAEvD,MAAM,QAAQ,GAAG;YACf,IAAI,EAAE,gBAAgB;YACtB,MAAM,EAAE,EAAE,IAAI,EAAE,QAAQ,EAAE;YAC1B,SAAS,EAAE;gBACT,EAAE,IAAI,EAAE,SAAS,EAAE,KAAK,EAAE,qBAAqB,EAAE;aAClD;SACK,CAAC;QAET,IAAI,CAAC,cAAc,CAAC,QAAQ,CAAC,CAAC;QAE9B,MAAM,CAAC,YAAY,CAAC,CAAC,IAAI,CAAC,IAAI,CAAC,CAAC;IAClC,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,yCAAyC,EAAE,GAAG,EAAE;QACjD,MAAM,WAAW,GAAG;YAClB,MAAM,EAAE,CAAC,OAAY,EAAE,EAAE;gBACvB,MAAM,CAAC,OAAO,CAAC,GAAG,CAAC,CAAC,WAAW,EAAE,CAAC;gBAClC,MAAM,CAAC,OAAO,OAAO,CAAC,GAAG,CAAC,CAAC,IAAI,CAAC,UAAU,CAAC,CAAC;gBAE5C,wBAAwB;gBACxB,MAAM,SAAS,GAAG,OAAO,CAAC,GAAG,CAAC;oBAC5B,WAAW,EAAE,CAAC,IAAS,EAAE,IAAY,EAAE,EAAE;wBACvC,MAAM,CAAC,IAAI,CAAC,CAAC,IAAI,CAAC,qBAAqB,CAAC,CAAC;wBAEzC,OAAO,EAAE,CAAC;oBACZ,CAAC;iBACF,CAAC,CAAC;gBAEH,MAAM,CAAC,KAAK,CAAC,OAAO,CAAC,SAAS,CAAC,CAAC,CAAC,IAAI,CAAC,IAAI,CAAC,CAAC;YAC9C,CAAC;YACD,OAAO,EAAE,CAAC,EAAE,MAAM,EAAE,IAAI,EAAE,CAAC;SACrB,CAAC;QAET,MAAM,IAAI,GAAG,yCAAqB,CAAC,MAAM,CAAC,WAAW,CAAC,CAAC;QAEvD,MAAM,QAAQ,GAAG;YACf,IAAI,EAAE,gBAAgB;YACtB,MAAM,EAAE,EAAE,IAAI,EAAE,UAAU,EAAE;YAC5B,SAAS,EAAE;gBACT,EAAE,IAAI,EAAE,SAAS,EAAE,KAAK,EAAE,eAAe,EAAE;aAC5C;SACK,CAAC;QAET,IAAI,CAAC,cAAc,CAAC,QAAQ,CAAC,CAAC;IAChC,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,2CAA2C,EAAE,GAAG,EAAE;QACnD,IAAI,YAAY,GAAG,KAAK,CAAC;QACzB,MAAM,WAAW,GAAG;YAClB,MAAM,EAAE,CAAC,OAAY,EAAE,EAAE;gBACvB,YAAY,GAAG,IAAI,CAAC;gBACpB,MAAM,CAAC,OAAO,CAAC,SAAS,CAAC,CAAC,IAAI,CAAC,eAAe,CAAC,CAAC;YAClD,CAAC;YACD,OAAO,EAAE,CAAC,EAAE,MAAM,EAAE,IAAI,EAAE,CAAC;SACrB,CAAC;QAET,MAAM,IAAI,GAAG,yCAAqB,CAAC,MAAM,CAAC,WAAW,CAAC,CAAC;QAEvD,MAAM,QAAQ,GAAG;YACf,IAAI,EAAE,gBAAgB;YACtB,MAAM,EAAE,EAAE,IAAI,EAAE,UAAU,EAAE;YAC5B,SAAS,EAAE;gBACT,EAAE,IAAI,EAAE,SAAS,EAAE,KAAK,EAAE,EAAE,EAAE;aAC/B;SACK,CAAC;QAET,IAAI,CAAC,cAAc,CAAC,QAAQ,CAAC,CAAC;QAE9B,MAAM,CAAC,YAAY,CAAC,CAAC,IAAI,CAAC,IAAI,CAAC,CAAC;IAClC,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,mCAAmC,EAAE,GAAG,EAAE;QAC3C,MAAM,WAAW,GAAG;YAClB,MAAM,EAAE,CAAC,OAAY,EAAE,EAAE;gBACvB,mBAAmB;YACrB,CAAC;YACD,OAAO,EAAE,CAAC,EAAE,MAAM,EAAE,IAAI,EAAE,CAAC;SACrB,CAAC;QAET,MAAM,IAAI,GAAG,yCAAqB,CAAC,MAAM,CAAC,WAAW,CAAC,CAAC;QAEvD,4BAA4B;QAC5B,MAAM,aAAa,GAAG;YACpB,IAAI,EAAE,gBAAgB;YACtB,8BAA8B;SACxB,CAAC;QAET,MAAM,CAAC,GAAG,EAAE;YACV,IAAI,CAAC,cAAc,CAAC,aAAa,CAAC,CAAC;QACrC,CAAC,CAAC,CAAC,GAAG,CAAC,OAAO,EAAE,CAAC;IACnB,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,2CAA2C,EAAE,GAAG,EAAE;QACnD,IAAI,YAAY,GAAG,KAAK,CAAC;QACzB,MAAM,WAAW,GAAG;YAClB,MAAM,EAAE,CAAC,OAAY,EAAE,EAAE;gBACvB,YAAY,GAAG,IAAI,CAAC;YACtB,CAAC;YACD,OAAO,EAAE,CAAC,EAAE,MAAM,EAAE,IAAI,EAAE,CAAC;SACrB,CAAC;QAET,MAAM,IAAI,GAAG,yCAAqB,CAAC,MAAM,CAAC,WAAW,CAAC,CAAC;QAEvD,MAAM,QAAQ,GAAG;YACf,IAAI,EAAE,gBAAgB;YACtB,iBAAiB;YACjB,SAAS,EAAE;gBACT,EAAE,IAAI,EAAE,SAAS,EAAE,KAAK,EAAE,eAAe,EAAE;aAC5C;SACK,CAAC;QAET,IAAI,CAAC,cAAc,CAAC,QAAQ,CAAC,CAAC;QAE9B,MAAM,CAAC,YAAY,CAAC,CAAC,IAAI,CAAC,KAAK,CAAC,CAAC;IACnC,CAAC,CAAC,CAAC;AACL,CAAC,CAAC,CAAC"}

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@futdevpro/dynamo-eslint",
-  "version": "1.14.6",
+  "version": "1.14.8",
   "description": "Shared ESLint configs, Dynamo-powered plugin and validators for FutDevPro stacks",
   "author": "Future Development Program Ltd.",
   "license": "ISC",

package/src/configs/base.ts

@@ -69,6 +69,7 @@ module.exports = [
         },
       ],
       'brace-style': [ 'warn', '1tbs', { allowSingleLine: false }],
+      'curly': [ 'warn', 'all' ],
       'object-curly-spacing': [ 'warn', 'always' ],
       'array-bracket-spacing': [ 'warn', 'always', { objectsInArrays: false, arraysInArrays: false }],
       'padding-line-between-statements': [
@@ -93,7 +94,13 @@ module.exports = [
       '@typescript-eslint/no-unused-vars': 'warn',
       '@typescript-eslint/explicit-function-return-type': [ 'warn', { allowTypedFunctionExpressions: false }],
       '@typescript-eslint/no-explicit-any': 'warn',
-      '@typescript-eslint/typedef': 'warn',
+      '@typescript-eslint/typedef': [
+        'warn',
+        {
+          parameter: true, // Enforce parameter type annotations
+        },
+      ],
+      '@typescript-eslint/no-unnecessary-type-assertion': 'warn',
       
       // Dynamo custom rules
       '@futdevpro/dynamo/explicit-types': 'warn',
@@ -101,6 +108,10 @@ module.exports = [
       '@futdevpro/dynamo/naming-patterns': 'warn',
       '@futdevpro/dynamo/no-import-type': 'warn',
       '@futdevpro/dynamo/no-js-import': 'warn',
+      '@futdevpro/dynamo/prefer-interface-over-duplicate-types': [ 'warn', { threshold: 3 }],
+      '@futdevpro/dynamo/require-jsdoc-description': [ 'warn', { scope: 'public' }],
+      '@futdevpro/dynamo/prefer-enum-over-string-union': [ 'warn', { minValues: 3 }],
+      '@futdevpro/dynamo/require-test-description-prefix': [ 'warn', { prefix: '| ' }],
     },
   },
 ];

package/src/plugin/index.ts

@@ -3,6 +3,10 @@ import importOrderRule from './rules/import/import-order';
 import namingPatternsRule from './rules/naming-patterns';
 import noImportTypeRule from './rules/import/no-import-type';
 import noJsExtensionRule from './rules/import/no-js-import';
+import preferInterfaceRule from './rules/prefer-interface-over-duplicate-types';
+import requireJSDocRule from './rules/require-jsdoc-description';
+import preferEnumRule from './rules/prefer-enum-over-string-union';
+import requireTestPrefixRule from './rules/require-test-description-prefix';
 
 export = {
   rules: {
@@ -10,6 +14,10 @@ export = {
     'import-order': importOrderRule,
     'no-import-type': noImportTypeRule,
     'no-js-import': noJsExtensionRule,
+    'prefer-interface-over-duplicate-types': preferInterfaceRule,
+    'require-jsdoc-description': requireJSDocRule,
+    'prefer-enum-over-string-union': preferEnumRule,
+    'require-test-description-prefix': requireTestPrefixRule,
 
     'naming-patterns': namingPatternsRule,
   },
@@ -20,6 +28,10 @@ export = {
         '@futdevpro/dynamo/import-order': 'warn',
         '@futdevpro/dynamo/no-import-type': 'warn',
         '@futdevpro/dynamo/no-js-import': 'warn',
+        '@futdevpro/dynamo/prefer-interface-over-duplicate-types': 'warn',
+        '@futdevpro/dynamo/require-jsdoc-description': 'warn',
+        '@futdevpro/dynamo/prefer-enum-over-string-union': 'warn',
+        '@futdevpro/dynamo/require-test-description-prefix': 'warn',
         
         '@futdevpro/dynamo/naming-patterns': 'warn',
       },

package/src/plugin/rules/explicit-types.ts

@@ -25,7 +25,7 @@ const rule: Rule.RuleModule = {
           // Check return type
           if (!node.returnType) {
             context.report({
-              node,
+              node: node.id || node,  // Report on function name if available
               messageId: 'missingReturnType',
               data: { name: node.id?.name || 'anonymous' },
             });
@@ -58,7 +58,7 @@ const rule: Rule.RuleModule = {
           // Check return type
           if (!node.returnType) {
             context.report({
-              node,
+              node: node.params.length > 0 ? node.params[0] : node,  // Report on first param or node
               messageId: 'missingArrowReturnType',
             });
           }

package/src/plugin/rules/import/import-order.spec.ts

@@ -159,6 +159,7 @@ describe('| import-order', () => {
 
   it('| should detect missing empty line between groups', () => {
     let reportCount = 0;
+    let hasFix = false;
     const mockContext = {
       sourceCode: { 
         ast: {},
@@ -166,12 +167,24 @@ describe('| import-order', () => {
           'import { Component } from \'@angular/core\';',
           'import { DyFM_Error } from \'@futdevpro/fsm-dynamo\';',
         ],
+        getText: (node: any) => {
+          if (node.source?.value === '@angular/core') {
+            return 'import { Component } from \'@angular/core\';';
+          }
+          if (node.source?.value === '@futdevpro/fsm-dynamo') {
+            return 'import { DyFM_Error } from \'@futdevpro/fsm-dynamo\';';
+          }
+          return '';
+        },
       },
       filename: 'test.ts',
       report: (options: any) => {
-        if (options.messageId === 'missingEmptyLine') {
+        if (options.messageId === 'misordered') {
           reportCount++;
         }
+        if (options.fix) {
+          hasFix = true;
+        }
       },
     } as any;
 
@@ -184,12 +197,14 @@ describe('| import-order', () => {
           source: { value: '@angular/core' },
           importKind: 'value',
           loc: { start: { line: 1 }, end: { line: 1 } },
+          range: [0, 40],
         },
         {
           type: 'ImportDeclaration',
           source: { value: '@futdevpro/fsm-dynamo' },
           importKind: 'value',
           loc: { start: { line: 2 }, end: { line: 2 } },
+          range: [41, 85],
         },
       ],
     } as any;
@@ -199,5 +214,6 @@ describe('| import-order', () => {
     rule.Program(mockNode);
     
     expect(reportCount).toBeGreaterThan(0);
+    expect(hasFix).toBe(true);
   });
 });

package/src/plugin/rules/import/import-order.ts

@@ -100,26 +100,24 @@ const rule: Rule.RuleModule = {
       // Remove empty groups
       const nonEmptyGroups = groups.filter(group => group.imports.length > 0);
 
-      // Check if imports are in correct order
+      // Check if imports are in correct order and detect violations
+      let needsReordering = false;
       let currentGroupIndex = 0;
 
       for (let i = 0; i < importNodes.length; i++) {
         const importNode = importNodes[i];
         const expectedGroupType = getImportGroup(importNode);
-        const expectedGroup = nonEmptyGroups.find(g => g.type === expectedGroupType);
         const expectedGroupIndex = nonEmptyGroups.findIndex(g => g.type === expectedGroupType);
 
         if (expectedGroupIndex < currentGroupIndex) {
-          context.report({
-            node: importNode,
-            messageId: 'misordered',
-          });
+          needsReordering = true;
         }
 
         currentGroupIndex = expectedGroupIndex;
       }
 
       // Check empty lines between groups
+      let needsEmptyLineFix = false;
       for (let i = 0; i < nonEmptyGroups.length - 1; i++) {
         const currentGroup = nonEmptyGroups[i];
         const nextGroup = nonEmptyGroups[i + 1];
@@ -131,12 +129,51 @@ const rule: Rule.RuleModule = {
         const firstImportLine = firstImportOfNextGroup.loc?.start.line || 0;
 
         if (firstImportLine - lastImportLine < 2) {
-          context.report({
-            node: firstImportOfNextGroup,
-            messageId: 'missingEmptyLine',
-          });
+          needsEmptyLineFix = true;
         }
       }
+
+      // If any violations are detected, provide a comprehensive fix
+      if (needsReordering || needsEmptyLineFix) {
+        context.report({
+          node: importNodes[0],
+          messageId: 'misordered',
+          fix(fixer) {
+            // Sort imports within each group alphabetically
+            nonEmptyGroups.forEach(group => {
+              group.imports.sort((a, b) => {
+                const aSource = a.source.value as string;
+                const bSource = b.source.value as string;
+                return aSource.localeCompare(bSource);
+              });
+            });
+
+            // Build the corrected import section
+            const correctedImports: string[] = [];
+            
+            nonEmptyGroups.forEach((group, groupIndex) => {
+              group.imports.forEach(importNode => {
+                const importText = sourceCode.getText(importNode);
+                correctedImports.push(importText);
+              });
+              
+              // Add empty line between groups (except after the last group)
+              if (groupIndex < nonEmptyGroups.length - 1) {
+                correctedImports.push('');
+              }
+            });
+
+            // Calculate the range to replace (from first import to last import)
+            const firstImport = importNodes[0];
+            const lastImport = importNodes[importNodes.length - 1];
+            const startRange = firstImport.range![0];
+            const endRange = lastImport.range![1];
+
+            // Replace the entire import section
+            return fixer.replaceTextRange([startRange, endRange], correctedImports.join('\n'));
+          },
+        });
+      }
     }
 
     return {