Skip to content

Fix/get prompt description #640

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

Draft
wants to merge 16 commits into
base: main
Choose a base branch
from

Conversation

Vinisha-projects
Copy link

Fixed the issue with get_prompt() not returning the correct description. The function was modified to ensure the prompt description is correctly fetched and returned.

Motivation and Context

Why is this change needed?

The get_prompt() method was not properly retrieving the description associated with the prompt. This change fixes that behavior, ensuring that the description is returned as expected.

What problem does it solve?

This resolves the issue where the description for certain prompts was missing or not properly accessible.

How Has This Been Tested?

Yes, I tested the get_prompt() method locally using unit tests to ensure that the description is returned correctly.

I also ran the existing tests in test_prompts.py to confirm that no other functionality was broken.

Breaking Changes

No breaking changes were introduced. This is a non-breaking bug fix that addresses the behavior of get_prompt().

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

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

This change only affects the behavior of get_prompt() and is tested with the current version of the codebase. If any further adjustments are required, we will address them based on feedback.

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.

1 participant