fix(rpc): update RPC parameter passing to use object configs

Updated both AI prompts and RPC bridge to properly handle Solana RPC params:

AI Prompt changes (ai_service.rs):
- Changed tool call syntax from :param style to object config style
- Example: (getSignaturesForAddress addr {:limit 1000})
- Updated pagination example to use config objects

RPC Bridge changes (rpc_bridge.rs):
- Improved config object building for getSignaturesForAddress
- Now properly wraps limit/before in config struct
- Handles both object and positional parameter styles

This fixes the RPC error:
  "invalid type: integer 1000, expected struct RpcSignaturesForAddressConfig"

Now generates correct Solana RPC format:
  ["address", {"limit": 1000}] ✅

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude <[email protected]>

Author: 0xrinegade

src/services/ai_service.rs

@@ -633,7 +633,7 @@ Data Structures:
 
 Tool Calls:
   (toolName arg1 arg2)
-  (toolName :param1 value1 :param2 value2)
+  (toolName "primary_arg" {:param1 value1 :param2 value2})
 
 # Plan Structure
 
@@ -812,8 +812,8 @@ The Solana RPC has a **HARD LIMIT of 1000 results per call** for methods like `g
 (while continue
   (define batch
     (if (== before null)
-        (getSignaturesForAddress :address address :limit MAX_RESULTS_PER_CALL)
-        (getSignaturesForAddress :address address :limit MAX_RESULTS_PER_CALL :before before)))
+        (getSignaturesForAddress address {:limit MAX_RESULTS_PER_CALL})
+        (getSignaturesForAddress address {:limit MAX_RESULTS_PER_CALL :before before})))
 
   (define batch_size (COUNT batch))
 

src/utils/rpc_bridge.rs

@@ -79,35 +79,38 @@ impl Tool for RpcBridgeTool {
             rpc_params.push(ovsm_value_to_json(&args[0]));
 
             // Build config object from remaining arguments
-            // For now, we use generic field names since OVSM doesn't preserve parameter names
-            // The specific RPC method determines how to interpret these
-            let mut config = serde_json::Map::new();
+            // Check if second argument is already an object (named params)
+            if args.len() == 2 && matches!(&args[1], OvsmValue::Object(_)) {
+                // Second arg is a config object, use it directly
+                rpc_params.push(ovsm_value_to_json(&args[1]));
+            } else {
+                // Build config from positional args
+                let mut config = serde_json::Map::new();
 
-            // Common Solana RPC config fields
-            if args.len() == 2 {
-                // Second arg is typically 'limit' for getSignaturesForAddress
+                // For getSignaturesForAddress specifically
                 if matches!(self.name.as_str(), "getSignaturesForAddress") {
-                    config.insert("limit".to_string(), ovsm_value_to_json(&args[1]));
+                    if args.len() >= 2 {
+                        config.insert("limit".to_string(), ovsm_value_to_json(&args[1]));
+                    }
+                    if args.len() >= 3 {
+                        config.insert("before".to_string(), ovsm_value_to_json(&args[2]));
+                    }
                 } else {
-                    // For other methods, treat as config object
-                    if let OvsmValue::Object(obj) = &args[1] {
-                        for (k, v) in obj.iter() {
-                            config.insert(k.clone(), ovsm_value_to_json(v));
+                    // For other methods, try to infer config structure
+                    for (i, arg) in args.iter().enumerate().skip(1) {
+                        if let OvsmValue::Object(obj) = arg {
+                            for (k, v) in obj.iter() {
+                                config.insert(k.clone(), ovsm_value_to_json(v));
+                            }
+                        } else {
+                            config.insert(format!("param{}", i), ovsm_value_to_json(arg));
                         }
-                    } else {
-                        config.insert("config".to_string(), ovsm_value_to_json(&args[1]));
                     }
                 }
-            } else if args.len() == 3 {
-                // Third arg could be 'before' for pagination
-                if matches!(self.name.as_str(), "getSignaturesForAddress") {
-                    config.insert("limit".to_string(), ovsm_value_to_json(&args[1]));
-                    config.insert("before".to_string(), ovsm_value_to_json(&args[2]));
-                }
-            }
 
-            if !config.is_empty() {
-                rpc_params.push(Value::Object(config));
+                if !config.is_empty() {
+                    rpc_params.push(Value::Object(config));
+                }
             }
         }