Skip to content

Update @github/copilot to 0.0.405#396

Merged
devm33 merged 1 commit intomainfrom
devm33/update
Feb 7, 2026
Merged

Update @github/copilot to 0.0.405#396
devm33 merged 1 commit intomainfrom
devm33/update

Conversation

@devm33
Copy link
Member

@devm33 devm33 commented Feb 7, 2026

No description provided.

@devm33 devm33 requested a review from a team as a code owner February 7, 2026 02:53
Copilot AI review requested due to automatic review settings February 7, 2026 02:53
Copy link
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

Updates the Node.js SDK’s @github/copilot dependency to 0.0.405 and regenerates session event type definitions across the language SDKs to reflect the updated schema (notably adding requestId on session.compaction_complete data).

Changes:

  • Bump Node dependency @github/copilot from ^0.0.403 to ^0.0.405.
  • Regenerate session event types for Node.js, Python, Go, and .NET.
  • Add requestId to the generated type shapes for session.compaction_complete event payloads.

Reviewed changes

Copilot reviewed 2 out of 6 changed files in this pull request and generated 1 comment.

Show a summary per file
File Description
python/copilot/generated/session_events.py Regenerated Python session event model; adds request_id field and (de)serialization support.
nodejs/src/generated/session-events.ts Regenerated TS union types; adds optional requestId to session.compaction_complete data.
nodejs/package.json Updates @github/copilot dependency version to ^0.0.405.
nodejs/package-lock.json Updates lockfile to @github/copilot@0.0.405 and records engines.node as >=24.0.0.
go/generated_session_events.go Regenerated Go Data struct; adds RequestID field.
dotnet/src/Generated/SessionEvents.cs Regenerated C# models; adds RequestId property to SessionCompactionCompleteData.
Files not reviewed (1)
  • nodejs/package-lock.json: Language not supported

Comment on lines 33 to 35
"engines": {
"node": ">=18.0.0"
"node": ">=24.0.0"
}
Copy link

Copilot AI Feb 7, 2026

Choose a reason for hiding this comment

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

The lockfile now records the root package engines.node as >=24.0.0 (was >=18.0.0). This effectively raises the minimum Node version for nodejs/ installs and will be inconsistent with repo workflows still using Node 22 (e.g. docs validation / publish), which can cause engine warnings or failures depending on npm config. Either revert this engine bump (and regenerate the lockfile) or update the affected CI/workflows to use Node 24+ consistently.

Copilot uses AI. Check for mistakes.
@github-actions
Copy link

github-actions bot commented Feb 7, 2026

✅ Cross-SDK Consistency Review

This PR maintains excellent cross-language consistency. The update to @github/copilot version 0.0.405 has been properly propagated across all four SDK implementations:

Changes Summary

  • Node.js/TypeScript: Added requestId?: string field to SessionCompactionCompleteData
  • Python: Added request_id: Optional[str] field to Data class
  • Go: Added RequestID *string field to Data struct
  • .NET: Added RequestId property to SessionCompactionCompleteData class

Consistency Analysis ✅

API Naming: All four implementations correctly follow their language-specific naming conventions:

  • TypeScript: requestId (camelCase) ✓
  • Python: request_id (snake_case) ✓
  • Go: RequestID (PascalCase for JSON tag requestId) ✓
  • .NET: RequestId (PascalCase) ✓

Type Mapping: All implementations correctly map to their language's optional string type:

  • TypeScript: string | undefined
  • Python: Optional[str]
  • Go: *string (pointer for optionality) ✓
  • .NET: string? (nullable reference type) ✓

Serialization: All implementations use the same JSON property name (requestId) and handle null/undefined values appropriately.

Conclusion

This is an automated schema update that correctly maintains feature parity across all SDK implementations. The generated code follows established patterns and conventions for each language. No manual consistency fixes are needed.


Generated by SDK Consistency Review Agent

AI generated by SDK Consistency Review Agent

Copy link
Contributor

@jmoseley jmoseley left a comment

Choose a reason for hiding this comment

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

Where do you change log entries come from the SDK? It'd be nice to have the addition of the request ID field recorded in the change log

@devm33
Copy link
Member Author

devm33 commented Feb 7, 2026

Where do you change log entries come from the SDK? It'd be nice to have the addition of the request ID field recorded in the change log

I don't think there is a changelog for the sdk. Just the PR titles on the releases.

@devm33 devm33 added this pull request to the merge queue Feb 7, 2026
Merged via the queue into main with commit 2186bf2 Feb 7, 2026
40 checks passed
@devm33 devm33 deleted the devm33/update branch February 7, 2026 16:11
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