Allow `diff-closures` to output `json`

[Pamplemousse]

[edolstra]

What does the JSON output look like?

[Pamplemousse]

Here's an excerpt of what I can get with outputs/out/bin/nix --extra-experimental-features nix-command store diff-closures /run/current-system /nix/store/5ks2pk9n0lwmck32552ysjf1d7662pmv-nixos-system-foo-22.05.20221023.471d921

{
  "bash-interactive": {
    "after": "∅",
    "before": "∅",
    "sizeDelta": "+6384.3"
  },
  "bind": {
    "after": "9.18.7",
    "before": "9.18.3",
    "sizeDelta": "+44.8"
  },
  "blas": {
    "after": "∅",
    "before": "3",
    "sizeDelta": "-56241.8"
  },
  "chrootenv": {
    "after": "ε",
    "before": "∅",
    "sizeDelta": "+17.5"
  },
 ...
}

[SuperSandro2000]

    "after": "ε",
    "before": "∅",

Can we use null, empty string or something that everyone knows how to type on their keyboard?

[edolstra]

Yeah, using null sounds good.

sizeDelta should just be a number, not a string, and should be the delta in bytes.

before and after should probably be versionBefore and versionAfter, to make clear what it's referring to.

It would be nice to include the before/after store paths that are being compared.

[Pamplemousse]

Can we use null, empty string or something that everyone knows how to type on their keyboard?

I favored consistency and kept the symbols in use in the "regular" display.

[Pamplemousse]

It would be nice to include the before/after store paths that are being compared.

I was just looking for a mean to exploit the data that is currently being rendered by diff-closures automatically, not necessarily try to "better" the output of the command.

[Ericson2314]

It would be cool to finish this! Nice to have JSON for all commands.

[Pamplemousse]

It would be cool to finish this!

I addressed the comments from the PR, either by updating the code, or by explaining why the display makes use of certain symbols.

I didn't get any extra feedback here, and have asked for some on Matrix months ago without any replies. As far as I can tell, this is "finished".

[Ericson2314]

I enqueued this on the project board to try to get some feedback from the rest of the team.

[nixos-discourse]

This pull request has been mentioned on NixOS Discourse. There might be relevant details there:

https://discourse.nixos.org/t/2023-07-24-nix-team-meeting-minutes-75/31112/1

[bew]

When the derivation name does not strictly follow the name-version format, diff-closure usually displays wrong derivation name (or ""). For exemple when making custom intermediate derivations for my configs, adding a version doesn't really make sense, and adding a -0 or -unversioned seems like a workaround for nice display.

To help downstream users of this json, have you considered adding the Nix store paths for before/after in addition to only the version?

That would also allow doing more than just display the before/after versions and size delta, like using external diff tools on the paths to get more diff information.