feat(check): configurable check concurrency, defaulting to CPU core count#148
Merged
Conversation
The anthropic-beta header fix (MULTI-1367) is now upstreamed to the wack/cersei fork's trunk branch, so the local third_party/cersei-provider vendor and [patch.crates-io] override are no longer needed. Point cersei-provider, cersei-agent, cersei-tools, and cersei-types at the git fork directly, matching the existing multitool-sdk dependency pattern. Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
Mirrors hare/CLAUDE.md's guidance: prefer cargo make clippy/test over raw cargo invocations, since only the Makefile.toml tasks run --locked and match what CI and bacon actually execute. Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
Contributor
Author
This stack of pull requests is managed by Graphite. Learn more about stacking. |
v2025_08_07 is now deprecated (superseded by v2026_01_12), which clippy's -D warnings gate flags as an error under a newer aws-config resolved by the cersei-provider fork update.
…ount The hardcoded concurrency cap of 2 left most machines' cores idle during `multi check` runs. Add a `--concurrency` flag (layered flag > env > file, like --provider/--model/--effort/--executor) that plumbs through to the execution actor's semaphore, validated to be greater than 0. Default it to std::thread::available_parallelism() instead of a fixed constant, and add a CHECKS.md requirement asserting the default tracks the host's core count. Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
RobbieMcKinstry
force-pushed
the
robbie/concurrency
branch
from
f494530 to
89db22c
Compare
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.
The hardcoded concurrency cap of 2 left most machines' cores idle during
multi checkruns. Add a--concurrencyflag (layered flag > env >file, like --provider/--model/--effort/--executor) that plumbs through
to the execution actor's semaphore, validated to be greater than 0.
Default it to std::thread::available_parallelism() instead of a fixed
constant, and add a CHECKS.md requirement asserting the default tracks
the host's core count.
Co-Authored-By: Claude Sonnet 5 noreply@anthropic.com