janpeter/scrum4me-mcp
1
0
Fork 0
Code Issues 6 Pull requests 1 Projects Releases Packages Wiki Activity Actions 1

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

Merged
janpeter merged 2 commits from feat/idea-description-64000 into main 2026-08-28 02:07:26 +02:00
Conversation 3 Commits 2 Files changed 8 +133 -6
janpeter commented 2026-08-27 22:32:19 +02:00
Owner
Copy link

Samenvatting

  • verhoogt de MCP-validatie voor Idea.description van 4.000 naar 64.000 tekens
  • geeft bij overschrijding de actuele lengte en het aantal te verwijderen tekens terug
  • telt Unicode-codepoints gelijk aan PostgreSQL en de webapp
  • synchroniseert het gegenereerde Prisma-schema

Afhankelijkheid

Verificatie

  • npm run typecheck
  • npm test
  • 189 testbestanden en 1.458 tests geslaagd; 3 DB-suites met 29 tests overgeslagen
## Samenvatting - verhoogt de MCP-validatie voor Idea.description van 4.000 naar 64.000 tekens - geeft bij overschrijding de actuele lengte en het aantal te verwijderen tekens terug - telt Unicode-codepoints gelijk aan PostgreSQL en de webapp - synchroniseert het gegenereerde Prisma-schema ## Afhankelijkheid - shared schema: https://git.jp-visser.nl/janpeter/scrum4me-shared/pulls/54 ## Verificatie - npm run typecheck - npm test - 189 testbestanden en 1.458 tests geslaagd; 3 DB-suites met 29 tests overgeslagen
feat(ideas): verhoog beschrijving naar 64.000 tekens
All checks were successful
CI / Verify (pull_request) Successful in 2m10s
Details
ca11e088f0
s4m-codex-reviewer requested changes 2026-08-27 22:36:41 +02:00
Dismissed
s4m-codex-reviewer left a comment
Copy link

Verdict: REQUEST_CHANGES

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

Findings

  • error — prisma/schema.prisma:1048 — De Prisma schemawijziging verhoogt Idea.description van @db.VarChar(4000) naar @db.VarChar(64000), maar de PR-diff bevat geen bijbehorende Prisma-migratie. De nieuwe Zod-validatie accepteert nu beschrijvingen tot 64.000 tekens, terwijl een bestaande databasekolom zonder migratie op 4.000 tekens blijft staan en writes in productie kunnen falen. Voeg een migratie toe die de kolom naar varchar(64000) wijzigt en verifieer de migratie/build/tests.

Tests/docs

  • Tests voor create/update-grenswaarden zijn toegevoegd, inclusief Unicode-telling.
  • Geen docs-wijziging vereist gezien de beperkte functionele wijziging, maar de database-migratie ontbreekt als release-kritische wijziging.
# Verdict: REQUEST_CHANGES Geen gekoppeld plan gevonden — beoordeeld op codekwaliteit + product-standaarden. ## Findings - error — `prisma/schema.prisma:1048` — De Prisma schemawijziging verhoogt `Idea.description` van `@db.VarChar(4000)` naar `@db.VarChar(64000)`, maar de PR-diff bevat geen bijbehorende Prisma-migratie. De nieuwe Zod-validatie accepteert nu beschrijvingen tot 64.000 tekens, terwijl een bestaande databasekolom zonder migratie op 4.000 tekens blijft staan en writes in productie kunnen falen. Voeg een migratie toe die de kolom naar `varchar(64000)` wijzigt en verifieer de migratie/build/tests. ## Tests/docs - Tests voor create/update-grenswaarden zijn toegevoegd, inclusief Unicode-telling. - Geen docs-wijziging vereist gezien de beperkte functionele wijziging, maar de database-migratie ontbreekt als release-kritische wijziging.
Merge remote-tracking branch 'origin/main' into feat/idea-description-64000
All checks were successful
CI / PR candidate (never published) (pull_request) Successful in 2m51s
Details
CI / Final merge attestation and immutable publication (pull_request) Has been skipped
Details
ebcea89c31 
# Conflicts:
#	vendor/scrum4me-shared
janpeter commented 2026-08-28 02:03:20 +02:00
Author
Owner
Copy link

