crewli

bert.hausmans/crewli

Fork 0

Commit Graph

Author	SHA1	Message	Date
bert.hausmans	dcb8d53b61	style(app): apply eslint --fix to Tier 3 config files WS-3 session 1b-ii Task 2 (audit Bucket X — 134 items). Hand-reviewed quote-style/semi/arrow-parens autofixes on the three config files explicitly excluded from session 1b-i: - vite.config.ts (91 items, @typescript-eslint/quotes / semi / arrow-parens / curly). Plugin order verified unchanged by diff inspection (VueRouter → vue → vueJsx → vuetify → MetaLayouts → Components → AutoImport → svgLoader). One curly-add at the apexcharts resolver: \`if (componentName === 'VueApexCharts') { return {...} }\` — behaviorally identical to the prior braces-omitted form. Build smoke: pnpm build succeeded in 12.13s, zero warnings. - themeConfig.ts (42 items, quotes / semi). Theme token values byte-identical, only surrounding quotes flipped from double to single. App title 'Crewli', logo span style, language labels and i18n codes all preserved. - vitest.config.ts (1 item, lines-around-comment). Trivial: blank line added before the dts:false comment block. No semantic changes. apps/app vitest 49 passed, vue-tsc clean, pnpm build succeeded. Lint baseline: 231 → 97 (Bucket X resolved). Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>	2026-04-29 14:07:13 +02:00
bert.hausmans	d95e68423d	test(apps/app): set up Vitest harness — closes TECH-APP-VITEST (WS-6) Mirrors apps/portal's Vitest setup so the SPA can take frontend unit + component tests. Required prerequisite for WS-6 sessie 3b's admin UI work — apps/portal had 113+ tests, apps/app had zero, and launching WS-6's organizer UI uncovered while the portal SPA is well-tested would be asymmetric quality. Setup: - vitest, happy-dom, @vue/test-utils, @testing-library/vue installed - vitest.config.ts mirrors portal config: trimmed auto-imports (no pinia/vue-router/vue-i18n/@vueuse/math) so tests run fast in happy-dom without loading the full Vuexy bundle - AutoImport's dts:false prevents the trimmed test-only set from clobbering the dev-server's full auto-imports.d.ts (apps/app's auto-import surface is bigger than the portal's) - tests/setup.ts mocks vue-router by default; tests that exercise the real router can override per-suite - Sample sanity test confirms the harness works end-to-end Adds `pnpm test` and `pnpm test:watch` scripts to package.json. Refs: BACKLOG TECH-APP-VITEST, WS-6 sessie 3b prerequisite Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>	2026-04-29 00:14:18 +02:00

Author

SHA1

Message

Date

bert.hausmans

dcb8d53b61

style(app): apply eslint --fix to Tier 3 config files

WS-3 session 1b-ii Task 2 (audit Bucket X — 134 items).

Hand-reviewed quote-style/semi/arrow-parens autofixes on the three
config files explicitly excluded from session 1b-i:

- vite.config.ts (91 items, @typescript-eslint/quotes / semi /
  arrow-parens / curly). Plugin order verified unchanged by diff
  inspection (VueRouter → vue → vueJsx → vuetify → MetaLayouts →
  Components → AutoImport → svgLoader). One curly-add at the
  apexcharts resolver: \`if (componentName === 'VueApexCharts') { return {...} }\`
  — behaviorally identical to the prior braces-omitted form.
  Build smoke: pnpm build succeeded in 12.13s, zero warnings.

- themeConfig.ts (42 items, quotes / semi). Theme token values
  byte-identical, only surrounding quotes flipped from double to
  single. App title 'Crewli', logo span style, language labels and
  i18n codes all preserved.

- vitest.config.ts (1 item, lines-around-comment). Trivial: blank
  line added before the dts:false comment block.

No semantic changes. apps/app vitest 49 passed, vue-tsc clean,
pnpm build succeeded.

Lint baseline: 231 → 97 (Bucket X resolved).

Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>

2026-04-29 14:07:13 +02:00

bert.hausmans

d95e68423d

test(apps/app): set up Vitest harness — closes TECH-APP-VITEST (WS-6)

Mirrors apps/portal's Vitest setup so the SPA can take frontend
unit + component tests. Required prerequisite for WS-6 sessie 3b's
admin UI work — apps/portal had 113+ tests, apps/app had zero, and
launching WS-6's organizer UI uncovered while the portal SPA is
well-tested would be asymmetric quality.

Setup:
- vitest, happy-dom, @vue/test-utils, @testing-library/vue installed
- vitest.config.ts mirrors portal config: trimmed auto-imports
  (no pinia/vue-router/vue-i18n/@vueuse/math) so tests run fast
  in happy-dom without loading the full Vuexy bundle
- AutoImport's dts:false prevents the trimmed test-only set from
  clobbering the dev-server's full auto-imports.d.ts (apps/app's
  auto-import surface is bigger than the portal's)
- tests/setup.ts mocks vue-router by default; tests that exercise
  the real router can override per-suite
- Sample sanity test confirms the harness works end-to-end

Adds `pnpm test` and `pnpm test:watch` scripts to package.json.

Refs: BACKLOG TECH-APP-VITEST, WS-6 sessie 3b prerequisite

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>

2026-04-29 00:14:18 +02:00

2 Commits