chore: Upgrade to deno 1.42.3 (#931)

Author: jespertheend

.github/workflows/build.yml

@@ -9,7 +9,7 @@ on:
 env:
   # be sure to also update these in other workflows
   DENO_DIR: deno_dir
-  DENO_VERSION: "1.40.3"
+  DENO_VERSION: "1.42.3"
 
 jobs:
   build:

.github/workflows/ci.yml

@@ -10,7 +10,7 @@ on:
 env:
   # be sure to also update these in other workflows
   DENO_DIR: deno_dir
-  DENO_VERSION: "1.40.3"
+  DENO_VERSION: "1.42.3"
 
 jobs:
   lint:

.github/workflows/prepare-release.yml

@@ -11,7 +11,7 @@ on:
 env:
   # be sure to also update these in other workflows
   DENO_DIR: deno_dir
-  DENO_VERSION: "1.40.3"
+  DENO_VERSION: "1.42.3"
 
 jobs:
   prepare-release:

.github/workflows/release-jsr.yml

@@ -13,7 +13,7 @@ on:
 env:
   # be sure to also update these in other workflows
   DENO_DIR: deno_dir
-  DENO_VERSION: "1.40.3"
+  DENO_VERSION: "1.42.3"
 
 permissions:
   contents: read

.github/workflows/release-npm.yml

@@ -13,7 +13,7 @@ on:
 env:
   # be sure to also update these in other workflows
   DENO_DIR: deno_dir
-  DENO_VERSION: "1.40.3"
+  DENO_VERSION: "1.42.3"
 
 permissions: {}
 jobs:

src/util/binarySerialization.js

@@ -161,7 +161,7 @@ export function binaryToUuid(buffer, offset = 0) {
 		if (i == 3 || i == 5 || i == 7 || i == 9) str += "-";
 	}
 	if (allZeros) return null;
-	return str;
+	return /** @type {import("./util.js").UuidString} */ (str);
 }
 
 /**

studio/src/network/studioConnections/StudioConnectionsManager.js

@@ -42,7 +42,7 @@ export class StudioConnectionsManager {
 	 * But pages created by the build view should always be allowed.
 	 * Therefore, we create tokens for every page created by the build view.
 	 * Inspectors can provide these tokens when connecting, and we'll always allow the connection when the token is valid.
-	 * @type {Set<string>}
+	 * @type {Set<import("../../../../src/mod.js").UuidString>}
 	 */
 	#connectionTokens = new Set();
 
@@ -397,7 +397,7 @@ export class StudioConnectionsManager {
 	 * regardless of its origin, the connection type, or whether internal connections are enabled.
 	 */
 	createConnectionToken() {
-		const token = crypto.randomUUID();
+		const token = /** @type {import("../../../../src/mod.js").UuidString} */ (crypto.randomUUID());
 		this.#connectionTokens.add(token);
 		return token;
 	}

test/unit/studio/src/network/studioConnections/StudioConnectionsManager.test.js

@@ -50,7 +50,7 @@ async function basicTest({
 	let randomUuid = 0;
 	const mockUuid = stub(crypto, "randomUUID", () => {
 		randomUuid++;
-		return "random_uuid_" + randomUuid;
+		return /** @type {`${string}-${string}-${string}-${string}-${string}`} */ ("random_uuid_" + randomUuid);
 	});
 	const oldLocation = window.location;
 	try {

test/unit/studio/src/ui/popoverMenus/PopoverManager.test.js

@@ -120,7 +120,7 @@ testTypes({
 
 Deno.test({
 	name: "context menu creation",
-	fn() {
+	async fn() {
 		const { manager, uninstall } = basicManager();
 		try {
 			assertEquals(manager.curtainEl.parentElement, null);
@@ -137,6 +137,10 @@ Deno.test({
 
 			assertEquals(manager.removePopover(contextMenu), true);
 			assertEquals(manager.removePopover(contextMenu), false);
+
+			// Some extra code is run in the next event loop when creating a popover.
+			// We need to wait for this code to run, otherwise the test sanitizer will complain.
+			await waitForMicrotasks();
 		} finally {
 			uninstall();
 		}