Skip to content

fix: consider prefix in the query tool description for describe - #72

Open
stefanrudi wants to merge 6 commits into
mainfrom
fix/prefix-query
Open

fix: consider prefix in the query tool description for describe#72
stefanrudi wants to merge 6 commits into
mainfrom
fix/prefix-query

Conversation

@stefanrudi

@stefanrudi stefanrudi commented Aug 27, 2026

Copy link
Copy Markdown
Contributor

Copy of #70

@stefanrudi
stefanrudi marked this pull request as ready for review August 27, 2026 14:44
@stefanrudi
stefanrudi requested review from a team as code owners August 27, 2026 14:44
@hyperspace-pr-bot

Copy link
Copy Markdown

Summary

The following content is AI-generated and provides a summary of the pull request:


fix: Consider Prefix in query Tool Description for describe Reference

Description

This fix ensures that when tools are registered with a prefix (e.g., CatalogService-query), the query tool's description correctly references the prefixed describe tool (e.g., CatalogService-describe) instead of the bare describe tool name.

Changes

  • lib/tools/query.js: Updated both CQN and default tool description strings to include the prefix variable when referencing the describe tool. Also cleaned up a minor formatting inconsistency (missing space before "Ensure").
  • tests/integration/prefix.test.js: Added a new test case to verify that the query tool description includes the correct prefixed describe tool reference (CatalogService-describe).
  • tests/integration/__snapshots__/catalog-service-card.json: Updated snapshot to reflect the corrected description (fixed missing space, removed backtick formatting around describe).
  • CHANGELOG.md: Added entry for version 1.4.4 documenting this fix.

Category

🐛 Bug Fix

Have you...

  • Added relevant entry to the change log?

Related: #70


  • 🔄 Regenerate and Update Summary
  • ✏️ Insert as PR Description (deletes this comment)
  • 🗑️ Delete comment
PR Bot Information

Version: 1.29.54

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