chore(shared): bump scrum4me-shared naar 9812ae5 (kimi in QUEUE_MODELS) #74

Merged

janpeter merged 1 commit from feat/shared-kimi-bump into main 2026-07-27 00:27:54 +02:00

Conversation 1 Commits 1 Files changed 1 +1 -1

janpeter commented 2026-07-27 00:03:59 +02:00

Owner

Copy link

Bumpt de scrum4me-shared-submodule van 69cb33f naar 9812ae5, de commit die het queue-vocabulaire uitbreidt.

Eén gitlink-wijziging, verder niets. Tests, typecheck en lint zijn vóór en ná identiek — geen enkele testtelling beweegt.

Wat de bump functioneel meebrengt is precies twee dingen: kimi in QUEUE_MODELS, en expliciete .js-extensies op vier type-only imports in lib/realtime-payload.ts (een module zonder consument buiten shared's eigen test). Geen schemawijziging — prisma/schema.prisma in shared is in dit hele bereik ongewijzigd.

🤖 Generated with Claude Code

Blast radius hier: QUEUE_MODELS loopt via lib/queue/types.ts door als MODELS en wordt gebruikt als dropdown-opties in de Messages-view (filter + verstuurformulier). Die lijst wordt door geen test gepind, dus een nieuw model verschijnt daar vanzelf.

Let op voor later: postinstall draait hier gen-schema.sh … || true, dus een mislukte generatie wordt stil geslikt. Positief geverifieerd na de bump: 39 modellen, 32 enums.

Bumpt de `scrum4me-shared`-submodule van `69cb33f` naar `9812ae5`, de commit die het queue-vocabulaire uitbreidt. Eén gitlink-wijziging, verder niets. Tests, typecheck en lint zijn vóór en ná identiek — geen enkele testtelling beweegt. **Wat de bump functioneel meebrengt is precies twee dingen:** `kimi` in `QUEUE_MODELS`, en expliciete `.js`-extensies op vier type-only imports in `lib/realtime-payload.ts` (een module zonder consument buiten shared's eigen test). Geen schemawijziging — `prisma/schema.prisma` in shared is in dit hele bereik ongewijzigd. 🤖 Generated with [Claude Code](https://claude.com/claude-code) **Blast radius hier:** `QUEUE_MODELS` loopt via `lib/queue/types.ts` door als `MODELS` en wordt gebruikt als dropdown-opties in de Messages-view (filter + verstuurformulier). Die lijst wordt door geen test gepind, dus een nieuw model verschijnt daar vanzelf. Let op voor later: `postinstall` draait hier `gen-schema.sh … || true`, dus een mislukte generatie wordt stil geslikt. Positief geverifieerd na de bump: 39 modellen, 32 enums.

janpeter added 1 commit 2026-07-27 00:03:59 +02:00

chore(shared): bump naar 9812ae5 voor kimi in QUEUE_MODELS ... 6a9dd4e95e

QUEUE_MODELS wordt ['claude', 'codex', 'kimi', 'jp']. Workers re-exporteert
die lijst als MODELS via lib/queue/types.ts; de Messages-view gebruikt hem
alleen als dropdown-opties (filter + verstuur-formulier), dus kimi verschijnt
daar zonder verdere aanpassing.

Verder in 69cb33f..9812ae5: expliciete .js-extensies op de type-imports in
lib/realtime-payload.ts. Die module heeft in deze repo geen consumenten.

prisma/schema.prisma verandert niet — de shared canonical is in dit bereik
ongemoeid; postinstall genereert dezelfde 39 modellen / 32 enums.

Verificatie: lint 0 errors, typecheck schoon, 778 tests groen (113 files,
10 + 1 skipped) — identiek aan de nulmeting vóór de bump.

s4m-codex-reviewer approved these changes 2026-07-27 00:06:49 +02:00

s4m-codex-reviewer left a comment

Copy link

Verdict: APPROVED

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

Findings

Geen blokkerende of kleinere findings.

Review-notities

De PR-diff in scrum4me-workers is uitsluitend een bump van vendor/scrum4me-shared van 69cb33f naar 9812ae5. De submodule-range is inhoudelijk gecontroleerd: lib/queue-identity.ts:44 voegt kimi toe aan QUEUE_MODELS, __tests__/queue-identity.test.ts pint expliciet dat isQueueModel('kimi') waar is, en de meegekomen lib/realtime-payload.ts wijziging corrigeert type-only relatieve imports naar .js-extensies voor ESM-output.

Dit past bij de productdocs: scrum4me-workers consumeert het gedeelde vocabulaire via de shared submodule, en voor queue-modellen bestaat bewust geen DB-CHECK/migratiepad in workers. Geen docs- of testgat gezien dat deze bump zou moeten blokkeren.

# Verdict: APPROVED geen gekoppeld plan gevonden — beoordeeld op codekwaliteit + product-standaarden. ## Findings Geen blokkerende of kleinere findings. ## Review-notities De PR-diff in `scrum4me-workers` is uitsluitend een bump van `vendor/scrum4me-shared` van `69cb33f` naar `9812ae5`. De submodule-range is inhoudelijk gecontroleerd: `lib/queue-identity.ts:44` voegt `kimi` toe aan `QUEUE_MODELS`, `__tests__/queue-identity.test.ts` pint expliciet dat `isQueueModel('kimi')` waar is, en de meegekomen `lib/realtime-payload.ts` wijziging corrigeert type-only relatieve imports naar `.js`-extensies voor ESM-output. Dit past bij de productdocs: `scrum4me-workers` consumeert het gedeelde vocabulaire via de shared submodule, en voor queue-modellen bestaat bewust geen DB-CHECK/migratiepad in workers. Geen docs- of testgat gezien dat deze bump zou moeten blokkeren.

janpeter merged commit bd27f2ed5f into main 2026-07-27 00:27:54 +02:00

janpeter referenced this pull request from a commit 2026-07-27 00:27:55 +02:00

Merge pull request 'chore(shared): bump scrum4me-shared naar 9812ae5 (kimi in QUEUE_MODELS)' (#74) from feat/shared-kimi-bump into main

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

Subscribe

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!74

No description provided.