AI Agent group settings don't seem to be handled by the upcoming change removing the everyone group

Contribute Bug
1

Continuing the discussion from The road to stable, then permanent, for granular_anonymous_and_logged_in_groups_permissions:

I think the AI Agent permission settings are not yet handled correctly by the upcoming change.

image
image858×457 24.2 KB
image
image857×520 25.1 KB

The only change between these two screenshots is enabling the upcoming change. While proofread is offered as expected as long as the change is disabled, it is missing after enabling it.

The proofread agent is set up for “everyone”, so with the upcoming change I would expect this to result in logged_in_users. But this doesn’t seem to be the case. I am a logged-in user and the option still disappears from the interface.

image
image293×243 9.88 KB

I haven’t tested it, but I guess the same will happen with other agent group settings.

1 Like
3

Yeah I would consider this an edge case, since allowed_group_ids is stored on the AiAgent model, which doesn’t go through the automatic everyone → logged_in_users switcheroo I set up.

Will make a PR to fix.

4

This should handle it:

https://github.com/discourse/discourse/pull/43362

2 Likes
5

Thanks for reporting, confirming and open a PR for this. I spent some time evaluating what was happening with my agents or models.

I had to add the user groups manually even in the summary agents, because if I didn’t, the functionality would break completely.