refactor: remove utils.home_directory and use get_home_directory from…

… Aspect bazel-lib utils instead (#1606)

npm/extensions.bzl

@@ -4,6 +4,7 @@ See https://bazel.build/docs/bzlmod#extension-definition
 
 load("@bazel_features//:features.bzl", "bazel_features")
 load("@aspect_bazel_lib//lib:repo_utils.bzl", "repo_utils")
+load("@aspect_bazel_lib//lib:utils.bzl", bazel_lib_utils = "utils")
 load("//npm:repositories.bzl", "npm_import", "pnpm_repository", _LATEST_PNPM_VERSION = "LATEST_PNPM_VERSION")
 load("//npm/private:npm_translate_lock.bzl", "npm_translate_lock", "npm_translate_lock_lib")
 load("//npm/private:npm_translate_lock_helpers.bzl", npm_translate_lock_helpers = "helpers")
@@ -107,7 +108,7 @@ def _npm_extension_impl(module_ctx):
                 (registries, npm_auth) = npm_translate_lock_helpers.get_npm_auth(npmrc, module_ctx.path(attr.npmrc), module_ctx.os.environ)
 
             if attr.use_home_npmrc:
-                home_directory = utils.home_directory(module_ctx)
+                home_directory = bazel_lib_utils.get_home_directory(module_ctx)
                 if home_directory:
                     home_npmrc_path = "{}/{}".format(home_directory, ".npmrc")
                     home_npmrc = parse_npmrc(module_ctx.read(home_npmrc_path))

npm/private/npm_translate_lock_state.bzl

@@ -5,6 +5,7 @@ load("@bazel_skylib//lib:paths.bzl", "paths")
 load("@bazel_skylib//lib:dicts.bzl", "dicts")
 load("@aspect_bazel_lib//lib:base64.bzl", "base64")
 load("@aspect_bazel_lib//lib:repo_utils.bzl", "repo_utils")
+load("@aspect_bazel_lib//lib:utils.bzl", bazel_lib_utils = "utils")
 load(":repository_label_store.bzl", "repository_label_store")
 load(":npm_translate_lock_helpers.bzl", "helpers")
 load(":utils.bzl", "INTERNAL_ERROR_MSG", "utils")
@@ -483,7 +484,7 @@ def _load_npmrc(priv, rctx, npmrc_path):
 
 ################################################################################
 def _load_home_npmrc(priv, rctx):
-    home_directory = utils.home_directory(rctx)
+    home_directory = bazel_lib_utils.get_home_directory(rctx)
     if not home_directory:
         # buildifier: disable=print
         print("""

npm/private/utils.bzl

@@ -378,9 +378,6 @@ if [ ! -f $1 ]; then exit 42; fi
     else:
         fail(INTERNAL_ERROR_MSG)
 
-def _home_directory(rctx):
-    return repo_utils.get_env_var(rctx, "HOME", None)
-
 def _replace_npmrc_token_envvar(token, npmrc_path, environ):
     # A token can be a reference to an environment variable
     if token.startswith("$"):
@@ -463,7 +460,6 @@ utils = struct(
     dicts_match = _dicts_match,
     reverse_force_copy = _reverse_force_copy,
     exists = _exists,
-    home_directory = _home_directory,
     replace_npmrc_token_envvar = _replace_npmrc_token_envvar,
     is_vendored_tarfile = _is_vendored_tarfile,
     is_tarball_extension = _is_tarball_extension,