Add experimental GitHub telemetry redirection across all SDKs

[MackinnonBuck]

Why

The runtime can now forward per-session GitHub (hydro) telemetry to opted-in host connections via a new JSON-RPC server to client notification, gitHubTelemetry.event. This PR implements the consumer side of that contract in every language SDK so hosts can receive those events. The feature is internal and experimental, so it is intentionally kept low-profile and undocumented for public consumers.

What

For each SDK this adds an onGitHubTelemetry callback (per-language naming) and a hand-written opt-in: when a host registers a handler, the client sets enableGitHubTelemetryRedirection on every session it creates or resumes, and dispatches each gitHubTelemetry.event notification to the callback.

Key pieces:

• Generated types. Regenerated the RPC types in all languages from the new schema (GitHubTelemetryNotification, GitHubTelemetryEvent, GitHubTelemetryClientInfo, and the gitHubTelemetry.event clientGlobal method). The shared codegen change adds a notification flag to RpcMethod so generators emit notification-style dispatch (no JSON-RPC reply) for void clientGlobal methods.
• Hand-written glue. Each SDK surfaces the callback using its existing llmInference / notification idiom and wires the opt-in flag into the create/resume requests. We deliberately reuse the generated RPC types directly rather than introducing friendly wrapper types.
• Tests. Real-wire tests per language verify the handler fires with a correctly typed payload and that the opt-in flag is present when a handler is registered and omitted otherwise.

Notable decisions

• One commit per language (plus a small shared codegen-infra commit for utils.ts). This makes the feature trivially revertible language-by-language: reverting a single language's commit removes its support without touching the others. The shared utils.ts change is purely additive (an optional notification? field), so it stays put and never breaks the remaining generators.
• "Unadvertised" where the language supports it. C# uses [EditorBrowsable(EditorBrowsableState.Never)] and Rust uses #[doc(hidden)]. Languages without a true "exported but hidden" mechanism stay visible + experimental with minimal docs.
• No public docs. The types are internal/experimental by design.

Review notes

This is the SDK side of a contract whose runtime counterpart is merged-in-progress on the @github/copilot (copilot-agent-runtime) side. The schema is the source of truth; generated files were produced by codegen, not hand-edited.

[Copilot]

Pull request overview

Implements an experimental cross-SDK surface for receiving per-session GitHub (hydro) telemetry forwarded by the runtime via a new JSON-RPC client-global notification (gitHubTelemetry.event). The PR extends shared codegen to understand notification-style RPC methods and wires opt-in + dispatch behavior across TypeScript/Node, Python, Go, Rust, Java, and .NET, with tests validating forwarding and handler invocation.

Changes:

• Added schema override + shared codegen support for notification-style client-global methods (RpcMethod.notification) and for injecting additive API schema entries during schema loading.
• Implemented per-SDK “on GitHub telemetry” callback wiring and session create/resume opt-in (enableGitHubTelemetryForwarding) when a handler is registered.
• Added/updated unit/integration tests across SDKs to validate handler dispatch and the opt-in flag behavior.

Show a summary per file

