| 13 Nov 2023 |
 Lily Foster | what the hell? | 17:39:15 |
| Lily Foster | oh we actually remove that | 17:39:50 |
| Lily Foster | in the hook | 17:39:51 |
| Lily Foster | the bugs are coming from inside the nixpkgs | 17:39:56 |
| Lily Foster | okay | 17:39:57 |
| Lily Foster | well give me a bit and i'll be able to have a "fix" for that | 17:40:10 |
| Lily Foster | (it is probably bug on our end, but npm still shouldn't be trying to re-reify stuff regardless when pruning) | 17:40:25 |
| Lily Foster | * (it is probably partly a bug on our end, but npm still shouldn't be trying to re-reify stuff regardless when pruning) | 17:40:35 |
| Lily Foster | teto: can you try from github:lilyinstarlight/nixpkgs/fix/npm-pack-shenanigans and see if that works without compromising prune? | 17:44:51 |
 @keiichi:matrix.org | awesome. There is no hurry on my side since dontNpmPrune works well. Now there is one question I've asked on the discourse without any answer is how people work with typescript on nix ? I had solved it with dream2nix but with the rewrite I stopped using it and I would like to use buildNpmPackage since it works reasonably well. Usually in my shellHook, I export NODE_PATH=${drvBuiltBy_buildNpmPackage}/lib/node_modules but typescript ignores NODE_PATH I believe so I end with TSError: ⨯ Unable to compile TypeScript: and index.ts(1,25): error TS2307: Cannot find module '@pulumi/pulumi' or its corresponding type declarations. | 17:45:42 |
| @keiichi:matrix.org | I'll have to merge that in my own nixpkgs. Gotta do some shopping before it closes than I try ty ! | 17:46:29 |
| Lily Foster | In reply to @keiichi:matrix.org
awesome. There is no hurry on my side since dontNpmPrune works well. Now there is one question I've asked on the discourse without any answer is how people work with typescript on nix ? I had solved it with dream2nix but with the rewrite I stopped using it and I would like to use buildNpmPackage since it works reasonably well. Usually in my shellHook, I export NODE_PATH=${drvBuiltBy_buildNpmPackage}/lib/node_modules but typescript ignores NODE_PATH I believe so I end with TSError: ⨯ Unable to compile TypeScript: and index.ts(1,25): error TS2307: Cannot find module '@pulumi/pulumi' or its corresponding type declarations. It should "just work" for the most part. Is there a reproducer on the discourse post? (feel free to tag me on those btw, i don't watch discourse super closely) | 17:46:33 |
| Lily Foster | In reply to @keiichi:matrix.org
I'll have to merge that in my own nixpkgs. Gotta do some shopping before it closes than I try ty ! okay, sounds good! let me know when you try it | 17:46:49 |
| Lily Foster | (i did just try it with the derivation and package{,-lock}.json files you posted above and it does seem to work with the above patch. i will PR it imminently but i'm gonna see if i can convince someone to merge https://github.com/NixOS/nixpkgs/pull/254420 first so i don't have to worry about merge conflicts since they both modify the same line) | 18:01:35 |
| @keiichi:matrix.org | Lily Foster: I've just tested it and it fixed it \o/ | 20:21:28 |
| @keiichi:matrix.org | just found my initial post, still valid https://discourse.nixos.org/t/help-typescript-to-find-modules/26851 | 21:06:33 |
 @ambroisie:belanyi.fr | Hey 👋
Does anyone here with more knowledge about our JS build helpers know why I need to do the whole dance with substituteAll in this MR, or rather if there's a better fix?
The vite.config.ts for reference, which sets up a bunch of file copies which fail if I don't fix the paths | 22:48:59 |
| Lily Foster | In reply to @ambroisie:belanyi.fr
Hey 👋
Does anyone here with more knowledge about our JS build helpers know why I need to do the whole dance with substituteAll in this MR, or rather if there's a better fix?
The vite.config.ts for reference, which sets up a bunch of file copies which fail if I don't fix the paths yarn2nix does weird stuff but idk why exactly you're needing that here. i doubt there's a better fix than that though | 22:51:05 |
| Lily Foster | (i still want to yeet yarn2nix into the abyss at some point once we have mature and modern yarn tooling) | 22:51:34 |
| @ambroisie:belanyi.fr | I'm just waiting for buildPnpmPackage for this specific package, we're really in a https://xkcd.com/2347/ there for now | 22:52:32 |
| @ambroisie:belanyi.fr | Anyways, thanks for the confirmation 👍️ | 22:53:00 |
| Lily Foster | i'm getting motivation to make it since the more i look at pnpm, the less hellish it seems | 22:53:10 |
| @ambroisie:belanyi.fr | * I'm just waiting for buildPnpmPackage for this specific package, we're really in a https://xkcd.com/2347/ situation there for now | 22:53:14 |
| Lily Foster | which the same cannot have been said for npm | 22:53:15 |
| @ambroisie:belanyi.fr | I've been out of cycles lately to properly shave the yak, but yeah recent discussions on that issue have been very promising | 22:53:55 |
| Lily Foster | yeah i feel the low-on-cycles bit | 22:55:01 |
| Lily Foster | (i've been able to surprisingly do some stuff of late, but i haven't been able to choose what i am capable of working on and it's mostly been to distract from life stuff 🙈) | 22:55:24 |
| @ambroisie:belanyi.fr | In reply to @lily:lily.flowers
(i still want to yeet yarn2nix into the abyss at some point once we have mature and modern yarn tooling) What's the most proper way to do yarn packaging at the moment? I thought the last part of yarn2nix was the fixup_yarn_lock (or is it the one with dashes?) that's in a bunch of packages | 22:56:28 |
| * @ambroisie:belanyi.fr In reply to @lily:lily.flowers
(i've been able to surprisingly do some stuff of late, but i haven't been able to choose what i am capable of working on and it's mostly been to distract from life stuff 🙈) sends internet accolades | 22:56:57 |
| Lily Foster | In reply to @ambroisie:belanyi.fr
What's the most proper way to do yarn packaging at the moment? I thought the last part of yarn2nix was the fixup_yarn_lock (or is it the one with dashes?) that's in a bunch of packages well i finally got the new fixup stuff in, so someone just needs to dust off https://github.com/NixOS/nixpkgs/pull/210814 now | 22:58:40 |
