Fix yarn workspace isolation for node-client (#97)

* Update .gitignore to allow tracking .yarnrc.yml files

Change .yarn* to .yarn/ and .pnp.* to be more specific.
This allows .yarnrc.yml configuration files to be tracked.

* Add .yarnrc.yml files for yarn workspace isolation

Add standalone .yarnrc.yml to spec/node-client/ so Yarn 4 treats it
as an independent project rather than detecting it as a workspace
member of the root project.

* Remove npm install workaround from node client test

With yarn workspace isolation fixed, we no longer need to run
npm install on every test run. The test now just executes the
pre-built node client directly.

- Remove npm install command from test
- Delete package-lock.json (use yarn consistently)
- Update yarn.lock files

* Trigger gh actions

Author: joemsak

.gitignore

@@ -10,7 +10,8 @@ coverage
 
 node_modules
 .bundle
-.yarn*
+.yarn/
+.pnp.*
 
 # Workspace
 .idea

.yarnrc.yml

@@ -0,0 +1 @@
+nodeLinker: node-modules

spec/integration/ruby_server_nodejs_client_spec.rb

@@ -9,10 +9,8 @@
 
   let(:node_client_dir) { File.expand_path('../node-client', __dir__) }
   let(:node_cmd) do
-    # Use npm for dependency management (avoids yarn workspace detection issues)
-    # Run npm install first to ensure dependencies are available
     [
-      "cd #{node_client_dir} && npm install --silent 2>/dev/null && node dist/client.js",
+      "cd #{node_client_dir} && node dist/client.js",
       server_url,
       grpc_method,
       basic_username,

spec/node-client/.yarnrc.yml

@@ -0,0 +1,2 @@
+nodeLinker: node-modules
+

spec/node-client/package-lock.json

@@ -1,47 +1,96 @@
-# THIS IS AN AUTOGENERATED FILE. DO NOT EDIT THIS FILE DIRECTLY.
-# yarn lockfile v1
+# This file is generated by running "yarn install" inside your project.
+# Manual changes might be lost - proceed with caution!
 
+__metadata:
+  version: 8
+  cacheKey: 10c0
 
-"@improbable-eng/[email protected]":
-  version "0.15.0"
-  resolved "https://registry.npmjs.org/@improbable-eng/grpc-web-node-http-transport/-/grpc-web-node-http-transport-0.15.0.tgz"
-  integrity sha512-HLgJfVolGGpjc9DWPhmMmXJx8YGzkek7jcCFO1YYkSOoO81MWRZentPOd/JiKiZuU08wtc4BG+WNuGzsQB5jZA==
+"@improbable-eng/grpc-web-node-http-transport@npm:0.15.0":
+  version: 0.15.0
+  resolution: "@improbable-eng/grpc-web-node-http-transport@npm:0.15.0"
+  peerDependencies:
+    "@improbable-eng/grpc-web": ">=0.13.0"
+  checksum: 10c0/3a33cbc8b5182fa7caed005fe09eadb6018dcbbd981bbd49fd512a6b82275c329e945aadce394125e17708949aac951ac3235925df32f841c62adceb24f16762
+  languageName: node
+  linkType: hard
 
-"@improbable-eng/grpc-web@>=0.13.0", "@improbable-eng/[email protected]":
-  version "0.15.0"
-  resolved "https://registry.npmjs.org/@improbable-eng/grpc-web/-/grpc-web-0.15.0.tgz"
-  integrity sha512-ERft9/0/8CmYalqOVnJnpdDry28q+j+nAlFFARdjyxXDJ+Mhgv9+F600QC8BR9ygOfrXRlAk6CvST2j+JCpQPg==
+"@improbable-eng/grpc-web@npm:0.15.0":
+  version: 0.15.0
+  resolution: "@improbable-eng/grpc-web@npm:0.15.0"
   dependencies:
-    browser-headers "^0.4.1"
+    browser-headers: "npm:^0.4.1"
+  peerDependencies:
+    google-protobuf: ^3.14.0
+  checksum: 10c0/98949a73bf98a54e5df02a3a13e3dadad83a610a35ed57600e566a842777e02bec71de413e42a7728267a38c5ccf30af67fe516486e68546c2701e95d2446f5c
+  languageName: node
+  linkType: hard
 
-"@types/google-protobuf@^3.15.12":
-  version "3.15.12"
-  resolved "https://registry.npmjs.org/@types/google-protobuf/-/google-protobuf-3.15.12.tgz"
-  integrity sha512-40um9QqwHjRS92qnOaDpL7RmDK15NuZYo9HihiJRbYkMQZlWnuH8AdvbMy8/o6lgLmKbDUKa+OALCltHdbOTpQ==
+"@types/google-protobuf@npm:^3.15.12":
+  version: 3.15.12
+  resolution: "@types/google-protobuf@npm:3.15.12"
+  checksum: 10c0/721783234e627f367dd710c345a1eaa9dca4ac64910032cef0c851c1821e05d06ffb51e9d1693080f1c0797a8674f89130fa56a390395ec7791ea8506d2f3bfb
+  languageName: node
+  linkType: hard
 
-"@types/node@^22.14.1":
-  version "22.19.2"
-  resolved "https://registry.npmjs.org/@types/node/-/node-22.19.2.tgz"
-  integrity sha512-LPM2G3Syo1GLzXLGJAKdqoU35XvrWzGJ21/7sgZTUpbkBaOasTj8tjwn6w+hCkqaa1TfJ/w67rJSwYItlJ2mYw==
+"@types/node@npm:^22.14.1":
+  version: 22.19.2
+  resolution: "@types/node@npm:22.19.2"
   dependencies:
-    undici-types "~6.21.0"
-
-browser-headers@^0.4.1:
-  version "0.4.1"
-  resolved "https://registry.npmjs.org/browser-headers/-/browser-headers-0.4.1.tgz"
-  integrity sha512-CA9hsySZVo9371qEHjHZtYxV2cFtVj5Wj/ZHi8ooEsrtm4vOnl9Y9HmyYWk9q+05d7K3rdoAE0j3MVEFVvtQtg==
-
-google-protobuf@^3.14.0, google-protobuf@^3.21.4:
-  version "3.21.4"
-  resolved "https://registry.npmjs.org/google-protobuf/-/google-protobuf-3.21.4.tgz"
-  integrity sha512-MnG7N936zcKTco4Jd2PX2U96Kf9PxygAPKBug+74LHzmHXmceN16MmRcdgZv+DGef/S9YvQAfRsNCn4cjf9yyQ==
-
-typescript@^5.8.3:
-  version "5.9.3"
-  resolved "https://registry.npmjs.org/typescript/-/typescript-5.9.3.tgz"
-  integrity sha512-jl1vZzPDinLr9eUt3J/t7V6FgNEw9QjvBPdysz9KfQDD41fQrC2Y4vKQdiaUpFT4bXlb1RHhLpp8wtm6M5TgSw==
-
-undici-types@~6.21.0:
-  version "6.21.0"
-  resolved "https://registry.npmjs.org/undici-types/-/undici-types-6.21.0.tgz"
-  integrity sha512-iwDZqg0QAGrg9Rav5H4n0M64c3mkR59cJ6wQp+7C4nI0gsmExaedaYLNO44eT4AtBBwjbTiGPMlt2Md0T9H9JQ==
+    undici-types: "npm:~6.21.0"
+  checksum: 10c0/fd56fc727bdc2cd9582e05db5daab6c8eb7bc5e5640a54cbc64ccc160a61cf50940742be5536f83796bffc83ce7709d5580a9ca85806f57501958821d230ab78
+  languageName: node
+  linkType: hard
+
+"browser-headers@npm:^0.4.1":
+  version: 0.4.1
+  resolution: "browser-headers@npm:0.4.1"
+  checksum: 10c0/3b08864bb955b295ab3dd6ab775c7798096c2e85486571803b4070ec484de83ccceebe531a8b00d5daf4463fada5e7ca18cd1a71cc1ee0dfdbab705332318cef
+  languageName: node
+  linkType: hard
+
+"google-protobuf@npm:^3.21.4":
+  version: 3.21.4
+  resolution: "google-protobuf@npm:3.21.4"
+  checksum: 10c0/28f2800f7fe1a8fc55eb58ba76e158268407bfb3b90646eaf8a177dd92a2e522459b773f8132ae546e60ac3b6f5947557a1cf3d963a05bb594f43bcde640f54f
+  languageName: node
+  linkType: hard
+
+"grpc-web-node-ts-client@workspace:.":
+  version: 0.0.0-use.local
+  resolution: "grpc-web-node-ts-client@workspace:."
+  dependencies:
+    "@improbable-eng/grpc-web": "npm:0.15.0"
+    "@improbable-eng/grpc-web-node-http-transport": "npm:0.15.0"
+    "@types/google-protobuf": "npm:^3.15.12"
+    "@types/node": "npm:^22.14.1"
+    google-protobuf: "npm:^3.21.4"
+    typescript: "npm:^5.8.3"
+  languageName: unknown
+  linkType: soft
+
+"typescript@npm:^5.8.3":
+  version: 5.9.3
+  resolution: "typescript@npm:5.9.3"
+  bin:
+    tsc: bin/tsc
+    tsserver: bin/tsserver
+  checksum: 10c0/6bd7552ce39f97e711db5aa048f6f9995b53f1c52f7d8667c1abdc1700c68a76a308f579cd309ce6b53646deb4e9a1be7c813a93baaf0a28ccd536a30270e1c5
+  languageName: node
+  linkType: hard
+
+"typescript@patch:typescript@npm%3A^5.8.3#optional!builtin<compat/typescript>":
+  version: 5.9.3
+  resolution: "typescript@patch:typescript@npm%3A5.9.3#optional!builtin<compat/typescript>::version=5.9.3&hash=5786d5"
+  bin:
+    tsc: bin/tsc
+    tsserver: bin/tsserver
+  checksum: 10c0/ad09fdf7a756814dce65bc60c1657b40d44451346858eea230e10f2e95a289d9183b6e32e5c11e95acc0ccc214b4f36289dcad4bf1886b0adb84d711d336a430
+  languageName: node
+  linkType: hard
+
+"undici-types@npm:~6.21.0":
+  version: 6.21.0
+  resolution: "undici-types@npm:6.21.0"
+  checksum: 10c0/c01ed51829b10aa72fc3ce64b747f8e74ae9b60eafa19a7b46ef624403508a54c526ffab06a14a26b3120d055e1104d7abe7c9017e83ced038ea5cf52f8d5e04
+  languageName: node
+  linkType: hard