Closes the four production gaps that emerged from sessie 3b's admin UI.
What we ship here is final: no further rework planned before production.
Backend
- IndexFailuresRequest validates state/search/failed_at_from/failed_at_to/
listener_class. orgIndex + platformIndex apply them via a single
applyIndexFilters() helper. Search runs case-insensitive `LIKE` on
exception_message; SQL wildcards in user input are escaped.
- New /kpis aggregate endpoint per scope (orgKpis, platformKpis) returns
open / resolved_30d / dismissed_30d / total_submissions in O(1) COUNTs.
Replaces sessie 3b's client-side bucketing of an oversized list.
- Resource expansion: organisation_name, form_schema_label,
resolved_by_user_name, dismissed_by_user_name, exception_trace,
retry_history[]. Eager-loading via indexEagerLoads()/detailEagerLoads()
prevents N+1 (verified by query-count assertion in test).
- New 2026_04_28_181000 migration adds exception_trace (longtext nullable)
to form_submission_action_failures. ApplyBindingsOnFormSubmit listener
now captures $e->getTraceAsString() at failure time.
- New FormSubmissionActionFailureRetryAttemptResource exposes per-attempt
data (timestamp, actor name, outcome, exception details) inside
retry_history[]. Index payloads omit the field via whenLoaded() to keep
list responses lean.
Frontend (apps/app)
- Types updated to mirror the expanded resource shape and the new KPI
endpoint contract. FormFailuresKpis is now { open, resolved_30d,
dismissed_30d, total_submissions } (server-aggregate).
- useFormFailures composable forwards all 5 server filters via
buildIndexParams() (strips empty/whitespace). useFormFailuresKpis hits
the dedicated /kpis endpoint per scope.
- FormFailuresTable replaces client-side bucketing with server-side
filtering, adds listener_class + date-range filter inputs, and renames
the 4th KPI tile to "Submissions" (was "Totaal").
- FormFailureDetail renders organisation_name + form_schema_label in the
header, surfaces an expandable stack-trace card, names the resolved/
dismissed actor in the timeline, and replaces the "v1 placeholder"
retry-history card with a full per-attempt timeline.
ESLint config gap (apps/app)
- New .eslintrc.cjs adapted from the Vuexy reference, minus Vuexy-internal
rules. `pnpm lint` now runs successfully (was previously broken — the
package.json script referenced a missing config). The 80 baseline
violations across the codebase are pre-existing and out of scope for
this session.
Tests + gates
- 24 new backend tests across filter, kpis, and resource-shape suites.
Backend: 1462 → 1486 passing, 0 → 0 failing. Larastan clean. Rector
dry-run unchanged at 354 (pre-Task-1 baseline from f18b55b).
- 3 new vitest tests in apps/app (filter wiring, KPI endpoint, KPI tile
values from /kpis). Vitest: 38 → 41 passing. tsc clean. Portal
unchanged (113 vitest, tsc clean).
- 5 backfill rollback tests bumped --step counts +1 for the new migration.
- Ws6FoundationMigrationTest down/up chain now includes exception_trace
before the parent table is restored.
Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
209 lines
6.1 KiB
JavaScript
209 lines
6.1 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',
|
|
],
|
|
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],
|
|
'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: {},
|
|
},
|
|
},
|
|
}
|