Prompt for `chroma_query_documents` and or similar needs clarification about `include` argument.

[calbrecht]

It reads

Optional list of what to include in response. Can contain any of: ["documents", "embeddings", "metadatas", "distances"]

and model thinks

{
  "collection_name": "personal_memories",
  "query_texts": ["coding style feedback"],
  "n_results": 1
}

and server responses

Error:
Error executing tool chroma_query_documents: Failed to query documents from collection 'personal_memories': Expected include to be a list, got None in query.

[calbrecht]

And given the opportunity to add a comment with suggestions to this ticket, model enthusiastically responds with:

Suggested improvements for the chroma_query_documents tool:

1. Update the parameter description to clearly indicate that 'include' is required:
   Current: "Optional list of what to include in response. Can contain any of: ["documents", "embeddings", "metadatas", "distances"]"
   Suggested: "Required list of what to include in response. Must be a non-empty list containing any of: ["documents", "embeddings", "metadatas", "distances"]"

2. Improve the error message to be more helpful:
   Current: "Expected include to be a list, got None in query."
   Suggested: "The 'include' parameter is required and must be a non-empty list. Please specify which elements to include in the response, e.g., include=["documents", "metadatas"]"

3. Alternatively, make the parameter truly optional by providing a default value in the implementation (e.g., default to ["documents"] if not specified).

These changes would align the documentation with the implementation and provide clearer guidance to users.

[jairad26]

thanks, will fix and release this

[jairad26]

released in v0.2.2