Skip to content

fix: preserve OpenClaw approval ownership#290

Merged
peg merged 1 commit intostagingfrom
fix/openclaw-approval-ownership
May 1, 2026
Merged

fix: preserve OpenClaw approval ownership#290
peg merged 1 commit intostagingfrom
fix/openclaw-approval-ownership

Conversation

@peg
Copy link
Copy Markdown
Owner

@peg peg commented May 1, 2026

Summary

  • stop Rampart bridge from auto-consuming OpenClaw approvals when the native Rampart plugin is installed
  • keep deny enforcement automatic while leaving OpenClaw/manual approval records human-owned
  • preserve legacy bridge-first auto-allow behavior when the Rampart plugin is not installed
  • document the ownership split and add regression coverage for forced/OpenClaw approval modes

Tests

  • go test ./internal/bridge ./cmd/rampart/cli ./internal/proxy -count=1
  • go test ./... -count=1
  • go vet ./...
  • go build ./cmd/rampart

@peg peg merged commit c65635f into staging May 1, 2026
4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant