fix: changes from #19837 causing flake in e2e_cookies_spec

[tgriesser]

We are seeing flake in CI around the e2e_cookies_spec

https://app.circleci.com/pipelines/github/cypress-io/cypress/31590/workflows/9fdd5ccd-d68e-4b77-8388-404cc96d2948/jobs/1234911/tests#failed-test-0

Tracked it down to most likely being due to a change in #19837 which changed null -> false on some values which might have had some unintended consequences. (Not 100% sure on this though, was just the first thing I tried with open mode locally and it seemed to make a difference)

Also fixed up a pesky yarn install bug that comes up occasionally where our patched jQuery types are broken by the double uncommenting, requiring blowing away node_modules

[cypress-bot]

Thanks for taking the time to open a PR!

• Create a Draft Pull Request if your PR is not ready for review. Mark the PR as Ready for Review when you're ready for a Cypress team member to review the PR.
• Become familiar with the Code Review Checklist for guidelines on coding standards and what needs to be done before a PR can be merged.

[cypress]

---

Test summary

19185 • 0 • 218 • 0 • 1

---

Run details

Project	cypress
Status	Passed
Commit	9866e2f
Started	Feb 10, 2022 4:57 AM
Ended	Feb 10, 2022 5:08 AM
Duration	11:35 💡
OS	Linux Debian - 10.10
Browser	Multiple

View run in Cypress Dashboard ➡️

---

Flakiness

	cypress/integration/commands/xhr_spec.js		1
1	... > can alias a route without stubbing it

---

This comment has been generated by cypress-bot as a result of this project's GitHub integration settings. You can manage this integration in this project's settings in the Cypress Dashboard

[tgriesser]

This actually doesn't seem to be the issue. Needs more investigation