# Observations

Issues noticed incidentally during development. See `CLAUDE.md` for logging rules.

| Date | Type | File | Observation |
|------|------|------|-------------|
| 2026-08-19 | `bug` | `tests/i18n-server.integration.test.ts` | The test updates a presumed `expert@example.com` fixture without creating it, so it fails on a freshly migrated database when the update returns no user |
| 2026-08-19 | `debt` | `tests/expert-reply.integration.test.ts` | The route test duplicates reply-route logic instead of exercising the route, allowing its permitted-status list to drift from the UI and production API |
| 2026-08-19 | `debt` | `tests/db.smoke.test.ts` | Test still expects the removed `resolution_summary` column, causing the full suite to fail independently of current changes |
| 2026-05-16 | `i18n` | `src/inngest/completeness-agent.ts` | Hardcoded English strings in the missing-fields clarification path (system message body, artifact content, question labels like "Please provide: X") even though `submitterLanguage` is already fetched — Spanish-speaking submitters receive English clarification prompts when structured fields are absent |
| 2026-05-15 | `data` | `src/db/migrations/0014_yielding_karnak.sql` | `preferred_language` column added with `DEFAULT 'en'` — all pre-migration users have `'en'` stored regardless of actual preference, so Spanish-speaking users who never revisited their profile settings will silently get English on pre-auth pages |
