Files
crewli/apps/app/.eslintrc.cjs
bert.hausmans 2fc2a569e7 fix(app): allow comments at SFC <script> block start
WS-3 session 1b-iii Task 2.

In Vue SFCs, the lines-around-comment rule conflicted with
vue/block-tag-newline at the <script setup>/comment boundary:
- lines-around-comment wants a blank line BEFORE the comment.
- vue/block-tag-newline wants exactly 1 line break after <script>.

Both can't be satisfied simultaneously when a script-block opens with
a leading comment. The session 1b-ii experiment (adding then reverting
blank lines) confirmed empirically these are mutually exclusive.

Resolution: per-*.vue override on lines-around-comment with
beforeBlockComment: false and beforeLineComment: false. Vue SFC
script blocks may now open with a leading comment without requiring
a preceding blank line. The base rule's allowBlockStart: true does
not help here because the <script> tag is not a JS block-start as
far as the AST sees it. All other rule options preserved
(allowBlockStart, allowClassStart, allowObjectStart, allowArrayStart,
ignorePattern: !SECTION).

Resolves the 3 items in PortalLayout.vue, PublicLayout.vue,
AppKpiCard.vue. Base rule remains in force for *.ts files.

Lint baseline: 6 → 3.

Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>
2026-04-29 18:46:21 +02:00

234 lines
7.0 KiB
JavaScript

