build: Run tsc --noEmit as prebuild for vite build

[samestep]

Description

Fixes #866. @wodeni and I made some progress toward this in #941 and #942, but #943 seems to have caused some regression, because Vite doesn't typecheck code:

Vite only performs transpilation on .ts files and does NOT perform type checking. It assumes type checking is taken care of by your IDE and build process (you can run tsc --noEmit in the build script or install vue-tsc and run vue-tsc --noEmit to also type check your *.vue files).

Thus, this PR adds tsc --noEmit as a prebuild script before every vite build script (except for @penrose/components, which seems to do something special and for which tsc --noEmit fails). This exposes a couple type errors caused by #906, which this PR also fixes.

Checklist

• I have commented my code, particularly in hard-to-understand areas
• My changes generate no new ESLint warnings
• I have reviewed any generated changes to the diagrams/ folder

[codecov]

Codecov Report

Merging #973 (9d16a87) into main (bfdbe43) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main     #973   +/-   ##
=======================================
  Coverage   63.41%   63.41%           
=======================================
  Files          62       62           
  Lines        7886     7886           
  Branches     1827     1827           
=======================================
  Hits         5001     5001           
  Misses       2766     2766           
  Partials      119      119           

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update bfdbe43...9d16a87. Read the comment docs.

[cloudflare-pages]

Deploying with    Cloudflare Pages

Latest commit:	9d16a87
Status:	✅  Deploy successful!
Preview URL:	https://5f073630.penrose-panes.pages.dev

View logs