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

chore(shared): submodule naar c63f0bd — issue-tracker-schema #83

Merged
janpeter merged 1 commit from bump/shared-issue-tracker into main 2026-08-17 10:17:55 +02:00
Conversation 1 Commits 1 Files changed 1 +1 -1
janpeter commented 2026-08-17 09:13:15 +02:00
Owner
Copy link

Runbook issue-tracker-rollout — stap 1, laatste openstaande onderdeel.

Waarom

De submodule stond hier nog op 0cc70beb, terwijl Scrum4Me en scrum4me-mcp allebei al op c63f0bd7 staan. De runbook noemt dit een hygiëne-bump: de nieuwe tabellen raken geen bestaande query in deze consument, maar de lockstep-regel blijft gelden — een consument die achterblijft crasht zodra er een rij met een nieuwe enum-waarde ontstaat.

Het schema van deze repo is niet ingecheckt maar wordt bij de build gegenereerd uit de submodule (gen-schema.sh in postinstall/prebuild), dus de pointer verzetten ís de hele wijziging.

Verificatie

Niet op "het is maar een pointer" afgegaan — de bump brengt 150 regels schema mee, dus dit is precies het moment om te toetsen of de consument dat verdraagt:

  • npm ci → schema gegenereerd, 1642 regels, met het Issue-model erin
  • tsc --noEmit → 0 fouten
  • vitest run → 121 passed, 1 skipped, geen failures

Stand van stap 1 na deze PR

repo submodule status
scrum4me-shared — main op c63f0bd7
Scrum4Me c63f0bd7 al binnen via #174
scrum4me-mcp c63f0bd7 al binnen; sync-schema gaf geen diff
scrum4me-workers 0cc70beb → c63f0bd7 deze PR

Daarmee is stap 1 compleet en kan stap 2 (migraties) aan.

Runbook [issue-tracker-rollout](../src/branch/main/docs/runbooks/issue-tracker-rollout.md) — **stap 1**, laatste openstaande onderdeel. ## Waarom De submodule stond hier nog op `0cc70beb`, terwijl `Scrum4Me` en `scrum4me-mcp` allebei al op `c63f0bd7` staan. De runbook noemt dit een hygiëne-bump: de nieuwe tabellen raken geen bestaande query in deze consument, maar de lockstep-regel blijft gelden — een consument die achterblijft crasht zodra er een rij met een nieuwe enum-waarde ontstaat. Het schema van deze repo is niet ingecheckt maar wordt bij de build gegenereerd uit de submodule (`gen-schema.sh` in `postinstall`/`prebuild`), dus de pointer verzetten ís de hele wijziging. ## Verificatie Niet op "het is maar een pointer" afgegaan — de bump brengt 150 regels schema mee, dus dit is precies het moment om te toetsen of de consument dat verdraagt: - `npm ci` → schema gegenereerd, **1642 regels**, met het `Issue`-model erin - `tsc --noEmit` → **0 fouten** - `vitest run` → **121 passed, 1 skipped**, geen failures ## Stand van stap 1 na deze PR | repo | submodule | status | |---|---|---| | `scrum4me-shared` | — | main op `c63f0bd7` | | `Scrum4Me` | `c63f0bd7` | al binnen via #174 | | `scrum4me-mcp` | `c63f0bd7` | al binnen; `sync-schema` gaf geen diff | | `scrum4me-workers` | `0cc70beb` → `c63f0bd7` | **deze PR** | Daarmee is stap 1 compleet en kan stap 2 (migraties) aan.
chore(shared): submodule naar c63f0bd — issue-tracker-schema
Some checks failed
CI / Verify (pull_request) Failing after 2m35s
Details
7578a06502 
Lockstep-bump horend bij de issue-tracker-rollout (runbook stap 1). De nieuwe
tabellen raken geen bestaande query in deze consument, maar de lockstep-regel
blijft de norm: een consument die achterblijft crasht zodra er een rij met een
nieuwe enum-waarde ontstaat.

Brengt 150 regels schema mee (Issue-model, HubPermissionRule en bijbehorende
enums). Het schema van deze repo wordt bij de build gegenereerd uit de
submodule, dus de pointer verzetten is de hele wijziging.

Geverifieerd na de bump: npm ci genereert een schema van 1642 regels met het
Issue-model erin, tsc --noEmit geeft 0 fouten en de suite draait 121 passed /
1 skipped zonder failures.

Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
s4m-codex-reviewer approved these changes 2026-08-17 09:16:46 +02:00
s4m-codex-reviewer left a comment
Copy link

Verdict: APPROVED

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

Findings

  • info — vendor/scrum4me-shared:1 — Submodule-bump van 0cc70beb74c4d67729651f2473c172f9951065ef naar c63f0bd7128d2b681b3cf7e28e3f38787e2537b1 is resolvebaar en staat op scrum4me-shared/main. Dit past bij de productstandaard dat het canonieke Prisma-schema uit vendor/scrum4me-shared komt en schemawijzigingen eerst in shared landen.
  • info — vendor/scrum4me-shared:1 — De target-range bevat nieuwe shared tests voor issue-status en Forgejo-mirror helpers. Lokale uitvoering van npm test in de tijdelijke shared-clone was niet mogelijk omdat vitest/dev-dependencies daar niet geinstalleerd waren; geen blocker voor deze pointer-only PR.

Geen blockerende of error-severity findings gevonden.

# Verdict: APPROVED Geen gekoppeld plan gevonden — beoordeeld op codekwaliteit + product-standaarden. ## Findings - info — vendor/scrum4me-shared:1 — Submodule-bump van `0cc70beb74c4d67729651f2473c172f9951065ef` naar `c63f0bd7128d2b681b3cf7e28e3f38787e2537b1` is resolvebaar en staat op `scrum4me-shared/main`. Dit past bij de productstandaard dat het canonieke Prisma-schema uit `vendor/scrum4me-shared` komt en schemawijzigingen eerst in shared landen. - info — vendor/scrum4me-shared:1 — De target-range bevat nieuwe shared tests voor issue-status en Forgejo-mirror helpers. Lokale uitvoering van `npm test` in de tijdelijke shared-clone was niet mogelijk omdat `vitest`/dev-dependencies daar niet geinstalleerd waren; geen blocker voor deze pointer-only PR. Geen blockerende of error-severity findings gevonden.
Sign in to join this conversation.
Reviewers
No reviewers
s4m-codex-reviewer
Labels
Clear labels
No items
No labels
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-workers!83
No description provided.