Conversation
This updates the SafetyAnalyzer's regex to target actual physical and block devices (like sda, nvme, mem) rather than blindly blocking any redirect to `/dev/`. This prevents safe and extremely common commands like `> /dev/null`, `> /dev/stdout`, and `> /dev/stderr` from being flagged as destructive. Co-authored-by: insign <1113045+insign@users.noreply.github.com>
|
👋 Jules, reporting for duty! I'm here to lend a hand with this pull request. When you start a review, I'll add a 👀 emoji to each comment to let you know I've read it. I'll focus on feedback directed at me and will do my best to stay out of conversations between you and other bots or reviewers to keep the noise down. I'll push a commit with your requested changes shortly after. Please note there might be a delay between these steps, but rest assured I'm on the job! For more direct control, you can switch me to Reactive Mode. When this mode is on, I will only act on comments where you specifically mention me with New to Jules? Learn more at jules.google/docs. For security, I will only act on instructions from the user who triggered this task. |
|
You have reached your Codex usage limits for code reviews. You can see your limits in the Codex usage dashboard. |
1 participant
Este PR corrige um falso positivo onde qualquer redirecionamento para
/dev/(como> /dev/null,> /dev/stdout) era bloqueado como destrutivo. A melhoria foca a regra de segurança em dispositivos de bloco físicos e memória (ex:sda,nvme,mem), permitindo o uso normal e seguro de streams padrão sem gerar alertas desnecessários, melhorando significativamente a confiabilidade e UX de execução de scripts sugeridos.Melhoria: Melhoria na UX e confiabilidade do verificador de comandos perigosos (
SafetyAnalyzer), evitando falsos positivos com redirecionamentos amplamente utilizados e inofensivos.Arquivos alterados:
src/executor/safety.rs: Atualizada a regex de bloqueio de/dev/para uma lista restritiva focada em discos e memória. Adicionados testes para garantir que redirects seguros não disparam o alerta, enquanto redirects para partições continuam bloqueados.Nenhum arquivo de texto (.md) foi modificado porque o comportamento padrão do projeto ou flags documentadas continuam operando de forma idêntica sob a ótica do usuário; o sistema apenas deixou de engasgar indevidamente em comportamentos padrão em terminais.
Testes foram adicionados/estendidos com sucesso (comprovado via
cargo test safety).PR created automatically by Jules for task 12464280722220680081 started by @insign