Guillaume Sabran

Is it `beta.3` or `beta.4` ? You said it's .3: https://github.com/gadicc/meteor-hmr/issues/51#issuecomment-220356254 but here it's .4 https://github.com/gadicc/meteor-hmr/blob/master/packages/ecmascript-hot/package.js#L5 (and as mentioned by @jbbr I can't install .4 either)

with `beta.3` I'm getting ``` => Errors prevented startup: While loading plugin `compile-ecmascript-hot` from package `gadicc:ecmascript-hot`: error: Can't load npm module 'ws' from packages/gadicc_hot-build.js. Check your Npm.depends(). packages/gadicc_hot-build/accelerator.js:53:1: undefined is...

Also related: https://github.com/Flight-School/AnyCodable/pull/76, but this PR should do a better job at preventing future regressions

Bummer! It would be great to have similar support for MacOS

Do you think there should be a few integration tests to help Gemini CLI developers who likely don't think about the ACP integration not break it? Something like https://github.com/zed-industries/gemini-cli/pull/2

Sounds good I'll follow up. Let's keep this fix small

@mergesort could you create a PR? I was talking with @jamesrochabrun. Your change overall looks great, but we were hesitant to migrate everyone off URLSession that for all its issues...

In addition to feedback about the goal of this change, and its implementation, I was curious if there already exist: - snapshot testing of generated code - tests that use...

Thank you. Yes I remember reading something about why it was rejected in the past (but I could not find a pointer for this PR). My recollection was that this...

Thanks, all those points make sense. I'll defer to you and the team on what you think are the best trade offs here. Maybe worth noting: it is possible to...