feat: Show edit graph link on dev startup, send operations library edits to UI, and run prettier (if available) on code output

[sgrove]

🎉 Thanks for submitting a pull request! 🎉

Summary

Fixes #4180

Makes editing the Graph operations library file locally work while also sending updates to the ui to make the DX much smoother via file monitoring.

It also runs prettier (if available) on the output so the code conforms to the user's local expectations.

Shows a link when starting netlify dev --graph that takes the user to the graph editing UI (requested by @biilmann)

---

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.
• Make sure the status checks below are successful ✅

[ehmicky]

Small note: if filePath has a space in its path (which is common on Windows, e.g. C:\Program Files), it will be interpreted as two different arguments by execa.command().
Instead, using execa('prettier', ['--write', filePath]) will prevent this problem.

[sgrove]

Oh, great note, I'll fix that up tomorrow. Thank you!

…

On Sat, Feb 5, 2022 at 8:05 AM ehmicky ***@***.***> wrote: ***@***.**** commented on this pull request. ------------------------------ In src/lib/one-graph/cli-netlify-graph.js <#4183 (comment)>: > @@ -241,6 +241,31 @@ const ensureFunctionsPath = (netlifyGraphConfig) => { fs.mkdirSync(fullPath, { recursive: true }) } +let disablePrettierDueToPreviousError = false + +const runPrettier = async (filePath) => { + if (disablePrettierDueToPreviousError) { + return + } + + const command = `prettier --write ${filePath}` + try { + const commandProcess = execa.command(command, { Small note: if filePath has a space in its path (which is common on Windows, e.g. C:\Program Files), it will be interpreted as two different arguments by execa.command(). Instead, using execa('prettier', ['--write', filePath]) will prevent this problem. — Reply to this email directly, view it on GitHub <#4183 (review)>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/AAAITYCGE5MSYO22M5CVO4DUZVDELANCNFSM5NTBVPNQ> . You are receiving this because you were assigned.Message ID: ***@***.***>