fix(mcp): move webhooks init before createFeedbackStore to fix TDZ crash

Declare `categories` and `webhooks` before passing `webhooks` into
`createFeedbackStore`, avoiding a TDZ ReferenceError at MCP startup.

Also type-narrow the catch clause in `fireWebhook` (webhook.ts) instead
of using `any`, as required under strict mode.

Author: igmagollo

src/mcp.ts

@@ -22,6 +22,8 @@ export async function startMcpServer(): Promise<void> {
   const embed = await createEmbedder();
 
   const sessionId = randomUUID();
+  const categories = getCategories();
+  const webhooks = getWebhooks();
   const store = createFeedbackStore({
     dbPath,
     sessionId,
@@ -31,9 +33,6 @@ export async function startMcpServer(): Promise<void> {
   });
 
   await store.init();
-
-  const categories = getCategories();
-  const webhooks = getWebhooks();
   const submitFeedbackSchema = createSubmitFeedbackSchema(categories);
   const listFeedbackSchema = createListFeedbackSchema(categories);
 

src/webhook.ts

@@ -50,8 +50,9 @@ export async function fireWebhook(url: string, payload: Record<string, unknown>)
         `[suggestion-box] webhook POST to ${url} failed: HTTP ${response.status}`
       );
     }
-  } catch (e: any) {
-    console.error(`[suggestion-box] webhook POST to ${url} error: ${e.message}`);
+  } catch (e) {
+    const message = e instanceof Error ? e.message : String(e);
+    console.error(`[suggestion-box] webhook POST to ${url} error: ${message}`);
   }
 }