Repository navigation
Conversation
This was referenced Oct 5, 2026
Pouyanpi
force-pushed
the
pouyanpi/transparent-proxy-streaming-kernel-1
branch
from
October 6, 2026 11:05
4ec5c7f to
f4a6606
Compare
Pouyanpi
force-pushed
the
pouyanpi/openai-chat-streaming-contract-2
branch
from
October 6, 2026 11:05
73f46d5 to
625e4cb
Compare
Pouyanpi
marked this pull request as ready for review
October 6, 2026 11:06
Pouyanpi
force-pushed
the
pouyanpi/openai-chat-streaming-contract-2
branch
from
October 6, 2026 15:22
625e4cb to
80703c1
Compare
Contributor
|
Signed-off-by: Pouyanpi <13303554+Pouyanpi@users.noreply.github.com>
Reject unreviewed choice fields before releasing the original event bytes. Regenerate the choice projection and align the maintained choice and delta closure declarations with the inspected stream profile. Track the actual DONE sentinel in per-stream hooks, require it for successful completion, and reject payloads after termination. Preserve data-less keepalives and provider-error endings with or without a final sentinel. Cover unchecked sibling content, missing and duplicate completion, later payloads, pending-window withholding, raw preservation, and isolated hook state. Signed-off-by: Pouyanpi <13303554+Pouyanpi@users.noreply.github.com>
Pouyanpi
force-pushed
the
pouyanpi/openai-chat-streaming-contract-2
branch
from
October 7, 2026 09:00
c312770 to
d5510f4
Compare
Pouyanpi
force-pushed
the
pouyanpi/transparent-proxy-streaming-kernel-1
branch
from
October 7, 2026 09:00
7521a53 to
9052700
Compare
This branch has not been deployed
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
Implements the stream surface already declared by the public OpenAI Chat guard contract, with staged projection/classifier modules and handwritten native error framing.
What Changed
Review Notes
Review the declared event inventory, unsupported content channels, strict JSON, and native SSE error framing. Endpoint/HTTP integration lands in the final PR.
AI Assistance
Checklist
Stack Position
Part 2 of 4.
Stack Context
Adds OpenAI Chat streaming on top of #2414 in four independently tested slices: provider-neutral execution, OpenAI stream modules, injected HTTP lifecycle handling, and integration with the existing buffered request pipeline. Concrete outbound HTTP, deployment configuration, and replacement execution remain separate work.
Please review each PR against its parent branch, not directly against the root base branch, except for part 1.
pouyanpi/transparent-proxy-streaming-kernel-1pouyanpi/openai-chat-buffered-integration-4pouyanpi/openai-chat-streaming-contract-2pouyanpi/transparent-proxy-streaming-kernel-1pouyanpi/transparent-proxy-streaming-http-3pouyanpi/openai-chat-streaming-contract-2pouyanpi/openai-chat-streaming-integration-4pouyanpi/transparent-proxy-streaming-http-3