Nightly security sweep: turn Dependabot and code-scanning (CodeQL) alerts into board-tracked issues
メンテナーはふだん 1 日以内に返信
まだ誰も着手していません。
評価
- 難易度
- 4/5
- 見積もり時間
- 3〜5日
- 初心者へのやさしさ
- 48/100
- issue の種類
- 機能追加
- 明瞭さ
- おおむね明確
- 活発さ
- 活発
- 技術スタック
- github, github-actions, javascript
調査の方向性
Start with scripts/dependabot-alerts.mjs, dependabot-alerts.yml, and the relevant rules in AGENTS.md; compare the existing Dependabot sweep with the CodeQL API and workflow permissions described here. Check the credential dependency in #2544 and the v2/main scanning split in #2545 before implementation. Done means nightly, deduplicated Dependabot and code-scanning findings are filed and boarded, while malformed listings write nothing.
索引モデルが issue の本文から書いたものです。
説明
Problem
Security alerts reach this repo from two GitHub sources, and only one of them becomes tracked work:
| Source | What produces it | Swept into issues today? |
|---|---|---|
| Dependabot alerts | vulnerable dependency ranges in a lockfile | Yes, daily (dependabot-alerts.yml, #2233). It files issues, but cannot board them: PROJECT_TOKEN does not exist (#2462). |
| Code scanning alerts | CodeQL (default setup), plus any other tool that uploads SARIF | No. Nothing reads them. |
CodeQL is not a third source: it is the tool behind the code-scanning alerts on Security → Code scanning. A sweep reads them all through one API, GET /repos/{owner}/{repo}/code-scanning/alerts, and can split by tool.name.
What that gap cost during the v2.9.0 release:
- A high-severity ReDoS reached the release PR undetected. CodeQL flagged
clients/cli/src/error-handler.ts:169only as a check on the milestone-merge PR (#2536). It was a regression from this milestone's #2423 work, fixed in #2540 / #2541, with the release on hold meanwhile. - Three high-severity alerts sit open with no issue: #74, #75, #76, all
js/incomplete-sanitization("does not escape backslash characters"), inscripts/dependabot-alerts.mjs:435,:679andscripts/sdk-watch.mjs:442. Nothing tracks them, and the code is unchanged onv2/main.
The deeper problem: CodeQL never looks at v2/main
CodeQL runs as default setup (languages: actions, javascript-typescript, schedule: weekly). Default setup analyzes the default branch (main) and PRs that target it. Every analysis on record is refs/heads/main or a PR into main, and the only PRs analyzed are the four milestone merges (#2304, #2381, #2456, #2536).
So no v2 development work is scanned until the milestone merge. A nightly sweep of the alerts cannot fix that on its own: an alert for code on v2/main does not exist until that code reaches a PR against main. It is the same blind spot AGENTS.md already records for Dependabot ("a vulnerable dependency introduced on v2/main and not yet merged to main produces no alert at all"), and for CodeQL it covers every line of first-party code.
Goal
A nightly sweep turns open Dependabot and code-scanning alerts into board-tracked issues, one per actionable finding, with no duplicates, the same way dependabot-alerts.yml does for Dependabot today.
Scanning v2/main itself, so that those alerts exist for v2 code before the release PR, is split out as #2545.
Design sketch
The sweep
Extend scripts/dependabot-alerts.mjs into a security-alert sweep, or add a sibling scripts/code-scanning-alerts.mjs with a shared filing and boarding core. It keeps the existing sweep's invariants:
- Re-check against
v2/mainbefore filing. Dependabot re-checks each vulnerable range againstv2/main's lockfile. For code scanning, the equivalent is to take the alert on thev2/mainref once #2545 makes that ref exist. Until then, check that the alert's file and line still hold the flagged code onv2/main, and skip one already fixed there. - One issue per finding, deduplicated by a marker that encodes tool + rule + alert number, with markers trusted only on automation-authored issues (the
AGENTS.mdrule, since the repo is public). - Board it directly at
Todo/Highonce the credential exists, the same standing override the Dependabot sweep uses. Without the credential, fall back to the labeled, milestoned triage hand-off. - Fail loudly on a bad listing (rate limit, truncated or 5xx response) and write nothing, as #2425 made the Dependabot sweep do.
- Never close an issue. A fixed alert closes itself; closing its issue is a maintainer act.
- No model in any write-capable job. This sweep is deterministic and needs none.
Out of scope unless we choose it: secret scanning (GET …/secret-scanning/alerts). It is a third alert type, would need its own read permission, and its alerts should not be copied into a public issue body. Decide explicitly rather than by omission.
Credential
The GitHub App is now its own issue, #2544, and its permission list there already includes what this sweep needs:
| Permission (GitHub App) | Needed for | Workflow permissions: equivalent |
|---|---|---|
| Repository → Dependabot alerts: read | GET …/dependabot/alerts |
vulnerability-alerts: read |
| Repository → Code scanning alerts: read | GET …/code-scanning/alerts |
security-events: read |
| Repository → Administration: read (optional) | turns the sweep's automated-security-fixes check from UNVERIFIED into a real assertion |
none: GITHUB_TOKEN cannot hold it |
| Repository → Secret scanning alerts: read (only if secret scanning is in scope) | GET …/secret-scanning/alerts |
secret-scanning-alerts: read |
Both reads are available to GITHUB_TOKEN through workflow permissions:, so the listing half does not strictly need the App. Only the board write does. Putting everything on the App still keeps the sweep to one identity with one scope list, so add the two read permissions when the App is created.
Blocked on
#2544 (the GitHub App), for the board write. Part of tracker #2543.
Acceptance
- A nightly sweep files one issue per open, unfixed-on-
v2/mainDependabot and code-scanning alert, deduplicated, and boards it atTodo/Highwhen the credential is present - The sweep fails loudly, writing nothing, on a rate-limited or malformed listing
- The sweep would file today's open alerts; #74, #75 and #76 are being fixed directly in #2546
- 主要言語
- TypeScript
- スター
- 11.1k
- フォーク
- 1.6k
- 平均マージ
- 5時間 29分
- マージ済み PR(30日)
- 158
環境構築
- Dockerfile または Docker Compose ファイルあり
- プルリクエストのテンプレートあり
- コントリビューションガイドを読む
はじめの一歩
- issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
- 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
- リポジトリをフォークし、ブランチを切って変更します。
- issue 番号を参照したプルリクエストを送ります。
modelcontextprotocol/inspector のほかの issue
-
enhancement v2
難易度 2/5 1〜3時間 初心者へのやさしさ 67/100
modelcontextprotocol/inspector#2782 ·
メンテナーはふだん 1 日以内に返信
-
CLI tests seed the developer's real ~/.mcp-inspector/mcp.json (cli.test.ts, completion.test.ts)オープンbug v2
難易度 2/5 1〜3時間 初心者へのやさしさ 76/100
modelcontextprotocol/inspector#2765 ·
メンテナーはふだん 1 日以内に返信
-
chore v2
難易度 2/5 1〜3時間 初心者へのやさしさ 78/100
modelcontextprotocol/inspector#2752 ·
メンテナーはふだん 1 日以内に返信
-
SSE transport: connection succeeds then marked 'Failed' after first event, works fine with raw curlオープン
難易度 4/5 1〜2日 初心者へのやさしさ 53/100
modelcontextprotocol/inspector#2783 ·
メンテナーはふだん 1 日以内に返信
-
enhancement v2
難易度 3/5 1〜2日 初心者へのやさしさ 72/100
modelcontextprotocol/inspector#2772 ·
メンテナーはふだん 1 日以内に返信
modelcontextprotocol/inspector の issue をすべて見る
似ている issue
-
難易度 2/5 1〜3時間 初心者へのやさしさ 72/100
braintrustdata/braintrust-sdk-javascript#2659 ·
メンテナーはふだん 1 日以内に返信
-
body-parser 2.3.0 in the published ai-dial-chat-bff image is vulnerable (CVE-2026-111181, HIGH)対応中かも @illya-havsiyevych が今日担当しました。 オープンbug
難易度 1/5 1〜3時間 初心者へのやさしさ 87/100
epam/ai-dial-chat#9566 ·
メンテナーはふだん 1 日以内に返信
-
bug
難易度 2/5 1〜3時間 初心者へのやさしさ 83/100
メンテナーはふだん 1 日以内に返信
-
はさみアイコンを修正するオープンbug
難易度 1/5 1時間未満 初心者へのやさしさ 68/100
-
area: core
難易度 2/5 1〜3時間 初心者へのやさしさ 78/100
メンテナーはふだん 1 日以内に返信