Update @github/copilot to 1.0.56-2#1495
Conversation
- Updated nodejs and test harness dependencies - Re-ran code generators - Formatted generated code
|
@dmytrostruk I see the runtime updates let you read what the model context tier is, but it doesn't look like we still don't pass any context tier option to session.create for any of the SDK languages, so maybe there's no way to control it. Were you expecting that to be added? |
@SteveSandersonMS Let me check this, I thought based on schema from runtime this will be auto-generated or am I missing something? |
|
@SteveSandersonMS I see, |
|
@dmytrostruk Yes, after GA we should move session.create/resume to be generated from schema, but we didn't yet migrate all the original APIs. Adding the parameters to the clients manually is the right thing for now. |
Cross-SDK Consistency ReviewThis PR adds The following Java files need
The pattern follows exactly how
|
| /// Context window tier for models that support it. | ||
| /// Valid values: "default", "long_context". | ||
| /// </summary> | ||
| public string? ContextTier { get; set; } |
There was a problem hiding this comment.
This should be used a strongly-typed string enum, no?
Automated update of
@github/copilotto version1.0.56-2.Changes
@github/copilotinnodejs/package.jsonandtest/harness/package.jsonscripts/codegen)Next steps
When ready, click Ready for review to trigger CI checks.