Mark internal deps extension as reproducible

This avoids an unnecessary lockfile entry for users of rules_python.

Also adopts `bazel mod tidy` to update the `use_repo` call.

.bazelversion

@@ -1 +1 @@
-7.0.0
+7.1.2

DEVELOPING.md

@@ -5,6 +5,7 @@
 1. Modify the `./python/pip_install/tools/requirements.txt` file and run:
    ```
    bazel run //tools/private/update_deps:update_pip_deps
+   bazel mod tidy
    ```
 1. Bump the coverage dependencies using the script using:
    ```

MODULE.bazel

@@ -5,7 +5,7 @@ module(
 )
 
 bazel_dep(name = "bazel_features", version = "1.9.1")
-bazel_dep(name = "bazel_skylib", version = "1.3.0")
+bazel_dep(name = "bazel_skylib", version = "1.6.1")
 bazel_dep(name = "rules_cc", version = "0.0.9")
 bazel_dep(name = "platforms", version = "0.0.4")
 
@@ -14,11 +14,9 @@ bazel_dep(name = "rules_proto", version = "6.0.0-rc1")
 bazel_dep(name = "protobuf", version = "21.7", repo_name = "com_google_protobuf")
 
 internal_deps = use_extension("//python/private/bzlmod:internal_deps.bzl", "internal_deps")
-internal_deps.install()
 use_repo(
     internal_deps,
-    "rules_python_internal",
-    # START: maintained by 'bazel run //tools/private/update_deps:update_pip_deps'
+    # Updated by bazel mod tidy.
     "pypi__build",
     "pypi__click",
     "pypi__colorama",
@@ -34,7 +32,7 @@ use_repo(
     "pypi__tomli",
     "pypi__wheel",
     "pypi__zipp",
-    # END: maintained by 'bazel run //tools/private/update_deps:update_pip_deps'
+    "rules_python_internal",
 )
 
 # We need to do another use_extension call to expose the "pythons_hub"

python/private/bzlmod/internal_deps.bzl

@@ -8,18 +8,15 @@
 
 "Python toolchain module extension for internal rule use"
 
+load("@bazel_skylib//lib:modules.bzl", "modules")
 load("//python/pip_install:repositories.bzl", "pip_install_dependencies")
 load("//python/private:internal_config_repo.bzl", "internal_config_repo")
 
-# buildifier: disable=unused-variable
-def _internal_deps_impl(module_ctx):
+def _internal_deps():
     internal_config_repo(name = "rules_python_internal")
     pip_install_dependencies()
 
-internal_deps = module_extension(
-    doc = "This extension to register internal rules_python dependecies.",
-    implementation = _internal_deps_impl,
-    tag_classes = {
-        "install": tag_class(attrs = dict()),
-    },
+internal_deps = modules.as_extension(
+    _internal_deps,
+    doc = "This extension registers internal rules_python dependencies.",
 )

tools/private/update_deps/BUILD.bazel

@@ -50,12 +50,10 @@ py_binary(
     name = "update_pip_deps",
     srcs = ["update_pip_deps.py"],
     data = [
-        "//:MODULE.bazel",
         "//python/pip_install:repositories",
         "//python/pip_install:requirements_txt",
     ],
     env = {
-        "MODULE_BAZEL": "$(rlocationpath //:MODULE.bazel)",
         "REPOSITORIES_BZL": "$(rlocationpath //python/pip_install:repositories)",
         "REQUIREMENTS_TXT": "$(rlocationpath //python/pip_install:requirements_txt)",
     },

tools/private/update_deps/update_pip_deps.py

@@ -56,14 +56,6 @@ def _dep_snippet(deps: list[Dep]) -> str:
     return textwrap.indent("".join(lines), " " * 4)
 
 
-def _module_snippet(deps: list[Dep]) -> str:
-    lines = []
-    for dep in deps:
-        lines.append(f'"{dep.name}",\n')
-
-    return textwrap.indent("".join(lines), " " * 4)
-
-
 def _generate_report(requirements_txt: pathlib.Path) -> dict:
     with tempfile.NamedTemporaryFile() as tmp:
         tmp_path = pathlib.Path(tmp.name)
@@ -131,12 +123,6 @@ def main():
         default=os.environ.get("REQUIREMENTS_TXT"),
         help="The requirements.txt path for the pip_install tools, defaults to the value taken from REQUIREMENTS_TXT",
     )
-    parser.add_argument(
-        "--module-bazel",
-        type=path_from_runfiles,
-        default=os.environ.get("MODULE_BAZEL"),
-        help="The path for the file to be updated, defaults to the value taken from MODULE_BAZEL",
-    )
     parser.add_argument(
         "--repositories-bzl",
         type=path_from_runfiles,
@@ -156,14 +142,6 @@ def main():
         dry_run=args.dry_run,
     )
 
-    update_file(
-        path=args.module_bazel,
-        snippet=_module_snippet(deps),
-        start_marker=args.start,
-        end_marker=args.end,
-        dry_run=args.dry_run,
-    )
-
 
 if __name__ == "__main__":
     main()