chore: Update/remove comments

Author: PartMan7

src/database/points.ts

@@ -32,9 +32,7 @@ const schema = new mongoose.Schema<MapModel>({
 	},
 });
 
-export const model = mongoose.model<MapModel>('point', schema, 'points', {
-	overwriteModels: true,
-});
+export const model = mongoose.model<MapModel>('point', schema, 'points', { overwriteModels: true });
 
 const DEFAULT_LB_USERS_CAP = 50;
 

src/database/seens.ts

@@ -1,5 +1,6 @@
 import mongoose from 'mongoose';
 
+import { IS_ENABLED } from '@/enabled';
 import { toId } from '@/tools';
 
 // We don't really care about storing joins (since we can just use their 'online' status)
@@ -36,16 +37,19 @@ const schema = new mongoose.Schema<Model>({
 
 const model = mongoose.model<Model>('seen', schema, 'seens', { overwriteModels: true });
 
-export function seeUser(user: string, rooms: string[] = [], at = new Date()): Promise<Model> {
+export async function seeUser(user: string, rooms: string[] = [], at = new Date()): Promise<Model | null> {
+	if (!IS_ENABLED.DB) return null;
 	const userId = toId(user);
 	return model.findOneAndUpdate({ id: userId }, { id: userId, name: user, seenIn: rooms, at }, { upsert: true, new: true });
 }
 
-export function lastSeen(user: string): Promise<Model | null> {
+export async function lastSeen(user: string): Promise<Model | null> {
+	if (!IS_ENABLED.DB) return null;
 	const userId = toId(user);
 	return model.findOne({ id: userId });
 }
 
-export function fetchAllSeens(): Promise<Model[]> {
+export async function fetchAllSeens(): Promise<Model[]> {
+	if (!IS_ENABLED.DB) return [];
 	return model.find({}).lean();
 }

src/ps/handlers/commands/index.ts

@@ -116,6 +116,7 @@ export async function commandHandler(message: PSMessage, indirect: IndirectCtx |
 		return await commandObj.run({ ...context, message });
 	} catch (err) {
 		if (err instanceof Error) {
+			// TODO: Ping the user in case they're in another room! (Eg: for spoof messages)
 			message.privateReply(err.message as string);
 			if (err.name !== 'ChatError') Logger.errorLog(err);
 		} else {

src/ps/handlers/joins.ts

@@ -29,8 +29,6 @@ export function nickHandler(this: Client, room: string, newName: string, oldName
 		to = toId(newName),
 		id = `${from}-${to}`;
 	if (from === to) return;
-	// Throttling cache updates at once per 5s per rename (A-B)
-	// TODO: Use debounce
 	DebounceAltCache[id] ??= {
 		at: new Date(),
 		call: debounce(() => rename(oldName, newName), fromHumanTime('5 seconds')),
@@ -44,11 +42,10 @@ const DebounceSeenCache: Record<string, { name: string; at: Date; call: (rooms:
 export function leaveHandler(this: Client, room: string, name: string, isIntro: boolean): void {
 	if (isIntro) return;
 	const userId = toId(name);
-	// Throttling cache updates at once per 5s per leave
 	DebounceSeenCache[userId] ??= {
 		name,
 		at: new Date(),
-		call: debounce(rooms => seeUser(name, rooms, DebounceSeenCache[userId].at), fromHumanTime('5 seconds')),
+		call: debounce((rooms: string[]) => seeUser(name, rooms.unique(), DebounceSeenCache[userId].at), fromHumanTime('5 seconds')),
 	};
 	DebounceSeenCache[userId].at = new Date();
 	const userObj = this.getUser(name);