Add resources section to README to include ai-declare skill#22
Conversation
|
@yujqiao, thank you for the skill. I believe it will help people a lot and make things easier for them! I invite new tooling so this is welcome! That said, I have a few nitpicks. Isn't it a huge overhead to keep updating the skill with the right version instead of it somehow fetching or inferring the latest version from the spec? Again, I don't mind but I think you will have to update it each time the spec updates and it would be a chore for you. Up to you on this one. (source) Also, although verbose, I would suggest that the options a user gets remain the same as the spec's usage definitions (and be updated accordingly) if future specs change things. These are discussed and deliberated over on GitHub, Reddit, and other places, and the phrasing is a feature in a way. Also, I know the spec will be written correctly, but then this defeats a sort of assumed WYSIWYG for the user of the skill. (source). |
DeepanshKhurana
left a comment
There was a problem hiding this comment.
This is a great idea, and I thank you for the initiative. Let's discuss on my other comments. I'm curious to hear what you think!
| ䷼ Hexagram 61 or Hexagram For Inner Truth (Unicode: `U+4DFC`) is one of 64 hexagrams in the Yi (I) Ching to illustrate principles where each line is either Yin (broken) or Yang (solid). ([source](https://en.wikipedia.org/wiki/List_of_hexagrams_of_the_I_Ching#Hexagram_61)) | ||
|
|
||
| ## Resources | ||
| - [ai-declare](https://github.com/yujqiao/ai-declare) agent skill to generate `AI-DECLARATION.md` |
There was a problem hiding this comment.
I recommend changing this to:
- [yujqiao/ai-declare](https://github.com/yujqiao/ai-declare) agent skill to generate `AI-DECLARATION.md`
|
💡 If the spec exposed an API that was a simple, no auth, REST GET endpoint that simply gave the current schema of the spec, would a skill be able to consume it? |
Pull Request Template
Thank you for your contribution!
Note
Before submitting this PR, please ensure You have opened a related issue and referenced it below.
Related Issue:
This is a simple and straight-forward change without an issue. Feel free to close this if you think it's not the right place to include
Description of Changes:
Add resources section to README to include
ai-declareskillAI-DECLARATION.md:
AI-DECLARATION.mdAI-DECLARATION.mdto reflect AI usage in this contribution (or confirmed no update is needed)Additional Notes:
Feel free to close this if you think it's not the right place to include