Auto generate skill for agent-skills-sandbox repo#2466
Auto generate skill for agent-skills-sandbox repo#2466tehbooom wants to merge 4 commits intoelastic:mainfrom
Conversation
|
❌ PR Changelog Check failed The following issues were found with the
Expected formatOr add the |
| | Subsystem | Endpoint env | Auth envs | | ||
| |---|---|---| | ||
| | Elasticsearch | `ELASTICSEARCH_ENDPOINTS` | `ELASTICSEARCH_USERNAME` + `ELASTICSEARCH_PASSWORD`, or `ELASTICSEARCH_API_KEY`, or `ELASTICSEARCH_BEARER_TOKEN` | | ||
| | Kibana | `KIBANA_ENDPOINT` | `KIBANA_USERNAME` + `KIBANA_PASSWORD`, or `KIBANA_API_KEY` | |
There was a problem hiding this comment.
🟢 Low references/provider.md:38
The Kibana auth environment variables table at line 38 omits KIBANA_BEARER_TOKEN, which is a supported authentication mechanism per the provider implementation. Since this document is used to generate AI skills, the missing token type means generated skills will never suggest bearer token authentication for Kibana even when it is valid.
-| Kibana | `KIBANA_ENDPOINT` | `KIBANA_USERNAME` + `KIBANA_PASSWORD`, or `KIBANA_API_KEY` |
+| Kibana | `KIBANA_ENDPOINT` | `KIBANA_USERNAME` + `KIBANA_PASSWORD`, or `KIBANA_API_KEY`, or `KIBANA_BEARER_TOKEN` |🤖 Copy this AI Prompt to have your agent fix this:
In file scripts/generate-skill/assets/references/provider.md around line 38:
The Kibana auth environment variables table at line 38 omits `KIBANA_BEARER_TOKEN`, which is a supported authentication mechanism per the provider implementation. Since this document is used to generate AI skills, the missing token type means generated skills will never suggest bearer token authentication for Kibana even when it is valid.
tobio
left a comment
There was a problem hiding this comment.
I'm curious what this solves over one of:
- Referencing https://registry.terraform.io/providers/elastic/elasticstack/latest/docs
- Embedding the
docsdirectory (well formed, generated from the implementation so no chance of drift)
I suppose referencing HTTP docs in a skill might be less effective than sourcing local content, but basing this off the generated docs feels like a better path (they're for the right audience, where the specs are more developer focused).
You're right using the docs directory is better than the specs |
Auto generates a skill to put in elastic/agent-skills-sandbox repo
Will need to discuss if this fits here or if we would like to have another repo pull in the providers to generate and then push to that repo.
Note
Add skill generator CLI for the elasticstack-terraform agent-skills-sandbox repo
scripts/generate-skillthat produces a deterministic skill directory tree underdist/skill/by combining hand-authored assets and provider docs.docs/resourcesanddocs/data-sources, parses frontmatter, and merges entities into a generatedreferences/index.mdwith descriptions and file links.SKILL.mdand reference docs) into the output tree, substituting{{VERSION}}with the value passed via--provider-version.GENERATED.mdprovenance file summarising entity counts and provider version.make skill-generateandmake skill-testtargets to invoke the generator and its unit tests respectively.Macroscope summarized 6f18ed1.