rs-utcp 0.3.2

Rust implementation of the Universal Tool Calling Protocol (UTCP).
Documentation
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
90
91
92
93
94
95
96
97
98
99
100
101
102
103
104
105
106
107
108
109
110
111
112
113
114
115
116
117
118
119
120
121
122
123
124
125
126
127
128
129
130
131
132
133
134
135
136
137
138
139
140
141
142
143
144
145
146
147
148
149
150
151
152
153
154
155
156
157
158
159
160
161
162
163
164
165
166
167
168
169
170
171
172
173
174
175
176
177
178
179
180
181
182
183
184
185
186
187
188
189
190
191
192
193
194
195
196
197
198
199
200
201
202
203
204
205
206
207
208
209
210
211
212
213
214
215
216
217
218
219
220
221
222
223
224
225
226
227
228
229
230
231
232
233
234
235
236
237
238
239
240
241
242
243
244
245
246
247
248
249
250
251
252
253
254
255
256
257
258
259
260
261
262
263
264
265
266
267
268
269
270
271
272
273
274
275
276
277
278
279
280
281
282
283
284
285
286
287
288
289
290
291
292
293
294
295
296
297
298
299
300
301
302
303
304
305
306
307
308
309
310
311
312
313
314
315
316
317
318
319
320
321
322
323
324
325
326
327
328
329
330
331
332
333
334
335
336
337
338
339
340
341
342
343
344
345
346
347
348
349
350
351
352
353
354
355
356
357
358
359
360
361
362
363
364
365
366
367
368
369
370
371
372
373
374
375
376
377
378
379
380
381
382
383
384
385
386
387
388
389
390
391
392
393
394
395
396
397
398
399
400
401
402
403
404
405
406
407
408
409
410
411
412
413
414
415
416
417
418
419
420
421
422
423
424
425
426
427
428
429
430
431
432
433
434
435
436
437
438
439
440
441
442
443
444
445
446
447
448
449
450
451
452
453
454
455
456
457
458
459
460
461
462
463
464
465
466
467
468
469
470
471
472
473
474
475
476
477
478
479
480
481
482
483
484
485
486
487
488
489
490
491
492
493
494
495
496
497
498
499
500
501
502
503
504
505
506
507
508
509
510
511
512
513
514
515
516
517
518
519
520
521
522
523
524
525
526
527
528
529
530
531
532
533
534
535
536
537
538
539
540
541
542
543
544
545
546
547
548
549
550
551
552
553
554
555
556
557
558
559
560
561
562
563
564
565
566
567
568
569
570
571
572
573
574
575
576
577
578
579
580
581
582
583
584
585
586
587
588
589
590
591
592
593
594
595
596
597
598
599
600
601
602
603
604
605
606
607
608
609
610
611
612
613
614
615
616
617
618
619
620
621
622
623
624
625
626
627
628
629
630
631
632
633
634
635
636
637
638
639
640
641
642
643
644
645
646
647
648
649
650
651
652
653
654
655
656
657
658
659
660
661
662
663
664
665
666
667
668
669
670
671
672
673
674
675
676
677
678
679
680
681
682
683
684
685
686
687
688
689
690
691
692
693
694
// Provider loading from JSON files
use anyhow::{anyhow, Result};
use serde_json::Value;
use std::path::Path;
use std::sync::Arc;

use crate::call_templates;
use crate::config::UtcpClientConfig;
use crate::migration::{migrate_v01_config, validate_v1_config, validate_v1_manual};
use crate::providers::base::Provider;
use crate::providers::cli::CliProvider;
use crate::providers::graphql::GraphqlProvider;
use crate::providers::grpc::GrpcProvider;
use crate::providers::http::HttpProvider;
use crate::providers::http_stream::StreamableHttpProvider;
use crate::providers::mcp::McpProvider;
use crate::providers::sse::SseProvider;
use crate::providers::tcp::TcpProvider;
use crate::providers::text::TextProvider;
use crate::providers::udp::UdpProvider;
use crate::providers::webrtc::WebRtcProvider;
use crate::providers::websocket::WebSocketProvider;
use crate::spec::ManualV1;

