Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Streamline AI Variable Service #14817

Open
sdirix opened this issue Feb 3, 2025 · 1 comment · May be fixed by #14827
Open

Streamline AI Variable Service #14817

sdirix opened this issue Feb 3, 2025 · 1 comment · May be fixed by #14827
Labels
TheiaAI

Comments

@sdirix
Copy link
Member

sdirix commented Feb 3, 2025 •
edited
Loading

The initial contribution adds an own AI variable service. Theia already has a generic variable service. It must be evaluated whether they should be fully merged instead of just integrated as they are right now.
@sdirix sdirix mentioned this issue Feb 3, 2025
Open
24 tasks
@sdirix sdirix changed the title The initial contribution adds an own AI variable service. Theia already has a generic variable service. It must be evaluated whether they should be fully merged instead of just integrated as they are right now. Streamline AI Variable Service Feb 3, 2025
@sdirix sdirix added the TheiaAI label Feb 3, 2025
@JonasHelming
Copy link
Contributor

Please keep future use cases in mind, specifically, we might extend the variable service to also resolve prompt fragments and for this we might need several arguments for variables and auto completion

CamilleLetavernier added a commit to eclipsesource/theia that referenced this issue Feb 4, 2025
@CamilleLetavernier
improve integration between VariableRegistry and AI
6db21d6 
VariableService

- make TheiaVariableContribution more easily to customize/override

fixes eclipse-theia#14817

Signed-off-by: Camille Letavernier <[email protected]>
@CamilleLetavernier CamilleLetavernier linked a pull request Feb 4, 2025 that will close this issue
Open
2 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
TheiaAI
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

improve integration between VariableRegistry and AI VariableService eclipsesource/theia
2 participants
@JonasHelming @sdirix