Skip to content

[to #12345678] docs: update excludeTools documentation in extensions … - #904

Merged
pomelo-nwu merged 1 commit into
QwenLM:mainfrom
Willam2004:fix/docs
Oct 29, 2025
Merged

[to #12345678] docs: update excludeTools documentation in extensions …#904
pomelo-nwu merged 1 commit into
QwenLM:mainfrom
Willam2004:fix/docs

Conversation

@Willam2004

Copy link
Copy Markdown
Contributor

…guide

  • Added clarification that tools specified in excludeTools will be disabled for the entire conversation context
  • Added note that excludeTools configuration affects all subsequent queries in the current session

This change improves documentation clarity for extension developers by better explaining the scope and impact of the excludeTools configuration.

TLDR

This PR updates the documentation to clarify that the excludeTools setting applies globally across the entire conversation session—not just the current turn. This helps extension developers avoid unexpected behavior when configuring tool exclusions.

Dive Deeper

Previously, the documentation did not explicitly state that excludeTools has a session-wide effect. Developers might have assumed the exclusion was limited to a single query or turn, leading to confusion or bugs. This update adds two concise sentences to make the behavior unambiguous, aligning the docs with the actual runtime behavior.

No code changes—only documentation improvements in the developer guide.

Reviewer Test Plan

Since this is a documentation-only change:

  • Reviewers should verify that the added sentences are clear, accurate, and consistent with the system’s actual behavior.
  • Confirm that the wording matches the project’s documentation style (e.g., tone, terminology).
  • No runtime testing is required, but reviewers may optionally build the docs site (if applicable) to ensure formatting renders correctly.

Testing Matrix

🍏 🪟 🐧
npm run
npx N/A N/A N/A
Docker N/A N/A N/A
Podman - - -
Seatbelt - - -

✅ = Verified that docs build successfully and changes appear as expected
N/A = Not applicable for documentation-only changes

Linked issues / bugs

…guide

- Added clarification that tools specified in excludeTools will be disabled for the entire conversation context
- Added note that excludeTools configuration affects all subsequent queries in the current session

This change improves documentation clarity for extension developers by better explaining the scope and impact of the excludeTools configuration.

Co-authored-by: Qwen-Coder <qwen-coder@alibabacloud.com>
@pomelo-nwu
pomelo-nwu self-requested a review October 29, 2025 06:02

@pomelo-nwu pomelo-nwu left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@pomelo-nwu

Copy link
Copy Markdown
Collaborator

@Willam2004 Thanks for your contribution!

@pomelo-nwu
pomelo-nwu merged commit 84957bb into QwenLM:main Oct 29, 2025
1 check passed
xaelistic pushed a commit to xaelistic/qwen-code that referenced this pull request Jun 7, 2026
[to #12345678] docs: update excludeTools documentation in extensions …
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