fix: use startsWith for error status checks instead of case-insensitive includes

Error statuses are always formatted as "Error: ...", so startsWith is
more precise and avoids false positives from status messages that
happen to contain the word "error".

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

Author: critesjosh

src/tools/sync.ts

@@ -105,7 +105,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" && r.status.startsWith("Error:"),
   );
   if (aztecFailed && (force || version)) {
     return {
@@ -160,7 +160,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 || result.status.startsWith("Error:")) continue;
 
     for (const sparsePath of repo.sparse || []) {
       if (!sparsePath.includes(effectiveVersion)) continue;
@@ -174,7 +174,7 @@ export async function syncRepos(options: {
   }
 
   const allSuccess = results.every(
-    (r) => !r.status.toLowerCase().includes("error")
+    (r) => !r.status.startsWith("Error:")
   );
 
   log?.(`Sync complete: ${results.length} repos, ${allSuccess ? "all succeeded" : "some failed"}`, "info");

src/utils/format.ts

@@ -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 = repo.status.startsWith("Error:") ? "✗" : "✓";
     lines.push(`  ${icon} ${repo.name}: ${repo.status}`);
   }