cljs-dev

dnolen 2023-09-29T13:24:42.140659Z

@thheller yes I saw all kinds of weird stuff, but really we only need this for top-levels anyway - happy to fix stuff as people report them, or for people to submit patches

dnolen 2023-09-29T13:25:03.625889Z

the logic was pretty inscrutable before, but I think it’s now pretty easy to add these extra cases and write the unit tests

dnolen 2023-09-29T00:37:54.703819Z

Yes tho you can probably version detect and macroize

thheller 2023-09-29T05:36:26.216789Z

@dnolen since I implemented full support for "exports" recently in shadow-cljs, all these styles are surprisingly common https://github.com/thheller/shadow-cljs/blob/master/test-env/node_modules/exports/package.json#L9-L12

😧 2
thheller 2023-09-29T05:37:39.204049Z

I thought I could get away with not implementing wildcards, or the prefix style, but they are very widely used

thheller 2023-09-29T05:38:36.939229Z

also this abomination https://unpkg.com/@babel/runtime@7.22.5/package.json, although this is not that common

😭 1