Skip to content

fix: Correctly update version metadata when setting a value for the first time#2139

Open
aklinker1 wants to merge 4 commits intomainfrom
set-version-when-setting-value
Open

fix: Correctly update version metadata when setting a value for the first time#2139
aklinker1 wants to merge 4 commits intomainfrom
set-version-when-setting-value

Conversation

@aklinker1
Copy link
Member

@aklinker1 aklinker1 commented Feb 19, 2026

Overview

Addresses the issues around updating a storage item's version when setting values and preventing excessive migrations when unnecessary.

Manual Testing

See unit tests. To try it out in your extension, see #2139 (comment).

Related Issue

This closes #1775
This closes #2130
Confirm that the test created in #1774 also passes with these changes.

@aklinker1 aklinker1 self-assigned this Feb 19, 2026
@netlify
Copy link

netlify bot commented Feb 19, 2026

Deploy Preview for creative-fairy-df92c4 ready!

Name Link
🔨 Latest commit 59c6d11
🔍 Latest deploy log https://app.netlify.com/projects/creative-fairy-df92c4/deploys/69967de0ead9c40008aecf8b
😎 Deploy Preview https://deploy-preview-2139--creative-fairy-df92c4.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@pkg-pr-new
Copy link

pkg-pr-new bot commented Feb 19, 2026

Open in StackBlitz

@wxt-dev/analytics

npm i https://pkg.pr.new/@wxt-dev/analytics@2139

@wxt-dev/auto-icons

npm i https://pkg.pr.new/@wxt-dev/auto-icons@2139

@wxt-dev/browser

npm i https://pkg.pr.new/@wxt-dev/browser@2139

@wxt-dev/i18n

npm i https://pkg.pr.new/@wxt-dev/i18n@2139

@wxt-dev/module-react

npm i https://pkg.pr.new/@wxt-dev/module-react@2139

@wxt-dev/module-solid

npm i https://pkg.pr.new/@wxt-dev/module-solid@2139

@wxt-dev/module-svelte

npm i https://pkg.pr.new/@wxt-dev/module-svelte@2139

@wxt-dev/module-vue

npm i https://pkg.pr.new/@wxt-dev/module-vue@2139

@wxt-dev/runner

npm i https://pkg.pr.new/@wxt-dev/runner@2139

@wxt-dev/storage

npm i https://pkg.pr.new/@wxt-dev/storage@2139

@wxt-dev/unocss

npm i https://pkg.pr.new/@wxt-dev/unocss@2139

@wxt-dev/webextension-polyfill

npm i https://pkg.pr.new/@wxt-dev/webextension-polyfill@2139

wxt

npm i https://pkg.pr.new/wxt@2139

commit: 59c6d11

@codecov
Copy link

codecov bot commented Feb 19, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 76.15%. Comparing base (9dea0cf) to head (59c6d11).
⚠️ Report is 8 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #2139      +/-   ##
==========================================
- Coverage   76.37%   76.15%   -0.22%     
==========================================
  Files         113      113              
  Lines        3031     3045      +14     
  Branches      680      683       +3     
==========================================
+ Hits         2315     2319       +4     
- Misses        633      641       +8     
- Partials       83       85       +2     

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

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

aklinker1 and others added 3 commits February 18, 2026 21:03
Co-authored-by: absdjfh <absdjfh@github.noreply.com>
Co-authored-by: willow <42willow@pm.me>
Co-authored-by: absdjfh <absdjfh@noreply.github.com>
Co-authored-by: willow <42willow@pm.me>
Co-authored-by: absdjfh <absdjfh@users.noreply.github.com>
Co-authored-by: willow <42willow@pm.me>
@aklinker1 aklinker1 marked this pull request as ready for review February 19, 2026 03:05
@aklinker1 aklinker1 requested a review from Timeraa as a code owner February 19, 2026 03:05
@42willow
Copy link
Contributor

thanks for this ❤️
i've tested and it works with my extension 👍

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.

Meta is not set when storage item with migrations defined is initially created

2 participants

Comments