fix: handle logging edge cases from review feedback

- Catch sendLoggingMessage rejections to prevent unhandled promise
  rejections when the client disconnects during sync
- Only pass logger arg to cloneRepo when defined, preserving the
  two-argument call contract for callers without logging

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>

Author: critesjosh

src/index.ts

@@ -203,7 +203,7 @@ server.setRequestHandler(CallToolRequestSchema, async (request) => {
             level: level as "info" | "debug" | "warning" | "error",
             logger: "aztec-sync",
             data: message,
-          });
+          }).catch(() => {});
         };
         const result = await syncRepos({
           version: args?.version as string | undefined,

src/tools/sync.ts

@@ -55,7 +55,7 @@ export async function syncRepos(options: {
     syncIndex++;
     log?.(`Syncing ${syncIndex}/${reposToSync.length}: ${config.name}`, "info");
     try {
-      const status = await cloneRepo(config, force, log);
+      const status = log ? await cloneRepo(config, force, log) : await cloneRepo(config, force);
       results.push({ name: config.name, status: statusTransform ? statusTransform(status) : status });
     } catch (error) {
       log?.(`${config.name}: Failed: ${error instanceof Error ? error.message : String(error)}`, "error");