/// Parse a providers JSON file
/// Supports multiple formats:
/// - Array: [{"provider_type": "http", ...}, ...]
/// - Object with providers array: {"providers": [{...}, ...]}
/// - Object with single provider: {"providers": {...}}
/// - Single provider: {"provider_type": "http", ...}
pub async fn load_providers_from_file(
    path: impl AsRef<Path>,
    config: &UtcpClientConfig,
) -> Result<Vec<Arc<dyn Provider>>> {
    Ok(load_providers_with_tools_from_file(path, config)
        .await?
        .into_iter()
        .map(|p| p.provider)
        .collect())
}

/// LoadedProvider represents a provider that has been loaded from a configuration file,
/// optionally including a list of tools if they were defined in the file (e.g. in a manual).
pub struct LoadedProvider {
    /// The loaded provider instance.
    pub provider: Arc<dyn Provider>,
    /// Optional list of tools associated with this provider.
    pub tools: Option<Vec<crate::tools::Tool>>,
}

/// Load providers or manuals (v0.1 or v1.0), returning providers and any embedded tools.
pub async fn load_providers_with_tools_from_file(
    path: impl AsRef<Path>,
    config: &UtcpClientConfig,
) -> Result<Vec<LoadedProvider>> {
    let contents = tokio::fs::read_to_string(path).await?;
    let json_raw: Value = serde_json::from_str(&contents)?;
    // Apply v0.1 -> v1.0 migration for configs if needed
    let json = migrate_v01_config(&json_raw);

    // Validate v1.0 shapes when applicable
    if let Some(obj) = json.as_object() {
        if obj.contains_key("manual_call_templates") {
            validate_v1_config(&json)?;
        }
        if obj.contains_key("tools") {
            validate_v1_manual(&json)?;
        }
    }

    // If this is a manual with tools, collect tools per provider
    if let Some(obj) = json.as_object() {
        if obj.get("tools").is_some() {
            let _manual: ManualV1 = serde_json::from_value(json.clone())
                .map_err(|e| anyhow!("Invalid v1.0 manual: {}", e))?;

            let (providers, tools) = parse_manual_tools_with_providers(json.clone(), config)?;
            return Ok(providers
                .into_iter()
                .zip(tools.into_iter())
                .map(|(provider, tools)| LoadedProvider {
                    provider,
                    tools: Some(tools),
                })
                .collect());
        }
    }

    let provider_values = parse_providers_json(json)?;

    let mut providers = Vec::new();
    for (index, mut provider_value) in provider_values.into_iter().enumerate() {
        // Perform variable substitution
        substitute_variables(&mut provider_value, config);

        // Create provider
        let provider = create_provider_from_value(provider_value, index)?;
        providers.push(LoadedProvider {
            provider,
            tools: None,
        });
    }

    Ok(providers)
}

/// Parses the raw JSON value into a list of provider JSON objects.
/// Handles various formats: array, object with "providers", object with "manual_call_templates", or single provider object.
fn parse_providers_json(json: Value) -> Result<Vec<Value>> {
    match json {
        // Direct array of providers
        Value::Array(arr) => Ok(arr),

        // Object that might contain providers
        Value::Object(obj) => {
            if obj.get("tools").is_some() {
                return parse_manual_tools(Value::Object(obj));
            }

            // v1.0 migration: manual_call_templates -> providers
            if let Some(templates_value) = obj.get("manual_call_templates") {
                if let Some(arr) = templates_value.as_array() {
                    let mut providers = Vec::new();
                    for template in arr {
                        providers
                            .push(call_templates::call_template_to_provider(template.clone())?);
                    }
                    return Ok(providers);
                }
            }

            if let Some(providers_value) = obj.get("providers") {
                match providers_value {
                    // providers is an array
                    Value::Array(arr) => Ok(arr.clone()),
                    // providers is a single object
                    Value::Object(_) => Ok(vec![providers_value.clone()]),
                    _ => Err(anyhow!("'providers' field must be an array or object")),
                }
            } else {
                // Single provider object (no "providers" wrapper)
                Ok(vec![Value::Object(obj)])
            }
        }

        _ => Err(anyhow!("JSON root must be array or object")),
    }
}

