Skip to content

test#3

Open
MelvinKl wants to merge 10 commits into
mainfrom
ai/issue-2-mcp-server-should-not-create-s
Open

test#3
MelvinKl wants to merge 10 commits into
mainfrom
ai/issue-2-mcp-server-should-not-create-s

Conversation

@MelvinKl

Copy link
Copy Markdown
Owner

Thank you for contributing to the RAG Core Library!

Please ensure your PR meets the following requirements:

  • PR Title: Follow the format "type: description"

  • PR Description: Replace this checklist with:

    • Description: Provide a detailed description of the changes made.
    • Issue: Mention the issue number this PR fixes, if applicable.
    • Dependencies: List any dependencies required for this change.

Additional Guidelines:

  • Ensure your code follows established coding conventions
  • Include relevant tests and documentation updates.
  • If no one reviews your PR within a few days, please @-mention a-klos.

Thank you for your contribution!

MelvinKl and others added 9 commits April 2, 2026 10:34
The OpenAPI client's ChatRequest was missing the skip_answer_generation
field, so the MCP server's flag was silently ignored during serialization
and never sent to the backend. This adds the field to the OpenAPI spec,
both ChatRequest models (core API and client), and updates documentation
to reflect that the MCP server returns only citations.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
…eneration parameter.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
…server returns only citations without generating answers.
…tion-only responses.

Fix skip_answer_generation path in chat_graph to properly merge node results
into the state instead of replacing it. Add REST API test verifying that
skip_answer_generation=True returns empty answer with populated citations.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
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