docs: speak the new subscription provider names on every user surface - #2016
Merged
Merged
Conversation
Renames the provider display names and ids that a user reads: the provider guide headings and table, the /fast help and its RPC error message, the service-tier note in settings, and the provider list in the pi-ai README. Deliberately NOT renamed, each verified line by line: - docs/skills.md "skills from Claude Code or OpenAI Codex" names OpenAI's actual Codex CLI product, whose skill directories a user configures. It is not senpi's old display name and renaming it would make the sentence false. - the frozen wire api id `openai-codex-responses` and its human name "OpenAI Codex Responses API" (custom-provider.md, pi-ai README). - "OpenAI"/"OpenAI-compatible" as the company/API, and the outbound `originator` identity sent to the codex backend. - module and test paths (`claude-sdk-oauth` extension dir, release-guide entries), which the behaviour-free internal rename owns. Also fixes doc drift found while sweeping: rpc.md quoted the fast-mode error as "Fast mode is only available for OpenAI Codex models." while the shipped string in service-tier.ts already reads "ChatGPT Subscription models." Part of #1989
code-yeongyu
force-pushed
the
docs/1989-string-sweep
branch
from
September 22, 2026 15:29
b78f8a8 to
c1f105b
Compare
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.
Makes every user-facing surface speak the new provider names. Part of #1989 (todo 12).
Renamed (14 places)
Provider guide headings + table, the
/fasthelp text and its RPC error message, the service-tier note insettings.md, the RPC JSON examples'"provider"values, and the provider list in the pi-ai README.Deliberately NOT renamed — each verified line by line
This is the part that matters, because a blind sweep corrupts real content:
docs/skills.md:47"skills from Claude Code or OpenAI Codex"openai-codex-responses+ "OpenAI Codex Responses API"supportsOpenAIGrammarToolsoriginatoridentityclaude-sdk-oauthextension dir + release-guide test pathsOpenAI Codexis simultaneously senpi's old display name (must go) and a real external product senpi interoperates with (must stay). The disambiguation is what the sentence is about, not the token.Doc drift fixed while sweeping
rpc.mddocumented the fast-mode error asFast mode is only available for OpenAI Codex models.— but the shipped string inservice-tier.ts:145already reads "ChatGPT Subscription models." The doc was quoting a message that no longer exists. Now synced to what actually ships.Verification
Changelog gate PASS (docs-only, no runtime source change). Every remaining legacy mention in
docs/+ READMEs is classified above and intentional.Summary by cubic
Renames the subscription provider display names and ids on every user-facing surface — provider guide headings,
/fasthelp and its RPC error, the service-tier note insettings.md, RPC JSON examples'"provider"values, and thepi-aiREADME — as part of #1989. Also renames the "Claude SDK OAuth" heading to "Anthropic Subscription". Docs-only change; no runtime behavior.Deliberately not renamed
openai-codex-responseswire API id, and the outboundoriginatoridentity refer to the company/API, not the provider.claude-sdk-oauthextension dir, release-guide entries) belong to the behavior-free internal rename.Doc drift fixed
rpc.mdquoted a fast-mode error that no longer ships; it's now synced to the actual "ChatGPT Subscription models." string.Written for commit c1f105b. Summary will update on new commits.