feat(fmt): enable parallel formatting - #132
Conversation
Deploying rstack-cli with
|
| Latest commit: |
d191590
|
| Status: | ✅ Deploy successful! |
| Preview URL: | https://fb4b85a9.rstack-cli.pages.dev |
| Branch Preview URL: | https://chenjiahan-feat-fmt-parallel.rstack-cli.pages.dev |
![chatgpt-codex-connector[bot]](https://avatars.githubusercontent.com/in/1144995?s=60&v=4){kind=small}
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 6bb7addffc
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
| FROM, OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER | ||
| DEALINGS IN THE SOFTWARE. | ||
|
|
||
| ## isotimer |
There was a problem hiding this comment.
The published formatter bundle now includes isotimer, whose newly added dependency chain includes immediato@1.1.0 (pnpm-lock.yaml:3419-3421), but this notice lists the other WorkTank transitive packages without including immediato. Because its MIT license requires its copyright and permission notice to accompany substantial distributed copies, add immediato’s license entry before publishing the bundled package.
Useful? React with 👍 / 👎.
1 participant
Running
rs fmtserially leaves available CPU capacity unused on multi-file workloads. This PR enables worker-based parallel formatting by default, lazily loads the worker pool, and preserves serial execution for single-file or non-cloneable requests. It also adds--no-paralleland--noParallelas explicit opt-outs, with coverage for worker startup, fallback, result ordering, and CLI behavior.