fix:remove agentStartBlock and use mailbox.deployedBlock() instead

[aroralanuk]

Description

• CLI was using a static start block numbers for core chains or getting the latest block number from a PI chain for agent config which is redundant and dangerous. Instead, I updated to using the mailbox.deployedBlock() which should precede all other indexable contract deployment and is hence safer.

Drive-by changes

• filtering the agent configs in CLI is redundant since the HyperlaneDeploymentArtifactsSchema requires all the specified entries and if writeAgentConfig gets an artifacts which doesn't contain these, we should throw an error and not filter them.

Related issues

• related to https://github.com/hyperlane-xyz/issues/issues/736

Backward compatibility

Yes

Testing

Manual b/w anvil1 and anvil2

[changeset-bot]

🦋 Changeset detected

Latest commit: 325e11f

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 6 packages

Name	Type
@hyperlane-xyz/infra	Patch
@hyperlane-xyz/cli	Patch
@hyperlane-xyz/sdk	Patch
@hyperlane-xyz/helloworld	Patch
@hyperlane-xyz/core	Patch
@hyperlane-xyz/utils	Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

[yorhodes]

nice cleanup :)