Skip to content

docs: three generation endpoints, and the streaming behaviour as shipped - #71

Merged
askalf merged 1 commit into
mainfrom
docs/responses-scope
Sep 22, 2026
Merged

askalf merged 1 commit into
mainfrom
docs/responses-scope

Conversation

@askalf

@askalf askalf commented Sep 22, 2026

Copy link
Copy Markdown
Owner

Follow-up to #70. The 'What it does not do' scope line still said two generation endpoints, and the changelog entry described the design as first written rather than what four review rounds produced: per-part hold-back buffers keyed by item_id/output_index/content_index, output_item.done flushing only its own item, every end-of-stream path flushing what is held, and prior assistant turns (output_text/refusal) redacted on the way back up.

The scope line still named two endpoints. The changelog described the
Responses streaming design before review: per-part buffers, the flush
scope of output_item.done, the no-terminal-frame path and the prior
assistant turns are what actually shipped.
@github-actions github-actions Bot added documentation Improvements or additions to documentation size/XS Under 10 hand-written lines labels Sep 22, 2026
@askalf
askalf enabled auto-merge (squash) September 22, 2026 11:50

@sprayberry-redline sprayberry-redline left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Deterministic approval — no LLM reviewed this diff.

Classified low-risk: docs-only (2 file(s)).

This repository requires an approving review before auto-merge can proceed, so this
approval exists to satisfy that gate without spending a full Code Reviewer execution
on a diff whose risk is provable from the file list alone. This approval satisfies ONLY
the review requirement — the merge still separately requires green CI, which this sweep
does not and cannot short-circuit.

Anything outside the docs allowlist, any major dependency bump, any non-Dependabot
dependency change, and any unknown diff keep the full reviewer path.

@askalf
askalf merged commit c19bddd into main Sep 22, 2026
5 checks passed
@askalf
askalf deleted the docs/responses-scope branch September 22, 2026 11:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

documentation Improvements or additions to documentation size/XS Under 10 hand-written lines

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants