-
Notifications
You must be signed in to change notification settings - Fork 206
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
4383 durable zoe #5879
4383 durable zoe #5879
Conversation
4a35fdd
to
4d4b514
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM!
* @param {ShutdownWithFailure | undefined} shutdownZoeVat | ||
*/ | ||
export const createInvitationKit = (shutdownZoeVat = undefined) => { | ||
const invitationKit = makeIssuerKit( | ||
export const createInvitationKit = (baggage, shutdownZoeVat = undefined) => { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
What do you think of a naming convention where functions that
- take a baggage first argument,
- call a
vivifySomething
- therefore need to be called during the first crank, if they might be revivifying
should themselves be called vivifySomething
?
export const createInvitationKit = (baggage, shutdownZoeVat = undefined) => { | |
export const vivifyInvitationKit = (baggage, shutdownZoeVat = undefined) => { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
sounds like a good rule.
4d4b514
to
917e1cd
Compare
refs: #4383
Description
First step of making Zoe reachably durable. This covers all of Zoe's global state. Remaining tasks are
I recommend reviewing this by commits. The commits separately update InvitationIssuer, escrow storage, installationStorage, and proposalSchema storage.
Security Considerations
Maintain Zoe invariants
Documentation Considerations
None.
Testing Considerations
Ensure existing tests continue to pass.