SyntaxError: '[inert] *):not([tabindex^="-"]):not(:disabled' is not a valid selector

[sebnitu]

Hello there! Firstly thank you for the work you do on this project!

I'm currently using a package called focusable-selectors which provides an array of selectors for grabbing all focusable HTML elements. I was previously locked to using nwsapi v2.2.2 because I was getting a "not valid selector" syntax error but recently updated to the most recent version used by jsdom nswapi v2.2.12 and am now getting this syntax error:

SyntaxError: '[inert] *):not([tabindex^="-"]):not(:disabled' is not a valid selector
 ❯ emit node_modules/nwsapi/src/nwsapi.js:575:17
 ❯ Object._matches [as match] node_modules/nwsapi/src/nwsapi.js:1417:9
 ❯ Array.Resolver node_modules/nwsapi/src/nwsapi.js:777:17
 ❯ collect node_modules/nwsapi/src/nwsapi.js:1573:21
 ❯ Object._querySelectorAll [as select] node_modules/nwsapi/src/nwsapi.js:1528:36
 ❯ HTMLDivElementImpl.querySelectorAll node_modules/jsdom/lib/jsdom/living/nodes/ParentNode-impl.js:78:26
 ❯ HTMLDivElement.querySelectorAll node_modules/jsdom/lib/jsdom/living/generated/Element.js:1119:58

Here's a discussion about the issue here for some more context: KittyGiraudel/focusable-selectors#17

Essentially the problem was resolved by removing a complex selector from specifically three array items (note: I removed the selectors that work as expected):

const notInert = ':not([inert]):not([inert] *)'
const notNegTabIndex = ':not([tabindex^="-"])'
const notDisabled = ':not(:disabled)'

export default [
  `button${notInert}${notNegTabIndex}${notDisabled}`,
  `[contenteditable]${notInert}${notNegTabIndex}`,
  `[tabindex]${notInert}${notNegTabIndex}`,
]

Source

The interesting thing is that in the array, there are many selectors that use the ${notInert}${notNegTabIndex}${notDisabled} chained selectors and do not produce a syntax error, only the button, [contenteditable] and [tabindex] do.

Temporary solution

The temporary solution I'm using to resolve my issue was to simply remove this portion in the notInert const: :not([inert] *).

[dperini]

All reported issues having the error similar to "is not a valid selectors" had a common cause.
The nesting Regular Expression in the recently added :has() pseudo-class resolver was wrong.
Closing this now.