move: external resolver reads multiple lock contents

[rvantonder]

Description

This makes changes to the external resolver so that multiple package graphs are read per resolver (null-separated strings). This also makes it so that an externally resolved dependency entry does not actually have to correspond to the package name. E.g., from #19561, A can be any name when r.foo is specified:

A = { r.foo = "bar", <other args if you want here> }

Test plan

Added a test to exercise parsing multiple null-separated lock contents. This also tests that it is not required for the dependency to be named as a recognized package.

---

Release notes

Check each box that your changes affect. If none of the boxes relate to your changes, release notes aren't required.

For each box you select, include information after the relevant heading that describes the impact of your changes that a user might notice and any actions they must take to implement updates.

• Protocol:
• Nodes (Validators and Full nodes):
• Indexer:
• JSON-RPC:
• GraphQL:
• CLI: Changes to external dependency resolution: allows processing multiple null-separated contents to extend the package graph per resolution call.
• Rust SDK:
• REST API:

[vercel]

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name	Status	Preview	Comments	Updated (UTC)
sui-docs	✅ Ready (Inspect)	Visit Preview	💬 Add feedback	Sep 26, 2024 7:51pm

3 Skipped Deployments

Name	Status	Preview	Comments	Updated (UTC)
multisig-toolkit	⬜️ Ignored (Inspect)	Visit Preview		Sep 26, 2024 7:51pm
sui-kiosk	⬜️ Ignored (Inspect)	Visit Preview		Sep 26, 2024 7:51pm
sui-typescript-docs	⬜️ Ignored (Inspect)	Visit Preview		Sep 26, 2024 7:51pm

[tzakian]

[package]
name = "name"
edition = "2024.beta"

[dependencies]
foo = { r.mvr = "foo" }
bar = { r.mvr = "bar" }

With configs

[package]
name = "name"
edition = "2024.beta"

[dependencies]
foo = { r.mvr = "foo" }
bar = { r.mvr = "bar" }

[resolver.mvr]
network = "testnet"

[tnowacki]

Maybe

[package]
name = "name"
edition = "2024.beta"

[dependencies]
foo = { r.mvr = "foo" }
bar = { r.mvr = "bar" }

[r.mvr]
network = "testnet"

[amnn]

[package]
name = "name"
edition = "2024.beta"

[dependencies.mvr.packages]
foo = "foo" 
bar = "bar"

[resolver.mvr]
network = "testnet"

[amnn]

I think this is good to go, although you'll need to coordinate with @tzakian on the manifest parser changes.

Note the comment I left about the restriction that a lock file from an external resolver only containing one edge from the root -- this wasn't an intended restriction/expectation, but given that it may be awkward to lift it for now, I think it's okay to keep, just leave a comment with the context there.

[rvantonder]

Thanks @amnn! Updated and

• Test uses new resolver syntax based on [move] Update syntax and usage for external resolvers in Move packages #19561
• Checked that this still works with external resolvers
• Left TODOs for optimization / generalization

Going ahead with merge