fix: used identifiers in handler

[CanardMandarin]

This PR fixes the issue #55

It is a bit old, I hope it is still relevant.
The nonce identifier does not seem to be used anymore.

[acheroncrypto]

Hey, thank you for fixing this. Could you add a test case for it in tests/misc so that we don't break it again in the future.

[vercel]

@CanardMandarin is attempting to deploy a commit to the coral-xyz Team on Vercel.

A member of the Team first needs to authorize it.

[CanardMandarin]

Sure! What do you think of #91e6abe?

[acheroncrypto]

This is looking good to merge once the review comment is adressed and CHANGELOG is updated.

[CanardMandarin]

I'm glad you ask a test with the instruction macro, It caused some other collisions that I fixed with 6f862ee.

[acheroncrypto]

This is a breaking change for programs that depend on program_id, accounts, ix_data, remaining_accounts in their contexts. I doubt anybody uses those identifiers but I'm noting this here since it was being used in zero-copy tests.

Example:

#[derive(Accounts)]
pub struct MyInstruction<'info> {
    #[account(owner = *program_id)]
    my_account: AccountInfo<'info>
}

Programs shouldn't rely on variables that are defined inside codegen.