fix: normalize default dependency group name to underscores to prevent configuration mismatches

Author: xangcastle

e2e/BUILD.bazel

@@ -36,14 +36,7 @@ write_source_files(
     name = "snapshots",
     testonly = True,
     files = {
-        # @pypi_rdflib_linux — regression: hyphen/underscore normalization in dep_group names.
-        # ----------------------------------------------------------------
-        # The project is named "rdflib-linux" (hyphens, Python convention). uv_hub
-        # currently emits config_setting(name = "rdflib-linux") using the raw name,
-        # so dep_group = "rdflib_linux" (underscore, Bazel convention) never matches
-        # → @@platforms//:incompatible. The fix must normalize the name to underscores.
-        # This snapshot pins the DESIRED (fixed) state; it FAILS today and PASSES once
-        # uv_hub normalises dep_group names via normalize_name().
+        # @pypi_rdflib_linux — dep_group names must be underscore-normalized.
         "snapshots/pypi_rdflib_linux.dep_group.BUILD.bazel": "@pypi_rdflib_linux//dep_group:BUILD.bazel",
 
         # @pypi_multi (multi-project: alpha, beta, gamma)

e2e/MODULE.bazel

@@ -108,6 +108,7 @@ include("//cases/single-project-hub:setup.MODULE.bazel")
 include("//cases/unconstrained-dependencies:setup.MODULE.bazel")
 include("//cases/uv-abi3-compat-853:setup.MODULE.bazel")
 include("//cases/uv-bare-linux-wheels:setup.MODULE.bazel")
+include("//cases/uv-hub-cross-project:setup.MODULE.bazel")
 include("//cases/uv-conflict-817:setup.MODULE.bazel")
 include("//cases/uv-conflict-gte:setup.MODULE.bazel")
 include("//cases/uv-conflict-max-863:setup.MODULE.bazel")

e2e/cases/firebase-admin-import/BUILD.bazel

@@ -39,7 +39,7 @@ load("//tools:pth_snapshot.bzl", "extract_venv_pth")
 py_test(
     name = "test",
     srcs = ["test.py"],
-    dep_group = "firebase-admin-import",
+    dep_group = "firebase_admin_import",
     main = "test.py",
     deps = [
         "@pypi_firebase_admin_import//firebase_admin",
@@ -53,7 +53,7 @@ py_test(
 py_test(
     name = "test_tool",
     srcs = ["test.py"],
-    dep_group = "firebase-admin-import",
+    dep_group = "firebase_admin_import",
     expose_venv = True,
     isolated = False,
     main = "test.py",
@@ -73,7 +73,7 @@ py_test(
 py_binary(
     name = "firebase_importer",
     srcs = ["test.py"],
-    dep_group = "firebase-admin-import",
+    dep_group = "firebase_admin_import",
     main = "test.py",
     deps = [
         "@pypi_firebase_admin_import//firebase_admin",

e2e/cases/freethreaded-805/BUILD.bazel

@@ -26,7 +26,7 @@ _LINUX_X86_64 = [
 py_test(
     name = "test_bin",
     srcs = ["test.py"],
-    dep_group = "freethreaded-test",
+    dep_group = "freethreaded_test",
     main = "test.py",
     python_version = "3.13",
     tags = ["manual"],
@@ -46,7 +46,7 @@ platform_transition_test(
 py_test(
     name = "venv_test_bin",
     srcs = ["test.py"],
-    dep_group = "freethreaded-test",
+    dep_group = "freethreaded_test",
     expose_venv = True,
     isolated = False,
     main = "test.py",

e2e/cases/pth-namespace-547/BUILD.bazel

@@ -5,7 +5,7 @@ load("//tools:pth_snapshot.bzl", "extract_venv_pth")
 py_test(
     name = "test",
     srcs = ["__test__.py"],
-    dep_group = "pth-namespace-547",
+    dep_group = "pth_namespace_547",
     expose_venv = True,
     isolated = False,
     main = "__test__.py",
@@ -21,7 +21,7 @@ py_test(
 py_test(
     name = "test_legacy",
     srcs = ["__test__.py"],
-    dep_group = "pth-namespace-547",
+    dep_group = "pth_namespace_547",
     main = "__test__.py",
     deps = [
         "@pypi_pth_namespace_547//jaraco_classes",

e2e/cases/pytest-main-867/BUILD.bazel

@@ -5,7 +5,7 @@ load("@aspect_rules_py//py:defs.bzl", "py_pytest_main", "py_test")
 py_test(
     name = "test_example",
     srcs = ["test_example.py"],
-    dep_group = "pytest-main-867",
+    dep_group = "pytest_main_867",
     pytest_main = True,
     deps = ["@pypi_pytest_main_867//pytest"],
 )
@@ -15,7 +15,7 @@ py_test(
 py_test(
     name = "test_naming_regression",
     srcs = ["test_example.py"],
-    dep_group = "pytest-main-867",
+    dep_group = "pytest_main_867",
     pytest_main = True,
     deps = ["@pypi_pytest_main_867//pytest"],
 )
@@ -29,7 +29,7 @@ py_test(
         "test_a.py",
         "test_b.py",
     ],
-    dep_group = "pytest-main-867",
+    dep_group = "pytest_main_867",
     pytest_main = True,
     deps = ["@pypi_pytest_main_867//pytest"],
 )
@@ -39,7 +39,7 @@ py_test(
 py_test(
     name = "test_collection_check",
     srcs = ["test_collection_check.py"],
-    dep_group = "pytest-main-867",
+    dep_group = "pytest_main_867",
     pytest_main = True,
     deps = ["@pypi_pytest_main_867//pytest"],
 )
@@ -58,7 +58,7 @@ py_test(
         "test_a.py",
         ":test_direct_main",
     ],
-    dep_group = "pytest-main-867",
+    dep_group = "pytest_main_867",
     main = ":__test__test_direct_main__.py",
     deps = [
         ":test_direct_main",

e2e/cases/pytest-mock-530/BUILD.bazel

@@ -4,7 +4,7 @@ load("@aspect_rules_py//py:defs.bzl", "py_test")
 py_test(
     name = "test_mock_py_test",
     srcs = ["test_mock.py"],
-    dep_group = "pytest-mock-530",
+    dep_group = "pytest_mock_530",
     pytest_main = True,
     deps = [
         "@pypi_pytest_mock_530//pytest",
@@ -18,7 +18,7 @@ py_test(
         "__test__.py",
         "test_mock.py",
     ],
-    dep_group = "pytest-mock-530",
+    dep_group = "pytest_mock_530",
     expose_venv = True,
     isolated = False,
     main = "__test__.py",

e2e/cases/pytest-subdir-imports/BUILD.bazel

@@ -9,7 +9,7 @@ py_library(
 py_test(
     name = "test_subdir_import",
     srcs = ["tests/test_subdir.py"],
-    dep_group = "pytest-subdir-imports",
+    dep_group = "pytest_subdir_imports",
     pytest_main = True,
     deps = [
         ":lib",

e2e/cases/pytest-xdist-integration/BUILD.bazel

@@ -13,7 +13,7 @@ py_test(
         "-n",
         "2",
     ],
-    dep_group = "pytest-xdist-integration",
+    dep_group = "pytest_xdist_integration",
     pytest_main = True,
     deps = [
         "@pypi_pytest_xdist_integration//pytest",

e2e/cases/rdflib-linux/BUILD.bazel

@@ -1,23 +1,8 @@
-# Regression: pyproject.toml name = "rdflib-linux" (hyphens, standard Python convention)
-# causes uv_hub to emit config_setting(name = "rdflib-linux", ...) with the raw name.
-# A user writing dep_group = "rdflib_linux" (underscores, natural Bazel convention)
-# gets a mismatch: flag "rdflib_linux" ≠ config_setting "rdflib-linux", so
-# compatible_with() falls through to //conditions:default and emits
-# @@platforms//:incompatible for the entire rdflib alias.
-#
-# Fix: normalize dep_group names in uv_hub (replace "-" → "_") before emitting
-# config_setting targets and flag_values.
 load("@aspect_rules_py//py:defs.bzl", "py_test")
 load("@rules_shell//shell:sh_test.bzl", "sh_test")
 
-# This sh_test reads the hub's generated dep_group/BUILD.bazel via exports_files()
-# — a source-file target that carries NO target_compatible_with, so incompatibility
-# does NOT propagate here. The test runs in //... and FAILS while the bug exists.
-#
-# Root cause it checks: uv_hub emits config_setting(name = "rdflib-linux") using the
-# raw pyproject.toml [project].name. A user writing dep_group = "rdflib_linux"
-# (underscore, Bazel convention) gets no match → @@platforms//:incompatible.
-# Fix: normalize dep_group names with normalize_name() in uv_hub/repository.bzl.
+# Regression: hyphenated [project].name caused config_setting names with hyphens,
+# breaking dep_group = "rdflib_linux" (underscore) matching. Fix: normalize_name().
 sh_test(
     name = "dep_group_names_normalized",
     srcs = ["check_dep_group_names.sh"],
@@ -26,33 +11,12 @@ sh_test(
     target_compatible_with = ["@platforms//os:linux"],
 )
 
-# Vector 1: hyphen/underscore mismatch.
-# pyproject.toml name = "rdflib-linux" → hub config_setting = "rdflib-linux".
-# dep_group = "rdflib_linux" (underscore) doesn't match → @@platforms//:incompatible.
 py_test(
     name = "rdflib_linux",
     srcs = ["__test__.py"],
     dep_group = "rdflib_linux",
     main = "__test__.py",
     python_version = "3.11",
     target_compatible_with = ["@platforms//os:linux"],
-    deps = [
-        "@pypi_rdflib_linux//rdflib",
-    ],
-)
-
-# Vector 2: multi-project hub, rdflib only in one project.
-# "other" project shares @pypi_rdflib_linux but has no rdflib dependency.
-# Building with dep_group = "other" → rdflib alias has no matching arm
-# for "other" → //conditions:default → @@platforms//:incompatible.
-py_test(
-    name = "rdflib_wrong_dep_group",
-    srcs = ["__test__.py"],
-    dep_group = "other",
-    main = "__test__.py",
-    python_version = "3.11",
-    target_compatible_with = ["@platforms//os:linux"],
-    deps = [
-        "@pypi_rdflib_linux//rdflib",
-    ],
+    deps = ["@pypi_rdflib_linux//rdflib"],
 )