refactor: rename the codev-proxy surface to "backend" - #164
Merged
Conversation
Rename the codev-proxy concept to "backend" across identifiers, comments,
log labels, and error text. The live HTTP route /codev-proxy and the
generic network-proxy mentions (Login/vscode error causes, the Continue
marketplace test) are intentionally left untouched.
- src/lib/proxy.ts -> src/lib/backend.ts (and the matching test file);
update all @/lib/proxy.js imports and the `import * as proxy` /
`spyOn(proxy, …)` test namespaces to `backend`.
- PROXY_TIMEOUT_MS -> BACKEND_TIMEOUT_MS.
- Log endpoint labels "proxy.*" -> "backend.*"; thrown error prefixes
"Proxy /… failed" -> "Backend /… failed"; update the tests pinning them.
- Reword codev-proxy/proxy comments to "backend" (log, upload, supabase,
auth, AGENTS.md); fix a stale upload.ts comment that referenced the
removed custom proxy-URL.
Kept: the `${BASE_URL}/codev-proxy` route and every `/codev-proxy/…` test
matcher/fixture (deployed server contract).
Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
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.
Summary
Renames the codev-proxy concept to backend across the codebase — identifiers, comments, log labels, and error text — per the agreed scope. ("proxy url → backend url" was already settled when the hidden custom-proxy-URL step was removed in #163; the constant is
BACKEND_URL.)Changes
Code identifiers
src/lib/proxy.ts→src/lib/backend.ts(andtests/lib/proxy.test.ts→tests/lib/backend.test.ts)@/lib/proxy.jsimports →@/lib/backend.js(6 source files)import * as proxy→backendand everyvi.spyOn(proxy, …)→vi.spyOn(backend, …)(5 test files)PROXY_TIMEOUT_MS→BACKEND_TIMEOUT_MSStrings (per the "labels + error text" scope decision)
"proxy.auth-exchange"/"proxy.config"/"proxy.supabase-exchange"→"backend.*""Proxy /… failed"→"Backend /… failed"(+ all tests pinning them)Comments
log.ts,upload.ts,supabase.ts,auth.ts,AGENTS.md, a couple of test descriptions)upload.tscomment that still referenced the removed custom proxy-URLIntentionally NOT changed
${BASE_URL}/codev-proxyand everyurl.includes("/codev-proxy/…")test matcher / fixture URL — that's the deployed server contract.Login.tsx,vscode.ts,vscode.test.ts, and the Continue-marketplace "Proxy returned 502" test inInstallApp.test.tsx.Validation
pnpm fix— clean (Biome reordered imports in the renamed files)pnpm typecheck— cleanpnpm test— 720 passed / 50 filespnpm build && node dist/index.js --version→0.3.7🤖 Generated with Claude Code