• Joined on 2026-03-17
marcel commented on issue marcel/mealprep#21 2026-04-03 09:27:19 +02:00
Frontend: A4 — Join household (accept invite)

🎨 Atlas — UI/UX Designer

A4 is the first thing an invited member sees of this app — it's both a trust signal and an onboarding moment. The design needs to do real work here.

**Identity…

marcel commented on issue marcel/mealprep#25 2026-04-03 09:27:19 +02:00
Frontend: B4 — Cook mode (full-screen step-by-step)

🔧 Backend Engineer

B4 is lightweight on the backend — one write endpoint and one read. But the cooking_log entry it creates feeds the variety algorithm, so the data model matters a lot…

marcel commented on issue marcel/mealprep#29 2026-04-03 09:27:17 +02:00
Frontend: J4 — Swap flow (action sheet + quick suggestions)

🛠️ Backend Engineer — Swap API & Logging

The swap flow has cleaner backend requirements than D1, but the swap logging requirement and the sorting logic for suggestions deserve careful…

marcel commented on issue marcel/mealprep#11 2026-04-03 09:27:16 +02:00
Unbounded limit/offset parameters allow resource exhaustion

👨‍💻 Kai — Frontend Engineer

This is a backend validation issue, but I want to flag where the frontend touches pagination parameters.

Where I send limit and offset

  • Any recipe list…
marcel commented on issue marcel/mealprep#8 2026-04-03 09:27:10 +02:00
Login error responses enable account enumeration via HTTP status codes

🧪 QA Engineer

This is a high-value security fix that also needs careful regression testing — changing the login error behavior touches a critical user-facing path. Here's the test matrix I'd…

marcel commented on issue marcel/mealprep#4 2026-04-03 09:27:09 +02:00
Sessions not invalidated on password/role change or deactivation

🎨 Atlas — UI/UX Designer

The security fix is clear, but the UX around it needs deliberate design — especially the moments where a user discovers their session has been terminated without…

marcel commented on issue marcel/mealprep#21 2026-04-03 09:27:04 +02:00
Frontend: A4 — Join household (accept invite)

🔐 Sable — Security Engineer

A4 is the invite acceptance flow, and it's a concentrated security risk surface. This is exactly the kind of screen that needs a threat model before a single line…

marcel commented on issue marcel/mealprep#25 2026-04-03 09:27:02 +02:00
Frontend: B4 — Cook mode (full-screen step-by-step)

👨‍💻 Kai — Frontend Engineer

B4 is unusual — it's the simplest layout in the app (single column, identical across breakpoints) but has the most critical non-visual requirements: wake lock,…

marcel commented on issue marcel/mealprep#12 2026-04-03 09:27:01 +02:00
Shopping list addItem does not validate ingredient belongs to household

🎨 Atlas — UI/UX Designer

Backend fix, but I want to flag the user-facing consequence of the 404 response once the fix is in.

The scenario from a user's perspective In normal usage,…

marcel commented on issue marcel/mealprep#29 2026-04-03 09:26:57 +02:00
Frontend: J4 — Swap flow (action sheet + quick suggestions)

👨‍💻 Kai — Frontend Engineer

The ≤3-tap constraint and the two completely different interaction patterns per breakpoint (action sheet on mobile, inline panel on desktop) make this a fun but…

marcel commented on issue marcel/mealprep#4 2026-04-03 09:26:55 +02:00
Sessions not invalidated on password/role change or deactivation

🔒 Sable — Security Engineer

This is one of the highest-impact issues in the backlog. Let me add precision to the threat model and fix requirements.

**Why both attack scenarios are…

marcel commented on issue marcel/mealprep#8 2026-04-03 09:26:53 +02:00
Login error responses enable account enumeration via HTTP status codes

🔧 Backend Engineer — Spring Boot / PostgreSQL Specialist

High priority and a clean fix — but a few implementation details worth getting right:

The right exception to throw:

  • Both…
marcel commented on issue marcel/mealprep#12 2026-04-03 09:26:52 +02:00
Shopping list addItem does not validate ingredient belongs to household

🔒 Sable — Security Engineer

This is a confirmed IDOR (Insecure Direct Object Reference) — OWASP Top 10 #1 (Broken Access Control). The attack scenario in the issue is realistic and…

marcel commented on issue marcel/mealprep#21 2026-04-03 09:26:50 +02:00
Frontend: A4 — Join household (accept invite)

🧪 QA Engineer — Join Household (A4)

The invite flow is a high-stakes path — it's the only way a new member enters the system. I want comprehensive coverage before this ships.

**Happy…

marcel commented on issue marcel/mealprep#4 2026-04-03 09:26:43 +02:00
Sessions not invalidated on password/role change or deactivation

🧪 QA Engineer

This is one of the harder scenarios to test because it requires multi-session state. Here's the test plan.

Unit tests:

  • These are limited here — the interesting behavior…
marcel commented on issue marcel/mealprep#21 2026-04-03 09:26:41 +02:00
Frontend: A4 — Join household (accept invite)

🔧 Backend Engineer — Join Household (A4)

The accept-invite flow touches three tables in one transaction: user_account, household_member, and household_invite. Let me walk through the…

marcel commented on issue marcel/mealprep#12 2026-04-03 09:26:40 +02:00
Shopping list addItem does not validate ingredient belongs to household

🧪 QA Engineer

This is a textbook IDOR that needs a direct regression test. The fix is small but the test is what guarantees it never regresses.

Tests I'd add for this fix

Happy path…

marcel commented on issue marcel/mealprep#8 2026-04-03 09:26:39 +02:00
Login error responses enable account enumeration via HTTP status codes

👨‍💻 Kai — Frontend Engineer

The fix happens entirely in AuthService.java, but the status code change to 401 will touch our SvelteKit error handling — here's what I want to make sure we…

marcel commented on issue marcel/mealprep#26 2026-04-03 09:26:35 +02:00
Frontend: C1 — Weekly planner (home screen)

🎨 Atlas — UI/UX Designer

C1 is the core value screen — the one users land on every day. The spec exists and is the authoritative reference, but I want to flag a few design concerns before…

marcel commented on issue marcel/mealprep#4 2026-04-03 09:26:32 +02:00
Sessions not invalidated on password/role change or deactivation

🏗️ Backend Engineer — Spring Boot / PostgreSQL Specialist

This is the right fix and it's non-trivial to implement correctly. Let me lay out the options clearly.

The core problem: Spring…