File	Description
scripts/codegen/utils.ts	Loads API schema and applies additive overrides; adds notification?: boolean to RpcMethod.
scripts/codegen/typescript.ts	Emits onNotification wiring for notification-style client-global methods.
scripts/codegen/schema-overrides/api-additions.schema.json	Adds gitHubTelemetry.event + telemetry types via additive schema override (marked notification + experimental).
scripts/codegen/rust.ts	Switches schema loading to shared loadSchemaJson (enabling overrides).
scripts/codegen/python.ts	Emits notification-handler wiring for notification-style client-global methods.
scripts/codegen/go.ts	Adjusts Go codegen for client-global void/notification methods and emits notification-safe handler signatures.
rust/src/github_telemetry.rs	New Rust module re-exporting telemetry types + callback alias (doc-hidden).
rust/src/lib.rs	Adds client option + plumbing for telemetry callback; adds test constructor helper.
rust/src/router.rs	Routes gitHubTelemetry.event client-global notifications to callback (if set).
rust/src/session.rs	Sets enableGitHubTelemetryForwarding on create/resume when callback registered.
rust/src/types.rs	Initializes new wire fields to None in config→wire conversion.
rust/src/wire.rs	Adds enableGitHubTelemetryForwarding wire field to create/resume request DTOs.
rust/src/generated/api_types.rs	Generated telemetry DTOs.
rust/tests/session_test.rs	Rust integration tests validating opt-in flag and callback dispatch.
python/copilot/_jsonrpc.py	Adds method-specific notification handler registry + dispatch path.
python/copilot/generated/rpc.py	Generated telemetry DTOs + client-global notification handler registration for gitHubTelemetry.event.
python/copilot/client.py	Adds on_github_telemetry option, opt-in flag wiring, and registers client-global handlers.
python/copilot/init.py	Re-exports telemetry DTOs at package level.
python/test_client.py	Python unit tests for opt-in flag and notification dispatch path.
nodejs/src/generated/rpc.ts	Generated telemetry DTOs + onNotification registration for gitHubTelemetry.event.
nodejs/src/types.ts	Exposes telemetry DTOs and adds onGitHubTelemetry option type.
nodejs/src/index.ts	Re-exports telemetry DTO types from the public entrypoint.
nodejs/src/client.ts	Wires onGitHubTelemetry, registers client-global handlers, and sets session opt-in flag.
nodejs/test/client.test.ts	Node tests for opt-in flag behavior and notification dispatch.
go/rpc/zrpc.go	Generated Go telemetry DTOs + client-global handler interface + registration.
go/types.go	Adds OnGitHubTelemetry client option and wire fields on create/resume DTOs.
go/client.go	Sets opt-in flag on create/resume and registers telemetry callback adapter.
go/client_test.go	Go tests validating opt-in flag serialization and notification routing to callback.
java/src/main/java/com/github/copilot/GitHubTelemetryAdapter.java	Java adapter bridging gitHubTelemetry.event into consumer callback.
java/src/main/java/com/github/copilot/CopilotClient.java	Registers telemetry adapter on connect; sets opt-in flag on create/resume when handler set.
java/src/main/java/com/github/copilot/rpc/CopilotClientOptions.java	Adds experimental setOnGitHubTelemetry option and clones it.
java/src/main/java/com/github/copilot/rpc/CreateSessionRequest.java	Adds enableGitHubTelemetryForwarding field + accessors.
java/src/main/java/com/github/copilot/rpc/ResumeSessionRequest.java	Adds enableGitHubTelemetryForwarding field + accessors.
java/src/main/java/com/github/copilot/rpc/GitHubTelemetryClientInfo.java	Java telemetry DTO.
java/src/main/java/com/github/copilot/rpc/GitHubTelemetryEvent.java	Java telemetry DTO.
java/src/main/java/com/github/copilot/rpc/GitHubTelemetryNotification.java	Java telemetry DTO.
java/src/test/java/com/github/copilot/GitHubTelemetryTest.java	Java tests verifying adapter dispatch and opt-in flag on create/resume.
dotnet/src/Generated/Rpc.cs	Generated telemetry DTOs + client-global handler interface and registration.
dotnet/src/Types.cs	Adds hidden experimental OnGitHubTelemetry option to CopilotClientOptions.
dotnet/src/Client.cs	Sets opt-in flag when handler present; registers telemetry adapter in client-global handlers.
dotnet/test/Unit/GitHubTelemetryTests.cs	.NET unit tests validating opt-in behavior and notification forwarding.

Review details

Files not reviewed (1)

• go/rpc/zrpc.go: Generated file

• Files reviewed: 36/41 changed files
• Comments generated: 6
• Review effort level: Low

[github-actions]

Cross-SDK Consistency Review ✅

This PR adds the experimental GitHub telemetry forwarding feature across all six SDK implementations. Overall, it maintains excellent cross-SDK consistency.

What's consistent ✅

Aspect	Node.js	Python	Go	.NET	Java	Rust
Option name	onGitHubTelemetry	on_github_telemetry	OnGitHubTelemetry	OnGitHubTelemetry	setOnGitHubTelemetry	on_github_telemetry
Wire flag in create/resume	✅	✅	✅	✅	✅	✅
Experimental annotation	✅	✅	✅	✅	✅	✅
Generated types exported	✅	✅	✅	✅	✅	✅
Adapter pattern	✅	✅	✅	✅	✅	✅

All naming follows each language's conventions (camelCase/snake_case/PascalCase/bean-style). The enableGitHubTelemetryForwarding wire flag is set consistently in both createSession and resumeSession across all SDKs. Nice use of per-language hide mechanisms (C#'s [EditorBrowsable(Never)], Rust's #[doc(hidden)]).

Minor observation: error handling in callback dispatch

There's a small inconsistency in how user-callback exceptions are handled across SDKs:

• Node.js: try { await callback(...) } catch { /* silent */ } — silently swallowed
• Go: defer func() { recover() }() — panics recovered
• Python: Caught by the underlying _dispatch_notification logging layer — safe
• Java: catch (Exception e) { LOG.log(Level.SEVERE, ...) } — logged
• .NET (GitHubTelemetryAdapter.EventAsync): No try/catch at the adapter level — exception propagates up to the RPC dispatch layer

This isn't a blocking issue (the .NET RPC layer may handle it upstream, and the telemetry path is "best-effort" by nature), but it's worth noting for consistency. If a user's OnGitHubTelemetry Action throws in .NET, the behavior differs from other SDKs that swallow or log the error.

Summary

This is a well-implemented, genuinely cross-language feature addition. All SDKs have feature parity and consistent API design. No blocking consistency issues found.

Generated by SDK Consistency Review Agent for issue #1835 · sonnet46 2.2M · ◷