Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

offer result vows in smart wallet #9635

Merged
merged 9 commits into from
Jul 5, 2024
Merged

offer result vows in smart wallet #9635

merged 9 commits into from
Jul 5, 2024

Conversation

turadg
Copy link
Member

@turadg turadg commented Jul 2, 2024

closes: #9308

Description

Makes the offer result watcher able to handle offer results that are vows or promises for vows.

Security Considerations

none

Scaling Considerations

none

Documentation Considerations

none

Testing Considerations

adds a test for a vow resolving after its source vat restarts

Upgrade Considerations

This requires an upgrade of the smart-wallet contract, which will have to be deployed in a core-eval.

adds a smart wallet contract upgrade test

Copy link

cloudflare-workers-and-pages bot commented Jul 2, 2024

Deploying agoric-sdk with  Cloudflare Pages  Cloudflare Pages

Latest commit: 5439ea8
Status: ✅  Deploy successful!
Preview URL: https://87b385a2.agoric-sdk.pages.dev
Branch Preview URL: https://9308-smart-wallet-vows.agoric-sdk.pages.dev

View logs

@turadg turadg force-pushed the 9308-smart-wallet-vows branch 2 times, most recently from e23962c to 46b8881 Compare July 2, 2024 14:40
Comment on lines +287 to +288
// throw so the vow watcher propagates the rejection
throw reason;
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Curious how rejections were propagated before - this change seems unrelated to the rest

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

the function returned the promise, but now that's wrapped in vow watchers

@turadg turadg force-pushed the 9308-smart-wallet-vows branch from 46b8881 to 8f46beb Compare July 2, 2024 21:45
@turadg turadg added the force:integration Force integration tests to run on PR label Jul 2, 2024
@turadg turadg force-pushed the 9308-smart-wallet-vows branch 4 times, most recently from 10667e2 to eaf225f Compare July 4, 2024 01:16
@turadg turadg requested a review from michaelfig July 4, 2024 01:17
@turadg turadg marked this pull request as ready for review July 4, 2024 01:17
Copy link
Member

@michaelfig michaelfig left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please address my question about the commented-out block. Otherwise, LGTM!

Comment on lines 62 to 66
// FIXME fails if valueVow is restarted
// t.log('restart valueVow');
// await evalProposal(
// buildProposal('@agoric/builders/scripts/testing/restart-valueVow.js'),
// );
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Did you intend to leave this commented out?

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Oops, bad rebase. This must be tested. Will fixup

@turadg turadg force-pushed the 9308-smart-wallet-vows branch from eaf225f to 5439ea8 Compare July 5, 2024 15:41
@turadg turadg added the automerge:rebase Automatically rebase updates, then merge label Jul 5, 2024
@mergify mergify bot merged commit 115adc4 into master Jul 5, 2024
81 of 88 checks passed
@mergify mergify bot deleted the 9308-smart-wallet-vows branch July 5, 2024 16:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
automerge:rebase Automatically rebase updates, then merge force:integration Force integration tests to run on PR
Projects
None yet
Development

Successfully merging this pull request may close these issues.

add support for Vows to smart-wallet
3 participants