fix(ci): suppress option_as_ref_deref on credential refs

This commit is contained in:
Chummy 2026-02-17 16:32:26 +08:00
parent a6ca68a4fb
commit e5a8cd3f57
2 changed files with 4 additions and 10 deletions

View file

@ -198,11 +198,8 @@ pub fn create_provider_with_url(
api_url: Option<&str>, api_url: Option<&str>,
) -> anyhow::Result<Box<dyn Provider>> { ) -> anyhow::Result<Box<dyn Provider>> {
let resolved_credential = resolve_provider_credential(name, api_key); let resolved_credential = resolve_provider_credential(name, api_key);
let key = if let Some(value) = resolved_credential.as_ref() { #[allow(clippy::option_as_ref_deref)]
Some(value.as_str()) let key = resolved_credential.as_ref().map(String::as_str);
} else {
None
};
match name { match name {
// ── Primary providers (custom implementations) ─────── // ── Primary providers (custom implementations) ───────
"openrouter" => Ok(Box::new(openrouter::OpenRouterProvider::new(key))), "openrouter" => Ok(Box::new(openrouter::OpenRouterProvider::new(key))),

View file

@ -169,11 +169,8 @@ impl Tool for DelegateTool {
.api_key .api_key
.clone() .clone()
.or_else(|| self.fallback_credential.clone()); .or_else(|| self.fallback_credential.clone());
let provider_credential = if let Some(value) = provider_credential_owned.as_ref() { #[allow(clippy::option_as_ref_deref)]
Some(value.as_str()) let provider_credential = provider_credential_owned.as_ref().map(String::as_str);
} else {
None
};
let provider: Box<dyn Provider> = let provider: Box<dyn Provider> =
match providers::create_provider(&agent_config.provider, provider_credential) { match providers::create_provider(&agent_config.provider, provider_credential) {