Skip to content

TIN-1606: Normalize RBAC management order#13

Merged
Jess Sullivan (Jesssullivan) merged 1 commit into
mainfrom
jess/tin-1606-rbac-matrix
May 25, 2026
Merged

TIN-1606: Normalize RBAC management order#13
Jess Sullivan (Jesssullivan) merged 1 commit into
mainfrom
jess/tin-1606-rbac-matrix

Conversation

@Jesssullivan

Copy link
Copy Markdown
Contributor

Summary

  • export ADMIN_ROLES and ROLE_MANAGEMENT_ORDER alongside ROLE_HIERARCHY
  • make canManageRole() use ROLE_HIERARCHY instead of a private divergent ordering
  • document the RBAC distinction: role management order is hierarchical, role permissions are a capability matrix and not strict supersets
  • add regression coverage for all roles, role alias normalization, and the event-manager/contributor capability split that broke the downstream PBT assumption

Validation

  • pnpm exec tsc --noEmit
  • pnpm exec vitest run tests/permissions.test.ts (34 tests)
  • pnpm test (15 files, 363 tests)
  • pnpm build
  • pnpm check:package
  • bazel test //:test
  • git diff --check

Linear

TIN-1606

@Jesssullivan Jess Sullivan (Jesssullivan) merged commit 26d2bf6 into main May 25, 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