nixpkgs review fails for me with:

       > sed: can't read node_modules/elmi-to-json/package.json: No such file or directory
       For full logs, run:
         nix log /nix/store/gk2n0fpqy8410siw4njh8iqjhlayxj3i-elm-verify-examples-6.0.3.drv
error: 1 dependencies of derivation '/nix/store/lkf8d6rbl8zd9d2i3p6f9m47b1q18mbb-review-shell.drv' failed to build

Link to currently reviewing PR:
https://github.com/NixOS/nixpkgs/pull/413234
--------- Report for 'x86_64-linux' ---------
1 package failed to build:
elmPackages.elm-verify-examples

I'm not sure if this is regression caused by this or not (yet).

Confirmed that this is not regression caused by changes in this PR. elm-verify-examples is failing in master too.

I think there is only elmi-to-json left there. Which might be a bit harder but it shouldn't be rocket science either.

Ah, I see elm-format was already moved. I also see that stoeffel/elmi-to-json#48 hasn’t moved at all, so maybe it’d be worth considering dropping elmi-to-json if possible…?

elmi-to-json is dependecy of other package and that's it sole purpose (similar to elm-instrument). So it needs to be decided based on if we want to keep these other packages or not.

The relevant packages seem to be elm-verify-examples (for elmi-to-json) and elm-coverage (for elmi-to-json and elm-instrument).

elm-verify-examples seems to have moved off of elmi-to-json a while ago: stoeffel/elm-verify-examples#99

elm-coverage doesn't seem to be maintained, but seems to be on the same versions of elm and elm-test that elm-verify-examples is on after that PR: https://github.com/zwilias/elm-coverage/blob/main/package.json

I can't find any other references to elmi-to-json in nixpkgs - so maybe we can just remove the references to it from those two preRebuild hooks without any further packages needed to drop?

That seems correct. Elm-coverage is something that @domenkozar requested and I packaged myself if I'm correct. I don't see it as an important package so I think we can consider dropping it if keeping it will be proven difficult.

While elm-coverage specifies recent enough elm and elm-test, somehow the generate-node-packages.sh script doesn't resolve to those versions for it, but instead to much older versions. I have no idea why that happens. But this causes elmi-to-json to stick around, even after updating the node packages.

I don't see it as an important package so I think we can consider dropping it if keeping it will be proven difficult.

Since I didn't succeed with keeping it, here's my attempt to drop it: #415911.