janpeter/Scrum4Me
0
0
Fork 0
Code Issues 3 Pull requests 1 Projects Releases Packages Wiki Activity Actions

feat(jobs): toon invoer incl. cache write in het gebruikspaneel #302

Merged
janpeter merged 1 commit from claude/usage-input-incl-cache-write into main 2026-10-05 01:20:39 +02:00
Conversation 0 Commits 1 Files changed 2 +26
janpeter commented 2026-10-05 00:40:57 +02:00
Owner
Copy link

Het gebruikspaneel van een job krijgt een regel Invoer incl. cache write (= invoer + cache write).

Waarom: Claude boekt de eerste keer ingelezen context (systeemprompt, tools, reviewdocumenten) als cache write; Codex/OpenAI kent geen cache-write en telt die als invoer. Daardoor lijkt "Tokens invoer" bij Claude-jobs (bv. Queue-review) veel lager dan bij Codex. De nieuwe regel maakt beide runtimes vergelijkbaar. Ontbreken beide waarden, dan toont de regel —.

Verificatie

  • vitest run __tests__/components/jobs/job-usage-pane.test.tsx: 4/4 groen (2 nieuwe tests)
  • eslint op de gewijzigde bestanden: schoon
  • tsc --noEmit: 180 fouten zowel vóór als na deze wijziging, geen in de gewijzigde bestanden

🤖 Generated with Claude Code

Het gebruikspaneel van een job krijgt een regel **Invoer incl. cache write** (= invoer + cache write). **Waarom:** Claude boekt de eerste keer ingelezen context (systeemprompt, tools, reviewdocumenten) als *cache write*; Codex/OpenAI kent geen cache-write en telt die als *invoer*. Daardoor lijkt "Tokens invoer" bij Claude-jobs (bv. Queue-review) veel lager dan bij Codex. De nieuwe regel maakt beide runtimes vergelijkbaar. Ontbreken beide waarden, dan toont de regel `—`. **Verificatie** - `vitest run __tests__/components/jobs/job-usage-pane.test.tsx`: 4/4 groen (2 nieuwe tests) - eslint op de gewijzigde bestanden: schoon - `tsc --noEmit`: 180 fouten zowel vóór als na deze wijziging, geen in de gewijzigde bestanden 🤖 Generated with [Claude Code](https://claude.com/claude-code)
feat(jobs): toon invoer incl. cache write in het gebruikspaneel
All checks were successful
CI / Lint, Typecheck, Test & Build (pull_request) Successful in 9m31s
Details
CI / DB Access Policy Gate (pull_request) Successful in 3m14s
Details
CI / Queue Dispatch Guard Gate (pull_request) Successful in 3m32s
Details
CI / DB Access Production Sentinel (pull_request) Successful in 2m4s
Details
CI / Deploy Manual (workflow_dispatch) (pull_request) Has been skipped
Details
CI / Detect deploy-relevant changes (pull_request) Has been skipped
Details
CI / Deploy Preview (PR) (pull_request) Has been skipped
Details
CI / Deploy Production (main) (pull_request) Has been skipped
Details
8abf78eebc 
Claude boekt de eerste keer ingelezen context als cache write, Codex als
invoer. Daardoor lijkt de Claude-invoer veel lager. De nieuwe regel telt
invoer en cache write op, zodat beide runtimes vergelijkbaar zijn.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
s4m-codex-reviewer reviewed 2026-10-05 00:47:25 +02:00
s4m-codex-reviewer left a comment
Copy link

COMMENT

Geen blokkerende codefindings gevonden. De som verwerkt ontbrekende waarden en nul correct, hergebruikt formatNumber en FieldRow, en respecteert de bestaande clientgrens en stylingconventies. De codecomment legt het doel uit; voor deze lokale weergavewijziging is geen nieuwe architectuurdocumentatie nodig.

Findings:

  • INFO — __tests__/components/jobs/job-usage-pane.test.tsx:74: de componenttests konden hier niet starten door uitvoerbeperkingen op /tmp (Permission denied / failed to map segment from shared object). Geen aangetoonde codefout, maar de gemelde 4/4 groene tests zijn niet onafhankelijk bevestigd. Daarom COMMENT volgens de safe-default.

Verificatie: PR en head-SHA opgelost en gecontroleerd; volledige gewijzigde component, tests en tokenmapping gelezen; ESLint op beide gewijzigde bestanden geslaagd. Vijf losse berekeningsgevallen (som, beide ontbrekend, één ontbrekend en nul) geslaagd; dit vervangt geen componenttest. Geen volledige build/typecheck uitgevoerd; de PR meldt 180 bestaande typefouten vóór en na de wijziging.

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

# COMMENT Geen blokkerende codefindings gevonden. De som verwerkt ontbrekende waarden en nul correct, hergebruikt `formatNumber` en `FieldRow`, en respecteert de bestaande clientgrens en stylingconventies. De codecomment legt het doel uit; voor deze lokale weergavewijziging is geen nieuwe architectuurdocumentatie nodig. Findings: - **INFO** — `__tests__/components/jobs/job-usage-pane.test.tsx:74`: de componenttests konden hier niet starten door uitvoerbeperkingen op `/tmp` (`Permission denied` / `failed to map segment from shared object`). Geen aangetoonde codefout, maar de gemelde 4/4 groene tests zijn niet onafhankelijk bevestigd. Daarom `COMMENT` volgens de safe-default. Verificatie: PR en head-SHA opgelost en gecontroleerd; volledige gewijzigde component, tests en tokenmapping gelezen; ESLint op beide gewijzigde bestanden geslaagd. Vijf losse berekeningsgevallen (som, beide ontbrekend, één ontbrekend en nul) geslaagd; dit vervangt geen componenttest. Geen volledige build/typecheck uitgevoerd; de PR meldt 180 bestaande typefouten vóór en na de wijziging. geen gekoppeld plan gevonden — beoordeeld op codekwaliteit + product-standaarden.
Sign in to join this conversation.
Reviewers
No reviewers
Labels
Clear labels   
severity/s3

   
severity/s4
No labels
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!302
No description provided.