Skip to content

feat: Improve context parameter with detailed instructions #8

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 3 commits into from
Nov 6, 2025
Merged

feat: Improve context parameter with detailed instructions #8

merged 3 commits into from
Nov 6, 2025

Conversation

@naji247
Copy link
Member

@naji247 naji247 commented Nov 6, 2025

Summary

  • Enhanced the context parameter description with comprehensive instructions for LLMs
  • Added word count requirement (15-25 words)
  • Enforced third-person perspective to avoid first-person pronouns
  • Added explicit warning against including sensitive information
  • Provided example to demonstrate expected format
  • Updated package.json to v0.1.7
  • Fixed CommonJS export path from .js to .cjs

Changes

  • constants.ts: Replaced simple context description with detailed multi-line instructions
  • tests: Updated tests to reference the constant instead of hardcoding descriptions
  • package.json: Version bump to 0.1.7 and corrected require export path

Test plan

  • All existing tests pass
  • Context parameter tests updated to use constant
  • Verified backward compatibility with existing tools

@naji247
feat: improve context param
7f98c3c
@naji247
v0.1.7
d8a7e10
@naji247
fix: handle MCP SDK 1.21.0 error response behavior change
973176f 
MCP SDK 1.21.0 introduced a breaking change where validation errors
now return CallToolResult with isError: true instead of throwing
exceptions. This aligns with the MCP specification.

Updated the "should fail when context parameter is missing" test to
handle both behaviors for backward compatibility:
- SDK <1.21.0: throws exceptions (caught in try/catch)
- SDK >=1.21.0: returns error responses (checks isError: true)

This ensures compatibility across all supported MCP SDK versions
(1.11.5 through 1.21.0+).
@naji247 naji247 requested a review from kashishhora November 6, 2025 17:22
@naji247 naji247 merged commit 4b64244 into main Nov 6, 2025
9 checks passed
@naji247 naji247 deleted the feat/improve-context branch November 6, 2025 17:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@kashishhora kashishhora kashishhora approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@naji247 @kashishhora