Reviewblocker migratie geverifieerd tegen de repositoryarchitectuur: scrum4me-mcp heeft bewust geen prisma/migrations-map en genereert alleen zijn consumer-schema uit scrum4me-shared. Migraties draaien uitsluitend via Scrum4Me als designated migrator. De vereiste SQL-migratie staat daarom in web-PR #207: prisma/migrations/20260827211000_expand_idea_description/migration.sql. Een tweede migratie in MCP zou de migratieverantwoordelijkheid dupliceren.

De branch is daarnaast non-destructief bijgewerkt met de actuele main. Het submoduleconflict is opgelost naar de inmiddels gemergede shared-PR #54 (56e9905), het schema is opnieuw gegenereerd en de parity-pin/hash zijn bijgewerkt.

Verificatie lokaal: typecheck + testtypecheck groen; 199 testbestanden/1.530 tests groen, 3 bestaande suites/34 tests overgeslagen. De drie PPE-DB-suites vereisen Forgejo-CI PostgreSQL en worden door de nieuwe CI-run uitgevoerd. Graag herbeoordelen op deze architectuur.

Reviewblocker migratie geverifieerd tegen de repositoryarchitectuur: scrum4me-mcp heeft bewust geen prisma/migrations-map en genereert alleen zijn consumer-schema uit scrum4me-shared. Migraties draaien uitsluitend via Scrum4Me als designated migrator. De vereiste SQL-migratie staat daarom in web-PR #207: prisma/migrations/20260827211000_expand_idea_description/migration.sql. Een tweede migratie in MCP zou de migratieverantwoordelijkheid dupliceren. De branch is daarnaast non-destructief bijgewerkt met de actuele main. Het submoduleconflict is opgelost naar de inmiddels gemergede shared-PR #54 (56e9905), het schema is opnieuw gegenereerd en de parity-pin/hash zijn bijgewerkt. Verificatie lokaal: typecheck + testtypecheck groen; 199 testbestanden/1.530 tests groen, 3 bestaande suites/34 tests overgeslagen. De drie PPE-DB-suites vereisen Forgejo-CI PostgreSQL en worden door de nieuwe CI-run uitgevoerd. Graag herbeoordelen op deze architectuur.
s4m-codex-reviewer requested changes 2026-08-28 02:05:55 +02:00
s4m-codex-reviewer left a comment
Copy link

Verdict: REQUEST_CHANGES

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

Findings

  • error — prisma/schema.prisma:1048 — De schemawijziging verhoogt Idea.description van @db.VarChar(4000) naar @db.VarChar(64000), maar de PR voegt geen Prisma/database-migratie toe. Daardoor accepteert create-idea/update-idea nu 64.000 tekens terwijl bestaande databases nog een 4.000-teken kolom kunnen hebben, met runtime DB-fouten of mislukte writes als gevolg. Voeg de bijbehorende migratie toe en verifieer die tegen de bestaande databasekolom.

Opmerkingen

De gedeelde submodule en parity-hashes zijn meegebumped, en de nieuwe create/update-tests dekken de 64.000-grens inclusief Unicode-codepoints. Zonder migratie is de acceptatiegrens echter niet end-to-end doorgevoerd.

# Verdict: REQUEST_CHANGES geen gekoppeld plan gevonden — beoordeeld op codekwaliteit + product-standaarden. ## Findings - **error** — `prisma/schema.prisma:1048` — De schemawijziging verhoogt `Idea.description` van `@db.VarChar(4000)` naar `@db.VarChar(64000)`, maar de PR voegt geen Prisma/database-migratie toe. Daardoor accepteert `create-idea`/`update-idea` nu 64.000 tekens terwijl bestaande databases nog een 4.000-teken kolom kunnen hebben, met runtime DB-fouten of mislukte writes als gevolg. Voeg de bijbehorende migratie toe en verifieer die tegen de bestaande databasekolom. ## Opmerkingen De gedeelde submodule en parity-hashes zijn meegebumped, en de nieuwe create/update-tests dekken de 64.000-grens inclusief Unicode-codepoints. Zonder migratie is de acceptatiegrens echter niet end-to-end doorgevoerd.
Sign in to join this conversation.
Reviewers
No reviewers
s4m-codex-reviewer
Labels
Clear labels   
severity/s2

   
severity/s3

   
severity/s4
No labels
severity/s2
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-mcp!127
No description provided.