Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Conversation

xiehuc
Copy link

@xiehuc xiehuc commented Jul 3, 2025

Description

  1. support inject trace id, very helpful to debug with http see mcp servers.
  2. support --verbose flag, very helpful to see trace id, and other header, like curl -v

Fixes # (issue)

Type of change

Please delete options that are not relevant.

  • 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 not work as expected)
  • This change requires a documentation update

How Has This Been Tested?

Please describe the tests that you ran to verify your changes. Provide instructions so we can reproduce.

Checklist:

  • My code follows the style guidelines of this project
  • I have performed a self-review of my own code
  • I have commented my code, particularly in hard-to-understand areas
  • I have made corresponding changes to the documentation
  • My changes generate no new warnings
  • I have added tests that prove my fix is effective or that my feature works
  • New and existing unit tests pass locally with my changes
  • Any dependent changes have been merged and published in downstream modules

@xiehuc
Copy link
Author

xiehuc commented Jul 3, 2025

effect:

企业微信截图_899d9a21-d83a-4033-b731-ac637f38ecc1

@xiehuc
Copy link
Author

xiehuc commented Jul 3, 2025

use
mcptools call --verbose
mcptools tools --verbose
to enable

@f
Copy link
Owner

f commented Jul 6, 2025

It fails on lint. Can you fix them?

@xiehuc
Copy link
Author

xiehuc commented Jul 7, 2025 via email

@f
Copy link
Owner

f commented Jul 7, 2025

Did you fix it or will fix? Seems not passing the tests and the linter.

@xiehuc
Copy link
Author

xiehuc commented Jul 7, 2025 via email

@xiehuc
Copy link
Author

xiehuc commented Jul 8, 2025

@f updated

@xiehuc
Copy link
Author

xiehuc commented Jul 9, 2025

@f fixed,

sorry for not familar with github action.

this time i ran golangci-lint locally, i can make sure, this time can pass workflow

@f
Copy link
Owner

f commented Jul 17, 2025

can you resolve conflicts?

@xiehuc
Copy link
Author

xiehuc commented Jul 17, 2025

@f fixed

@xiehuc xiehuc closed this Aug 11, 2025
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