[Rename] Change arguments in the transaction builder to functionArguments, change references in docs

[xbtmatt]

Description

Because arguments is a reserved keyword in typescript, when you try to use the spread operator with it in strict mode, it doesn't let you.

For example:

const transactionBuilder = async(args: {
    aptos: Aptos,
    sender: Account,
    moduleFunction: string,
    arguments: Array<EntryFunctionArgumentTypes>,
    typeArguments?: TypeTag[],
}): Promise<UserTransactionResponse> => {
    const { aptos, sender, moduleFunction, arguments, typeArguments } = args;
    // ..

The compiler won't let you destructure args because arguments is reserved. You could of course do arguments: functionArguments, but perhaps we should just avoid the reserved keyword entirely to avoid issues

I've also changed each instance of type_arguments in the docs to typeArguments

I also made sure not to change arguments to functionArguments in API calls where the JSON structure is required

Test Plan

pnpm jest

[gregnazario]

Can we add a CHANGELOG?

[0xmigo]

LGTM