@elizaos/plugin-wallet 2.0.0-beta.1 → 2.0.3-beta.5

package/src/analytics/lpinfo/kamino/services/kaminoService.ts

@@ -1,4 +1,3 @@
-// @ts-nocheck — legacy code from absorbed plugins (lp-manager, lpinfo, dexscreener, defi-news, birdeye); strict types pending cleanup
 import type { IAgentRuntime } from "@elizaos/core";
 import { logger, Service } from "@elizaos/core";
 
@@ -28,9 +27,17 @@ interface KaminoUserMarketRef {
   discovered: boolean;
 }
 
+export interface KaminoUserPosition {
+  token?: string;
+  amount?: number;
+  value?: number;
+  apy?: number;
+  market?: string;
+}
+
 interface KaminoUserPositionsOk {
-  lending: unknown[];
-  borrowing: unknown[];
+  lending: KaminoUserPosition[];
+  borrowing: KaminoUserPosition[];
   totalValue: number;
   markets: KaminoUserMarketRef[];
   userAccounts: number;
@@ -38,15 +45,17 @@ interface KaminoUserPositionsOk {
 }
 
 interface KaminoUserPositionsErr {
-  lending: unknown[];
-  borrowing: unknown[];
+  lending: KaminoUserPosition[];
+  borrowing: KaminoUserPosition[];
   totalValue: number;
   error: string;
 }
 
-type KaminoUserPositions = KaminoUserPositionsOk | KaminoUserPositionsErr;
+export type KaminoUserPositions =
+  | KaminoUserPositionsOk
+  | KaminoUserPositionsErr;
 
-interface KaminoMarketOverviewRow {
+export interface KaminoMarketOverviewRow {
   address: string;
   marketName: string;
   dataSize: number;
@@ -80,7 +89,7 @@ interface KaminoMarketOverviewErr {
 
 type KaminoMarketOverview = KaminoMarketOverviewOk | KaminoMarketOverviewErr;
 
-interface KaminoReserve {
+export interface KaminoReserve {
   market: string;
   marketName: string;
   dataSize: number;
@@ -113,6 +122,19 @@ interface KaminoConnectionTestResult {
   timestamp: string;
 }
 
+function getStringSetting(
+  runtime: IAgentRuntime | undefined,
+  key: string,
+  fallback: string,
+): string {
+  const value = runtime?.getSetting(key);
+  return typeof value === "string" && value.length > 0 ? value : fallback;
+}
+
+function formatLogError(error: unknown): string {
+  return error instanceof Error ? error.message : String(error);
+}
+
 /**
  * Kamino Lending Protocol Service
  * Handles interactions with Kamino lending protocol using the official API
@@ -126,11 +148,14 @@ export class KaminoService extends Service {
   capabilityDescription =
     "Provides standardized access to Kamino lending protocol via the official API." as const;
 
-  constructor(runtime: IAgentRuntime) {
+  constructor(runtime?: IAgentRuntime) {
     super(runtime);
 
-    this.apiBaseUrl =
-      runtime.getSetting("KAMINO_API_URL") || KAMINO_API_BASE_URL;
+    this.apiBaseUrl = getStringSetting(
+      runtime,
+      "KAMINO_API_URL",
+      KAMINO_API_BASE_URL,
+    );
 
     logger.log(`KaminoService initialized with API: ${this.apiBaseUrl}`);
     logger.log(`Program ID: ${KAMINO_LEND_PROGRAM_ID}`);
@@ -161,7 +186,10 @@ export class KaminoService extends Service {
 
       return await response.json();
     } catch (error) {
-      logger.error(`API request failed for ${endpoint}:`, error);
+      logger.error(
+        `API request failed for ${endpoint}:`,
+        formatLogError(error),
+      );
       throw error;
     }
   }
@@ -195,7 +223,7 @@ export class KaminoService extends Service {
       );
       return userPositions;
     } catch (error) {
-      logger.error("Error fetching user positions:", error);
+      logger.error("Error fetching user positions:", formatLogError(error));
       return {
         lending: [],
         borrowing: [],
@@ -231,7 +259,7 @@ export class KaminoService extends Service {
         }
         logger.log(`Found ${stakingYields.length} staking yields`);
       } catch (error) {
-        logger.warn("Error fetching staking yields:", error);
+        logger.warn("Error fetching staking yields:", formatLogError(error));
       }
 
       // Get Limo trades to discover trading pairs
@@ -248,7 +276,7 @@ export class KaminoService extends Service {
         }
         logger.log(`Found ${limoTrades.length} Limo trades`);
       } catch (error) {
-        logger.warn("Error fetching Limo trades:", error);
+        logger.warn("Error fetching Limo trades:", formatLogError(error));
       }
 
       // Limit to first 20 markets to avoid overwhelming
@@ -257,7 +285,7 @@ export class KaminoService extends Service {
 
       return limitedMarkets;
     } catch (error) {
-      logger.error("Error discovering markets:", error);
+      logger.error("Error discovering markets:", formatLogError(error));
       return [];
     }
   }
@@ -314,7 +342,10 @@ export class KaminoService extends Service {
           ),
         );
       } catch (error) {
-        logger.warn("Error fetching staking yields for overview:", error);
+        logger.warn(
+          "Error fetching staking yields for overview:",
+          formatLogError(error),
+        );
       }
 
       // Get Limo trades for volume data
@@ -326,7 +357,10 @@ export class KaminoService extends Service {
         }, 0);
         overview.limoTrades = limoTrades.length;
       } catch (error) {
-        logger.warn("Error fetching Limo trades for overview:", error);
+        logger.warn(
+          "Error fetching Limo trades for overview:",
+          formatLogError(error),
+        );
       }
 
       // Create market entries
@@ -346,7 +380,7 @@ export class KaminoService extends Service {
       );
       return overview;
     } catch (error) {
-      logger.error("Error fetching market overview:", error);
+      logger.error("Error fetching market overview:", formatLogError(error));
       return {
         totalMarkets: 0,
         totalTvl: 0,
@@ -394,7 +428,10 @@ export class KaminoService extends Service {
           }
         }
       } catch (error) {
-        logger.warn("Error fetching staking yields for reserves:", error);
+        logger.warn(
+          "Error fetching staking yields for reserves:",
+          formatLogError(error),
+        );
       }
 
       // Get Limo trades to understand trading opportunities
@@ -424,13 +461,16 @@ export class KaminoService extends Service {
           }
         }
       } catch (error) {
-        logger.warn("Error fetching Limo trades for reserves:", error);
+        logger.warn(
+          "Error fetching Limo trades for reserves:",
+          formatLogError(error),
+        );
       }
 
       logger.log(`Found ${reserves.length} reserves`);
       return reserves;
     } catch (error) {
-      logger.error("Error fetching available reserves:", error);
+      logger.error("Error fetching available reserves:", formatLogError(error));
       return [];
     }
   }
@@ -447,7 +487,7 @@ export class KaminoService extends Service {
         multisig: KAMINO_MULTISIG,
         dataSize: 0, // Not available via API
         lamports: 0, // Not available via API
-        owner: "System Program", // Placeholder
+        owner: "System Program", // Default owner label when account metadata is absent.
         executable: false,
         rentEpoch: 0,
       };
@@ -455,7 +495,7 @@ export class KaminoService extends Service {
       logger.log("Program info retrieved successfully");
       return info;
     } catch (error) {
-      logger.error("Error fetching program info:", error);
+      logger.error("Error fetching program info:", formatLogError(error));
       throw error;
     }
   }
@@ -665,7 +705,7 @@ export class KaminoService extends Service {
           `API connection test passed. Found ${stakingYields.length} staking yields`,
         );
       } catch (error) {
-        logger.error("API connection test failed:", error);
+        logger.error("API connection test failed:", formatLogError(error));
       }
 
       // Test Limo trades endpoint
@@ -677,7 +717,7 @@ export class KaminoService extends Service {
           `Limo trades test passed. Found ${limoTrades.length} trades`,
         );
       } catch (error) {
-        logger.error("Limo trades test failed:", error);
+        logger.error("Limo trades test failed:", formatLogError(error));
       }
 
       // Test market discovery
@@ -686,13 +726,13 @@ export class KaminoService extends Service {
         results.marketCount = markets.length;
         logger.log(`Market discovery test: ${markets.length} markets found`);
       } catch (error) {
-        logger.error("Market discovery test failed:", error);
+        logger.error("Market discovery test failed:", formatLogError(error));
       }
 
       logger.log("Connection test completed");
       return results;
     } catch (error) {
-      logger.error("Error in connection test:", error);
+      logger.error("Error in connection test:", formatLogError(error));
       throw error;
     }
   }
@@ -727,12 +767,12 @@ export class KaminoService extends Service {
 
       // Test connection on startup
       const testResults = await this.testConnection();
-      logger.log("Startup connection test results:", testResults);
+      logger.log({ testResults }, "Startup connection test results");
 
       this.isRunning = true;
       logger.log("KaminoService started successfully");
     } catch (error) {
-      logger.error("Failed to start KaminoService:", error);
+      logger.error("Failed to start KaminoService:", formatLogError(error));
       throw error;
     }
   }
@@ -747,7 +787,7 @@ export class KaminoService extends Service {
       this.isRunning = false;
       logger.log("KaminoService stopped successfully");
     } catch (error) {
-      logger.error("Failed to stop KaminoService:", error);
+      logger.error("Failed to stop KaminoService:", formatLogError(error));
       throw error;
     }
   }

package/src/analytics/lpinfo/steer/index.ts

@@ -1,5 +1,4 @@
-// @ts-nocheck — legacy code from absorbed plugins (lp-manager, lpinfo, dexscreener, defi-news, birdeye); strict types pending cleanup
-import type { Plugin } from "@elizaos/core";
+import type { IAgentRuntime, Plugin } from "@elizaos/core";
 
 // Providers
 import { steerLiquidityProvider } from "./providers/steerLiquidityProvider";
@@ -18,6 +17,12 @@ export const steerPlugin: Plugin = {
   providers: [steerLiquidityProvider],
   actions: [],
   services: [SteerLiquidityService],
+  async dispose(runtime: IAgentRuntime) {
+    const svc = runtime.getService<SteerLiquidityService>(
+      SteerLiquidityService.serviceType,
+    );
+    await svc?.stop();
+  },
 };
 
 export default steerPlugin;

package/src/analytics/lpinfo/steer/providers/steerLiquidityProvider.ts

@@ -1,4 +1,3 @@
-// @ts-nocheck — legacy code from absorbed plugins (lp-manager, lpinfo, dexscreener, defi-news, birdeye); strict types pending cleanup
 import type { IAgentRuntime, Memory, Provider, State } from "@elizaos/core";
 import type { SteerLiquidityService } from "../services/steerLiquidityService";
 import type {
@@ -8,6 +7,18 @@ import type {
 
 const STEER_LIQUIDITY_TEXT_LIMIT = 4000;
 
+function getVaultTokenAddress(
+  vaultToken: SteerVaultDetailInput["token0"],
+): string {
+  if (!vaultToken || vaultToken === "Unknown") {
+    return "Unknown";
+  }
+
+  return typeof vaultToken === "string"
+    ? vaultToken
+    : vaultToken.address || "Unknown";
+}
+
 /**
  * Steer Finance Liquidity Protocol Provider
  * Provides information about Steer Finance vaults and staking pools
@@ -17,7 +28,7 @@ export const steerLiquidityProvider: Provider = {
   description:
     "Provides information about Steer Finance vaults, staking pools, and token-specific liquidity data across multiple chains",
   descriptionCompressed:
-    "provide information Steer Finance vault, stak pool, token-specific liquidity data across multiple chain",
+    "Steer Finance vault/staking pool/token liquidity across chains",
   dynamic: true,
   contexts: ["finance", "crypto", "wallet"],
   contextGate: { anyOf: ["finance", "crypto", "wallet"] },
@@ -25,15 +36,6 @@ export const steerLiquidityProvider: Provider = {
   cacheScope: "turn",
   roleGate: { minRole: "USER" },
   get: async (runtime: IAgentRuntime, message: Memory, _state: State) => {
-    console.log("🚀 STEER_LIQUIDITY provider called");
-    console.log("📝 Message content:", message.content.text);
-    console.log("📺 Channel type:", message.content.channelType);
-    console.log("🔧 Runtime methods available:", {
-      getService: !!runtime.getService,
-      getCache: !!runtime.getCache,
-      getSetting: !!runtime.getSetting,
-    });
-
     let liquidityInfo = "";
 
     try {
@@ -42,7 +44,6 @@ export const steerLiquidityProvider: Provider = {
         "STEER_LIQUIDITY_SERVICE",
       ) as SteerLiquidityService;
       if (!steerLiquidityService) {
-        console.log("Steer liquidity service not available");
         return {
           data: {},
           values: {},
@@ -50,27 +51,21 @@ export const steerLiquidityProvider: Provider = {
         };
       }
 
-      console.log("Steer liquidity service found, generating report...");
-
       liquidityInfo += `=== STEER FINANCE LIQUIDITY POOLS REPORT ===\n\n`;
 
       // Extract token address and chain from message content
       const content = message.content.text || "";
-      console.log("Searching for token address and chain in content:", content);
 
       // More flexible regex to catch various formats
       const tokenMatch = content.match(/(0x[a-fA-F0-9]{40})/);
-      console.log("Token match result:", tokenMatch);
 
       // Also try to find any 0x pattern that might be a token address
       const anyHexMatch = content.match(/(0x[a-fA-F0-9]+)/);
-      console.log("Any hex match result:", anyHexMatch);
 
       // Extract chain name from content
       const chainMatch = content.match(
         /\b(base|ethereum|mainnet|polygon|arbitrum|optimism)\b/i,
       );
-      console.log("Chain match result:", chainMatch);
 
       // Map chain names to chain IDs
       const chainNameToId: { [key: string]: number } = {
@@ -85,7 +80,6 @@ export const steerLiquidityProvider: Provider = {
       const targetChainId = chainMatch
         ? chainNameToId[chainMatch[1].toLowerCase()]
         : null;
-      console.log("Target chain ID:", targetChainId);
 
       // Validate chain if specified
       if (
@@ -104,7 +98,6 @@ export const steerLiquidityProvider: Provider = {
 
       if (tokenMatch) {
         const tokenIdentifier = tokenMatch[1];
-        console.log(`Token identifier found: ${tokenIdentifier}`);
 
         // Validate token address format
         if (!isValidEthereumAddress(tokenIdentifier)) {
@@ -180,7 +173,7 @@ export const steerLiquidityProvider: Provider = {
 
     return {
       data,
-      values: {} as Record<string, unknown>,
+      values: {},
       text,
     };
   },
@@ -272,7 +265,7 @@ async function getVaultDetails(vault: SteerVaultDetailInput): Promise<string> {
     const token0Address =
       typeof vault.token0 === "string"
         ? vault.token0
-        : vault.token0?.address || "Unknown";
+        : vault.token0.address || "Unknown";
     const token0Symbol = getTokenSymbol(token0Address);
     details += `   🪙 Token0: ${token0Address} (${token0Symbol})\n`;
   } else {
@@ -283,7 +276,7 @@ async function getVaultDetails(vault: SteerVaultDetailInput): Promise<string> {
     const token1Address =
       typeof vault.token1 === "string"
         ? vault.token1
-        : vault.token1?.address || "Unknown";
+        : vault.token1.address || "Unknown";
     const token1Symbol = getTokenSymbol(token1Address);
     details += `   🪙 Token1: ${token1Address} (${token1Symbol})\n`;
   } else {
@@ -474,8 +467,10 @@ async function getSingleAssetDepositInfo(
         depositInfo += `   🔄 Strategy: ${vault.strategyType}\n`;
         depositInfo += `   🏊 Pool: ${vault.poolAddress || "N/A"}\n`;
         depositInfo += `   📝 Single-Asset Contract: ${vault.singleAssetDepositContract}\n`;
-        depositInfo += `   🪙 Token0: ${vault.token0} (${getTokenSymbol(typeof vault.token0 === "string" ? vault.token0 : vault.token0?.address || "Unknown")})\n`;
-        depositInfo += `   🪙 Token1: ${vault.token1} (${getTokenSymbol(typeof vault.token1 === "string" ? vault.token1 : vault.token1?.address || "Unknown")})\n`;
+        const token0Address = getVaultTokenAddress(vault.token0);
+        const token1Address = getVaultTokenAddress(vault.token1);
+        depositInfo += `   🪙 Token0: ${token0Address} (${getTokenSymbol(token0Address)})\n`;
+        depositInfo += `   🪙 Token1: ${token1Address} (${getTokenSymbol(token1Address)})\n`;
 
         // Add additional APY breakdown if available
         if (vault.apr1d || vault.apr7d || vault.apr14d) {
@@ -501,7 +496,11 @@ async function getSingleAssetDepositInfo(
         depositInfo += `\n`;
 
         depositInfo += `   💡 Single-Asset Deposit Features:\n`;
-        depositInfo += `      • Deposit only one token (${tokenIdentifier === vault.token0.toLowerCase() ? "Token0" : "Token1"})\n`;
+        const depositSide =
+          tokenIdentifier.toLowerCase() === token0Address.toLowerCase()
+            ? "Token0"
+            : "Token1";
+        depositInfo += `      • Deposit only one token (${depositSide})\n`;
         depositInfo += `      • Automatic internal swap to balance the pair\n`;
         depositInfo += `      • Configurable slippage protection\n`;
         depositInfo += `      • Preview functionality before execution\n`;