Skip to content

Expose AI configuration in profile#4143

Open
nelsonkopliku wants to merge 5 commits intobasic-ai-config-functionsfrom
expose-ai-configuration-in-profile
Open

Expose AI configuration in profile#4143
nelsonkopliku wants to merge 5 commits intobasic-ai-config-functionsfrom
expose-ai-configuration-in-profile

Conversation

@nelsonkopliku
Copy link
Copy Markdown
Member

@nelsonkopliku nelsonkopliku commented Apr 2, 2026

Description

This PR exposes a user's AI configuration as part of the /profile endpoint.

We need to decide wether a user manager can operate on specific users' AI configuration, similarly to what happens with PATs.

Additionally:

  • when a user is deleted, its AI config is deleted
  • took the chance to add the missing "when a user is deleted, its PATs are deleted" (it does not change the overall behavior, it just helps in keeping things clean)

Depends on #4140

How was this tested?

Automated and IRL tests.

@nelsonkopliku nelsonkopliku self-assigned this Apr 2, 2026
@nelsonkopliku nelsonkopliku added the enhancement New feature or request label Apr 2, 2026
@nelsonkopliku nelsonkopliku changed the title Expose ai configuration in profile Expose AI configuration in profile Apr 2, 2026
@nelsonkopliku nelsonkopliku force-pushed the basic-ai-config-functions branch from 58daf83 to e8f9f91 Compare April 7, 2026 14:46
@nelsonkopliku nelsonkopliku force-pushed the expose-ai-configuration-in-profile branch from 0b8c6b2 to 41dc558 Compare April 7, 2026 14:47
Copy link
Copy Markdown
Contributor

@arbulu89 arbulu89 left a comment

Choose a reason for hiding this comment

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

Looks good!
I didn't check the code where the AIConfiguration is implemented, but the code is simple enough to see that it makes sense.
I understand that you don't include the AI data in the users list as it is not planned to be displayed in the frontend, right?

@nelsonkopliku
Copy link
Copy Markdown
Member Author

the code where the AIConfiguration is implemented

It's here #4140, just in case

I understand that you don't include the AI data in the users list as it is not planned to be displayed in the frontend, right?

Indeed, not right now.
I don't exclude though that at some point we will need user admins to operate on specific users' AI configuration. Similarly to what happens for PATs.

We will add that when needed.

@nelsonkopliku nelsonkopliku force-pushed the basic-ai-config-functions branch from e8f9f91 to 6095286 Compare April 8, 2026 08:12
@nelsonkopliku nelsonkopliku force-pushed the expose-ai-configuration-in-profile branch from d795031 to b06dc2b Compare April 8, 2026 08:14
@nelsonkopliku nelsonkopliku force-pushed the basic-ai-config-functions branch from 6095286 to ecb3839 Compare April 8, 2026 20:56
@nelsonkopliku nelsonkopliku force-pushed the expose-ai-configuration-in-profile branch from b06dc2b to 8c061b2 Compare April 8, 2026 20:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Development

Successfully merging this pull request may close these issues.

4 participants