Use less method property declarations

Author: TheKodeToad

backend/src/plugin/core/public/command.ts

@@ -33,7 +33,7 @@ export interface BaseContext {
 	user: User;
 	member: Member;
 	channel: AnyTextableGuildChannel;
-	respond(reply: Reply): Promise<void>;
+	respond: (reply: Reply) => Promise<void>;
 }
 
 export interface CommandContext extends BaseContext {
@@ -45,7 +45,7 @@ export interface CommandContext extends BaseContext {
 export interface ComponentContext extends BaseContext {
 	/** The ID of the user who initially ran the command */
 	originalUserID: string;
-	edit(reply: Reply): Promise<void>;
+	edit: (reply: Reply) => Promise<void>;
 }
 
 export interface ReplyObject extends Omit<CreateMessageOptions, "messageReference" | "tts" | "content"> {

backend/src/plugin/core/public/extensionPoints.ts

@@ -6,9 +6,9 @@ import type { ClientEvents } from "oceanic.js";
 
 // custom implementations as fancy stuff with generics is required
 
-export interface BotEventListener<T extends keyof ClientEvents = keyof ClientEvents> {
+export interface BotEventListener<T extends keyof ClientEvents = any> {
 	type: T;
-	listener(...args: ClientEvents[T]): Promise<void> | void;
+	listener: (...args: ClientEvents[T]) => Promise<void> | void;
 }
 
 export function onBotEvent<T extends keyof ClientEvents>(listener: BotEventListener<T>): Contribution {

backend/src/plugin/core/public/helper/paginator.ts

@@ -4,9 +4,9 @@ import { ComponentTypes } from "oceanic.js";
 
 export interface Paginator<E, K> {
 	pageSize: number;
-	getKey(entry: E): K;
-	lookUp(context: BaseContext, query: PaginatorQuery<K>): Promise<E[]>;
-	render(entries: E[]): Promise<ReplyObject> | ReplyObject;
+	getKey: (entry: E) => K;
+	lookUp: (context: BaseContext, query: PaginatorQuery<K>) => Promise<E[]>;
+	render: (entries: E[]) => Promise<ReplyObject> | ReplyObject;
 }
 
 export interface PaginatorQuery<K> {

backend/src/plugin/logging/logger/messages.ts

@@ -6,7 +6,7 @@ import { onBotEvent } from "#plugin/core/public/extensionPoints.ts";
 import { logWithLogger } from "#plugin/logging/helper/webhooks.ts";
 import { loggingConfigStore } from "#plugin/logging/index.ts";
 import { cleanUpMessageCacheEntries, getMessageCacheEntry, takeMessageCacheEntry, upsertMessageCacheEntry, type MessageCacheEntry } from "#plugin/logging/storage/messageCache.ts";
-import { Message, Routes } from "oceanic.js";
+import { Message, Routes, type PossiblyUncachedMessage } from "oceanic.js";
 
 const logger = moduleLogger();
 
@@ -89,7 +89,7 @@ async function handleUpdate(message: Message): Promise<void> {
 	}
 }
 
-async function handleDelete(message: Message): Promise<void> {
+async function handleDelete(message: PossiblyUncachedMessage): Promise<void> {
 	if (message.guild == null)
 		return;
 
@@ -103,8 +103,8 @@ async function handleDelete(message: Message): Promise<void> {
 	if (entry === null)
 		return;
 
-	const avatarURL = entry.authorAvatarHash !== null ?
-		bot.util.formatImage(Routes.USER_AVATAR(entry.authorID, entry.authorAvatarHash))
+	const avatarURL = entry.authorAvatarHash !== null
+		? bot.util.formatImage(Routes.USER_AVATAR(entry.authorID, entry.authorAvatarHash))
 		: undefined;
 
 	for (const logger of config.loggers) {

backend/src/plugin/moderation/helper/bulkAction.ts

@@ -26,8 +26,8 @@ type BulkAction =
 		duration?: number;
 		memberRanking: MemberRanking;
 
-		check?(member: Member): Promise<string | true> | string | true;
-		makeCase(options: Pick<CreateCaseOptions, "createdAt" | "expiresAt" | "actorID" | "targetID" | "dmDelivered">): CreateCaseOptions;
+		check?: (member: Member) => Promise<string | true> | string | true;
+		makeCase: (options: Pick<CreateCaseOptions, "createdAt" | "expiresAt" | "actorID" | "targetID" | "dmDelivered">) => CreateCaseOptions;
 	};
 
 export interface BulkSuccessEntry {
@@ -51,7 +51,6 @@ export async function doBulkAction(action: BulkAction): Promise<BulkResult> {
 
 	const members = await fetchMembersCached(action.guild, action.ids);
 
-	// eslint-disable-next-line @typescript-eslint/unbound-method
 	action.check ??= () => true;
 
 	for (const targetID of action.ids) {