Skip to content

feat: expose ProviderRegistry.list() - #18

Merged
royalpinto007 merged 1 commit into
AgentPostmortem:mainfrom
pachara-n:feat/provider-registry-list
Aug 6, 2026
Merged

feat: expose ProviderRegistry.list()#18
royalpinto007 merged 1 commit into
AgentPostmortem:mainfrom
pachara-n:feat/provider-registry-list

Conversation

@pachara-n

Copy link
Copy Markdown
Contributor

What & why

This PR adds ProviderRegistry.list() and updates cmdList() to derive the provider list from the registry instead of using a hardcoded string.

This keeps the CLI output in sync with registered providers and allows runtime-registered providers to be reflected correctly.

Changes

  • Added ProviderRegistry.list()
  • Updated cmdList() to use the registry
  • Added tests covering built-in and custom providers

Checklist

@github-actions

github-actions Bot commented Aug 6, 2026

Copy link
Copy Markdown

Welcome, @pachara-n, and thanks for your first pull request to Evalgate.

A quick look at CONTRIBUTING.md covers how things are set up and run here.

I'll review this shortly. Ask anything in the thread if something is unclear.

@royalpinto007
royalpinto007 merged commit 541efbe into AgentPostmortem:main Aug 6, 2026
6 checks passed
@royalpinto007

Copy link
Copy Markdown
Collaborator

Merged, thanks @pachara-n. Deriving the list from the registry means the CLI can no longer drift from what is actually registered, which was the real bug behind the hardcoded string. Verified locally: 68 tests, typecheck and lint all clean.

If the project is useful to you, a star helps more people find it.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

evalgate list hardcodes the provider names instead of asking the registry

2 participants