feat(iota_account): implementation of a Move package to represent a basic abstracted IOTA account #8250
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
|
The latest updates on your projects. Learn more about Vercel for GitHub. 6 Skipped Deployments
|
iota-ci
added
the
sc-platform
A verifier for validating `authenticate` functions introduced for `Move Authentication` (Account Abstraction). Contrary to other verifiers this shouldn't/can't be executed statically. This is why it doesn't appear among the list of other verifiers in: iota_verify_module_metered function. By design it has to be executed during runtime, as it is only executed for a single user specified function. The exact identity of this function will not be known until runtime.
Additionally all relevant components which lacked documentation have been updated.`
…l gas Otherwise we would need a new cut, which we don't want to have.
valeriyr
force-pushed
the
vm-lang/aa-auth/8238-basic-abstracted-IOTA-account
branch
from
502da43 to
bd23d71
Compare
valeriyr
changed the base branch from
vm-lang/aa-auth/8116-feature-branch
to
vm-lang/aa-auth/8117-create_auth_info
valeriyr
force-pushed
the
vm-lang/aa-auth/8238-basic-abstracted-IOTA-account
branch
from
d4dfee1 to
52761af
Compare
TheMrAI
force-pushed
the
vm-lang/aa-auth/8117-create_auth_info
branch
from
2a17efb to
6ac6db5
Compare
valeriyr
force-pushed
the
vm-lang/aa-auth/8238-basic-abstracted-IOTA-account
branch
from
52761af to
daaa870
Compare
valeriyr
requested review from
TheMrAI and
theiari
and removed request for
a team
miker83z
reviewed
Sep 9, 2025
theiari
reviewed
Sep 10, 2025
|
|
||
| /// Checks that the sender of this transaction is the account. | ||
| fun ensure_tx_sender_is_account(self: &IOTAccount, ctx: &TxContext) { | ||
| assert!(self.id.uid_to_address() == ctx.sender()); |
Contributor
There was a problem hiding this comment.
Probably I'm missing something, but I guess this assert is enough and we don't need any signature verification, right?
Contributor
There was a problem hiding this comment.
No, this is a utility function just to check the sender address, nothing more.
theiari
reviewed
Sep 10, 2025
theiari
reviewed
Sep 10, 2025
Contributor
theiari
left a comment
theiari
left a comment
There was a problem hiding this comment.
Looks good to me!
I only left two small comments that are more like personal doubts.
valeriyr
force-pushed
the
vm-lang/aa-auth/8238-basic-abstracted-IOTA-account
branch
2 times, most recently
from
a1b6c73 to
92d67ba
Compare
miker83z
approved these changes
Sep 10, 2025
With this the drop function was removed as well.
TheMrAI
force-pushed
the
vm-lang/aa-auth/8117-create_auth_info
branch
from
9f9daca to
458aa71
Compare
…asic abstracted IOTA account
valeriyr
force-pushed
the
vm-lang/aa-auth/8238-basic-abstracted-IOTA-account
branch
from
92d67ba to
aaa3a95
Compare
Base automatically changed from
vm-lang/aa-auth/8117-create_auth_info
to
vm-lang/aa-auth/8116-feature-branch
September 11, 2025 12:05
…th/8238-basic-abstracted-IOTA-account
valeriyr
merged commit 25f4b30
into
vm-lang/aa-auth/8116-feature-branch
33 of 36 checks passed
valeriyr
deleted the
vm-lang/aa-auth/8238-basic-abstracted-IOTA-account
branch
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Description of change
A new Move type has been introduced:
What can we do with the introduced account:
Links to any relevant issues
Fixes #8238 .