From 2ef46afb48b8ac868d6cbde148bd8d9b314811d4 Mon Sep 17 00:00:00 2001
From: Mike Hardy <github@mikehardy.net>
Date: Sat, 25 Jan 2025 19:57:30 -0500
Subject: [PATCH] build: specify the canvas transitive node dependency build
 failure workaround in future-compatible way (#3752)

* fix: specify canvas npm optional dep ignore workaround more cleanly

this should allow it to work with yarn classic and yarn modern

* chore: add self to CONTRIBUTORS
---
 CONTRIBUTORS | 1 +
 package.json | 3 +--
 2 files changed, 2 insertions(+), 2 deletions(-)

diff --git a/CONTRIBUTORS b/CONTRIBUTORS
index e858d68ad..45c454c9c 100644
--- a/CONTRIBUTORS
+++ b/CONTRIBUTORS
@@ -211,6 +211,7 @@ David Elizalde <david.elizalde.r.a@gmail.com>
 Yuki <https://github.com/YukiNagat0>
 wackbyte <wackbyte@protonmail.com>
 GithubAnon0000 <GithubAnon0000@users.noreply.github.com>
+Mike Hardy <github@mikehardy.net>
 
 ********************
 
diff --git a/package.json b/package.json
index e3160b25f..2f954324f 100644
--- a/package.json
+++ b/package.json
@@ -67,7 +67,6 @@
         "@popperjs/core": "^2.11.8",
         "bootstrap": "^5.3.0",
         "bootstrap-icons": "^1.10.5",
-        "canvas": "npm:empty-npm-package",
         "codemirror": "^5.63.1",
         "d3": "^7.0.0",
         "fabric": "^5.3.0",
@@ -80,7 +79,7 @@
         "mathjax": "^3.1.2"
     },
     "resolutions": {
-        "canvas": "npm:empty-npm-package",
+        "canvas": "npm:empty-npm-package@1.0.0",
         "cookie": "0.7.0"
     },
     "browserslist": [