Feat/playwright os 2

[bijin-bruno]

Description

Updated branch from #7253

Contribution Checklist:

• I've used AI significantly to create this pull request
• The pull request only addresses one issue or adds one feature.
• The pull request does not introduce any breaking changes
• I have added screenshots or gifs to help explain the change if applicable.
• I have read the contribution guidelines.
• Create an issue and link to the pull request.

Note: Keeping the PR small and focused helps make it easier to review and merge. If you have multiple changes you want to make, please consider submitting them as separate pull requests.

Publishing to New Package Managers

Please see here for more information.

[coderabbitai]

Important

Review skipped

Draft detected.

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

Run ID: 5c4d269c-551a-47b6-ab32-6af282b751c3

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

• 🔍 Trigger review

✨ Finishing Touches

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Commit unit tests in branch feat/playwright-os-2

Tip

💬 Introducing Slack Agent: The best way for teams to turn conversations into code.

Slack Agent is built on CodeRabbit's deep understanding of your code, so your team can collaborate across the entire SDLC without losing context.

• Generate code and open pull requests
• Plan features and break down work
• Investigate incidents and troubleshoot customer tickets together
• Automate recurring tasks and respond to alerts with triggers
• Summarize progress and report instantly

Built for teams:

• Shared memory across your entire org—no repeating context
• Per-thread sandboxes to safely plan and execute work
• Governance built-in—scoped access, auditability, and budget controls

One agent for your entire SDLC. Right inside Slack.

👉 Get started

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands and usage tips.}

[github-actions]

CLI Test Results (Linux)

  1 files  211 suites   1m 8s ⏱️
589 tests 589 ✅ 0 💤 0 ❌
702 runs  695 ✅ 7 💤 0 ❌

Results for commit f60f3e4.

♻️ This comment has been updated with latest results.

[github-actions]

⚠️ Warning: You modified flaky/failed test files

The following test files you modified have reliability issues:

❌ Failed: tests/import/insomnia/import-insomnia-v5-environments.spec.ts

Test: Import Insomnia v5 collection with nested environments and verify flattening
Status: failed

To debug locally, run:

npx playwright test tests/import/insomnia/import-insomnia-v5-environments.spec.ts --repeat-each=5 --workers=1

❌ Failed: tests/shortcuts/bound-actions.spec.ts

Test: default Cmd/Ctrl+W closes the active tab
Status: failed

To debug locally, run:

npx playwright test tests/shortcuts/bound-actions.spec.ts --repeat-each=5 --workers=1

⚠️ Flaky: tests/shortcuts/bound-actions.spec.ts

Test: default Cmd/Ctrl+F open sidebar search
Status: passed
Retry Attempt: 1

To debug locally, run:

npx playwright test tests/shortcuts/bound-actions.spec.ts --repeat-each=5 --workers=1

⚠️ Flaky: tests/shortcuts/bound-actions.spec.ts

Test: customized Alt+K Global Search Modal
Status: passed
Retry Attempt: 1

To debug locally, run:

npx playwright test tests/shortcuts/bound-actions.spec.ts --repeat-each=5 --workers=1

❌ Failed: tests/shortcuts/bound-actions.spec.ts

Test: sends request when cursor is in pre-request Vars value editor
Status: failed

To debug locally, run:

npx playwright test tests/shortcuts/bound-actions.spec.ts --repeat-each=5 --workers=1

---

Note: Flaky tests passed after retrying, failed tests did not pass. Please investigate and fix the root cause before merging.

[sid-bruno]

Why were these tests removed?

[bijin-bruno]

Those assertions would fail on windows. Also, doesn't provide much value since all of them are validating an external module path. Few validations that are remaining is enough to validate the bruno scripting.