[deps]: Update electron-store to v11 - abandoned

[renovate]

This PR contains the following updates:

Package	Change	Age	Confidence
electron-store	8.2.0 -> 11.0.2

---

Release Notes

sindresorhus/electron-store (electron-store)

v11.0.2

Compare Source

• Update dependencies ccf6f15

---

v11.0.1

Compare Source

• Update dependencies (#​297) 8ecbb6b

---

v11.0.0

Compare Source

• Update conf dependency 3d690aa

---

v10.1.0

Compare Source

• Update dependencies dcf42b7

---

v10.0.1

Compare Source

• Fix importing electron abc1d2f

---

v10.0.0

Compare Source

Breaking

• Switch from JSON Schema draft-v7 to draft-2020-12
  • Please read about the breaking changes.

This is only a breaking change if you use the schema option.

v9.0.0

Compare Source

Breaking

• Require Node.js 20 and Electron 30 7ddf0c6
• This package is now pure ESM. Please read this and this.
  • For TypeScript users, make sure you have the correct tsconfig.

Improvements

• Update dependencies 7ddf0c6

---

Configuration

📅 Schedule: Branch creation - "every 2nd week starting on the 2 week of the year before 4am on Monday" (UTC), Automerge - At any time (no schedule defined).

🚦 Automerge: Disabled by config. Please merge this manually once you are satisfied.

♻ Rebasing: Whenever PR becomes conflicted, or you tick the rebase/retry checkbox.

🔕 Ignore: Close this PR and you won't be reminded about this update again.

---

• If you want to rebase/retry this PR, check this box

---

This PR was generated by Mend Renovate. View the repository job log.

[bitwarden-bot]

Internal tracking:

• ID: PM-26891
• Link: https://bitwarden.atlassian.net/browse/PM-26891

[codecov]

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 16.01%. Comparing base (9a719c9) to head (de8dc97).
⚠️ Report is 86 commits behind head on main.

Additional details and impacted files

@@           Coverage Diff           @@
##             main     #901   +/-   ##
=======================================
  Coverage   16.01%   16.01%           
=======================================
  Files          67       67           
  Lines        2791     2791           
  Branches      481      481           
=======================================
  Hits          447      447           
  Misses       2306     2306           
  Partials       38       38           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:

• 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

[github-actions]

Checkmarx One – Scan Summary & Details – f3c6022b-0cdb-4adb-ba4e-7d652818435c

Great job! No new security vulnerabilities introduced in this pull request

[claude]

Claude finished @renovate[bot]'s task in 2m 46s —— View job

---

PR Review: electron-store v8 → v11 Update

Tasks:

• Review the dependency update scope
• Analyze code changes for breaking change handling
• Run automated code review

Review Complete ✅

The code review agent has analyzed the changes and posted detailed inline comments on the PR. Key findings:

Breaking Changes Handled:

• ✅ ESM import migration (replaced require() with import)
• ✅ Node 20 & Electron 30+ requirements met
• ⚠️ TypeScript config change needs validation

Issues Identified:

1. Type safety: store property uses any instead of proper typing
2. Webpack config: May need to add electron-store to externals
3. moduleResolution change: From "node" to "bundler" needs testing

Please review the inline comments for detailed recommendations.

---

[claude]

Type Safety Issue: The store property is typed as any, which defeats TypeScript's type safety.

Since electron-store v11 is now written in TypeScript with proper type definitions, you should use proper typing:

Suggested change

	private store: any;
	private store: Store<Record<string, unknown>>;

Or if you know the specific shape of your data, define a proper interface for the store schema.

[claude]

Configuration Concern: Changing moduleResolution from "node" to "bundler" is a significant change that affects how TypeScript resolves module imports.

While "bundler" is appropriate for bundled applications, this change should be carefully tested because:

1. It changes how conditional exports are resolved
2. It allows importing ESM modules from CJS contexts (which may hide issues)
3. It affects how package.json exports fields are interpreted

Since you're using Webpack to bundle the application, this change seems reasonable, but you should verify:

• The application builds successfully
• All imports resolve correctly at runtime
• No module resolution errors occur in production

Consider also checking if you need to update the webpack configuration's resolve.conditionNames to match this change.

[claude]

Good: The migration from CommonJS require() to ESM import is correct for electron-store v11.

However, there are some additional concerns to address:

1. Missing webpack configuration: Since electron-store v11 is pure ESM, you may need to add it to webpack externals in webpack.main.js to prevent bundling issues:

   externals: {
     "electron-reload": "commonjs2 electron-reload",
     "electron-store": "commonjs2 electron-store",  // Add this
     keytar: "commonjs2 keytar",
   },

2. Type safety: The store property on line 11 is still typed as any. Consider using proper typing:

   private store: Store<Record<string, unknown>>;

3. Verify compatibility: Electron-store v11 requires Electron 30+. Ensure your electron dependency meets this requirement (I see you have v39.2.1, so this is fine).

[eliykat]

As discussed, I suggest we hold off until we look at ESM support properly next year.

[renovate]

Edited/Blocked Notification

Renovate will not automatically rebase this PR, because it does not recognize the last commit author and assumes somebody else may have edited the PR.

You can manually request rebase by checking the rebase/retry box above.

⚠️ Warning: custom changes will be lost.

[renovate]

Autoclosing Skipped

This PR has been flagged for autoclosing. However, it is being skipped due to the branch being already modified. Please close/delete it manually or report a bug if you think this is in error.

[eliykat]

Updated to 11 in #1034