Greptile Summary

This PR adds a system-wide pass-through toggle that mirrors the existing User-Agent pass-through setting. ChannelSettings.PassThroughBody is promoted from bool to *bool so that nil channels inherit the global default, while an explicit true/false still overrides it. The change is well-structured and backwards-compatible (the old omitempty bool field never persisted false, so existing rows deserialise correctly as nil).

Confidence Score: 5/5

Safe to merge; only P2 findings remain after the previously-raised P1 (multiple DB queries per request) was already noted in an earlier review round.

All findings are P2 or below. The implementation is consistent with the existing User-Agent pass-through pattern, backwards-compatible JSON serialisation is preserved, and the GraphQL schema change is additive.

internal/server/orchestrator/pass_through.go — the global DB lookup is still called up to 3× per request on the inherit path (pre-existing open issue). frontend/src/features/channels/components/channels-action-dialog.tsx — inherit state does not surface the prompt-injection warning.

Important Files Changed

Flowchart

%%{init: {'theme': 'neutral'}}%%
flowchart TD
    A[Request enters orchestrator] --> B[isPassThroughEnabled called]
    B --> C{channel == nil?}
    C -- yes --> D[return false]
    C -- no --> E{API formats match?}
    E -- no --> D
    E -- yes --> F{channel.Settings.PassThroughBody != nil?}
    F -- yes --> G[use channel-level value]
    F -- no --> H{systemService != nil?}
    H -- no --> I[return false default]
    H -- yes --> J[systemService.PassThrough DB lookup]
    J --> K{error?}
    K -- yes --> L[log warn, return false]
    K -- no --> M[use global value]
    G --> N[return enabled]
    M --> N

_{Reviews (2): Last reviewed commit: "fix: lint issues from global pass-throug..." | Re-trigger Greptile}