Export ProviderConfig from Node.js and Python SDKs

[stephentoub]

ProviderConfig is the only way to configure the Responses API (wireApi: 'responses') and BYOK providers, but it wasn't re-exported from the public API of the Node.js or Python SDKs. Consumers had to duplicate the type locally to reference it in their own code.

This adds ProviderConfig to the public exports of both SDKs:

• Node.js — added to the export type { ... } block in src/index.ts
• Python — imported from session and added to __all__ in copilot/__init__.py

Go and .NET were already fine — Go exports via capitalized naming and .NET via public class.

Fixes #968

[Copilot]

Pull request overview

This PR exposes ProviderConfig as part of the public API surface for the Node.js and Python SDKs, so consumers can reference the SDK’s canonical BYOK/Responses provider type instead of duplicating it locally.

Changes:

• Node.js: re-exported ProviderConfig from the top-level src/index.ts type exports.
• Python: re-exported ProviderConfig by importing it from session and adding it to __all__.

Show a summary per file

File	Description
python/copilot/init.py	Adds ProviderConfig to the package import surface and __all__ for public export.
nodejs/src/index.ts	Adds ProviderConfig to the top-level export type { ... } block for public type re-export.

Copilot's findings

• Files reviewed: 2/2 changed files
• Comments generated: 0

[github-actions]

Cross-SDK Consistency Review ✅

This PR correctly addresses a cross-SDK export gap and maintains consistency across all four SDK implementations.

SDK	Status	How ProviderConfig is exported
Go	✅ Already exported	Capitalized struct name (ProviderConfig) in go/types.go
.NET	✅ Already exported	public class ProviderConfig in dotnet/src/Types.cs
Node.js	✅ Fixed in this PR	Added to export type { ... } block in src/index.ts
Python	✅ Fixed in this PR	Imported and added to __all__ in copilot/__init__.py

No further changes needed — this PR brings all SDKs to parity.

Generated by SDK Consistency Review Agent for issue #1048 · ◷

[SteveSandersonMS]

Seems worth doing to simplify things for consumers.

I mentioned to Jakub earlier this week that it's also possible to reference the type without it being exported (it's SessionConfig["provider"]), so it's not really necessary to duplicate it. Jakub thought that was sufficient, but it's certainly not as obvious, so exporting it with a name makes sense.

[SteveSandersonMS]

Thanks!