Attach a typed lifetime constraint field when we compile a transaction

[mcintyre94]

This PR adds a typed lifetimeConstraint field to the return of compileTransaction, ie to the Transaction object produced by compiling a TransactionMessage. This is required to distinguish between a blockhash and a nonce - the value of these is stored in the lifetimeToken field (and ultimately serialized) but the type is not.

This is required for transaction confirmation which relies on the lifetime constraint to know how to confirm the transaction. Our new Transaction object alone is insufficient for this.

In a future PR I'll need to add code that can take a Transaction and find its lifetime constraint, which will require decoding it and inspecting its first instruction. But in the case where we have an uncompiled TransactionMessage we already know the constraint type and can attach it.

[changeset-bot]

⚠️ No Changeset found

Latest commit: ef42435

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

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

[mcintyre94]

• Refactor send-transaction to use NewTransaction #2487 : 2 dependent PRs (#2488 , #2492 )
• add TransactionWithXLifetime types #2486
• Refactor transaction-confirmation to use new transaction lifetimes #2485
• Maintain transaction lifetime on transactions #2484
• Attach a typed lifetime constraint field when we compile a transaction #2481 👈
• master

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

Join @mcintyre94 and the rest of your teammates on Graphite

[buffalojoec]

Looks like you might just be missing something little based on CI, but lgtm!

[lorisleiva]

I like that this mimics the way it's structured on the TransactionMessage (but with slightly less information needed at this stage).

[buffalojoec]

[github-actions]

🎉 This PR is included in version 1.91.5 🎉

The release is available on:

• npm package (@latest dist-tag)
• GitHub release

Your semantic-release bot 📦🚀

[github-actions]

Because there has been no activity on this PR for 14 days since it was merged, it has been automatically locked. Please open a new issue if it requires a follow up.