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

ST-054: actieve Codex-registrymodellen kiezen en veilig opslaan #112

Merged
janpeter merged 4 commits from codex/modelregistry-20261001 into main 2026-10-01 03:10:22 +02:00
Conversation 2 Commits 4 Files changed 19 +880 -43
janpeter commented 2026-10-01 02:27:21 +02:00
Owner
Copy link

GPT-6 Astra stond actief in de registry maar ontbrak in de editor. De pagina geeft nu actieve CODEX-modellen door aan het formulier en de server valideert de lijst opnieuw bij iedere save. Opgeslagen ontbrekende/inactieve waarden worden niet stil vervangen. Bestaande admin- en managed-jobguards blijven behouden.

Validatie: 25 regressieassertions eerst rood; 103 gerichte tests en volledige verify met 1328 tests groen (18 bestaande skips; één bestaande lintwaarschuwing). Volledige lokale Docker-build geslaagd. Echte browser/serveractie/DB-readback op geïsoleerde PostgreSQL17: Astra opslaan en herladen, alleen Sprint/Codex en updated_at gewijzigd over 16 rijen. Bewijs en screenshot: docs/superpowers/reports/2026-10-01-modelregistry-workers-verificatie.md.

Onafhankelijke mac:claude-review: GO. De twee daaropvolgende auto-review MINORs zijn hersteld: afzonderlijke melding voor lege modelwaarden en consistent bewijs dat een lege modelstring de --model-vlag weglaat. Formuliertests 23/23 en volledige verify 1328 passed / 18 bestaande skips. Rapport is meegeleverd. ST-054/T-167/T-169. T-171 blijft open: afzonderlijke merge/deployautorisatie, SCRUM4ME_ENABLE_CODEX_WORKERS-startvoorwaarde en positieve geïsoleerde productiecanary. Runtime vóór UI uitrollen; geen productieconfig gewijzigd.

Afhankelijkheden:

GPT-6 Astra stond actief in de registry maar ontbrak in de editor. De pagina geeft nu actieve CODEX-modellen door aan het formulier en de server valideert de lijst opnieuw bij iedere save. Opgeslagen ontbrekende/inactieve waarden worden niet stil vervangen. Bestaande admin- en managed-jobguards blijven behouden. Validatie: 25 regressieassertions eerst rood; 103 gerichte tests en volledige verify met 1328 tests groen (18 bestaande skips; één bestaande lintwaarschuwing). Volledige lokale Docker-build geslaagd. Echte browser/serveractie/DB-readback op geïsoleerde PostgreSQL17: Astra opslaan en herladen, alleen Sprint/Codex en updated_at gewijzigd over 16 rijen. Bewijs en screenshot: docs/superpowers/reports/2026-10-01-modelregistry-workers-verificatie.md. Onafhankelijke mac:claude-review: GO. De twee daaropvolgende auto-review MINORs zijn hersteld: afzonderlijke melding voor lege modelwaarden en consistent bewijs dat een lege modelstring de --model-vlag weglaat. Formuliertests 23/23 en volledige verify 1328 passed / 18 bestaande skips. Rapport is meegeleverd. ST-054/T-167/T-169. T-171 blijft open: afzonderlijke merge/deployautorisatie, SCRUM4ME_ENABLE_CODEX_WORKERS-startvoorwaarde en positieve geïsoleerde productiecanary. Runtime vóór UI uitrollen; geen productieconfig gewijzigd. Afhankelijkheden: - https://git.jp-visser.nl/janpeter/scrum4me-shared/pulls/67 - https://git.jp-visser.nl/janpeter/scrum4me-mcp/pulls/174
docs(ST-054): record independent implementation review GO
All checks were successful
CI / Verify (pull_request) Successful in 2m5s
Details
80919e3b7c
janpeter changed title from WIP: ST-054: actieve Codex-registrymodellen kiezen en veilig opslaan to ST-054: actieve Codex-registrymodellen kiezen en veilig opslaan 2026-10-01 02:51:53 +02:00
s4m-codex-reviewer reviewed 2026-10-01 02:58:12 +02:00
s4m-codex-reviewer left a comment
Copy link