/// Parse a manual (v0.1 or v1.0) into a list of providers by lifting tool_call_templates.
fn parse_manual_tools(json: Value) -> Result<Vec<Value>> {
    let obj = json
        .as_object()
        .ok_or_else(|| anyhow!("Manual must be an object"))?;
    let tools = obj
        .get("tools")
        .and_then(|v| v.as_array())
        .ok_or_else(|| anyhow!("Manual missing tools array"))?;

    let mut providers = Vec::new();
    for tool in tools {
        if let Some(provider) = tool_to_provider(tool)? {
            providers.push(provider);
        }
    }
    Ok(providers)
}

/// Parses a manual JSON object to extract both providers and their associated tools.
/// Returns a tuple of (providers, tools_per_provider).
fn parse_manual_tools_with_providers(
    json: Value,
    config: &UtcpClientConfig,
) -> Result<(Vec<Arc<dyn Provider>>, Vec<Vec<crate::tools::Tool>>)> {
    let obj = json
        .as_object()
        .ok_or_else(|| anyhow!("Manual must be an object"))?;
    let tools = obj
        .get("tools")
        .and_then(|v| v.as_array())
        .ok_or_else(|| anyhow!("Manual missing tools array"))?;

    // Extract allowed_communication_protocols from manual
    let manual_allowed_protocols: Option<Vec<String>> = obj
        .get("allowed_communication_protocols")
        .and_then(|v| v.as_array())
        .map(|arr| {
            arr.iter()
                .filter_map(|v| v.as_str().map(|s| s.to_string()))
                .collect()
        });

    let mut providers = Vec::new();
    let mut tools_per_provider = Vec::new();

    for (idx, tool_val) in tools.iter().enumerate() {
        if let Some(provider_val) = tool_to_provider(tool_val)? {
            let mut provider_val = provider_val.clone();
            substitute_variables(&mut provider_val, config);

            // If missing provider_type, derive from call_template_type
            let provider_obj = provider_val
                .as_object_mut()
                .ok_or_else(|| anyhow!("Provider must be object"))?;

            let provider_type_str = if provider_obj
                .get("provider_type")
                .or_else(|| provider_obj.get("type"))
                .is_none()
            {
                if let Some(ct) = provider_obj.get("call_template_type").cloned() {
                    provider_obj.insert("provider_type".to_string(), ct.clone());
                    provider_obj.insert("type".to_string(), ct.clone());
                    ct.as_str().unwrap_or("http").to_string()
                } else {
                  if provider_obj
                        .get("allowed_communication_protocols")
                        .is_none()
                    {
                        provider_obj.insert(
                            "allowed_communication_protocols".to_string(),
                            serde_json::to_value("http".to_string())?,
                        );
                    }
                    provider_obj.insert("type".to_string(), Value::String("http".to_string()));
                    "http".to_string()
                }
            } else {
                provider_obj
                    .get("provider_type")
                    .or_else(|| provider_obj.get("type"))
                    .and_then(|v| v.as_str())
                    .unwrap_or("http")
                    .to_string()
            };

            // When the tool_call_template has no explicit "name" field, normalize_common_template
            // defaults the provider name to the call_template_type string (e.g. "cli"). This means
            // multiple tools of the same type would all get the same provider name and overwrite each
            // other in the repository (issue #28). Fix: if the provider's current name equals the
            // bare call_template_type, replace it with the tool's outer name so that every tool gets
            // its own unique provider slot.
            let tool_outer_name = tool_val.get("name").and_then(|v| v.as_str());
            if let Some(outer_name) = tool_outer_name {
                let current_prov_name = provider_obj
                    .get("name")
                    .and_then(|v| v.as_str())
                    .unwrap_or("");
                // Only override when the name was auto-defaulted to the type string (not explicitly set).
                if current_prov_name == provider_type_str || current_prov_name.is_empty() {
                    provider_obj.insert("name".to_string(), Value::String(outer_name.to_string()));
                }
            }

            // Propagate manual's allowed_communication_protocols to provider if set
            if let Some(ref allowed) = manual_allowed_protocols {
                if !allowed.is_empty() {
                    // Check if this tool's protocol is allowed
                    if !allowed.contains(&provider_type_str) {
                        // Log warning and skip this tool
                        if let Some(tool_name) = tool_val.get("name").and_then(|v| v.as_str()) {
                            eprintln!(
                                "Warning: Tool '{}' uses communication protocol '{}' which is not in allowed protocols {:?} for manual. Tool will not be registered.",
                                tool_name, provider_type_str, allowed
                            );
                        }
                        continue; // Skip this tool
                    }

                    // Add allowed_communication_protocols to provider
                    provider_obj.insert(
                        "allowed_communication_protocols".to_string(),
                        serde_json::to_value(allowed.clone())?,
                    );
                }
            }

            let provider = create_provider_from_value(provider_val, idx)?;
            let prov_name = provider.name();
            providers.push(provider);

            let mut tool_value = tool_val.clone();
            if let Some(obj) = tool_value.as_object_mut() {
                obj.entry("tags")
                    .or_insert_with(|| Value::Array(Vec::new()));
            }
            let mut tool: crate::tools::Tool = serde_json::from_value(tool_value)?;
            // Prefix tool name with provider to keep existing naming
            if !tool.name.starts_with(&format!("{}.", prov_name)) {
                tool.name = format!("{}.{}", prov_name, tool.name);
            }
            tools_per_provider.push(vec![tool]);
        }
    }

    Ok((providers, tools_per_provider))
}

