Merge pull request #13 from AztecProtocol/feat/broaden-sparse-checkout-paths

feat: broaden sparse checkout paths and add --skip-checks flag

Author: critesjosh

src/tools/sync.ts

@@ -9,16 +9,22 @@ import { cloneRepo, getReposStatus, getNoirCommitFromAztec, getRepoPath, REPOS_D
 import { writeSyncMetadata, stampMetadataMcpVersion, readSyncMetadata, SyncMetadata } from "../utils/sync-metadata.js";
 import { clearErrorCache } from "../utils/error-lookup.js";
 
+export interface RepoSyncStatus {
+  name: string;
+  status: string;
+  commit?: string;
+}
+
+export function isRepoError(repo: RepoSyncStatus): boolean {
+  return repo.status.startsWith("Error:");
+}
+
 export interface SyncResult {
   success: boolean;
   metadataSafe: boolean;
   message: string;
   version: string;
-  repos: {
-    name: string;
-    status: string;
-    commit?: string;
-  }[];
+  repos: RepoSyncStatus[];
 }
 
 /**
@@ -105,7 +111,7 @@ export async function syncRepos(options: {
   // leaves the old checkout while other repos sync to the new tag, producing a
   // mixed-version workspace.
   const aztecFailed = results.some(
-    (r) => r.name === "aztec-packages" && r.status.toLowerCase().includes("error"),
+    (r) => r.name === "aztec-packages" && isRepoError(r),
   );
   if (aztecFailed && (force || version)) {
     return {
@@ -160,7 +166,7 @@ export async function syncRepos(options: {
   let versionedDocsMissing = false;
   for (const repo of syntheticRepos) {
     const result = results.find((r) => r.name === repo.name);
-    if (!result || result.status.toLowerCase().includes("error")) continue;
+    if (!result || isRepoError(result)) continue;
 
     for (const sparsePath of repo.sparse || []) {
       if (!sparsePath.includes(effectiveVersion)) continue;
@@ -174,7 +180,7 @@ export async function syncRepos(options: {
   }
 
   const allSuccess = results.every(
-    (r) => !r.status.toLowerCase().includes("error")
+    (r) => !isRepoError(r)
   );
 
   log?.(`Sync complete: ${results.length} repos, ${allSuccess ? "all succeeded" : "some failed"}`, "info");

src/utils/format.ts

@@ -2,7 +2,7 @@
  * Formatting utilities for MCP tool responses
  */
 
-import type { SyncResult } from "../tools/sync.js";
+import { isRepoError, type SyncResult } from "../tools/sync.js";
 import type { SearchResult, FileInfo } from "./search.js";
 import type { SyncMetadata } from "./sync-metadata.js";
 import type { ErrorLookupResult } from "./error-lookup.js";
@@ -18,7 +18,7 @@ export function formatSyncResult(result: SyncResult): string {
   ];
 
   for (const repo of result.repos) {
-    const icon = repo.status.toLowerCase().includes("error") ? "✗" : "✓";
+    const icon = isRepoError(repo) ? "✗" : "✓";
     lines.push(`  ${icon} ${repo.name}: ${repo.status}`);
   }