SWI-11363 chore: replace "adapter" with "translator" throughout - #24
Merged
Conversation
Finishes the Migration Toolkit rebrand (#21), which renamed the product in prose but left "adapter" in env vars, code identifiers, filenames, and runtime strings. No backward-compatible aliases. Three referents, three words: - The running proxy is the **translator** — env vars (ADAPTER_* -> TRANSLATOR_*), user-facing strings, matrix notes, OpenAPI spec, docs, comments, and test fixture hostnames. - The `src/server` types are the **server** — AdapterConfig/Deps/Error and adapterErrors become ServerConfig/Deps/Error and serverErrors. Naming them Translator* would collide with the narrower `src/translator` module. - Whole-product framing is the **toolkit** — AGENTS.md's paired model, CONTRIBUTING.md, and the issue templates cover all three tools, not just the translator. Also: openapi/twilio-compat-adapter.yml -> twilio-compat.yml, scripts/bench-adapter-overhead.ts -> bench-translator-overhead.ts, and the stale bw-voice-adapter repo URL in the issue-template config. BREAKING: TRANSLATOR_ACCOUNT_SID, TRANSLATOR_AUTH_TOKEN, TRANSLATOR_CAPTURE_DIR, and TRANSLATOR_LOG replace their ADAPTER_* equivalents. Existing deployments must update their environment.
✅ Snyk checks have passed. No issues have been found so far.
💻 Catch issues earlier using the plugins for VS Code, JetBrains IDEs, Visual Studio, and Eclipse. |
joshraub-bw
approved these changes
Aug 20, 2026
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.
BLUF
Finishes the Migration Toolkit rebrand (#21), which renamed the product in prose but left 229 references to "adapter" in env vars, code identifiers, filenames, and runtime strings. Zero remain. No backward-compatible aliases — existing deployments must update their environment.
Three referents, three words
The word "adapter" was doing three different jobs, so a blanket find-and-replace would have made things worse:
src/servertypesTranslatorConfigwould collide with the much narrowersrc/translatormodule — it's the server's config, not the translation engine'sADAPTER_ACCOUNT_SIDTRANSLATOR_ACCOUNT_SIDADAPTER_AUTH_TOKENTRANSLATOR_AUTH_TOKENADAPTER_CAPTURE_DIRTRANSLATOR_CAPTURE_DIRADAPTER_LOGTRANSLATOR_LOGWhat changed
Renamed identifiers —
AdapterConfig/AdapterDeps/AdapterError/adapterErrors→ServerConfig/ServerDeps/ServerError/serverErrors.User-visible strings — the
90002body is now"Internal translator error"; the message played into the call on an unsupported verb; thetranslator listening on…startup log; 7 finding messages intranslate.ts; 8notesfields intwilio-voice.json(these surface in the playground and compat reports).OpenAPI spec — retitled Real-Time Translator (Voice),
adapterHost→translatorHost.Files renamed —
openapi/twilio-compat-adapter.yml→twilio-compat.yml("compat" already carries it);scripts/bench-adapter-overhead.ts→bench-translator-overhead.ts. Thebench:overheadnpm script name is unchanged.Docs — README, SECURITY,
docs/demo.md, AGENTS.md, CONTRIBUTING,web/README.md, and both issue templates. Also fixed the stalebw-voice-adapterrepo URL in the issue-template config (it was redirect-only).Tests — ~30 inert fixture hostnames (
adapter.test→translator.test) plus the assertions that move in lockstep with the message changes above.Verification
npm run typecheck— cleannpm test— 303/303 passing across 43 filesnpm run playground:build— builds (506 KB)npm run compatibility-check -- examples/full-pv-app— smoke-tested, correct outputReviewer notes
web/fonts/*.woff2binaries. I verified them byte-for-byte againstHEAD— unchanged — and removed three zero-byte temp files macOS sed left behind. Worth a glance..env" that isn't tracked in the repo. Happy to clean that up separately.adapter/*branch names are left alone — they're historical.🤖 Generated with Claude Code