feat: Remove entrypoint collection

[spalladino]

Attempt at removing the horrifying EntrypointCollection class. This allows us to remove the { sender: address } option when sending a tx, and instead just rely on the Wallet attached to the Contract instance to decide which sender to use. In other words, there is now only one way to choose which account sends a tx, following ethers conventions, rather than web3js. It also allows us to clean up some utils that are no longer necessary.

[spalladino]

So the CI is failing to build due to an error on a file that was deleted on this PR:

18:37:59| Step 4/6 : RUN yarn build && yarn formatting && yarn test
18:38:00| ---> Running in 5dae2e9d77ea
18:38:07| ../aztec.js/src/account/entrypoint/entrypoint_collection.ts(5,10): error TS2305: Module '"./index.js"' has no exported member 'CreateTxRequestOpts'.
18:38:07| ../aztec.js/src/account/entrypoint/entrypoint_collection.ts(49,54): error TS2554: Expected 1 arguments, but got 2.
The command '/bin/sh -c yarn build && yarn formatting && yarn test' returned a non-zero code: 2

[ludamad]

Does this seem like a rebuild issue? (The fact that this got thru CI)

[ludamad]

I guess given our current setup, this should really be all ts files. That or we shouldn't be getting the source from yarn-project-base

  "yarn-project-base": {
    "buildDir": "yarn-project",
    "dockerfile": "yarn-project-base/Dockerfile",
    "rebuildPatterns": [
      "^circuits/.*\\.(cpp|cc|cxx|c\\+\\+|h|hpp|hxx|h\\+\\+|c|h|inl|inc|ipp|tpp|cmake)$|^circuits/.*CMakeLists\\.txt$",
      "^l1-contracts/",
      "^yarn-project/l1-artifacts/",
      "^yarn-project/noir-contracts/",
      "^yarn-project/noir-libs/",
      "^yarn-project/noir-compiler/",
      "^yarn-project/yarn-project-base/",
      "^yarn-project/yarn.lock"
    ],

[ludamad]

See #2151

[spalladino]

Yup, looks like this line copies the entire yarn-project on yarn-project-base:

aztec-packages/yarn-project/yarn-project-base/Dockerfile

Line 117 in 27d5c76

COPY . .

I wouldn't go as far as setting all ts files as a rebuild pattern for yarn-project-base, or that would cause everything to rebuild every time. Maybe we can remove that line from the project base dockerfile?

[ludamad]

Will take this up Monday alright