[FEATURE] Function getToolsParametersMap in ToolStep should parse connector id

[yuye-aws]

Is your feature request related to a problem?

When creating Connector tool with connector_id from previous node inputs, connector id is missing.

What solution would you like?

Add connector_id into the parsing function in ToolStep.

What alternatives have you considered?

A clear and concise description of any alternative solutions or features you've considered.

Do you have any additional context?

Add any other context or screenshots about the feature request here.

[dbwiddis]

Great request, @yuye-aws . Feel free to submit a PR.

You'd just need to add Connector ID to this line:

flow-framework/src/main/java/org/opensearch/flowframework/workflow/ToolStep.java

Line 51 in 3a88199

Set<String> optionalKeys = Set.of(NAME_FIELD, DESCRIPTION_FIELD, PARAMETERS_FIELD, INCLUDE_OUTPUT_IN_AGENT_RESPONSE);

and add a getter like this:

flow-framework/src/main/java/org/opensearch/flowframework/workflow/ToolStep.java

Lines 63 to 67 in 3a88199

	String type = (String) inputs.get(TYPE);
	String name = (String) inputs.get(NAME_FIELD);
	String description = (String) inputs.get(DESCRIPTION_FIELD);
	Boolean includeOutputInAgentResponse = ParseUtils.parseIfExists(inputs, INCLUDE_OUTPUT_IN_AGENT_RESPONSE, Boolean.class);
	Map<String, String> parameters = getToolsParametersMap(inputs.get(PARAMETERS_FIELD), previousNodeInputs, outputs);

and a null-checking builder line like this:

flow-framework/src/main/java/org/opensearch/flowframework/workflow/ToolStep.java

Lines 72 to 83 in 3a88199

	if (name != null) {
	builder.name(name);
	}
	if (description != null) {
	builder.description(description);
	}
	if (parameters != null) {
	builder.parameters(parameters);
	}
	if (includeOutputInAgentResponse != null) {
	builder.includeOutputInAgentResponse(includeOutputInAgentResponse);
	}

[yuye-aws]

Thanks! TBH, I am new to this repo. Will submit a PR soon.

[dbwiddis]

Let us know if you need any more help! There may be additional steps (not sure if the builder has the setter you need, etc.) but the above steps should get you started!

[yuye-aws]

Thanks. I think the connector_id is just like the model_id. I will implement this feature within function getToolsParametersMap.