Skip to content

[AI Infra] Add docLinks to GenAI Settings Documentation section#246657

Merged
spong merged 3 commits intoelastic:mainfrom
spong:genai-settings-doclinks
Dec 17, 2025
Merged

[AI Infra] Add docLinks to GenAI Settings Documentation section#246657
spong merged 3 commits intoelastic:mainfrom
spong:genai-settings-doclinks

Conversation

@spong
Copy link
Copy Markdown
Member

@spong spong commented Dec 16, 2025

Summary

As a follow-up to #246099 and #246334, this PR adds docLinks off to the AI Assistant Settings Docs to both the Documentation panel and error toasts when the artifact fails to install (in aid of those in air-gapped environments).

Note: Yes, air-gapped deployments won't be able to load the documentation link, but they'll at least have the URL they need to get further information.

Checklist

Check the PR satisfies following conditions.

Reviewers should verify this PR satisfies this list as well.

PR developed with Cursor + GPT 5.2

@spong spong self-assigned this Dec 16, 2025
@spong spong requested review from a team as code owners December 16, 2025 22:50
@spong spong added release_note:skip Skip the PR/issue when compiling release notes backport:skip This PR does not require backporting v9.3.0 labels Dec 16, 2025
Copy link
Copy Markdown
Member

@florent-leborgne florent-leborgne left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

Copy link
Copy Markdown
Member

@qn895 qn895 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM 🎉

@spong spong enabled auto-merge (squash) December 16, 2025 23:42
@spong spong merged commit b690ed0 into elastic:main Dec 17, 2025
14 checks passed
@elasticmachine
Copy link
Copy Markdown
Contributor

💚 Build Succeeded

Metrics [docs]

Public APIs missing comments

Total count of every public API that lacks a comment. Target amount is 0. Run node scripts/build_api_docs --plugin [yourplugin] --stats comments for more detailed information.

id before after diff
@kbn/doc-links 88 89 +1

Async chunks

Total size of all lazy-loaded chunks that will be downloaded as the user navigates the app

id before after diff
aiAssistantManagementSelection 90.8KB 90.9KB +89.0B
genAiSettings 58.5KB 59.0KB +592.0B
lists 129.7KB 129.8KB +89.0B
total +770.0B

Page load bundle

Size of the bundles that are downloaded on every page load. Target size is below 100kb

id before after diff
core 496.6KB 496.6KB +89.0B
genAiSettings 5.0KB 5.1KB +70.0B
total +159.0B
Unknown metric groups

API count

id before after diff
@kbn/doc-links 88 89 +1

History

cc @spong

@spong spong deleted the genai-settings-doclinks branch December 17, 2025 02:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backport:skip This PR does not require backporting release_note:skip Skip the PR/issue when compiling release notes v9.3.0

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants