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

feat: add UpdateValidator and CancelAutoStake invitations to auto-stake-it #9786

Draft
wants to merge 4 commits into
base: master
Choose a base branch
from

Conversation

0xpatrickdev
Copy link
Member

@0xpatrickdev 0xpatrickdev commented Jul 26, 2024

closes: #XXXX
refs: #9042

Description

While the title includes the primary changes, this was also a side-quest to explore patterns for object composition with exos. Here, an opts bag is added to preparePortfolioHolder that allows the caller to supply extra invitation methods. This pattern seems useful as it allows developers to easily add new functionality to an existing kit while ensuring safety-critical invitations (i.e. Transfer or Withdraw funds) are still available to the end wallet user.

To Do

  • type support for dynamic guards + methods in an exo class kit
  • bootstrap test needed?

Security Considerations

Scaling Considerations

Documentation Considerations

Testing Considerations

Upgrade Considerations

0xpatrickdev and others added 4 commits July 25, 2024 17:45
- adds an example of extending a 'prepareSomeHolderKit' with extra invitation makers
- adds opts?: { extraInvitationMakerGuards, extraInvitationMakerMethods, extraStateShape } as third argument to facilitate this

Co-authored-by: Mark S. Miller <[email protected]>
Copy link

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

Deploying agoric-sdk with  Cloudflare Pages  Cloudflare Pages

Latest commit: e6adef2
Status: ✅  Deploy successful!
Preview URL: https://2729aea9.agoric-sdk.pages.dev
Branch Preview URL: https://pc-portfolio-holder-kit-exte.agoric-sdk.pages.dev

View logs

Copy link

codecov bot commented Jul 26, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 56.37%. Comparing base (0c96ea5) to head (e6adef2).
Report is 3 commits behind head on master.

Additional details and impacted files
Components Coverage Δ
SwingSet/kernel 73.36% <ø> (ø)
ERTP 92.45% <ø> (ø)
Orchestration 94.48% <100.00%> (+0.04%) ⬆️
swing-store 95.95% <ø> (ø)
Files Coverage Δ
...ges/orchestration/src/exos/portfolio-holder-kit.js 99.05% <100.00%> (+0.20%) ⬆️
packages/orchestration/tools/ibc-mocks.ts 100.00% <100.00%> (ø)

... and 9 files with indirect coverage changes

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant