C# API review fixes#1343

Merged

SteveSandersonMS merged 40 commits into

mainfrom

stevesandersonms/stevesa-api-review-fixes

May 21, 2026

Commits on May 21, 2026

Phase 1: codegen falls back to JsonElement instead of object

SteveSandersonMS
and
Copilot
committed
Phase 3: seal hand-written public types

SteveSandersonMS
and
Copilot
committed
Phase 4a: simple property/method renames

SteveSandersonMS
and
Copilot
committed
Phase 4b: expand SessionFsProvider abbreviations

SteveSandersonMS
and
Copilot
committed
Phase 4c: shape changes (Streaming, Tools, ToolBinaryResultType, cleanups)

SteveSandersonMS
and
Copilot
committed
Phase 4d/4e: remove CopilotClient.State, retype LogLevel

SteveSandersonMS
and
Copilot
committed
Fix mojibake from cp1252/UTF-8 round-trip in earlier phase commits

SteveSandersonMS
and
Copilot
committed
Phase 4f: generic OnLifecycle<T> + On<T>, polymorphic lifecycle events

SteveSandersonMS
and
Copilot
committed
Phase 4g: DateTimeOffset timestamps + PermissionRequestResult.Feedback

SteveSandersonMS
and
Copilot
committed
Phase 7: SendAsync(string) / SendAndWaitAsync(string) convenience overloads

SteveSandersonMS
and
Copilot
committed
Phase 9 (partial): rename Cwd/Remote properties on hand-written types

SteveSandersonMS
and
Copilot
committed
Phase 8: README updates for OnLifecycle<T> and lifecycle event types

SteveSandersonMS
and
Copilot
committed
Phase 5: extract SessionConfigBase; seal config classes

SteveSandersonMS
and
Copilot
committed
Phase 6: remove named delegate types, use Func<...>

SteveSandersonMS
and
Copilot
committed
Phase 9: introduce RuntimeConnection discriminated config

SteveSandersonMS
and
Copilot
committed
README: update Custom Permission Handler section for Phase 6 delegate removal

SteveSandersonMS
and
Copilot
committed
Fix snapshot capture locally
SteveSandersonMS
committed
Fix MCP config serialization
SteveSandersonMS
committed
Revert "Phase 1: codegen falls back to JsonElement instead of object"
SteveSandersonMS
committed
Post-revert cleanup for Phase 1 revert

SteveSandersonMS
and
Copilot
committed
Revert "Fix MCP config serialization"
SteveSandersonMS
committed
Fixes to E2E tests to restore main behavior
SteveSandersonMS
committed
Restore TCP transport for two ClientOptions tests

SteveSandersonMS
and
Copilot
committed
Test harness: throw if both useStdio and Connection are supplied

SteveSandersonMS
and
Copilot
committed
Revert accidental CapiProxy.cs change

SteveSandersonMS
and
Copilot
committed
Test fix
SteveSandersonMS
committed
Update C# doc snippets for renamed APIs

SteveSandersonMS
and
Copilot
committed
Update C# test scenarios for renamed APIs

SteveSandersonMS
and
Copilot
committed
Drop .SDK from C# namespaces

SteveSandersonMS
and
Copilot
committed
Fix local test runs
SteveSandersonMS
committed
Drop unused uriConn binding in Client.cs is-check

SteveSandersonMS
and
Copilot
committed
Address Copilot review feedback

SteveSandersonMS
and
Copilot
committed
Docs validation: prepend 'using GitHub.Copilot;' after namespace rename

SteveSandersonMS
and
Copilot
committed
Rename RuntimeConnection factories: Stdio/Tcp/Uri -> ForStdio/ForTcp/ForUri

SteveSandersonMS
and
Copilot
committed
Apply RuntimeConnection.For* rename in E2ETestContext.cs

SteveSandersonMS
and
Copilot
committed
Docs: restore 'dotnet add package GitHub.Copilot.SDK'

SteveSandersonMS
and
Copilot
committed
Dev version
SteveSandersonMS
committed
Codegen: drop full GitHub.Copilot qualifier from GeneratedStringEnumJson refs

SteveSandersonMS
and
Copilot
committed
Regenerate Rpc.cs after rebase onto origin/main

SteveSandersonMS
and
Copilot
committed
Test: make Should_Get_Null_Last_Session_Id resilient to test order

SteveSandersonMS
and
Copilot
committed