Add model related field for tools

[xinyual]

Description

To check downstream task before deleting, we need to add list of model related fields for model related tools. The background PR is: opensearch-project/ml-commons#3209

Related Issues

Resolves #[Issue number to be closed when this PR is merged]

Check List

• New functionality includes testing.
• New functionality has been documented.
• API changes companion pull request created.
• Commits are signed per the DCO using --signoff.
• Public documentation issue/PR created.

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.

[zhichao-aws]

Can we use this static variable to replace "model_id" constant in factory?

[zane-neo]

I see this constant been used in multiple classes, let's create a common constant class, eg. CommonConstants to put it there, and that class can be used to put more global constants in the future.

[xinyual]

I see this constant been used in multiple classes, let's create a common constant class, eg. CommonConstants to put it there, and that class can be used to put more global constants in the future.

I think each tool should have its own related tool field so we cannot use a generate one for all tools

[zane-neo]

Not quite understand, I mean this constant:
https://github.com/opensearch-project/skills/pull/491/files#diff-e7cc44d1e3c01541c76a3d3f5308487536871c349b84cf388baa9fb8c45a15a6R74 and
https://github.com/opensearch-project/skills/pull/491/files#diff-cacec66258e0f1410446a7f762a38af26fb9d2874e562151820f0d0e55f95f89R36 and
https://github.com/opensearch-project/skills/pull/491/files#diff-fa8cc057d2cd63132a2728fabab484d8b9b254faab603f50c189e50e12882d18R72
I see they're all string with value model_id, this can be placed to a common place. This is not a big change and if you think you can optimize this in future PRs, I'm fine to approve.

[zhichao-aws]

It seems we have a compile error. Please fix it before merge.

/Users/runner/work/skills/skills/src/main/java/org/opensearch/agent/tools/VectorDBTool.java:115: error: Factory cannot be inherited with different arguments: <org.opensearch.agent.tools.NeuralSparseSearchTool> and <org.opensearch.agent.tools.VectorDBTool>
    public static class Factory extends AbstractRetrieverTool.Factory<VectorDBTool>

[codecov]

Codecov Report

Attention: Patch coverage is 65.00000% with 7 lines in your changes missing coverage. Please review.

Project coverage is 81.90%. Comparing base (2b76e3c) to head (c67debe).
Report is 43 commits behind head on main.

Files with missing lines	Patch %	Lines
...va/org/opensearch/agent/tools/CreateAlertTool.java	66.66%	1 Missing ⚠️
...nsearch/agent/tools/CreateAnomalyDetectorTool.java	66.66%	1 Missing ⚠️
...opensearch/agent/tools/NeuralSparseSearchTool.java	66.66%	1 Missing ⚠️
.../main/java/org/opensearch/agent/tools/PPLTool.java	66.66%	1 Missing ⚠️
.../main/java/org/opensearch/agent/tools/RAGTool.java	75.00%	1 Missing ⚠️
.../java/org/opensearch/agent/tools/VectorDBTool.java	66.66%	1 Missing ⚠️
.../opensearch/agent/tools/utils/CommonConstants.java	0.00%	1 Missing ⚠️

Additional details and impacted files

@@             Coverage Diff              @@
##               main     #491      +/-   ##
============================================
+ Coverage     81.78%   81.90%   +0.11%     
- Complexity      193      335     +142     
============================================
  Files            11       17       +6     
  Lines           961     1586     +625     
  Branches        137      222      +85     
============================================
+ Hits            786     1299     +513     
- Misses          121      201      +80     
- Partials         54       86      +32     

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