docs: consolidated RBAC docs (accuracy fixes, transition guide, dynamic-auth OAuth)#658
docs: consolidated RBAC docs (accuracy fixes, transition guide, dynamic-auth OAuth)#658jordanc-relevanceai wants to merge 7 commits into
Conversation
|
Preview deployment for your docs. Learn more about Mintlify Previews.
💡 Tip: Enable Workflows to automatically generate PRs for you. |
Combines two drafter PRs that both modify add-members.mdx and enterprise/rbac.mdx: - TSP-1150 (#583): correct inaccuracies — adds Chat role to add-members, simplifies Viewer accordion, adds clarifying callouts about Editor scoping, Viewer access scope, and asset-level visibility, and adds "Permission inheritance and cascading" + "Technical implementation notes" sections - TSP-1152 (#586): adds the "Transitioning to RBAC" section covering what changes during migration, role mapping, and admin action items Reconciled the Info banner at the top of add-members.mdx so both PRs' framing co-exists: the page now describes itself as the standard permission system for orgs not yet migrated to RBAC, with links to both the RBAC docs and the transition guide.
Non-admin team members can now add their own OAuth accounts when dynamic authentication is enabled on a shared agent. Update user-level-authentication.mdx to clarify this capability in the setup section, first-time auth flow, privacy section, and a new FAQ entry. Update rbac.mdx permissions table and add a note distinguishing project-level OAuth management (admin-only) from personal OAuth for dynamic auth (all members). Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
Fix the permission inheritance table and warning to match the OpenFGA model: only Admin and Editor cascade to asset Admin; Member, Viewer, and Chat get no asset access without an explicit grant. Restyle the new Transitioning to RBAC, cascading, and technical notes sections with cards, steps, and tabs, and convert the first-time auth steps to a Steps component. Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
aed8180 to
6cd00d6
Compare
Switch the After RBAC is enabled list from a CardGroup to an AccordionGroup so each subsection uses a distinct component. Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
🎯 Vibe checkReviewed: 3 files (3 with issues, 0 clean) Scores
Score key: 🟢 9–10, 🟡 6–8, 🔴 1–5. Scores are a single overall judgment across the whole PR. ✨ Overall vibe: The content itself is solid — comprehensive RBAC reference, a clear transition guide, and a well-structured user auth how-to. What drags the scores down is mechanical: casing errors across all three files, two banned-word uses, and a genuine factual contradiction about whether legacy Viewers could run agents that needs resolving before these pages can be trusted as a source of truth. 🔧 Issues (13)
🧩 Component suggestions (2)
🏗️ Page structure (2)
|
| Item | Count |
|---|---|
| Files reviewed | 3 |
| Context pages read | 3 |
| Total lines processed | ~1,123 |
Files read: admin/project-management/add-members.mdx (109 lines), enterprise/rbac.mdx (365 lines), enterprise/user-level-authentication.mdx (219 lines), admin/project-management/remove-members.mdx (58 lines), enterprise/rbac-groups.mdx (328 lines), enterprise/asset-controls.mdx (44 lines)
Convert the embed style objects to the single-quoted format the structure-check linter requires (paddingTop: '56.25%', the purple border, and rounded corners), fixing pre-existing embeds in the files this branch touches so the lint check passes. Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
🎯 Vibe checkReviewed: 3 files (3 with issues, 0 clean) Scores
Score key: 🟢 9–10, 🟡 6–8, 🔴 1–5. Scores are a single overall judgment about the whole PR — not per file. ✨ Overall vibe: The content itself is solid — the RBAC page is the most comprehensive treatment this topic has had, the ULA page explains a tricky feature clearly, and the add-members page adds useful detail about the invitation flow. The main damage is cosmetic (heading and card title casing is inconsistent throughout 🔧 Issues (19)Heading and title case
Banned words
Callout / markup
Tone / style
Misplaced content
🧩 Component suggestions (3)
🏗️ Page structure (1)
|
| Item | Count |
|---|---|
| Files reviewed | 3 |
| Context pages read | 2 |
| Total lines processed | ~1081 |
Files read: admin/project-management/add-members.mdx (109 lines), enterprise/rbac.mdx (365 lines), enterprise/user-level-authentication.mdx (220 lines), admin/project-management/remove-members.mdx (59 lines), enterprise/rbac-groups.mdx (328 lines)
This reverts commit 6342571.
Why two checks are red1. Documentation Lint Checks — Supademo embeds Reverted the Supademo embed normalization (390b126). The
These embeds are pre-existing (not added in this PR) and render correctly with their original wrapper, so I've left them as-is rather than forcing the standard snippet. The lint rule may need to allow a taller aspect ratio for demos like these. 2. afdocs check — unrelated This failure is on |
Resolves the structure-check warning about key-value bullet lists by presenting the project-level role guidance as a table. Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
🎯 Vibe checkReviewed: 3 files (3 with issues, 0 clean) Scores
Score key: 🟢 9–10, 🟡 6–8, 🔴 1–5. Scores are a single overall judgment about the whole PR. ✨ Overall vibe: The content is genuinely solid — the RBAC tables are comprehensive, the transition guide is well-structured, and user-level authentication is explained with good depth. The PR is held back by two direct factual contradictions between 🔧 Issues (15)Heading sentence case —
Card title sentence case —
Bold label inside callout —
Misplaced content —
Potentially wrong anchor —
Product term capitalization (pattern across files)
🧩 Component suggestions (2)
🏗️ Page structure (2)
|
| Item | Count |
|---|---|
| Files reviewed | 3 |
| Context pages read | 2 |
| Total lines processed | ~1,080 |
Files read: admin/project-management/add-members.mdx (109 lines), enterprise/rbac.mdx (365 lines), enterprise/user-level-authentication.mdx (219 lines), admin/project-management/remove-members.mdx (59 lines), enterprise/rbac-groups.mdx (328 lines)
🎯 Vibe checkReviewed: 3 files (3 with issues, 0 clean) Scores
Score key: 🟢 9–10, 🟡 6–8, 🔴 1–5. ✨ Overall vibe: The RBAC and ULA pages are substantively good — comprehensive, well-organized, with tables and steps where they belong. The main drag is a cluster of mechanical consistency failures (heading case, card titles, banned words) that are quick to fix, plus one genuine content bug (a misplaced Warning that talks about region lock-in in the middle of the "Add members to your organization" section). The Viewer-role contradiction between the legacy permissions page and the RBAC migration guide also needs a decision: one of them is wrong about whether legacy Viewers can run agents. 🔧 Issues (18)Heading capitalization —
Card title capitalization —
Banned words —
Bold labels inside callouts
Misplaced content —
Inconsistent capitalization of feature name —
Terminology mismatch across files
🧩 Component suggestions (2)
|
| Item | Count |
|---|---|
| Files reviewed | 3 |
| Context pages read | 2 |
| Total lines processed | ~1081 |
Files read: admin/project-management/add-members.mdx (109 lines), enterprise/rbac.mdx (367 lines), enterprise/user-level-authentication.mdx (220 lines), admin/project-management/remove-members.mdx (58 lines), enterprise/rbac-groups.mdx (327 lines)

This PR consolidates the open RBAC documentation work. Opened as draft for review before the source PRs are closed.
Source PRs (being closed in favor of this one)
Why consolidated
All three edit
enterprise/rbac.mdx. Shipping them separately would mean each merge conflicts with the next and the permissions tables / role descriptions drift. #606 already reconciled the two RBAC-accuracy PRs; folding #593 in here keeps the permissions table and role notes coherent in a single pass.Changes by source PR
#606 — TSP-1150 + TSP-1152
admin/project-management/add-members.mdx— adds the Chat role accordion, simplifies the Viewer description (removes the incorrect "can run agents" claim), reframes the Info banner to link both the RBAC docs and the transition guide.enterprise/rbac.mdx— Editor/Viewer/asset-visibility clarifying callouts, plus three new sections: Transitioning to RBAC, Permission inheritance and cascading, and Technical implementation notes.#593 — TSP-1169
enterprise/rbac.mdx— adds an "Add personal OAuth accounts (dynamic auth)" row to the project-level permissions table (all roles ✅) and a note distinguishing project-level OAuth management (admin-only) from personal OAuth for dynamic auth (all members).enterprise/user-level-authentication.mdx— Info callout, updated first-time auth steps, privacy paragraph, and a new FAQ entry documenting that non-admin members can add their own OAuth accounts when dynamic auth is enabled.Reconciliation note
No conflicts between the two — #593's permissions-table row and note land inside the existing Project-level section, while #606's additions are separate sections. Verified no duplicate headings or table rows in the merged
rbac.mdx.