Skip to content

Core Loader, Compilation, and Definition Validation #410

Closed

Description

@BryanttV

Description

Implement the logic to read definition files (e.g., YAML), compile the rules in memory, and apply the early validations defined in the ADR. For now, the openedx-authz module will load its own definitions by pointing directly to its local path, setting the foundation for future extensibility.

Acceptance Criteria

  • Create the Loader function/class that receives a list of definition file paths.
  • Implement early validation: detection of duplicate identifiers, unsupported scope types, and broken references.
  • Define and implement error handling: specify whether the application fails on startup (Fail Fast) or if the invalid file is skipped leaving a critical error log.
  • The openedx-authz module successfully loads its own roles/permissions by passing its static path to the Loader.

Proposed ADR: #421

Activity

  1. moved this to In Grooming in RBAC AuthZ Boardon Aug 21, 2026
  2. moved this from In Grooming to Blocked in RBAC AuthZ Boardon Aug 21, 2026
  3. changed the title [-]Plugin definition discovery[/-] [+]2. Plugin definition discovery[/+] on Aug 21, 2026
  4. changed the title [-]2. Plugin definition discovery[/-] [+]Core Loader, Compilation, and Definition Validation[/+] on Sep 2, 2026
  5. moved this from Blocked to In Progress in RBAC AuthZ Boardon Sep 11, 2026
  6. linked a pull request that will close this issuefeat: add authz schema definition files #445on Sep 11, 2026
  7. 8 remaining items

  8. linked a pull request that will close this issuefeat: add authz schema definition models #478on Oct 2, 2026
  9. linked a pull request that will close this issuefeat: add authz schema policy renderer #479on Oct 2, 2026
  10. linked a pull request that will close this issuefeat: add authz schema applier #480on Oct 2, 2026
  11. linked a pull request that will close this issuefeat: validate Paragon icons on authz schemas #472on Oct 2, 2026
  12. rodmgwgu commented on Oct 8, 2026

    @rodmgwgu
    Contributor

    @BryanttV I think with the PRs that have been merged so far, the acceptance criteria of this ticket would be complete, what do you think?

  13. BryanttV commented on Oct 8, 2026

    @BryanttV
    ContributorAuthor

    Yes, according to the criteria, it's complete. I opened this because the other PRs lacked linked issues, so I linked them here. I'll close this one and open new issues to map the work for the remaining PRs. Thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

Labels

willowReleased in Willow

Type

No type

Projects

Milestone

No milestone

Relationships

None yet

Development

No branches or pull requests

Issue actions