Add a search AD results tool

[ohltyler]

Description

This PR adds a search AD results tool. It exposes several different parameters to search and sort the desired results. Note these are subject to change as more integration testing is done with agents configured with the tool.

Testing:

• manually tested the generated queries against a test cluster to ensure they work as expected against different scenarios and different types of anomaly results
• adds limited UT consistent with existing coverage of tools for now. Again this may be improved when the inputs/outputs/parameters become more standardized and we can have more robust coverage and detailed test cases. We can see how the performance is when we benchmark the tools in integration tests with an equipped agent configured with an LLM

Issues Resolved

opensearch-project/ml-commons#1548

Check List

• New functionality includes testing.
  • All tests pass
• New functionality has been documented.
  • New functionality has javadoc added
• Commits are signed per the DCO using --signoff

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

[codecov]

Codecov Report

Attention: 16 lines in your changes are missing coverage. Please review.

Comparison is base (cd1cfc4) 44.41% compared to head (bdfdc28) 81.57%.
Report is 6 commits behind head on main.

Files	Patch %	Lines
...ensearch/agent/tools/SearchAnomalyResultsTool.java	80.72%	7 Missing and 9 partials ⚠️

Additional details and impacted files

@@              Coverage Diff              @@
##               main      #52       +/-   ##
=============================================
+ Coverage     44.41%   81.57%   +37.16%     
- Complexity       30       78       +48     
=============================================
  Files             6        7        +1     
  Lines           358      456       +98     
  Branches         42       63       +21     
=============================================
+ Hits            159      372      +213     
+ Misses          178       45      -133     
- Partials         21       39       +18     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[dhrubo-os]

can we add sufficient test for this method?

[ohltyler]

Added

[dhrubo-os]

can't we add more info in the test like anomalyGradeThreshold, dataStartTime, dataEndTime etc?

[ohltyler]

Added for other valid cases with the other params. But there won't be any other invalid cases since the other params are tested with isNumeric, which if it fails, will default to null values and not throw exceptions.

I don't use isNumeric specifically because of #52 (comment)

[zane-neo]

Do we need to check if the parameters.get("anomalyGradeThreshold") is a number before parsing to Double?

[ohltyler]

isNumeric requires each character to be a digit, but users may want to pass -1 for example. Prefer to leave as-is, as -1 may be a common value (within LLM prompt potentially) to indicate they want to fetch all anomaly results, regardless if they are actual anomalies with grades > 0.