Error: invalid range description

[kermage]

Running pkgx +yazi +lazygit produces the error but swapping the order (pkgx +lazygit +yazi) does not.

Also weird enough, this only happens on v2 and not on v1.

eval $(pkgx +yazi +lazygit)
eval $(pkgx +lazygit +yazi)
eval $(pkgx pkgx@1 +yazi +lazygit)
eval $(pkgx pkgx@1 +lazygit +yazi)

[jhheider]

the order-dependence is very odd. comparing the output of:

$ pkgx --json=v2 +lazygit | pkgx jq .env.LIBRARY_PATH | sort
...
$ pkgx --json=v2 +yazi | pkgx jq .env.LIBRARY_PATH | sort
...

shows lazygit has nearly a perfect super-set of yazi deps. it looks like somewhere along the line we're triggering one of these two lines:

• parsing a range that doesn't begin with [^~<@=]: https://github.com/jhheider/semverator/blob/be09dde40a415c9dd4d604036451acd5aa2197fe/lib/src/range/parse.rs#L151
• parsing a range that doesn't match CONSTANT_REGEX_SIMPLE => Regex::new(r"^>=((\d+\.)*\d+)\s*(<((\d+\.)*\d+))?$"): https://github.com/jhheider/semverator/blob/be09dde40a415c9dd4d604036451acd5aa2197fe/lib/src/range/parse.rs#L154

again, the fact that it's order-dependent is probably the most interesting part. i'll try and hunt this down and see what we're doing here.

[jhheider]

yeah, the backtrace shows it's an intersection (which is the obvious guess). order might matter there, a little.

[jhheider]

the bad run is:

intersecting constraints: Range { raw: ">=5.0.0<Infinity.Infinity.Infinity", set: [Contiguous(Semver { components: [5], major: 5, minor: 0, patch: 0, prerelease: [], build: [], raw: "5.0.0" }, Semver { components: [18446744073709551615, 18446744073709551615, 18446744073709551615], major: 18446744073709551615, minor: 18446744073709551615, patch: 18446744073709551615, prerelease: [], build: [], raw: "Infinity.Infinity.Infinity" })] } and Range { raw: "*", set: [Any] }

versus

intersecting constraints: Range { raw: "*", set: [Any] } and Range { raw: ">=5", set: [Contiguous(Semver { components: [5], major: 5, minor: 0, patch: 0, prerelease: [], build: [], raw: "5.0.0" }, Semver { components: [18446744073709551615, 18446744073709551615, 18446744073709551615], major: 18446744073709551615, minor: 18446744073709551615, patch: 18446744073709551615, prerelease: [], build: [], raw: "Infinity.Infinity.Infinity" })] }

which seems to be a correct construction. so, I suspect something is being double-converted, and the output isn't input. should be easy to trap and fix on the libsemverator side.

[jhheider]

fix pushed to semverator; just need to update the dep in pkgx.