search

kheina-com / Blue-Blocker

Blocks all Twitter Blue verified users on twitter.com
Mozilla Public License 2.0
352 stars 27 forks source link

[fix] Dynamic URL fix #366

Closed rougetimelord closed 1 month ago

rougetimelord commented 1 month ago

Bumps the version of @crxjs/vite-plugin that is used because it includes a fix for Chrome not recognizing dynamic URLs, see: https://github.com/crxjs/chrome-extension-tools/pull/928. Closes #365. Don't love that it means using a pre-release version of the package, but it is what it is... Hardcoded the specific version just in case a new beta drops.

I also specified a specific version of rollup in the package so that npm doesn't install a vulnerable version of it automatically... See: https://github.com/advisories/GHSA-gcx4-mw62-g8wm

Tested. Building works, and the extension no longer gets hit with a CSP error.

cooljeanius commented 1 month ago

CI failure says:

 warnings when minifying css:
Warning: G] CSS nesting syntax is not supported in the configured target environment ("chrome87", "edge88", "es2020", "firefox78", "safari13") [invalid-@nest]

    <stdin>:328:1:
      328 │   & p {
          ╵   ^

makefile:11: *** Extension version mismatch. manifest: "0.4.110", package.json: "0.4.11".  Stop.
Error: Process completed with exit code 2.
rougetimelord commented 1 month ago

The syntax warning is also present on main and the version mismatch was a problem with the script that runs on npm version runs