/// Extracts a provider definition from a tool JSON object.
/// Checks for "tool_call_template" or "provider" fields.
fn tool_to_provider(tool: &Value) -> Result<Option<Value>> {
    let tool_obj = tool
        .as_object()
        .ok_or_else(|| anyhow!("Tool must be an object"))?;

    if let Some(tmpl) = tool_obj.get("tool_call_template") {
        Ok(Some(call_templates::call_template_to_provider(
            tmpl.clone(),
        )?))
    } else if let Some(prov) = tool_obj.get("provider") {
        Ok(Some(call_templates::call_template_to_provider(
            prov.clone(),
        )?))
    } else {
        Ok(None)
    }
}

/// Creates a Provider instance from a JSON value.
/// Handles type normalization and defaults.
fn create_provider_from_value(mut value: Value, index: usize) -> Result<Arc<dyn Provider>> {
    // Normalize type field: accept both "type" and "provider_type"
    let provider_type = {
        let obj = value
            .as_object_mut()
            .ok_or_else(|| anyhow!("Provider must be an object"))?;

        if obj.get("provider_type").is_none() && obj.get("type").is_none() {
            obj.insert(
                "provider_type".to_string(),
                Value::String("http".to_string()),
            );
            obj.insert("type".to_string(), Value::String("http".to_string()));
        }

        let ptype = obj
            .get("provider_type")
            .or_else(|| obj.get("type"))
            .and_then(|v| v.as_str())
            .ok_or_else(|| anyhow!("Missing provider_type field"))?
            .to_string();

        // Ensure both fields exist
        obj.insert("type".to_string(), Value::String(ptype.clone()));
        obj.insert("provider_type".to_string(), Value::String(ptype.clone()));

        // Ensure name field exists
        if !obj.contains_key("name") {
            obj.insert(
                "name".to_string(),
                Value::String(format!("{}_{}", ptype, index)),
            );
        }

        ptype
    };

    // Create provider based on type
    match provider_type.as_str() {
        "http" => {
            if !value
                .get("http_method")
                .or_else(|| value.get("method"))
                .is_some()
            {
                if let Some(obj) = value.as_object_mut() {
                    obj.insert("http_method".to_string(), Value::String("GET".to_string()));
                }
            }
            if !value.get("url").is_some() {
                if let Some(obj) = value.as_object_mut() {
                    obj.insert(
                        "url".to_string(),
                        Value::String("http://localhost".to_string()),
                    );
                }
            }
            let provider: HttpProvider = serde_json::from_value(value)?;
            Ok(Arc::new(provider))
        }
        "cli" => {
            let provider: CliProvider = serde_json::from_value(value)?;
            Ok(Arc::new(provider))
        }
        "sse" => {
            let provider: SseProvider = serde_json::from_value(value)?;
            Ok(Arc::new(provider))
        }
        "websocket" => {
            let provider: WebSocketProvider = serde_json::from_value(value)?;
            Ok(Arc::new(provider))
        }
        "grpc" => {
            let provider: GrpcProvider = serde_json::from_value(value)?;
            Ok(Arc::new(provider))
        }
        "graphql" => {
            let provider: GraphqlProvider = serde_json::from_value(value)?;
            Ok(Arc::new(provider))
        }
        "tcp" => {
            let provider: TcpProvider = serde_json::from_value(value)?;
            Ok(Arc::new(provider))
        }
        "udp" => {
            let provider: UdpProvider = serde_json::from_value(value)?;
            Ok(Arc::new(provider))
        }
        "http_stream" => {
            let provider: StreamableHttpProvider = serde_json::from_value(value)?;
            Ok(Arc::new(provider))
        }
        "mcp" => {
            let provider: McpProvider = serde_json::from_value(value)?;
            Ok(Arc::new(provider))
        }
        "webrtc" => {
            let provider: WebRtcProvider = serde_json::from_value(value)?;
            Ok(Arc::new(provider))
        }
        "text" => {
            let provider: TextProvider = serde_json::from_value(value)?;
            Ok(Arc::new(provider))
        }
        // Add more provider types as they are implemented
        _ => Err(anyhow!("Unsupported provider type: {}", provider_type)),
    }
}

