Graduated guard with secure-exec sandbox tier and learning feedback loop
Nadie ha tomado este issue todavía.
Evaluación
- Dificultad
- 5/5
- Tiempo estimado
- Más de una semana
- Aptitud para principiantes
- 25/100
- Tipo de issue
- Nueva funcionalidad
- Claridad
- Bien especificado
- Estado de actividad
- Estancado
- Stack tecnológico
- rust, shell, typescript
Línea de trabajo
Read crates/terraphim_agent/src/guard_patterns.rs and plans/terraphim-secure-exec-convergence.md first, then evaluate whether secure-exec's child_process bridge enforces the proposed restrictions. Review the hook entry points in ~/.claude/hooks/pre_tool_use.sh and post_tool_use.sh. Done requires the feasibility gate and the phased three-tier guard, sandbox execution, learning, and review flow described in the issue.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
Summary
Extend the CommandGuard from binary allow/deny to a three-tier decision system (allow/sandbox/deny), integrate rivet-dev/secure-exec as the sandbox execution layer, and wire the PostToolUse learning capture into an adaptive pattern evolution loop.
Motivation
Today the PreToolUse guard has no middle ground: a command is either blocked entirely (destructive pattern match) or runs with full host access. Commands that are suspicious but not outright destructive (e.g., curl | sh, writes to ~/.ssh/, chmod 777, eval of dynamic strings) pass through unchecked.
The learning capture system records failures but does not feed back into the guard patterns. Guard patterns are static KG entries that only change via manual editing.
Proposed Design
Three convergence layers
Layer 1 -- Graduated Guard Decisions
- Add
GuardDecision::Sandboxtoguard_patterns.rsalongside existingAllowandBlock - Add a third pattern set:
suspicious_patterns(between safe and destructive) - Create
~/.config/terraphim/kg/guard-suspicious/directory with initial entries - The PreToolUse hook reads the three-valued decision and routes accordingly
Layer 2 -- Secure Exec as Sandbox Tier
- When guard returns
sandbox, run the command inside a V8 isolate via secure-exec - Permission profiles scoped by command type: read-only project dir, network deny-by-default, no process spawning, 64 MB memory cap, 10s CPU time limit
- 17ms cold start makes this viable for interactive development
- Thin wrapper script at
~/.claude/hooks/sandbox-exec.ts
Layer 3 -- Learning-Driven Pattern Evolution
- Extend
learn hookto capture sandbox outcomes (permission violations, timeouts, clean passes) - Promotion/demotion rules: 3+ clean sandbox runs suggests promotion to allow; permission violation promotes to deny
- Auto-generate KG entries into
guard-staging/directory - Human review gate via
terraphim-agent guard reviewbefore patterns become active
Architecture
Command arrives
|
v
Guard (Aho-Corasick pattern match)
|
+-- destructive match ---------> DENY (never runs)
+-- suspicious match ----------> SANDBOX (V8 isolate via secure-exec)
+-- KG-recognised safe match --> ALLOW (full host access)
+-- unknown --------------------> SANDBOX (default-to-sandbox)
|
v
Replace (KG synonym substitution, runs regardless of tier)
|
v
Execute (tier-appropriate: raw shell or sandboxed)
|
v
Learn (capture outcome, feed back into guard patterns)
|
+-- repeated sandbox success --> promote to ALLOW
+-- sandbox violation/failure -> promote to DENY
+-- new pattern discovered ----> add to KG
Implementation Phases
- Phase 0 (1 day): Evaluation spike -- verify secure-exec
child_processbridge enforces filesystem restrictions inside V8 isolate. This is the critical gate for Layer 2. - Phase 1 (2 days): Add
GuardDecision::Sandboxtoguard_patterns.rs, create suspicious pattern KG entries, updatepre_tool_use.sh - Phase 2 (3 days): Create
sandbox-exec.tswrapper, define permission profiles, wire into PreToolUse hook - Phase 3 (3 days): Extend learning capture for sandbox outcomes, build promotion/demotion engine, KG auto-generation with staging directory
- Phase 4 (1 day): Integrate guard pattern review into daily sweep
Key Risk
Whether secure-exec's bridged child_process module enforces isolate-level filesystem restrictions on spawned shell commands. If not, Layer 2 applies only to AI-generated JS/TS code execution, not arbitrary shell commands. Layers 1 and 3 remain valuable regardless.
References
- Design document:
plans/terraphim-secure-exec-convergence.mdin cto-executive-system - Secure Exec: https://github.com/rivet-dev/secure-exec (Apache-2.0, 17ms cold start, 3.4 MB memory)
- Current guard implementation:
crates/terraphim_agent/src/guard_patterns.rs - Current hook scripts:
~/.claude/hooks/pre_tool_use.sh,post_tool_use.sh - Related: secure-exec knowledge entry at
knowledge/external/context-engineering/secure-exec-sandboxless-code-execution.md
- Lenguaje dominante
- Rust
- Estrellas
- 62
- Forks
- 5
- Merge medio
- 2 h 27 min
- PR fusionados (30 d)
- 1
Guía de contribución
Primeros pasos
- Lee el issue completo y luego la guía de contribución del proyecto.
- Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
- Haz un fork del repositorio y trabaja en una rama.
- Abre un pull request que haga referencia al número del issue.
Más de terraphim/terraphim-ai
-
Dificultad 5/5 Más de una semana Aptitud para principiantes 25/100
terraphim/terraphim-ai#885 ·
-
Dificultad 4/5 3-5 días Aptitud para principiantes 55/100
terraphim/terraphim-ai#871 ·
-
enhancement
Dificultad 5/5 Más de una semana Aptitud para principiantes 25/100
terraphim/terraphim-ai#810 · 2 comentarios ·
-
enhancement
Dificultad 5/5 Más de una semana Aptitud para principiantes 35/100
terraphim/terraphim-ai#729 ·
-
enhancement
Dificultad 5/5 Más de una semana Aptitud para principiantes 35/100
terraphim/terraphim-ai#728 ·
Todos los issues de terraphim/terraphim-ai
Issues similares
-
Browser (wasm) relay client cannot connect to relays whose URL has a trailing-dot FQDN hostname Abierto
Dificultad 2/5 1-3 horas Aptitud para principiantes 75/100
n0-computer/iroh#4550 ·
-
impl detach for native Abierto
Dificultad 2/5 1-3 horas Aptitud para principiantes 65/100
paritytech/zombienet-sdk#591 ·
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 75/100
farion1231/cc-switch#7638 · 1 comentario ·
-
onnx-ir re-exports ModelProto and GraphProto but not NodeProto, AttributeProto and AttributeType Abierto
Dificultad 2/5 1-3 horas Aptitud para principiantes 75/100
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 75/100