diff --git a/docs/npm_import.md b/docs/npm_import.md
index 346d8bc45..430a1b55d 100644
--- a/docs/npm_import.md
+++ b/docs/npm_import.md
@@ -27,8 +27,7 @@ npm_import(name, pa
lifecycle_hooks_env, lifecycle_hooks_use_default_shell_env, integrity, url, commit,
replace_package, package_visibility, patch_args, patches, custom_postinstall, npm_auth,
npm_auth_basic, npm_auth_username, npm_auth_password, bins, dev,
- register_copy_directory_toolchains, register_copy_to_directory_toolchains,
- lifecycle_hooks_no_sandbox, kwargs)
+ register_copy_directory_toolchains, register_copy_to_directory_toolchains, kwargs)
Import a single npm package into Bazel.
@@ -162,7 +161,6 @@ Read more about the downloader config: dev | Whether this npm package is a dev dependency | `False` |
| register_copy_directory_toolchains | if True, `@aspect_bazel_lib//lib:repositories.bzl` `register_copy_directory_toolchains()` is called if the toolchain is not already registered | `True` |
| register_copy_to_directory_toolchains | if True, `@aspect_bazel_lib//lib:repositories.bzl` `register_copy_to_directory_toolchains()` is called if the toolchain is not already registered | `True` |
-| lifecycle_hooks_no_sandbox | If True, adds "no-sandbox" to `lifecycle_hooks_execution_requirements`.
Deprecated. Add "no-sandbox" to `lifecycle_hooks_execution_requirements` instead. | `None` |
| kwargs | Internal use only | none |
diff --git a/npm/extensions.bzl b/npm/extensions.bzl
index f2634d5e7..4b30bff67 100644
--- a/npm/extensions.bzl
+++ b/npm/extensions.bzl
@@ -185,7 +185,6 @@ def _npm_import_bzlmod(i):
lifecycle_hooks = i.lifecycle_hooks,
lifecycle_hooks_env = i.lifecycle_hooks_env,
lifecycle_hooks_execution_requirements = i.lifecycle_hooks_execution_requirements,
- lifecycle_hooks_no_sandbox = i.lifecycle_hooks_no_sandbox,
lifecycle_hooks_use_default_shell_env = i.lifecycle_hooks_use_default_shell_env,
link_packages = i.link_packages,
link_workspace = i.link_workspace,
diff --git a/npm/private/npm_import.bzl b/npm/private/npm_import.bzl
index 5fa557a1e..85314734f 100644
--- a/npm/private/npm_import.bzl
+++ b/npm/private/npm_import.bzl
@@ -920,8 +920,6 @@ def npm_import(
dev = False,
register_copy_directory_toolchains = True,
register_copy_to_directory_toolchains = True,
- # TODO(2.0): remove lifecycle_hooks_no_sandbox from npm_import
- lifecycle_hooks_no_sandbox = None,
**kwargs):
"""Import a single npm package into Bazel.
@@ -1170,10 +1168,6 @@ def npm_import(
register_copy_to_directory_toolchains: if True, `@aspect_bazel_lib//lib:repositories.bzl` `register_copy_to_directory_toolchains()` is called if the toolchain is not already registered
- lifecycle_hooks_no_sandbox: If True, adds "no-sandbox" to `lifecycle_hooks_execution_requirements`.
-
- Deprecated. Add "no-sandbox" to `lifecycle_hooks_execution_requirements` instead.
-
**kwargs: Internal use only
"""
@@ -1216,10 +1210,6 @@ def npm_import(
extract_full_archive = extract_full_archive,
)
- if lifecycle_hooks_no_sandbox:
- if "no-sandbox" not in lifecycle_hooks_execution_requirements:
- lifecycle_hooks_execution_requirements.append("no-sandbox")
-
has_custom_postinstall = not (not custom_postinstall)
has_lifecycle_hooks = not (not lifecycle_hooks)