Skip to content

Conversation

louis-bompart
Copy link
Collaborator

@louis-bompart louis-bompart commented Oct 16, 2025

Cypress notwithstanding (it's about to be removed), analysis of the recent results shows that in the case of Headless changes, Quantic is the second bottleneck, with execution time in the worst-case scenario falling within the 15' range.

To alleviate this:

  • chore(quantic): move playwright setup to docker #6167 will help; it will reduce the avg of setup time and bring back down the worst-case scenario where apt gets slow (in the worst-case scenario, apt dependencies alone were around 4-5', and I guestimate them to be in the 2-3' average)
  • This PR: we noticed that the setup & teardown phase of Scratch Orgs was done sequentially.
  • The setup phase has an average runtime of 11m15s over the last year, and most of it now is on the sfdx interactions. By parallelizing those two processes, we should have a bit less than ~50% improvement (ought to consider the base overhead). The experiment validates the theory:
image
  • The gain for the teardown would be negligible: we see an avg run time <2min. We systematically need to have 1 to 2 min of setup (npm install and whatnot), so the gain should be in seconds. Recent runs validate this theory: https://github.com/coveo/ui-kit/actions/runs/18539880218/job/52846626897; if we parallelize the two teardown runs, we would save... 4 seconds.

All and all, #6167 combined with this PR could shave up to 10' in the worst-case scenario, and over 5' in average.

Data for Coveans:


KIT-5134

@developer-experience-bot
Copy link
Contributor

developer-experience-bot bot commented Oct 16, 2025

Pull Request Report

PR Title

✅ Title follows the conventional commit spec.

Live demo links

Bundle Size

File Old (kb) New (kb) Change (%)
case-assist 255.6 255.6 0
commerce 369.4 369.4 0
search 427.1 427.1 0
insight 418.3 418.3 0
recommendation 266.4 266.4 0
ssr 421.5 421.5 0
ssr-commerce 386.5 386.5 0
ssr-commerce-next 387.8 387.8 0
ssr-next 422.1 422.1 0

@louis-bompart louis-bompart force-pushed the chore/parallelize-scratch-org-setup branch from c57e033 to 06b8ef5 Compare October 16, 2025 05:07
@louis-bompart louis-bompart force-pushed the chore/parallelize-scratch-org-setup branch from 06b8ef5 to a2158aa Compare October 16, 2025 05:09
@louis-bompart louis-bompart force-pushed the chore/parallelize-scratch-org-setup branch from a2158aa to 2d7f2d9 Compare October 16, 2025 05:35
@louis-bompart louis-bompart force-pushed the chore/parallelize-scratch-org-setup branch from 2d7f2d9 to 5e1b3ee Compare October 16, 2025 05:52
@louis-bompart louis-bompart changed the base branch from main to chore/docker-playwright-quantic-too October 16, 2025 07:02
@louis-bompart louis-bompart changed the title chore/parallelize scratch org setup chore(quantic): parallelize scratch org setup Oct 16, 2025
@louis-bompart louis-bompart marked this pull request as ready for review October 16, 2025 07:02
@louis-bompart louis-bompart requested review from a team as code owners October 16, 2025 07:02
Copy link
Contributor

@mmitiche mmitiche left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Very nice, thank you! 🚀

Base automatically changed from chore/docker-playwright-quantic-too to main October 16, 2025 20:48
@louis-bompart louis-bompart added this pull request to the merge queue Oct 17, 2025
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to failed status checks Oct 17, 2025
@louis-bompart louis-bompart added this pull request to the merge queue Oct 17, 2025
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to failed status checks Oct 17, 2025
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.

4 participants