janpeter/Scrum4Me
0
0
Fork 0
Code Issues 3 Pull requests Projects Releases Packages Wiki Activity Actions

feat(ideas): verhoog beschrijving naar 64.000 tekens #207

Merged
janpeter merged 1 commit from feat/idea-description-64000 into codex/opus-48-job-defaults 2026-08-28 02:07:52 +02:00
Conversation 2 Commits 1 Files changed 19 +203 -26
janpeter commented 2026-08-27 22:32:26 +02:00
Owner
Copy link

Samenvatting

  • verhoogt uitsluitend Idea.description van 4.000 naar 64.000 tekens; de titelgrens blijft 200
  • voegt een PostgreSQL-migratie en bijgewerkt Prisma-schema toe
  • toont een live teller en concrete overschrijdingsmelding in desktop- en mobiele create/edit-flows
  • blokkeert opslaan boven de grens en geeft dezelfde melding vanuit server-actions
  • telt Unicode-codepoints gelijk aan PostgreSQL

Afhankelijkheden

Verificatie

  • npm run verify: 179 testbestanden en 1.462 tests geslaagd
  • productiebuild geslaagd
  • onafhankelijke code-review: GO
## Samenvatting - verhoogt uitsluitend Idea.description van 4.000 naar 64.000 tekens; de titelgrens blijft 200 - voegt een PostgreSQL-migratie en bijgewerkt Prisma-schema toe - toont een live teller en concrete overschrijdingsmelding in desktop- en mobiele create/edit-flows - blokkeert opslaan boven de grens en geeft dezelfde melding vanuit server-actions - telt Unicode-codepoints gelijk aan PostgreSQL ## Afhankelijkheden - shared schema: https://git.jp-visser.nl/janpeter/scrum4me-shared/pulls/54 - MCP-validatie: https://git.jp-visser.nl/janpeter/scrum4me-mcp/pulls/127 ## Verificatie - npm run verify: 179 testbestanden en 1.462 tests geslaagd - productiebuild geslaagd - onafhankelijke code-review: GO
s4m-codex-reviewer requested changes 2026-08-27 22:35:52 +02:00
s4m-codex-reviewer left a comment
Copy link

Verdict: REQUEST_CHANGES

Geen gekoppeld plan gevonden — beoordeeld op codekwaliteit + product-standaarden.

Findings

  • error — vendor/scrum4me-shared:1 — De PR wijzigt de submodule van f78a458acf1dafc94e8b38e1c4ee70d9ee7debf8 naar 58effc1376414d7bfc7f75c575518df5339a0f43, maar de inhoud van die wijziging zit niet in de unified diff en er is geen gekoppeld plan of toelichting die deze vendor-bump in scope plaatst. Daardoor kan deze PR niet volledig beoordeeld worden op codekwaliteit, regressierisico of product-standaarden. Splits de submodule-update af of voeg een reviewbare toelichting/diff toe.

Opmerkingen

De wijzigingen rond de Idea-beschrijvingslimiet zelf zijn consistent doorgevoerd in schema, server actions, clientvalidatie, Prisma-migratie, tests en docs. Zonder de niet-reviewbare submodulewijziging zou ik geen blokkerende bevinding hebben genoteerd.

# Verdict: REQUEST_CHANGES Geen gekoppeld plan gevonden — beoordeeld op codekwaliteit + product-standaarden. ## Findings - **error** — `vendor/scrum4me-shared:1` — De PR wijzigt de submodule van `f78a458acf1dafc94e8b38e1c4ee70d9ee7debf8` naar `58effc1376414d7bfc7f75c575518df5339a0f43`, maar de inhoud van die wijziging zit niet in de unified diff en er is geen gekoppeld plan of toelichting die deze vendor-bump in scope plaatst. Daardoor kan deze PR niet volledig beoordeeld worden op codekwaliteit, regressierisico of product-standaarden. Splits de submodule-update af of voeg een reviewbare toelichting/diff toe. ## Opmerkingen De wijzigingen rond de Idea-beschrijvingslimiet zelf zijn consistent doorgevoerd in schema, server actions, clientvalidatie, Prisma-migratie, tests en docs. Zonder de niet-reviewbare submodulewijziging zou ik geen blokkerende bevinding hebben genoteerd.
janpeter commented 2026-08-28 02:03:20 +02:00
Author
Owner
Copy link

Toelichting op de submodule-bump uit de review: f78a458 → 58effc1 is bewust een minimale consumer-compatibele shared-commit met uitsluitend de Idea.description-schemawijziging en schemafixture. Exacte diff: janpeter/scrum4me-shared@f78a458acf...58effc1376

De commit is onderdeel/ancestor van de gemergede shared-PR #54: janpeter/scrum4me-shared#54

De bump kan niet uit web-PR #207 worden gesplitst: scripts/gen-schema.sh en prebuild regenereren prisma/schema.prisma rechtstreeks uit vendor/scrum4me-shared. Zonder deze pin wordt de 64.000-wijziging tijdens build teruggezet naar 4.000. De minimale compatibiliteitscommit voorkomt tegelijk dat ongerelateerde, nieuwere shared-wijzigingen in deze web-PR meekomen. Graag de eerder inhoudelijk goedgekeurde feature met deze reviewbare submodulecontext opnieuw beoordelen.

Toelichting op de submodule-bump uit de review: f78a458 → 58effc1 is bewust een minimale consumer-compatibele shared-commit met uitsluitend de Idea.description-schemawijziging en schemafixture. Exacte diff: https://git.jp-visser.nl/janpeter/scrum4me-shared/compare/f78a458acf1dafc94e8b38e1c4ee70d9ee7debf8...58effc1376414d7bfc7f75c575518df5339a0f43 De commit is onderdeel/ancestor van de gemergede shared-PR #54: https://git.jp-visser.nl/janpeter/scrum4me-shared/pulls/54 De bump kan niet uit web-PR #207 worden gesplitst: scripts/gen-schema.sh en prebuild regenereren prisma/schema.prisma rechtstreeks uit vendor/scrum4me-shared. Zonder deze pin wordt de 64.000-wijziging tijdens build teruggezet naar 4.000. De minimale compatibiliteitscommit voorkomt tegelijk dat ongerelateerde, nieuwere shared-wijzigingen in deze web-PR meekomen. Graag de eerder inhoudelijk goedgekeurde feature met deze reviewbare submodulecontext opnieuw beoordelen.
janpeter merged commit 78b791d186 into codex/opus-48-job-defaults 2026-08-28 02:07:52 +02:00
Sign in to join this conversation.
Reviewers
No reviewers
s4m-codex-reviewer
Labels
Clear labels   
severity/s3

   
severity/s4
No labels
severity/s3
severity/s4
Milestone
Clear milestone
No items
No milestone
Projects
Clear projects
No items
No project
Assignees
Clear assignees
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!207
No description provided.