Skip to content

Use structs from updated SDK #886

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 1 commit into from
Aug 14, 2025
Merged

Use structs from updated SDK #886

merged 1 commit into from
Aug 14, 2025

Conversation

almaleksia
Copy link
Contributor

@almaleksia almaleksia commented Aug 14, 2025

use server.ServerResourceTemplate and server.ServerPrompt from SDK and remove previous ServerResourceTemplate and ServerPrompt wrappers.

Base automatically changed from bump-sdk-0.36.0 to main August 14, 2025 15:31
@almaleksia almaleksia marked this pull request as ready for review August 14, 2025 15:35
@almaleksia almaleksia requested a review from a team as a code owner August 14, 2025 15:35
@Copilot Copilot AI review requested due to automatic review settings August 14, 2025 15:35
Copy link
Contributor

@Copilot Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

This PR refactors the toolsets package to use structs from the updated SDK instead of local wrapper types. It removes custom ServerResourceTemplate and ServerPrompt wrapper structs and replaces them with the equivalent types from the server package.

Key changes:

  • Removed local ServerResourceTemplate and ServerPrompt struct definitions
  • Updated function signatures and struct fields to use server.ServerResourceTemplate and server.ServerPrompt
  • Updated field access patterns to match the new struct field names (Template and Handler instead of resourceTemplate and handler)

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.
You can also share your feedback on Copilot code review for a chance to win a $100 gift card. Take the survey.

@almaleksia almaleksia merged commit 587272b into main Aug 14, 2025
17 checks passed
@almaleksia almaleksia deleted the use-sdk-structs branch August 14, 2025 15:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants