Prod changes - #2
Closed
andyne13 wants to merge 6 commits into
Closed
Conversation
dodekapod
added a commit
that referenced
this pull request
Nov 24, 2025
EnjoyBacon7
added a commit
that referenced
this pull request
Mar 12, 2026
EnjoyBacon7
added a commit
that referenced
this pull request
May 7, 2026
Replace all `from openrag.core.X` / `from openrag.services.X` imports with bare `from core.X` / `from services.X` to match the pythonpath convention (pythonpath = ./openrag in pytest.ini). With both the installed package and the PYTHONPATH entry reachable, the prefixed and bare forms resolve to two distinct module objects. This breaks isinstance checks, ABC registration, and registry lookups -- the exact bug documented in REFACTORING_DECISION_LOG.md Phase 5D #2 for the exception shims.
andyne13
added a commit
that referenced
this pull request
Jun 29, 2026
#10) The capability layer offered actions the API rejects. Align the UI with the backend so it never shows a control guaranteed to 403: - #10 usePermissions honors SUPER_ADMIN_MODE: the admin→all-partitions bypass (canRead/canWrite/canManageMembers, new canConfigurePartition) only applies when the backend flag is on, read from /config (admin-gated query). A plain admin without it is treated by their actual partition role, like the API does. - #2 documents/detail gates delete/replace on the caller's partition role and filters copy targets to partitions the user can write — parity with the list. - #3 partitions GeneralTab is role-aware: read-only for non-owners (banner, no Save, disabled inputs); preset/model registries (admin-only) are only fetched for admins so a member's view doesn't fire guaranteed-403 requests. Also fixes the partition-list role lookup (key is `partition`, not `name`) in GeneralTab and UsersTab, which #10 would otherwise leave undefined once the blanket is_admin bypass is gone. Mock /config exposes super_admin_mode top-level to match api/main.py.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
3 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
No description provided.