Merge pull request #2: chore: two tooling cleanups (admin .vscode entry + dead views/ dir)
This commit is contained in:
4
.vscode/settings.json
vendored
4
.vscode/settings.json
vendored
@@ -17,10 +17,6 @@
|
||||
"eslint.nodePath": "apps/app/node_modules",
|
||||
"eslint.run": "onSave",
|
||||
"eslint.workingDirectories": [
|
||||
{
|
||||
"directory": "apps/admin",
|
||||
"changeProcessCWD": true
|
||||
},
|
||||
{
|
||||
"directory": "apps/app",
|
||||
"changeProcessCWD": true
|
||||
|
||||
@@ -255,7 +255,6 @@ module.exports = {
|
||||
'boundaries/ignore': [
|
||||
'src/@core/**', // vendored Vuexy
|
||||
'src/@layouts/**', // vendored Vuexy
|
||||
'src/views/**', // single dead Vuexy file (zero importers); see TECH-DELETE-DEAD-VIEWS
|
||||
'src/App.vue', // orchestration root
|
||||
'src/main.ts', // orchestration root
|
||||
'src/assets/**', // static media
|
||||
|
||||
@@ -1,46 +0,0 @@
|
||||
<script setup lang="ts">
|
||||
import { useTheme } from 'vuetify'
|
||||
|
||||
const { global } = useTheme()
|
||||
|
||||
const authProviders = [
|
||||
{
|
||||
icon: 'tabler-brand-facebook-filled',
|
||||
color: '#4267b2',
|
||||
colorInDark: '#497CE2',
|
||||
},
|
||||
{
|
||||
icon: 'tabler-brand-twitter-filled',
|
||||
color: '#1da1f2',
|
||||
colorInDark: '#1da1f2',
|
||||
},
|
||||
{
|
||||
icon: 'tabler-brand-github-filled',
|
||||
color: '#272727',
|
||||
colorInDark: '#fff',
|
||||
},
|
||||
{
|
||||
icon: 'tabler-brand-google-filled',
|
||||
color: '#dd4b39',
|
||||
colorInDark: '#db4437',
|
||||
},
|
||||
]
|
||||
</script>
|
||||
|
||||
<template>
|
||||
<div class="d-flex justify-center flex-wrap gap-1">
|
||||
<VBtn
|
||||
v-for="link in authProviders"
|
||||
:key="link.icon"
|
||||
icon
|
||||
variant="text"
|
||||
size="small"
|
||||
:color="global.name.value === 'dark' ? link.colorInDark : link.color"
|
||||
>
|
||||
<VIcon
|
||||
size="20"
|
||||
:icon="link.icon"
|
||||
/>
|
||||
</VBtn>
|
||||
</div>
|
||||
</template>
|
||||
@@ -602,26 +602,6 @@ sprint waard, geen meelift-pad.
|
||||
|
||||
---
|
||||
|
||||
### TECH-VSCODE-STALE-ADMIN-ENTRY — Verwijder apps/admin uit .vscode/settings.json eslint.workingDirectories
|
||||
|
||||
**Aanleiding:** `.vscode/settings.json` bevat in
|
||||
`eslint.workingDirectories` nog een entry voor `apps/admin`, een
|
||||
directory die in april 2026 volledig is verwijderd (admin SPA gemerged
|
||||
in apps/app/ onder `/platform/*`). Cursor's ESLint extension skipt
|
||||
missende directories silently dus is harmless, maar het is dode config
|
||||
die bij toekomstig debugging voor verwarring kan zorgen.
|
||||
**Wat:**
|
||||
- Open `.vscode/settings.json`.
|
||||
- Verwijder uit `eslint.workingDirectories` de regel:
|
||||
`{ "directory": "apps/admin", "changeProcessCWD": true },`
|
||||
- Commit als `chore(tooling): remove stale apps/admin entry from
|
||||
.vscode/settings.json`.
|
||||
**Prioriteit:** Laag — triviaal cleanup-item, kan in elke gerelateerde
|
||||
tooling-sprint meeliften (TECH-PORTAL-ESLINT-DEPS of
|
||||
TECH-ESLINT-V9-MIGRATION zijn natuurlijke kandidaten).
|
||||
|
||||
---
|
||||
|
||||
### TECH-AXIOS-STORE-COUPLING — Decouple lib/axios.ts from stores layer
|
||||
|
||||
**Aanleiding:** WS-3 sessie 1c (eslint-plugin-boundaries enforcement)
|
||||
@@ -662,28 +642,6 @@ of `plugins/` raakt, of een dedicated 2-3 uur sessie na WS-6 sluiting.
|
||||
|
||||
---
|
||||
|
||||
### TECH-DELETE-DEAD-VIEWS — Verwijder src/views/ uit apps/app/
|
||||
|
||||
**Aanleiding:** WS-3 sessie 1c audit (`dev-docs/WS-3-SESSION-1C-AUDIT.md`
|
||||
§A.1) constateerde dat `apps/app/src/views/` precies één bestand bevat
|
||||
(`views/pages/authentication/AuthProvider.vue`) met nul importers in de
|
||||
hele repo. Het is overgebleven Vuexy-template dode code. De §4.2
|
||||
post-consolidatie target layout drop `views/` volledig. Het bestand is
|
||||
nu in `boundaries/ignore` opgenomen om sessie 1c te laten landen, maar
|
||||
de natuurlijke vervolgstap is fysieke verwijdering.
|
||||
|
||||
**Wat:**
|
||||
- Verwijder `apps/app/src/views/` recursief.
|
||||
- Verwijder de bijbehorende `'src/views/**'` regel uit
|
||||
`apps/app/.eslintrc.cjs` `boundaries/ignore`.
|
||||
- Verifieer dat `pnpm lint`, `pnpm build` en `pnpm test` groen blijven.
|
||||
- Eén commit: `chore(cleanup): delete dead Vuexy views/ directory`.
|
||||
|
||||
**Prioriteit:** Laag — triviaal cleanup-item, kan in elke gerelateerde
|
||||
housekeeping-sprint meeliften (bijvoorbeeld vóór of na WS-3 PR-B).
|
||||
|
||||
---
|
||||
|
||||
### TECH-WS3-BOUNDARIES-SUBZONES — Sub-zone import-boundaries inside components/ and pages/
|
||||
|
||||
**Aanleiding:** WS-3 sessie 1c heeft top-level zone-boundaries in
|
||||
|
||||
Reference in New Issue
Block a user