Greptile Summary

This PR adds Namespace field propagation across the full OpenAI Responses API conversion pipeline so that namespace-qualified function calls (e.g., mcp__gitnexus.query) are no longer stripped to bare function names.

• Adds Namespace string to llm.FunctionCall, responses.Item, and responses.StreamEvent, then threads it through every inbound/outbound conversion, stream handler, and the stream aggregator.
• Fixes a subtle bug in outbound_stream.go where the FunctionCallArgumentsDone handler was unconditionally overwriting tc.Function.Name with a potentially empty string from the done event; it is now guarded with a non-empty check, consistent with how the aggregator path already handled the same field.

Confidence Score: 5/5

Safe to merge — the change is additive field propagation with omitempty on all new fields, leaving non-namespace function calls entirely unaffected.

Every conversion site (streaming and non-streaming, inbound and outbound, aggregator and response builder) now consistently threads the new Namespace field. The previously noted gap in the aggregator's FunctionCallArgumentsDone handler is addressed in this revision. The Name guard added to outbound_stream.go is a correct defensive fix that prevents an empty done-event name from overwriting a previously captured name.

No files require special attention.

Important Files Changed

Sequence Diagram

sequenceDiagram
    participant UP as Upstream Provider
    participant OS as outbound_stream.go
    participant IS as inbound_stream.go
    participant AGG as aggregator.go
    participant Client

    UP->>OS: "output_item.added {type:function_call, name, namespace}"
    OS->>OS: "store toolCall{Name, Namespace} in state"
    OS->>Client: "stream delta ToolCall{Name, Namespace}"

    UP->>OS: "function_call_arguments.delta {delta}"
    OS->>Client: "stream delta ToolCall{Arguments delta}"

    UP->>OS: "function_call_arguments.done {name?, namespace?, arguments}"
    OS->>OS: guard update Name (if non-empty), Namespace (if non-empty)
    Note over OS: Returns nil — no event emitted

    UP->>OS: response.completed
    OS->>Client: "finish_reason=tool_calls"

    Note over IS,AGG: Aggregator path (non-streaming client)
    IS->>AGG: "OutputItemAdded {Item{Name, Namespace}}"
    AGG->>AGG: "aggregatedItem.Namespace = item.Namespace"
    IS->>AGG: FunctionCallArgumentsDelta
    IS->>AGG: "FunctionCallArgumentsDone {namespace? (guarded)}"
    AGG->>AGG: "if ev.Namespace != empty then item.Namespace = ev.Namespace"
    IS->>AGG: OutputItemDone
    AGG->>AGG: "buildResponse() with Item{Name, Namespace, Arguments}"

_{Reviews (2): Last reviewed commit: "fix: preserve namespace in function call..." | Re-trigger Greptile}

已补充 aggregator 在 StreamEventTypeFunctionCallArgumentsDone 中对 ev.Namespace 的处理，与 outbound_stream.go 的逻辑保持一致。