Skip to content

Fix E2E cleanup#7784

Open
gonzaloriestra wants to merge 2 commits into
mainfrom
fix-e2e-cleanup
Open

Fix E2E cleanup#7784
gonzaloriestra wants to merge 2 commits into
mainfrom
fix-e2e-cleanup

Conversation

@gonzaloriestra

@gonzaloriestra gonzaloriestra commented Jun 11, 2026

Copy link
Copy Markdown
Contributor

WHY are these changes introduced?

E2E cleanup can leave apps behind when the dashboard errors during bulk discovery or when uninstall confirmation stays disabled until a reason is selected.

WHAT is this pull request doing?

  • Cleans matching apps page-by-page, with retries and per-app stats.
  • Restarts dashboard scanning after mutations to avoid stale pagination cursors.
  • Parses dashboard app names/install counts more safely.
  • Selects an uninstall reason and feedback before confirming app uninstall.

How to test your changes?

CI and pnpm test:e2e-cleanup-apps

Checklist

  • I've considered possible cross-platform impacts (Mac, Linux, Windows)
  • I've considered possible documentation changes
  • I've considered analytics changes to measure impact
  • This change is not user-facing; no changeset is needed

@github-actions github-actions Bot added the no-changelog This PR doesn't include a changeset entry. Is an internal only change not relevant to end users. label Jun 11, 2026
@gonzaloriestra gonzaloriestra marked this pull request as ready for review June 11, 2026 12:24
@gonzaloriestra gonzaloriestra requested a review from a team as a code owner June 11, 2026 12:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

no-changelog This PR doesn't include a changeset entry. Is an internal only change not relevant to end users.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants