Skip to content

Conversation

kpollich
Copy link
Member

@kpollich kpollich commented Oct 14, 2025

Proposed commit message

Make existing README available as a knowledge base document for Elastic AI assistants.

Checklist

  • I have reviewed tips for building integrations and this pull request is aligned with them.
  • I have verified that all data streams collect metrics or logs.
  • I have added an entry to my package's changelog.yml file.
  • I have verified that Kibana version constraints are current according to guidelines.
  • I have verified that any added dashboard complies with Kibana's Dashboard good practices

@kpollich kpollich self-assigned this Oct 14, 2025
@kpollich kpollich requested review from a team as code owners October 14, 2025 14:44
@kpollich kpollich added the enhancement New feature or request label Oct 14, 2025
@kpollich kpollich requested review from a team as code owners October 14, 2025 14:46
@pierrehilbert pierrehilbert added the Team:Security-Linux Platform Linux Platform Security team [elastic/sec-linux-platform] label Oct 14, 2025
@elasticmachine
Copy link

Pinging @elastic/sec-linux-platform (Team:Security-Linux Platform)

@pierrehilbert pierrehilbert added Team:Elastic-Agent-Data-Plane Agent Data Plane team [elastic/elastic-agent-data-plane] Team:Obs-InfraObs Observability Infrastructure Monitoring team [elastic/obs-infraobs-integrations] labels Oct 14, 2025
@elasticmachine
Copy link

Pinging @elastic/elastic-agent-data-plane (Team:Elastic-Agent-Data-Plane)

@pierrehilbert pierrehilbert added the Team:Elastic-Agent-Control-Plane Label for the Agent Control Plane team label Oct 14, 2025
@elasticmachine
Copy link

Pinging @elastic/elastic-agent-control-plane (Team:Elastic-Agent-Control-Plane)

@elastic-vault-github-plugin-prod

🚀 Benchmarks report

To see the full report comment with /test benchmark fullreport

@elasticmachine
Copy link

💚 Build Succeeded

cc @kpollich

@andrewkroh andrewkroh added documentation Improvements or additions to documentation. Applied to PRs that modify *.md files. Integration:elastic_agent Elastic Agent Integration:system System Team:Elastic-Agent Platform - Ingest - Agent [elastic/elastic-agent] labels Oct 14, 2025
@jen-huang
Copy link
Contributor

jen-huang commented Oct 14, 2025

I did something similar for testing, but coping the exact READMEs does place additional burden on package authors to remember to look for and update them in two places. Not sure how folks feel about that...

It would be interesting if Fleet treated the base READMEs as additional KB content that is automatically indexed, I remember we discussed this during design of KB content but don't remember if we deferred or decided to not do that.

@ishleenk17
Copy link
Member

ishleenk17 commented Oct 15, 2025

I did something similar for testing, but coping the exact READMEs does place additional burden on package authors to remember to look for and update them in two places. Not sure how folks feel about that...

It would be interesting if Fleet treated the base READMEs as additional KB content that is automatically indexed, I remember we discussed this during design of KB content but don't remember if we deferred or decided to not do that.

I tend to agree with @jen-huang on this @kpollich, since the KB README's are manually duplicated and not generated as part of the build.
Could you please share what is the motive for this change and why can't the current README'e be referred to ?

@jen-huang
Copy link
Contributor

I tend to agree with @jen-huang on this @kpollich, since the KB README's are manually duplicated and not generated as part of the build.

Could you please share what is the motive for this change and why can't the current README'e be referred to ?

Fleet currently looks specifically in this /docs/knowledge directory for content to index for consumption by AI assistants. It will be additional work (but not a ton) to also index the base README.

But actually I like the idea of automatically duplicating the base README to that directory during the package build process. That would require no additional Fleet changes or work for package developers. We would just have to come to the consensus that all base READMEs are useful for the AI assistants to consume.

@nimarezainia
Copy link
Contributor

@jen-huang I think at this point in time all the READMEs should be copied into the knowledge directory to give us an immediate content for AI to noodle on. We can always refine/remove that if better content becomes available.

@ishleenk17
Copy link
Member

@jen-huang I think at this point in time all the READMEs should be copied into the knowledge directory to give us an immediate content for AI to noodle on. We can always refine/remove that if better content becomes available.

@nimarezainia : I understand we're planning to roll this out across all Integrations.

Given that Jen mentioned here that it's relatively straightforward, could we incorporate this README generation into the elastic-package build process? Since we update Integrations and their READMEs frequently, manually maintaining these could quickly become outdated without automated generation."

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

Labels

documentation Improvements or additions to documentation. Applied to PRs that modify *.md files. enhancement New feature or request Integration:elastic_agent Elastic Agent Integration:system System Team:Elastic-Agent Platform - Ingest - Agent [elastic/elastic-agent] Team:Elastic-Agent-Control-Plane Label for the Agent Control Plane team Team:Elastic-Agent-Data-Plane Agent Data Plane team [elastic/elastic-agent-data-plane] Team:Obs-InfraObs Observability Infrastructure Monitoring team [elastic/obs-infraobs-integrations] Team:Security-Linux Platform Linux Platform Security team [elastic/sec-linux-platform]

Projects

None yet

Development

Successfully merging this pull request may close these issues.

7 participants