Skip to content

Update @github/copilot to 1.0.49-6#1327

Open
github-actions[bot] wants to merge 6 commits into
mainfrom
update-copilot-1.0.49-6
Open

Update @github/copilot to 1.0.49-6#1327
github-actions[bot] wants to merge 6 commits into
mainfrom
update-copilot-1.0.49-6

Conversation

@github-actions
Copy link
Copy Markdown
Contributor

Automated update of @github/copilot to version 1.0.49-6.

Changes

  • Updated @github/copilot in nodejs/package.json and test/harness/package.json
  • Re-ran all code generators (scripts/codegen)
  • Formatted generated output

Next steps

When ready, click Ready for review to trigger CI checks.

Created by the Update @github/copilot Dependency workflow.

@stephentoub stephentoub marked this pull request as ready for review May 18, 2026 22:18
@stephentoub stephentoub requested a review from a team as a code owner May 18, 2026 22:18
Copilot AI review requested due to automatic review settings May 18, 2026 22:18
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot encountered an error and was unable to review this pull request. You can try again by re-requesting a review.

Comment thread dotnet/src/Generated/SessionEvents.cs Fixed
@stephentoub stephentoub self-assigned this May 18, 2026
Comment thread dotnet/test/E2E/ModeHandlersE2ETests.cs Fixed
Comment thread dotnet/src/SessionFsProvider.cs Fixed
Comment thread dotnet/src/SessionFsProvider.cs Fixed
@github-actions

This comment has been minimized.

Copy link
Copy Markdown
Contributor Author

@github-actions github-actions Bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Generated by SDK Consistency Review Agent for issue #1327 · ● 1.1M

Comment thread rust/src/generated/api_types.rs
Comment thread rust/src/session_fs.rs
Comment thread dotnet/test/E2E/ModeHandlersE2ETests.cs Fixed
@github-actions

This comment has been minimized.

@github-actions

This comment has been minimized.

@github-actions

This comment has been minimized.

Copy link
Copy Markdown
Contributor Author

@github-actions github-actions Bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Generated by SDK Consistency Review Agent for issue #1327 · ● 2M

Comment thread rust/src/session_fs.rs Outdated
github-actions Bot and others added 6 commits May 18, 2026 23:25
- Updated nodejs and test harness dependencies
- Re-ran code generators
- Formatted generated code
Add SessionFS SQLite support across SDKs and fix generated type issues exposed by the updated Copilot schema.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Tighten C# SessionFS exception handling and make Python SessionFS adapter params type-check cleanly.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Teach Rust codegen to prefer richer inline RPC parameter schemas when definitions omit wire fields, and forward the SQLite query session ID through the provider trait.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
@stephentoub stephentoub force-pushed the update-copilot-1.0.49-6 branch from b05364e to 8ba82ae Compare May 19, 2026 03:34
@github-actions
Copy link
Copy Markdown
Contributor Author

Cross-SDK Consistency Review ✅

This automated codegen PR maintains excellent consistency across all five SDK implementations (Node.js, Python, Go, .NET, Rust). Here's a summary of what was verified:

New Features — Applied Consistently to All SDKs

Feature Node.js Python Go .NET Rust
sqliteQuery / sqlite_query on SessionFsProvider
sqliteExists / sqlite_exists on SessionFsProvider
SessionFsSetProviderCapabilities with sqlite field
ModelPolicyState typed enum (was string)
McpServerSource replacing DiscoveredMcpServerSource
SkillSource typed enum for ServerSkill.source
AutoModeSwitchResponse moved to generated session events + new response field on AutoModeSwitchCompletedData
SessionFsSqliteQueryResult.lastInsertRowid field

Language-Idiomatic Differences (Expected, Not Inconsistencies)

  • Date fields (ResetDate, StaleAt, ModifiedTime, StartTime): .NET uses DateTimeOffset, Go uses typed time values, while Node.js/Python/Rust keep ISO 8601 strings — all idiomatic for each language.
  • SQLite method implementations: Rust uses optional trait methods with default implementations returning errors (non-breaking, consistent with the existing rename pattern), while Node.js, Python, Go, and .NET make them required interface members. This follows each SDK's established conventions.
  • AutoModeSwitchResponse in Go: Cleanly moved from the hand-written types.go into the generated zsession_events.go, with re-exports maintained in zsession_events.go.
  • AutoModeSwitchResponse in .NET: Moved from the hand-written Types.cs (C# enum) to a generated forward-compatible readonly struct in SessionEvents.cs, consistent with the SDK's existing pattern for session-event string enums.

No cross-SDK consistency issues found. The changes are well-coordinated across all SDKs. 🎉

Generated by SDK Consistency Review Agent for issue #1327 · ● 996K ·

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.

3 participants