[UX] add option to register multiple tools in one go

[sks]

No description provided.

[strowk]

Hey, I am honestly not quite sure how this would increase usability. Could you clarify why writing every tool in separate line is a problem for you? Is there very many tools?

I mean, you can just write

WithTool(NewTool1).
WithTool(NewTool2).
// etc ...

This does not look to me like a particularly problematic way compared to

WithTool(NewTool1, NewTool2).

, while second requires a bit more work internally and besides then the method probably should be WithTools, thus increasing amount of methods too which would complicate scanning over available options for everyone who reads that file.

[sks]

Hey, I am honestly not quite sure how this would increase usability. Could you clarify why writing every tool in separate line is a problem for you? Is there very many tools?

I mean, you can just write

WithTool(NewTool1).
WithTool(NewTool2).
// etc ...

This does not look to me like a particularly problematic way compared to

WithTool(NewTool1, NewTool2).

, while second requires a bit more work internally and besides then the method probably should be WithTools, thus increasing amount of methods too which would complicate scanning over available options for everyone who reads that file.

I am on the edge on this one,

I have couple of existing structs that I would like to expose as tools

type MCPToolProvider interface {
	Tools() []fxctx.Tool
}

Current approach is to get all the tool one by one and then add them.
If I had a spread operator , then I could do removing the need to declare another var to iterate over

WithTool(provider.Tool()...).

[sks]

Closing the PR as I have a workaround and the func signature is not compatible anyway

for _, provider := range mcpProviders {
		for _, tool := range provider.Tools(ctx) {
			appBuilder.WithTool(tool)
		}
	}