chore: remove redundant methods and use hash instead of compress in the interface

[kevaundray]

This removes quite a few methods from the interface:

• compress is effectively the same as hash now, so I've moved to using hash everywhere on the interface
• the withIndex methods are a superset of the methods without it -- I've left pedersenHash and pedersenHashWithIndex because the 0 hash index seems to be quite useful for circuits.js, however since circuits.js does not use barretenberg_api, we could likely get rid of it and just define it in circuits.js.

Related to #3029

Checklist:

Remove the checklist to signal you've completed it. Enable auto-merge if the PR is ready to merge.

• If the pull request requires a cryptography review (e.g. cryptographic algorithm implementations) I have added the 'crypto' tag.
• I have reviewed my diff in github, line by line and removed unexpected formatting changes, testing logs, or commented-out code.
• Every change is related to the PR description.
• I have linked this pull request to relevant issues (if any exist).

[kevaundray]

This has been merged into master piecewise by the linked commits, the issues encountered were:

• arguments are in the wrong order for the hash variant
• calling convention changes subtly which see the nuance between hash_index being a u32 and being a u32 const*