Proposal: update specification — extend agents diagnostics example to show all 9 supported providers #5320

New issue

Closed

opened 2026-04-09 05:50:14 +00:00 by HAL9000 · 0 comments

HAL9000 commented

2026-04-09 05:50:14 +00:00

Owner

Specification Update Proposal

Triggered by: Merged PR #3469 (fix(cli): extend agents diagnostics to check all 9 supported providers)
Classification: Implementation is BETTER than spec — update spec to match

What Changed in the Implementation

PR #3469 extended _check_providers() in src/cleveragents/cli/commands/system.py to report diagnostic status for all 9 providers supported by ProviderRegistry. Previously only 4 providers were checked (OpenAI, Anthropic, Google, OpenRouter); now all 9 are covered including Azure, Gemini, Cohere, Groq, and Together AI.

The ProviderRegistry supports these 9 provider types:

openai (OPENAI_API_KEY)
anthropic (ANTHROPIC_API_KEY)
google (GOOGLE_API_KEY)
gemini (GEMINI_API_KEY)
azure (AZURE_OPENAI_API_KEY)
openrouter (OPENROUTER_API_KEY)
cohere (COHERE_API_KEY)
groq (GROQ_API_KEY)
together (TOGETHER_API_KEY)

What Spec Sections Need Updating

Section: `agents diagnostics` (line ~853)

Current spec (lines 876-878 in the Rich example):

│ OPENAI_API_KEY   WARN    missing        │
│ Anthropic key    OK      configured     │

Only shows 2 providers.

Proposed update: Expand the example to show all 9 providers, demonstrating the comprehensive provider coverage. The example should show a realistic mix of configured and unconfigured providers.

Section: `agents diagnostics` JSON example (line ~946)

Current spec: Only includes OPENAI_API_KEY and Anthropic key checks.

Proposed update: Include all 9 provider checks in the JSON output example.

Rationale

The spec example is misleading — it implies only 2 providers are checked, when the implementation actually checks all 9. Users reading the spec would not know that agents diagnostics provides comprehensive provider coverage. Updating the example to show all 9 providers accurately reflects the implementation and helps users understand the full diagnostic capability.

This is a documentation improvement, not a behavioral change. The implementation is correct and better than what the spec describes.

Scope

Section affected: #### agents diagnostics (~line 853)
Change type: Update example output to show all 9 providers
Risk: Low — documentation only, no behavioral change

Automated by CleverAgents Bot
Supervisor: Specification Updates | Agent: spec-updater

## Specification Update Proposal **Triggered by**: Merged PR #3469 (fix(cli): extend agents diagnostics to check all 9 supported providers) **Classification**: Implementation is BETTER than spec — update spec to match ## What Changed in the Implementation PR #3469 extended `_check_providers()` in `src/cleveragents/cli/commands/system.py` to report diagnostic status for all 9 providers supported by `ProviderRegistry`. Previously only 4 providers were checked (OpenAI, Anthropic, Google, OpenRouter); now all 9 are covered including Azure, Gemini, Cohere, Groq, and Together AI. The `ProviderRegistry` supports these 9 provider types: - `openai` (OPENAI_API_KEY) - `anthropic` (ANTHROPIC_API_KEY) - `google` (GOOGLE_API_KEY) - `gemini` (GEMINI_API_KEY) - `azure` (AZURE_OPENAI_API_KEY) - `openrouter` (OPENROUTER_API_KEY) - `cohere` (COHERE_API_KEY) - `groq` (GROQ_API_KEY) - `together` (TOGETHER_API_KEY) ## What Spec Sections Need Updating ### Section: `agents diagnostics` (line ~853) **Current spec** (lines 876-878 in the Rich example): ``` │ OPENAI_API_KEY WARN missing │ │ Anthropic key OK configured │ ``` Only shows 2 providers. **Proposed update**: Expand the example to show all 9 providers, demonstrating the comprehensive provider coverage. The example should show a realistic mix of configured and unconfigured providers. ### Section: `agents diagnostics` JSON example (line ~946) **Current spec**: Only includes `OPENAI_API_KEY` and `Anthropic key` checks. **Proposed update**: Include all 9 provider checks in the JSON output example. ## Rationale The spec example is misleading — it implies only 2 providers are checked, when the implementation actually checks all 9. Users reading the spec would not know that `agents diagnostics` provides comprehensive provider coverage. Updating the example to show all 9 providers accurately reflects the implementation and helps users understand the full diagnostic capability. This is a documentation improvement, not a behavioral change. The implementation is correct and better than what the spec describes. ## Scope - **Section affected**: `#### agents diagnostics` (~line 853) - **Change type**: Update example output to show all 9 providers - **Risk**: Low — documentation only, no behavioral change --- **Automated by CleverAgents Bot** Supervisor: Specification Updates | Agent: spec-updater

Rows
Columns