Update @github/copilot to 1.0.51-1

[github-actions]

Automated update of @github/copilot to version 1.0.51-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.

[Copilot]

Pull request overview

Automated dependency/schema refresh for @github/copilot 1.0.51-1, updating package locks and regenerated SDK bindings across languages.

Changes:

• Bumped @github/copilot dependency metadata in Node.js packages and harness locks.
• Regenerated session-event types across Node.js, Python, Go, Rust, and .NET.
• Added/relocated generated Go JSON union encoding helpers for new/updated RPC schema shapes.

Show a summary per file

File	Description
test/harness/package.json	Updates harness @github/copilot dev dependency.
test/harness/package-lock.json	Locks harness Copilot packages to 1.0.51-1.
rust/src/generated/session_events.rs	Regenerates Rust session-event data types and docs.
python/copilot/generated/session_events.py	Regenerates Python session-event dataclasses and serializers.
nodejs/src/generated/session-events.ts	Updates generated TypeScript session-event comments.
nodejs/samples/package-lock.json	Updates sample lockfile dependency resolution.
nodejs/package.json	Updates Node SDK @github/copilot dependency.
nodejs/package-lock.json	Locks Node SDK Copilot packages to 1.0.51-1.
go/zsession_events.go	Updates top-level Go aliases for generated session events.
go/rpc/zsession_events.go	Regenerates Go RPC session-event types.
go/rpc/zsession_encoding.go	Removes relocated session-event JSON encoding helpers.
go/rpc/zrpc_encoding.go	Adds generated Go RPC union JSON encoding helpers.
dotnet/src/Generated/SessionEvents.cs	Regenerates .NET session-event types, validation attributes, and docs.

Copilot's findings

Files not reviewed (7)

• go/rpc/zrpc_encoding.go: Language not supported
• go/rpc/zsession_encoding.go: Language not supported
• go/rpc/zsession_events.go: Language not supported
• go/zsession_events.go: Language not supported
• 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: 2/19 changed files
• Comments generated: 2

[github-actions]

Cross-SDK Consistency Review

This is an automated dependency update PR that re-generates code for all 5 SDK implementations (Node.js, Python, Go, .NET, Rust) from an updated @github/copilot schema — so the generated files are expected to be consistent by design.

I did notice one cross-SDK inconsistency worth flagging in the manual (non-generated) changes:

PingResponse.timestamp type — inconsistent across SDKs

The timestamp field changed from an integer (Unix epoch ms) to an ISO 8601 string on the wire. Three SDKs parse this into their language's native datetime type, while two leave it as a raw string:

SDK	New timestamp type
Python	datetime ✅ (parsed)
Go	time.Time ✅ (parsed)
.NET	DateTimeOffset ✅ (parsed)
Rust	String ⚠️ (raw wire string)
Node.js	string ⚠️ (raw wire string)

Suggestion: For consistency, consider whether:

• Node.js (nodejs/src/client.ts) should return Date instead of string — or at minimum provide a helper/note in JSDoc that it's an ISO 8601 string. TypeScript/JavaScript has Date and widely-used ISO 8601 string conventions.
• Rust (rust/src/types.rs) should use a datetime crate type (e.g., chrono::DateTime<Utc> or time::OffsetDateTime). That said, adding a new transitive dependency just for PingResponse (a diagnostic-only method) may not be worth it — so a String here is understandable.

This is a minor inconsistency in a diagnostic API (ping), so it's not blocking. Just worth aligning intentionally either way.

---

All other changes look consistent across SDKs:

• The SQLite last_insert_rowid float→int fix is applied in all relevant SDKs (Python, Go, Rust; Node.js didn't have the issue)
• The duration/seconds property handling is consistently updated in the codegen scripts
• The session log API change in .NET is a defensive named-argument refactor with no public API impact

Generated by SDK Consistency Review Agent for issue #1340 · ● 732.3K · ◷