Skip to content

Reapply "[Security Solution] [Attack discovery] Output chunking / refinement, LangGraph migration, and evaluation improvements (#195669)"#196440

Merged
jbudz merged 2 commits intoelastic:mainfrom
jbudz:reapply-195669
Oct 16, 2024
Merged

Conversation

@jbudz
Copy link
Copy Markdown
Contributor

@jbudz jbudz commented Oct 15, 2024

Reapplies a revert I made in dbe6d82. See #195669 for full description.

Also includes #196381 to fix an import statement.

@jbudz
Copy link
Copy Markdown
Contributor Author

jbudz commented Oct 15, 2024

buildkite test this

1 similar comment
@jbudz
Copy link
Copy Markdown
Contributor Author

jbudz commented Oct 15, 2024

buildkite test this

@jbudz jbudz marked this pull request as ready for review October 15, 2024 23:44
@jbudz jbudz requested review from a team as code owners October 15, 2024 23:44
@elasticmachine
Copy link
Copy Markdown
Contributor

💚 Build Succeeded

Metrics [docs]

Module Count

Fewer modules leads to a faster build time

id before after diff
integrationAssistant 451 456 +5
securitySolution 6016 6030 +14
total +19

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/elastic-assistant 133 139 +6
@kbn/elastic-assistant-common 366 372 +6
total +12

Async chunks

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

id before after diff
securitySolution 20.6MB 20.7MB +16.4KB
Unknown metric groups

API count

id before after diff
@kbn/elastic-assistant 159 168 +9
@kbn/elastic-assistant-common 393 403 +10
total +19

ESLint disabled line counts

id before after diff
elasticAssistant 35 32 -3

Total ESLint disabled count

id before after diff
elasticAssistant 35 32 -3

History

@jbudz jbudz added backport:skip This PR does not require backporting release_note:skip Skip the PR/issue when compiling release notes labels Oct 16, 2024
@jbudz jbudz merged commit ad2ac71 into elastic:main Oct 16, 2024
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.0.0

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants