Repository navigation
feat: stacked-task-set 477, 478, 479 - #643
Merged
Merged
Conversation
Task 476 kept projections hand-written; no [Mapper] exists. Drop the package reference and its central version pin. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
…ed-riokmapperly-reference-from-web-appl chore(web-application): remove unused Riok.Mapperly reference
Role.AgentScopes now reads back in RoleCatalog.AgentScopes order whatever the set order or EF load order. GetRole, GetRoles, and SetRoleAgentScopes project it as is and no longer call AgentScopeOrder.OrderKnown; unknown scopes stay rejected (contract validator 400, Role.SetAgentScopes throws as the belt). AgentScopeOrder keeps the Principals and AgentScopeApproval callers. Task 478. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
…-scopes-in-roleagentscopes-so-roles-han refactor(roles): order agent scopes in Role.AgentScopes
…jection arguments GetRole and GetRoles now read Role members through RoleContractMap.Project, and the upload handler and FeedbackContractMap.AttachmentsAsync share FeedbackContractMap.Project. Every response, DTO, admin-view, activity, and attachment constructor call in the Roles and Feedback application layers uses named arguments. Behavior and wire shapes are unchanged. Task 479. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
…rojection-for-getrole-and-getroles-and refactor(roles,feedback): share one projection per shape and name projection arguments
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
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.
Summary
stacked-task-set merged children onto
feature/glue-followups.Task ids: 477, 478, 479
Test plan
Master merge stays a human Gate (
ganda pr mergewithout--into).