fix: prevent duplicate hub rules acceptance entries and handle errors

[seer-by-sentry]

👋 Hi there! This PR was automatically generated by Autofix 🤖

This fix was triggered by dev-737.

Fixes INTERCHAT-PRODUCTION-1F3. The issue was that: handleHubRulesAccept attempts to create HubRulesAcceptance without checking for existing records, causing a unique constraint error on duplicate requests.

• Replaced create with upsert to prevent duplicate entries in hubRulesAcceptance table.
• Added error handling around the database operation to catch potential issues.
• Added a check to prevent attempting to reply to an interaction that has already been replied to, avoiding errors.

If you have any questions or feedback for the Sentry team about this fix, please email autofix@sentry.io with the Run ID: 29894.

[Copilot]

Pull Request Overview

A bugfix PR to prevent duplicate hub rules acceptance entries and to improve error handling during interaction responses.

• Replaced the direct create call with an upsert to prevent unique constraint errors.
• Wrapped the database operation in a try‐catch block with updated error handling.
• Added a check to avoid replying to an interaction that has already been replied to.

[Copilot]

The repeated check for '!interaction.replied' in both the main execution path and the error handling block could lead to subtle logic issues if the interaction state changes unexpectedly. Consider consolidating this logic to ensure the interaction is only replied to once.

Suggested change

	if (!interaction.replied) {
	await interaction.followUp({ embeds: [embed], components: [], flags: ['Ephemeral'] });
	await this.redis.del(`${RedisKeys.RulesShown}:${interaction.user.id}:${hubId}`);
	}
	} catch (error) {
	Logger.error('Error while accepting hub rules:', error);
	// Don't attempt to reply if the interaction was already replied to
	if (!interaction.replied) {
	const locale = await fetchUserLocale(interaction.user.id);
	await interaction.followUp({
	content: t('errors.generalError', locale, {
	emoji: getEmoji('x_icon', interaction.client)
	}),
	flags: ['Ephemeral']
	});
	}
	}
	await this.safeReply(interaction, {
	embeds: [embed],
	components: [],
	flags: ['Ephemeral'],
	});
	await this.redis.del(`${RedisKeys.RulesShown}:${interaction.user.id}:${hubId}`);
	} catch (error) {
	Logger.error('Error while accepting hub rules:', error);
	// Don't attempt to reply if the interaction was already replied to
	const locale = await fetchUserLocale(interaction.user.id);
	await this.safeReply(interaction, {
	content: t('errors.generalError', locale, {
	emoji: getEmoji('x_icon', interaction.client),
	}),
	flags: ['Ephemeral'],
	});