Sécurité Shell AI — dry-run et confirmation #97

New Issue

2026-08-20T11:02:11-04:00

bruno commented

2026-08-20 11:02:11 -04:00

Définir et implémenter les garde-fous de sécurité pour la commande am ai afin d'éviter l'exécution accidentelle ou malveillante de commandes shell.

Règles de sécurité

Règle	Détail
`--dry-run` par défaut	Aucune commande modifiante n'est exécutée sans confirmation explicite
Classification `safe` / `risky`	Inspiré d'AI CLI : chaque commande est classée selon son impact
Confirmation utilisateur	Les commandes `risky` demandent une validation explicite (`y/n`)
Mode local possible	Support d'Ollama via `providers.rs` pour ne pas sortir les données
`certainty`	Indice de confiance affiché à l'utilisateur

UX proposée

$ am ai --exec "supprime tous les fichiers .tmp"
🔒 Commande générée : rm *.tmp
   Risk     : risky
   Certainty: 92%
   Exécuter ? [y/N]:

Configuration

settings:
  shell_ai:
    default_safety: dry-run  # dry-run | confirm | auto
    risky_patterns:
      - "rm -rf"
      - "> /dev"
      - "dd if"

Critères d'acceptation

--dry-run est le comportement par défaut
Classification automatique safe/risky implémentée
Confirmation avant exécution des commandes risky
Configuration utilisateur possible
Tests de sécurité

Effort : M (2 à 4 jours) — Phase P3

Définir et implémenter les garde-fous de sécurité pour la commande `am ai` afin d'éviter l'exécution accidentelle ou malveillante de commandes shell. ## Règles de sécurité | Règle | Détail | |---|---| | `--dry-run` par défaut | Aucune commande modifiante n'est exécutée sans confirmation explicite | | Classification `safe` / `risky` | Inspiré d'AI CLI : chaque commande est classée selon son impact | | Confirmation utilisateur | Les commandes `risky` demandent une validation explicite (`y/n`) | | Mode local possible | Support d'Ollama via `providers.rs` pour ne pas sortir les données | | `certainty` | Indice de confiance affiché à l'utilisateur | ## UX proposée ```bash $ am ai --exec "supprime tous les fichiers .tmp" 🔒 Commande générée : rm *.tmp Risk : risky Certainty: 92% Exécuter ? [y/N]: ``` ## Configuration ```yaml settings: shell_ai: default_safety: dry-run # dry-run | confirm | auto risky_patterns: - "rm -rf" - "> /dev" - "dd if" ``` ## Critères d'acceptation - [ ] `--dry-run` est le comportement par défaut - [ ] Classification automatique `safe`/`risky` implémentée - [ ] Confirmation avant exécution des commandes `risky` - [ ] Configuration utilisateur possible - [ ] Tests de sécurité ## Effort : **M** (2 à 4 jours) — Phase **P3**