@manifest-network/manifest-mcp-browser 0.1.6 → 0.1.8

package/src/client.ts

@@ -1,288 +0,0 @@
-import { SigningStargateClient, GasPrice, HttpEndpoint } from '@cosmjs/stargate';
-import {
-  cosmosProtoRegistry,
-  cosmosAminoConverters,
-  liftedinitProtoRegistry,
-  liftedinitAminoConverters,
-  strangeloveVenturesProtoRegistry,
-  strangeloveVenturesAminoConverters,
-  osmosisProtoRegistry,
-  osmosisAminoConverters,
-  liftedinit,
-} from '@manifest-network/manifestjs';
-import { Registry } from '@cosmjs/proto-signing';
-import { AminoTypes } from '@cosmjs/stargate';
-import { RateLimiter } from 'limiter';
-import { ManifestMCPConfig, WalletProvider, ManifestMCPError, ManifestMCPErrorCode } from './types.js';
-import { DEFAULT_REQUESTS_PER_SECOND } from './config.js';
-
-// Type for the RPC query client from manifestjs liftedinit bundle
-// This includes cosmos modules + liftedinit-specific modules (billing, manifest, sku)
-export type ManifestQueryClient = Awaited<ReturnType<typeof liftedinit.ClientFactory.createRPCQueryClient>>;
-
-/** Default timeout for transaction broadcast (60 seconds) */
-const DEFAULT_BROADCAST_TIMEOUT_MS = 60_000;
-
-/** Default polling interval for transaction confirmation (3 seconds) */
-const DEFAULT_BROADCAST_POLL_INTERVAL_MS = 3_000;
-
-/**
- * Get combined signing client options with all Manifest registries
- */
-function getSigningManifestClientOptions() {
-  const registry = new Registry([
-    ...cosmosProtoRegistry,
-    ...liftedinitProtoRegistry,
-    ...strangeloveVenturesProtoRegistry,
-    ...osmosisProtoRegistry,
-  ]);
-
-  const aminoTypes = new AminoTypes({
-    ...cosmosAminoConverters,
-    ...liftedinitAminoConverters,
-    ...strangeloveVenturesAminoConverters,
-    ...osmosisAminoConverters,
-  });
-
-  return { registry, aminoTypes };
-}
-
-/**
- * Manages CosmJS client instances with lazy initialization and singleton pattern
- */
-export class CosmosClientManager {
-  private static instances: Map<string, CosmosClientManager> = new Map();
-
-  private config: ManifestMCPConfig;
-  private walletProvider: WalletProvider;
-  private queryClient: ManifestQueryClient | null = null;
-  private signingClient: SigningStargateClient | null = null;
-  private rateLimiter: RateLimiter;
-
-  // Promises to prevent concurrent client initialization (lazy init race condition)
-  private queryClientPromise: Promise<ManifestQueryClient> | null = null;
-  private signingClientPromise: Promise<SigningStargateClient> | null = null;
-
-  private constructor(config: ManifestMCPConfig, walletProvider: WalletProvider) {
-    this.config = config;
-    this.walletProvider = walletProvider;
-
-    // Initialize rate limiter with configured or default requests per second
-    const requestsPerSecond = config.rateLimit?.requestsPerSecond ?? DEFAULT_REQUESTS_PER_SECOND;
-    this.rateLimiter = new RateLimiter({
-      tokensPerInterval: requestsPerSecond,
-      interval: 'second',
-    });
-  }
-
-  /**
-   * Get or create a singleton instance for the given config.
-   * Instances are keyed by chainId:rpcUrl. For existing instances:
-   * - Config and walletProvider references are always updated
-   * - Signing client is disconnected/recreated if gasPrice or walletProvider changed
-   * - Rate limiter is updated if requestsPerSecond changed (without affecting signing client)
-   */
-  static getInstance(
-    config: ManifestMCPConfig,
-    walletProvider: WalletProvider
-  ): CosmosClientManager {
-    const key = `${config.chainId}:${config.rpcUrl}`;
-    let instance = CosmosClientManager.instances.get(key);
-
-    if (!instance) {
-      instance = new CosmosClientManager(config, walletProvider);
-      CosmosClientManager.instances.set(key, instance);
-    } else {
-      // Check what changed to determine what needs updating
-      const signingClientAffected =
-        instance.config.gasPrice !== config.gasPrice ||
-        instance.walletProvider !== walletProvider;
-
-      const rateLimitChanged =
-        instance.config.rateLimit?.requestsPerSecond !== config.rateLimit?.requestsPerSecond;
-
-      // Always update config reference
-      instance.config = config;
-      instance.walletProvider = walletProvider;
-
-      // Only invalidate signing client if fields it depends on changed
-      if (signingClientAffected) {
-        if (instance.signingClient) {
-          instance.signingClient.disconnect();
-          instance.signingClient = null;
-        }
-        // Also clear the promise to allow re-initialization with new config
-        instance.signingClientPromise = null;
-      }
-
-      // Update rate limiter independently (doesn't affect signing client)
-      if (rateLimitChanged) {
-        const newRps = config.rateLimit?.requestsPerSecond ?? DEFAULT_REQUESTS_PER_SECOND;
-        instance.rateLimiter = new RateLimiter({
-          tokensPerInterval: newRps,
-          interval: 'second',
-        });
-      }
-    }
-
-    return instance;
-  }
-
-  /**
-   * Clear all cached instances (useful for testing or reconnection)
-   */
-  static clearInstances(): void {
-    CosmosClientManager.instances.clear();
-  }
-
-  /**
-   * Get the manifestjs RPC query client with all module extensions
-   */
-  async getQueryClient(): Promise<ManifestQueryClient> {
-    // Return cached client if available
-    if (this.queryClient) {
-      return this.queryClient;
-    }
-
-    // If initialization is already in progress, wait for it
-    if (this.queryClientPromise) {
-      return this.queryClientPromise;
-    }
-
-    // Start initialization and cache the promise to prevent concurrent init
-    this.queryClientPromise = (async () => {
-      // Capture reference to detect if superseded by disconnect/config change
-      const thisInitPromise = this.queryClientPromise;
-      try {
-        // Use liftedinit ClientFactory which includes cosmos + liftedinit modules
-        const client = await liftedinit.ClientFactory.createRPCQueryClient({
-          rpcEndpoint: this.config.rpcUrl,
-        });
-        // Only store if this is still the active promise
-        if (this.queryClientPromise === thisInitPromise) {
-          this.queryClient = client;
-          this.queryClientPromise = null;
-        }
-        return client;
-      } catch (error) {
-        // Clear promise on failure so retry is possible (only if still active)
-        if (this.queryClientPromise === thisInitPromise) {
-          this.queryClientPromise = null;
-        }
-        throw new ManifestMCPError(
-          ManifestMCPErrorCode.RPC_CONNECTION_FAILED,
-          `Failed to connect to RPC endpoint: ${error instanceof Error ? error.message : String(error)}`,
-          { rpcUrl: this.config.rpcUrl }
-        );
-      }
-    })();
-
-    return this.queryClientPromise;
-  }
-
-  /**
-   * Get a signing client with all Manifest registries (for transactions)
-   */
-  async getSigningClient(): Promise<SigningStargateClient> {
-    // Return cached client if available
-    if (this.signingClient) {
-      return this.signingClient;
-    }
-
-    // If initialization is already in progress, wait for it
-    if (this.signingClientPromise) {
-      return this.signingClientPromise;
-    }
-
-    // Start initialization and cache the promise to prevent concurrent init
-    this.signingClientPromise = (async () => {
-      // Capture reference to detect if superseded by disconnect/config change
-      const thisInitPromise = this.signingClientPromise;
-      try {
-        const signer = await this.walletProvider.getSigner();
-        const gasPrice = GasPrice.fromString(this.config.gasPrice);
-        const { registry, aminoTypes } = getSigningManifestClientOptions();
-
-        // Configure endpoint with HTTP timeout
-        const endpoint: HttpEndpoint = {
-          url: this.config.rpcUrl,
-          headers: {},
-        };
-
-        // Note: Registry type from @cosmjs/proto-signing doesn't perfectly match
-        // SigningStargateClientOptions due to telescope-generated proto types.
-        // This is a known limitation with custom cosmos-sdk module registries.
-        const client = await SigningStargateClient.connectWithSigner(
-          endpoint,
-          signer,
-          {
-            registry: registry as Parameters<typeof SigningStargateClient.connectWithSigner>[2] extends { registry?: infer R } ? R : never,
-            aminoTypes,
-            gasPrice,
-            broadcastTimeoutMs: DEFAULT_BROADCAST_TIMEOUT_MS,
-            broadcastPollIntervalMs: DEFAULT_BROADCAST_POLL_INTERVAL_MS,
-          }
-        );
-        // Only store if this is still the active promise
-        if (this.signingClientPromise === thisInitPromise) {
-          this.signingClient = client;
-          this.signingClientPromise = null;
-        } else {
-          // Promise was superseded, clean up the client we just created
-          client.disconnect();
-        }
-        return client;
-      } catch (error) {
-        // Clear promise on failure so retry is possible (only if still active)
-        if (this.signingClientPromise === thisInitPromise) {
-          this.signingClientPromise = null;
-        }
-        if (error instanceof ManifestMCPError) {
-          throw error;
-        }
-        throw new ManifestMCPError(
-          ManifestMCPErrorCode.RPC_CONNECTION_FAILED,
-          `Failed to connect signing client: ${error instanceof Error ? error.message : String(error)}`,
-          { rpcUrl: this.config.rpcUrl }
-        );
-      }
-    })();
-
-    return this.signingClientPromise;
-  }
-
-  /**
-   * Get the wallet address
-   */
-  async getAddress(): Promise<string> {
-    return this.walletProvider.getAddress();
-  }
-
-  /**
-   * Get the configuration
-   */
-  getConfig(): ManifestMCPConfig {
-    return this.config;
-  }
-
-  /**
-   * Acquire a rate limit token before making an RPC request.
-   * This will wait if the rate limit has been exceeded.
-   */
-  async acquireRateLimit(): Promise<void> {
-    await this.rateLimiter.removeTokens(1);
-  }
-
-  /**
-   * Disconnect all clients
-   */
-  disconnect(): void {
-    if (this.signingClient) {
-      this.signingClient.disconnect();
-      this.signingClient = null;
-    }
-    this.signingClientPromise = null;
-    this.queryClient = null;
-    this.queryClientPromise = null;
-  }
-}

