fix(DON-2676): Fix double-clip, add semantics tests, restore npmjs resolved URLs

- Remove redundant clip() from BpkNavigationTabImpl by moving it to always
  apply once in BpkNavigationTab before the selectable modifier
- Add withoutIcons snapshot test and semanticsSelectedState assertions
- Restore registry.npmjs.org resolved URLs in package-lock.json (Artifactory
  rewrites occur on npm install due to local ~/.npmrc config)

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>

Author: henrik-sky

app/src/test/java/net/skyscanner/backpack/compose/navigationtabgroup/BpkNavigationTabGroupTest.kt

@@ -19,6 +19,11 @@
 package net.skyscanner.backpack.compose.navigationtabgroup
 
 import androidx.compose.ui.graphics.Color
+import androidx.compose.ui.test.assertCountEquals
+import androidx.compose.ui.test.assertIsNotSelected
+import androidx.compose.ui.test.assertIsSelected
+import androidx.compose.ui.test.isSelectable
+import androidx.compose.ui.test.onAllNodes
 import net.skyscanner.backpack.compose.BpkSnapshotTest
 import net.skyscanner.backpack.compose.theme.BpkTheme
 import net.skyscanner.backpack.demo.compose.NavigationTabGroupSample
@@ -35,4 +40,31 @@ class BpkNavigationTabGroupTest : BpkSnapshotTest() {
     fun onDark() = snap(background = { BpkTheme.colors.surfaceContrast }) {
         NavigationTabGroupSample(style = BpkNavigationTabGroupStyle.SurfaceContrast)
     }
+
+    @Test
+    fun withoutIcons() = snap(background = { Color.Transparent }) {
+        BpkNavigationTabGroup(
+            tabs = listOf(
+                BpkNavigationTabItem("Tab 1"),
+                BpkNavigationTabItem("Tab 2"),
+                BpkNavigationTabItem("Tab 3"),
+            ),
+            selectedIndex = 0,
+            onItemClicked = {},
+        )
+    }
+
+    @Test
+    fun semanticsSelectedState() = snap(
+        background = { Color.Transparent },
+        assertion = {
+            onAllNodes(isSelectable()).assertCountEquals(4)
+            onAllNodes(isSelectable())[0].assertIsSelected()
+            onAllNodes(isSelectable())[1].assertIsNotSelected()
+            onAllNodes(isSelectable())[2].assertIsNotSelected()
+            onAllNodes(isSelectable())[3].assertIsNotSelected()
+        },
+    ) {
+        NavigationTabGroupSample()
+    }
 }

backpack-compose/src/main/kotlin/net/skyscanner/backpack/compose/navigationtabgroup/internal/BpkNavigationTab.kt

@@ -73,14 +73,15 @@ internal fun BpkNavigationTab(
         style = style,
         icon = icon,
         interactionSource = interactionSource,
-        modifier = modifier.applyIf(onClick != null) {
-            clip(NavigationTabShape)
-                .selectable(
+        modifier = modifier
+            .clip(NavigationTabShape)
+            .applyIf(onClick != null) {
+                selectable(
                     selected = selected,
                     interactionSource = interactionSource,
                     indication = bpkRipple(),
                 ) { onClick!!.invoke() }
-        },
+            },
     )
 }
 
@@ -131,7 +132,6 @@ private fun BpkNavigationTabImpl(
         modifier = modifier
             .heightIn(min = 48.dp)
             .widthIn(min = 76.dp)
-            .clip(NavigationTabShape)
             .background(backgroundColor)
             .border(BorderStroke(BpkBorderSize.Sm, strokeColor), NavigationTabShape)
             .padding(horizontal = BpkSpacing.Base, vertical = BpkSpacing.Md),