diff options
| author | Martin Kagamino Lehoux <martin@lehoux.net> | 2026-07-28 15:07:57 +0200 |
|---|---|---|
| committer | Martin Kagamino Lehoux <martin@lehoux.net> | 2026-07-28 15:07:57 +0200 |
| commit | 04fabf115ccea31ac504b06cc51d78423b697f60 (patch) | |
| tree | 26c21f8822ccd24fce3a80b7361382c8a87d0c96 /src/routes/catalog/+page.server.ts | |
| parent | 45cc0afb734a422ee7765dc5c60dccbcc6721367 (diff) | |
refactor: services layer, valibot validation, shared CSS utilities, aria labels, and drizzle migrate
- Extract groceries/categories/references services with db injection
- Add valibot for form validation and safe numeric id parsing
- Fix category delete FK behavior with onDelete: set null + migration
- Replace custom migrate.js with drizzle-orm/libsql/migrator
- Add shared CSS utility classes in app.css (forms, lists, buttons)
- Add aria-pressed and aria-label on interactive elements
- Remove redundant app.css imports and unused /stores import
- Add integration tests for all services using drizzle migrate helper
- Remove smoke test; add db:migrate and db:studio scripts
Diffstat (limited to 'src/routes/catalog/+page.server.ts')
| -rw-r--r-- | src/routes/catalog/+page.server.ts | 54 |
1 files changed, 15 insertions, 39 deletions
diff --git a/src/routes/catalog/+page.server.ts b/src/routes/catalog/+page.server.ts index a3a611e..69a775a 100644 --- a/src/routes/catalog/+page.server.ts +++ b/src/routes/catalog/+page.server.ts @@ -1,52 +1,28 @@ import { fail } from '@sveltejs/kit'; -import { db } from '$lib/server/db'; -import { groceries, categories, references } from '$lib/server/db/schema'; -import { eq, desc } from 'drizzle-orm'; +import { loadAll, toggle } from '$lib/server/services/groceries'; +import { numericId } from '$lib/server/services/validation'; +import type { EnrichedGrocery } from '$lib/server/services/groceries'; +import type { Category } from '$lib/server/services/types'; import type { Actions, PageServerLoad } from './$types'; -export const load: PageServerLoad = async () => { - const [items, cats, refs] = await Promise.all([ - db.select().from(groceries).orderBy(desc(groceries.createdAt)), - db.select().from(categories).orderBy(desc(categories.name)), - db.select().from(references).orderBy(desc(references.qualityScore)), - ]); - - const catMap = new Map(cats.map((c) => [c.id, c.name])); - const bestRef = new Map<number, (typeof refs)[number]>(); - for (const ref of refs) { - if (!bestRef.has(ref.groceryId)) { - bestRef.set(ref.groceryId, ref); - } - } +export interface CatalogOutput { + items: EnrichedGrocery[]; + categories: Category[]; +} - return { - items: items.map((i) => ({ - ...i, - categoryName: i.categoryId ? (catMap.get(i.categoryId) ?? null) : null, - bestRef: bestRef.get(i.id) ?? null, - })), - categories: cats, - }; +export const load: PageServerLoad = async () => { + return loadAll(); }; export const actions: Actions = { toggle: async ({ request }) => { const data = await request.formData(); - const id = Number(data.get('id')); - - const [item] = await db - .select() - .from(groceries) - .where(eq(groceries.id, id)); - if (!item) { - return fail(404, { error: 'Article introuvable' }); + try { + const id = numericId(data.get('id')); + await toggle(id); + } catch { + return fail(400, { error: 'ID invalide' }); } - - await db - .update(groceries) - .set({ checked: !item.checked }) - .where(eq(groceries.id, id)); - return { success: true }; }, }; |
