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

ST-054: behoud registrymodellen in context en managed snapshots #174

Merged
janpeter merged 3 commits from codex/modelregistry-20261001 into main 2026-10-01 02:58:43 +02:00
Conversation 1 Commits 3 Files changed 10 +188 -5
janpeter commented 2026-10-01 02:27:21 +02:00
Owner
Copy link

De shared resolver verving nieuwe Codex-registrymodellen door GPT-5.5. Deze wijziging pint de herstelde resolver en bewijst de doorgifte door getFullJobContext, managed dispatch/enqueue en de CLI-argumenten. Snapshotvoorrang en managed guards blijven intact.

Validatie: regressies eerst rood; typecheck en 2015 unit-/contracttests groen (69 bestaande skips), alle 268 PostgreSQL-dispatchtests groen. Een aparte echte runner met weigerende stub startte precies eenmaal met Astra, exit 2, rollback naar QUEUED, geen modelwissel. Bewijs: docs/reports/2026-10-01-modelregistry-verification.md.

Onafhankelijke mac:claude-review: GO, geen BLOCKER/MAJOR. Onderdeel van ST-054/T-170. Geen merge of deployment geautoriseerd. Alle runtime-instanties moeten bijgewerkt zijn vóór UI-activatie.

Afhankelijkheden:

De shared resolver verving nieuwe Codex-registrymodellen door GPT-5.5. Deze wijziging pint de herstelde resolver en bewijst de doorgifte door getFullJobContext, managed dispatch/enqueue en de CLI-argumenten. Snapshotvoorrang en managed guards blijven intact. Validatie: regressies eerst rood; typecheck en 2015 unit-/contracttests groen (69 bestaande skips), alle 268 PostgreSQL-dispatchtests groen. Een aparte echte runner met weigerende stub startte precies eenmaal met Astra, exit 2, rollback naar QUEUED, geen modelwissel. Bewijs: docs/reports/2026-10-01-modelregistry-verification.md. Onafhankelijke mac:claude-review: GO, geen BLOCKER/MAJOR. Onderdeel van ST-054/T-170. Geen merge of deployment geautoriseerd. Alle runtime-instanties moeten bijgewerkt zijn vóór UI-activatie. Afhankelijkheden: - https://git.jp-visser.nl/janpeter/scrum4me-shared/pulls/67
docs(ST-054): preserve sanitized runner proof logs
Some checks failed
CI / Final merge attestation and immutable publication (pull_request) Has been skipped
Details
CI / PR candidate (never published) (pull_request) Failing after 5m7s
Details
6a4d53e512
test(ST-054): align current consumer pin with registry resolver
All checks were successful
CI / Final merge attestation and immutable publication (pull_request) Has been skipped
Details
CI / PR candidate (never published) (pull_request) Successful in 8m19s
Details
2a4088d718
janpeter changed title from WIP: ST-054: behoud registrymodellen in context en managed snapshots to ST-054: behoud registrymodellen in context en managed snapshots 2026-10-01 02:51:52 +02:00
s4m-codex-reviewer approved these changes 2026-10-01 02:57:11 +02:00
s4m-codex-reviewer left a comment
Copy link

Verdict: APPROVED

  • Geen findings met blocker- of error-severity. De shared resolver behoudt expliciet geconfigureerde Codex-model-ID's, terwijl lege waarden veilig terugvallen op de default; de managed-configuratievanger blijft gehandhaafd.
  • Testdekking: de PR voegt gerichte resolver-, context-, CLI-argument- en PostgreSQL-integratietests toe. De geclaimde volledige suite kon lokaal niet opnieuw draaien doordat deze reviewomgeving /tmp met noexec mount en de testtool native executables laadt.
  • Docs: het opgenomen ketenbewijs documenteert scope, runnerbewijs en resterende CI-/canarygrenzen.

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

## Verdict: APPROVED - Geen findings met blocker- of error-severity. De shared resolver behoudt expliciet geconfigureerde Codex-model-ID's, terwijl lege waarden veilig terugvallen op de default; de managed-configuratievanger blijft gehandhaafd. - Testdekking: de PR voegt gerichte resolver-, context-, CLI-argument- en PostgreSQL-integratietests toe. De geclaimde volledige suite kon lokaal niet opnieuw draaien doordat deze reviewomgeving `/tmp` met `noexec` mount en de testtool native executables laadt. - Docs: het opgenomen ketenbewijs documenteert scope, runnerbewijs en resterende CI-/canarygrenzen. Geen gekoppeld plan gevonden — beoordeeld op codekwaliteit + product-standaarden.
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!174
No description provided.