chore(google-tag-manager): remove debug logs

- Remove all console.log statements from gtm-client.ts
- Remove all console.log statements from accounts.ts
- Clean code ready for production deployment

Author: JonasJesus42

google-tag-manager/server/lib/gtm-client.ts

@@ -42,8 +42,6 @@ export class GTMClient {
    * Make a request to the GTM API
    */
   private async request<T>(url: string, options: RequestInit = {}): Promise<T> {
-    console.log(`[GTM API Request] ${options.method || "GET"} ${url}`);
-
     const response = await fetch(url, {
       ...options,
       headers: {
@@ -53,33 +51,19 @@ export class GTMClient {
       },
     });
 
-    console.log(
-      `[GTM API Response] Status: ${response.status} ${response.statusText}`,
-    );
-
     if (!response.ok) {
       const error = await response.text();
-      console.error(`[GTM API Error] ${response.status} - ${error}`);
       throw new Error(
         `Google Tag Manager API error: ${response.status} - ${error}`,
       );
     }
 
     // Handle 204 No Content
     if (response.status === 204) {
-      console.log(`[GTM API] 204 No Content - returning empty object`);
       return {} as T;
     }
 
-    const data = await response.json();
-    console.log(
-      `[GTM API Data] Response type: ${Array.isArray(data) ? "Array" : typeof data}`,
-    );
-    if (typeof data === "object" && data !== null) {
-      console.log(`[GTM API Data] Keys:`, Object.keys(data));
-    }
-
-    return data as T;
+    return (await response.json()) as T;
   }
 
   // ==================== Account Methods ====================

google-tag-manager/server/tools/accounts.ts

@@ -44,54 +44,21 @@ export const createListAccountsTool = (env: Env) =>
       nextPageToken: z.string().optional(),
     }),
     execute: async ({ context }) => {
-      console.log(
-        "[list_accounts] Starting with pageToken:",
-        context.pageToken,
-      );
-
       const client = new GTMClient({
         accessToken: getAccessToken(env),
       });
 
       const response = await client.listAccounts(context.pageToken);
 
-      console.log("[list_accounts] Raw response structure:");
-      console.log("- Type:", typeof response);
-      console.log("- Keys:", Object.keys(response));
-      console.log("- Has 'account' property:", "account" in response);
-      console.log(
-        "- 'account' type:",
-        Array.isArray(response.account) ? "Array" : typeof response.account,
-      );
-      console.log("- 'account' length:", response.account?.length);
-
-      if (response.account && response.account.length > 0) {
-        console.log("[list_accounts] First account structure:");
-        console.log(JSON.stringify(response.account[0], null, 2));
-      }
-
-      const accounts = (response.account || []).map((acc) => {
-        console.log(
-          "[list_accounts] Processing account:",
-          acc.accountId,
-          acc.name,
-        );
-        return {
-          path: acc.path,
-          accountId: acc.accountId,
-          name: acc.name,
-          ...(acc.shareData !== undefined && { shareData: acc.shareData }),
-          ...(acc.fingerprint && { fingerprint: acc.fingerprint }),
-          ...(acc.tagManagerUrl && { tagManagerUrl: acc.tagManagerUrl }),
-          ...(acc.features && { features: acc.features }),
-        };
-      });
-
-      console.log(
-        "[list_accounts] Returning result with",
-        accounts.length,
-        "accounts",
-      );
+      const accounts = (response.account || []).map((acc) => ({
+        path: acc.path,
+        accountId: acc.accountId,
+        name: acc.name,
+        ...(acc.shareData !== undefined && { shareData: acc.shareData }),
+        ...(acc.fingerprint && { fingerprint: acc.fingerprint }),
+        ...(acc.tagManagerUrl && { tagManagerUrl: acc.tagManagerUrl }),
+        ...(acc.features && { features: acc.features }),
+      }));
 
       return {
         accounts,