Skip to content

Conversation

@kpavlov
Copy link
Contributor

@kpavlov kpavlov commented Dec 3, 2025

Update samples to use the new schema

Refactor samples to use kotlin-sdk.types, update integration tests to assert meta is null, and adjust coroutine logic.

Motivation and Context

Get rid of old types

How Has This Been Tested?

Integration tests, CI

Breaking Changes

No

Types of changes

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)
  • Documentation update
  • Sample update

Checklist

  • I have read the MCP Documentation
  • My code follows the repository's style guidelines
  • New and existing tests pass locally
  • I have added appropriate error handling
  • I have added or updated documentation as needed

Additional context

@kpavlov kpavlov added the samples label Dec 3, 2025
@kpavlov kpavlov marked this pull request as ready for review December 4, 2025 05:07
Refactor samples to use `kotlin-sdk.types`, update integration tests to assert `meta` is null, and adjust coroutine logic.
@kpavlov kpavlov force-pushed the kpavlov/update-api-in-samples branch from db3977f to 1159332 Compare December 4, 2025 07:19
@github-actions
Copy link

github-actions bot commented Dec 4, 2025

TestsPassed ✅Skipped ⚠️FailedTime ⏱
JUnit Test Report3399 ran3310 passed89 skipped3m 54s 638ms
TestResultTime ⏱
No test annotations available

@kpavlov kpavlov requested a review from sdubov December 4, 2025 07:46
@kpavlov kpavlov merged commit bb8de71 into main Dec 4, 2025
8 checks passed
@kpavlov kpavlov deleted the kpavlov/update-api-in-samples branch December 4, 2025 08:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants