Skip to content

chore(deps): update npm and pip dependencies across all suites#1092

Open
travagliad wants to merge 2 commits into
mainfrom
cursor/deps-update-all
Open

chore(deps): update npm and pip dependencies across all suites#1092
travagliad wants to merge 2 commits into
mainfrom
cursor/deps-update-all

Conversation

@travagliad

Copy link
Copy Markdown
Contributor

Why

Dependabot PR #1078 cannot be merged: it bumps TypeScript to 7.x which breaks npm ci with ERESOLVE (@typescript-eslint requires <6.1.0). All 30 E2E matrix jobs fail in ~20s at the install step.

How

Manually updated all suites to their latest compatible versions:

Suite Key updates
cli Playwright 1.61.1, adm-zip 0.6.0, shelljs 0.10, TS 5.9, ESLint 8.57
codeceptjs-e2e Playwright 1.61.1, adm-zip 0.6.0, axios/mysql2/pg/lodash patches, brace-expansion overrides
e2e_tests Playwright 1.61.1, typescript-eslint 8.65, mongodb 7.5, adm-zip 0.6.0
qa-integration ansible-runner 2.4.3, requests 2.32.5

Deferred (breaking majors): TypeScript 7, ESLint 10, codeceptjs 4, faker 10, mongodb 3→7 in codeceptjs-e2e.

Verified locally: npm ci passes in all three npm workspaces, tsc --noEmit passes in cli.

Dependabot PR #1078 fails because it bumps TypeScript to 7.x which
breaks @typescript-eslint peer constraints (npm ci ERESOLVE). This
update brings all suites to their latest compatible versions instead:

- Security: adm-zip 0.6.0, brace-expansion overrides (CVE-2026-13149)
- cli: Playwright 1.61.1, shelljs 0.10, TypeScript 5.9, ESLint 8.57
- codeceptjs-e2e: Playwright 1.61.1, axios/mysql2/pg/lodash patches
- e2e_tests: Playwright 1.61.1, typescript-eslint 8.65, mongodb 7.5
- qa-integration: ansible-runner 2.4.3, requests 2.32.5

Intentionally deferred breaking majors: TypeScript 7, ESLint 10,
codeceptjs 4, faker 10, mongodb 3→7 in codeceptjs-e2e.
@travagliad

Copy link
Copy Markdown
Contributor Author

Test Healer — PR #1092 validation results

Run: E2E tests Matrix #29782815075
Commit: 05dfb942

CI summary (30 matrix jobs)

Result Count
✅ Pass 28
❌ Fail 2
⏳ Pending 0

Verdict: Dependency updates are healthy — npm ci succeeds in all suites (unlike Dependabot #1078 which fails at install with ERESOLVE). The 2 failures look unrelated to this PR (infra flake + dashboard timing).


Local validation

Check Result
clinpm ci ✅ pass (0 vulnerabilities)
clitsc --noEmit ✅ pass
codeceptjs-e2enpm ci ✅ pass
e2e_testsnpm ci ✅ pass (0 vulnerabilities)

Failed jobs (not caused by deps)

1. @pgsm-pmm-integration — infra/setup flake

fatal: docker network create pmm-qa
Error: network with name pmm-qa already exists
Error: endpoint with name pmm-server already exists in network pmm-qa

Setup failed before any test ran. Classic parallel-matrix Docker network collision on the runner.

2. @mongodb-exporter — PMM-T1333 dashboard timing flake

Expected 2 Elements without data but found 4 on Dashboard
...mongodb-collections-overview...
Top 5 Hottest Collections by Read/Write (Total/Rate)

14 tests passed, 1 failed (PMM-T1333), 37 skipped. Assertion is on Grafana panels missing metrics data — timing/collection warmup, not npm/Playwright version change. PGSS (@pgss-pmm-integration) passed on the same run.


Comparison with Dependabot #1078

#1078 (Dependabot) #1092 (this PR)
npm ci ❌ ERESOLVE (TypeScript 7 vs eslint-plugin) ✅ all suites
Jobs reaching tests 0 / 30 30 / 30
Failures at install 30 0

Recommendation

Safe to merge from a dependency standpoint. The 2 red checks are pre-existing flake patterns (Docker setup race, dashboard metrics timing) — consider re-running those jobs or merging #1091 first so future pushes auto-cancel stale matrix runs.

@travagliad

Copy link
Copy Markdown
Contributor Author

@cursoragent fix conflicts

@cursor

cursor Bot commented Jul 23, 2026

Copy link
Copy Markdown
Contributor

Merge conflicts are resolved and pushed to cursor/deps-update-all.

What conflicted: Only codeceptjs-e2e/package.json and codeceptjs-e2e/package-lock.json, after merging main into the branch.

Resolution:

  • Kept main's tar@^7.5.20 security fix (replacing deprecated tar.gz from Fix Dependabot tar security update in codeceptjs-e2e #1093)
  • Kept this branch's dependency bumps (typescript@^5.9.3, yaml@^2.9.0, and the other updated packages)
  • Regenerated package-lock.json and verified npm ci passes in codeceptjs-e2e

Other changes from main (alert test fixes, dump API updates, workflow changes, etc.) merged cleanly without conflicts.

PR #1092 is updated and should be mergeable now.

Open in Web Open in Cursor 

Resolve codeceptjs-e2e package conflicts: keep main's tar@7.5.20
security fix (replacing deprecated tar.gz) alongside dependency bumps
from this branch.

Co-authored-by: travagliad <travagliad@users.noreply.github.com>
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