Permissions view dialog

[stanislavHamara]

[krissetto]

/review

[krissetto]

The "Evaluation" line at the bottom of the dialog is a bit confusing IMHO, i wonder if we can make things clearer for the users

Also it'd be better if we used the scrollbar for ux consistency

[github-actions]

Review Summary

✅ Approved - No issues found in the changed code.

This PR adds a permissions view dialog to display tool permission rules. The implementation is clean and follows existing code patterns:

• Proper nil checks where needed (runtime checks, session checks)
• Safe slice operations with boundary clamping in the scrolling logic
• Consistent with existing codebase patterns
• Appropriate separation of concerns between app, runtime, and UI layers

The code correctly:

• Merges team and session-level permissions for display
• Handles empty permission lists gracefully
• Implements scrolling with proper bounds checking
• Hides the command when no permissions are configured

Great work! 🚀

[stanislavHamara]

The "Evaluation" line at the bottom of the dialog is a bit confusing IMHO, i wonder if we can make things clearer for the users

Also it'd be better if we used the scrollbar for ux consistency

The scrollbar is there, I guess the user can do both? 🤷

[krissetto]

The scrollbar is there, I guess the user can do both?

oh, my bad. i got confused as i'm looking into other inconsistencies we have in dialogs and the up/down arrow threw me off :)