github / github/github-mcp-server

Feature: per-toolset (or per-tool) read-only mode instead of global GITHUB_READ_ONLY

オープン
#3,229 コメント 1 件 リアクション 0 件 担当者 0 名 GitHub で見る
enhancement policies & governance
主要言語
Go
スター
33k
フォーク
5k
平均マージ
2日 1時間
マージ済み PR(30日)
52

説明

### Feature request

`GITHUB_READ_ONLY` is currently all-or-nothing: when set, the whole server rejects every write operation. There is no way to keep some domains writable while others stay read-only.

### Use case

Running the server as a local coding-agent tool with a single GitHub token, I want a common "safe by default" posture:

- Reads everywhere (repos, issues, pull requests) allowed without human confirmation
- Writes (merge PR, close/label issues, push comments) gated behind explicit user approval

Today the only way to approximate this is to launch **two full server instances** — one with `GITHUB_READ_ONLY=1` and one without — and rely on agent-side conventions to route writes to the second instance. That is fragile because nothing on the server side prevents an agent from calling write tools on the writable instance, and it doubles the tool surface / process count.

### Proposed solution

Any of the following would fix it:

1. Per-toolset read-only, e.g. `GITHUB_READ_ONLY_TOOLSETS=issues,pull_requests` (writes rejected only for the listed toolsets), or
2. Per-tool read-only overrides, e.g. a `GITHUB_READ_ONLY_TOOLS=merge_pull_request,create_issue` deny list, or
3. A "write-confirm" layer that rejects write tools unless an opt-in env var for that specific call is present.

Option 1 seems the most consistent with the existing `GITHUB_TOOLSETS` design.

### Alternatives considered

- Token scoping (fine-grained PAT without write scopes): does not help, because the same token is also expected to perform approved writes.
- Dual-instance setup: works only as a convention, not enforcement (described above).

コントリビューションガイド

コントリビューションガイドを開く

調査の方向性

Start by tracing how the server handles GITHUB_READ_ONLY and the existing GITHUB_TOOLSETS configuration, then identify where write tools are dispatched. Compare the proposed per-toolset and per-tool scopes with current configuration behavior. Done means selected write operations are rejected while reads and explicitly permitted writes continue to work, with coverage for the chosen configuration.

索引モデルが issue の本文から書いたものです。

評価

技術スタック
github, go
領域
backend-api-design, security
issue の種類
機能追加
難易度
5/5
見積もり時間
1週間以上
活発さ
活発
明瞭さ
おおむね明確
初心者へのやさしさ
42/100

新しい issue をメールで受け取る

初心者向けの GitHub issue を短くまとめたダイジェスト。