FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

Log exception before returning tool call error result by fabianlange · Pull Request #3267 · modelcontextprotocol/python-sdk · GitHub

/ python-sdk Public

Log exception before returning tool call error result - #3267

Closed
fabianlange wants to merge 1 commit into
modelcontextprotocol:mainfrom
fabianlange:log-tool-call-exception
ConversationCommits (1)ChecksFiles changed
Closed

Log exception before returning tool call error result#3267
fabianlange wants to merge 1 commit into
modelcontextprotocol:mainfrom
fabianlange:log-tool-call-exception

Conversation

fabianlange commented Aug 7, 2026 •
edited
Loading

Copy link
Copy Markdown

Fixes #3266

Motivation and Context

How Has This Been Tested?

Breaking Changes

Types of changes

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)
  • Documentation update

Checklist

  • I have read the MCP Documentation
  • My code follows the repository's style guidelines
  • New and existing tests pass locally
  • I have added appropriate error handling
  • I have added or updated documentation as needed

Additional context

cubic-dev-ai Bot reviewed Aug 7, 2026
View reviewed changes

cubic-dev-ai Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Choose a reason Spam Abuse Off Topic Outdated Duplicate Resolved Low Quality

No issues found across 1 file

Re-trigger cubic

This was referenced Aug 11, 2026
Closed
Merged
github-actions Bot added the missing-issue-link Auto-closed: PR needs a linked issue assigned to its author (see CONTRIBUTING.md) label Aug 17, 2026

github-actions Bot commented Aug 17, 2026

Copy link
Copy Markdown
Contributor

Thanks for the contribution. This repository only keeps pull requests open when they're linked to an issue that a maintainer has assigned to the author — CONTRIBUTING.md explains why and how we work. This PR has been closed for now because you aren't currently assigned to #3266.

If a maintainer would like this change as a PR from you, they'll assign you to #3266 and this PR will reopen automatically — there's nothing more you need to do. (If you opened the issue, this PR already shows up on its timeline.)

There's no need to open a new PR — this one will be reopened. While it's closed, please push any updates as new commits rather than force-pushing, since GitHub can't reopen a PR whose branch has been rewritten.

Maintainers: reopening this PR, removing the missing-issue-link label, or adding bypass-issue-check bypasses the check.

github-actions Bot closed this Aug 17, 2026
niklasgruener mentioned this pull request Aug 20, 2026
Draft
9 tasks
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
Show hidden characters
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

cubic-dev-ai[bot] cubic-dev-ai[bot] left review comments

Assignees

No one assigned

Labels

missing-issue-link Auto-closed: PR needs a linked issue assigned to its author (see CONTRIBUTING.md)

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Log exceptions in tool calls

1 participant

Back | FazBrowse Home | New Git URL