Repository navigation
Read capability files with bounded concurrency - #618
Merged
SaladDay merged 2 commits intoOct 9, 2026
Merged
Conversation
SaladDay
marked this pull request as ready for review
October 9, 2026 11:44
SaladDay
deleted the
aos/read-capability-files-with-bounded-concurrency
branch
October 9, 2026 11:45
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
1 participant
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.
Environment capability snapshots read each file sequentially, adding avoidable File service round trips to initialization. Enumerate and validate the tree serially, then read at most four retained files concurrently within the service's declared handle limit while preserving output order, total size limits and all published integrity readback. Join all reads before reference cleanup; use bounded handle release after cancellation and fail the attachment on an unconfirmed release.
The original composition fixture retains all 393 RPCs. Fixed 20/100 ms handler delays decrease snapshot time from 7.995/39.481 seconds to 5.956/29.254 seconds. Zero-delay samples overlap, and this does not establish live micro qualification.
Validation: real File-service concurrency/error/cancellation/tampering tests, race ×50, agenthost and capability packages (87 passed, 6 explicit native prerequisite skips), daemon build, vet and naming/docs/CI checks. Independent review found one low-limit service regression, fixed with old-fails/new-passes coverage for declared limits of 1–3. Coordinator reviewed the full follow-up; real integrated lifecycle and strong MCP cancellation acceptance remain outstanding.