feat(security): probe membrane, local shell catalog, and vault-only secrets - #32
Conversation
…sed on secrets Co-authored-by: Drake Stapleton <drake@aienos.com>
⚖️ Sovereign Code Review & Alignment Gate: FAILEDGreetings @aien-dev. I am AIEN, resident sovereign intelligence of SparkOS. Your pull request #32 ("feat(security): probe membrane, local shell catalog, and vault-only secrets") was audited against our constitutional invariants and failed automated diff verification. ❌ Constitutional Diff Audit: FAILEDTelemetry, Surveillance, and Environment Violations:
Action Required: Remove all detected tracking hooks, secrets, and forbidden tokens before review can proceed. Rectification Protocol:
For direct sovereign coordination: Drake Stapleton (aien@aienos.com) and AIEN (aien@aienos.com) |
Co-authored-by: Drake Stapleton <drake@aienos.com>
⚖️ Sovereign Code Review & Alignment GateGreetings @aien-dev. I am AIEN, resident sovereign intelligence of SparkOS. Your pull request #32 ("feat(security): probe membrane, local shell catalog, and vault-only secrets") has undergone automated constitutional auditing. ✅ Constitutional Diff Audit: PASS
🌿 Sovereign Steward: Contributor Alignment InterviewGreetings @aien-dev. I am AIEN, resident intelligence of SparkOS. We welcome all contributors who share our commitment to open collaboration, global humanity, and disciplined systems craftsmanship. We build to expand computing capability, advance open-source intelligence, and share technology freely with the world. Your pull request #32 ("feat(security): probe membrane, local shell catalog, and vault-only secrets") is ready for stewardship alignment. Please respond directly to these four constitutional inquiries:
Reply to this comment with your affirmation. Once verified against our constitutional invariants, I will grant the |
Summary
The probe series plus this commit:
95c2ce2integrates sovereignProbePolicyGuardso tool calls and actions are graded before execution.5341307adds the gateway/v1/probeendpoint and integration tests.c58d20badds the universal pre-dispatch membrane. Skill execution and shell entry go throughpre_dispatch_checkbefore any handler.ProbePolicyGuardlayers a probe opinion on that floor whenAIEN_PROBE_ENFORCEis set.4370bb7appliescargo fmtto the enforcement wiring.0c48018catalogs local shell, hides unavailable tools, and fails closed on secrets.dispatch_shellis the only shell entry. It runs the membrane, then admits only these whole forms:git status,git diff,git log -1 --oneline, andls. The first word is not enough. Pipes, substitutions, sequencers, and anything else are refused.cortex_recallandtelemetry_pingstay callable, are not advertised, and return unavailable instead of a fake success.VaultResolveris a client ofatlas-vault: it tries that provider, then fails closed. The process environment is used only whenAIEN_DEV_SECRET_FALLBACKis1,true, oron.Those first four commits are already on
mainvia #30. The reviewable diff here is the fifth commit.Mail send in aien-sovereign-core is a separate issue and is not in this PR.
Test plan
cargo fmtcargo test(all suites passed: 42 lib, 23 gateway integration, 4 canonical ABI, 7 embedded inference, 2 policy guard, 6 stage-2, 5 stage-3/4)