Skip to content
Merged
Show file tree
Hide file tree
Changes from 10 commits
Commits
Show all changes
33 commits
Select commit Hold shift + click to select a range
d73ef96
refactor(events): extract ready handler to events/ready.js
Mar 4, 2026
bdfe4f8
refactor(events): extract messageCreate handler to events/messageCrea…
Mar 4, 2026
f13be71
refactor(events): extract guildMemberAdd and voiceState handlers
Mar 4, 2026
549a9e6
refactor(events): extract interaction handlers to events/interactionC…
Mar 4, 2026
5930224
refactor(events): extract reaction handlers to events/reactions.js
Mar 4, 2026
08b1fda
refactor(events): extract error handlers to events/errors.js
Mar 4, 2026
f250bfb
refactor(events): extract voice state handler to events/voiceState.js
Mar 4, 2026
e4c7b93
refactor(events): update main events.js to use extracted modules
Mar 4, 2026
24195e9
refactor(events): modularize event handlers
Mar 4, 2026
ecc6714
test(utils): add cronParser and flattenToLeafPaths tests
Mar 4, 2026
45e3569
fix(test): correct cronParser timezone handling and dangerousKeys imp…
Mar 4, 2026
7c1df72
Merge branch 'main' into feat/code-quality-improvements
BillChirico Mar 4, 2026
011237f
refactor: wire guildMemberAdd handler from dedicated module
Mar 4, 2026
2b6bcfa
fix: use safeReply when interaction not deferred, sanitize error mess…
Mar 4, 2026
de39ceb
fix: add error logging for background tasks
Mar 4, 2026
64d6e16
fix: add mobile responsive header
Mar 4, 2026
0b88cd2
fix(test): use JSON.parse for __proto__ test reliability
Mar 4, 2026
c53c502
style: apply biome lint fixes
Mar 4, 2026
d69a2d9
fix(a11y): resolve accessibility errors in landing page
Mar 4, 2026
a3e1415
fix(a11y): add aria-expanded and aria-controls to mobile nav toggle
Mar 4, 2026
0d942a8
fix: check both deferred and replied in showcase error handler
Mar 4, 2026
043d801
docs: fix markdownlint MD022/MD031 violations in task files
Mar 4, 2026
59d72f6
fix: await accumulateMessage before evaluateNow for immediate triage
Mar 4, 2026
4f9c4a8
fix: import DANGEROUS_KEYS instead of duplicating
Mar 4, 2026
a442741
fix(messageCreate): improve async error handling and abort on buffer …
Mar 4, 2026
80b6482
fix(interactionCreate): improve error handling for deferred interactions
Mar 4, 2026
7aba485
fix(events): wrap sendSpamAlert in try/catch to prevent unhandled rej…
Mar 4, 2026
2b5d10f
fix(interactionCreate): add config gates and wrap fallback replies
Mar 4, 2026
48d53db
fix(test): update ticket handler tests for sanitized error messages
Mar 4, 2026
c4ff232
fix: resolve biome lint issues
Mar 4, 2026
abd62a7
fix: update tests for events module refactoring
Mar 4, 2026
daf4354
fix(test): add config mocks for challenge and poll handlers
Mar 4, 2026
e9096a4
fix: resolve biome lint errors (duplicate keys + formatting)
Mar 4, 2026
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
969 changes: 42 additions & 927 deletions src/modules/events.js

Large diffs are not rendered by default.

40 changes: 40 additions & 0 deletions src/modules/events/errors.js
Original file line number Diff line number Diff line change
@@ -0,0 +1,40 @@
/**
* Error Event Handlers
* Handles Discord client errors and process-level error handling
*/

import { Events } from 'discord.js';
import { error as logError } from '../../logger.js';

/** @type {boolean} Guard against duplicate process-level handler registration */
let processHandlersRegistered = false;

/**
* Register error event handlers
* @param {Client} client - Discord client
*/
export function registerErrorHandlers(client) {
client.on(Events.Error, (err) => {
logError('Discord error', { error: err.message, stack: err.stack });
});

if (!processHandlersRegistered) {
process.on('unhandledRejection', (err) => {
logError('Unhandled rejection', { error: err?.message || String(err), stack: err?.stack });
});
process.on('uncaughtException', async (err) => {
logError('Uncaught exception — shutting down', {
error: err?.message || String(err),
stack: err?.stack,
});
try {
const { Sentry } = await import('../../sentry.js');
await Sentry.flush(2000);
} catch {
// ignore — best-effort flush
}
process.exit(1);
});
processHandlersRegistered = true;
}
}
20 changes: 20 additions & 0 deletions src/modules/events/guildMemberAdd.js
Original file line number Diff line number Diff line change
@@ -0,0 +1,20 @@
/**
* Guild Member Add Event Handler
* Handles welcome messages when users join a guild
*/

import { Events } from 'discord.js';
import { getConfig } from '../config.js';
import { sendWelcomeMessage } from '../welcome.js';

/**
* Register a handler that sends the configured welcome message when a user joins a guild.
* @param {Client} client - Discord client instance to attach the event listener to.
* @param {Object} _config - Unused (kept for API compatibility); handler resolves per-guild config via getConfig().
*/
export function registerGuildMemberAddHandler(client, _config) {
client.on(Events.GuildMemberAdd, async (member) => {
const guildConfig = getConfig(member.guild.id);
await sendWelcomeMessage(member, client, guildConfig);
});
}
Loading
Loading