feat: require one of key, instead of mandatory OD-166#15
Conversation
There was a problem hiding this comment.
Pull request overview
This PR updates the server-side pipeline entrypoint to support running configure-codacy-cloud with either Claude (Anthropic) or Gemini, requiring at least one LLM API key rather than mandating ANTHROPIC_API_KEY.
Changes:
- Removes
ANTHROPIC_API_KEYfrom the always-required environment variable list and replaces it with a “require at least one of ANTHROPIC_API_KEY or GEMINI_API_KEY” check. - Adds a provider switch to run the skill via
claudewhenANTHROPIC_API_KEYis set, otherwise viageminiwhenGEMINI_API_KEYis set. - Captures and appends run metadata for both LLM paths into the summary JSON.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
![gemini-code-assist[bot]](https://avatars.githubusercontent.com/in/956858?s=60&v=4){kind=small}
There was a problem hiding this comment.
Code Review
This pull request updates the docker/server-pipeline.sh script to support both Anthropic (Claude) and Gemini models. It removes ANTHROPIC_API_KEY from the strictly required variables list and instead adds a check to ensure that at least one of ANTHROPIC_API_KEY or GEMINI_API_KEY is provided. Depending on which key is present, the script executes the corresponding CLI tool (claude or gemini) and extracts the run metadata accordingly. There are no review comments, so I have no feedback to provide.
Important
The consumer version of Gemini Code Assist on GitHub is being sunset. Starting June 18, 2026, new organization installations will be blocked, and all code review activity will officially cease on July 17, 2026.
For more details on the timeline and next steps, please review the Help Documentation.
No description provided.