Improvement to the ai module

[reflog]

Hi @abenz1267

I've 'generalized' the ai module and created another provider - Gemini.
What do you think of this approach?

[abenz1267]

Hi,

thanks for working on that. I think the general approach is fine, this PR doesn't function yet though.

Small nitpicks:

• Anthropic API returned status code 200 => avoid console messages. If you wanna log something, use slog and hide it behind a debug-flag
• antropic.go => anthropic.go
• the label should say which model/company is being used, put that info in the sub, where the model is displayed

As for not working: console spits out Anthropic API returned status code 200, but the answer isn't shown.

[reflog]

@abenz1267 thanks. I've addressed your comments.
The only thing that is not clear to me is the 'sub' usage. When I put model info in it, it doesn't appear in the UI (in the screenshot I have gemini and claude with the same assistant name, but they are indistinguishable from each other)

[abenz1267]

@abenz1267 thanks. I've addressed your comments. The only thing that is not clear to me is the 'sub' usage. When I put model info in it, it doesn't appear in the UI (in the screenshot I have gemini and claude with the same assistant name, but they are indistinguishable from each other)

Modules/Plugins have a show_sub_when_single boolean setting that can be set to true. By default the sub won't be shown, when there's only 1 modules, f.e. when doing walker -m ai.

This could be simply changed in the default config.

[reflog]

@abenz1267 lovely, that works great. I've updated the default on the ai module

[abenz1267]

This looks rather repetitive to me. It should probably be handled in somekinda loop.

[abenz1267]

Also: do you really need the "New*Provider" functions? To me it looks like everything should be handled in the providers SetupData method, since you have a loop calling that already anyways. in ai.SetupData.