Fix config environment variable evaluation and secret key validation tests

Co-authored-by: unixfox <[email protected]>

Author: Copilot

src/lib/helpers/config.ts

@@ -12,12 +12,16 @@ export const ConfigSchema = z.object({
             Deno.env.get("SERVER_UNIX_SOCKET_PATH") ||
                 "/tmp/invidious-companion.sock",
         ),
-        secret_key: z.string().length(16).regex(
-            /^[a-zA-Z0-9]+$/,
-            "SERVER_SECRET_KEY contains invalid characters. Only alphanumeric characters (a-z, A-Z, 0-9) are allowed. Please generate a valid key using 'pwgen 16 1' or ensure your key contains only letters and numbers.",
-        ).default(
-            Deno.env.get("SERVER_SECRET_KEY") || "",
-        ),
+        secret_key: z.preprocess(
+            (val) =>
+                val === undefined
+                    ? Deno.env.get("SERVER_SECRET_KEY") || ""
+                    : val,
+            z.string().length(16).regex(
+                /^[a-zA-Z0-9]+$/,
+                "SERVER_SECRET_KEY contains invalid characters. Only alphanumeric characters (a-z, A-Z, 0-9) are allowed. Please generate a valid key using 'pwgen 16 1' or ensure your key contains only letters and numbers.",
+            ),
+        ).default(undefined),
         verify_requests: z.boolean().default(
             Deno.env.get("SERVER_VERIFY_REQUESTS") === "true" || false,
         ),
@@ -133,7 +137,13 @@ export async function parseConfig() {
         console.log(errorMessage);
         if (err instanceof ZodError) {
             console.log(err.issues);
-            throw new Error("Failed to parse configuration file");
+            // Include detailed error information in the thrown error for testing
+            const detailedMessage = err.issues.map((issue) =>
+                `${issue.path.join(".")}: ${issue.message}`
+            ).join("; ");
+            throw new Error(
+                `Failed to parse configuration file: ${detailedMessage}`,
+            );
         }
         // rethrow error if not Zod
         throw err;

src/tests/secret_key_validation_test.ts

@@ -9,7 +9,7 @@ import { parseConfig } from "../lib/helpers/config.ts";
 Deno.test("Secret key validation in Invidious companion config", async (t) => {
     // Clean up any existing environment variables that might interfere
     Deno.env.delete("SERVER_SECRET_KEY");
-    
+
     // Disable PO token generation for testing to avoid network calls
     Deno.env.set("JOBS_YOUTUBE_SESSION_PO_TOKEN_ENABLED", "false");
 
@@ -25,10 +25,6 @@ Deno.test("Secret key validation in Invidious companion config", async (t) => {
         for (const key of validKeys) {
             // Set the environment variable for each test
             Deno.env.set("SERVER_SECRET_KEY", key);
-            
-            // Debug: Check what environment variable is actually set
-            const actualEnvValue = Deno.env.get("SERVER_SECRET_KEY");
-            console.log(`Debug: Setting key "${key}", actual env value: "${actualEnvValue}"`);
 
             try {
                 const config = await parseConfig();