want volta list --format=json

[mcarifio]

Would be nice to have volta list --format=json with a json schema similar to npm's. I will sometimes use this format with jq or similar "pick out" useful information. Low priority item.

[chriskrycho]

Thanks for flagging it up – we'd like this as well, and it's actually probably a relatively easy lift, just (as you noted) a lower priority. If someone out there wants to implement it, however, it's actually far and away the easiest of the list --format=... variants to implement, because all it actually requires is using the existing logic for list and Serde definitions for the output types.

[saidsay-so]

I can take a look at it!

[charlespierce]

@musikid That would be great! I haven't dug in too deeply, but I imagine you can leverage serde_json to generate the actual JSON output. Let us know if you run into any issues or have questions!

[saidsay-so]

Hi!
I tried to implement this feature for a while now but I abandoned and forgot it because I faced a strange problem that I haven't been able to solve.
I tried in first to derive the Serialize trait but it failed. And when I tried to implement manually the trait, I get the same error.
command::list::_IMPL_SERIALIZE_FOR_Source::_serde::Serialize is not implemented for semver::version::Version
Do you have any clue about it ?

[charlespierce]

Hi @musikid! I think I understand that error: It looks like the Serialize trait, which is what serde needs to perform the serialization, isn't implemented for the Version struct, which is provided by semver. Unfortunately, due to Rust's orphaning rules, we can't just make our own with:

impl Serialize for Version {}

However, I do know that Version implements Display, as we use that often throughout the Volta code, so one solution would be for your data structure to use String everywhere you were previously using Version. Then when creating an instance, you can call .to_string() on every one of the Version values to convert it into a string. Serde knows how to handle Strings, so then it should be able to derive the Serialize trait correctly.

[saidsay-so]

What I forget to precise first is that Version actually implement the Serialize but I didn't understand why it gives that error. Nevermind, I just realized that the trait is just part of the serde feature and this was the reason why it didn't appear.

[saidsay-so]

Thanks @charlespierce for the help!

[chriskrycho]

Hey @musikid, how's this coming along? No worries if you're unable to finish it; just figured I'd check in!

[saidsay-so]

I almost finished it. I just need to write proper tests and it should be ready! Sorry for the time it took, I had some important things to deal with so I wasn't able to do it.

[chriskrycho]

@musikid no need to apologize, not even a little! Was just checking in as I was going through the open issues to see what their status is. Feel free to open a Draft PR and we can give you early feedback on it!