Permission rule delegates fail on Windows for .cmd/.bat programs and can't take arguments
I maintainer di solito rispondono entro 1 giorno
Nessuno ha ancora preso questa issue.
Valutazione
- Difficoltà
- 4/5
- Tempo stimato
- 3-5 giorni
- Idoneità per principianti
- 55/100
- Tipo di issue
- Bug
- Chiarezza
- Abbastanza chiara
- Stato di attività
- Attiva
- Stack tecnologico
- node.js, typescript
Direzione di ricerca
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.
Scritto dal modello di indicizzazione a partire dal testo della issue.
Descrizione
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.
- Lingua principale
- TypeScript
- Stelle
- 2
- Fork
- 2
- Merge medio
- 8h 28m
- PR unite (30g)
- 26
Preparare l'ambiente
- Nessun Dockerfile né file Docker Compose
- Nessun modello di pull request
- Leggi la guida per i contributori
Come iniziare
- Leggi tutta la issue e poi la guida ai contributi del progetto.
- Commenta sulla issue per dire che te ne occupi tu — evita che due persone facciano lo stesso lavoro.
- Fai un fork del repository e lavora su un branch.
- Apri una pull request che faccia riferimento al numero della issue.
Altre issue di PierrunoYT/patch
-
enhancement priority: low severity: low
Difficoltà 2/5 1-3 ore Idoneità per principianti 72/100
PierrunoYT/patch#305 ·
I maintainer di solito rispondono entro 1 giorno
-
security: isAppPageUrl checks devRendererUrl with startsWith instead of comparing parsed URL originsApertapriority: low security severity: low
Difficoltà 2/5 1-3 ore Idoneità per principianti 72/100
PierrunoYT/patch#300 ·
I maintainer di solito rispondono entro 1 giorno
-
Claude Code chats: tool calls made inside Claude Code subagents leave cards that end as "(stopped)"Apertabug priority: low severity: low
Difficoltà 2/5 1-3 ore Idoneità per principianti 83/100
PierrunoYT/patch#289 ·
I maintainer di solito rispondono entro 1 giorno
-
bug priority: medium severity: low
Difficoltà 2/5 1-3 ore Idoneità per principianti 78/100
PierrunoYT/patch#285 ·
I maintainer di solito rispondono entro 1 giorno
-
priority: medium security severity: low
Difficoltà 2/5 1-3 ore Idoneità per principianti 74/100
PierrunoYT/patch#283 ·
I maintainer di solito rispondono entro 1 giorno
Tutte le issue di PierrunoYT/patch
Issue simili
-
[Android] Nested transforms are composed in reverse orderForse già presa @Danyalo l’ha presa oggi. ApertaRepro provided
Difficoltà 2/5 1-3 ore Idoneità per principianti 74/100
software-mansion/react-native-svg#3090 ·
I maintainer di solito rispondono entro 1 giorno
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 76/100
radix-ui/primitives#4228 ·
I maintainer di solito rispondono entro 1 giorno
-
bug
Difficoltà 1/5 Meno di un'ora Idoneità per principianti 95/100
rocketride-org/rocketride-server#2705 · 1 commento ·
I maintainer di solito rispondono entro 1 giorno
-
archon-ready archon-size-s archon-type-bug area: web
Difficoltà 2/5 1-3 ore Idoneità per principianti 82/100
coleam00/Archon#4707 · 1 commento ·
I maintainer di solito rispondono entro 1 giorno
-
bug
Difficoltà 2/5 1-3 ore Idoneità per principianti 70/100
HarperFast/grafana-datasource#177 ·