chore: put types condition first in the syncing script

[Andarist]

Follow up to #26630 , fixes what @MikeMcC399 reported in the comment here: #26630 (comment)

[MikeMcC399]

@Andarist

I am still getting a diff, however this appears to be due to a typo.

Have you actually tested this yourself?

diff --git a/cli/package.json b/cli/package.json
index e0d0a3303f..f9c2d09f14 100644
--- a/cli/package.json
+++ b/cli/package.json
@@ -160,7 +160,7 @@
     "./svelte": {
       "types": "./svelte/dist/index.d.ts",
       "import": "./svelte/dist/cypress-svelte.esm-bundler.js",
-      "require": "./svelte/dist/cypress-selte.cjs.js"
+      "require": "./svelte/dist/cypress-svelte.cjs.js"
     }
   },
   "workspaces": {

[cypress]

26 flaky tests on run #46245 ↗︎

0

27290

1308

0

26

Details:

fix(internal): put types condition first in the syncing script
Project: cypress	Commit: 3b8c2a1011
Status: Passed	Duration: 22:22 💡
Started: May 12, 2023 1:19 PM	Ended: May 12, 2023 1:41 PM

  e2e/origin/navigation.cy.ts • 1 flaky test • 5x-driver-electron

View Output Video

Test		Artifacts
delayed navigation > errors > redirects to an unexpected cross-origin and calls another command in the cy.origin command		Output Video

  cypress/cypress.cy.js • 3 flaky tests • 5x-driver-electron

View Output Video

Test		Artifacts
... > correctly returns currentRetry		Output Video
... > correctly returns currentRetry		Output Video
... > correctly returns currentRetry		Output Video

  create-from-component.cy.ts • 1 flaky test • app-e2e

View Output Video

Test		Artifacts
... > runs generated spec		Output Screenshots Video

  specs_list_latest_runs.cy.ts • 1 flaky test • app-e2e

View Output Video

Test		Artifacts
App/Cloud Integration - Latest runs and Average duration > when no runs are recorded > shows placeholders for all visible specs		Output Screenshots Video

  cypress-origin-communicator.cy.ts • 1 flaky test • app-e2e

View Output Video

Test		Artifacts
Cypress In Cypress Origin Communicator > cy.origin passivity with app interactions > passes upon test reload mid test execution		Output Screenshots Video

The first 5 flaky specs are shown, see all 14 specs in Cypress Cloud.

_{This comment has been generated by cypress-bot as a result of this project's GitHub integration settings.}

[MikeMcC399]

• The typo ./svelte/dist/cypress-selte.cjs.js was introduced by fix: move types condition to the front #26630

[Andarist]

@MikeMcC399 pushed out a fix for this typo, good catch

Have you actually tested this yourself?

Yeah, perhaps not to the full extent. When I located what was reordering those keys I kept running the build script in only one of the directories so I missed the typo in another one.

[MikeMcC399]

After executing

yarn
git status

I am no longer seeing any git diff generated.

[chrisbreiding]

Thanks for the fix, @Andarist!

[cypress-bot]

Released in 12.13.0.

This comment thread has been locked. If you are still experiencing this issue after upgrading to
Cypress v12.13.0, please open a new issue.