Skip to content

feat(usage): record streaming /chat/completions (Task #8)#24

Open
lubobill1990 wants to merge 1 commit into
feat/07-wire-nonstreaming-chatfrom
feat/08-wire-streaming-chat
Open

feat(usage): record streaming /chat/completions (Task #8)#24
lubobill1990 wants to merge 1 commit into
feat/07-wire-nonstreaming-chatfrom
feat/08-wire-streaming-chat

Conversation

@lubobill1990
Copy link
Copy Markdown
Collaborator

Forces include_usage on streaming requests; records usage on stream close (ok/aborted/error).

Refs #8

🤖 Generated with Claude Code

- create-chat-completions.ts forces stream_options.include_usage=true on streaming requests so the upstream emits a final usage frame.
- Streaming branch uses an OpenAI accumulator: every chunk is parsed, fed, then forwarded untouched. After the stream closes, recordUsage is called with status='ok' (or 'aborted' if c.req.raw.signal.aborted, or 'error' on iteration error). Missing usage frame logs a warn and records zero tokens with status='error'.

Refs #8

Co-Authored-By: Claude Opus 4 <noreply@anthropic.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant