Refactor config command to utilize exported parsing functions

Author: ysdragon

src/commands/config.ts

@@ -1,6 +1,6 @@
 import { BaseCommand } from "./base.js";
 import { CommandContext } from "../types/index.js";
-import config from "../config.js";
+import config, { parseBoolean, parseVideoCodec, parsePreset } from "../config.js";
 import logger from "../utils/logger.js";
 
 export default class ConfigCommand extends BaseCommand {
@@ -83,7 +83,7 @@ export default class ConfigCommand extends BaseCommand {
 				case 'respect_video_params':
 				case 'hardwareacceleration':
 				case 'hardwareaccelerateddecoding':
-					const boolValue = this.parseBoolean(value);
+					const boolValue = parseBoolean(value);
 					if (param === 'hardwareacceleration' || param === 'hardwareaccelerateddecoding') {
 						config.hardwareAcceleratedDecoding = boolValue;
 						await this.sendSuccess(context.message, `Set hardwareAcceleratedDecoding to \`${boolValue}\``);
@@ -112,7 +112,7 @@ export default class ConfigCommand extends BaseCommand {
 
 				// Video codec
 				case 'videocodec':
-					const codec = this.parseVideoCodec(value);
+					const codec = parseVideoCodec(value);
 					if (!codec) {
 						await this.sendError(context.message, `Invalid video codec. Valid options: VP8, H264, H265`);
 						return;
@@ -125,7 +125,7 @@ export default class ConfigCommand extends BaseCommand {
 				// H26x preset
 				case 'h26xpreset':
 				case 'preset':
-					const preset = this.parsePreset(value);
+					const preset = parsePreset(value);
 					if (!preset) {
 						await this.sendError(context.message, `Invalid preset. Valid options: ultrafast, superfast, veryfast, faster, fast, medium, slow, slower, veryslow`);
 						return;
@@ -171,29 +171,6 @@ export default class ConfigCommand extends BaseCommand {
 		];
 	}
 
-	private parseBoolean(value: string): boolean {
-		const val = value.trim().toLowerCase();
-		return val === 'true' || val === '1' || val === 'yes' || val === 'on';
-	}
-
-	private parseVideoCodec(value: string): "VP8" | "H264" | "H265" | null {
-		const VALID_VIDEO_CODECS = ['VP8', 'H264', 'H265'];
-		const codec = value.trim().toUpperCase();
-		if (VALID_VIDEO_CODECS.includes(codec)) {
-			return codec as "VP8" | "H264" | "H265";
-		}
-		return null;
-	}
-
-	private parsePreset(value: string): "ultrafast" | "superfast" | "veryfast" | "faster" | "fast" | "medium" | "slow" | "slower" | "veryslow" | null {
-		const VALID_PRESETS = ["ultrafast", "superfast", "veryfast", "faster", "fast", "medium", "slow", "slower", "veryslow"];
-		const preset = value.trim().toLowerCase();
-		if (VALID_PRESETS.includes(preset)) {
-			return preset as "ultrafast" | "superfast" | "veryfast" | "faster" | "fast" | "medium" | "slow" | "slower" | "veryslow";
-		}
-		return null;
-	}
-
 	private isAdmin(userId: string): boolean {
 		// If no admins configured, allow all users (backwards compatibility)
 		if (!config.adminIds || config.adminIds.length === 0) {

src/config.ts

@@ -5,7 +5,7 @@ dotenv.config({ quiet: true });
 
 const VALID_VIDEO_CODECS = ['VP8', 'H264', 'H265', 'VP9', 'AV1'];
 
-function parseVideoCodec(value: string): "VP8" | "H264" | "H265" {
+export function parseVideoCodec(value: string): "VP8" | "H264" | "H265" {
 	if (typeof value === "string") {
 		value = value.trim().toUpperCase();
 	}
@@ -15,7 +15,7 @@ function parseVideoCodec(value: string): "VP8" | "H264" | "H265" {
 	return "H264";
 }
 
-function parsePreset(value: string): "ultrafast" | "superfast" | "veryfast" | "faster" | "fast" | "medium" | "slow" | "slower" | "veryslow" {
+export function parsePreset(value: string): "ultrafast" | "superfast" | "veryfast" | "faster" | "fast" | "medium" | "slow" | "slower" | "veryslow" {
 	if (typeof value === "string") {
 		value = value.trim().toLowerCase();
 	}
@@ -35,7 +35,7 @@ function parsePreset(value: string): "ultrafast" | "superfast" | "veryfast" | "f
 	}
 }
 
-function parseBoolean(value: string | undefined): boolean {
+export function parseBoolean(value: string | undefined): boolean {
 	if (typeof value === "string") {
 		value = value.trim().toLowerCase();
 	}