Repository navigation
TEST: Stabilize Operations E2E - #3356
Merged
Roman Lutz (romanlutz) merged 1 commit intoOct 20, 2026
Merged
Roman Lutz (romanlutz) merged 1 commit into
Roman Lutz (romanlutz) merged 1 commit into
Conversation
Wait for runtime readiness and configured defaults before clicking startup controls. Confirm selected and cleared operation preferences are saved before navigation or reload, and keep intentional early-startup coverage unchanged. Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com> Copilot-Session: 732186ad-b909-441f-93fb-93d18dac3c62
Richard Lundeen (richlundeen)
approved these changes
Oct 19, 2026
Roman Lutz (romanlutz)
deleted the
romanlutz-operations-e2e-investigation
branch
October 20, 2026 01:07
Roman Lutz (romanlutz)
added a commit
to Kaap10/PyRIT
that referenced
this pull request
Oct 20, 2026
Backport the existing startup and preference persistence fix from microsoft#3356 (7a8a341). Wait for runtime readiness and configured defaults before normal startup interactions, and confirm operation preferences are persisted before reloads. Preserve intentional early-startup coverage. Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com> Copilot-Session: f496faeb-6d01-47ac-9c54-1e0a9665078b
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
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.
Description
The seeded Operations test blocked the merge queue for #3315 in this run. The operation was registered correctly, but startup data moved the visible picker during its click. The dropdown stayed closed and never requested the operation list. Controlled browser diagnostics reproduced the same race when the runtime banner disappeared.
operation: nullvalues reach localStorage before navigation or reload. An empty picker alone is not proof of clearing: it is also empty while a saved-ID lookup is pending.No product behavior, converter code, timeout increases, or fixed sleeps are introduced. Related PR #3275 adds rename/delete coverage and does not contain these fixes.
Tests and Documentation
Validated locally on Windows with the provider-free seeded backend and isolated ports. Both suites ran with
CI=true,E2E_SEEDED_MODE=true, and zero retries.Exact validation commands and results, run from frontend
Session-only diagnostics also exercised the new helper with delayed defaults at both viewport widths and controlled runtime-banner reflow. All three checks passed. These diagnostic files and traces are not included in the PR.
Updated
frontend/README.mdwith startup and preference synchronization guidance. No notebooks changed; JupyText and live-provider tests were not run.