remix build hangs after build when no tty is available.

[tiagoad]

What version of Remix are you using?

1.9.0

Steps to Reproduce

Add a package.json script build that runs remix build, and another that runs npm-run-all -l build.

Expected Behavior

Build finishes as usual.

Actual Behavior

Build hangs after Built in 2.3s.

[tiagoad]

As a workaround, I changed the build script to script -qefc \"script -q /dev/null remix build\"

[cdeath]

remix build > /dev/null 2>&1 also works

[TrySound]

Reproducible with turborepo and pnpm run -r

[TrySound]

Remix build started to hang since https://github.com/remix-run/remix/releases/tag/remix%401.7.5. Probably in #4410

cc @pcattori

[TrySound]

Reproduced with pnpm run --parallel --filter=@webstudio-is/builder build

Found this incremental flag make build hang. With incremental: false all works.

remix/packages/remix-dev/compiler/compileBrowser.ts

Line 220 in 542c09a

incremental: true,

[TrySound]

@pcattori New esbuild version seems moved away from incremental flag so upgrade may solve the issue.

[pcattori]

Remix v1.16 shipped with more changes to compiler including upgrading to esbuild 0.17.6 and removing the incremental flag in favor of rebuilds via esbuild context. Can anyone here try things out with latest and see if this is still an issue?

[pcattori]

Tested on 1.18.1. Original issue seems to be resolved.

package.json scripts:

{
  "scripts": {
    "stuff": "npm-run-all -l build",
    "build": "remix build",
  }
}

Running npm run stuff finishes without hanging:

[pcattori]

@TrySound like I mentioned earlier, we've upgraded to esbuild 17 and are now using esbuild context (instead of incremental option) for incremental builds.

If issue reemerges, feel free to reopen.