refactor: remove deprecated lifecycle_hooks_no_sandbox from npm_import (

#1573)

npm/extensions.bzl

@@ -206,7 +206,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,

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)