code cleanup (#825)

MisskeyIO · Nov 26, 2024 · e7afbd1 · e7afbd1
1 parent 029d6da
commit e7afbd1
Show file tree

Hide file tree

Showing 22 changed files with 24 additions and 25 deletions.
diff --git a/packages/backend/src/core/FileInfoService.ts b/packages/backend/src/core/FileInfoService.ts
@@ -273,7 +273,9 @@ export class FileInfoService {
 			watcher.close();
 		});
 		command.run();
-		for (let i = 1; true; i++) { // eslint-disable-line @typescript-eslint/no-unnecessary-condition
+		let i = 0;
+		while (true) {
+			i++;
 			const current = `${i}.png`;
 			const next = `${i + 1}.png`;
 			const framePath = join(cwd, current);

diff --git a/packages/backend/src/decorators.ts b/packages/backend/src/decorators.ts
@@ -20,8 +20,7 @@ export function bindThis(target: any, key: string, descriptor: any) {
 	return {
 		configurable: true,
 		get() {
-			// eslint-disable-next-line no-prototype-builtins
-			if (this === target.prototype || this.hasOwnProperty(key) ||
+			if (this === target.prototype || Object.hasOwn(this, key) ||
         typeof fn !== 'function') {
 				return fn;
 			}

diff --git a/packages/backend/src/server/oauth/OAuth2ProviderService.ts b/packages/backend/src/server/oauth/OAuth2ProviderService.ts
@@ -195,7 +195,7 @@ function getQueryMode(issuerUrl: string): oauth2orize.grant.Options['modes'] {
 				parsed.searchParams.append(key, value as string);
 			}
 
-			return (res as OAuthHttpResponse).redirect(parsed.toString());
+			(res as OAuthHttpResponse).redirect(parsed.toString());
 		},
 	};
 }

diff --git a/packages/backend/src/server/web/bios.css b/packages/backend/src/server/web/bios.css
@@ -5,7 +5,7 @@
  */
 
 * {
-	font-family: Fira code, Fira Mono, Consolas, Menlo, Courier, monospace;
+	font-family: 'Fira code', 'Fira Mono', Consolas, Menlo, Courier, monospace;
 }
 
 html {

diff --git a/packages/backend/src/server/web/boot.js b/packages/backend/src/server/web/boot.js
@@ -33,7 +33,7 @@
 	}
 
 	//#region Detect language & fetch translations
-	if (!localStorage.hasOwnProperty('locale')) {
+	if (!Object.hasOwn(localStorage, 'locale')) {
 		let lang = localStorage.getItem('lang');
 		if (lang == null || lang.toString == null || lang.toString() === 'null') {
 			lang = 'ja-JP';

diff --git a/packages/backend/src/server/web/cli.css b/packages/backend/src/server/web/cli.css
@@ -5,7 +5,7 @@
  */
 
 * {
-	font-family: Fira code, Fira Mono, Consolas, Menlo, Courier, monospace;
+	font-family: 'Fira code', 'Fira Mono', Consolas, Menlo, Courier, monospace;
 }
 
 html {

diff --git a/packages/backend/src/server/web/flush.js b/packages/backend/src/server/web/flush.js
@@ -39,7 +39,7 @@ message('Start flushing.');
 
         console.error(e);
         setTimeout(() => {
-            location = '/';
+            window.location = '/';
         }, 10000)
     }
 })();

diff --git a/packages/backend/test-server/tsconfig.json b/packages/backend/test-server/tsconfig.json
@@ -9,7 +9,7 @@
 		"noFallthroughCasesInSwitch": true,
 		"declaration": false,
 		"sourceMap": true,
-		"target": "ES2022",
+		"target": "es2022",
 		"module": "nodenext",
 		"moduleResolution": "nodenext",
 		"allowSyntheticDefaultImports": true,

diff --git a/packages/backend/test/tsconfig.json b/packages/backend/test/tsconfig.json
@@ -9,7 +9,7 @@
 		"noFallthroughCasesInSwitch": true,
 		"declaration": false,
 		"sourceMap": true,
-		"target": "ES2022",
+		"target": "es2022",
 		"module": "nodenext",
 		"moduleResolution": "nodenext",
 		"allowSyntheticDefaultImports": true,

diff --git a/packages/backend/test/unit/ap-request.ts b/packages/backend/test/unit/ap-request.ts
@@ -9,7 +9,7 @@ import httpSignature from '@peertube/http-signature';
 import { genRsaKeyPair } from '@/misc/gen-key-pair.js';
 import { ApRequestCreator } from '@/core/activitypub/ApRequestService.js';
 
-export const buildParsedSignature = (signingString: string, signature: string, algorithm: string) => {
+const buildParsedSignature = (signingString: string, signature: string, algorithm: string) => {
 	return {
 		scheme: 'Signature',
 		params: {

diff --git a/packages/backend/tsconfig.json b/packages/backend/tsconfig.json
@@ -9,7 +9,7 @@
 		"noFallthroughCasesInSwitch": true,
 		"declaration": false,
 		"sourceMap": false,
-		"target": "ES2022",
+		"target": "es2022",
 		"module": "nodenext",
 		"moduleResolution": "nodenext",
 		"allowSyntheticDefaultImports": true,

diff --git a/packages/frontend/src/components/MkNote.vue b/packages/frontend/src/components/MkNote.vue
@@ -288,7 +288,7 @@ const keymap = {
 	'down|j|tab': focusAfter,
 	'esc': blur,
 	'm|o': () => showMenu(true),
-	's': () => showContent.value !== showContent.value,
+	's': () => { showContent.value = !showContent.value; focus(); },
 };
 
 provide('react', (reaction: string) => {

diff --git a/packages/frontend/src/components/MkNoteDetailed.vue b/packages/frontend/src/components/MkNoteDetailed.vue
@@ -305,7 +305,7 @@ const keymap = {
 	'q': () => renote(true),
 	'esc': blur,
 	'm|o': () => showMenu(true),
-	's': () => showContent.value !== showContent.value,
+	's': () => { showContent.value = !showContent.value; focus(); },
 };
 
 provide('react', (reaction: string) => {

diff --git a/packages/frontend/src/components/MkPollEditor.vue b/packages/frontend/src/components/MkPollEditor.vue
@@ -116,7 +116,7 @@ function get(): PollEditorModelValue {
 	};
 
 	const calcAfter = () => {
-		let base = parseInt(after.value.toString());
+		let base = Number.parseInt(after.value.toString());
 		switch (unit.value) {
 			// @ts-expect-error fallthrough
 			case 'day': base *= 24;

diff --git a/packages/frontend/src/unicode-emoji-indexes/en-US.json b/packages/frontend/src/unicode-emoji-indexes/en-US.json
@@ -305,8 +305,6 @@
 	"👸": ["girl", "woman", "female", "blond", "crown", "royal", "queen"],
 	"🤴": ["boy", "man", "male", "crown", "royal", "king"],
 	"👰": ["couple", "marriage", "wedding", "woman", "bride"],
-	"👰": ["couple", "marriage", "wedding", "woman", "bride"],
-	"🤵": ["couple", "marriage", "wedding", "groom"],
 	"🤵": ["couple", "marriage", "wedding", "groom"],
 	"🏃‍♀️": ["woman", "walking", "exercise", "race", "running", "female"],
 	"🏃": ["man", "walking", "exercise", "race", "running"],

diff --git a/packages/frontend/test/tsconfig.json b/packages/frontend/test/tsconfig.json
@@ -9,7 +9,7 @@
 		"noFallthroughCasesInSwitch": true,
 		"declaration": false,
 		"sourceMap": true,
-		"target": "ES2022",
+		"target": "es2022",
 		"module": "nodenext",
 		"moduleResolution": "nodenext",
 		"allowSyntheticDefaultImports": true,

diff --git a/packages/frontend/tsconfig.json b/packages/frontend/tsconfig.json
@@ -9,7 +9,7 @@
 		"noFallthroughCasesInSwitch": true,
 		"declaration": false,
 		"sourceMap": false,
-		"target": "ES2022",
+		"target": "es2022",
 		"module": "nodenext",
 		"moduleResolution": "nodenext",
 		"removeComments": false,

diff --git a/packages/misskey-bubble-game/src/game.ts b/packages/misskey-bubble-game/src/game.ts
@@ -204,10 +204,10 @@ export class DropAndFusionGame extends EventEmitter<{
 		} else if (mono.shape === 'rectangle') {
 			return Matter.Bodies.rectangle(x, y, mono.sizeX, mono.sizeY, options);
 		} else if (mono.shape === 'custom') {
-			return Matter.Bodies.fromVertices(x, y, mono.vertices!.map(i => i.map(j => ({
+			return Matter.Bodies.fromVertices(x, y, mono.vertices?.map(i => i.map(j => ({
 				x: (j.x / mono.verticesSize!) * mono.sizeX,
 				y: (j.y / mono.verticesSize!) * mono.sizeY,
-			}))), options);
+			}))) ?? [], options);
 		} else {
 			throw new Error('unrecognized shape');
 		}

diff --git a/packages/misskey-bubble-game/tsconfig.json b/packages/misskey-bubble-game/tsconfig.json
@@ -1,7 +1,7 @@
 {
 	"$schema": "https://json.schemastore.org/tsconfig",
 	"compilerOptions": {
-		"target": "ES2022",
+		"target": "es2022",
 		"module": "nodenext",
 		"moduleResolution": "nodenext",
 		"declaration": true,

diff --git a/packages/misskey-js/tsconfig.json b/packages/misskey-js/tsconfig.json
@@ -1,7 +1,7 @@
 {
 	"$schema": "https://json.schemastore.org/tsconfig",
 	"compilerOptions": {
-		"target": "ES2022",
+		"target": "es2022",
 		"module": "nodenext",
 		"moduleResolution": "nodenext",
 		"declaration": true,

diff --git a/packages/misskey-reversi/tsconfig.json b/packages/misskey-reversi/tsconfig.json
@@ -1,7 +1,7 @@
 {
 	"$schema": "https://json.schemastore.org/tsconfig",
 	"compilerOptions": {
-		"target": "ES2022",
+		"target": "es2022",
 		"module": "nodenext",
 		"moduleResolution": "nodenext",
 		"declaration": true,

diff --git a/packages/sw/tsconfig.json b/packages/sw/tsconfig.json
@@ -9,7 +9,7 @@
 		"noFallthroughCasesInSwitch": true,
 		"declaration": false,
 		"sourceMap": false,
-		"target": "ES2022",
+		"target": "es2022",
 		"module": "nodenext",
 		"moduleResolution": "nodenext",
 		"removeComments": false,