Repository navigation
Conversation
The autonomous WorkerSpawner path bypassed the existing stream parser and task cost ledger, so production runs emitted no cost data. Parse Claude stream-json in the live worker, retain readable diagnostics and PR detection, and persist one provider-estimated cost per classified ticket. Co-Authored-By: OpenAI Codex <noreply@openai.com>
Bound retained stream state, preserve distinct terminal output, record positive provider estimates without token metadata, and keep seeded autonomous tickets classified for cost reporting. Co-Authored-By: OpenAI Codex <noreply@openai.com>
Cap partial stream lines and diagnostic tails, validate numeric cost inputs before persistence, and only accept PR URLs from parsed assistant text. Document that cost reporting remains observational. Co-Authored-By: OpenAI Codex <noreply@openai.com>
Qodo reviews are paused for this user.Troubleshooting steps vary by plan Learn more → On a Teams plan? Using GitHub Enterprise Server, GitLab Self-Managed, or Bitbucket Data Center? |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
stream-jsonoutput in the liveWorkerSpawnerpath, route it through the shared parser, and retain bounded human-readable assistant/diagnostic tails while only detecting PR URLs in parsed assistant text.total_cost_usdis treated as a client-side estimate and used once when available; Relay's token-price table is only the fallback, so the two sources are never added together.Closes #243
Test plan
pnpm test(1,248 passed, 29 skipped)pnpm typecheckpnpm buildpnpm format:checkcargo check --workspace --lockedcargo test -p harness-data(91 passed)cd gui && pnpm buildHARNESS_LIVE=1 ./node_modules/.bin/vitest run test/integration/live-worker-cost.test.ts(real autonomous handler -> Claude -> cost ledger ->rly cost)Size check
Diff size:
1004lines added /95deleted.This exceeds 800 lines because 724 additions are focused regression and opt-in integration tests. The production/documentation change is 280 additions and 66 deletions across the existing worker, parser, cost-ledger, ticket-classification, and mirrored type paths; it does not introduce a parallel parsing or orchestration framework.
Breaking changes?
No. The new ticket
taskTypefield is optional and the Rust mirror uses a default for existing serialized tickets. Existing non-JSON diagnostics and failure/cleanup behavior remain supported.AI-assisted?
Yes. OpenAI Codex provided substantial assistance with investigation, implementation, tests, and review. Every commit includes the required
Co-Authored-By: OpenAI Codex <noreply@openai.com>footer. The final diff was reviewed and validated at commit666f4826b31dd1ea675475ff0ac8032609c4c7fd.