ci: add builds to test workflows

[erickzhao]

Adds yarn make to our test + post-merge flows.

Did a bit of cleanup for our workflows in general:

• Aligned .nvmrc with the version we use for releasing/testing.
• Updated @types/node to latest 22.

Note that these artifacts are unsigned.

[socket-security]

Review the following changes in direct dependencies. Learn more about Socket for GitHub.

Diff	Package	Supply Chain Security	Vulnerability	Quality	Maintenance	License
	npm/​@​types/​node@​22.10.10 ⏵ 22.19.1	+1		+1	+1

View full report

[coveralls]

coverage: 79.531% (-0.01%) from 79.543%
when pulling b4e0048 on build-workflow
into 5d25716 on main.

[dsanders11]

Discussed on Slack refactoring this to avoid a double build situation for the release workflow.

[erickzhao]

Added a universal build as well here!

[dsanders11]

Two things:

• Is there much benefit to adding a universal macOS build this late in the game? Not sure it's worth introducing, TBH
• Not sure we want to upload the artifacts for these builds. The artifacts will total several gigabyte for each run given all the variants, that's going to eat a lot of space quickly.

[erickzhao]

Is there much benefit to adding a universal macOS build this late in the game? Not sure it's worth introducing, TBH

@dsanders11 this stems from a previous Slack discussion we had about the Fiddle download page. Browser support for architecture detection is spotty and intrusive for users, so providing a single universal target is the easiest way of shipping the correct architecture for all users.

Looking at the download stats, we're still on 2/3 arm64 vs 1/3 x64 downloads for macOS.

Not sure we want to upload the artifacts for these builds. The artifacts will total several gigabyte for each run given all the variants, that's going to eat a lot of space quickly.

Good point. Looks like the free GHA limits are relatively low: https://docs.github.com/en/billing/concepts/product-billing/github-actions#free-use-of-github-actions

[erickzhao]

Had a brief discussion with @dsanders11. Going to add Universal in a follow-up if we choose to add the target.