fix: return full label paths from npm_link_targets (#2655)

Fix #2417

### Changes are visible to end-users: no

### Test plan

- Covered by existing test cases

Author: jbedard

e2e/npm_translate_lock_disable_hooks/snapshots/defs.bzl

@@ -552,7 +552,7 @@ def npm_link_targets(name = "node_modules", package = None, prod = True, dev = T
                 targets = starlark_codegen_utils.to_list_attr(lists["dev"], 3, 4, quote_value = False),
             ))
         first_link = False
-    npm_link_targets_bzl.append("""    return link_targets""")
+    npm_link_targets_bzl.append("""    return ["//%s%s" % (bazel_package, target) for target in link_targets]""")
     return npm_link_targets_bzl
 
 def _generate_npm_visibility_config(package_visibility_attr):

e2e/npm_translate_lock_empty/snapshots/npm_defs.bzl

@@ -18,7 +18,7 @@ load("@npm//:defs.bzl", "npm_link_targets", "npm_link_all_packages")
 
 # buildifier: disable=unused-variable
 def npm_link_targets(name = "node_modules", package = None, prod = True, dev = True):
-    """Generated list of target names that are linked by npm_link_all_packages()
+    """Generated list of target labels that are linked by npm_link_all_packages()
 
     Args:
         name: name of catch all target to generate for all packages linked
@@ -35,7 +35,7 @@ def npm_link_targets(name = "node_modules", package = None, prod = True, dev = T
             Default True includes development dependencies.
 
     Returns:
-        A list of target names that are linked by npm_link_all_packages()
+        A list of target labels (full package labels) that are linked by npm_link_all_packages()
     """
     pass