Movatterモバイル変換


[0]ホーム

URL:


Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Sign up
Appearance settings

fix: addedreason argument toget_me function#8

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to ourterms of service andprivacy statement. We’ll occasionally send you account related emails.

Already on GitHub?Sign in to your account

Merged
toby merged 1 commit intomainfromno-args-fix
Mar 14, 2025
Merged

Conversation

toby
Copy link
Member

Themark3labs/mcp-go library has a bug where it will error when there are no arguments specified for a tool. This adds areason argument, which works around that bug but is also somewhat helpful for debugging and potentially even returns better results because the LLM can see why it decided to call the tool.

@CopilotCopilotAI review requested due to automatic review settingsMarch 13, 2025 22:46
Copy link
Contributor

@CopilotCopilotAI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This pull request addresses a bug in the mark3labs/mcp-go library by adding a new "reason" argument to the get_me tool to avoid errors when no arguments are supplied and to provide additional context for debugging.

  • Added a "reason" argument with an accompanying description in the tool configuration.
  • Updated the tool initialization in pkg/github/server.go to include the new argument.

Tip: Copilot only keeps its highest confidence comments to reduce noise and keep you focused.Learn more

@tobytoby merged commit62b20f7 intomainMar 14, 2025
3 checks passed
@tobytoby deleted the no-args-fix branchMarch 14, 2025 16:19
festion referenced this pull request in festion/github-mcp-serverJul 5, 2025
Implements 18 new MCP tools for managing GitHub Projects API v2:Project Board Management (5 tools):- create_project_board: Create new project boards with templates- update_project_board: Update board settings and metadata- delete_project_board: Delete/archive project boards- list_project_boards: List accessible project boards- get_project_board: Get detailed board info and statisticsColumn Management (6 tools):- create_project_column: Create new columns (as Status field options)- update_project_column: Update column properties- delete_project_column: Delete columns with card archival- reorder_project_columns: Change column order- list_project_columns: List all columns in a board- get_project_column: Get column details and statisticsCard Operations (7 tools):- add_card_to_project: Add issues/PRs to project boards- move_project_card: Move cards between columns- update_project_card: Update card custom fields- remove_card_from_project: Remove/archive cards- bulk_move_cards: Bulk card operations- list_project_cards: List cards with filtering (Phase 2)- get_project_card: Get card details (Phase 2)Technical details:- Uses GitHub GraphQL API v4 for all operations- Follows existing patterns with proper error handling- Includes snapshot tests for all tool definitions- Maintains read/write tool separation for permissionsResolves#12 (Phase 1)Related to Epic#8🤖 Generated with [Claude Code](https://claude.ai/code)Co-Authored-By: Claude <noreply@anthropic.com>
Sign up for freeto join this conversation on GitHub. Already have an account?Sign in to comment
Reviewers

Copilot code reviewCopilotCopilot left review comments

@juruenjuruenjuruen approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

2 participants
@toby@juruen

[8]ページ先頭

©2009-2025 Movatter.jp