cpp-actions/eslint.config.mjs at develop · alandefreitas/cpp-actions · GitHub

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
90
91
92
93
94
95
96
97
98
99
100
101
102
103
104
105
106
107
108
109
110
111
112
113
114
115
116
117
118
119
120
121
122
123
import tseslint from 'typescript-eslint';

export default tseslint.config(
    // Global ignores
    {
        ignores: [
            '**/dist/**',
            '**/lib/**',
            '**/node_modules/**',
            'utils/esbuild/**',
            'docs/**',
            'coverage/**',
            '*.js',
        ],
    },

    // Base config for all TypeScript files
    ...tseslint.configs.recommended,

    // TypeScript-specific rules
    {
        files: ['**/*.ts'],
        rules: {
            // Moderate strictness: catch real issues, avoid busywork
            '@typescript-eslint/no-unused-vars': ['error', {
                argsIgnorePattern: '^_',
                varsIgnorePattern: '^_',
            }],
            '@typescript-eslint/no-explicit-any': 'warn',
            '@typescript-eslint/no-require-imports': 'error',
            '@typescript-eslint/consistent-type-imports': ['error', {
                prefer: 'type-imports',
                fixStyle: 'inline-type-imports',
            }],

            // Allow short-circuit expressions (condition && doSomething())
            '@typescript-eslint/no-unused-expressions': ['error', {
                allowShortCircuit: true,
                allowTernary: true,
            }],

            // Code quality
            'no-constant-condition': 'error',
            'no-duplicate-case': 'error',
            'eqeqeq': ['error', 'always', { null: 'ignore' }],
            'no-var': 'error',
            'prefer-const': 'error',

            // Naming conventions
            '@typescript-eslint/naming-convention': ['error',
                {
                    selector: 'default',
                    format: ['camelCase', 'PascalCase'],
                    leadingUnderscore: 'allow',
                },
                {
                    selector: 'variable',
                    format: ['camelCase', 'UPPER_CASE', 'PascalCase'],
                    leadingUnderscore: 'allow',
                },
                {
                    selector: 'parameter',
                    format: ['camelCase', 'PascalCase', 'snake_case'],
                    leadingUnderscore: 'allow',
                },
                {
                    selector: 'typeLike',
                    format: ['PascalCase'],
                },
                {
                    selector: 'enumMember',
                    format: ['PascalCase', 'UPPER_CASE'],
                },
                {
                    selector: 'import',
                    format: null,
                },
                // Allow snake_case for object properties (external API boundaries)
                {
                    selector: 'objectLiteralProperty',
                    format: null,
                },
                {
                    selector: 'typeProperty',
                    format: null,
                },
                // Allow kebab-case and other formats for class/object members
                // accessed via bracket notation or matching external APIs
                {
                    selector: 'classProperty',
                    format: ['camelCase', 'PascalCase'],
                    leadingUnderscore: 'allow',
                },
                {
                    selector: 'method',
                    format: ['camelCase'],
                    leadingUnderscore: 'allow',
                },
            ],
        },
    },

    // Action entry points: no console.log (use core.info/debug instead)
    {
        files: [
            '*/src/index.ts',
            'common/*/src/index.ts',
        ],
        rules: {
            'no-console': 'error',
        },
    },

    // Test files: relaxed rules
    {
        files: ['**/*.test.ts'],
        rules: {
            '@typescript-eslint/no-explicit-any': 'off',
            '@typescript-eslint/naming-convention': 'off',
            '@typescript-eslint/no-require-imports': 'off',
        },
    },
);