Skip to content

Add warning for missing Firefox extension ID in manifest#2293

Open
Sbrjt wants to merge 1 commit intowxt-dev:mainfrom
Sbrjt:main
Open

Add warning for missing Firefox extension ID in manifest#2293
Sbrjt wants to merge 1 commit intowxt-dev:mainfrom
Sbrjt:main

Conversation

@Sbrjt
Copy link
Copy Markdown

@Sbrjt Sbrjt commented Apr 23, 2026

Overview

This PR adds a warning when the Firefox extension ID is missing from the manifest to help prevent invalid builds.

https://developer.mozilla.org/en-US/docs/Mozilla/Add-ons/WebExtensions/manifest.json/browser_specific_settings#description

Manual Testing

Not sure

Related Issue

None

@Sbrjt Sbrjt requested a review from aklinker1 as a code owner April 23, 2026 19:05
@netlify
Copy link
Copy Markdown

netlify Bot commented Apr 23, 2026

Deploy Preview for creative-fairy-df92c4 ready!

Name Link
🔨 Latest commit 1fddf85
🔍 Latest deploy log https://app.netlify.com/projects/creative-fairy-df92c4/deploys/69ea6d7de37be20008762d27
😎 Deploy Preview https://deploy-preview-2293--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.

@github-actions github-actions Bot added the pkg/wxt Includes changes to the `packages/wxt` directory label Apr 23, 2026
@pkg-pr-new
Copy link
Copy Markdown

pkg-pr-new Bot commented Apr 24, 2026

Open in StackBlitz

@wxt-dev/analytics

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

@wxt-dev/auto-icons

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

@wxt-dev/browser

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

@wxt-dev/i18n

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

@wxt-dev/is-background

npm i https://pkg.pr.new/@wxt-dev/is-background@2293

@wxt-dev/module-react

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

@wxt-dev/module-solid

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

@wxt-dev/module-svelte

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

@wxt-dev/module-vue

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

@wxt-dev/runner

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

@wxt-dev/storage

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

@wxt-dev/unocss

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

@wxt-dev/webextension-polyfill

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

wxt

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

commit: 1fddf85

"Manifest 'version' is missing. Either:\n1. Add a version in your <rootDir>/package.json\n2. Pass the version via the manifest option in your wxt.config.ts",
);
}

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Let's use prettier here, i'll fix CI errors and anyway, that gap isn't necessary

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

pkg/wxt Includes changes to the `packages/wxt` directory

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants