Merge branch 'main' of https://github.com/MercuryWorkshop/scramjet

.eslintrc.json

@@ -16,6 +16,7 @@
 		"prefer-const": "warn",
 		"no-unreachable": "warn",
 		"no-undef": "off",
+		"no-empty": "off",
 		"@typescript-eslint/no-explicit-any": "off",
 		"@typescript-eslint/ban-ts-comment": "off",
 		"@typescript-eslint/ban-types": "off",

src/client/dom/element.ts

@@ -12,9 +12,9 @@ import {
 import type { URLMeta } from "../../shared/rewriters/url";
 
 export default function (client: ScramjetClient, self: typeof window) {
-	const nativeGetAttribute = self.Element.prototype.getAttribute;
+	const _nativeGetAttribute = self.Element.prototype.getAttribute;
 	const nativeSetAttribute = self.Element.prototype.setAttribute;
-	const nativeHasAttribute = self.Element.prototype.hasAttribute;
+	const _nativeHasAttribute = self.Element.prototype.hasAttribute;
 
 	const attrObject = {
 		nonce: [self.HTMLElement],

src/client/dom/fontface.ts

@@ -1,7 +1,7 @@
 import { ScramjetClient } from "../client";
-import { decodeUrl, rewriteCss } from "../../shared";
+import { rewriteCss } from "../../shared";
 
-export default function (client: ScramjetClient, self: typeof window) {
+export default function (client: ScramjetClient, _self: Self) {
 	client.Proxy("FontFace", {
 		construct(ctx) {
 			ctx.args[1] = rewriteCss(ctx.args[1], client.meta);

src/client/shared/error.ts

@@ -2,7 +2,7 @@ import { config, decodeUrl } from "../../shared";
 import { ScramjetClient } from "../client";
 
 export const enabled = () => self.$scramjet.config.flags.cleanerrors;
-export default function (client: ScramjetClient, self: Self) {
+export default function (client: ScramjetClient, _self: Self) {
 	// v8 only. all we need to do is clean the scramjet urls from stack traces
 	const closure = (error, stack) => {
 		let newstack = error.stack;
@@ -12,9 +12,9 @@ export default function (client: ScramjetClient, self: Self) {
 
 			if (url.endsWith(config.client)) {
 				// strip stack frames including scramjet handlers from the trace
-				let lines = newstack.split("\n");
-				let i = lines.find((l) => l.includes(url));
-				lines.splice(i, 1);
+				const lines = newstack.split("\n");
+				const line = lines.find((l) => l.includes(url));
+				lines.splice(line, 1);
 				newstack = lines.join("\n");
 				continue;
 			}
@@ -27,11 +27,11 @@ export default function (client: ScramjetClient, self: Self) {
 		return newstack;
 	};
 	client.Trap("Error.prepareStackTrace", {
-		get(ctx) {
+		get(_ctx) {
 			// this is a funny js quirk. the getter is ran every time you type something in console
 			return closure;
 		},
-		set(value) {
+		set(_value) {
 			// just ignore it if a site tries setting their own. not much we can really do
 		},
 	});

src/shared/rewriters/html.ts

@@ -270,7 +270,7 @@ export function rewriteSrcset(srcset: string, meta: URLMeta) {
 	return rewrittenUrls.join("");
 }
 
-function base64ToBytes(base64) {
+function _base64ToBytes(base64) {
 	const binString = atob(base64);
 
 	return Uint8Array.from(binString, (m) => m.codePointAt(0));