Skip to content

feat: prompt (WIP - do not merge) #31752

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
wants to merge 16 commits into
base: develop
Choose a base branch
from
Open

feat: prompt (WIP - do not merge) #31752

wants to merge 16 commits into from

Conversation

ryanthemanuel
Copy link
Collaborator

  • Closes

Additional details

Steps to test

How has the user experience changed?

PR Tasks

@ryanthemanuel ryanthemanuel self-assigned this May 20, 2025
Copy link

cypress bot commented May 20, 2025

cypress    Run #62977

Run Properties:  status check failed Failed #62977  •  git commit 30e48d172b: chore: handle errors (#31854)
Project cypress
Branch Review feat/cy-prompt
Run status status check failed Failed #62977
Run duration 16m 29s
Commit git commit 30e48d172b: chore: handle errors (#31854)
Committer Alejandro Estrada
View all properties for this run ↗︎

Test results
Tests that failed  Failures 1
Tests that were flaky  Flaky 2
Tests that did not run due to a developer annotating a test with .skip  Pending 29
Tests that did not run due to a failure in a mocha hook  Skipped 0
Tests that passed  Passing 816
View all changes introduced in this branch ↗︎
UI Coverage  63.64%
  Untested elements 30  
  Tested elements 56  
Accessibility  96.23%
  Failed rules  0 critical   4 serious   1 moderate   0 minor
  Failed elements 143  

Tests for review

Failed  cypress\e2e\config-files-error-handling.cy.ts • 1 failed test • launchpad-e2e

View Output

Test Artifacts
setupNodeEvents > handles multiple config errors and then recovers Test Replay Screenshots
Flakiness  cypress\e2e\studio\studio.cy.ts • 1 flaky test • app-e2e

View Output

Test Artifacts
Cypress Studio > remains in studio mode when the test name is changed on the file system and file watching is disabled Test Replay Screenshots
Flakiness  cypress\e2e\choose-a-browser.cy.ts • 1 flaky test • launchpad-e2e

View Output

Test Artifacts
Choose a browser page > System Browsers Detected > launches when --browser is passed alone through the command line Test Replay Screenshots

* feat: cy prompt infrastructure

* refactor and add tests

* refactor

* rename experimental config

* prompt

* fix test

* Update cy-prompt-development.md

* Update cy-prompt-development.md

* PR comments

* Update packages/server/lib/cloud/api/cy-prompt/get_cy_prompt_bundle.ts

Co-authored-by: Copilot <[email protected]>

* PR comments

---------

Co-authored-by: Copilot <[email protected]>
ryanthemanuel and others added 13 commits May 29, 2025 21:51
* chore: add cdp connection to cy prompt

* minor fix

* fix type build

* try to fix build

* Update packages/server/lib/browsers/browser-cri-client.ts

Co-authored-by: Bill Glesias <[email protected]>

* Update packages/server/lib/browsers/browser-cri-client.ts

Co-authored-by: Bill Glesias <[email protected]>

* do not support prompt in firefox or webkit

* rework timing of lifecycle

* refactor

* fix tests

* troubleshooting

* troubleshooting

* fix tests

* additional troubleshooting

* additional troubleshooting

* additional troubleshooting

* attempt to fix build

* add back

* debugging

* debugging

* debugging

* debugging

* clean up

* fix unit tests

* rework

---------

Co-authored-by: Bill Glesias <[email protected]>
…#31803)

* chore: add promptBackend as an additional Cypress-attached function

* Update packages/app/src/runner/event-manager.ts

* update types

* fix types

* fix spacing

* refactor

* additional refactor

* fix type build

* fix build

* refactor

* reword messages

* fix

* debugging

* undo debugging

* PR comment

* fix tests

* fix tests

* fix tests

* fix test
* chore: add watcher for cy-prompt development

* test caching

* fix types
…31835)

* internal: (cy.prompt) handle errors better in the command definition

* internal: (cy.prompt) add timeout and handle loading errors more cleanly

* add process environment variable

* clean up test

* update JSDoc
* chore: handle errors

* Fix ts, add test

* Fix error title

* Fix ts

* Fix ts
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.

2 participants