Permission rule delegates fail on Windows for .cmd/.bat programs and can't take arguments
メンテナーはふだん 1 日以内に返信
まだ誰も着手していません。
評価
- 難易度
- 4/5
- 見積もり時間
- 3〜5日
- 初心者へのやさしさ
- 55/100
- issue の種類
- バグ
- 明瞭さ
- おおむね明確
- 活発さ
- 活発
- 技術スタック
- node.js, typescript
調査の方向性
Start with src/main/agent/permissions.ts and inspect runDelegate around line 71, then trace its permission-rule callers and existing tests. Reproduce the Windows .cmd/.bat spawn failure and the to: "node check.js" case; add tests for both supported argument forms. Done when those cases work as intended and the tests pass.
索引モデルが issue の本文から書いたものです。
説明
Found in the 2026-10-07 code review and confirmed: spawning a .cmd without a shell throws spawn EINVAL on current Node.js.
Problem
runDelegate (src/main/agent/permissions.ts:71) calls spawn(program, [], …) without a shell. On Windows, .cmd/.bat files can't be spawned that way since the CVE-2024-27980 fix. Every matching tool call is then "Blocked by a permission rule … failed: spawn EINVAL". A value like to: "node check.js" fails with ENOENT on every OS, because the whole string is treated as the program name.
Fix
Accept to as { program, args } (or an argv array), or run .cmd/.bat through cmd.exe /d /s /c with properly quoted arguments. Document that to names a single executable. Add tests for both forms.
- 主要言語
- TypeScript
- スター
- 2
- フォーク
- 2
- 平均マージ
- 8時間 28分
- マージ済み PR（30日）
- 26
環境構築
- Dockerfile・Docker Compose ファイルなし
- プルリクエストのテンプレートなし
- コントリビューションガイドを読む
はじめの一歩
- issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
- 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
- リポジトリをフォークし、ブランチを切って変更します。
- issue 番号を参照したプルリクエストを送ります。
PierrunoYT/patch のほかの issue
-
難易度 2/5 1〜3時間 初心者へのやさしさ 66/100
PierrunoYT/patch#316 · コメント 1 件 ·
メンテナーはふだん 1 日以内に返信
-
難易度 2/5 1〜3時間 初心者へのやさしさ 72/100
PierrunoYT/patch#315 ·
メンテナーはふだん 1 日以内に返信
-
難易度 2/5 1〜3時間 初心者へのやさしさ 76/100
PierrunoYT/patch#313 ·
メンテナーはふだん 1 日以内に返信
-
enhancement priority: low severity: low
難易度 2/5 1〜3時間 初心者へのやさしさ 72/100
PierrunoYT/patch#305 ·
メンテナーはふだん 1 日以内に返信
-
security: isAppPageUrl checks devRendererUrl with startsWith instead of comparing parsed URL originsオープンpriority: low security severity: low
難易度 2/5 1〜3時間 初心者へのやさしさ 72/100
PierrunoYT/patch#300 ·
メンテナーはふだん 1 日以内に返信
PierrunoYT/patch の issue をすべて見る
似ている issue
-
難易度 2/5 1〜3時間 初心者へのやさしさ 74/100
paperclipai/paperclip#17151 ·
メンテナーはふだん 1 日以内に返信
-
needs-triage
難易度 2/5 1〜3時間 初心者へのやさしさ 83/100
AgoraIO-Extensions/agent-infra#3159 ·
メンテナーはふだん 1 日以内に返信
-
blocklist addition
難易度 1/5 1時間未満 初心者へのやさしさ 66/100
MetaMask/eth-phishing-detect#304596 ·
メンテナーはふだん 1 日以内に返信
-
friction
難易度 2/5 1〜3時間 初心者へのやさしさ 70/100
kentcdodds/kody#4142 ·
-
[Bug] Statistics | Task statistics are always empty on MikroORM with MySQL or SQLite対応中かも @NdekoCode が今日担当しました。 オープン
難易度 2/5 1〜3時間 初心者へのやさしさ 76/100
ever-co/ever-gauzy#11214 ·
メンテナーはふだん 1 日以内に返信