API key from Env Var support

[suprjinx]

Related GitHub Issue

Closes: #4337

Description

This PR adds support for obtaining API keys from the environment. A checkbox is added to each configuration that allows inputting an API key -- when checked, a given environment variable (eg, OPENAI_API_KEY, etc) is consulted for the key and the user is not required to provide one in the configuration. Env Var key is predictable but not editable.

Test Procedure

I tested this using VS Code run configurations, where the expected environment variable could be provided or omitted.

Pre-Submission Checklist

• Issue Linked: This PR is linked to an approved GitHub Issue (see "Related GitHub Issue" above).
• Scope: My changes are focused on the linked issue (one major feature/fix per PR).
• Self-Review: I have performed a thorough self-review of my code.
• Testing: New and/or updated tests have been added to cover my changes (if applicable).
• Documentation Impact: I have considered if my changes require documentation updates (see "Documentation Updates" section below).

A documentation update is advised, to describe environment variable usage to supply API keys.

• Contribution Guidelines: I have read and agree to the Contributor Guidelines.

Screenshots / Videos

Documentation Updates

Additional Notes

Get in Touch

---

Important

Adds support for using environment variables for API keys across multiple providers, updates UI components to reflect this, and enhances testing and translation capabilities.

• Behavior:
  • Adds support for API keys from environment variables for providers like anthropic, glama, openrouter, openai, gemini, mistral, deepseek, unbound, requesty, xai, groq, chutes, litellm in provider-settings.ts.
  • Updates validateModelsAndKeysProvided() in validate.ts to check for environment variable usage for API keys.
• UI Components:
  • Introduces ApiKey component in ApiKey.tsx to handle API key input and environment variable toggle.
  • Updates various provider components like Anthropic.tsx, Chutes.tsx, DeepSeek.tsx, etc., to use the new ApiKey component.
• Testing:
  • Adds tests for environment variable integration in index.spec.ts and ApiKey.spec.tsx.
• Misc:
  • Adds --create-missing option in find-missing-translations.js for generating placeholder translation keys.

^{This description was created by for f154eb2. You can customize this summary. It will automatically update as commits are pushed.}

[suprjinx]

Thanks, @daniel-lxs -- addressed these issues and added some new providers, let me know if you see any other problems!

[suprjinx]

@daniel-lxs would love to get this approved, can you take a look at the changes since your last comment?

[roomote]

Review Summary

I've reviewed the latest changes (commit 7b24b89) and found no new issues. All previously flagged issues have been successfully addressed.

Issues to Fix

• Fix typo in constant name: MOONSHOOT should be MOONSHOT in packages/types/src/constants.ts
• Remove trailing whitespace in packages/types/src/provider-settings.ts (lines 372, 397, 417)
• Add validation checks for new providers with env var support in webview-ui/src/utils/validate.ts (deepinfra, huggingface, cerebras, fireworks, featherless, io-intelligence, vercel-ai-gateway, doubao, moonshot)

Overall Assessment

The implementation is well-structured and follows a consistent pattern across all providers. The core functionality for reading API keys from environment variables is solid. All validation logic for the newly added providers has been properly implemented.

Follow Along on Roo Code Cloud

Previous Reviews

• 3823e1b: Review #1
• 564cc9a: Review #2
• 142412c: Review #3
• 4597469: Review #4
• 5048312: Review #5

[roomote]

Review complete. Found 3 issues that should be addressed before merging. Please see the inline comments for details.

[suprjinx]

@daniel-lxs I addressed the issues found by the review bot with 4597469

[roomote]

No issues found in the latest commit (142412c). The translation updates look good.

[roomote]

Review updated. The formatting commit looks good - all trailing whitespace issues have been resolved.