Update dependencies in tldraw-sync-worker, removing outdated Cloudflare types and adding workers-types. Refactor TldrawDurableObject to specify type for schedulePersistToR2. Adjust router type definitions in worker.ts for improved clarity.

Author: mdroidian

apps/tldraw-sync-worker/package.json

@@ -10,14 +10,14 @@
 		"check-types": "tsc --noEmit -p tsconfig.json"
 	},
 	"dependencies": {
-		"@cloudflare/types": "^6.29.0",
 		"@tldraw/sync-core": "2.4.6",
 		"@tldraw/tlschema": "2.4.6",
 		"cloudflare-workers-unfurl": "^0.0.7",
 		"itty-router": "^5.0.17",
 		"lodash.throttle": "^4.1.1"
 	},
 	"devDependencies": {
+		"@cloudflare/workers-types": "^4.20240208.0",
 		"@types/lodash.throttle": "^4",
 		"typescript": "^5.0.2",
 		"wrangler": "^3.64.0"

apps/tldraw-sync-worker/worker/TldrawDurableObject.ts

@@ -191,7 +191,7 @@ export class TldrawDurableObject {
 	}
 
 	// we throttle persistance so it only happens every 10 seconds
-	schedulePersistToR2 = throttle(async () => {
+	schedulePersistToR2: () => void = throttle(async () => {
 		if (!this.roomPromise || !this.roomId) return
 		const room = await this.getRoom()
 

apps/tldraw-sync-worker/worker/worker.ts

@@ -8,14 +8,11 @@ export { TldrawDurableObject } from "./TldrawDurableObject";
 // we use itty-router (https://itty.dev/) to handle routing. in this example we turn on CORS because
 // we're hosting the worker separately to the client. you should restrict this to your own domain.
 const { preflight, corsify } = cors({ origin: "*" });
-const router = AutoRouter<
-  IRequest,
-  [env: Environment, ctx: unknown /* ExecutionContext */]
->({
+const router = AutoRouter<IRequest, [env: Environment, ctx: ExecutionContext]>({
   before: [preflight],
   finally: [corsify],
   catch: (e) => {
-    console.error(e); // eslint-disable-line no-console ?
+    console.error(e);
     return error(e);
   },
 })
@@ -29,13 +26,6 @@ const router = AutoRouter<
     });
   })
 
-  //   We store assest in native apps
-  // assets can be uploaded to the bucket under /uploads:
-  //   .post("/uploads/:uploadId", handleAssetUpload)
-
-  // they can be retrieved from the bucket too:
-  //   .get("/uploads/:uploadId", handleAssetDownload)
-
   // bookmarks need to extract metadata from pasted URLs:
   .get("/unfurl", handleUnfurlRequest);