Skip to content

Update @github/copilot to 1.0.41-1#1202

Merged
stephentoub merged 2 commits into
mainfrom
update-copilot-1.0.41-1
May 5, 2026
Merged

Update @github/copilot to 1.0.41-1#1202
stephentoub merged 2 commits into
mainfrom
update-copilot-1.0.41-1

Conversation

@github-actions
Copy link
Copy Markdown
Contributor

@github-actions github-actions Bot commented May 5, 2026

Automated update of @github/copilot to version 1.0.41-1.

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 5, 2026 13:48
Copilot AI review requested due to automatic review settings May 5, 2026 13:48
@stephentoub stephentoub requested a review from a team as a code owner May 5, 2026 13:48
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.

Pull request overview

This PR updates the monorepo’s Node-side @github/copilot dependency to 1.0.41-1 and refreshes generated SDK event types to match the upstream schema change. It fits the repo’s normal flow where the language SDKs track Copilot CLI/JSON-RPC schema updates coming from the shared package.

Changes:

  • Bumped @github/copilot to ^1.0.41-1 in the Node package and test harness, with lockfiles updated to the resolved 1.0.41-1 artifacts.
  • Regenerated cross-language session event types so CustomAgentsUpdatedAgent.tools can be nullable where the upstream schema now allows null.
  • Propagated the dependency update into the Node samples lockfile.
Show a summary per file
File Description
test/harness/package.json Bumps harness devDependency to the new Copilot package version.
test/harness/package-lock.json Updates resolved harness package and platform-specific optional binaries to 1.0.41-1.
python/copilot/generated/session_events.py Regenerated Python event model to allow nullable tools.
nodejs/src/generated/session-events.ts Regenerated TypeScript event interface/docs for nullable tools.
nodejs/samples/package-lock.json Refreshes sample lockfile metadata after the package version bump.
nodejs/package.json Bumps main Node SDK dependency to ^1.0.41-1.
nodejs/package-lock.json Updates resolved Node package and platform-specific optional binaries to 1.0.41-1.
go/generated_session_events.go Regenerated Go event comments to reflect nullable tools semantics.
dotnet/src/Generated/SessionEvents.cs Regenerated .NET event model to allow nullable tools.

Copilot's findings

Files not reviewed (3)
  • nodejs/package-lock.json: Language not supported
  • nodejs/samples/package-lock.json: Language not supported
  • test/harness/package-lock.json: Language not supported
  • Files reviewed: 3/9 changed files
  • Comments generated: 0

github-actions Bot and others added 2 commits May 5, 2026 11:45
- Updated nodejs and test harness dependencies
- Re-ran code generators
- Formatted generated code
Avoid waiting for an assistant continuation after resuming with continuePendingWork=false. The runtime can interrupt pending work before a manual completion drives another model turn, so the test now only verifies the pending tool call remains handleable.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
@stephentoub stephentoub force-pushed the update-copilot-1.0.41-1 branch from 266a788 to d3f5f40 Compare May 5, 2026 15:57
@github-actions
Copy link
Copy Markdown
Contributor Author

github-actions Bot commented May 5, 2026

Cross-SDK Consistency Review ✅

This PR maintains consistent cross-SDK behavior for the CustomAgentsUpdatedAgent.Tools nullable change across all four language implementations:

SDK Change Correct?
TypeScript tools: string[]tools: string[] | null
Python tools: list[str]tools: list[str] | None + serialization logic
.NET required string[] Toolsstring[]? Tools (nullable, removed required)
Go Comment updated only; type stays []string

Note on Go: The Go type []string didn't need a type change — in Go, a nil slice naturally deserializes from JSON null and re-serializes as null, making it the idiomatic nullable representation for slices. No change to *[]string was needed. The comment-only update is correct.

No cross-SDK consistency issues found.

Generated by SDK Consistency Review Agent for issue #1202 · ● 268.9K ·

@stephentoub stephentoub merged commit 08b486d into main May 5, 2026
36 checks passed
@stephentoub stephentoub deleted the update-copilot-1.0.41-1 branch May 5, 2026 16:08
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.

2 participants