Skip to content

[WIP] Merge IsRoot feature into main branch#22

Merged
yongpengW merged 1 commit intodevfrom
copilot/sub-pr-21
Mar 3, 2026
Merged

[WIP] Merge IsRoot feature into main branch#22
yongpengW merged 1 commit intodevfrom
copilot/sub-pr-21

Conversation

Copy link
Copy Markdown
Contributor

Copilot AI commented Mar 3, 2026

  • Initial plan
  • Revert SystemRoleConstants to lowercase ("root"/"staff") matching DB data
  • Use case-insensitive OrdinalIgnoreCase for IsRoot detection in UserContextCacheService
  • Bump CacheVersion to v3 to invalidate stale caches
  • Update TokenOptions XML comment to remove non-existent validation claim

🔒 GitHub Advanced Security automatically protects Copilot coding agent pull requests. You can protect all pull requests by enabling Advanced Security for your repositories. Learn more about Advanced Security.

Copilot AI mentioned this pull request Mar 3, 2026
Merged
@yongpengW yongpengW marked this pull request as ready for review March 3, 2026 15:42
Copilot AI review requested due to automatic review settings March 3, 2026 15:42
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot wasn't able to review any files in this pull request.


💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@yongpengW yongpengW merged commit 86cab15 into dev Mar 3, 2026
1 check failed
Copilot stopped work on behalf of yongpengW due to an error March 3, 2026 15:42
@yongpengW yongpengW deleted the copilot/sub-pr-21 branch March 4, 2026 03:22
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.

3 participants