/// Substitutes variables in the JSON value using the provided configuration.
/// Replaces ${VAR} and $VAR with values from config or environment.
fn substitute_variables(value: &mut Value, config: &UtcpClientConfig) {
    match value {
        Value::String(s) => {
            // Replace ${VAR} or $VAR patterns
            let mut result = s.clone();

            // Replace from config variables
            for (key, val) in &config.variables {
                result = result.replace(&format!("${{{}}}", key), val);
                result = result.replace(&format!("${}", key), val);
            }

            // Also check environment variables for remaining variables
            if result.contains('$') {
                for key in config.variables.keys() {
                    if let Ok(env_val) = std::env::var(key) {
                        result = result.replace(&format!("${}", key), &env_val);
                        result = result.replace(&format!("${{{}}}", key), &env_val);
                    }
                }
            }

            *s = result;
        }
        Value::Object(obj) => {
            for (_, v) in obj.iter_mut() {
                substitute_variables(v, config);
            }
        }
        Value::Array(arr) => {
            for item in arr.iter_mut() {
                substitute_variables(item, config);
            }
        }
        _ => {}
    }
}

#[cfg(test)]
mod tests {
    use super::*;
    use std::io::Write;
    use tempfile::NamedTempFile;

    #[test]
    fn test_parse_providers_array() {
        let json = serde_json::json!([
            {"provider_type": "http", "url": "http://example.com"},
            {"provider_type": "cli", "command_name": "ls"}
        ]);

        let result = parse_providers_json(json).unwrap();
        assert_eq!(result.len(), 2);
    }

    #[test]
    fn test_parse_providers_object_with_array() {
        let json = serde_json::json!({
            "providers": [
                {"provider_type": "http", "url": "http://example.com"}
            ]
        });

        let result = parse_providers_json(json).unwrap();
        assert_eq!(result.len(), 1);
    }

    #[test]
    fn test_parse_single_provider() {
        let json = serde_json::json!({
            "provider_type": "http",
            "url": "http://example.com"
        });

        let result = parse_providers_json(json).unwrap();
        assert_eq!(result.len(), 1);
    }

    #[test]
    fn test_parse_manual_call_templates_converts_to_providers() {
        let json = serde_json::json!({
            "manual_call_templates": [
                {
                    "name": "weather_service",
                    "call_template_type": "http",
                    "url": "http://example.com",
                    "http_method": "GET"
                },
                {
                    "name": "cli_tool",
                    "call_template_type": "cli",
                    "command": "echo hi",
                    "working_dir": "/tmp"
                }
            ]
        });

        let result = parse_providers_json(json).unwrap();
        assert_eq!(result.len(), 2);
        assert_eq!(
            result[0]
                .get("provider_type")
                .and_then(|v| v.as_str())
                .unwrap(),
            "http"
        );
        assert_eq!(
            result[1]
                .get("command_name")
                .and_then(|v| v.as_str())
                .unwrap(),
            "echo hi"
        );
    }

