Add types for user input in send/call

packages/web3-eth-contract/src/contract.ts

@@ -503,7 +503,7 @@ export class Contract<Abi extends ContractAbi>
 	 *
 	 * ```ts
 	 * myContract.deploy({
-	 *   input: '0x12345...',
+	 *   input: '0x12345...', // data keyword can be used, too. If input is used, data will be ignored.
 	 *   arguments: [123, 'My String']
 	 * })
 	 * .send({
@@ -604,10 +604,6 @@ export class Contract<Abi extends ContractAbi>
 			> => {
 				const modifiedOptions = { ...options };
 
-				// Remove to address
-				// modifiedOptions.to = '0x0000000000000000000000000000000000000000';
-				delete modifiedOptions.to;
-
 				// eslint-disable-next-line @typescript-eslint/no-unsafe-return
 				return this._contractMethodDeploySend(
 					abi as AbiFunctionFragment,
@@ -622,9 +618,6 @@ export class Contract<Abi extends ContractAbi>
 			) => {
 				const modifiedOptions = { ...options };
 
-				// Remove to address
-				delete modifiedOptions.to;
-
 				return this._contractMethodEstimateGas({
 					abi: abi as AbiFunctionFragment,
 					params: args as unknown[],

packages/web3-eth-contract/src/types.ts

@@ -304,7 +304,7 @@ export interface NonPayableMethodObject<Inputs = unknown[], Outputs = unknown[]>
 	 * @param block - If you pass this parameter it will not use the default block set with contract.defaultBlock. Pre-defined block numbers as `earliest`, `latest`, `pending`, `safe` or `finalized can also be used. Useful for requesting data from or replaying transactions in past blocks.
 	 * @returns The returned data of the createAccessList,  e.g. The generated access list for transaction.
 	 *
-	 *  ```ts
+	 * ```ts
 	 *  const result = await MyContract.methods.myFunction().createAccessList();
 	 *  console.log(result);
 	 *
@@ -319,7 +319,7 @@ export interface NonPayableMethodObject<Inputs = unknown[], Outputs = unknown[]>
 	 *   ],
 	 *   "gasUsed": "0x7671"
 	 * }
-	 *  ```
+	 * ```
 	 */
 	createAccessList(
 		tx?: NonPayableCallOptions,
@@ -493,7 +493,7 @@ export interface PayableMethodObject<Inputs = unknown[], Outputs = unknown[]> {
 	 * @param block - If you pass this parameter it will not use the default block set with contract.defaultBlock. Pre-defined block numbers as `earliest`, `latest`, `pending`, `safe` or `finalized can also be used. Useful for requesting data from or replaying transactions in past blocks.
 	 * @returns The returned data of the createAccessList,  e.g. The generated access list for transaction.
 	 *
-	 *  ```ts
+	 * ```ts
 	 *  const result = await MyContract.methods.myFunction().createAccessList();
 	 *  console.log(result);
 	 *
@@ -508,12 +508,9 @@ export interface PayableMethodObject<Inputs = unknown[], Outputs = unknown[]> {
 	 *   ],
 	 *   "gasUsed": "0x7671"
 	 * }
-	 *  ```
+	 *```
 	 */
-	createAccessList(
-		tx?: NonPayableCallOptions,
-		block?: BlockNumberOrTag,
-	): Promise<AccessListResult>;
+	createAccessList(tx?: PayableCallOptions, block?: BlockNumberOrTag): Promise<AccessListResult>;
 }
 
 export type Web3ContractContext = Partial<

packages/web3-eth-contract/src/utils.ts

@@ -22,6 +22,7 @@ import {
 	TransactionWithSenderAPI,
 	TransactionCall,
 	HexString,
+	Address,
 	NonPayableCallOptions,
 	PayableCallOptions,
 	ContractInitOptions,
@@ -38,7 +39,11 @@ export const getSendTxParams = ({
 }: {
 	abi: AbiFunctionFragment;
 	params: unknown[];
-	options?: PayableCallOptions | NonPayableCallOptions;
+	options?: (PayableCallOptions | NonPayableCallOptions) & {
+		input?: HexString;
+		data?: HexString;
+		to?: Address;
+	};
 	contractOptions: ContractOptions;
 }): TransactionCall => {
 	const deploymentCall = options?.input ?? options?.data ?? contractOptions.input;
@@ -80,7 +85,7 @@ export const getEthTxCallParams = ({
 }: {
 	abi: AbiFunctionFragment;
 	params: unknown[];
-	options?: PayableCallOptions | NonPayableCallOptions;
+	options?: (PayableCallOptions | NonPayableCallOptions) & { to?: Address };
 	contractOptions: ContractOptions;
 }): TransactionCall => {
 	if (!options?.to && !contractOptions.address) {
@@ -162,7 +167,7 @@ export const getCreateAccessListParams = ({
 }: {
 	abi: AbiFunctionFragment;
 	params: unknown[];
-	options?: PayableCallOptions | NonPayableCallOptions;
+	options?: (PayableCallOptions | NonPayableCallOptions) & { to?: Address };
 	contractOptions: ContractOptions;
 }): TransactionForAccessList => {
 	if (!options?.to && !contractOptions.address) {