Skip to content

feat: security role privilege scaffold applier for pp-security-role-privilege - #113

Open
zekelinAlex wants to merge 1 commit into
users/alexander.zekelin/applier-pp-sitemap-subareafrom
users/alexander.zekelin/applier-pp-security-role-privilege
Open

zekelinAlex wants to merge 1 commit into
users/alexander.zekelin/applier-pp-sitemap-subareafrom
users/alexander.zekelin/applier-pp-security-role-privilege

Conversation

@zekelinAlex

Copy link
Copy Markdown
Contributor

moves pp-security-role-privilege's post-action logic into the library. the GenerateRolePrivileges dotnet-run codegen is gone, the applier parses the privilege spec in-process (Newtonsoft is already a dependency of Serialization.Xml) and merges privileges into Roles/.xml idempotently, alphabetically sorted like the old script did. dispatcher key SecurityRolePrivilege.

semantics follow the TALXIS/tools-devkit-templates#152 fix rather than the old broken codegen:

  • quoted and bare JSON specs both parse (the old pipeline died with CS1003 on the quoted form and the CLI rolled the whole scaffold back)
  • the "type" key alias works and produces the proper prvAppendTo name instead of the garbage prvudpp_ one
  • User/BusinessUnit/ParentChild map to Basic/Local/Deep, None omits the privilege, unknown type or level fails naming the valid values

parity sandbox (6 variants) matched the per-variant expectations one for one: the quoted-JSON call exists only in the updated run, level mapping corrected, update-in-place identical, alphabetical order kept.

@zekelinAlex
zekelinAlex force-pushed the users/alexander.zekelin/applier-pp-sitemap-subarea branch from 6ee474f to 87adaf1 Compare September 17, 2026 16:46
@zekelinAlex
zekelinAlex force-pushed the users/alexander.zekelin/applier-pp-security-role-privilege branch from a75da0e to 3572c89 Compare September 17, 2026 16:46
@zekelinAlex
zekelinAlex force-pushed the users/alexander.zekelin/applier-pp-sitemap-subarea branch from 87adaf1 to 510cd90 Compare September 21, 2026 14:13
@zekelinAlex
zekelinAlex force-pushed the users/alexander.zekelin/applier-pp-security-role-privilege branch from 3572c89 to 7abd987 Compare September 21, 2026 14:13
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