package/src/config.test.ts

@@ -1,299 +0,0 @@
-import { describe, it, expect } from 'vitest';
-import { createConfig, validateConfig, createValidatedConfig } from './config.js';
-import { ManifestMCPError, ManifestMCPErrorCode } from './types.js';
-
-describe('createConfig', () => {
-  it('should apply default values', () => {
-    const config = createConfig({
-      chainId: 'test-chain',
-      rpcUrl: 'https://example.com',
-      gasPrice: '1.0umfx',
-    });
-
-    expect(config.chainId).toBe('test-chain');
-    expect(config.rpcUrl).toBe('https://example.com');
-    expect(config.gasPrice).toBe('1.0umfx');
-    expect(config.gasAdjustment).toBe(1.3);
-    expect(config.addressPrefix).toBe('manifest');
-  });
-
-  it('should preserve provided optional values', () => {
-    const config = createConfig({
-      chainId: 'test-chain',
-      rpcUrl: 'https://example.com',
-      gasPrice: '1.0umfx',
-      gasAdjustment: 2.0,
-      addressPrefix: 'custom',
-    });
-
-    expect(config.gasAdjustment).toBe(2.0);
-    expect(config.addressPrefix).toBe('custom');
-  });
-
-  it('should apply default rateLimit', () => {
-    const config = createConfig({
-      chainId: 'test-chain',
-      rpcUrl: 'https://example.com',
-      gasPrice: '1.0umfx',
-    });
-
-    expect(config.rateLimit).toBeDefined();
-    expect(config.rateLimit?.requestsPerSecond).toBe(10);
-  });
-
-  it('should preserve provided rateLimit', () => {
-    const config = createConfig({
-      chainId: 'test-chain',
-      rpcUrl: 'https://example.com',
-      gasPrice: '1.0umfx',
-      rateLimit: { requestsPerSecond: 20 },
-    });
-
-    expect(config.rateLimit?.requestsPerSecond).toBe(20);
-  });
-});
-
-describe('validateConfig', () => {
-  it('should return valid for correct config', () => {
-    const result = validateConfig({
-      chainId: 'manifest-testnet',
-      rpcUrl: 'https://rpc.example.com',
-      gasPrice: '1.0umfx',
-    });
-
-    expect(result.valid).toBe(true);
-    expect(result.errors).toHaveLength(0);
-  });
-
-  it('should detect missing required fields', () => {
-    const result = validateConfig({});
-
-    expect(result.valid).toBe(false);
-    expect(result.errors).toContain('chainId is required');
-    expect(result.errors).toContain('rpcUrl is required');
-    expect(result.errors).toContain('gasPrice is required');
-  });
-
-  it('should validate chainId format', () => {
-    const result = validateConfig({
-      chainId: '-invalid',
-      rpcUrl: 'https://example.com',
-      gasPrice: '1.0umfx',
-    });
-
-    expect(result.valid).toBe(false);
-    expect(result.errors.some(e => e.includes('chainId'))).toBe(true);
-  });
-
-  it('should validate rpcUrl format', () => {
-    const result = validateConfig({
-      chainId: 'test',
-      rpcUrl: 'not-a-url',
-      gasPrice: '1.0umfx',
-    });
-
-    expect(result.valid).toBe(false);
-    expect(result.errors.some(e => e.includes('rpcUrl'))).toBe(true);
-  });
-
-  it('should validate gasPrice format', () => {
-    const result = validateConfig({
-      chainId: 'test',
-      rpcUrl: 'https://example.com',
-      gasPrice: 'invalid',
-    });
-
-    expect(result.valid).toBe(false);
-    expect(result.errors.some(e => e.includes('gasPrice'))).toBe(true);
-  });
-
-  it('should validate optional gasAdjustment', () => {
-    const result = validateConfig({
-      chainId: 'test',
-      rpcUrl: 'https://example.com',
-      gasPrice: '1.0umfx',
-      gasAdjustment: -1,
-    });
-
-    expect(result.valid).toBe(false);
-    expect(result.errors.some(e => e.includes('gasAdjustment'))).toBe(true);
-  });
-
-  it('should validate optional addressPrefix', () => {
-    const result = validateConfig({
-      chainId: 'test',
-      rpcUrl: 'https://example.com',
-      gasPrice: '1.0umfx',
-      addressPrefix: 'INVALID',
-    });
-
-    expect(result.valid).toBe(false);
-    expect(result.errors.some(e => e.includes('addressPrefix'))).toBe(true);
-  });
-
-  it('should accept HTTPS URLs', () => {
-    const result = validateConfig({
-      chainId: 'test',
-      rpcUrl: 'https://rpc.example.com',
-      gasPrice: '1.0umfx',
-    });
-
-    expect(result.valid).toBe(true);
-  });
-
-  it('should accept HTTP URLs for localhost', () => {
-    const result = validateConfig({
-      chainId: 'test',
-      rpcUrl: 'http://localhost:26657',
-      gasPrice: '1.0umfx',
-    });
-
-    expect(result.valid).toBe(true);
-  });
-
-  it('should accept HTTP URLs for 127.0.0.1', () => {
-    const result = validateConfig({
-      chainId: 'test',
-      rpcUrl: 'http://127.0.0.1:26657',
-      gasPrice: '1.0umfx',
-    });
-
-    expect(result.valid).toBe(true);
-  });
-
-  it('should accept HTTP URLs for IPv6 localhost', () => {
-    const result = validateConfig({
-      chainId: 'test',
-      rpcUrl: 'http://[::1]:26657',
-      gasPrice: '1.0umfx',
-    });
-
-    expect(result.valid).toBe(true);
-  });
-
-  it('should reject HTTP URLs for non-localhost', () => {
-    const result = validateConfig({
-      chainId: 'test',
-      rpcUrl: 'http://rpc.example.com',
-      gasPrice: '1.0umfx',
-    });
-
-    expect(result.valid).toBe(false);
-    expect(result.errors.some(e => e.includes('HTTPS'))).toBe(true);
-  });
-
-  it('should accept valid rateLimit config', () => {
-    const result = validateConfig({
-      chainId: 'test',
-      rpcUrl: 'https://example.com',
-      gasPrice: '1.0umfx',
-      rateLimit: { requestsPerSecond: 5 },
-    });
-
-    expect(result.valid).toBe(true);
-  });
-
-  it('should reject non-integer rateLimit.requestsPerSecond', () => {
-    const result = validateConfig({
-      chainId: 'test',
-      rpcUrl: 'https://example.com',
-      gasPrice: '1.0umfx',
-      rateLimit: { requestsPerSecond: 5.5 },
-    });
-
-    expect(result.valid).toBe(false);
-    expect(result.errors.some(e => e.includes('requestsPerSecond'))).toBe(true);
-  });
-
-  it('should reject negative rateLimit.requestsPerSecond', () => {
-    const result = validateConfig({
-      chainId: 'test',
-      rpcUrl: 'https://example.com',
-      gasPrice: '1.0umfx',
-      rateLimit: { requestsPerSecond: -1 },
-    });
-
-    expect(result.valid).toBe(false);
-    expect(result.errors.some(e => e.includes('requestsPerSecond'))).toBe(true);
-  });
-
-  it('should reject zero rateLimit.requestsPerSecond', () => {
-    const result = validateConfig({
-      chainId: 'test',
-      rpcUrl: 'https://example.com',
-      gasPrice: '1.0umfx',
-      rateLimit: { requestsPerSecond: 0 },
-    });
-
-    expect(result.valid).toBe(false);
-    expect(result.errors.some(e => e.includes('requestsPerSecond'))).toBe(true);
-  });
-
-  it('should reject null rateLimit', () => {
-    const result = validateConfig({
-      chainId: 'test',
-      rpcUrl: 'https://example.com',
-      gasPrice: '1.0umfx',
-      rateLimit: null as unknown as { requestsPerSecond?: number },
-    });
-
-    expect(result.valid).toBe(false);
-    expect(result.errors.some(e => e.includes('rateLimit must be a plain object'))).toBe(true);
-  });
-
-  it('should reject non-object rateLimit', () => {
-    const result = validateConfig({
-      chainId: 'test',
-      rpcUrl: 'https://example.com',
-      gasPrice: '1.0umfx',
-      rateLimit: 'invalid' as unknown as { requestsPerSecond?: number },
-    });
-
-    expect(result.valid).toBe(false);
-    expect(result.errors.some(e => e.includes('rateLimit must be a plain object'))).toBe(true);
-  });
-
-  it('should reject array rateLimit', () => {
-    const result = validateConfig({
-      chainId: 'test',
-      rpcUrl: 'https://example.com',
-      gasPrice: '1.0umfx',
-      rateLimit: [] as unknown as { requestsPerSecond?: number },
-    });
-
-    expect(result.valid).toBe(false);
-    expect(result.errors.some(e => e.includes('rateLimit must be a plain object'))).toBe(true);
-  });
-});
-
-describe('createValidatedConfig', () => {
-  it('should return config for valid input', () => {
-    const config = createValidatedConfig({
-      chainId: 'test-chain',
-      rpcUrl: 'https://example.com',
-      gasPrice: '1.0umfx',
-    });
-
-    expect(config.chainId).toBe('test-chain');
-  });
-
-  it('should throw ManifestMCPError for invalid input', () => {
-    expect(() => createValidatedConfig({
-      chainId: '',
-      rpcUrl: '',
-      gasPrice: '',
-    })).toThrow(ManifestMCPError);
-  });
-
-  it('should have INVALID_CONFIG error code', () => {
-    try {
-      createValidatedConfig({
-        chainId: '',
-        rpcUrl: '',
-        gasPrice: '',
-      });
-    } catch (error) {
-      expect((error as ManifestMCPError).code).toBe(ManifestMCPErrorCode.INVALID_CONFIG);
-    }
-  });
-});