Repository navigation
fix(perplexity-web): make the built-in-search hint opt-in - #10904
Merged
Merged
Conversation
Extracted from #8634: the two other fixes there (mode "search" -> "copilot" downgrade, pplx-opus -> claude50opus) are already on this release tip (open-sse/executors/perplexity-web/protocol.ts already uses "copilot" and maps pplx-opus to claude50opus). Only the search-hint opt-in remained uncovered. The built-in-search hint appended to every system message is now opt-in via OMNIROUTE_PPLX_SEARCH_HINT (off by default). Perplexity's answer engine searches anyway, and for coding clients the sentence leaks into replies as meta-commentary ("I need to search before responding per my instructions"). Co-authored-by: diegosouzapw <8016841+diegosouzapw@users.noreply.github.com>
muhamadgalihsaputra
pushed a commit
to niyatna/NiyatnaRoute
that referenced
this pull request
Sep 27, 2026
…ference#10904) Merged — extraction of the one still-uncovered fix from cheaper-inference#8634 (the other two items — mode "search"→CONCISE downgrade, pplx-opus generation — were already applied on this release tip). typecheck/file-size/changelog/complexity/cognitive-complexity gates all clean, 32/32 tests passing.
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.
Clean extraction of the one still-uncovered fix from #8634 (by @danscMax), which was a month-old branch CONFLICTING against the current release.
Auditing what #8634 fixed against the current tip: two of its three items are already applied —
mode: "search"downgrade to CONCISE — already fixed;open-sse/executors/perplexity-web/protocol.tsalready posts"copilot"throughoutMODEL_MAP/THINKING_MAP.pplx-opuspointing at the previous Opus generation — already fixed;MODEL_MAP["pplx-opus"]already resolves toclaude50opusand the catalog label already reads "Claude Opus 5.0".Only item 3 remained:
The built-in-search hint is now opt-in
The sentence "You have built-in web search. Answer questions directly using search results." was appended to every system message. Perplexity's answer engine searches anyway, and for coding clients it leaks into replies as meta-commentary ("I need to search before responding per my instructions"). Now behind
OMNIROUTE_PPLX_SEARCH_HINT(off by default, documented in.env.exampleanddocs/reference/ENVIRONMENT.md).Validation
npm run typecheck:core— cleannode scripts/check/check-file-size.mjs/check-changelog-integrity.mjs/check-complexity.mjs/check-cognitive-complexity.mjs— all OKtests/unit/perplexity-web.test.ts— 32/32 passing (including the new hint on/off/env-toggle case)Co-authored-by: dansc dansc@yandex.ru