@manifest-network/manifest-mcp-browser 0.1.1 → 0.1.5

package/src/queries/billing.ts

@@ -1,15 +1,34 @@
 import { ManifestQueryClient } from '../client.js';
-import { ManifestMCPError, ManifestMCPErrorCode } from '../types.js';
-import { parseBigInt, defaultPagination } from './utils.js';
+import {
+  BillingParamsResult, LeaseResult, LeasesResult, CreditAccountResult,
+  CreditAccountsResult, CreditAddressResult, WithdrawableAmountResult,
+  ProviderWithdrawableResult, CreditEstimateResult
+} from '../types.js';
+import { parseBigInt, requireArgs, extractPaginationArgs } from './utils.js';
+import { throwUnsupportedSubcommand } from '../modules.js';
+
+/** Billing query result union type */
+type BillingQueryResult =
+  | BillingParamsResult
+  | LeaseResult
+  | LeasesResult
+  | CreditAccountResult
+  | CreditAccountsResult
+  | CreditAddressResult
+  | WithdrawableAmountResult
+  | ProviderWithdrawableResult
+  | CreditEstimateResult;
 
 /**
  * Route billing module query to manifestjs query client
+ *
+ * Paginated queries support --limit flag (default: 100, max: 1000)
  */
 export async function routeBillingQuery(
   queryClient: ManifestQueryClient,
   subcommand: string,
   args: string[]
-): Promise<Record<string, unknown>> {
+): Promise<BillingQueryResult> {
   const billing = queryClient.liftedinit.billing.v1;
 
   switch (subcommand) {
@@ -19,146 +38,87 @@ export async function routeBillingQuery(
     }
 
     case 'lease': {
-      if (args.length < 1) {
-        throw new ManifestMCPError(
-          ManifestMCPErrorCode.QUERY_FAILED,
-          'lease requires lease-uuid argument'
-        );
-      }
+      requireArgs(args, 1, ['lease-uuid'], 'billing lease');
       const [leaseUuid] = args;
       const result = await billing.lease({ leaseUuid });
       return { lease: result.lease };
     }
 
     case 'leases': {
+      const { pagination } = extractPaginationArgs(args, 'billing leases');
       // stateFilter: 0 = LEASE_STATE_UNSPECIFIED (returns all)
-      const result = await billing.leases({ stateFilter: 0, pagination: defaultPagination });
+      const result = await billing.leases({ stateFilter: 0, pagination });
       return { leases: result.leases, pagination: result.pagination };
     }
 
     case 'leases-by-tenant': {
-      if (args.length < 1) {
-        throw new ManifestMCPError(
-          ManifestMCPErrorCode.QUERY_FAILED,
-          'leases-by-tenant requires tenant-address argument'
-        );
-      }
-      const [tenant] = args;
-      const result = await billing.leasesByTenant({ tenant, stateFilter: 0, pagination: defaultPagination });
+      const { pagination, remainingArgs } = extractPaginationArgs(args, 'billing leases-by-tenant');
+      requireArgs(remainingArgs, 1, ['tenant-address'], 'billing leases-by-tenant');
+      const [tenant] = remainingArgs;
+      const result = await billing.leasesByTenant({ tenant, stateFilter: 0, pagination });
       return { leases: result.leases, pagination: result.pagination };
     }
 
     case 'leases-by-provider': {
-      if (args.length < 1) {
-        throw new ManifestMCPError(
-          ManifestMCPErrorCode.QUERY_FAILED,
-          'leases-by-provider requires provider-uuid argument'
-        );
-      }
-      const [providerUuid] = args;
-      const result = await billing.leasesByProvider({ providerUuid, stateFilter: 0, pagination: defaultPagination });
+      const { pagination, remainingArgs } = extractPaginationArgs(args, 'billing leases-by-provider');
+      requireArgs(remainingArgs, 1, ['provider-uuid'], 'billing leases-by-provider');
+      const [providerUuid] = remainingArgs;
+      const result = await billing.leasesByProvider({ providerUuid, stateFilter: 0, pagination });
       return { leases: result.leases, pagination: result.pagination };
     }
 
     case 'leases-by-sku': {
-      if (args.length < 1) {
-        throw new ManifestMCPError(
-          ManifestMCPErrorCode.QUERY_FAILED,
-          'leases-by-sku requires sku-uuid argument'
-        );
-      }
-      const [skuUuid] = args;
-      const result = await billing.leasesBySKU({ skuUuid, stateFilter: 0, pagination: defaultPagination });
+      const { pagination, remainingArgs } = extractPaginationArgs(args, 'billing leases-by-sku');
+      requireArgs(remainingArgs, 1, ['sku-uuid'], 'billing leases-by-sku');
+      const [skuUuid] = remainingArgs;
+      const result = await billing.leasesBySKU({ skuUuid, stateFilter: 0, pagination });
       return { leases: result.leases, pagination: result.pagination };
     }
 
     case 'credit-account': {
-      if (args.length < 1) {
-        throw new ManifestMCPError(
-          ManifestMCPErrorCode.QUERY_FAILED,
-          'credit-account requires tenant-address argument'
-        );
-      }
+      requireArgs(args, 1, ['tenant-address'], 'billing credit-account');
       const [tenant] = args;
       const result = await billing.creditAccount({ tenant });
       return { creditAccount: result.creditAccount };
     }
 
     case 'credit-accounts': {
-      const result = await billing.creditAccounts({ pagination: defaultPagination });
+      const { pagination } = extractPaginationArgs(args, 'billing credit-accounts');
+      const result = await billing.creditAccounts({ pagination });
       return { creditAccounts: result.creditAccounts, pagination: result.pagination };
     }
 
     case 'credit-address': {
-      if (args.length < 1) {
-        throw new ManifestMCPError(
-          ManifestMCPErrorCode.QUERY_FAILED,
-          'credit-address requires tenant-address argument'
-        );
-      }
+      requireArgs(args, 1, ['tenant-address'], 'billing credit-address');
       const [tenant] = args;
       const result = await billing.creditAddress({ tenant });
       return { creditAddress: result.creditAddress };
     }
 
     case 'withdrawable-amount': {
-      if (args.length < 1) {
-        throw new ManifestMCPError(
-          ManifestMCPErrorCode.QUERY_FAILED,
-          'withdrawable-amount requires lease-uuid argument'
-        );
-      }
+      requireArgs(args, 1, ['lease-uuid'], 'billing withdrawable-amount');
       const [leaseUuid] = args;
       const result = await billing.withdrawableAmount({ leaseUuid });
       return { amounts: result.amounts };
     }
 
     case 'provider-withdrawable': {
-      if (args.length < 1) {
-        throw new ManifestMCPError(
-          ManifestMCPErrorCode.QUERY_FAILED,
-          'provider-withdrawable requires provider-uuid argument'
-        );
-      }
+      requireArgs(args, 1, ['provider-uuid'], 'billing provider-withdrawable');
       const [providerUuid] = args;
-      // limit: max leases to process (default 100, max 1000)
+      // Optional: limit for max leases to process (default 100, max 1000)
       const limit = args[1] ? parseBigInt(args[1], 'limit') : BigInt(100);
       const result = await billing.providerWithdrawable({ providerUuid, limit });
       return { amounts: result.amounts };
     }
 
     case 'credit-estimate': {
-      if (args.length < 1) {
-        throw new ManifestMCPError(
-          ManifestMCPErrorCode.QUERY_FAILED,
-          'credit-estimate requires tenant-address argument'
-        );
-      }
+      requireArgs(args, 1, ['tenant-address'], 'billing credit-estimate');
       const [tenant] = args;
       const result = await billing.creditEstimate({ tenant });
       return { estimate: result };
     }
 
     default:
-      throw new ManifestMCPError(
-        ManifestMCPErrorCode.UNSUPPORTED_QUERY,
-        `Unsupported billing query subcommand: ${subcommand}`,
-        {
-          availableSubcommands: [
-            'params',
-            'lease',
-            'leases',
-            'leases-by-tenant',
-            'leases-by-provider',
-            'leases-by-sku',
-            'credit-account',
-            'credit-accounts',
-            'credit-address',
-            'withdrawable-amount',
-            'provider-withdrawable',
-            'credit-estimate',
-          ],
-        }
-      );
+      throwUnsupportedSubcommand('query', 'billing', subcommand);
   }
 }

