What would you like to be added:
Move TokenizedPrompt from InferenceRequestBody.TokenizedPrompt to the per-request attribute store added in #1351. The existing token-producer plugin and its declared TokenizedPromptDataKey should be the writer; downstream consumers should read via scheduling.ReadRequestAttribute[*fwkrh.TokenizedPrompt](...).
Why is this needed:
TokenizedPrompt fits well into the per-request attribute store: it is request-scoped, produced once by a single plugin, consumed by several downstream plugins, and already has a DataKey declared in the producer DAG. Today the value lives on Body and not the top level requiest purely for historical reasons.
Moving it brings the producer/consumer wiring into agreement with how the framework expects request-scoped data to flow and removes a hidden mutation surface tat exists with having it on the Body.
What would you like to be added:
Move
TokenizedPromptfromInferenceRequestBody.TokenizedPromptto the per-request attribute store added in #1351. The existingtoken-producerplugin and its declaredTokenizedPromptDataKeyshould be the writer; downstream consumers should read viascheduling.ReadRequestAttribute[*fwkrh.TokenizedPrompt](...).Why is this needed:
TokenizedPromptfits well into the per-request attribute store: it is request-scoped, produced once by a single plugin, consumed by several downstream plugins, and already has aDataKeydeclared in the producer DAG. Today the value lives onBodyand not the top level requiest purely for historical reasons.Moving it brings the producer/consumer wiring into agreement with how the framework expects request-scoped data to flow and removes a hidden mutation surface tat exists with having it on the Body.