    #[tokio::test]
    async fn load_providers_supports_multiple_types() {
        let mut file = NamedTempFile::new().unwrap();
        write!(
            file,
            r#"{{
                "providers": [
                    {{ "provider_type": "sse", "name": "events", "url": "http://example.com/sse" }},
                    {{ "provider_type": "http_stream", "name": "stream", "url": "http://example.com/stream" }}
                ]
            }}"#
        )
        .unwrap();

        let config = UtcpClientConfig::default();
        let providers = load_providers_from_file(file.path(), &config)
            .await
            .unwrap();
        assert_eq!(providers.len(), 2);
        assert_eq!(
            providers[0].type_(),
            crate::providers::base::ProviderType::Sse
        );
        assert_eq!(
            providers[1].type_(),
            crate::providers::base::ProviderType::HttpStream
        );
    }

    #[tokio::test]
    async fn load_manual_with_tools_returns_tools() {
        let mut file = NamedTempFile::new().unwrap();
        write!(
            file,
            r#"{{
                "manual_version": "1.0.0",
                "utcp_version": "0.2.0",
                "info": {{ "title": "demo", "version": "1.0.0" }},
                "tools": [
                    {{
                        "name": "echo",
                        "description": "Echo",
                        "inputs": {{ "type": "object" }},
                        "outputs": {{ "type": "object" }},
                        "tool_call_template": {{
                            "call_template_type": "http",
                            "name": "http_tool",
                            "url": "http://example.com",
                            "http_method": "GET"
                        }}
                    }}
                ]
            }}"#
        )
        .unwrap();

        let config = UtcpClientConfig::default();
        let loaded = load_providers_with_tools_from_file(file.path(), &config)
            .await
            .unwrap();
        assert_eq!(loaded.len(), 1);
        assert!(loaded[0].tools.as_ref().unwrap()[0]
            .name
            .starts_with(&loaded[0].provider.name()));
    }

    /// Regression test for issue #28:
    /// Multiple tools with the same call_template_type and no explicit name in their
    /// tool_call_template should each get a unique provider name derived from the tool's
    /// outer name, preventing them from overwriting each other in the repository.
    #[tokio::test]
    async fn load_manual_multiple_tools_same_type_no_template_name_each_get_unique_provider() {
        let mut file = NamedTempFile::new().unwrap();
        write!(
            file,
            r#"{{
                "utcp_version": "1.0.1",
                "manual_version": "1.0.0",
                "info": {{ "title": "Bug Reproduction", "version": "1.0.0" }},
                "tools": [
                    {{
                        "name": "tool1",
                        "description": "First CLI tool",
                        "inputs": {{ "type": "object", "properties": {{}} }},
                        "outputs": {{ "type": "string" }},
                        "tool_call_template": {{
                            "call_template_type": "cli",
                            "commands": [{{"command": "echo 'tool1'", "append_to_final_output": true}}]
                        }}
                    }},
                    {{
                        "name": "tool2",
                        "description": "Second CLI tool",
                        "inputs": {{ "type": "object", "properties": {{}} }},
                        "outputs": {{ "type": "string" }},
                        "tool_call_template": {{
                            "call_template_type": "cli",
                            "commands": [{{"command": "echo 'tool2'", "append_to_final_output": true}}]
                        }}
                    }},
                    {{
                        "name": "tool3",
                        "description": "Third CLI tool",
                        "inputs": {{ "type": "object", "properties": {{}} }},
                        "outputs": {{ "type": "string" }},
                        "tool_call_template": {{
                            "call_template_type": "cli",
                            "commands": [{{"command": "echo 'tool3'", "append_to_final_output": true}}]
                        }}
                    }}
                ]
            }}"#
        )
        .unwrap();

        let config = UtcpClientConfig::default();
        let loaded = load_providers_with_tools_from_file(file.path(), &config)
            .await
            .unwrap();

        // All 3 tools must be present as separate providers
        assert_eq!(loaded.len(), 3, "Expected 3 providers, one per tool");

        // Each provider must have a distinct name
        let names: Vec<String> = loaded.iter().map(|p| p.provider.name()).collect();
        let mut unique_names = names.clone();
        unique_names.dedup();
        assert_eq!(
            unique_names.len(),
            names.len(),
            "Provider names are not unique: {:?}",
            names
        );

        // Every loaded entry must carry exactly 1 tool
        for lp in &loaded {
            let tools = lp.tools.as_ref().unwrap();
            assert_eq!(tools.len(), 1, "Expected exactly 1 tool per provider");
            // Tool name must be prefixed with the provider name
            assert!(
                tools[0].name.starts_with(&lp.provider.name()),
                "Tool name '{}' should start with provider name '{}'",
                tools[0].name,
                lp.provider.name()
            );
        }
    }
}