refactor: modify note processors and synchronizers to use complete address

[sklppy88]

I am modifying the note processors and synchronizers to use a complete address, instead of purely using the aztecaddress. This is because in the new address scheme, to decrypt, we will require the preaddress (a future member of the complete address), as well as the ivsk to calculate the "secret" of the address point. This is just scaffolding to be able to make the decrypting changes in the future.

The note processors specifically need this, but the synchronizer previously did not have it, but it was fetching the complete address anyways from the pxe db.

[sklppy88]

• feat: modify contract instance to include public keys #9153
• feat: fix encoding of public keys #9158
• refactor: modify note processors and synchronizers to use complete address #9152 👈
• refactor: Move contract stuff from types into circuits.js #9151
• refactor: move public keys to protocol circuits #9074 : 1 other dependent PR (#8950 )
• master

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @sklppy88 and the rest of your teammates on Graphite

[sklppy88]

Warning

This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
Learn more

• feat: modify contract instance to include public keys #9153
• refactor: modify note processors and synchronizers to use complete address #9152 👈
• refactor: Move contract stuff from types into circuits.js #9151
• refactor: move public keys to protocol circuits #9074 : 1 other dependent PR (#8950 )
• master

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @sklppy88 and the rest of your teammates on Graphite

[dbanks12]

I wouldn't consider myself an authority on these parts of the code, but this looks fine to me!