✅ Decisions Resolved
Three open questions from the persona reviews are now closed.
1. Font — Merriweather (no change)
Fraunces is not adopted. Story pages use the existing…
📋 Elicit — Requirements Engineer (spec review)
The four spec files are high quality — impl-ref tables, visual mockups, and explicit implementation decisions are exactly what makes a spec…
🛠️ Tobias Wendt — DevOps & Platform Engineer (spec review)
Short update — the specs don't change the infrastructure picture, but there are three new operational details worth noting.
###…
🎨 Leonie Voss — UX Designer & Accessibility Strategist (spec review)
The specs are well-structured — the impl-ref tables are exactly the right level of precision for handoff. A few places…
🧪 Sara Holt — QA Engineer (spec review)
The four specs add substantial surface area that has no AC coverage in the original issue. I'm cataloguing the gaps and the new journeys.
AC…
🔒 Nora "NullX" Steiner — Security Engineer (spec review)
The four specs add two new attack surfaces not fully addressed in my earlier review. The existing concerns (XSS, DRAFT exposure)…
👨💻 Felix Brandt — Senior Fullstack Developer (spec review)
The four spec files resolve the open decisions from my earlier review and add concrete implementation guidance. Here's what…
🏛️ Markus Keller — Application Architect (spec review)
The four spec files resolve several earlier open questions but also introduce scope that the issue body doesn't cover. I'm flagging…
🗳️ Decision Queue — Action Required
2 decisions need your input before implementation starts.
Architecture / Data Model
- Story body character limit — The
bodyfield is…
🛠️ Tobias Wendt — DevOps & Platform Engineer
Observations
- Zero new infrastructure required. Geschichten are stored in PostgreSQL and served via the existing Spring Boot +…
🎨 Leonie Voss — UX Designer & Accessibility Strategist
Observations
Navigation
The issue says "Geschichten" visible to all logged-in users. Looking at AppNav.svelte, the pattern…
🧪 Sara Holt — QA Engineer
Observations
Missing acceptance criteria / edge cases not covered by the spec:
- US-BLOG-001, empty body: The spec requires a title to save. What…
🔒 Nora "NullX" Steiner — Security Engineer
Observations
Critical: Stored XSS via rich text body
The body field is rich text HTML stored in PostgreSQL and rendered in the browser.…
👨💻 Felix Brandt — Senior Fullstack Developer
Observations
- Reusable components already exist:
PersonTypeaheadandPersonMultiSelectare the right building blocks for the…
🏛️ Markus Keller — Application Architect
Observations
- Module package: The existing layout is
controller/,service/,repository/,model/— but that's layer-first. The…
Implementation complete
Branch: feat/issue-370-at-mention-placeholder-hint
PR: #379
What was done
Implemented Leonie's design decision from the review comment: update the 3 existing…