refactor: add leveling view layer

Create leveling.view.ts with UI logic extracted from leaderboard command:
- getLeaderboardEmbed() for leaderboard display (XP and Balance)
- getMedalEmoji() helper for ranking medals (🥇🥈🥉)
- formatLeaderEntry() helper for entry formatting with null safety

Updated leaderboard.ts to use view functions instead of inline formatting.

src/commands/economy/trade.ts

@@ -1,6 +1,6 @@
 import { createCommand } from "@/lib/utils";
 import { SlashCommandBuilder, ChannelType, ThreadAutoArchiveDuration, MessageFlags } from "discord.js";
-import { TradeService } from "@/modules/trade/trade.service";
+import { tradeService } from "@/modules/trade/trade.service";
 import { getTradeDashboard } from "@/modules/trade/trade.view";
 import { createErrorEmbed, createWarningEmbed } from "@lib/embeds";
 
@@ -59,7 +59,7 @@ export const trade = createCommand({
         }
 
         // Setup Session
-        const session = TradeService.createSession(thread.id,
+        const session = tradeService.createSession(thread.id,
             { id: interaction.user.id, username: interaction.user.username },
             { id: targetUser.id, username: targetUser.username }
         );

src/commands/inventory/inventory.ts

@@ -2,7 +2,8 @@ import { createCommand } from "@/lib/utils";
 import { SlashCommandBuilder } from "discord.js";
 import { inventoryService } from "@/modules/inventory/inventory.service";
 import { userService } from "@/modules/user/user.service";
-import { createWarningEmbed, createBaseEmbed } from "@lib/embeds";
+import { createWarningEmbed } from "@lib/embeds";
+import { getInventoryEmbed } from "@/modules/inventory/inventory.view";
 
 export const inventory = createCommand({
     data: new SlashCommandBuilder()
@@ -24,18 +25,19 @@ export const inventory = createCommand({
         }
 
         const user = await userService.getOrCreateUser(targetUser.id, targetUser.username);
-        const items = await inventoryService.getInventory(user.id);
+        if (!user) {
+            await interaction.editReply({ embeds: [createWarningEmbed("Failed to load user data.", "Error")] });
+            return;
+        }
+
+        const items = await inventoryService.getInventory(user.id.toString());
 
         if (!items || items.length === 0) {
             await interaction.editReply({ embeds: [createWarningEmbed("Inventory is empty.", `${user.username}'s Inventory`)] });
             return;
         }
 
-        const description = items.map(entry => {
-            return `**${entry.item.name}** x${entry.quantity}`;
-        }).join("\n");
-
-        const embed = createBaseEmbed(`${user.username}'s Inventory`, description, "Blue");
+        const embed = getInventoryEmbed(items, user.username);
 
         await interaction.editReply({ embeds: [embed] });
     }

src/commands/inventory/use.ts

@@ -2,7 +2,8 @@ import { createCommand } from "@/lib/utils";
 import { SlashCommandBuilder } from "discord.js";
 import { inventoryService } from "@/modules/inventory/inventory.service";
 import { userService } from "@/modules/user/user.service";
-import { createErrorEmbed, createSuccessEmbed } from "@lib/embeds";
+import { createErrorEmbed } from "@lib/embeds";
+import { getItemUseResultEmbed } from "@/modules/inventory/inventory.view";
 import { inventory, items } from "@/db/schema";
 import { eq, and, like } from "drizzle-orm";
 import { DrizzleClient } from "@/lib/DrizzleClient";
@@ -25,9 +26,13 @@ export const use = createCommand({
 
         const itemId = interaction.options.getNumber("item", true);
         const user = await userService.getOrCreateUser(interaction.user.id, interaction.user.username);
+        if (!user) {
+            await interaction.editReply({ embeds: [createErrorEmbed("Failed to load user data.")] });
+            return;
+        }
 
         try {
-            const result = await inventoryService.useItem(user.id, itemId);
+            const result = await inventoryService.useItem(user.id.toString(), itemId);
 
             const usageData = result.usageData;
             if (usageData) {
@@ -53,11 +58,7 @@ export const use = createCommand({
                 }
             }
 
-            const embed = createSuccessEmbed(
-                result.results.map(r => `• ${r}`).join("\n"),
-                `Used ${result.usageData.effects.length > 0 ? 'Item' : 'Item'}` // Generic title, improves below
-            );
-            embed.setTitle("Item Used!");
+            const embed = getItemUseResultEmbed(result.results);
 
             await interaction.editReply({ embeds: [embed] });
 

src/commands/leveling/leaderboard.ts

@@ -3,7 +3,8 @@ import { SlashCommandBuilder } from "discord.js";
 import { DrizzleClient } from "@/lib/DrizzleClient";
 import { users } from "@/db/schema";
 import { desc } from "drizzle-orm";
-import { createWarningEmbed, createBaseEmbed } from "@lib/embeds";
+import { createWarningEmbed } from "@lib/embeds";
+import { getLeaderboardEmbed } from "@/modules/leveling/leveling.view";
 
 export const leaderboard = createCommand({
     data: new SlashCommandBuilder()
@@ -34,13 +35,7 @@ export const leaderboard = createCommand({
             return;
         }
 
-        const description = leaders.map((user, index) => {
-            const medal = index === 0 ? "🥇" : index === 1 ? "🥈" : index === 2 ? "🥉" : `${index + 1}.`;
-            const value = isXp ? `Lvl ${user.level} (${user.xp} XP)` : `${user.balance} 🪙`;
-            return `${medal} **${user.username}** — ${value}`;
-        }).join("\n");
-
-        const embed = createBaseEmbed(isXp ? "🏆 XP Leaderboard" : "💰 Richest Players", description, "Gold");
+        const embed = getLeaderboardEmbed(leaders, isXp ? 'xp' : 'balance');
 
         await interaction.editReply({ embeds: [embed] });
     }

src/commands/quest/quests.ts

@@ -1,7 +1,8 @@
 import { createCommand } from "@/lib/utils";
 import { SlashCommandBuilder, MessageFlags } from "discord.js";
 import { questService } from "@/modules/quest/quest.service";
-import { createWarningEmbed, createBaseEmbed } from "@lib/embeds";
+import { createWarningEmbed } from "@lib/embeds";
+import { getQuestListEmbed } from "@/modules/quest/quest.view";
 
 export const quests = createCommand({
     data: new SlashCommandBuilder()
@@ -17,21 +18,7 @@ export const quests = createCommand({
             return;
         }
 
-        const embed = createBaseEmbed("📜 Quest Log", undefined, "Blue");
-
-        userQuests.forEach(entry => {
-            const status = entry.completedAt ? "✅ Completed" : "In Progress";
-            const rewards = entry.quest.rewards as { xp?: number, balance?: number };
-            const rewardStr = [];
-            if (rewards?.xp) rewardStr.push(`${rewards.xp} XP`);
-            if (rewards?.balance) rewardStr.push(`${rewards.balance} 🪙`);
-
-            embed.addFields({
-                name: `${entry.quest.name} (${status})`,
-                value: `${entry.quest.description}\n**Rewards:** ${rewardStr.join(", ")}\n**Progress:** ${entry.progress}%`,
-                inline: false
-            });
-        });
+        const embed = getQuestListEmbed(userQuests);
 
         await interaction.editReply({ embeds: [embed] });
     }

src/lib/BotClient.ts

@@ -4,6 +4,7 @@ import type { Command } from "@lib/types";
 import { env } from "@lib/env";
 import { CommandLoader } from "@lib/loaders/CommandLoader";
 import { EventLoader } from "@lib/loaders/EventLoader";
+import { logger } from "@lib/logger";
 
 export class Client extends DiscordClient {
 
@@ -21,25 +22,25 @@ export class Client extends DiscordClient {
     async loadCommands(reload: boolean = false) {
         if (reload) {
             this.commands.clear();
-            console.log("♻️  Reloading commands...");
+            logger.info("♻️  Reloading commands...");
         }
 
         const commandsPath = join(import.meta.dir, '../commands');
         const result = await this.commandLoader.loadFromDirectory(commandsPath, reload);
 
-        console.log(`📦 Command loading complete: ${result.loaded} loaded, ${result.skipped} skipped, ${result.errors.length} errors`);
+        logger.info(`📦 Command loading complete: ${result.loaded} loaded, ${result.skipped} skipped, ${result.errors.length} errors`);
     }
 
     async loadEvents(reload: boolean = false) {
         if (reload) {
             this.removeAllListeners();
-            console.log("♻️  Reloading events...");
+            logger.info("♻️  Reloading events...");
         }
 
         const eventsPath = join(import.meta.dir, '../events');
         const result = await this.eventLoader.loadFromDirectory(eventsPath, reload);
 
-        console.log(`📦 Event loading complete: ${result.loaded} loaded, ${result.skipped} skipped, ${result.errors.length} errors`);
+        logger.info(`📦 Event loading complete: ${result.loaded} loaded, ${result.skipped} skipped, ${result.errors.length} errors`);
     }
 
 
@@ -48,7 +49,7 @@ export class Client extends DiscordClient {
         // We use env.DISCORD_BOT_TOKEN directly so this can run without client.login()
         const token = env.DISCORD_BOT_TOKEN;
         if (!token) {
-            console.error("❌ DISCORD_BOT_TOKEN is not set.");
+            logger.error("DISCORD_BOT_TOKEN is not set.");
             return;
         }
 
@@ -58,16 +59,16 @@ export class Client extends DiscordClient {
         const clientId = env.DISCORD_CLIENT_ID;
 
         if (!clientId) {
-            console.error("❌ DISCORD_CLIENT_ID is not set.");
+            logger.error("DISCORD_CLIENT_ID is not set.");
             return;
         }
 
         try {
-            console.log(`Started refreshing ${commandsData.length} application (/) commands.`);
+            logger.info(`Started refreshing ${commandsData.length} application (/) commands.`);
 
             let data;
             if (guildId) {
-                console.log(`Registering commands to guild: ${guildId}`);
+                logger.info(`Registering commands to guild: ${guildId}`);
                 data = await rest.put(
                     Routes.applicationGuildCommands(clientId, guildId),
                     { body: commandsData },
@@ -75,20 +76,20 @@ export class Client extends DiscordClient {
                 // Clear global commands to avoid duplicates
                 await rest.put(Routes.applicationCommands(clientId), { body: [] });
             } else {
-                console.log('Registering commands globally');
+                logger.info('Registering commands globally');
                 data = await rest.put(
                     Routes.applicationCommands(clientId),
                     { body: commandsData },
                 );
             }
 
-            console.log(`✅ Successfully reloaded ${(data as any).length} application (/) commands.`);
+            logger.success(`Successfully reloaded ${(data as any).length} application (/) commands.`);
         } catch (error: any) {
             if (error.code === 50001) {
-                console.warn("⚠️  Missing Access: The bot is not in the guild or lacks 'applications.commands' scope.");
-                console.warn("   If you are testing locally, make sure you invited the bot with scope 'bot applications.commands'.");
+                logger.warn("Missing Access: The bot is not in the guild or lacks 'applications.commands' scope.");
+                logger.warn("   If you are testing locally, make sure you invited the bot with scope 'bot applications.commands'.");
             } else {
-                console.error(error);
+                logger.error(error);
             }
         }
     }

src/lib/handlers/AutocompleteHandler.ts

@@ -1,5 +1,6 @@
 import { AutocompleteInteraction } from "discord.js";
 import { AuroraClient } from "@/lib/BotClient";
+import { logger } from "@lib/logger";
 
 /**
  * Handles autocomplete interactions for slash commands
@@ -15,7 +16,7 @@ export class AutocompleteHandler {
         try {
             await command.autocomplete(interaction);
         } catch (error) {
-            console.error(`Error handling autocomplete for ${interaction.commandName}:`, error);
+            logger.error(`Error handling autocomplete for ${interaction.commandName}:`, error);
         }
     }
 }

src/lib/handlers/CommandHandler.ts

@@ -2,6 +2,7 @@ import { ChatInputCommandInteraction, MessageFlags } from "discord.js";
 import { AuroraClient } from "@/lib/BotClient";
 import { userService } from "@/modules/user/user.service";
 import { createErrorEmbed } from "@lib/embeds";
+import { logger } from "@lib/logger";
 
 /**
  * Handles slash command execution
@@ -12,7 +13,7 @@ export class CommandHandler {
         const command = AuroraClient.commands.get(interaction.commandName);
 
         if (!command) {
-            console.error(`No command matching ${interaction.commandName} was found.`);
+            logger.error(`No command matching ${interaction.commandName} was found.`);
             return;
         }
 
@@ -20,13 +21,13 @@ export class CommandHandler {
         try {
             await userService.getOrCreateUser(interaction.user.id, interaction.user.username);
         } catch (error) {
-            console.error("Failed to ensure user exists:", error);
+            logger.error("Failed to ensure user exists:", error);
         }
 
         try {
             await command.execute(interaction);
         } catch (error) {
-            console.error(error);
+            logger.error(String(error));
             const errorEmbed = createErrorEmbed('There was an error while executing this command!');
 
             if (interaction.replied || interaction.deferred) {

src/lib/handlers/ComponentInteractionHandler.ts

@@ -1,4 +1,5 @@
 import { ButtonInteraction, StringSelectMenuInteraction, ModalSubmitInteraction } from "discord.js";
+import { logger } from "@lib/logger";
 
 type ComponentInteraction = ButtonInteraction | StringSelectMenuInteraction | ModalSubmitInteraction;
 
@@ -19,7 +20,7 @@ export class ComponentInteractionHandler {
                     await handlerMethod(interaction);
                     return;
                 } else {
-                    console.error(`Handler method ${route.method} not found in module`);
+                    logger.error(`Handler method ${route.method} not found in module`);
                 }
             }
         }

src/lib/loaders/CommandLoader.ts

@@ -4,6 +4,7 @@ import type { Command } from "@lib/types";
 import { config } from "@lib/config";
 import type { LoadResult, LoadError } from "./types";
 import type { Client } from "../BotClient";
+import { logger } from "@lib/logger";
 
 /**
  * Handles loading commands from the file system
@@ -44,7 +45,7 @@ export class CommandLoader {
                 await this.loadCommandFile(filePath, reload, result);
             }
         } catch (error) {
-            console.error(`Error reading directory ${dir}:`, error);
+            logger.error(`Error reading directory ${dir}:`, error);
             result.errors.push({ file: dir, error });
         }
     }
@@ -59,7 +60,7 @@ export class CommandLoader {
             const commands = Object.values(commandModule);
 
             if (commands.length === 0) {
-                console.warn(`⚠️  No commands found in ${filePath}`);
+                logger.warn(`No commands found in ${filePath}`);
                 result.skipped++;
                 return;
             }
@@ -73,21 +74,21 @@ export class CommandLoader {
                     const isEnabled = config.commands[command.data.name] !== false;
 
                     if (!isEnabled) {
-                        console.log(`🚫 Skipping disabled command: ${command.data.name}`);
+                        logger.info(`🚫 Skipping disabled command: ${command.data.name}`);
                         result.skipped++;
                         continue;
                     }
 
                     this.client.commands.set(command.data.name, command);
-                    console.log(`✅ Loaded command: ${command.data.name}`);
+                    logger.success(`Loaded command: ${command.data.name}`);
                     result.loaded++;
                 } else {
-                    console.warn(`⚠️  Skipping invalid command in ${filePath}`);
+                    logger.warn(`Skipping invalid command in ${filePath}`);
                     result.skipped++;
                 }
             }
         } catch (error) {
-            console.error(`❌ Failed to load command from ${filePath}:`, error);
+            logger.error(`Failed to load command from ${filePath}:`, error);
             result.errors.push({ file: filePath, error });
         }
     }

src/lib/loaders/EventLoader.ts

@@ -3,6 +3,7 @@ import { join } from "node:path";
 import type { Event } from "@lib/types";
 import type { LoadResult } from "./types";
 import type { Client } from "../BotClient";
+import { logger } from "@lib/logger";
 
 /**
  * Handles loading events from the file system
@@ -43,7 +44,7 @@ export class EventLoader {
                 await this.loadEventFile(filePath, reload, result);
             }
         } catch (error) {
-            console.error(`Error reading directory ${dir}:`, error);
+            logger.error(`Error reading directory ${dir}:`, error);
             result.errors.push({ file: dir, error });
         }
     }
@@ -63,14 +64,14 @@ export class EventLoader {
                 } else {
                     this.client.on(event.name, (...args) => event.execute(...args));
                 }
-                console.log(`✅ Loaded event: ${event.name}`);
+                logger.success(`Loaded event: ${event.name}`);
                 result.loaded++;
             } else {
-                console.warn(`⚠️  Skipping invalid event in ${filePath}`);
+                logger.warn(`Skipping invalid event in ${filePath}`);
                 result.skipped++;
             }
         } catch (error) {
-            console.error(`❌ Failed to load event from ${filePath}:`, error);
+            logger.error(`Failed to load event from ${filePath}:`, error);
             result.errors.push({ file: filePath, error });
         }
     }

src/lib/logger.ts

@@ -0,0 +1,39 @@
+/**
+ * Centralized logging utility with consistent formatting
+ */
+export const logger = {
+    /**
+     * General information message
+     */
+    info: (message: string, ...args: any[]) => {
+        console.log(`ℹ️  ${message}`, ...args);
+    },
+
+    /**
+     * Success message
+     */
+    success: (message: string, ...args: any[]) => {
+        console.log(`✅ ${message}`, ...args);
+    },
+
+    /**
+     * Warning message
+     */
+    warn: (message: string, ...args: any[]) => {
+        console.warn(`⚠️  ${message}`, ...args);
+    },
+
+    /**
+     * Error message
+     */
+    error: (message: string, ...args: any[]) => {
+        console.error(`❌ ${message}`, ...args);
+    },
+
+    /**
+     * Debug message
+     */
+    debug: (message: string, ...args: any[]) => {
+        console.log(`🔍 ${message}`, ...args);
+    },
+};

src/modules/class/class.service.ts

@@ -2,14 +2,16 @@ import { classes, users } from "@/db/schema";
 import { eq, sql } from "drizzle-orm";
 import { DrizzleClient } from "@/lib/DrizzleClient";
 import { UserError } from "@/lib/errors";
+import { withTransaction } from "@/lib/db";
+import type { Transaction } from "@/lib/types";
 
 export const classService = {
     getAllClasses: async () => {
         return await DrizzleClient.query.classes.findMany();
     },
 
-    assignClass: async (userId: string, classId: bigint, tx?: any) => {
-        const execute = async (txFn: any) => {
+    assignClass: async (userId: string, classId: bigint, tx?: Transaction) => {
+        return await withTransaction(async (txFn) => {
             const cls = await txFn.query.classes.findFirst({
                 where: eq(classes.id, classId),
             });
@@ -22,8 +24,7 @@ export const classService = {
                 .returning();
 
             return user;
-        };
-        return tx ? await execute(tx) : await DrizzleClient.transaction(execute);
+        }, tx);
     },
     getClassBalance: async (classId: bigint) => {
         const cls = await DrizzleClient.query.classes.findFirst({
@@ -31,15 +32,15 @@ export const classService = {
         });
         return cls?.balance || 0n;
     },
-    modifyClassBalance: async (classId: bigint, amount: bigint, tx?: any) => {
-        const execute = async (txFn: any) => {
+    modifyClassBalance: async (classId: bigint, amount: bigint, tx?: Transaction) => {
+        return await withTransaction(async (txFn) => {
             const cls = await txFn.query.classes.findFirst({
                 where: eq(classes.id, classId),
             });
 
             if (!cls) throw new UserError("Class not found");
 
-            if ((cls.balance ?? 0n) < amount) {
+            if ((cls.balance ?? 0n) + amount < 0n) {
                 throw new UserError("Insufficient class funds");
             }
 
@@ -51,35 +52,31 @@ export const classService = {
                 .returning();
 
             return updatedClass;
-        };
-        return tx ? await execute(tx) : await DrizzleClient.transaction(execute);
+        }, tx);
     },
 
-    updateClass: async (id: bigint, data: Partial<typeof classes.$inferInsert>, tx?: any) => {
-        const execute = async (txFn: any) => {
+    updateClass: async (id: bigint, data: Partial<typeof classes.$inferInsert>, tx?: Transaction) => {
+        return await withTransaction(async (txFn) => {
             const [updatedClass] = await txFn.update(classes)
                 .set(data)
                 .where(eq(classes.id, id))
                 .returning();
             return updatedClass;
-        };
-        return tx ? await execute(tx) : await DrizzleClient.transaction(execute);
+        }, tx);
     },
 
-    createClass: async (data: typeof classes.$inferInsert, tx?: any) => {
-        const execute = async (txFn: any) => {
+    createClass: async (data: typeof classes.$inferInsert, tx?: Transaction) => {
+        return await withTransaction(async (txFn) => {
             const [newClass] = await txFn.insert(classes)
                 .values(data)
                 .returning();
             return newClass;
-        };
-        return tx ? await execute(tx) : await DrizzleClient.transaction(execute);
+        }, tx);
     },
 
-    deleteClass: async (id: bigint, tx?: any) => {
-        const execute = async (txFn: any) => {
+    deleteClass: async (id: bigint, tx?: Transaction) => {
+        return await withTransaction(async (txFn) => {
             await txFn.delete(classes).where(eq(classes.id, id));
-        };
-        return tx ? await execute(tx) : await DrizzleClient.transaction(execute);
+        }, tx);
     }
 };

src/modules/inventory/inventory.view.ts

@@ -0,0 +1,40 @@
+import { EmbedBuilder } from "discord.js";
+import type { ItemUsageData } from "@/lib/types";
+
+/**
+ * Inventory entry with item details
+ */
+interface InventoryEntry {
+    quantity: bigint | null;
+    item: {
+        id: number;
+        name: string;
+        [key: string]: any;
+    };
+}
+
+/**
+ * Creates an embed displaying a user's inventory
+ */
+export function getInventoryEmbed(items: InventoryEntry[], username: string): EmbedBuilder {
+    const description = items.map(entry => {
+        return `**${entry.item.name}** x${entry.quantity}`;
+    }).join("\n");
+
+    return new EmbedBuilder()
+        .setTitle(`📦 ${username}'s Inventory`)
+        .setDescription(description)
+        .setColor(0x3498db); // Blue
+}
+
+/**
+ * Creates an embed showing the results of using an item
+ */
+export function getItemUseResultEmbed(results: string[], itemName?: string): EmbedBuilder {
+    const description = results.map(r => `• ${r}`).join("\n");
+
+    return new EmbedBuilder()
+        .setTitle("✅ Item Used!")
+        .setDescription(description)
+        .setColor(0x2ecc71); // Green/Success
+}

src/modules/leveling/leveling.view.ts

@@ -0,0 +1,48 @@
+import { EmbedBuilder } from "discord.js";
+
+/**
+ * User data for leaderboard display
+ */
+interface LeaderboardUser {
+    username: string;
+    level: number | null;
+    xp: bigint | null;
+    balance: bigint | null;
+}
+
+/**
+ * Returns the appropriate medal emoji for a ranking position
+ */
+function getMedalEmoji(index: number): string {
+    if (index === 0) return "🥇";
+    if (index === 1) return "🥈";
+    if (index === 2) return "🥉";
+    return `${index + 1}.`;
+}
+
+/**
+ * Formats a single leaderboard entry based on type
+ */
+function formatLeaderEntry(user: LeaderboardUser, index: number, type: 'xp' | 'balance'): string {
+    const medal = getMedalEmoji(index);
+    const value = type === 'xp'
+        ? `Lvl ${user.level ?? 1} (${user.xp ?? 0n} XP)`
+        : `${user.balance ?? 0n} 🪙`;
+    return `${medal} **${user.username}** — ${value}`;
+}
+
+/**
+ * Creates a leaderboard embed for either XP or Balance rankings
+ */
+export function getLeaderboardEmbed(leaders: LeaderboardUser[], type: 'xp' | 'balance'): EmbedBuilder {
+    const description = leaders.map((user, index) =>
+        formatLeaderEntry(user, index, type)
+    ).join("\n");
+
+    const title = type === 'xp' ? "🏆 XP Leaderboard" : "💰 Richest Players";
+
+    return new EmbedBuilder()
+        .setTitle(title)
+        .setDescription(description)
+        .setColor(0xFFD700); // Gold
+}

src/modules/quest/quest.view.ts

@@ -0,0 +1,54 @@
+import { EmbedBuilder } from "discord.js";
+
+/**
+ * Quest entry with quest details and progress
+ */
+interface QuestEntry {
+    progress: number | null;
+    completedAt: Date | null;
+    quest: {
+        name: string;
+        description: string | null;
+        rewards: any;
+    };
+}
+
+/**
+ * Formats quest rewards object into a human-readable string
+ */
+function formatQuestRewards(rewards: { xp?: number, balance?: number }): string {
+    const rewardStr: string[] = [];
+    if (rewards?.xp) rewardStr.push(`${rewards.xp} XP`);
+    if (rewards?.balance) rewardStr.push(`${rewards.balance} 🪙`);
+    return rewardStr.join(", ");
+}
+
+/**
+ * Returns the quest status display string
+ */
+function getQuestStatus(completedAt: Date | null): string {
+    return completedAt ? "✅ Completed" : "📝 In Progress";
+}
+
+/**
+ * Creates an embed displaying a user's quest log
+ */
+export function getQuestListEmbed(userQuests: QuestEntry[]): EmbedBuilder {
+    const embed = new EmbedBuilder()
+        .setTitle("📜 Quest Log")
+        .setColor(0x3498db); // Blue
+
+    userQuests.forEach(entry => {
+        const status = getQuestStatus(entry.completedAt);
+        const rewards = entry.quest.rewards as { xp?: number, balance?: number };
+        const rewardsText = formatQuestRewards(rewards);
+
+        embed.addFields({
+            name: `${entry.quest.name} (${status})`,
+            value: `${entry.quest.description}\n**Rewards:** ${rewardsText}\n**Progress:** ${entry.progress}%`,
+            inline: false
+        });
+    });
+
+    return embed;
+}

src/modules/trade/trade.interaction.ts

@@ -7,7 +7,7 @@ import {
     TextChannel,
     EmbedBuilder
 } from "discord.js";
-import { TradeService } from "./trade.service";
+import { tradeService } from "./trade.service";
 import { inventoryService } from "@/modules/inventory/inventory.service";
 import { createErrorEmbed, createWarningEmbed, createSuccessEmbed, createInfoEmbed } from "@lib/embeds";
 import { getTradeDashboard, getTradeMoneyModal, getItemSelectMenu, getTradeCompletedEmbed } from "./trade.view";
@@ -55,10 +55,10 @@ export async function handleTradeInteraction(interaction: Interaction) {
 }
 
 async function handleCancel(interaction: ButtonInteraction | StringSelectMenuInteraction | ModalSubmitInteraction, threadId: string) {
-    const session = TradeService.getSession(threadId);
+    const session = tradeService.getSession(threadId);
     const user = interaction.user;
 
-    TradeService.endSession(threadId);
+    tradeService.endSession(threadId);
 
     await interaction.deferUpdate();
 
@@ -70,11 +70,11 @@ async function handleCancel(interaction: ButtonInteraction | StringSelectMenuInt
 
 async function handleLock(interaction: ButtonInteraction | StringSelectMenuInteraction | ModalSubmitInteraction, threadId: string) {
     await interaction.deferUpdate();
-    const isLocked = TradeService.toggleLock(threadId, interaction.user.id);
+    const isLocked = tradeService.toggleLock(threadId, interaction.user.id);
     await updateTradeDashboard(interaction, threadId);
 
     // Check if trade executed (both locked)
-    const session = TradeService.getSession(threadId);
+    const session = tradeService.getSession(threadId);
     if (session && session.state === 'COMPLETED') {
         // Trade executed during updateTradeDashboard
         return;
@@ -95,7 +95,7 @@ async function handleMoneySubmit(interaction: ModalSubmitInteraction, threadId:
 
     if (amount < 0n) throw new Error("Amount must be positive");
 
-    TradeService.updateMoney(threadId, interaction.user.id, amount);
+    tradeService.updateMoney(threadId, interaction.user.id, amount);
     await interaction.deferUpdate(); // Acknowledge modal
     await updateTradeDashboard(interaction, threadId);
 }
@@ -128,14 +128,14 @@ async function handleItemSelect(interaction: StringSelectMenuInteraction, thread
     const item = await inventoryService.getItem(itemId);
     if (!item) throw new Error("Item not found");
 
-    TradeService.addItem(threadId, interaction.user.id, { id: item.id, name: item.name }, 1n);
+    tradeService.addItem(threadId, interaction.user.id, { id: item.id, name: item.name }, 1n);
 
     await interaction.update({ content: `Added ${item.name} x1`, components: [] });
     await updateTradeDashboard(interaction, threadId);
 }
 
 async function handleRemoveItemClick(interaction: ButtonInteraction, threadId: string) {
-    const session = TradeService.getSession(threadId);
+    const session = tradeService.getSession(threadId);
     if (!session) return;
 
     const participant = session.userA.id === interaction.user.id ? session.userA : session.userB;
@@ -158,7 +158,7 @@ async function handleRemoveItemSelect(interaction: StringSelectMenuInteraction,
     const value = interaction.values[0];
     if (!value) return;
     const itemId = parseInt(value);
-    TradeService.removeItem(threadId, interaction.user.id, itemId);
+    tradeService.removeItem(threadId, interaction.user.id, itemId);
 
     await interaction.update({ content: `Removed item.`, components: [] });
     await updateTradeDashboard(interaction, threadId);
@@ -168,14 +168,14 @@ async function handleRemoveItemSelect(interaction: StringSelectMenuInteraction,
 // --- DASHBOARD UPDATER ---
 
 export async function updateTradeDashboard(interaction: Interaction, threadId: string) {
-    const session = TradeService.getSession(threadId);
+    const session = tradeService.getSession(threadId);
     if (!session) return;
 
     // Check Auto-Execute (If both locked)
     if (session.userA.locked && session.userB.locked) {
         // Execute Trade
         try {
-            await TradeService.executeTrade(threadId);
+            await tradeService.executeTrade(threadId);
             const embed = getTradeCompletedEmbed(session);
             await updateDashboardMessage(interaction, { embeds: [embed], components: [] });
 

src/modules/trade/trade.service.ts

@@ -1,143 +1,25 @@
 import type { TradeSession, TradeParticipant } from "./trade.types";
-import { DrizzleClient } from "@/lib/DrizzleClient";
 import { economyService } from "@/modules/economy/economy.service";
 import { inventoryService } from "@/modules/inventory/inventory.service";
 import { itemTransactions } from "@/db/schema";
+import { withTransaction } from "@/lib/db";
 import type { Transaction } from "@/lib/types";
 
-export class TradeService {
-    private static sessions = new Map<string, TradeSession>();
+// Module-level session storage
+const sessions = new Map<string, TradeSession>();
 
-    /**
-     * Creates a new trade session
+/**
+ * Unlocks both participants in a trade session
  */
-    static createSession(threadId: string, userA: { id: string, username: string }, userB: { id: string, username: string }): TradeSession {
-        const session: TradeSession = {
-            threadId,
-            userA: {
-                id: userA.id,
-                username: userA.username,
-                locked: false,
-                offer: { money: 0n, items: [] }
-            },
-            userB: {
-                id: userB.id,
-                username: userB.username,
-                locked: false,
-                offer: { money: 0n, items: [] }
-            },
-            state: 'NEGOTIATING',
-            lastInteraction: Date.now()
-        };
-
-        this.sessions.set(threadId, session);
-        return session;
-    }
-
-    static getSession(threadId: string): TradeSession | undefined {
-        return this.sessions.get(threadId);
-    }
-
-    static endSession(threadId: string) {
-        this.sessions.delete(threadId);
-    }
-
-    /**
-     * Updates an offer. If allowed, validation checks should be done BEFORE calling this.
-     * unlocking logic is handled here (if offer changes, unlock both).
-     */
-    static updateMoney(threadId: string, userId: string, amount: bigint) {
-        const session = this.getSession(threadId);
-        if (!session) throw new Error("Session not found");
-        if (session.state !== 'NEGOTIATING') throw new Error("Trade is not active");
-
-        const participant = session.userA.id === userId ? session.userA : session.userB.id === userId ? session.userB : null;
-        if (!participant) throw new Error("User not in trade");
-
-        participant.offer.money = amount;
-        this.unlockAll(session);
-        session.lastInteraction = Date.now();
-    }
-
-    static addItem(threadId: string, userId: string, item: { id: number, name: string }, quantity: bigint) {
-        const session = this.getSession(threadId);
-        if (!session) throw new Error("Session not found");
-        if (session.state !== 'NEGOTIATING') throw new Error("Trade is not active");
-
-        const participant = session.userA.id === userId ? session.userA : session.userB.id === userId ? session.userB : null;
-        if (!participant) throw new Error("User not in trade");
-
-        const existing = participant.offer.items.find(i => i.id === item.id);
-        if (existing) {
-            existing.quantity += quantity;
-        } else {
-            participant.offer.items.push({ id: item.id, name: item.name, quantity });
-        }
-
-        this.unlockAll(session);
-        session.lastInteraction = Date.now();
-    }
-
-    static removeItem(threadId: string, userId: string, itemId: number) {
-        const session = this.getSession(threadId);
-        if (!session) throw new Error("Session not found");
-
-        const participant = session.userA.id === userId ? session.userA : session.userB.id === userId ? session.userB : null;
-        if (!participant) throw new Error("User not in trade");
-
-        participant.offer.items = participant.offer.items.filter(i => i.id !== itemId);
-
-        this.unlockAll(session);
-        session.lastInteraction = Date.now();
-    }
-
-    static toggleLock(threadId: string, userId: string): boolean {
-        const session = this.getSession(threadId);
-        if (!session) throw new Error("Session not found");
-
-        const participant = session.userA.id === userId ? session.userA : session.userB.id === userId ? session.userB : null;
-        if (!participant) throw new Error("User not in trade");
-
-        participant.locked = !participant.locked;
-        session.lastInteraction = Date.now();
-
-        return participant.locked;
-    }
-
-    private static unlockAll(session: TradeSession) {
+const unlockAll = (session: TradeSession) => {
     session.userA.locked = false;
     session.userB.locked = false;
-    }
+};
 
-    /**
-     * Executes the trade atomically.
-     * 1. Validates balances/inventory for both users.
-     * 2. Swaps money.
-     * 3. Swaps items.
-     * 4. Logs transactions.
+/**
+ * Processes a one-way transfer from one participant to another
  */
-    static async executeTrade(threadId: string): Promise<void> {
-        const session = this.getSession(threadId);
-        if (!session) throw new Error("Session not found");
-
-        if (!session.userA.locked || !session.userB.locked) {
-            throw new Error("Both players must accept the trade first.");
-        }
-
-        session.state = 'COMPLETED'; // Prevent double execution
-
-        await DrizzleClient.transaction(async (tx) => {
-            // -- Validate & Execute User A -> User B --
-            await this.processTransfer(tx, session.userA, session.userB, session.threadId);
-
-            // -- Validate & Execute User B -> User A --
-            await this.processTransfer(tx, session.userB, session.userA, session.threadId);
-        });
-
-        this.endSession(threadId);
-    }
-
-    private static async processTransfer(tx: Transaction, from: TradeParticipant, to: TradeParticipant, threadId: string) {
+const processTransfer = async (tx: Transaction, from: TradeParticipant, to: TradeParticipant, threadId: string) => {
     // 1. Money
     if (from.offer.money > 0n) {
         await economyService.modifyUserBalance(
@@ -186,5 +68,130 @@ export class TradeService {
             description: `Received from ${from.username}`,
         });
     }
+};
+
+export const tradeService = {
+    /**
+     * Creates a new trade session
+     */
+    createSession: (threadId: string, userA: { id: string, username: string }, userB: { id: string, username: string }): TradeSession => {
+        const session: TradeSession = {
+            threadId,
+            userA: {
+                id: userA.id,
+                username: userA.username,
+                locked: false,
+                offer: { money: 0n, items: [] }
+            },
+            userB: {
+                id: userB.id,
+                username: userB.username,
+                locked: false,
+                offer: { money: 0n, items: [] }
+            },
+            state: 'NEGOTIATING',
+            lastInteraction: Date.now()
+        };
+
+        sessions.set(threadId, session);
+        return session;
+    },
+
+    getSession: (threadId: string): TradeSession | undefined => {
+        return sessions.get(threadId);
+    },
+
+    endSession: (threadId: string) => {
+        sessions.delete(threadId);
+    },
+
+    /**
+     * Updates an offer. If allowed, validation checks should be done BEFORE calling this.
+     * unlocking logic is handled here (if offer changes, unlock both).
+     */
+    updateMoney: (threadId: string, userId: string, amount: bigint) => {
+        const session = tradeService.getSession(threadId);
+        if (!session) throw new Error("Session not found");
+        if (session.state !== 'NEGOTIATING') throw new Error("Trade is not active");
+
+        const participant = session.userA.id === userId ? session.userA : session.userB.id === userId ? session.userB : null;
+        if (!participant) throw new Error("User not in trade");
+
+        participant.offer.money = amount;
+        unlockAll(session);
+        session.lastInteraction = Date.now();
+    },
+
+    addItem: (threadId: string, userId: string, item: { id: number, name: string }, quantity: bigint) => {
+        const session = tradeService.getSession(threadId);
+        if (!session) throw new Error("Session not found");
+        if (session.state !== 'NEGOTIATING') throw new Error("Trade is not active");
+
+        const participant = session.userA.id === userId ? session.userA : session.userB.id === userId ? session.userB : null;
+        if (!participant) throw new Error("User not in trade");
+
+        const existing = participant.offer.items.find(i => i.id === item.id);
+        if (existing) {
+            existing.quantity += quantity;
+        } else {
+            participant.offer.items.push({ id: item.id, name: item.name, quantity });
         }
-}
+
+        unlockAll(session);
+        session.lastInteraction = Date.now();
+    },
+
+    removeItem: (threadId: string, userId: string, itemId: number) => {
+        const session = tradeService.getSession(threadId);
+        if (!session) throw new Error("Session not found");
+
+        const participant = session.userA.id === userId ? session.userA : session.userB.id === userId ? session.userB : null;
+        if (!participant) throw new Error("User not in trade");
+
+        participant.offer.items = participant.offer.items.filter(i => i.id !== itemId);
+
+        unlockAll(session);
+        session.lastInteraction = Date.now();
+    },
+
+    toggleLock: (threadId: string, userId: string): boolean => {
+        const session = tradeService.getSession(threadId);
+        if (!session) throw new Error("Session not found");
+
+        const participant = session.userA.id === userId ? session.userA : session.userB.id === userId ? session.userB : null;
+        if (!participant) throw new Error("User not in trade");
+
+        participant.locked = !participant.locked;
+        session.lastInteraction = Date.now();
+
+        return participant.locked;
+    },
+
+    /**
+     * Executes the trade atomically.
+     * 1. Validates balances/inventory for both users.
+     * 2. Swaps money.
+     * 3. Swaps items.
+     * 4. Logs transactions.
+     */
+    executeTrade: async (threadId: string): Promise<void> => {
+        const session = tradeService.getSession(threadId);
+        if (!session) throw new Error("Session not found");
+
+        if (!session.userA.locked || !session.userB.locked) {
+            throw new Error("Both players must accept the trade first.");
+        }
+
+        session.state = 'COMPLETED'; // Prevent double execution
+
+        await withTransaction(async (tx) => {
+            // -- Validate & Execute User A -> User B --
+            await processTransfer(tx, session.userA, session.userB, session.threadId);
+
+            // -- Validate & Execute User B -> User A --
+            await processTransfer(tx, session.userB, session.userA, session.threadId);
+        });
+
+        tradeService.endSession(threadId);
+    }
+};