// Sessie 3c (WS-6) — closes the apps/app ESLint config gap.
// Adapted from the Vuexy reference (resources/vuexy-admin-v10.11.1/.../full-version/.eslintrc.cjs)
// minus the Vuexy-internal lint rules (valid-appcardcode-*, internal regex
// rules) that don't apply outside the demo project. Plugin set matches
// what's installed in apps/app's package.json.
module.exports = {
root: true,
env: {
browser: true,
node: true,
es2022: true,
},
extends: [
'@antfu/eslint-config-vue',
'plugin:vue/vue3-recommended',
'plugin:import/recommended',
'plugin:import/typescript',
'plugin:promise/recommended',
'plugin:sonarjs/recommended',
'plugin:@typescript-eslint/recommended',
'plugin:case-police/recommended',
'plugin:regexp/recommended',
],
parser: 'vue-eslint-parser',
parserOptions: {
ecmaVersion: 13,
parser: '@typescript-eslint/parser',
sourceType: 'module',
},
plugins: [
'vue',
'@typescript-eslint',
'regex',
'regexp',
],
ignorePatterns: [
'src/plugins/iconify/*.js',
'node_modules',
'dist',
'*.d.ts',
'vendor',
'*.json',
'src/@core/**',
'src/@layouts/**',
],
rules: {
'no-console': process.env.NODE_ENV === 'production' ? 'warn' : 'off',
'no-debugger': process.env.NODE_ENV === 'production' ? 'warn' : 'off',
'comma-spacing': ['error', { before: false, after: true }],
'key-spacing': ['error', { afterColon: true }],
'n/prefer-global/process': ['off'],
'sonarjs/cognitive-complexity': ['off'],
'vue/first-attribute-linebreak': ['error', {
singleline: 'beside',
multiline: 'below',
}],
'antfu/top-level-function': 'off',
// Project rule (CLAUDE.md frontend rules): no `any`. Override the
// Vuexy reference (which sets this off) — Crewli's stricter posture.
'@typescript-eslint/no-explicit-any': 'error',
'indent': ['error', 2, { SwitchCase: 1 }],
'comma-dangle': ['error', 'always-multiline'],
'object-curly-spacing': ['error', 'always'],
'camelcase': 'error',
'max-len': 'off',
'semi': ['error', 'never'],
'arrow-parens': ['error', 'as-needed'],
'newline-before-return': 'error',
'lines-around-comment': [
'error',
{
beforeBlockComment: true,
beforeLineComment: true,
allowBlockStart: true,
allowClassStart: true,
allowObjectStart: true,
allowArrayStart: true,
ignorePattern: '!SECTION',
},
],
'@typescript-eslint/no-unused-vars': ['error', {
varsIgnorePattern: '^_+$',
argsIgnorePattern: '^_+$',
}],
'array-element-newline': ['error', 'consistent'],
'array-bracket-newline': ['error', 'consistent'],
'vue/multi-word-component-names': 'off',
'padding-line-between-statements': [
'error',
{ blankLine: 'always', prev: 'expression', next: 'const' },
{ blankLine: 'always', prev: 'const', next: 'expression' },
{ blankLine: 'always', prev: 'multiline-const', next: '*' },
{ blankLine: 'always', prev: '*', next: 'multiline-const' },
],
'import/prefer-default-export': 'off',
'import/newline-after-import': ['error', { count: 1 }],
'no-restricted-imports': ['error', 'vuetify/components', {
name: 'vue3-apexcharts',
message: 'apexcharts are auto imported',
}],
'import/extensions': [
'error',
'ignorePackages',
{
js: 'never',
jsx: 'never',
ts: 'never',
tsx: 'never',
},
],
'import/no-unresolved': [2, {
ignore: [
'~pages$',
'virtual:meta-layouts',
'#auth$',
'#components$',
'.*\\?raw',
],
}],
'no-shadow': 'off',
'@typescript-eslint/no-shadow': ['error'],
'@typescript-eslint/consistent-type-imports': 'error',
// CLAUDE.md frontend convention — backend enums are mirrored as
// `as const` objects WITH a same-named `type` alias. The two live
// in different namespaces (value vs. type) and are intentional;
// both base `no-redeclare` and the typed variant flag them anyway.
'no-redeclare': 'off',
'@typescript-eslint/no-redeclare': 'off',
'promise/always-return': 'off',
'promise/catch-or-return': 'off',
'vue/block-tag-newline': 'error',
'vue/component-api-style': 'error',
'vue/component-name-in-template-casing': ['error', 'PascalCase', {
registeredComponentsOnly: false,
ignores: ['/^swiper-/'],
}],
'vue/custom-event-name-casing': ['error', 'camelCase', {
ignores: [
'/^(click):[a-z]+((\\d)|([A-Z0-9][a-z0-9]+))*([A-Z])?/',
],
}],
'vue/define-macros-order': 'error',
'vue/html-comment-content-newline': 'error',
'vue/html-comment-content-spacing': 'error',
'vue/html-comment-indent': 'error',
'vue/match-component-file-name': 'error',
'vue/no-child-content': 'error',
'vue/require-default-prop': 'off',
'vue/no-duplicate-attr-inheritance': 'error',
'vue/no-empty-component-block': 'error',
'vue/no-multiple-objects-in-class': 'error',
'vue/no-reserved-component-names': 'error',
'vue/no-template-target-blank': 'error',
'vue/no-useless-mustaches': 'error',
'vue/no-useless-v-bind': 'error',
'vue/padding-line-between-blocks': 'error',
'vue/prefer-separate-static-class': 'error',
'vue/prefer-true-attribute-shorthand': 'error',
'vue/v-on-function-call': 'error',
'vue/no-restricted-class': ['error', '/^(p|m)(l|r)-/'],
'vue/valid-v-slot': ['error', { allowModifiers: true }],
'vue/no-irregular-whitespace': 'error',
'vue/template-curly-spacing': 'error',
'sonarjs/no-duplicate-string': 'off',
'sonarjs/no-nested-template-literals': 'off',
'regex/invalid': [
'error',
[
{
regex: '@/assets/images',
replacement: '@images',
message: 'Use \'@images\' path alias for image imports',
},
{
regex: '@/assets/styles',
replacement: '@styles',
message: 'Use \'@styles\' path alias for importing styles from \'src/assets/styles\'',
},
],
'\\.eslintrc\\.cjs',
],
},
settings: {
'import/resolver': {
node: true,
typescript: {},
},
},
overrides: [
// Vue SFCs: the base lines-around-comment rule conflicts with
// vue/block-tag-newline at the <script setup>/comment boundary
// (the <script> tag isn't a JS block-start so allowBlockStart
// doesn't kick in, while vue/block-tag-newline forbids the blank
// line that lines-around-comment wants). Disable both
// beforeBlockComment and beforeLineComment for *.vue so a leading
// comment in <script setup> is allowed without a preceding blank.
{
files: ['*.vue'],
rules: {
'lines-around-comment': ['error', {
beforeBlockComment: false,
beforeLineComment: false,
allowBlockStart: true,
allowClassStart: true,
allowObjectStart: true,
allowArrayStart: true,
ignorePattern: '!SECTION',
}],
},
},
],
}