chore: two tooling cleanups (admin .vscode entry + dead views/ dir) #2

bert.hausmans · 2026-05-04T21:44:08+02:00

bert.hausmans commented

2026-05-04 21:44:08 +02:00

Twee triviale cleanup-items uit de backlog die in dezelfde tooling-batch
mee kunnen. Beide ontstonden tijdens / na WS-3 sessie 1c.

Commits

b9f8f55 — chore(tooling): remove stale apps/admin entry from .vscode/settings.json
apps/admin/ is in april 2026 verwijderd (admin SPA gemerged in
apps/app/ onder /platform/*). De eslint.workingDirectories
verwijzing was achtergebleven; Cursor's ESLint extension skipte de
missende directory silently maar de dode config-regel was verwarrend
tijdens debugging. Closes TECH-VSCODE-STALE-ADMIN-ENTRY.
bdbd5b0 — chore(cleanup): delete dead Vuexy views/ directory
apps/app/src/views/ bevatte één Vuexy-template-bestand
(AuthProvider.vue) met nul importers in de hele repo. WS-3 sessie 1c
audit identificeerde het als dead vendored code. De §4.2 post-
consolidation target layout drop views/ volledig. Inclusief
verwijdering van de 'src/views/**' regel uit boundaries/ignore
in apps/app/.eslintrc.cjs. Closes TECH-DELETE-DEAD-VIEWS.
617a6d2 — docs(backlog): remove two closed tooling items
BACKLOG.md opschonen. Beide entries fysiek verwijderd; closure-
referenties leven in de twee chore-commits hierboven (en zijn
eeuwig findbaar via git log --grep=TECH-VSCODE-STALE /
--grep=TECH-DELETE-DEAD-VIEWS).

Acceptance

✅ pnpm lint 0 problems in apps/app/
✅ pnpm build succeeds (11.09s)
✅ pnpm test 49 passed (geen regressie)
✅ Geen wijziging in apps/portal/, api/, packages/, docs/
✅ Conventional Commits + Co-Authored-By: Claude op alle commits

Pre-existing warnings (niet door deze PR veroorzaakt)

Voor de duidelijkheid bij review:

TypeScript 5.9.3 buiten range van @typescript-eslint/typescript-estree — wordt opgelost door TECH-ESLINT-V9-MIGRATION.
boundaries/element-types v5→v6 deprecation warnings — wordt opgelost door TECH-BOUNDARIES-V6-SELECTOR-MIGRATION.
Vite/Rollup mixed static+dynamic import warnings voor useAuthStore / useImpersonationStore — wordt opgelost door TECH-AXIOS-STORE-COUPLING.

Merge

--no-ff (web UI: kies "Create merge commit") per CLAUDE.md commit hygiene.

Twee triviale cleanup-items uit de backlog die in dezelfde tooling-batch mee kunnen. Beide ontstonden tijdens / na WS-3 sessie 1c. ## Commits 1. **b9f8f55** — `chore(tooling): remove stale apps/admin entry from .vscode/settings.json` `apps/admin/` is in april 2026 verwijderd (admin SPA gemerged in `apps/app/` onder `/platform/*`). De `eslint.workingDirectories` verwijzing was achtergebleven; Cursor's ESLint extension skipte de missende directory silently maar de dode config-regel was verwarrend tijdens debugging. Closes `TECH-VSCODE-STALE-ADMIN-ENTRY`. 2. **bdbd5b0** — `chore(cleanup): delete dead Vuexy views/ directory` `apps/app/src/views/` bevatte één Vuexy-template-bestand (`AuthProvider.vue`) met nul importers in de hele repo. WS-3 sessie 1c audit identificeerde het als dead vendored code. De §4.2 post- consolidation target layout drop `views/` volledig. Inclusief verwijdering van de `'src/views/**'` regel uit `boundaries/ignore` in `apps/app/.eslintrc.cjs`. Closes `TECH-DELETE-DEAD-VIEWS`. 3. **617a6d2** — `docs(backlog): remove two closed tooling items` BACKLOG.md opschonen. Beide entries fysiek verwijderd; closure- referenties leven in de twee chore-commits hierboven (en zijn eeuwig findbaar via `git log --grep=TECH-VSCODE-STALE` / `--grep=TECH-DELETE-DEAD-VIEWS`). ## Acceptance - ✅ `pnpm lint` 0 problems in `apps/app/` - ✅ `pnpm build` succeeds (11.09s) - ✅ `pnpm test` 49 passed (geen regressie) - ✅ Geen wijziging in `apps/portal/`, `api/`, `packages/`, `docs/` - ✅ Conventional Commits + `Co-Authored-By: Claude` op alle commits ## Pre-existing warnings (niet door deze PR veroorzaakt) Voor de duidelijkheid bij review: - TypeScript 5.9.3 buiten range van `@typescript-eslint/typescript-estree` — wordt opgelost door `TECH-ESLINT-V9-MIGRATION`. - `boundaries/element-types` v5→v6 deprecation warnings — wordt opgelost door `TECH-BOUNDARIES-V6-SELECTOR-MIGRATION`. - Vite/Rollup mixed static+dynamic import warnings voor `useAuthStore` / `useImpersonationStore` — wordt opgelost door `TECH-AXIOS-STORE-COUPLING`. ## Merge `--no-ff` (web UI: kies "Create merge commit") per CLAUDE.md commit hygiene.

bert.hausmans added 3 commits 2026-05-04 21:44:08 +02:00

chore(tooling): remove stale apps/admin entry from .vscode/settings.json b9f8f558d1

apps/admin/ was removed in April 2026 (admin SPA merged into apps/app/
under /platform/*). Cursor's ESLint extension silently skipped the
missing directory, but the dead config entry caused confusion when
debugging extension activation issues.

Closes TECH-VSCODE-STALE-ADMIN-ENTRY.

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

chore(cleanup): delete dead Vuexy views/ directory bdbd5b0335

src/views/ contained a single Vuexy-template file
(views/pages/authentication/AuthProvider.vue) with zero importers
in the repo. Vendored dead code from the original Vuexy template;
the §4.2 post-consolidation target layout drops views/ entirely.

Removed:
- apps/app/src/views/ (recursive)
- 'src/views/**' line from boundaries/ignore in .eslintrc.cjs

Closes TECH-DELETE-DEAD-VIEWS.

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

docs(backlog): remove two closed tooling items 617a6d2d13

- TECH-VSCODE-STALE-ADMIN-ENTRY: closed in b9f8f558d1
- TECH-DELETE-DEAD-VIEWS: closed in bdbd5b0335

Both items shipped; references preserved in git history.

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

bert.hausmans referenced this issue from a commit

2026-05-04 21:45:49 +02:00

Merge pull request #2: chore: two tooling cleanups (admin .vscode entry + dead views/ dir)

bert.hausmans closed this pull request

2026-05-04 21:46:06 +02:00

bert.hausmans deleted branch chore/tooling-cleanups-may

2026-05-04 21:46:06 +02:00