Repository navigation
llm plane: the OpenAI Responses webhook receiver, ported before the busbar-llm delete (F26/D4, option A) - #966
Draft
MattJackson wants to merge 1 commit into
Draft
MattJackson wants to merge 1 commit into
MattJackson wants to merge 1 commit into
Conversation
…lm, switched on by the plane's own webhooks: section, verified by the auth kind, answered ROUTE_LOCAL (ARCHITECT 2026-10-14 option A)
Collaborator
Author
|
Row 155 $ #991 edits busbar-plane-llm src/plane_door.rs (one compile-time drift assertion beside the billable class list) and src/meta.rs (MeterClassDecl derived from the door's TOKEN_CLASSES). The hunks are disjoint from this PR's, and the merge with predev is clean. |
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.
PARITY-SCOPE item 3. The OpenAI Responses webhook receiver lived only in legacy busbar-llm (
webhook-receiverfeature, Cargo.toml:56,openai_responses_webhook.rs), so row 155's C delete would have dropped it with no port. Row 155 (af5c074a15290b0f8) has confirmed this lane owns the port and that C waits for it. Spec: F26 + correction (BUSBAR-1.6.0.md l.4642-4643), D4 (l.4664), E table (l.5496); ARCHITECT 2026-10-14, option A.Design (as ruled)
webhooks:section, stated with flags 0 (neither declaring nor consumed).webhooks: { openai_responses: true }switches the receiver on, and it is read at open fromPlaneOpenIn::owned. The section usesdeny_unknown_fields, so a misspelt receiver is refused at boot namingwebhooks. There is no kernel or core switch.config/prepass.rsDest::for_declarednow carries any section a registered plane owns beside its own, other than the endpoint door, RAW toplane_raw, and from theredoor_sectionshands it to the plane'sowned. Nowebhooksliteral appears in kernel or root source. The kernel is net -1 line, and only MCP'smcpendpoint door was an owned section before, which is unchanged. Top-level unknown-key refusals keep 1.5.5's list: lifted keys are not serde fields, as withtools/agents.POST /v1/llm/webhooks/openaiclaim wearing the Responses dialect.auth.chainnames abusbar-auth-webhook-signaturestandard-webhooksinstance (pinned and linked by busbar-auth-webhook-signature pinned and linked, default-on (Twilio / Standard Webhooks inbound verify) #948). All-PASS is 401, a forged signature is REJECT, and a replayedwebhook-idis refused through the claim record. The plane never sees the secret, and its unit requires a principal.ROUTE_LOCAL. There is no walk and no far end, and the fee unit is never incurred (fee_unitexcludes local units), so it bills nothing. On the caller's last piece the plane answers200 application/json {"received":true,"response_id":…,"type":…}, the ported receiver's acknowledgement. A body that is not a Responses event isDecline::WebhookBody, a 400invalid_requestin the Responses envelope.REDs (before the port there was no receiver, so none of these builds)
busbar-plane-llm/tests/exchange_webhook.rs: off by default with no extra claim; switched on gives one exact POST claim in the Responses dialect; a misspelt or mistyped section is refused naming it; the Statement ownswebhookswith flags 0; only a POST on the exact path is a delivery; read and ack bytes; ten malformed bodies are refused.busbar-kernel config::tests::a_section_a_plane_owns_beside_its_own_is_lifted_raw_for_it: before the fix the document was refused at the key as an unknown field.Register. This is new 1.6.0 surface with no 1.5.5 golden (F26 correction). It carries a CHANGELOG line under 1.6.0 Added. No oracle cell exercises the path. A cell for it belongs to the oracle-new-cells lane, and it records only after #911.
DRAFT. It waits for:
plane_door.rs. My hunks are the SECTIONS row, snapshot/claims,UnitState(two fields),LlmDoor(one field),fee_unit, the FROM_CALLER arm ofanswer, and open/refresh/arrive.config/tests/tests.rs(my test is appended at the end).