fix(ideas): herstel 64.000-tekenlimiet voor beschrijving in web #280

Merged
janpeter merged 1 commit from claude/sleepy-dirac-162dc6 into main 2026-10-01 20:39:00 +02:00
Owner

Probleem

Opslaan van een idee gaf "Validatie mislukt" zodra de beschrijving langer was dan 4.000 tekens (bv. via MCP geschreven). DB (VARCHAR(64000)) en MCP (ideaDescriptionSchema) staan al op 64.000; het web-Zod-schema nog op 4.000.

Oorzaak

PR #207 (cf904849) is gemerged naar codex/opus-48-job-defaults, een branch die in juni al via #85 op main stond. De webkant kwam daardoor nooit op main; alleen de kolomverbreding landde later via M39 (0db91116).

Wijziging

Cherry-pick van cf904849 zónder de dubbele migratie, schema- en submodule-wijziging (al op main):

  • lib/idea-description.ts + lib/schemas/idea.ts: limiet 64.000 (Unicode-tekens), zelfde melding als MCP
  • actions/ideas.ts: toont de concrete veldfout i.p.v. alleen "Validatie mislukt"
  • IdeaDescriptionField (teller + blokkade) in desktop- en mobiele idee-formulieren
  • REST-contract en dialog-spec bijgewerkt
  • Test kreeg de inmiddels verplichte userId-prop

Verificatie

  • Gerichte tests (idea-schemas, ideas-crud, idea-list, mobile-idea-detail): 105/105 groen
  • RED-check: met het oude schema falen 5 limiettests
  • npm run verify: lint 0 errors, typecheck groen, 3094 tests passed
  • npm run build niet gedraaid (worktree zonder .env)

Geen migratie; geen gevolgen voor MCP.

🤖 Generated with Claude Code

## Probleem Opslaan van een idee gaf "Validatie mislukt" zodra de beschrijving langer was dan 4.000 tekens (bv. via MCP geschreven). DB (`VARCHAR(64000)`) en MCP (`ideaDescriptionSchema`) staan al op 64.000; het web-Zod-schema nog op 4.000. ## Oorzaak PR #207 (`cf904849`) is gemerged naar `codex/opus-48-job-defaults`, een branch die in juni al via #85 op main stond. De webkant kwam daardoor nooit op main; alleen de kolomverbreding landde later via M39 (`0db91116`). ## Wijziging Cherry-pick van `cf904849` zónder de dubbele migratie, schema- en submodule-wijziging (al op main): - `lib/idea-description.ts` + `lib/schemas/idea.ts`: limiet 64.000 (Unicode-tekens), zelfde melding als MCP - `actions/ideas.ts`: toont de concrete veldfout i.p.v. alleen "Validatie mislukt" - `IdeaDescriptionField` (teller + blokkade) in desktop- en mobiele idee-formulieren - REST-contract en dialog-spec bijgewerkt - Test kreeg de inmiddels verplichte `userId`-prop ## Verificatie - Gerichte tests (idea-schemas, ideas-crud, idea-list, mobile-idea-detail): 105/105 groen - RED-check: met het oude schema falen 5 limiettests - `npm run verify`: lint 0 errors, typecheck groen, 3094 tests passed - `npm run build` niet gedraaid (worktree zonder `.env`) Geen migratie; geen gevolgen voor MCP. 🤖 Generated with [Claude Code](https://claude.com/claude-code)
fix(ideas): herstel 64.000-tekenlimiet voor beschrijving in web
All checks were successful
CI / DB Access Policy Gate (pull_request) Successful in 3m5s
CI / Lint, Typecheck, Test & Build (pull_request) Successful in 5m4s
CI / Queue Dispatch Guard Gate (pull_request) Successful in 3m6s
CI / Deploy Manual (workflow_dispatch) (pull_request) Has been skipped
CI / DB Access Production Sentinel (pull_request) Successful in 1m29s
CI / Detect deploy-relevant changes (pull_request) Has been skipped
CI / Deploy Preview (PR) (pull_request) Has been skipped
CI / Deploy Production (main) (pull_request) Has been skipped
85fb8cf53a
PR #207 (cf904849) is gemerged naar codex/opus-48-job-defaults, een branch
die in juni al via #85 op main stond. De webkant kwam daardoor nooit op main;
alleen de kolomverbreding landde later via M39 (0db91116). Gevolg: DB en MCP
staan op 64.000 tekens, web-Zod nog op 4.000, en opslaan gaf "Validatie
mislukt" zodra een (bv. via MCP geschreven) beschrijving langer was.

Cherry-pick van cf904849 zonder de dubbele migratie, schema- en
submodule-wijziging (al op main). Test krijgt de inmiddels verplichte
userId-prop.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
s4m-codex-reviewer left a comment

APPROVED

Geen blokkerende of error-severity findings.

  • Geen gekoppeld plan gevonden — beoordeeld op codekwaliteit + product-standaarden.
  • De 64.000-tekenlimiet wordt centraal en Unicode-codepoint-correct gevalideerd, in de server action afgedwongen en in alle gewijzigde idea-invoerpunten met toegankelijke live feedback getoond.
  • Tests dekken de grenswaarden, Unicode-overflow en de belangrijkste create/edit UI-flows; de API- en dialogdocumentatie is bijgewerkt.
## APPROVED Geen blokkerende of error-severity findings. - Geen gekoppeld plan gevonden — beoordeeld op codekwaliteit + product-standaarden. - De 64.000-tekenlimiet wordt centraal en Unicode-codepoint-correct gevalideerd, in de server action afgedwongen en in alle gewijzigde idea-invoerpunten met toegankelijke live feedback getoond. - Tests dekken de grenswaarden, Unicode-overflow en de belangrijkste create/edit UI-flows; de API- en dialogdocumentatie is bijgewerkt.
janpeter scheduled this pull request to auto merge when all checks succeed 2026-10-01 20:09:40 +02:00
Sign in to join this conversation.
No reviewers
No milestone
No project
No assignees
2 participants
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference
janpeter/Scrum4Me!280
No description provided.