Skip to content

Conversation

p-spacek
Copy link
Contributor

@p-spacek p-spacek commented Sep 8, 2025

What does this PR do?

Add expected properties to the diagnostic message

image

The reason for displaying property information is AI agents. AI chat tools like Copilot or Claude can identify issues during the generation of YAML and the validation process and fix generated issues in the next iteration.

Note that a similar implementation already has enums.
image

Note that it can't see the 'data' property (some extra information is already there, but not accessible by AI).

What issues does this PR fix or reference?

Is it tested? How?

modified existing tests

Petr Spacek added 30 commits December 8, 2021 16:08
 - with short nextLine - nested object
 - with a new line inside the object
 - on the first array item
p-spacek added 24 commits March 25, 2025 21:59
* feat: filePatternAssociation modification
 - glob vs regexp customization

* chore: add test
commit cb38345
Author: Petr Spacek <[email protected]>
Date:   Tue Apr 22 15:00:26 2025 +0200

    chore: update comments

commit 07d741d
Author: Petr Spacek <[email protected]>
Date:   Tue Apr 22 14:53:56 2025 +0200

    fix: exclude not suggest properties from possible properties error
* fix: hover detail const and enum

* fix: improve hover for array type
* feat: value completion condition support

* chore: remove empty lines
@coveralls
Copy link

coveralls commented Sep 8, 2025

Coverage Status

coverage: 35.229% (-48.7%) from 83.956%
when pulling 8f139db on jigx-com:feat/add-expected-properties-to-diagnostic-message
into 3821411 on redhat-developer:main.

Copy link

@github-advanced-security github-advanced-security bot left a comment

Choose a reason for hiding this comment

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

ESLint found more than 20 potential problems in the proposed changes. Check the Files changed tab for more details.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

7 participants