Skip to content

Fix command-center permanent allowlist backend crash#9

Draft
cursor[bot] wants to merge 2 commits into
mainfrom
cursor/critical-bug-investigation-7717
Draft

Fix command-center permanent allowlist backend crash#9
cursor[bot] wants to merge 2 commits into
mainfrom
cursor/critical-bug-investigation-7717

Conversation

@cursor

@cursor cursor Bot commented Jun 12, 2026

Copy link
Copy Markdown

Summary

  • Add the missing brain.permanent_allowlist backend module used by command-center approval endpoints and orchestrator paths.
  • Persist scoped permanent approval rules locally with exact payload matching and server-side rejection for never-permanent actions.
  • Fix a permanent approval creation logging crash caused by passing action both positionally and as a keyword.
  • Add focused regression tests for rule CRUD, matching, endpoint creation, and forbidden actions.

Validation

  • python3 -m pytest tests/test_events_router.py tests/test_permanent_allowlist.py -> 10 passed
  • python3 -m pytest -> 24 passed, 1 known unrelated failure in tests/test_repo_router.py::test_repo_tree_returns_empty_when_governance_root_missing
  • Explicit import check for routers.events, services.supervisor_bridge, and brain.orchestrator.main passed
Open in Web View Automation 

cursoragent and others added 2 commits June 12, 2026 20:09
Co-authored-by: AlexGuruz <AlexGuruz@users.noreply.github.com>
Co-authored-by: AlexGuruz <AlexGuruz@users.noreply.github.com>
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