@webpieces/dev-config 0.0.0-dev → 0.2.17

package/eslint-plugin/__tests__/catch-error-pattern.test.ts

@@ -2,7 +2,6 @@
  * Tests for catch-error-pattern ESLint rule
  */
 
-/* eslint-disable @webpieces/catch-error-pattern */
 import { RuleTester } from 'eslint';
 import rule from '../rules/catch-error-pattern';
 

package/eslint-plugin/index.d.ts

@@ -7,8 +7,6 @@
 declare const _default: {
     rules: {
         'catch-error-pattern': import("eslint").Rule.RuleModule;
-        'max-method-lines': import("eslint").Rule.RuleModule;
-        'max-file-lines': import("eslint").Rule.RuleModule;
     };
 };
 export = _default;

package/eslint-plugin/index.js

@@ -7,13 +7,9 @@
  */
 const tslib_1 = require("tslib");
 const catch_error_pattern_1 = tslib_1.__importDefault(require("./rules/catch-error-pattern"));
-const max_method_lines_1 = tslib_1.__importDefault(require("./rules/max-method-lines"));
-const max_file_lines_1 = tslib_1.__importDefault(require("./rules/max-file-lines"));
 module.exports = {
     rules: {
         'catch-error-pattern': catch_error_pattern_1.default,
-        'max-method-lines': max_method_lines_1.default,
-        'max-file-lines': max_file_lines_1.default,
     },
 };
 //# sourceMappingURL=index.js.map

package/eslint-plugin/index.js.map

@@ -1 +1 @@
-{"version":3,"file":"index.js","sourceRoot":"","sources":["../../../../../packages/tooling/dev-config/eslint-plugin/index.ts"],"names":[],"mappings":";AAAA;;;;;GAKG;;AAEH,8FAA4D;AAC5D,wFAAsD;AACtD,oFAAkD;AAElD,iBAAS;IACL,KAAK,EAAE;QACH,qBAAqB,EAAE,6BAAiB;QACxC,kBAAkB,EAAE,0BAAc;QAClC,gBAAgB,EAAE,wBAAY;KACjC;CACJ,CAAC","sourcesContent":["/**\n * ESLint plugin for WebPieces\n * Provides rules for enforcing WebPieces code patterns\n *\n * This plugin is automatically included in @webpieces/dev-config\n */\n\nimport catchErrorPattern from './rules/catch-error-pattern';\nimport maxMethodLines from './rules/max-method-lines';\nimport maxFileLines from './rules/max-file-lines';\n\nexport = {\n    rules: {\n        'catch-error-pattern': catchErrorPattern,\n        'max-method-lines': maxMethodLines,\n        'max-file-lines': maxFileLines,\n    },\n};\n"]}
+{"version":3,"file":"index.js","sourceRoot":"","sources":["../../../../../packages/tooling/dev-config/eslint-plugin/index.ts"],"names":[],"mappings":";AAAA;;;;;GAKG;;AAEH,8FAA4D;AAE5D,iBAAS;IACL,KAAK,EAAE;QACH,qBAAqB,EAAE,6BAAiB;KAC3C;CACJ,CAAC","sourcesContent":["/**\n * ESLint plugin for WebPieces\n * Provides rules for enforcing WebPieces code patterns\n *\n * This plugin is automatically included in @webpieces/dev-config\n */\n\nimport catchErrorPattern from './rules/catch-error-pattern';\n\nexport = {\n    rules: {\n        'catch-error-pattern': catchErrorPattern,\n    },\n};\n"]}

package/eslint-plugin/index.ts

@@ -6,13 +6,9 @@
  */
 
 import catchErrorPattern from './rules/catch-error-pattern';
