apunkt/plano
1
0
Fork 0
mirror of https://github.com/katanemo/plano.git synced 2026-05-10 16:22:42 +02:00
Code Issues Projects Releases Packages Wiki Activity Actions

fix(config): accept vercel and openrouter as provider_interface values (#915)

Browse source 
The Python CLI (#902) and the JSON schema both allow `vercel` and
`openrouter` as `provider_interface`, and `hermesllm::ProviderId` knows
how to dispatch them — but `crates/common::LlmProviderType` was never
extended to deserialize them. As a result, `planoai up` with no user
config (which synthesizes both providers via `cli/planoai/defaults.py`)
caused brightstaff to crash on startup with:

    unknown variant `vercel`, expected one of `anthropic`, ..., `digitalocean`

Add the missing enum variants and Display arms, plus a regression test
that asserts both round-trip through serde and resolve through
`to_provider_id()` (the exact path that previously panicked at parse).
This commit is contained in:
Musa 2026-04-24 16:32:00 -07:00 • committed by GitHub
parent 92f82a9c68
commit 2954ae258f
No known key found for this signature in database
GPG key ID: B5690EEEBB952194
1 changed files with 26 additions and 0 deletions
Download patch file Download diff file Expand all files Collapse all files

26
crates/common/src/configuration.rs
View file

@ -396,6 +396,10 @@ pub enum LlmProviderType {
ChatGPT, ChatGPT,
#[serde(rename = "digitalocean")] #[serde(rename = "digitalocean")]
DigitalOcean, DigitalOcean,
#[serde(rename = "vercel")]
Vercel,
#[serde(rename = "openrouter")]
OpenRouter,
} }
impl Display for LlmProviderType { impl Display for LlmProviderType {
@ -419,6 +423,8 @@ impl Display for LlmProviderType {
LlmProviderType::Plano => write!(f, "plano"), LlmProviderType::Plano => write!(f, "plano"),
LlmProviderType::ChatGPT => write!(f, "chatgpt"), LlmProviderType::ChatGPT => write!(f, "chatgpt"),
LlmProviderType::DigitalOcean => write!(f, "digitalocean"), LlmProviderType::DigitalOcean => write!(f, "digitalocean"),
LlmProviderType::Vercel => write!(f, "vercel"),
LlmProviderType::OpenRouter => write!(f, "openrouter"),
} }
} }
} }
@ -757,6 +763,26 @@ mod test {
assert!(!model_ids.contains(&"plano-orchestrator".to_string())); assert!(!model_ids.contains(&"plano-orchestrator".to_string()));
} }
#[test]
fn test_llm_provider_type_vercel_and_openrouter_roundtrip() {
// Regression: brightstaff used to reject `provider_interface: vercel`
// (and `openrouter`) because these variants were missing from
// `LlmProviderType`, causing `planoai up` with the synthesized default
// config to crash on startup.
for (yaml_value, expected) in [
("vercel", LlmProviderType::Vercel),
("openrouter", LlmProviderType::OpenRouter),
] {
let parsed: LlmProviderType =
serde_yaml::from_str(yaml_value).expect("variant should deserialize");
assert_eq!(parsed, expected);
assert_eq!(parsed.to_string(), yaml_value);
// to_provider_id() bridges into hermesllm; both providers must be
// recognized there as well or this panics.
let _ = parsed.to_provider_id();
}
}
#[test] #[test]
fn test_overrides_disable_signals_default_none() { fn test_overrides_disable_signals_default_none() {
let overrides = super::Overrides::default(); let overrides = super::Overrides::default();