AIDER-1977: Add a new argument litellm-extra-params

[mark-bradshaw]

Add a new config argument litellm-extra-params that takes a string of JSON that should be used as a global litellm config base. Values set in the model settings will override those in the global config.

[fry69]

This is what I meant with my "update" in the other comment:

If you do not have the LiteLLM repo installed alongside the aider repo (as a sibling folder, not inside the aider repo of course) you'll delete all these generated API key names.

@paul-gauthier This is an unmaintainable nightmare IMHO and should get fixed.

[mark-bradshaw]

Whoops. I’ll revert that.

[paul-gauthier]

Thanks for your interest in contributing to aider. It seems very unusual to have a command line parameter that accepts a json blob. I don't think we can adopt that approach. Unfortunately, I'm not sure I have a better suggestion yet though.

[mark-bradshaw]

@paul-gauthier Well, my need is tied just to adding headers, so if it feels better I can tighten it down to a semicolon separated list of headers to add. Ex "header1=value1;header2=value2".

[mark-bradshaw]

OK, here's another try. I switched to a semicolon separated list of header values. It's just a string, so no JSON parsing any more. Code is simpler too.

[CLAassistant]

All committers have signed the CLA.

[mark-bradshaw]

@paul-gauthier I rebased the code to trim it down and fix up the wrong name/email on commits. I also signed the CLA. Ready to go.

[mark-bradshaw]

@paul-gauthier @fry69 Is there anything in the updated approach that you would like changed?