chore(deps): replace execa with node:child_process in perf tests#12570
Open
chore(deps): replace execa with node:child_process in perf tests#12570
Conversation
execa was only used for simple git command execution in perf test utilities. Node's built-in child_process module handles this without needing an external dependency. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
|
The latest updates on your projects. Learn more about Vercel for GitHub.
1 Skipped Deployment
|
Contributor
📦 Bundle Stats —
|
| Metric | Value | vs main (48a5609) | vs v5.19.0 |
|---|---|---|---|
| Internal (raw) | 4.41 MB | - | -202 B, -0.0% |
| Internal (gzip) | 1.01 MB | - | -19 B, -0.0% |
| Bundled (raw) | 12.09 MB | - | -34.0 KB, -0.3% |
| Bundled (gzip) | 2.72 MB | - | -7.1 KB, -0.3% |
| Import time | 1.40s | -4ms, -0.3% | +46ms, +3.4% |
bin:sanity
| Metric | Value | vs main (48a5609) | vs v5.19.0 |
|---|---|---|---|
| Internal (raw) | 7.1 KB | - | - |
| Internal (gzip) | 2.9 KB | - | - |
| Bundled (raw) | 7.1 KB | - | - |
| Bundled (gzip) | 2.8 KB | - | - |
| Import time | 5ms | -0ms, -0.4% | +0ms, +6.9% |
🗺️ View treemap · Artifacts
Details
- Import time regressions over 10% are flagged with
⚠️ - Sizes shown as raw / gzip 🗜️. Internal bytes = own code only. Total bytes = with all dependencies. Import time = Node.js cold-start median.
Contributor
🧪 E2E Preview environment🔑 Environment Variables for Local TestingThis is the preview URL for the E2E tests: https://e2e-studio-cmbiqlooq.sanity.dev To run the E2E tests locally, you can use the following environment variables, then run 💬 Remember to build the project first with |
Contributor
📊 Playwright Test ReportThis report contains test results, including videos of failing tests. |
Contributor
⚡️ Editor Performance ReportUpdated Thu, 02 Apr 2026 16:19:03 GMT
Detailed information🏠 Reference resultThe performance result of
🧪 Experiment resultThe performance result of this branch
📚 Glossary
|
Contributor
Coverage Report
File CoverageNo changed files found. |
Contributor
|
@bjoerge this looks good to me, but do you mind having a sanity check? I don't know much about the perf section and nothing looks critical but just to double check |
RitaDias
approved these changes
Apr 7, 2026
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
2 participants
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.
Description
Replaces the
execadependency with Node's built-innode:child_processmodule in the perf test utilities.execa@^2.1.0was only used for simple synchronous and async git command execution, whichexecFile/execFileSynchandle natively.What to review
perf/tests/runner/utils/gitUtils.ts— allexecacalls replaced withexecFile/execFileSyncfromnode:child_processperf/tests/package.json—execaremoved from dependenciespnpm-lock.yaml— lockfile updated,pnpm dedupeverifiedTesting
Notes for release
No user-facing changes. Internal dependency cleanup only.
🤖 Generated with Claude Code