Skip to content

perf(fmt): infer parsers in workers - #143

Merged
chenjiahan merged 2 commits into
mainfrom
chenjiahan/perf-fmt-parser-worker
Aug 3, 2026
Merged

perf(fmt): infer parsers in workers#143
chenjiahan merged 2 commits into
mainfrom
chenjiahan/perf-fmt-parser-worker

Conversation

@chenjiahan

@chenjiahan chenjiahan commented Aug 3, 2026

Copy link
Copy Markdown
Member

Summary

This moves Prettier parser inference and unsupported-file detection into formatting workers, so the main CLI path no longer loads Prettier or Yuku. It reuses the resolved plugin list for inference and formatting while preserving the existing silent skip behavior for unsupported files.

Benchmark time redecued from 5.246s to 5.103s with one worker and from 2.034s to 1.938s with the default worker count.

@cloudflare-workers-and-pages

cloudflare-workers-and-pages Bot commented Aug 3, 2026

Copy link
Copy Markdown

Deploying rstack-cli with  Cloudflare Pages  Cloudflare Pages

Latest commit: 606013e
Status: ✅  Deploy successful!
Preview URL: https://a0ccad64.rstack-cli.pages.dev
Branch Preview URL: https://chenjiahan-perf-fmt-parser-w.rstack-cli.pages.dev

View logs

…arser-worker

# Conflicts:
#	packages/rstack/src/fmt/worker.ts
#	packages/rstack/tests/fmt/worker.test.ts
@chenjiahan
chenjiahan merged commit 71ce5ba into main Aug 3, 2026
4 checks passed
@chenjiahan
chenjiahan deleted the chenjiahan/perf-fmt-parser-worker branch August 3, 2026 05:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant