fix(getHighlightedIndexOnOpen): remove added check from migration

src/hooks/utils/__tests__/getHighlightedIndexOnOpen.test.ts

@@ -186,3 +186,33 @@ test('returns -1 when no conditions match', () => {
     ),
   ).toBe(-1)
 })
+
+test('returns -1 when defaultHighlightedIndex is -1, even when selectedItem is set', () => {
+  expect(
+    getHighlightedIndexOnOpen(
+      items,
+      undefined,
+      -1,
+      isItemDisabled,
+      itemToKey,
+      'b',
+      -1,
+      1,
+    ),
+  ).toBe(-1)
+})
+
+test('returns -1 when initialHighlightedIndex is -1 and highlightedIndex matches', () => {
+  expect(
+    getHighlightedIndexOnOpen(
+      items,
+      -1,
+      undefined,
+      isItemDisabled,
+      itemToKey,
+      'b',
+      -1,
+      1,
+    ),
+  ).toBe(-1)
+})

src/hooks/utils/getHighlightedIndexOnOpen.ts

@@ -32,16 +32,17 @@ export function getHighlightedIndexOnOpen<Item>(
   if (
     initialHighlightedIndex !== undefined &&
     highlightedIndex === initialHighlightedIndex &&
-    items[initialHighlightedIndex] &&
-    !isItemDisabled(items[initialHighlightedIndex], initialHighlightedIndex)
+    (items[initialHighlightedIndex] === undefined ||
+      !isItemDisabled(items[initialHighlightedIndex], initialHighlightedIndex))
   ) {
     return initialHighlightedIndex
   }
 
   if (
     defaultHighlightedIndex !== undefined &&
-    items[defaultHighlightedIndex] &&
-    !isItemDisabled(items[defaultHighlightedIndex], defaultHighlightedIndex)
+    (defaultHighlightedIndex === -1 ||
+      items[defaultHighlightedIndex] === undefined ||
+      !isItemDisabled(items[defaultHighlightedIndex], defaultHighlightedIndex))
   ) {
     return defaultHighlightedIndex
   }