More functions need to allow bigints as args

[jeapostrophe]

I am modifying Reach to always use bigint int decoding, but there are some problems...

• SuggestParams doesn't allow bigints in its fields
• makeAssetTransferTxnWithSuggestedParams -- specifically the assetIndex could be a bigint
• makeApplicationXYZ --- specifically the app id, foreign, and asset ids
• indexer.lookupApplications
• indexer.searchForTransactions().minRound()

[michaeldiamant]

Summarizing outcomes from a group brainstorm:

• Philosophically, it feels most correct to change fields from number to bigint everywhere that go-algorand defines a uint64 data type. In practice, it causes a backwards incompatibility and adds some friction to developer ergonomics (can't perform arithmetic with number and bigint).
• Given that changing field types feels out of reach, we agreed it'd be reasonable to expand function types to accept number | bigint + perform safe coercion to number.
• We confirmed via visual inspection that Transaction construction defends against unsafe conversions to number using Number.isSafeInteger. Here's an example check:

  js-algorand-sdk/src/transaction.ts

  Lines 402 to 411 in 894e287

  	if (txn.appForeignApps !== undefined) {
  	if (!Array.isArray(txn.appForeignApps))
  	throw Error('appForeignApps must be an Array of integers.');
  	txn.appForeignApps = txn.appForeignApps.slice();
  	txn.appForeignApps.forEach((foreignAppIndex) => {
  	if (!Number.isSafeInteger(foreignAppIndex) || foreignAppIndex < 0)
  	throw Error(
  	'each foreign application index must be a positive number and smaller than 2^53-1'
  	);
  	});