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

#275 update to HDI and holochain 0.0.152 #340

Merged
merged 123 commits into from
Aug 13, 2022

Conversation

Connoropolous
Copy link
Contributor

@Connoropolous Connoropolous commented Jul 21, 2022

In case not using nix-shell, these are the package versions for this branch:
cargo install holochain --version 0.0.152
cargo install holochain_cli --version 0.0.50
cargo install lair_keystore --version 0.0.11

additionally you will need the following:

  • Node v16
  • pnpm
  • install the wasm32-unknown-unknown target: rustup target add wasm32-unknown-unknown

Current HDK: 0.0.144

closes #275

  • refactor bundles/dna_templates to have 'integrity' and 'coordinator' zomes
  • add updated dna-auth-resolver
  • set up nix-shell for holochain v0.0.150
  • update HDK versions to 0.0.142, instead of anywhere that it has 0.0.136 set, fix compilation issues
  • rename any reference from Header to Action
  • rename any reference from Element to Record
  • introduce entry enum type
  • separate out an integrity zome for each zomes
  • work those integrity zomes into the bundles/dna_templates
  • update @holochain/client (ask Connor about this)
  • define IndexSegment entry type and its method implementations in the integrity zome, or remove
  • define UUID entry type and wrapper enum in integrity, or remove (working with hashes only)
  • define separate link types in hdk_records, along with integrity zome crate (or define in link type in each object type integrity zome and pass in the type to hdk_records functions)
  • fix the integration tests

@Connoropolous
Copy link
Contributor Author

@pospi FYI @weswalla is back and they're taking over this from me

@Connoropolous
Copy link
Contributor Author

Connoropolous commented Jul 25, 2022

we've just landed the migration on the Acorn project, so have learned a few lessons along the way and feel prepared for this

@Connoropolous
Copy link
Contributor Author

Pushed the fix, that is letting us get the integration tests running

@weswalla
Copy link
Contributor

weswalla commented Aug 10, 2022

It seems like the remaining errors (there may be others I haven't investigated yet) are related to dna_auth_resolver, as the following error is consistent across tests:

Err(Wasm(WasmError { file: "/Users/weswalla/.cargo/git/checkouts/dna-auth-resolver-bb2f8d37890db247/2c9951a/core/src/lib.rs", line: 32, error: Guest("AvailableCapability does not map to any ZomeId and LinkType that is in scope for this zome.") }))])

which is related to https://github.com/holochain-open-dev/dna-auth-resolver/blob/2c9951ae5cc567b7560773d94ec6c34a0afe0758/core/src/lib.rs#L32

where the error in generated in holochain: https://github.com/holochain/holochain/blob/29f7cf536c030c5d79c6595f4bdbcb0c61c3b9d1/crates/hdk_derive/src/link_types.rs#L71

we had a bad assumption in play, which is that the relationship between the indexing integrity zomes could be one-to-many (one integrity zome, many indexing zomes), but that was bad. it meant that when calling get_links, links from other indexing zomes could be mixed in with the links for the intended indexing zome
@Connoropolous Connoropolous marked this pull request as ready for review August 11, 2022 16:54
@Connoropolous
Copy link
Contributor Author

merrrrrrrging

@Connoropolous Connoropolous merged commit 9750fc7 into sprout Aug 13, 2022
@Connoropolous Connoropolous deleted the feature/275-update-holochain-150 branch August 13, 2022 23:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
No open projects
Status: Done
Development

Successfully merging this pull request may close these issues.

After release of the first HDI (or integrity kit) of Holochain, update versions to that
3 participants