test(dispatch): Claude-probe in de model-image-integratietest (M41 5b) #102

Merged

janpeter merged 1 commit from feat/m41-claude-probe-integration into master

2026-10-03 12:42:54 +02:00

janpeter commented

2026-10-03 12:36:13 +02:00

Owner

Wat

npm run test:dispatch-model-image draait de egress-keten nu voor beide runtimes: eerst model-codex-probe, daarna model-claude-probe (M41 5b). De Claude-run gaat zoals de broker van het Claude-slot hem start: runtime: 'CLAUDE' in broker.create, credential-bron claude_oauth_token.

Nieuwe checks per runtime

runtimeEnvOnly — exacte child-env (Claude: OAuth-token, CLAUDE_CODE_DISABLE_NONESSENTIAL_TRAFFIC, DISABLE_AUTOUPDATER plus de gedeelde allowlist).
childRuntimeMarked — DISPATCH_CHILD_RUNTIME=CLAUDE alleen op het Claude-kind; het Codex-kind blijft zonder.
modelAndEffortInArgv — childModel {probe-model, high} komt aan in de CLI-argv (--model/--effort, resp. --model/-c model_reasoning_effort=).
cwd — het Claude-kind start in /work (bevinding uit de review van #101).

De stubs melden daarvoor hun cwd; de Claude-stub laat de prompt (waarde van -p) weg uit argv_tail. docs/dispatch-runtime-operator.md beschrijft de uitgebreide test.

Verificatie

npm run test:dispatch-model-image (lokaal Docker): CODEX 13/13 en CLAUDE 13/13 checks true, beide passed, alles opgeruimd.
RED: met cwd:undefined en zonder effort in de Claude-adapter faalden precies cwd en modelAndEffortInArgv (MODEL_IMAGE_EGRESS_CHECK_FAILED); daarna hersteld.
vitest run: 1068 groen; 2 bestaande falers in transcript-retention (BSD-chmod op macOS), los van deze diff.
Geen runtimecode gewijzigd; de test draait niet in CI (lokaal Docker).

🤖 Generated with Claude Code

## Wat `npm run test:dispatch-model-image` draait de egress-keten nu voor **beide** runtimes: eerst `model-codex-probe`, daarna `model-claude-probe` (M41 5b). De Claude-run gaat zoals de broker van het Claude-slot hem start: `runtime: 'CLAUDE'` in `broker.create`, credential-bron `claude_oauth_token`. ## Nieuwe checks per runtime - `runtimeEnvOnly` — exacte child-env (Claude: OAuth-token, `CLAUDE_CODE_DISABLE_NONESSENTIAL_TRAFFIC`, `DISABLE_AUTOUPDATER` plus de gedeelde allowlist). - `childRuntimeMarked` — `DISPATCH_CHILD_RUNTIME=CLAUDE` alleen op het Claude-kind; het Codex-kind blijft zonder. - `modelAndEffortInArgv` — `childModel {probe-model, high}` komt aan in de CLI-argv (`--model`/`--effort`, resp. `--model`/`-c model_reasoning_effort=`). - `cwd` — het Claude-kind start in `/work` (bevinding uit de review van #101). De stubs melden daarvoor hun `cwd`; de Claude-stub laat de prompt (waarde van `-p`) weg uit `argv_tail`. `docs/dispatch-runtime-operator.md` beschrijft de uitgebreide test. ## Verificatie - `npm run test:dispatch-model-image` (lokaal Docker): CODEX 13/13 en CLAUDE 13/13 checks `true`, beide `passed`, alles opgeruimd. - RED: met `cwd:undefined` en zonder effort in de Claude-adapter faalden precies `cwd` en `modelAndEffortInArgv` (`MODEL_IMAGE_EGRESS_CHECK_FAILED`); daarna hersteld. - `vitest run`: 1068 groen; 2 bestaande falers in `transcript-retention` (BSD-`chmod` op macOS), los van deze diff. - Geen runtimecode gewijzigd; de test draait niet in CI (lokaal Docker). 🤖 Generated with [Claude Code](https://claude.com/claude-code)

janpeter added 1 commit

2026-10-03 12:36:13 +02:00

test(dispatch): Claude-probe in de model-image-integratietest (M41 5b)

CI / Compose config (pull_request) Successful in 10s

Details

CI / Build-arg coverage (pull_request) Successful in 9s

Details

CI / Docker build (pull_request) Successful in 1m54s

Details

fc16464191

npm run test:dispatch-model-image draait de keten nu voor beide runtimes:
eerst model-codex-probe, dan model-claude-probe met runtime CLAUDE en
claude_oauth_token als credential. Per runtime controleert de test ook de
exacte child-env, DISPATCH_CHILD_RUNTIME alleen op het Claude-kind, model en
effort in de argv van de CLI, en cwd /work voor Claude. De stubs melden
daarvoor hun cwd; de Claude-stub laat de prompt weg uit argv_tail.

RED gecontroleerd: zonder cwd en effort in de Claude-adapter falen cwd en
modelAndEffortInArgv.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>

s4m-codex-reviewer reviewed

2026-10-03 12:42:46 +02:00

s4m-codex-reviewer left a comment

COMMENT

Geen blokkerende codefinding gevonden in de diff op commit fc16464191585f4811771fe55bc6f9cc13cf2d0f. De runtimeconfiguratie, credentialselectie, env-allowlist, model/effort-asserties en Claude-werkmap sluiten aan op de bestaande broker en entrypoint. De operator-documentatie beschrijft de uitbreiding correct. De geraadpleegde productdocumentatie (architectuuroverzicht en agent-guidance) levert geen strijdigheid op.

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

Findings

INFO — tests/dispatch-model-image.integration.sh:10 — De nieuwe Claude-image-integratierun kon hier niet worden uitgevoerd omdat Docker ontbreekt. Vitest kon evenmin starten: de native binding faalt met failed to map segment from shared object. Het gedrag van beide echte probe-images blijft daardoor onafhankelijk onbevestigd. Dit is een verificatiebeperking, geen aangetoonde coderegressie; conform de safe-default is het verdict COMMENT.

Verificatie

Exacte head-commit uitgecheckt; vijf gewijzigde bestanden komen overeen met de aangeleverde diff.
Dispatch-typecheck geslaagd na initialisatie van de gepinde shared-submodule (node node_modules/typescript/bin/tsc -p tsconfig.dispatch.json).
git diff --check, bash -n voor het integratiescript en node --check voor beide CLI-stubs geslaagd.
Volledige Docker-integratierun en Vitest-suite niet bevestigd wegens bovenstaande omgevingbeperkingen.

# COMMENT Geen blokkerende codefinding gevonden in de diff op commit `fc16464191585f4811771fe55bc6f9cc13cf2d0f`. De runtimeconfiguratie, credentialselectie, env-allowlist, model/effort-asserties en Claude-werkmap sluiten aan op de bestaande broker en entrypoint. De operator-documentatie beschrijft de uitbreiding correct. De geraadpleegde productdocumentatie (architectuuroverzicht en agent-guidance) levert geen strijdigheid op. geen gekoppeld plan gevonden — beoordeeld op codekwaliteit + product-standaarden. ## Findings - **INFO — tests/dispatch-model-image.integration.sh:10** — De nieuwe Claude-image-integratierun kon hier niet worden uitgevoerd omdat Docker ontbreekt. Vitest kon evenmin starten: de native binding faalt met `failed to map segment from shared object`. Het gedrag van beide echte probe-images blijft daardoor onafhankelijk onbevestigd. Dit is een verificatiebeperking, geen aangetoonde coderegressie; conform de safe-default is het verdict COMMENT. ## Verificatie - Exacte head-commit uitgecheckt; vijf gewijzigde bestanden komen overeen met de aangeleverde diff. - Dispatch-typecheck geslaagd na initialisatie van de gepinde shared-submodule (`node node_modules/typescript/bin/tsc -p tsconfig.dispatch.json`). - `git diff --check`, `bash -n` voor het integratiescript en `node --check` voor beide CLI-stubs geslaagd. - Volledige Docker-integratierun en Vitest-suite niet bevestigd wegens bovenstaande omgevingbeperkingen.

janpeter merged commit 6f163765c3 into master

2026-10-03 12:42:54 +02:00

janpeter referenced this pull request from a commit

2026-10-03 12:42:56 +02:00

Merge pull request 'test(dispatch): Claude-probe in de model-image-integratietest (M41 5b)' (#102)

janpeter referenced this pull request from a commit

2026-10-04 07:32:53 +02:00

docs(audit): model-claude proxy proof in README (PR #102)