objectStoreNames's type seems wrong

[Ptival]

idb/lib/entry.ts

Line 149 in 809a75c

readonly objectStoreNames: StoreNames<DBTypes>[];

As far as I can tell, within my browser, this method returns a DOMStringList, which is not an array.

In particular, it contains a method contains, and does not contain array methods such as indexOf, which leads to both static and run-time errors.

[jakearchibald]

Good point.

Ugh, fixing this isn't totally easy, as DOMStringList doesn't support types for its content.

(fwiw, in Edge, it does return an array, as this property is polyfilled. However I don't know how much I care about this difference, as I intend to remove EdgeHTML support in the next major version)

[NotWoods]

I'd be interested in submitted a PR for this if @jakearchibald hasn't worked on it yet 😄.

[jakearchibald]

Go for it!

[shadow-light]

I think there's still an issue with objectStoreNames when it is accessed off a transaction:

idb/src/entry.ts

Line 531 in abce3c7

readonly objectStoreNames: TxStores;

It is array like, rather than DOMStringList. Just got burnt by this in a migration where I used includes instead of contains 😞

[jakearchibald]

I'll fix this tomorrow

[jakearchibald]

Fix published as 6.1.4.

[shadow-light]

Woohoo pipeline fixed, thanks!