[Bug fix]: CLI overwriting an app does not update configuration

[aybruhm]

Description

This PR resolves the bug that overwrites an app from the CLI, which leads to the configuration not been updated.

Related Issue

Closes AGE-396

Changes

• Set api_key to an empty string (in OSS) when it is None to avoid fern raising a NoneType error during request header serialization.
• Removed the overwrite field from the SaveConfigPayload API model and associated arguments to eliminate redundancy and potential confusion.
• Removed the use of the redundant payload.overwrite and replaced print statements with logger.info for better logging practices in save_config API router.
• Ensured that variant parameters are updated accordingly during the overwrite process.
• Removed redundant code in the serve_cli variant command in agenta-cli.

[vercel]

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name	Status	Preview	Comments	Updated (UTC)
agenta	✅ Ready (Inspect)	Visit Preview	💬 Add feedback	Jul 9, 2024 2:54pm

[mmabrouk]

Wouldn't the right fix just modify line 43 in agenta_init.py to be
api_key: Optional[str] = ""?

I don't understand the reason for all the other changes (except 480 in variant_commands which looks like dead code, and 1784 in db_manager which fixed an edge case and the use of logger instead of print)

[aybruhm]

Wouldn't the right fix just modify line 43 in agenta_init.py to be api_key: Optional[str] = ""?

I don't understand the reason for all the other changes (except 480 in variant_commands which looks like dead code, and 1784 in db_manager which fixed an edge case and the use of logger instead of print)

Noted. I'll create a new PR with the right changes.