fix(deps): update dependency watchpack to v2

[renovate]

This PR contains the following updates:

Package	Change	Age	Adoption	Passing	Confidence
watchpack	1.7.5 -> 2.1.1

---

Release Notes

webpack/watchpack

v2.1.1

Compare Source

Bugfix

• fix warnings with ENOENT when symlinks are resolved by watchpack

v2.1.0

Compare Source

Bugfix

• use creation time instead of modified time for folder entries that are only watch for existence
  • fixes a problem where webpack rebuilds unnecessarily

v2.0.1

Compare Source

Bugfix

• fix incorrectly reported changes when attaching to an already watched directory

v2.0.0

Compare Source

Features

• Polling no longer uses fs.watchFile but polls the directory directly
• add ignored option
• add followSymlinks option
• allow all Iterables instead of only Array passes as arguments
• Changed API to take an options object
  • Old API is also accepted without warning (it may be deprecated in the next major)
• emit remove event when files or directories are not found during initial scan
  • It's assumed that they are removed between reading and watching start
• add watching of missing items
• Support /RegExp/ for ignored option
• limit maximum number of os watchers on OSX to 2000 and on windows to 10000
  • limit can be changed with WATCHPACK_WATCHER_LIMIT environment variable
  • once limit is reached watchers are merged into recursive watchers
• merge multiple os watchers from watchers with different options into one os watcher
• add logging of recursive watchers when WATCHPACK_RECURSIVE_WATCHER_LOGGING env var is set

Bugfixes

• Normalize readdir result to NFC
• Skip watching the root of the filesystem as file even when requested
• use options identify instead of stringified options to merge watchers
• removals and changes in the aggregated event no longer both contain a file
• aggregateTimeout: 0 now works correctly
• don't fire change events when files are only read
• fix handling of renames that only change the casing of files on a case-insensitive filesystem

Performance

• avoid calling process.nextTick when unneeded on watcher creation
• cache time entry map in Watcher
• close watchers faster
• avoid an unneeded filter call

Dependencies

• remove neo-async dependency to be smaller

Contributing

• added prettier
• node 14 in CI
• test watcher limit of 1 on CI

Changes

• increase minimum node.js version to 10
• initial scan do not fire change events when no start time is provided
• improve polling schedule to not try to poll faster than the scanning

---

Renovate configuration

📅 Schedule: At any time (no schedule defined).

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

♻️ Rebasing: Renovate will not automatically rebase this PR, because other commits have been found.

🔕 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 has been generated by WhiteSource Renovate. View repository job log here.

[rpl]

There are test failures triggered by this watchpack updates, we need to look into those before merging this.

[rpl]

Apparently one test is still failing when running on windows:

• https://app.circleci.com/pipelines/github/mozilla/web-ext/179/workflows/86126945-b0e4-4277-ac96-4650e23e798d/jobs/726/parallel-runs/0/steps/0-106

I'll take a look at that and then add back Rob as a reviewer on this PR.

[rpl]

Let's hope that nobody uses \ to escape sequences.

yeah, I was thinking about that too, if I'm not mistaken that replace call shouldn't affect a path where the user did escape a whitespace character (which I think is not that uncommon on Windows), e.g.:

❯ node
> "C:\\Users\\My\ User\ Name\\Some\\Path"
'C:\\Users\\My User Name\\Some\\Path'
> "C:\\Users\\My\ User\ Name\\Some\\Path".replace(/\\/g, '/')
'C:/Users/My User Name/Some/Path'