Verdict: COMMENT

Findings

  • MINOR — components/settings/job-kind-config-form.tsx:62 Een lege of uitsluitend-witruimte opgeslagen codex_model krijgt de tekst “Het opgeslagen Codex-model is niet beschikbaar”. Er is dan juist geen opgeslagen model; dit wijkt af van de T3-meldingseis en maakt hersteldiagnose verwarrend. Splits de foutmelding: vraag bij een lege waarde om een actief model te kiezen, en gebruik de huidige melding alleen voor een niet-lege, niet-beschikbare id.
  • MINOR — docs/superpowers/reviews/2026-10-01-codex-modelregistry-implementatiereview.md:8 / :63 Het bewijsdocument is intern tegenstrijdig over een lege requested_model: regel 8 stelt dat de args-builder de --model-vlag weglaat, terwijl regel 63 stelt dat --model '' wordt doorgegeven. Corrigeer één van beide naar het geteste gedrag en verwijs naar het betreffende test- of logbewijs.

De registrygestuurde servervalidatie en de shared-resolver-bump zijn verder planconform beoordeeld. De diff- en submodulecontrole zijn schoon; gerichte tests konden lokaal niet starten doordat de aangewezen tijdelijke mount noexec is.

## Verdict: COMMENT ### Findings - **MINOR — `components/settings/job-kind-config-form.tsx:62`** Een lege of uitsluitend-witruimte opgeslagen `codex_model` krijgt de tekst “Het opgeslagen Codex-model is niet beschikbaar”. Er is dan juist geen opgeslagen model; dit wijkt af van de T3-meldingseis en maakt hersteldiagnose verwarrend. Splits de foutmelding: vraag bij een lege waarde om een actief model te kiezen, en gebruik de huidige melding alleen voor een niet-lege, niet-beschikbare id. - **MINOR — `docs/superpowers/reviews/2026-10-01-codex-modelregistry-implementatiereview.md:8` / `:63`** Het bewijsdocument is intern tegenstrijdig over een lege `requested_model`: regel 8 stelt dat de args-builder de `--model`-vlag weglaat, terwijl regel 63 stelt dat `--model ''` wordt doorgegeven. Corrigeer één van beide naar het geteste gedrag en verwijs naar het betreffende test- of logbewijs. De registrygestuurde servervalidatie en de shared-resolver-bump zijn verder planconform beoordeeld. De diff- en submodulecontrole zijn schoon; gerichte tests konden lokaal niet starten doordat de aangewezen tijdelijke mount `noexec` is.
fix(ST-054): clarify empty model feedback and review evidence
All checks were successful
CI / Verify (pull_request) Successful in 2m4s
Details
d0e94b3ab5
s4m-codex-reviewer approved these changes 2026-10-01 03:07:25 +02:00
s4m-codex-reviewer left a comment
Copy link

Verdict: APPROVED

Findings

  • Geen findings met blocker-, error- of comment-severity.

De gekoppelde planlaag voor registrygestuurde Codex-modelkeuze is conform uitgevoerd: expliciete actieve model-IDs blijven behouden, de editor gebruikt runtimegescheiden registry-opties zonder stille fallback, en de server valideert de actuele CODEX-registry vóór de upsert. De relevante regressietests zijn aanwezig; lokale uitvoering in deze container werd beperkt door de noexec-mount op /tmp (waar de reviewcheckout staat), niet door een testfout.

# Verdict: APPROVED ## Findings - Geen findings met blocker-, error- of comment-severity. De gekoppelde planlaag voor registrygestuurde Codex-modelkeuze is conform uitgevoerd: expliciete actieve model-IDs blijven behouden, de editor gebruikt runtimegescheiden registry-opties zonder stille fallback, en de server valideert de actuele CODEX-registry vóór de upsert. De relevante regressietests zijn aanwezig; lokale uitvoering in deze container werd beperkt door de `noexec`-mount op `/tmp` (waar de reviewcheckout staat), niet door een testfout.
Sign in to join this conversation.
Reviewers
No reviewers
s4m-codex-reviewer
Labels
Clear labels   
severity/s4
No labels
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-workers!112
No description provided.