Pull request overview

Updates validation messaging to correctly associate unsupported attributes with VS Code skills rather than agents.

Changes:

• Corrected the localized unsupported skill attribute message.

This is the case. Is there anything you can do?

Copilot can you do this or not buddy?

The change is incomplete and/or incorrect and does not seem to solve a specific problem.

The change is incomplete and/or incorrect and does not seem to solve a specific problem.

Hi Dmitriy Vasyura (@dmitrivMS). Was this review manual or automated?

If manual, did you take a peek at the diff, it tells the story best, especially if you compare the changed line with line 311. This is a one-word change, where it seems a copy/ paste error occurred. My take is that the author meant to copy the string from the agents error message to use for the skills error message, but the word agents never got updated correctly to the word skills.

If automated, that's exciting, but is your automation pulling in the commentary from Copilot's first post? Its summary is better than what I would have generated or what I put in the prior paragraph just now really :).

Hi, I reviewed, commented, and closed, not agent.
I did look at the diff. There are other strings that mention "agents", so I think the change is inconsistent.

I see the miscommunication (as I get better at sharing context with the harness, I'm worried I'm getting rusty at sharing it with actual humans 😅).

Below is a screenshot of VS Code surfacing the error that this PR updates. I added a key that isn't listed in the Skill schema, and can see the error message when hovering over the invalid key.

A Skill file is open, but the error message suggests that it's interpreting the SKILL.md file as an AGENT file (e.g. foo.agent.md). This confused me and created a suspicion that VS Code wasn't correctly processing the schema for Skill files. This led to a fun bug hunt adventure/ distraction into VS Code that went as far as discovering it was just a simple typo.

That word should be corrected to accurately indicate that the file in question is correctly being interpreted as a Skill file and not an Agent file, right?

TheNotary Thanks a lot for the explanation and proving me wrong!

يجب تغير الوجه لي الاتنين

والرساله تم ارسلها خطأ

تم

TheNotary Thanks a lot for the explanation and proving me wrong!

Ha, any time 😄 and thanks for the work you're all doing on the app!