feat(bot): add guild events

[GalvinPython]

This PR just adds guild events into the database and bot. Merging into the refactor/database branch because well.. it updates the database

[Copilot]

Pull Request Overview

This PR implements guild event handling for a Discord bot by adding database tracking for guild joins/leaves and synchronization logic for guilds that changed while the bot was offline. The implementation includes real-time event handlers and a startup reconciliation process.

• Adds Discord guild event handlers for GuildCreate and GuildDelete events with database integration
• Implements startup synchronization to detect and update guilds that were added/removed while offline
• Extends the database schema with a memberCount field and adds helper functions for guild management

Reviewed Changes

Copilot reviewed 6 out of 6 changed files in this pull request and generated 6 comments.

Show a summary per file

File	Description
src/utils/discord/updateGuildsOnStartup.ts	Startup reconciliation logic to sync guild state between Discord cache and database
src/index.ts	Integration of startup guild synchronization into bot initialization
src/events/guildDelete.ts	Event handler for when the bot leaves a guild
src/events/guildCreate.ts	Event handler for when the bot joins a new guild
src/db/schema.ts	Database schema update adding memberCount field
src/db/discord.ts	Helper functions for adding and removing guilds from database tracking

[Copilot]

Using forEach with async callbacks can lead to unhandled promise rejections and doesn't properly handle errors. Use Promise.all with map or a for...of loop instead.

Suggested change

	missingGuilds.forEach(async (guild) => {
	console.log(`Removing guild from tracking: ${guild.guildId}`);
	const result = await db
	.update(dbDiscordTable)
	.set({ isInServer: false })
	.where(eq(dbDiscordTable.guildId, guild.guildId))
	.returning();
	if (result) {
	console.log(
	`Successfully removed guild ${guild.guildId} from tracking.`,
	);
	} else {
	console.error(
	`Failed to remove guild ${guild.guildId} from tracking.`,
	);
	}
	});
	newGuilds.forEach(async (guildId) => {
	console.log(`Adding new guild to tracking: ${guildId}`);
	const result = await db
	.insert(dbDiscordTable)
	.values({
	guildId,
	allowedPublicSharing: false,
	feedrUpdatesChannelId: null,
	isInServer: true,
	memberCount: 0,
	})
	.returning();
	if (result) {
	console.log(`Successfully added guild ${guildId} to tracking.`);
	} else {
	console.error(`Failed to add guild ${guildId} to tracking.`);
	}
	});
	for (const guild of missingGuilds) {
	try {
	console.log(`Removing guild from tracking: ${guild.guildId}`);
	const result = await db
	.update(dbDiscordTable)
	.set({ isInServer: false })
	.where(eq(dbDiscordTable.guildId, guild.guildId))
	.returning();
	if (result) {
	console.log(
	`Successfully removed guild ${guild.guildId} from tracking.`,
	);
	} else {
	console.error(
	`Failed to remove guild ${guild.guildId} from tracking.`,
	);
	}
	} catch (error) {
	console.error(
	`Error while removing guild ${guild.guildId} from tracking:`,
	error,
	);
	}
	}
	for (const guildId of newGuilds) {
	try {
	console.log(`Adding new guild to tracking: ${guildId}`);
	const result = await db
	.insert(dbDiscordTable)
	.values({
	guildId,
	allowedPublicSharing: false,
	feedrUpdatesChannelId: null,
	isInServer: true,
	memberCount: 0,
	})
	.returning();
	if (result) {
	console.log(`Successfully added guild ${guildId} to tracking.`);
	} else {
	console.error(`Failed to add guild ${guildId} to tracking.`);
	}
	} catch (error) {
	console.error(
	`Error while adding guild ${guildId} to tracking:`,
	error,
	);
	}
	}

[Copilot]

The condition if (result) is incorrect. Database update operations return an array, which is always truthy even when empty. Check result.length > 0 instead.

Suggested change

	if (result) {
	if (result.length > 0) {

[Copilot]

The condition if (result) is incorrect. Database insert operations return an array, which is always truthy even when empty. Check result.length > 0 instead.

Suggested change

	if (result) {
	if (result.length > 0) {

[Copilot]

[nitpick] The 10-second timeout is a magic number. Consider extracting this to a named constant like GUILD_LOAD_RETRY_INTERVAL for better maintainability.

Suggested change

	await new Promise((resolve) => setTimeout(resolve, 10000));
	await new Promise((resolve) => setTimeout(resolve, GUILD_LOAD_RETRY_INTERVAL));