Skip to content

Bump electron devDependencies (@types/node, typescript-eslint) - #1383

Merged
Tim020 merged 5 commits into
devfrom
feature/electron-dep-bumps
Sep 18, 2026
Merged

Tim020 merged 5 commits into
devfrom
feature/electron-dep-bumps

Conversation

@Tim020

@Tim020 Tim020 commented Sep 18, 2026

Copy link
Copy Markdown
Contributor

Summary

Recreates the changes from dependabot PRs #1379, #1376, #1375 by regenerating electron/package-lock.json locally on macOS instead of copying the Linux-generated lockfile from the dependabot runner (see repo convention for client-side dependency PRs).

  • @types/node: 26.4.1 → 26.6.1 (dependabot targeted 26.5.1; 26.6.1 is now latest in the same ^26.5.1 range)
  • @typescript-eslint/eslint-plugin: 8.69.0 → 8.70.0
  • @typescript-eslint/parser: 8.69.0 → 8.70.0

All devDependencies. npm run typecheck and npm run ci-lint pass clean with no new violations.

Closes the following dependabot PRs once merged: #1379, #1376, #1375

Test plan

  • npm run typecheck passes
  • npm run ci-lint passes (prettier + eslint)
  • Lockfile regenerated via npm ci + npm install <pkg>@<version> on macOS

🤖 Generated with Claude Code

Tim020 and others added 2 commits September 18, 2026 13:10
Recreated locally from dependabot PR #1379 (which targeted 26.5.1)
since the lockfile must be regenerated on macOS, not copied from the
Linux-based dependabot runner. 26.6.1 is now the latest patch
satisfying the same ^26.5.1 caret range.

Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
…rom 8.69.0 to 8.70.0 in /electron

Recreated locally from dependabot PRs #1376 and #1375. Combined into
one commit since these two packages must stay version-matched.
Lockfile regenerated on macOS rather than copied from the Linux-based
dependabot runner. No new lint violations from the bump.

Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
@github-actions

github-actions Bot commented Sep 18, 2026

Copy link
Copy Markdown

Client Test Results

132 tests   132 ✅  0s ⏱️
  7 suites    0 💤
  1 files      0 ❌

Results for commit 2a6605d.

♻️ This comment has been updated with latest results.

@github-actions

github-actions Bot commented Sep 18, 2026

Copy link
Copy Markdown

Client V3 Test Results

39 tests   39 ✅  0s ⏱️
 4 suites   0 💤
 1 files     0 ❌

Results for commit 2a6605d.

♻️ This comment has been updated with latest results.

@github-actions

github-actions Bot commented Sep 18, 2026

Copy link
Copy Markdown

Playwright E2E Results (firefox)

221 tests   221 ✅  2m 8s ⏱️
 14 suites    0 💤
  1 files      0 ❌

Results for commit 2a6605d.

♻️ This comment has been updated with latest results.

@github-actions

github-actions Bot commented Sep 18, 2026

Copy link
Copy Markdown

Python Test Results

  1 files    1 suites   2m 24s ⏱️
863 tests 863 ✅ 0 💤 0 ❌
868 runs  868 ✅ 0 💤 0 ❌

Results for commit 2a6605d.

♻️ This comment has been updated with latest results.

@github-actions

github-actions Bot commented Sep 18, 2026

Copy link
Copy Markdown

Playwright E2E Results (chromium)

221 tests   221 ✅  2m 16s ⏱️
 14 suites    0 💤
  1 files      0 ❌

Results for commit 2a6605d.

♻️ This comment has been updated with latest results.

@Tim020
Tim020 enabled auto-merge (squash) September 18, 2026 12:23
Tim020 and others added 3 commits September 18, 2026 14:04
CI failed on this branch with `npm error Missing: encoding@0.1.13
from lock file` / `Missing: iconv-lite@0.6.3 from lock file` — the
local macOS `npm install <pkg>` regeneration in the prior two commits
silently dropped minipass-fetch's optional `encoding` dependency (and
its nested iconv-lite@0.6.3) from the lockfile, even though `npm ci`
installs it fine and it's present on `dev`. Restored the two entries
verbatim from `dev`'s lockfile. `npm ci`, typecheck and ci-lint all
pass clean with this fix.

Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
@sonarqubecloud

Copy link
Copy Markdown

@Tim020
Tim020 merged commit 254ae0a into dev Sep 18, 2026
35 checks passed
@Tim020
Tim020 deleted the feature/electron-dep-bumps branch September 18, 2026 13:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

claude Issues created by Claude client Pull requests changing front end code medium-diff

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant