fix: handle missing packages in _collect_dep_constraints

When an optional dependency is a "link:" dependency or otherwise not present in the `packages` dictionary, `npm_translate_lock` would fail with a "key not found" error such as: `Error: key "link:@angular/core|packages/core" not found in dictionary`

This occurs because `_collect_dep_constraints` iterates over optional dependencies but assumes they all exist in the `packages` dictionary. Link dependencies are often not represented in the lockfile as standard
packages, leading to this missing key.

Author: alan-agius4

e2e/pnpm_lockfiles/.bazelignore

@@ -2,6 +2,8 @@ node_modules/
 cases/versionless-patch-v9/node_modules
 cases/docusaurus-direct-peer-v9/node_modules
 cases/nested-peer-v9/node_modules
+cases/workspace-peer-v9/node_modules
+cases/workspace-peer-v9/libs/chokidar/node_modules
 cases/isaacs-cliui-v90/node_modules
 cases/override-with-alias-url-v9/node_modules
 projects/a/node_modules

e2e/pnpm_lockfiles/MODULE.bazel

@@ -17,6 +17,7 @@ PNPM_LOCK_TEST_CASES = [
     "docusaurus-direct-peer-v9",
     "versionless-patch-v9",
     "nested-peer-v9",
+    "workspace-peer-v9",
 ]
 
 bazel_dep(name = "bazel_lib", version = "3.0.0")

e2e/pnpm_lockfiles/cases/workspace-peer-v9/BUILD.bazel

@@ -0,0 +1,14 @@
+load("@bazel_skylib//rules:build_test.bzl", "build_test")
+load("@workspace-peer-v9//:defs.bzl", "npm_link_all_packages")
+
+exports_files(["pnpm-lock.yaml"])
+
+npm_link_all_packages()
+
+build_test(
+    name = "optional_peers_targets",
+    targets = [
+        ":node_modules",
+        ":node_modules/chokidar",
+    ],
+)

e2e/pnpm_lockfiles/cases/workspace-peer-v9/libs/chokidar/BUILD.bazel

@@ -0,0 +1,7 @@
+load("@aspect_rules_js//js:defs.bzl", "js_library")
+
+js_library(
+    name = "pkg",
+    srcs = ["package.json"],
+    visibility = ["//visibility:public"],
+)

e2e/pnpm_lockfiles/cases/workspace-peer-v9/libs/chokidar/package.json

@@ -0,0 +1,4 @@
+{
+  "private": true,
+  "name": "chokidar"
+}

e2e/pnpm_lockfiles/cases/workspace-peer-v9/package.json

@@ -0,0 +1,11 @@
+{
+    "name": "workspace-peer",
+    "private": true,
+    "pnpm": {
+        "onlyBuiltDependencies": []
+    },
+    "devDependencies": {
+        "@angular-devkit/core": "21.2.1",
+        "chokidar": "workspace:*"
+    }
+}

e2e/pnpm_lockfiles/cases/workspace-peer-v9/pnpm-lock.yaml

@@ -0,0 +1,3 @@
+packages:
+    - '.'
+    - 'libs/*'

e2e/pnpm_lockfiles/cases/workspace-peer-v9/pnpm-workspace.yaml

@@ -555,6 +555,9 @@ def _collect_dep_constraints(packages, package_info):
 
     optional_keys = package_info["optional_dependencies"].values() + package_info.get("transitive_optional_closure", {}).keys()
     for dep_key in optional_keys:
+        if dep_key.startswith("link:"):
+            continue
+
         dep = packages[dep_key]
         if dep["os"]:
             constraints_os[dep_key] = dep["os"]