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

Bootstrap core: refactor, diagnostics, chain support #4387

Merged
merged 28 commits into from
Jan 26, 2022
Merged

Conversation

dckc
Copy link
Member

@dckc dckc commented Jan 26, 2022

refs: #4165

  • add @agoric/vats/decentral-demo-config.json
  • reorganise modules to separate behaviors and other purposes
  • show remaining consumers without a producer yet
  • throw exceptions when properties outside the permit list are accessed
  • provide support for Cosmos deployments (local-chain and production)
  • allow incremental addition to clientHome (such as by governance)

dckc and others added 27 commits January 25, 2022 21:01
probably crashes in sim-chain mode
 - factor out registerNetworkProtocols
 - factor out makeChainBundler
   - add vatParameters arg
   - import types for vats
     - avoid casting result of buildZoe
   - turn bootMsg example comment into machine-checked type
 - default attMakerFor to non-bridge case rather than undefined
 - fix array arg to buildDistributor
 - factor out createLocalBundle
   - add vatPowers arg
 - refactor: unroll setupCommandDevice
 - feat: don't await vatAdminSvc
   I can't imagine any need for it to be local.
 - docs: prune refs to python provisioning service
   Is there anything left that's worth saying about "scenario #1"?
 - docs: declare type of makeFeeCollector
   move function comment into JSDoc
some styling issues remain

 - toward authorityViz: read bootstrapManifest
 - feat: visualize vatPowers
 - feat: authority viz with distinctive styles
   with --sim-chain option
 - chore(vats): clean up debug logging
 - fix(vats): buildZoe was missing permit for nameAdmins
 - refactor: hoist wellKnownERights
 - factor out startTimerService
 - factor out bootEconomy.js
 - reserve installations, instances as well as issuers, brands
   in agoricNames
Connecting vattp / mailbox is more part of the SwingSet API.
Note conflict around OptionalBridgeManager.
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.

LGTM! I'm willing to accept responsibility. :)

@michaelfig michaelfig added the automerge:no-update (expert!) Automatically merge without updates label Jan 26, 2022
@mergify mergify bot merged commit 8a8335e into master Jan 26, 2022
@mergify mergify bot deleted the bootstrap-core branch January 26, 2022 03:36
mhofman added a commit to Agoric/testnet-load-generator that referenced this pull request Feb 4, 2022
Fix for BLD removed as collateral in Bootstrap change Agoric/agoric-sdk#4387
mhofman added a commit to Agoric/testnet-load-generator that referenced this pull request Feb 4, 2022
Fix for BLD removed as collateral in Bootstrap change Agoric/agoric-sdk#4387
mhofman added a commit to Agoric/testnet-load-generator that referenced this pull request Feb 5, 2022
Use USDC for compatibility with newer SDKs
Fix for BLD removed as collateral in Bootstrap change Agoric/agoric-sdk#4387
mhofman added a commit to Agoric/testnet-load-generator that referenced this pull request Feb 5, 2022
Use USDC for compatibility with newer SDKs
Fix for BLD removed as collateral in Bootstrap change Agoric/agoric-sdk#4387
mhofman added a commit to Agoric/testnet-load-generator that referenced this pull request Feb 5, 2022
Use USDC for compatibility with newer SDKs
Fix for BLD removed as collateral in Bootstrap change Agoric/agoric-sdk#4387
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
automerge:no-update (expert!) Automatically merge without updates
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants