Repository navigation
fix(managed): remove deprecated Skill update API - #11
Merged
Merged
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Change
Remove the deprecated Managed
PUT /skills/{skill_id}wrapper from the unpublished Java 0.2.0 candidate. Delete synchronous, asynchronous andwithResponse()update overloads andmanaged.SkillUpdateParams; adjust fixtures and the coverage inventory to 107 operations.Managed Skill content updates use
skills().versions().create(...). Version creation does not replace Skill metadata updates. This removal is Java-specific; Forward Skill APIs and other SDKs are unchanged.Validation
make qualityon JDK 17: format, repository checks and 38 maintenance tests passed.make teston JDK 8 and JDK 17: each passed 1,239 SDK tests and 75 independent JAR consumer tests, with no failures or skips.updateand the removed parameter class is absent from the JAR.No live requests were made for this removal. GitHub's JDK 11/21/25 matrix has not run locally. After merge, regenerate the 0.2.0 QA package from the merged SHA; earlier QA packages do not contain this change.