-import maxMethodLines from './rules/max-method-lines';
-import maxFileLines from './rules/max-file-lines';
 
 export = {
     rules: {
         'catch-error-pattern': catchErrorPattern,
-        'max-method-lines': maxMethodLines,
-        'max-file-lines': maxFileLines,
     },
 };

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@webpieces/dev-config",
-  "version": "0.0.0-dev",
+  "version": "0.2.17",
   "description": "Development configuration, scripts, and patterns for WebPieces projects",
   "type": "commonjs",
   "bin": {
@@ -27,10 +27,6 @@
   "peerDependencies": {
     "eslint": ">=8.0.0"
   },
-  "devDependencies": {
-    "@types/eslint": "^9.6.1",
-    "eslint": "^9.39.1"
-  },
   "keywords": [
     "webpieces",
     "config",

package/eslint-plugin/__tests__/max-file-lines.test.ts

@@ -1,195 +0,0 @@
-/**
- * Tests for max-file-lines ESLint rule
- */
-
-import { RuleTester } from 'eslint';
-import rule from '../rules/max-file-lines';
-import * as fs from 'fs';
-import * as path from 'path';
-
-// Use require to load parser at runtime (avoids TypeScript import issues)
-const tsParser = require('@typescript-eslint/parser');
-
-const ruleTester = new RuleTester({
-    languageOptions: {
-        parser: tsParser,
-        parserOptions: {
-            ecmaVersion: 2020,
-            sourceType: 'module',
-        },
-    },
-});
-
-ruleTester.run('max-file-lines', rule, {
-    valid: [
-        // Short file (well under limit)
-        {
-            code: `function shortFunc() {
-    return 42;
-}`,
-        },
-        // File with exactly 700 lines (default limit)
-        {
-            code: Array(700)
-                .fill(0)
-                .map((_, i) => `const line${i} = ${i};`)
-                .join('\n'),
-        },
-        // File with 699 lines (just under default limit)
-        {
-            code: Array(699)
-                .fill(0)
-                .map((_, i) => `const line${i} = ${i};`)
-                .join('\n'),
-        },
-        // Custom limit: 10 lines
-        {
-            code: `function shortFunc() {
-    const a = 1;
-    const b = 2;
-    const c = 3;
-    const d = 4;
-    const e = 5;
-    return a + b + c + d + e;
-}`,
-            options: [{ max: 10 }],
-        },
-        // Empty file
-        {
-            code: '',
-        },
-        // File with comments and blank lines (all count)
-        {
-            code: `// Comment line 1
-// Comment line 2
-
-function func() {
-    return 42;
-}
-
-// Another comment`,
-            options: [{ max: 10 }],
-        },
-    ],
-
-    invalid: [
-        // File with 701 lines (exceeds default limit)
-        {
-            code: Array(701)
-                .fill(0)
-                .map((_, i) => `const line${i} = ${i};`)
-                .join('\n'),
-            errors: [
-                {
-                    messageId: 'tooLong',
-                    data: { actual: '701', max: '700' },
-                },
-            ],
-        },
-        // File with 1000 lines (way over limit)
-        {
-            code: Array(1000)
-                .fill(0)
-                .map((_, i) => `const line${i} = ${i};`)
-                .join('\n'),
-            errors: [
-                {
-                    messageId: 'tooLong',
-                    data: { actual: '1000', max: '700' },
-                },
-            ],
-        },
-        // Custom limit: exceed 5 lines
-        {
-            code: `function func() {
-    const a = 1;
-    const b = 2;
-    const c = 3;
-    const d = 4;
-    return a + b + c + d;
-}`,
-            options: [{ max: 5 }],
-            errors: [
-                {
-                    messageId: 'tooLong',
-                    data: { actual: '7', max: '5' },
-                },
-            ],
-        },
-        // Custom limit: exceed 100 lines
-        {
-            code: Array(101)
-                .fill(0)
-                .map((_, i) => `const line${i} = ${i};`)
-                .join('\n'),
-            options: [{ max: 100 }],
-            errors: [
-                {
-                    messageId: 'tooLong',
-                    data: { actual: '101', max: '100' },
-                },
-            ],
-        },
-        // File with blank lines and comments (all lines count)
-        {
-            code: `// Line 1
-// Line 2
-// Line 3
-
-function func() {
-    return 42;
-}
-
-// Line 9
-// Line 10
-// Line 11`,
-            options: [{ max: 10 }],
-            errors: [
-                {
-                    messageId: 'tooLong',
-                    data: { actual: '11', max: '10' },
-                },
-            ],
-        },
-    ],
-});
-
-console.log('✅ All max-file-lines rule tests passed!');
-
-// Test documentation file creation
-const docPath = path.join(process.cwd(), 'tmp', 'webpieces', 'webpieces.filesize.md');
-
-// Run a test that triggers violation
-try {
-    ruleTester.run('max-file-lines-doc-test', rule, {
-        valid: [],
-        invalid: [
-            {
-                code: Array(800)
-                    .fill(0)
-                    .map((_, i) => `const line${i} = ${i};`)
-                    .join('\n'),
-                errors: [{ messageId: 'tooLong' }],
-            },
-        ],
-    });
-} catch (err: any) {
-    //const error = toError(err);
-    // Test may fail, but file should be created
-}
-
-// Verify file was created
-if (!fs.existsSync(docPath)) {
-    throw new Error('Documentation file was not created at ' + docPath);
-}
-
-// Verify content has AI directive
-const content = fs.readFileSync(docPath, 'utf-8');
-if (!content.includes('READ THIS FILE to fix files that are too long')) {
-    throw new Error('Documentation file missing AI directive');
-}
-if (!content.includes('SINGLE COHESIVE UNIT')) {
-    throw new Error('Documentation file missing single cohesive unit principle');
-}
-
-console.log('✅ Documentation file creation test passed!');

package/eslint-plugin/__tests__/max-method-lines.test.ts

@@ -1,246 +0,0 @@
-/**
- * Tests for max-method-lines ESLint rule
- */
-
-import { RuleTester } from 'eslint';
-import rule from '../rules/max-method-lines';
-import * as fs from 'fs';
-import * as path from 'path';
-
-// Use require to load parser at runtime (avoids TypeScript import issues)
-const tsParser = require('@typescript-eslint/parser');
-
-const ruleTester = new RuleTester({
-    languageOptions: {
-        parser: tsParser,
-        parserOptions: {
-            ecmaVersion: 2020,
-            sourceType: 'module',
-        },
-    },
-});
-
-ruleTester.run('max-method-lines', rule, {
-    valid: [
-        // Short function (well under limit)
-        {
-            code: `function shortFunc() {
-    return 42;
-}`,
-        },
-        // Function with exactly 70 lines (default limit)
-        {
-            code: `function exactlySeventyLines() {
-${Array(68)
-    .fill(0)
-    .map((_, i) => `    const line${i} = ${i};`)
-    .join('\n')}
-}`,
-        },
-        // Function with 69 lines (just under default limit)
-        {
-            code: `function sixtyNineLines() {
-${Array(67)
-    .fill(0)
-    .map((_, i) => `    const line${i} = ${i};`)
-    .join('\n')}
-}`,
-        },
-        // Custom limit: 10 lines
-        {
-            code: `function shortFunc() {
-    const a = 1;
-    const b = 2;
-    const c = 3;
-    const d = 4;
-    const e = 5;
-    const f = 6;
-    const g = 7;
-    return a + b + c + d + e + f + g;
-}`,
-            options: [{ max: 10 }],
-        },
-        // Arrow function under limit
-        {
-            code: `const shortArrow = () => {
-    return 42;
-};`,
-        },
-        // Method definition under limit
-        {
-            code: `class MyClass {
-    shortMethod() {
-        return 42;
-    }
-}`,
-        },
-        // Function expression under limit
-        {
-            code: `const func = function() {
-    return 42;
-};`,
-        },
-    ],
-
-    invalid: [
-        // Function with 71 lines (exceeds default limit)
-        {
-            code: `function tooLong() {
-${Array(69)
-    .fill(0)
-    .map((_, i) => `    const line${i} = ${i};`)
-    .join('\n')}
-}`,
-            errors: [
-                {
-                    messageId: 'tooLong',
-                    data: { name: 'tooLong', actual: '71', max: '70' },
-                },
-            ],
-        },
-        // Function with 100 lines (way over limit)
-        {
-            code: `function wayTooLong() {
-${Array(98)
-    .fill(0)
-    .map((_, i) => `    const line${i} = ${i};`)
-    .join('\n')}
-}`,
-            errors: [
-                {
-                    messageId: 'tooLong',
-                    data: { name: 'wayTooLong', actual: '100', max: '70' },
-                },
-            ],
-        },
-        // Custom limit: exceed 5 lines
-        {
-            code: `function tooLongForCustom() {
-    const a = 1;
-    const b = 2;
-    const c = 3;
-    const d = 4;
-    return a + b + c + d;
-}`,
-            options: [{ max: 5 }],
-            errors: [
-                {
-                    messageId: 'tooLong',
-                    data: { name: 'tooLongForCustom', actual: '7', max: '5' },
-                },
-            ],
-        },
-        // Arrow function exceeding limit
-        {
-            code: `const tooLongArrow = () => {
-${Array(69)
-    .fill(0)
-    .map((_, i) => `    const line${i} = ${i};`)
-    .join('\n')}
-};`,
-            errors: [
-                {
-                    messageId: 'tooLong',
-                    data: { name: 'anonymous', actual: '71', max: '70' },
-                },
-            ],
-        },
-        // Method definition exceeding limit
-        {
-            code: `class MyClass {
-    tooLongMethod() {
-${Array(69)
-    .fill(0)
-    .map((_, i) => `        const line${i} = ${i};`)
-    .join('\n')}
-    }
-}`,
-            errors: [
-                {
-                    messageId: 'tooLong',
-                    data: { name: 'tooLongMethod', actual: '71', max: '70' },
-                },
-            ],
-        },
-        // Function expression exceeding limit
-        {
-            code: `const func = function tooLongFunc() {
-${Array(69)
-    .fill(0)
-    .map((_, i) => `    const line${i} = ${i};`)
-    .join('\n')}
-};`,
-            errors: [
-                {
-                    messageId: 'tooLong',
-                    data: { name: 'tooLongFunc', actual: '71', max: '70' },
-                },
-            ],
-        },
-        // Multiple functions, one exceeds limit
-        {
-            code: `function shortFunc() {
-    return 42;
-}
-
-function tooLong() {
-${Array(69)
-    .fill(0)
-    .map((_, i) => `    const line${i} = ${i};`)
-    .join('\n')}
-}
-
-function anotherShort() {
-    return 24;
-}`,
-            errors: [
-                {
-                    messageId: 'tooLong',
-                    data: { name: 'tooLong', actual: '71', max: '70' },
-                },
-            ],
-        },
-    ],
-});
-
-console.log('✅ All max-method-lines rule tests passed!');
-
-// Test documentation file creation
-const docPath = path.join(process.cwd(), 'tmp', 'webpieces', 'webpieces.methods.md');
-
-// Run a test that triggers violation (will create doc file)
-try {
-    ruleTester.run('max-method-lines-doc-test', rule, {
-        valid: [],
-        invalid: [
-            {
-                code: `function veryLongMethod() {
-${Array(100)
-    .fill(0)
-    .map((_, i) => `    const line${i} = ${i};`)
-    .join('\n')}
-}`,
-                errors: [{ messageId: 'tooLong' }],
-            },
-        ],
-    });
-} catch (err: any) {
-    //const error = toError(err);
-    // Test may fail due to too many errors, but file should be created
-}
-
-// Verify file was created
-if (!fs.existsSync(docPath)) {
-    throw new Error('Documentation file was not created at ' + docPath);
-}
-
-// Verify content has AI directive
-const content = fs.readFileSync(docPath, 'utf-8');
-if (!content.includes('READ THIS FILE to fix methods that are too long')) {
-    throw new Error('Documentation file missing AI directive');
-}
-if (!content.includes('TABLE OF CONTENTS')) {
-    throw new Error('Documentation file missing table of contents principle');
-}
-
-console.log('✅ Documentation file creation test passed!');

package/eslint-plugin/rules/max-file-lines.d.ts

@@ -1,12 +0,0 @@
-/**
- * ESLint rule to enforce maximum file length
- *
- * Enforces a configurable maximum line count for files.
- * Default: 700 lines
- *
- * Configuration:
- * '@webpieces/max-file-lines': ['error', { max: 700 }]
- */
-import type { Rule } from 'eslint';
-declare const rule: Rule.RuleModule;
-export = rule;