Repository navigation
Conversation
docxology
added a commit
that referenced
this pull request
Nov 2, 2026
…ntegration-header-carried-forward Carries forward #1 by @qirh as our own commit so the change runs through CI here. The original PR's head branch was deleted upstream, so it could not be merged directly; the content is unchanged and the contributor's test assertions are kept as written. Validation: full suite 621 passed / 9 skipped / coverage 88.82%; ruff check and format clean; the Perplexity assertion fails without the source change. Remote CI run 41376147431 passed on 3.12, 3.13 and 3.14.
Collaborator
|
The head branch for this pull request was deleted upstream, so it could not be merged directly. The change is carried forward in this repository as commit 4e1cde8 (merge b4d5e21), with credit to @qirh: Closing this PR as superseded by #4. Thank you for the change, @qirh. |
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.
Adds a static
X-Pplx-Integration: symposiumdefault header to the OpenAI-compatible client inPerplexityProvider, which always targetshttps://api.perplexity.ai. The OpenRouter provider is unchanged and does not receive the header.This lets Perplexity attribute API traffic to Symposium. Authentication, models, request bodies and response handling are unchanged.
Tests:
TestPerplexityProvider.test_initnow asserts the client is built with the Perplexity base URL and the header, andTestOpenRouterProvider.test_initasserts OpenRouter gets nodefault_headers.Verification (locally, matching CI):
ruff check src/ tests/andruff format --check src/ tests/pass.python -m pytest --no-header: 188 passed, 8 skipped, coverage 86.26% (gate 80%). The updated Perplexity assertion fails without the source change.