fix(lint): remove unnecessary type assertions after dep updates

Author: nonlooped

packages/shardwire/src/bridge/transport/socket.ts

@@ -76,13 +76,13 @@ class StandardWebSocketAdapter implements WebSocketLike {
 				this.openListeners.add(listener as () => void);
 				break;
 			case 'message':
-				this.messageListeners.add(listener as (data: unknown) => void);
+				this.messageListeners.add(listener);
 				break;
 			case 'close':
 				this.closeListeners.add(listener as () => void);
 				break;
 			case 'error':
-				this.errorListeners.add(listener as (error: unknown) => void);
+				this.errorListeners.add(listener);
 				break;
 			default:
 				break;

packages/shardwire/src/discord/runtime/discordjs-adapter.ts

@@ -1114,7 +1114,7 @@ export class DiscordJsRuntimeAdapter implements DiscordRuntimeAdapter {
 			...(payload.topic !== undefined ? { topic: payload.topic } : {}),
 			...(payload.reason ? { reason: payload.reason } : {}),
 		} as never);
-		return serializeChannel(edited as Channel);
+		return serializeChannel(edited);
 	}
 
 	private async deleteChannel(payload: BotActionPayloadMap['deleteChannel']) {
@@ -1234,7 +1234,7 @@ export class DiscordJsRuntimeAdapter implements DiscordRuntimeAdapter {
 			);
 		}
 
-		let owner: unknown = this.client as unknown;
+		let owner: unknown = this.client;
 		let target: unknown = owner;
 		for (const part of parts) {
 			if (!target || (typeof target !== 'object' && typeof target !== 'function')) {

packages/shardwire/test/helpers/fake-runtime.ts

@@ -45,8 +45,8 @@ export class FakeDiscordRuntime implements DiscordRuntimeAdapter {
 			existing.add(handler);
 		} else {
 			this.eventHandlers[name] = new Set([
-				handler as never as (payload: BotEventPayloadMap[BotEventName]) => void,
-			]) as EventHandlers[K];
+				handler as never,
+			]);
 		}
 		return () => {
 			const current = this.eventHandlers[name] as Set<(payload: BotEventPayloadMap[K]) => void> | undefined;
@@ -62,7 +62,7 @@ export class FakeDiscordRuntime implements DiscordRuntimeAdapter {
 		if (!handler) {
 			throw new Error(`No fake action handler registered for "${name}".`);
 		}
-		return handler(payload as never) as Promise<BotActionResultDataMap[K]>;
+		return handler(payload as never);
 	}
 
 	getClient(): unknown {

packages/shardwire/test/in-process.spec.ts

@@ -136,7 +136,7 @@ describe('in-process app bridge', () => {
 		const handler = ({ message }: { message: { id: string } }) => {
 			seen.push(message.id);
 		};
-		const unsub = app.on('messageCreate', handler as never, { channelId: 'c1' });
+		const unsub = app.on('messageCreate', handler, { channelId: 'c1' });
 		runtime.emit('messageCreate', {
 			receivedAt: Date.now(),
 			message: {
@@ -159,7 +159,7 @@ describe('in-process app bridge', () => {
 		});
 		expect(seen).toEqual(['m1']);
 
-		app.off('messageCreate', handler as never);
+		app.off('messageCreate', handler);
 		runtime.emit('messageCreate', {
 			receivedAt: Date.now(),
 			message: {
@@ -172,7 +172,7 @@ describe('in-process app bridge', () => {
 		});
 		expect(seen).toEqual(['m1']);
 
-		const second = app.on('messageCreate', handler as never);
+		const second = app.on('messageCreate', handler);
 		unsub();
 		second();
 		await app.close();
@@ -236,7 +236,7 @@ describe('in-process app bridge', () => {
 			capabilities: { events: ['ready', 'messageCreate'], actions: ['runRaw'] },
 		});
 
-		app.off('ready', (() => undefined) as never);
+		app.off('ready', (() => undefined));
 		const h1 = (() => undefined) as never;
 		const h2 = (() => undefined) as never;
 		const unsub1 = app.on('ready', h1);
@@ -258,11 +258,11 @@ describe('in-process app bridge', () => {
 		unsub1();
 		unsub1();
 		unsub2();
-		const missingUnsub = app.on('messageCreate', (() => undefined) as never);
+		const missingUnsub = app.on('messageCreate', (() => undefined));
 		missingUnsub();
-		app.off('messageCreate', (() => undefined) as never);
+		app.off('messageCreate', (() => undefined));
 
-		expect(() => app.on('guildDelete', (() => undefined) as never)).toThrow(/not available/i);
+		expect(() => app.on('guildDelete', (() => undefined))).toThrow(/not available/i);
 		await app.close();
 
 		const afterClose = await app.raw('guilds.fetch', ['x']);

packages/shardwire/test/validation.spec.ts

@@ -179,7 +179,7 @@ describe('discord-first bridge validation', () => {
 			connectBotBridge({
 				url: 'http://example.com',
 				secret: '',
-			} as never),
+			}),
 		).toThrow(/url|secret/i);
 	});
 
@@ -655,7 +655,7 @@ describe('discord-first bridge validation', () => {
 		});
 
 		await bot.ready();
-		await expect(app.ready({ strict: true } as never)).rejects.toThrow(/requires `manifest`/);
+		await expect(app.ready({ strict: true })).rejects.toThrow(/requires `manifest`/);
 
 		await app.close();
 		await bot.close();