From 75bd2b125ac1d181c78e718e9b4d337a28ae30e6 Mon Sep 17 00:00:00 2001 From: Kiran Kashyap Date: Sun, 1 Mar 2026 20:32:52 +0530 Subject: [PATCH 1/3] Add per-language install recipes (#585) --- justfile | 33 ++++++++++++++++++++++++++------- 1 file changed, 26 insertions(+), 7 deletions(-) diff --git a/justfile b/justfile index 85cd8c61bf..fe0246353e 100644 --- a/justfile +++ b/justfile @@ -71,15 +71,34 @@ test-dotnet: @echo "=== Testing .NET code ===" @cd dotnet && dotnet test test/GitHub.Copilot.SDK.Test.csproj -# Install all dependencies -install: - @echo "=== Installing dependencies ===" - @cd nodejs && npm ci - @cd python && uv pip install -e ".[dev]" +# Install all dependencies across all languages +install: install-go install-python install-nodejs install-dotnet + @echo "✅ All dependencies installed" + +# Install Go dependencies and prerequisites for tests +install-go: install-nodejs install-test-harness + @echo "=== Installing Go dependencies ===" @cd go && go mod download + +# Install Python dependencies and prerequisites for tests +install-python: install-nodejs install-test-harness + @echo "=== Installing Python dependencies ===" + @cd python && uv pip install -e ".[dev]" + +# Install .NET dependencies and prerequisites for tests +install-dotnet: install-nodejs install-test-harness + @echo "=== Installing .NET dependencies ===" @cd dotnet && dotnet restore - @cd test/harness && npm ci --ignore-scripts - @echo "✅ All dependencies installed" + +# Install Node.js dependencies +install-nodejs: + @echo "=== Installing Node.js dependencies ===" + @cd nodejs && npm ci + +# Install test harness dependencies (used by E2E tests in all languages) +install-test-harness: + @echo "=== Installing test harness dependencies ===" + @cd test/harness && npm ci # Run interactive SDK playground playground: From 8fd1e1ad0c1c43dd2071263553a200f874fda792 Mon Sep 17 00:00:00 2001 From: Kiran Kashyap Date: Sun, 1 Mar 2026 21:13:04 +0530 Subject: [PATCH 2/3] fix #585 --- justfile | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/justfile b/justfile index fe0246353e..5cc138e162 100644 --- a/justfile +++ b/justfile @@ -83,7 +83,7 @@ install-go: install-nodejs install-test-harness # Install Python dependencies and prerequisites for tests install-python: install-nodejs install-test-harness @echo "=== Installing Python dependencies ===" - @cd python && uv pip install -e ".[dev]" + @cd python && uv venv && uv pip install -e ".[dev]" # Install .NET dependencies and prerequisites for tests install-dotnet: install-nodejs install-test-harness From 7ef0050ea642038f8028e3f78b8176e2e606b5a1 Mon Sep 17 00:00:00 2001 From: Steve Sanderson Date: Thu, 19 Mar 2026 14:49:25 +0000 Subject: [PATCH 3/3] Revert unnecessary changes in install recipes - Restore --ignore-scripts flag for test harness npm ci - Remove extraneous uv venv from Python install (uv run manages venvs) Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> --- justfile | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/justfile b/justfile index 5cc138e162..fd7fc3adbe 100644 --- a/justfile +++ b/justfile @@ -83,7 +83,7 @@ install-go: install-nodejs install-test-harness # Install Python dependencies and prerequisites for tests install-python: install-nodejs install-test-harness @echo "=== Installing Python dependencies ===" - @cd python && uv venv && uv pip install -e ".[dev]" + @cd python && uv pip install -e ".[dev]" # Install .NET dependencies and prerequisites for tests install-dotnet: install-nodejs install-test-harness @@ -98,7 +98,7 @@ install-nodejs: # Install test harness dependencies (used by E2E tests in all languages) install-test-harness: @echo "=== Installing test harness dependencies ===" - @cd test/harness && npm ci + @cd test/harness && npm ci --ignore-scripts # Run interactive SDK playground playground: