Skip to content

Re-generate sdk files based on updates#1159

Closed
DonJayamanne wants to merge 1 commit intogithub:mainfrom
DonJayamanne:don/broad-primate
Closed

Re-generate sdk files based on updates#1159
DonJayamanne wants to merge 1 commit intogithub:mainfrom
DonJayamanne:don/broad-primate

Conversation

@DonJayamanne
Copy link
Copy Markdown

@DonJayamanne DonJayamanne commented Apr 28, 2026

What i did:

  • Updated github copilot version to 1.0.39-0
  • Ran npm run generate from the scripts folder.

Copilot AI review requested due to automatic review settings April 28, 2026 23:53
@DonJayamanne DonJayamanne requested a review from a team as a code owner April 28, 2026 23:53
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

Regenerates SDK-generated artifacts across Python/Node/Go/.NET to reflect updated JSON schemas and CLI package versions, adding new session event shapes and new session-scoped Tasks RPC APIs.

Changes:

  • Add new session event model.call_failure (with ModelCallFailureData/Source) and extend existing session event payloads (rate-limit auto-switch metadata, info tips, model-change cause).
  • Add experimental Tasks session APIs (startAgent, list, promoteToBackground, cancel, remove) and associated task data types across SDKs.
  • Bump Node dependency @github/copilot and refresh generated RPC/session-event type outputs.
Show a summary per file
File Description
python/copilot/generated/session_events.py Adds MODEL_CALL_FAILURE event + new/extended event payload fields.
python/copilot/generated/rpc.py Adds experimental Tasks API/types; extends AgentInfo, Workspace, and tweaks docs.
nodejs/src/generated/session-events.ts Adds ModelCallFailureEvent/Data and extends existing event payload types.
nodejs/src/generated/rpc.ts Adds Tasks RPC group/types; extends AgentInfo and workspace fields.
nodejs/package.json Bumps @github/copilot version range.
nodejs/package-lock.json Updates resolved @github/copilot package and platform optional deps.
go/rpc/generated_rpc.go Adds Tasks RPC client + types; extends AgentInfo/workspace fields.
go/generated_session_events.go Adds model.call_failure event + extends event payloads and enums.
dotnet/src/Generated/SessionEvents.cs Adds ModelCallFailureEvent/Data/Source and extends event payload types.
dotnet/src/Generated/Rpc.cs Adds Tasks RPC APIs/types; extends AgentInfo and workspace fields.

Copilot's findings

Files not reviewed (1)
  • nodejs/package-lock.json: Language not supported
  • Files reviewed: 3/10 changed files
  • Comments generated: 1

Comment thread go/rpc/generated_rpc.go
Comment on lines 6 to 13
import (
"context"
"encoding/json"
"time"
"errors"
"fmt"
"github.com/github/copilot-sdk/go/internal/jsonrpc2"
"time"
)
Copy link

Copilot AI Apr 28, 2026

Choose a reason for hiding this comment

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

The import block isn’t gofmt-sorted (e.g., time appears before errors). CI runs go fmt ./... and fails if it produces changes, so please re-run go fmt ./... in the go/ directory (or ensure the generator output is already gofmt-clean) and commit the formatted result.

See below for a potential fix:

	"errors"
	"fmt"
	"time"

	"github.com/github/copilot-sdk/go/internal/jsonrpc2"
)

type RPCTypes struct {
	AccountGetQuotaRequest                                  AccountGetQuotaRequest                                  `json:"AccountGetQuotaRequest"`
	AccountGetQuotaResult                                   AccountGetQuotaResult                                   `json:"AccountGetQuotaResult"`
	AccountQuotaSnapshot                                    AccountQuotaSnapshot                                    `json:"AccountGetQuotaSnapshot"`

Copilot uses AI. Check for mistakes.
@stephentoub
Copy link
Copy Markdown
Collaborator

Thanks, @DonJayamanne. We have a GitHub Actions that does the update, but it also required some manual intervention to fix up a few things. I've merged updating to 1.0.39-0.

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