hasAccess checking done in getTools but not executeTool

Created on 6 June 2025, about 1 month ago

In the most recent 1.x-dev version it looks like an attempt was made to fix the hasAccess checking to allow it to return a boolean instead of just an AccessResult. However, this change was only made to the getTools function and not to the executeTool function. Here is a patch that cleans this up with putting the access check into a helper function and then using the helper function in both places.

πŸ› Bug report
Status

Active

Version

1.0

Component

Code

Created by

πŸ‡ΊπŸ‡ΈUnited States mpotter

Live updates comments and jobs are added and updated live.
Sign in to follow issues

Comments & Activities

Production build 0.71.5 2024