Skip to content

Conversation

JonasHelming
Copy link
Contributor

fixed #15174

What it does

registers a schema for the agent settings

How to test

Open the settings.json and see that agent settings do not show a warning anymore

Follow-ups

Breaking changes

  • This PR introduces breaking changes and requires careful review. If yes, the breaking changes section in the changelog has been updated.

Attribution

Review checklist

Reminder for reviewers

@JonasHelming JonasHelming requested a review from sdirix March 12, 2025 15:43
@github-project-automation github-project-automation bot moved this to Waiting on reviewers in PR Backlog Mar 12, 2025
Copy link
Member

@sdirix sdirix left a comment

Choose a reason for hiding this comment

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

This works in the JSON Editor. However as a consequence this adds an empty title into the preferences.

image

Ideally we should get rid of it. Here or in a follow up issue.

@github-project-automation github-project-automation bot moved this from Waiting on reviewers to Needs merge in PR Backlog Mar 13, 2025
Co-authored-by: Stefan Dirix <sdirix@eclipsesource.com>
@colin-grant-work
Copy link
Contributor

However as a consequence this adds an empty title into the preferences.

I believe that that isn't a consequence of how this preference is declared, but of this bug:

#15165

@colin-grant-work colin-grant-work merged commit 678af5d into master Mar 13, 2025
9 of 11 checks passed
@github-project-automation github-project-automation bot moved this from Needs merge to Done in PR Backlog Mar 13, 2025
@github-actions github-actions bot added this to the 1.60.0 milestone Mar 13, 2025
@sdirix
Copy link
Member

sdirix commented Mar 14, 2025

@colin-grant-work good catch. When adapting the labels.length code as indicated by you, the preference renders properly.

laemmleint pushed a commit to mvtecsoftware/theia that referenced this pull request Aug 18, 2025
Fixes eclipse-theia#15174

Co-authored-by: Stefan Dirix <sdirix@eclipsesource.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

[Theia AI] Agent settings do not validate in preferences
3 participants