Add E2E test for session.providerEndpoint.get#1621
Conversation
This comment has been minimized.
This comment has been minimized.
c310f9d to
9d87a1e
Compare
Validates the new shared API exposed by copilot-agent-runtime (see github/copilot-sdk-internal#133) end-to-end from the Node SDK. Covers both BYOK (returns provider config + headers) and CAPI (returns the resolved CAPI base URL). Regenerates rpc.ts and session-events.ts against the runtime schema that adds the providerEndpoint RPC. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
9d87a1e to
d0916b9
Compare
Cross-SDK Consistency ReviewThis PR regenerates Node.js generated types against an updated runtime schema and adds E2E tests for New APIs added (Node.js only)The regenerated
Schema field changes that also need propagationThe following field-level schema changes in Node.js generated types will also need to be reflected in the other SDKs' generated files:
E2E tests to addOnce the runtime ships,
No action needed nowSince this is a draft waiting on the runtime release, the above is expected to be incomplete at this stage. When the runtime PR lands and this is undrafted, running The Node.js test implementation looks correct and comprehensive (covers both BYOK and CAPI-OAuth cases). 👍
|
Adds a Node E2E test for the new
session.providerEndpoint.getshared API (not yet shipped).The new RPC returns the provider endpoint + credentials for a session — the BYOK config if one is set, otherwise the resolved CAPI endpoint — so SDK consumers can call the LLM backend directly with their own OpenAI/Anthropic client.
Changes
nodejs/src/generated/rpc.tsandsession-events.tsagainst the runtime schema that addsproviderEndpoint.nodejs/test/e2e/provider_endpoint.e2e.test.tscovering:protocol,baseUrl,apiKey,headersfrom the configured provider.CapiProxyso no real network calls).Validating
Built the runtime branch locally and pointed the harness at it:
Both tests pass.
Draft because this depends on the runtime PR landing + a release that includes the new schema before it can run in CI.