From e9aac70bd7b79587938ef41aef22792086be4f44 Mon Sep 17 00:00:00 2001 From: "copilot-swe-agent[bot]" <198982749+Copilot@users.noreply.github.com> Date: Thu, 22 Jan 2026 04:31:36 +0000 Subject: [PATCH 1/5] Initial plan From 394f1d80a59e86ca134d4ed1036a98c9eabe202a Mon Sep 17 00:00:00 2001 From: "copilot-swe-agent[bot]" <198982749+Copilot@users.noreply.github.com> Date: Thu, 22 Jan 2026 04:33:54 +0000 Subject: [PATCH 2/5] Update CONTRIBUTING.md: clarify .NET SDK prerequisites - Mark 'just' as optional since commands can be run directly - Add Node.js and TypeScript as requirements for .NET SDK (needed for test harness) - Add npm install instructions for ./nodejs and ./test/harness directories - Update "Running tests and linters" section to clarify 'just' is optional Co-authored-by: friggeri <106686+friggeri@users.noreply.github.com> --- CONTRIBUTING.md | 12 +++++++++--- 1 file changed, 9 insertions(+), 3 deletions(-) diff --git a/CONTRIBUTING.md b/CONTRIBUTING.md index 8584e360..ada8ae96 100644 --- a/CONTRIBUTING.md +++ b/CONTRIBUTING.md @@ -26,7 +26,7 @@ Currently **we are not looking to add SDKs for other languages**. If you want to This is a multi-language SDK repository. Install the tools for the SDK(s) you plan to work on: ### All SDKs -1. Install [just](https://github.com/casey/just) command runner +1. (Optional) Install [just](https://github.com/casey/just) command runner for convenience ### Node.js/TypeScript SDK 1. Install [Node.js](https://nodejs.org/) (v18+) @@ -44,7 +44,13 @@ This is a multi-language SDK repository. Install the tools for the SDK(s) you pl ### .NET SDK 1. Install [.NET 8.0+](https://dotnet.microsoft.com/download) -1. Install dependencies: `cd dotnet && dotnet restore` +1. Install [Node.js](https://nodejs.org/) (v18+) and [TypeScript](https://www.typescriptlang.org/) (the .NET tests depend on a TypeScript-based test harness) +1. Install npm dependencies: + ```bash + cd nodejs && npm ci + cd ../test/harness && npm ci + ``` +1. Install .NET dependencies: `cd dotnet && dotnet restore` ## Submitting a pull request @@ -59,7 +65,7 @@ This is a multi-language SDK repository. Install the tools for the SDK(s) you pl ### Running tests and linters -Use `just` to run tests and linters across all SDKs or for specific languages: +If you installed `just`, you can use it to run tests and linters across all SDKs or for specific languages: ```bash # All SDKs From 1cb312ebff62016daa4c2f51e1f0e5a3d1543fed Mon Sep 17 00:00:00 2001 From: "copilot-swe-agent[bot]" <198982749+Copilot@users.noreply.github.com> Date: Thu, 22 Jan 2026 04:34:27 +0000 Subject: [PATCH 3/5] Fix: Remove unnecessary TypeScript installation step TypeScript and tsx are included as npm dependencies and don't need separate installation Co-authored-by: friggeri <106686+friggeri@users.noreply.github.com> --- CONTRIBUTING.md | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/CONTRIBUTING.md b/CONTRIBUTING.md index ada8ae96..bf9672f4 100644 --- a/CONTRIBUTING.md +++ b/CONTRIBUTING.md @@ -44,7 +44,7 @@ This is a multi-language SDK repository. Install the tools for the SDK(s) you pl ### .NET SDK 1. Install [.NET 8.0+](https://dotnet.microsoft.com/download) -1. Install [Node.js](https://nodejs.org/) (v18+) and [TypeScript](https://www.typescriptlang.org/) (the .NET tests depend on a TypeScript-based test harness) +1. Install [Node.js](https://nodejs.org/) (v18+) (the .NET tests depend on a TypeScript-based test harness) 1. Install npm dependencies: ```bash cd nodejs && npm ci From 9ecf9aab7b662223d03fd95bb749ce39e44572df Mon Sep 17 00:00:00 2001 From: "copilot-swe-agent[bot]" <198982749+Copilot@users.noreply.github.com> Date: Thu, 22 Jan 2026 04:35:49 +0000 Subject: [PATCH 4/5] Fix: Use consistent path navigation from repository root Changed from relative path (cd ../test/harness) to absolute path from repo root (cd test/harness) for consistency Co-authored-by: friggeri <106686+friggeri@users.noreply.github.com> --- CONTRIBUTING.md | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/CONTRIBUTING.md b/CONTRIBUTING.md index bf9672f4..a4a48c6c 100644 --- a/CONTRIBUTING.md +++ b/CONTRIBUTING.md @@ -48,7 +48,7 @@ This is a multi-language SDK repository. Install the tools for the SDK(s) you pl 1. Install npm dependencies: ```bash cd nodejs && npm ci - cd ../test/harness && npm ci + cd test/harness && npm ci ``` 1. Install .NET dependencies: `cd dotnet && dotnet restore` From 06a5efcab1e659f365c5e2530667f55b5e9ec7e4 Mon Sep 17 00:00:00 2001 From: "copilot-swe-agent[bot]" <198982749+Copilot@users.noreply.github.com> Date: Thu, 22 Jan 2026 04:37:08 +0000 Subject: [PATCH 5/5] Clarify commands are run from repository root Added "(from the repository root)" to npm install step for clarity Co-authored-by: friggeri <106686+friggeri@users.noreply.github.com> --- CONTRIBUTING.md | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/CONTRIBUTING.md b/CONTRIBUTING.md index a4a48c6c..0bf829f3 100644 --- a/CONTRIBUTING.md +++ b/CONTRIBUTING.md @@ -45,7 +45,7 @@ This is a multi-language SDK repository. Install the tools for the SDK(s) you pl ### .NET SDK 1. Install [.NET 8.0+](https://dotnet.microsoft.com/download) 1. Install [Node.js](https://nodejs.org/) (v18+) (the .NET tests depend on a TypeScript-based test harness) -1. Install npm dependencies: +1. Install npm dependencies (from the repository root): ```bash cd nodejs && npm ci cd test/harness && npm ci