feat: add MCP logging, split docs clone, and parallelize sync#7
Merged
critesjosh merged 3 commits intomainfrom Mar 3, 2026
Merged
feat: add MCP logging, split docs clone, and parallelize sync#7critesjosh merged 3 commits intomainfrom
critesjosh merged 3 commits intomainfrom
Conversation
…ility Adds structured logging throughout the clone/update lifecycle so MCP clients can surface progress messages (repo-by-repo status, git fetch/checkout stages, sparse-checkout paths, and error details). Also includes the gc.auto=0 fix for blobless clones to prevent the race condition. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
87a24ff to
b340300
Compare
- 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>
Instead of overlaying docs from the `next` branch into the blobless tag clone of aztec-packages (which triggered slow one-by-one lazy blob fetching or SIGSEGV), create a separate shallow clone (`aztec-packages-docs`) for the docs paths. All independent repos now clone in parallel via Promise.all after the blocking aztec-packages clone completes. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
|
🎉 This PR is included in version 1.10.0 🎉 The release is available on: Your semantic-release bot 📦🚀 |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
server.sendLoggingMessage) throughout the sync/clone/update lifecycle so clients get real-time progress visibilitygc.auto=0fix for blobless clones to prevent the gc race condition (previously onfix/gc-race-conditionbranch)sparsePathOverridesinto a separate clone: instead of overlayingnextbranch content into the blobless tag clone ofaztec-packages(which triggered slow one-by-one lazy blob fetching or SIGSEGV for ~2600 files), creates a separate shallow clone (aztec-packages-docs) for the docs pathsaztec-packagesclone + noir commit extraction, all remaining repos (noir, noir-examples, aztec-examples, aztec-starter, demo-wallet, gregoswap, aztec-packages-docs) clone in parallel viaPromise.allTest plan
aztec_sync_repos --forceand verify all 8 repos clone successfullyaztec-packagesclones at tag without sparsePathOverrides overlay — no SIGSEGV or slow blob fetchingaztec-packages-docsclones separately fromnextbranch with docs sparse pathssearchAztecDocsreturns results fromaztec-packages-docs🤖 Generated with Claude Code