evalWithDocument + stricterd restricted globals

Author: fongandrew

eslint.config.js

@@ -225,10 +225,12 @@ export default tseslint.config(
 		rules: {
 			'no-restricted-globals': [
 				'error',
-				...['window', 'document', 'history', 'location'].map((name) => ({
-					name,
-					message: `Use the document-setup utility instead`,
-				})),
+				...['window', 'document', 'history', 'location', 'requestAnimationFrame'].map(
+					(name) => ({
+						name,
+						message: `Use the document-setup utility instead`,
+					}),
+				),
 			],
 		},
 	},

src/lib/utility/callback-attrs/mount.ts

@@ -25,7 +25,7 @@ const [processed, setProcessed] = createMagicProp<boolean>();
  * Special property to stock on window or document to mark that a mount request
  * has been scheduled.
  */
-const [raf, setRaf] = createMagicProp<ReturnType<typeof requestAnimationFrame>>();
+const [raf, setRaf] = createMagicProp<number>();
 
 /**
  * Process all mount attributes for a given node

src/lib/utility/document-setup.ts

@@ -1,12 +1,12 @@
 import { parentDocument } from '~/lib/utility/multi-view';
 
-export type DocumentSetupFunction = (document: Document) => void;
+export type DocumentCallbackFunction = (document: Document) => void;
 
 /**
  * Track all the setup functions (so we don't run twice and can re-run on all new
  * open documents).
  */
-const setupFunctions = new Set<DocumentSetupFunction>();
+const setupFunctions = new Set<DocumentCallbackFunction>();
 
 /**
  * Track which documents are open (so when adding a new setup function, we can call the
@@ -19,7 +19,7 @@ export const activeDocuments = new Set<Document>(parentDocument ? [parentDocumen
  * on the current document after idle period. Will also be triggered for new documents (if
  * doing child windows or something like that).
  */
-export const registerDocumentSetup = (setup: DocumentSetupFunction) => {
+export const registerDocumentSetup = (setup: DocumentCallbackFunction) => {
 	if (!setupFunctions.has(setup)) {
 		setupFunctions.add(setup);
 		for (const document of activeDocuments) {
@@ -42,3 +42,10 @@ export function registerDocument(document: Document) {
 		}
 	}
 }
+
+/** Run a function on all active documents */
+export function evalWithDocument(cb: DocumentCallbackFunction) {
+	for (const document of activeDocuments) {
+		cb(document);
+	}
+}