chore: cy prompt infrastructure #31748
Merged
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
ryanthemanuel
requested review from
brian-mann and
jennifer-shehane
as code owners
cypress
|
||||||||||||||||||||||||||||||||||||||||
| Project |
cypress
|
| Branch Review |
ryanm/feat/cy-prompt-infra
|
| Run status |
|
| Run duration | 18m 21s |
| Commit |
|
| Committer | Ryan Manuel |
| View all properties for this run ↗︎ | |
| Test results | |
|---|---|
Failures
|
0
|
Flaky
|
11
|
Pending
|
1232
|
Skipped
|
0
|
Passing
|
32164
|
| View all changes introduced in this branch ↗︎ | |
UI Coverage
45.71%
|
|
|---|---|
|
Untested elements
|
191
|
|
Tested elements
|
165
|
Accessibility
92.74%
|
|
|---|---|
|
Failed rules
|
3 critical
9 serious
2 moderate
2 minor
|
|
Failed elements
|
695
|
mschile
reviewed
May 22, 2025
mschile
reviewed
May 22, 2025
mschile
approved these changes
May 23, 2025
There was a problem hiding this comment.
Pull Request Overview
This PR provides the foundational infrastructure for the new cy.prompt() command by pulling down, verifying, and loading a remotely hosted bundle, wiring in lifecycle managers, and enabling the experimental flag across server, driver, config, and docs.
- Adds bundle download/extract helpers and cloud API calls for cy-prompt
- Introduces
CyPromptManagerandCyPromptLifecycleManagerfor runtime orchestration - Registers the experimentalPromptCommand flag in config, driver, tests, and documentation
Reviewed Changes
Copilot reviewed 43 out of 43 changed files in this pull request and generated 4 comments.
Show a summary per file
| File | Description |
|---|---|
| packages/server/lib/cloud/cy-prompt/ensure_cy_prompt_bundle.ts | Fetches or copies and extracts the cy-prompt bundle |
| packages/server/lib/cloud/cy-prompt/CyPromptManager.ts | Invokes methods on dynamically loaded cy-prompt server scripts |
| packages/server/lib/cloud/cy-prompt/CyPromptLifecycleManager.ts | Orchestrates session creation, bundle setup, and manager ready |
| packages/server/lib/cloud/api/cy-prompt/post_cy_prompt_session.ts | Implements POST to start a cy-prompt session |
| packages/server/lib/cloud/api/cy-prompt/get_cy_prompt_bundle.ts | Downloads and verifies the signed cy-prompt bundle |
| packages/data-context/src/data/coreDataShape.ts | Adds cyPromptLifecycleManager to the shared data context shape |
| packages/driver/src/cy/commands/prompt/prompt-driver-types.ts | Defines driver interface for cy.prompt |
| packages/driver/src/cy/commands/prompt/index.ts | Implements the prompt command using module federation runtime |
| packages/driver/src/cy/commands/index.ts | Registers the prompt command under driver commands |
| packages/driver/package.json | Adds @module-federation/runtime dependency |
| packages/driver/cypress/e2e/commands/prompt.cy.ts | Basic end-to-end smoke test for cy.prompt |
| packages/driver/cypress/e2e/e2e/origin/commands/misc.cy.ts | Updated command list to include prompt |
| packages/driver/cypress.config.ts | Enables experimentalPromptCommand |
| packages/config/src/options.ts | Introduces new experimental flag |
| packages/config/test/project/utils.spec.ts | Updates default and public config keys tests |
| packages/config/snapshots/index.spec.ts.js | Updates snapshots to include the new flag |
| packages/frontend-shared/src/locales/en-US.json | Adds UI copy for the new experimental flag |
| guides/cy-prompt-development.md | Documents local and remote development workflows for cy.prompt |
Comments suppressed due to low confidence (1)
packages/driver/cypress/e2e/commands/prompt.cy.ts:7
- [nitpick] This test only verifies that the command does not throw—consider adding assertions on expected side effects or backend calls to improve coverage.
cy.prompt('Hello, world!')
packages/server/lib/cloud/api/cy-prompt/get_cy_prompt_bundle.ts
Outdated
Show resolved
Hide resolved
packages/server/lib/cloud/api/cy-prompt/get_cy_prompt_bundle.ts
Outdated
Show resolved
Hide resolved
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Additional details
This PR lays the ground work for consuming the cloud code that will be responsible for supporting
cy.prompt()Steps to test
How has the user experience changed?
PR Tasks
cypress-documentation?type definitions?