feat(dev): enable the usage of the command property without port

[lukasholzer]

🎉 Thanks for submitting a pull request! 🎉

Summary

Fixes #3654

---

For us to review and ship your PR efficiently, please perform the following steps:

• Open a bug/issue before writing your code 🧑‍💻. This ensures we can discuss the changes and get feedback from everyone that should be involved. If you`re fixing a typo or something that`s on fire 🔥 (e.g. incident related), you can skip this step.
• Read the contribution guidelines 📖. This ensures your code follows our style guide and
  passes our tests.
• Update or add tests (if any source code was changed or added) 🧪
• Update or add documentation (if features were changed or added) 📝
• Make sure the status checks below are successful ✅

A picture of a cute animal (not mandatory, but encouraged)

[github-actions]

📊 Benchmark results

Comparing with 51bde9a

Package size: 365 MB

⬇️ 0.00% decrease vs. 51bde9a

^  364 MB  364 MB  364 MB  364 MB  364 MB  364 MB  364 MB  364 MB  364 MB  365 MB  365 MB  365 MB  365 MB 
│   ┌──┐    ┌──┐    ┌──┐    ┌──┐    ┌──┐    ┌──┐    ┌──┐    ┌──┐    ┌──┐    ┌──┐    ┌──┐    ┌──┐    ┌──┐  
│   |  |    |  |    |  |    |  |    |  |    |  |    |  |    |  |    |  |    |  |    |  |    |  |    |▒▒|  
│   |  |    |  |    |  |    |  |    |  |    |  |    |  |    |  |    |  |    |  |    |  |    |  |    |▒▒|  
│   |  |    |  |    |  |    |  |    |  |    |  |    |  |    |  |    |  |    |  |    |  |    |  |    |▒▒|  
│   |  |    |  |    |  |    |  |    |  |    |  |    |  |    |  |    |  |    |  |    |  |    |  |    |▒▒|  
│   |  |    |  |    |  |    |  |    |  |    |  |    |  |    |  |    |  |    |  |    |  |    |  |    |▒▒|  
│   |  |    |  |    |  |    |  |    |  |    |  |    |  |    |  |    |  |    |  |    |  |    |  |    |▒▒|  
│   |  |    |  |    |  |    |  |    |  |    |  |    |  |    |  |    |  |    |  |    |  |    |  |    |▒▒|  
│   |  |    |  |    |  |    |  |    |  |    |  |    |  |    |  |    |  |    |  |    |  |    |  |    |▒▒|  
│   |  |    |  |    |  |    |  |    |  |    |  |    |  |    |  |    |  |    |  |    |  |    |  |    |▒▒|  
│   |  |    |  |    |  |    |  |    |  |    |  |    |  |    |  |    |  |    |  |    |  |    |  |    |▒▒|  
│   |  |    |  |    |  |    |  |    |  |    |  |    |  |    |  |    |  |    |  |    |  |    |  |    |▒▒|  
│   |  |    |  |    |  |    |  |    |  |    |  |    |  |    |  |    |  |    |  |    |  |    |  |    |▒▒|  
│   |  |    |  |    |  |    |  |    |  |    |  |    |  |    |  |    |  |    |  |    |  |    |  |    |▒▒|  
│   |  |    |  |    |  |    |  |    |  |    |  |    |  |    |  |    |  |    |  |    |  |    |  |    |▒▒|  
│   |  |    |  |    |  |    |  |    |  |    |  |    |  |    |  |    |  |    |  |    |  |    |  |    |▒▒|  
│   |  |    |  |    |  |    |  |    |  |    |  |    |  |    |  |    |  |    |  |    |  |    |  |    |▒▒|  
│   |  |    |  |    |  |    |  |    |  |    |  |    |  |    |  |    |  |    |  |    |  |    |  |    |▒▒|  
│   |  |    |  |    |  |    |  |    |  |    |  |    |  |    |  |    |  |    |  |    |  |    |  |    |▒▒|  
│   |  |    |  |    |  |    |  |    |  |    |  |    |  |    |  |    |  |    |  |    |  |    |  |    |▒▒|  
│   |  |    |  |    |  |    |  |    |  |    |  |    |  |    |  |    |  |    |  |    |  |    |  |    |▒▒|  
└───┴──┴────┴──┴────┴──┴────┴──┴────┴──┴────┴──┴────┴──┴────┴──┴────┴──┴────┴──┴────┴──┴────┴──┴────┴──┴──>
    T-12    T-11    T-10    T-9     T-8     T-7     T-6     T-5     T-4     T-3     T-2     T-1      T    

Legend

• T-30 (51bde9a): 365 MB
• T-29 (53643b4): 365 MB
• T-28 (bb09481): 365 MB
• T-27 (f7ca0d1): 365 MB
• T-26 (65cf7cc): 365 MB
• T-25 (e2ef338): 365 MB
• T-24 (134e441): 365 MB
• T-23 (4357279): 365 MB
• T-22 (2effcdb): 364 MB
• T-21 (0fcf5ec): 364 MB
• T-20 (e87d1e1): 364 MB
• T-19 (eaeced2): 364 MB
• T-18 (b2f77f3): 364 MB
• T-17 (e47ded9): 364 MB
• T-16 (fc3cecb): 364 MB
• T-15 (8ad90d0): 364 MB
• T-14 (cf1d3f1): 364 MB
• T-13 (e690ee5): 364 MB
• T-12 (a894ef4): 364 MB
• T-11 (86ffb8a): 364 MB
• T-10 (5f59329): 364 MB
• T-9 (4b49ca8): 364 MB
• T-8 (618d997): 364 MB
• T-7 (5272de7): 364 MB
• T-6 (86931c6): 364 MB
• T-5 (a875fb7): 364 MB
• T-4 (05fe45a): 364 MB
• T-3 (089d65e): 365 MB
• T-2 (d769ed2): 365 MB
• T-1 (86f4b8b): 365 MB
• T (current commit): 365 MB

[ascorbic]

This is looking good so far, and works for me locally with Remix. The behaviour is a significant change though, so you should ensure that you test various scenarios to avoid breaking existing sites.

[ascorbic]

This will probably need a docs update, explaing that if targetPort is provided then it will wait for that port to open, and proxy to it, and if it's not provided then the static server will be used.

[lukasholzer]

This will probably need a docs update, explaing that if targetPort is provided then it will wait for that port to open, and proxy to it, and if it's not provided then the static server will be used.

I think it should be updated here as well: https://docs.netlify.com/configure-builds/file-based-configuration/#netlify-dev

Hey as the docs in the repo are "autogenerated" <!-- AUTO-GENERATED-CONTENT:START (GENERATE_COMMANDS_DOCS) --> – can you point me in the direction on where to adapt it?

[ascorbic]

What's needed to land this? It would be great to get this out, now that Remix has been released

[lukasholzer]

@ascorbic I'm opening up a PR on the docs site as well to change it but IMHO we can release it first ;) From my point, it should be good to go but I'm the PR issuer so maybe @ehmicky can provide here more information.

The doc's site does not say that this is not working what we implemented so we only need to explain this use case a little bit more there:

https://docs.netlify.com/configure-builds/file-based-configuration/#netlify-dev

[ascorbic]

Great! Could you complete the PR description now it's ready for review?