README: GITHUB_TOOLSETS doesn't take precedence over --toolsets (viper gives the flag priority)
Maintainer thường phản hồi trong vòng 2 ngày
Đánh giá
- Độ khó
- 1/5
- Thời gian dự kiến
- Dưới một giờ
- Mức phù hợp với người mới
- 90/100
- Loại issue
- Tài liệu
- Độ rõ ràng
- Đặc tả rõ ràng
- Mức độ hoạt động
- Sôi nổi
- Công nghệ
- go
- Lĩnh vực
- documentation
Hướng nghiên cứu
Đọc README.md khoảng dòng 468 và các binding viper trong cmd/github-mcp-server/main.go (các dòng 87-90, 280, 313-317) để xác nhận thứ tự ưu tiên flag. Sửa câu trong README để nêu rõ rằng --toolsets ghi đè GITHUB_TOOLSETS. Hoàn thành khi docs khớp với thứ tự lookup của viper.
Do mô hình lập chỉ mục viết ra từ nội dung của issue.
Mô tả
The README says:
"The environment variable GITHUB_TOOLSETS takes precedence over the command line argument if both are provided."
https://github.com/github/github-mcp-server/blob/f38526264abb14961b32cea386e38ec6d3cf674f/README.md?plain=1#L468
--toolsets is bound through viper, and the env is wired with AutomaticEnv:
https://github.com/github/github-mcp-server/blob/f38526264abb14961b32cea386e38ec6d3cf674f/cmd/github-mcp-server/main.go#L280
https://github.com/github/github-mcp-server/blob/f38526264abb14961b32cea386e38ec6d3cf674f/cmd/github-mcp-server/main.go#L313-L317
https://github.com/github/github-mcp-server/blob/f38526264abb14961b32cea386e38ec6d3cf674f/cmd/github-mcp-server/main.go#L87-L90
Viper's lookup order is Set() > flag (if changed) > env > config > default, so when both are given the flag wins, the opposite of the README. That matters a bit for anyone trying to narrow toolsets via env on top of a launcher that already passes --toolsets. Probably just the README sentence that needs flipping.
Found while testing a docs-vs-code checker I'm building (Elliot), then verified by hand.
- Ngôn ngữ chính
- Go
- Star
- 33.8k
- Fork
- 5.2k
- Merge trung bình
- 2 ngày 18 giờ
- Pull request đã merge (30 ngày)
- 38
Chuẩn bị môi trường
- Có Dockerfile hoặc tệp Docker Compose
- Có mẫu pull request
- Đọc hướng dẫn đóng góp
Bắt đầu từ đâu
- Đọc hết issue, rồi đọc hướng dẫn đóng góp của dự án.
- Bình luận trên issue rằng bạn sẽ nhận — tránh hai người làm cùng một việc.
- Fork repository và làm thay đổi trên một nhánh.
- Mở pull request có tham chiếu số hiệu của issue.
Issue khác của github/github-mcp-server
-
Repository resource completions hide branch, tag, and commit lookup errorsCó thể đã có người làm Có pull request liên kết đang mở hoặc đã được merge. Đang mở
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 75/100
github/github-mcp-server#3581 ·
Maintainer thường phản hồi trong vòng 2 ngày
-
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 85/100
github/github-mcp-server#3572 ·
Maintainer thường phản hồi trong vòng 2 ngày
-
Add Zero-Docker quick start and clarify no‑Docker options in READMECó thể đã có người làm @Flanxiyum001 đã nhận 12 ngày trước. Đang mởdocumentation
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 68/100
github/github-mcp-server#3495 ·
Maintainer thường phản hồi trong vòng 2 ngày
-
Integer parameters equal to 2^63 pass the range check in toInt and wrap to a negative numberCó thể đã có người làm @xiao115255 đã nhận 12 ngày trước. Đang mởbug
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 82/100
github/github-mcp-server#3489 ·
Maintainer thường phản hồi trong vòng 2 ngày
-
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 78/100
github/github-mcp-server#3475 ·
Maintainer thường phản hồi trong vòng 2 ngày
Tất cả issue của github/github-mcp-server
Issue tương tự
-
fix(handlers): explicitly set Content-Type header in static Index handlerCó thể đã có người làm @kage-zero đã nhận hôm nay. Đang mở
Độ khó 1/5 Dưới một giờ Mức phù hợp với người mới 85/100
Maintainer thường phản hồi trong vòng 1 ngày
-
Quick Open: restore dependency-directory exclusions in filesystem fallbackCó thể đã có người làm Có pull request liên kết đang mở hoặc đã được merge. Đang mở
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 82/100
eugenioenko/ttt#1206 ·
Maintainer thường phản hồi trong vòng 1 ngày
-
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 68/100
open-policy-agent/opa#9426 ·
Maintainer thường phản hồi trong vòng 1 ngày
-
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 72/100
rorkai/App-Store-Connect-CLI#3192 ·
Maintainer thường phản hồi trong vòng 1 ngày
-
t:urlDecodeUni does not map U+3002 and U+FF61 to a dot, unlike ModSecurityCó thể đã có người làm @fzipi đã nhận hôm nay. Đang mởbug
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 78/100
Maintainer thường phản hồi trong vòng 1 ngày