package/src/queries/distribution.ts

@@ -1,26 +1,40 @@
 import { ManifestQueryClient } from '../client.js';
-import { ManifestMCPError, ManifestMCPErrorCode } from '../types.js';
-import { parseBigInt, defaultPagination } from './utils.js';
+import {
+  RewardsResult, CommissionResult, CommunityPoolResult, DistributionParamsResult,
+  ValidatorOutstandingRewardsResult, SlashesResult, DelegatorValidatorsResult,
+  DelegatorWithdrawAddressResult
+} from '../types.js';
+import { parseBigInt, requireArgs, extractPaginationArgs } from './utils.js';
+import { throwUnsupportedSubcommand } from '../modules.js';
+
+/** Distribution query result union type */
+type DistributionQueryResult =
+  | RewardsResult
+  | CommissionResult
+  | CommunityPoolResult
+  | DistributionParamsResult
+  | ValidatorOutstandingRewardsResult
+  | SlashesResult
+  | DelegatorValidatorsResult
+  | DelegatorWithdrawAddressResult;
 
 /**
  * Route distribution query to manifestjs query client
+ *
+ * Paginated queries support --limit flag (default: 100, max: 1000)
  */
 export async function routeDistributionQuery(
   queryClient: ManifestQueryClient,
   subcommand: string,
   args: string[]
-): Promise<Record<string, unknown>> {
+): Promise<DistributionQueryResult> {
   const distribution = queryClient.cosmos.distribution.v1beta1;
 
   switch (subcommand) {
     case 'rewards': {
-      if (args.length < 1) {
-        throw new ManifestMCPError(
-          ManifestMCPErrorCode.QUERY_FAILED,
-          'rewards requires delegator-address argument'
-        );
-      }
+      requireArgs(args, 1, ['delegator-address'], 'distribution rewards');
       const [delegatorAddress] = args;
+      // Optional: validator address for specific validator rewards
       const validatorAddress = args[1];
 
       if (validatorAddress) {
@@ -41,12 +55,7 @@ export async function routeDistributionQuery(
     }
 
     case 'commission': {
-      if (args.length < 1) {
-        throw new ManifestMCPError(
-          ManifestMCPErrorCode.QUERY_FAILED,
-          'commission requires validator-address argument'
-        );
-      }
+      requireArgs(args, 1, ['validator-address'], 'distribution commission');
       const [validatorAddress] = args;
       const result = await distribution.validatorCommission({ validatorAddress });
       return { commission: result.commission };
@@ -63,76 +72,43 @@ export async function routeDistributionQuery(
     }
 
     case 'validator-outstanding-rewards': {
-      if (args.length < 1) {
-        throw new ManifestMCPError(
-          ManifestMCPErrorCode.QUERY_FAILED,
-          'validator-outstanding-rewards requires validator-address argument'
-        );
-      }
+      requireArgs(args, 1, ['validator-address'], 'distribution validator-outstanding-rewards');
       const [validatorAddress] = args;
       const result = await distribution.validatorOutstandingRewards({ validatorAddress });
       return { rewards: result.rewards };
     }
 
     case 'slashes': {
-      if (args.length < 1) {
-        throw new ManifestMCPError(
-          ManifestMCPErrorCode.QUERY_FAILED,
-          'slashes requires validator-address argument'
-        );
-      }
-      const [validatorAddress] = args;
-      const startingHeight = args[1] ? parseBigInt(args[1], 'starting-height') : BigInt(0);
-      const endingHeight = args[2] ? parseBigInt(args[2], 'ending-height') : BigInt(Number.MAX_SAFE_INTEGER);
+      const { pagination, remainingArgs } = extractPaginationArgs(args, 'distribution slashes');
+      requireArgs(remainingArgs, 1, ['validator-address'], 'distribution slashes');
+      const [validatorAddress] = remainingArgs;
+      // Optional: starting and ending height for filtering
+      const startingHeight = remainingArgs[1] ? parseBigInt(remainingArgs[1], 'starting-height') : BigInt(0);
+      const endingHeight = remainingArgs[2] ? parseBigInt(remainingArgs[2], 'ending-height') : BigInt(Number.MAX_SAFE_INTEGER);
       const result = await distribution.validatorSlashes({
         validatorAddress,
         startingHeight,
         endingHeight,
-        pagination: defaultPagination,
+        pagination,
       });
       return { slashes: result.slashes, pagination: result.pagination };
     }
 
     case 'delegator-validators': {
-      if (args.length < 1) {
-        throw new ManifestMCPError(
-          ManifestMCPErrorCode.QUERY_FAILED,
-          'delegator-validators requires delegator-address argument'
-        );
-      }
+      requireArgs(args, 1, ['delegator-address'], 'distribution delegator-validators');
       const [delegatorAddress] = args;
       const result = await distribution.delegatorValidators({ delegatorAddress });
       return { validators: result.validators };
     }
 
     case 'delegator-withdraw-address': {
-      if (args.length < 1) {
-        throw new ManifestMCPError(
-          ManifestMCPErrorCode.QUERY_FAILED,
-          'delegator-withdraw-address requires delegator-address argument'
-        );
-      }
+      requireArgs(args, 1, ['delegator-address'], 'distribution delegator-withdraw-address');
       const [delegatorAddress] = args;
       const result = await distribution.delegatorWithdrawAddress({ delegatorAddress });
       return { withdrawAddress: result.withdrawAddress };
     }
 
     default:
-      throw new ManifestMCPError(
-        ManifestMCPErrorCode.UNSUPPORTED_QUERY,
-        `Unsupported distribution query subcommand: ${subcommand}`,
-        {
-          availableSubcommands: [
-            'rewards',
-            'commission',
-            'community-pool',
-            'params',
-            'validator-outstanding-rewards',
-            'slashes',
-            'delegator-validators',
-            'delegator-withdraw-address',
-          ],
-        }
-      );
+      throwUnsupportedSubcommand('query', 'distribution', subcommand);
   }
 }

package/src/queries/gov.ts

@@ -1,46 +1,54 @@
 import { ManifestQueryClient } from '../client.js';
-import { ManifestMCPError, ManifestMCPErrorCode } from '../types.js';
-import { parseBigInt, parseInt, defaultPagination } from './utils.js';
+import {
+  ProposalResult, ProposalsResult, VoteResult, VotesResult,
+  DepositResult, DepositsResult, TallyResult, GovParamsResult
+} from '../types.js';
+import { parseBigInt, parseInteger, requireArgs, extractPaginationArgs } from './utils.js';
+import { throwUnsupportedSubcommand } from '../modules.js';
+
+/** Gov query result union type */
+type GovQueryResult =
+  | ProposalResult
+  | ProposalsResult
+  | VoteResult
+  | VotesResult
+  | DepositResult
+  | DepositsResult
+  | TallyResult
+  | GovParamsResult;
 
 /**
  * Route gov query to manifestjs query client
+ *
+ * Paginated queries support --limit flag (default: 100, max: 1000)
  */
 export async function routeGovQuery(
   queryClient: ManifestQueryClient,
   subcommand: string,
   args: string[]
-): Promise<Record<string, unknown>> {
+): Promise<GovQueryResult> {
   const gov = queryClient.cosmos.gov.v1;
 
   switch (subcommand) {
     case 'proposal': {
-      if (args.length < 1) {
-        throw new ManifestMCPError(
-          ManifestMCPErrorCode.QUERY_FAILED,
-          'proposal requires proposal-id argument'
-        );
-      }
+      requireArgs(args, 1, ['proposal-id'], 'gov proposal');
       const proposalId = parseBigInt(args[0], 'proposal-id');
       const result = await gov.proposal({ proposalId });
       return { proposal: result.proposal };
     }
 
     case 'proposals': {
-      // Parse optional status filter
-      const proposalStatus = args[0] ? parseInt(args[0], 'status') : 0;
-      const voter = args[1] || '';
-      const depositor = args[2] || '';
-      const result = await gov.proposals({ proposalStatus, voter, depositor, pagination: defaultPagination });
+      const { pagination, remainingArgs } = extractPaginationArgs(args, 'gov proposals');
+      // All optional: status filter, voter, depositor
+      const proposalStatus = remainingArgs[0] ? parseInteger(remainingArgs[0], 'status') : 0;
+      const voter = remainingArgs[1] || '';
+      const depositor = remainingArgs[2] || '';
+      const result = await gov.proposals({ proposalStatus, voter, depositor, pagination });
       return { proposals: result.proposals, pagination: result.pagination };
     }
 
     case 'vote': {
-      if (args.length < 2) {
-        throw new ManifestMCPError(
-          ManifestMCPErrorCode.QUERY_FAILED,
-          'vote requires proposal-id and voter-address arguments'
-        );
-      }
+      requireArgs(args, 2, ['proposal-id', 'voter-address'], 'gov vote');
       const proposalId = parseBigInt(args[0], 'proposal-id');
       const voter = args[1];
       const result = await gov.vote({ proposalId, voter });
@@ -48,24 +56,15 @@ export async function routeGovQuery(
     }
 
     case 'votes': {
-      if (args.length < 1) {
-        throw new ManifestMCPError(
-          ManifestMCPErrorCode.QUERY_FAILED,
-          'votes requires proposal-id argument'
-        );
-      }
-      const proposalId = parseBigInt(args[0], 'proposal-id');
-      const result = await gov.votes({ proposalId, pagination: defaultPagination });
+      const { pagination, remainingArgs } = extractPaginationArgs(args, 'gov votes');
+      requireArgs(remainingArgs, 1, ['proposal-id'], 'gov votes');
+      const proposalId = parseBigInt(remainingArgs[0], 'proposal-id');
+      const result = await gov.votes({ proposalId, pagination });
       return { votes: result.votes, pagination: result.pagination };
     }
 
     case 'deposit': {
-      if (args.length < 2) {
-        throw new ManifestMCPError(
-          ManifestMCPErrorCode.QUERY_FAILED,
-          'deposit requires proposal-id and depositor-address arguments'
-        );
-      }
+      requireArgs(args, 2, ['proposal-id', 'depositor-address'], 'gov deposit');
       const proposalId = parseBigInt(args[0], 'proposal-id');
       const depositor = args[1];
       const result = await gov.deposit({ proposalId, depositor });
@@ -73,30 +72,22 @@ export async function routeGovQuery(
     }
 
     case 'deposits': {
-      if (args.length < 1) {
-        throw new ManifestMCPError(
-          ManifestMCPErrorCode.QUERY_FAILED,
-          'deposits requires proposal-id argument'
-        );
-      }
-      const proposalId = parseBigInt(args[0], 'proposal-id');
-      const result = await gov.deposits({ proposalId, pagination: defaultPagination });
+      const { pagination, remainingArgs } = extractPaginationArgs(args, 'gov deposits');
+      requireArgs(remainingArgs, 1, ['proposal-id'], 'gov deposits');
+      const proposalId = parseBigInt(remainingArgs[0], 'proposal-id');
+      const result = await gov.deposits({ proposalId, pagination });
       return { deposits: result.deposits, pagination: result.pagination };
     }
 
     case 'tally': {
-      if (args.length < 1) {
-        throw new ManifestMCPError(
-          ManifestMCPErrorCode.QUERY_FAILED,
-          'tally requires proposal-id argument'
-        );
-      }
+      requireArgs(args, 1, ['proposal-id'], 'gov tally');
       const proposalId = parseBigInt(args[0], 'proposal-id');
       const result = await gov.tallyResult({ proposalId });
       return { tally: result.tally };
     }
 
     case 'params': {
+      // Optional: params type (defaults to 'tallying')
       const paramsType = args[0] || 'tallying';
       const result = await gov.params({ paramsType });
       return {
@@ -108,21 +99,6 @@ export async function routeGovQuery(
     }
 
     default:
-      throw new ManifestMCPError(
-        ManifestMCPErrorCode.UNSUPPORTED_QUERY,
-        `Unsupported gov query subcommand: ${subcommand}`,
-        {
-          availableSubcommands: [
-            'proposal',
-            'proposals',
-            'vote',
-            'votes',
-            'deposit',
-            'deposits',
-            'tally',
-            'params',
-          ],
-        }
-      );
+      throwUnsupportedSubcommand('query', 'gov', subcommand);
   }
 }

package/src/queries/index.ts

@@ -1,4 +1,13 @@
-export { parseBigInt, parseInt, defaultPagination, DEFAULT_PAGE_LIMIT } from './utils.js';
+export {
+  parseBigInt,
+  parseInteger,
+  createPagination,
+  extractPaginationArgs,
+  DEFAULT_PAGE_LIMIT,
+  MAX_PAGE_LIMIT,
+  defaultPagination,
+} from './utils.js';
+export type { PaginationConfig } from './utils.js';
 export { routeBankQuery } from './bank.js';
 export { routeStakingQuery } from './staking.js';
 export { routeDistributionQuery } from './distribution.js';