Skip to content

refactor(sdk): hard-rename provider proxy client#1736

Merged
christso merged 1 commit into
mainfrom
av-kfik535-sdk-provider-client
Jul 7, 2026
Merged

refactor(sdk): hard-rename provider proxy client#1736
christso merged 1 commit into
mainfrom
av-kfik535-sdk-provider-client

Conversation

@christso

@christso christso commented Jul 7, 2026

Copy link
Copy Markdown
Collaborator

Summary

Script graders now use provider-native proxy naming end to end. SDK consumers import createProviderClient/ProviderClient, script evaluator config enables proxy access with provider, proxy requests override with provider, /info reports providerLabel and availableProviderLabels, and AgentV injects AGENTV_PROVIDER_PROXY_URL/AGENTV_PROVIDER_PROXY_TOKEN.

This intentionally removes the target-era SDK/proxy surface rather than keeping aliases. There is no retained target-named env or wire fallback in this slice; run artifact metadata such as target_proxy remains untouched because av-kfik.53.4 owns artifact-field migration.

Exported Surface Impact

  • Removed from @agentv/sdk: createTargetClient, TargetClient, TargetInfo, TargetInvokeRequest, TargetInvokeResponse, TargetNotAvailableError, and TargetInvocationError.
  • Added replacements in @agentv/sdk: createProviderClient, ProviderClient, ProviderInfo, ProviderInvokeRequest, ProviderInvokeResponse, ProviderNotAvailableError, and ProviderInvocationError.
  • Renamed core runtime proxy exports from TargetProxy*/createTargetProxy to ProviderProxy*/createProviderProxy.
  • Script evaluator provider proxy config is now provider: true or provider: { max_calls }; script evaluator target is rejected with migration guidance.

Validation

  • bun test packages/sdk/test/provider-client.test.ts
  • bun test packages/core/test/runtime/provider-proxy.test.ts packages/core/test/evaluation/token-usage.test.ts packages/core/test/evaluation/loaders/grader-parser.test.ts
  • bun run validate:eval-schema
  • bun --filter @agentv/core build
  • bun --filter @agentv/sdk build
  • bun --filter @agentv/core typecheck
  • bun --filter @agentv/sdk typecheck
  • cd examples/features/sdk-python && uv run pytest
  • bunx biome check <touched files>
  • git diff --check
  • Removed-name scan for SDK/proxy target-era names returned no matches outside excluded historical plan/ADR/archive paths.

No live provider dogfood was run: this slice renames the local script-grader provider proxy surface and does not alter provider invocation or grading semantics beyond field/env names. The focused SDK client, runtime proxy, parser/schema, and Python helper tests exercise the changed contract.

Post-Deploy Monitoring & Validation

No additional operational monitoring required. This is a published API/config rename in local SDK/runtime proxy code with no hosted service, background job, dashboard, or production runtime path to monitor after merge; GitHub Actions and package consumers are the validation gates.

Healthy signals after merge:

  • GitHub Actions pass on the PR merge commit.
  • Docs/examples reference createProviderClient, provider, providerLabel, availableProviderLabels, and AGENTV_PROVIDER_PROXY_*.
  • No new references to removed SDK/proxy target-era names appear in CI or follow-up branches.

Failure signals and mitigation:

  • CI import/type failures referencing removed target client/proxy names. Mitigation: migrate the missed callsite to provider naming in this PR before merge.
  • Example/schema failures using script evaluator target. Mitigation: update to provider unless the path is explicitly owned by a separate non-goal bead.

Validation window and owner: PR review and GitHub Actions before merge; owner Christopher Tso.

Related

Related: av-kfik.53.5


Compound Engineering
GPT-5

@cloudflare-workers-and-pages

Copy link
Copy Markdown

Deploying agentv with  Cloudflare Pages  Cloudflare Pages

Latest commit: f6da5c7
Status: ✅  Deploy successful!
Preview URL: https://4fe69180.agentv.pages.dev
Branch Preview URL: https://av-kfik535-sdk-provider-clie.agentv.pages.dev

View logs

@christso christso merged commit 1d88a66 into main Jul 7, 2026
9 checks passed
@christso christso deleted the av-kfik535-sdk-provider-client branch July 7, 2026 22:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant