Conversation
- Updated nodejs and test harness dependencies - Re-ran code generators - Formatted generated code
|
@copilot is there any documentation or tests or anything that needs to be updated in reaction to these changes? Please ensure everything builds and all tests pass. |
Agent-Logs-Url: https://github.com/github/copilot-sdk/sessions/3afab108-c1a1-4942-9bce-f4c96ce1936a Co-authored-by: stephentoub <[email protected]>
Yes — the generated code renamed
Builds pass for all SDKs. Unit tests pass: Node.js (205 passed, 3 pre-existing vision timeout flakes), Python (87 passed), Go (all passed), .NET (all passed). Go E2E tests also compile successfully with the updated API. Warning Firewall rules blocked me from connecting to one or more addresses (expand for details)I tried to connect to the following addresses, but was blocked by firewall rules:
If you need me to access, download, or install something from one of these locations, you can either:
|
…ilot 1.0.21 Agent-Logs-Url: https://github.com/github/copilot-sdk/sessions/3afab108-c1a1-4942-9bce-f4c96ce1936a Co-authored-by: stephentoub <[email protected]>
This comment has been minimized.
This comment has been minimized.
Agent-Logs-Url: https://github.com/github/copilot-sdk/sessions/2078ab96-88a2-4213-bd0a-a40e7b755bc3 Co-authored-by: stephentoub <[email protected]>
|
Just as a heads up, I was blocked by some firewall rules while working on your feedback. Expand below for details. Warning Firewall rules blocked me from connecting to one or more addresses (expand for details)I tried to connect to the following addresses, but was blocked by firewall rules:
If you need me to access, download, or install something from one of these locations, you can either:
|
Cross-SDK Consistency Review ✅This PR maintains excellent feature parity and consistency across all four SDK implementations. Here's a summary of the changes and their cross-SDK coverage: Changes Reviewed
Observations
No inconsistencies found. 🎉
|
Automated bump of
@github/copilotto 1.0.21. The codegen renamedsession.compaction.compact→session.history.compactand added newsession.history.truncateandsessions.forkRPCs, requiring updates across tests and docs.Generated code
@github/copilotinnodejs/package.jsonandtest/harness/package.jsonE2E test fixes
compaction.compact()tohistory.compact()across Node.js, Python, Go, and .NET E2E testsDocumentation
docs/troubleshooting/compatibility.mdto reflect the rename and added missing entries forsession.history.truncateandsessions.fork