-
Notifications
You must be signed in to change notification settings - Fork 1
Update skills and add scripts and prompts to keep skills up to date #4
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.
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
base: main
Are you sure you want to change the base?
Conversation
|
@simon-johansson I'm not sure whether it's the best approach to use prompts in favor of scripts due to accuracy. I've had experience in the past with a similar prompt-based approach when I had to fix broken links in certain pages which I did by telling the LLM to run the contents against a sitemap and to ensure that all of the links are valid and exist - but that worked only once out of a couple of times. Few questions:
Other than that, looks great - much cleaner and efficient than the instructions we have now. |
|
Good questions @ivancernja, sorry for not being more clear.
Quite good. It is of course not deterministic and I don't expect the AI to be perfect in suggesting what skills to update but I think this is a good complement to updating the skills by hand.
It's only for us internally. So when we have done some changes to the docs I think we can run the prompts to get some guidance in what skills needs to be updated.
I don't think we need to make it automatic. I think it's good enough to run the prompts manually. The prompts will help when updating the skills, which we will need to do anyway. |
|
Thanks @simon-johansson.
My only concern here is that the skills repository is public and we accept external contributions. Since the prompts and scripts reference internal workflows ("for us internally"), it might not be clear to outside contributors how they fit into the maintenance process or whether they should use them when proposing updates. This makes me wonder if we should reconsider whether the repo should remain public, or if we're actually seeing value from external contributions that justifies keeping it open. If it's primarily an internal tool, we might want to make it private to avoid confusion. |
|
Updated the PR to remove the prompts and the scripts. |
docs-summaries/README.md
Outdated
|
|
||
| ## How to use | ||
|
|
||
| 1. Run the `update-docs-summaries-prompt.md` using Claude Code prompt to update the `ts-docs-summary.md`, `go-docs-summary.md`, and `platform-docs-summary.md` files. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
| 1. Run the `update-docs-summaries-prompt.md` using Claude Code prompt to update the `ts-docs-summary.md`, `go-docs-summary.md`, and `platform-docs-summary.md` files. | |
| 1. Run the `update-docs-summaries-prompt.md` prompt using Claude Code to update the `ts-docs-summary.md`, `go-docs-summary.md`, and `platform-docs-summary.md` files. |
I added some scripts and prompts we can run to update the skills when the docs gets updated. These prompts should probably run for the
llm-instructions.txtas well but that is not in this repo so.... figure that out later.