Skip to content

Commit e88df1b

Browse files
patnikoCopilot
andcommitted
Fix scenario tests: paths, verifications, streaming, and parallel execution
- Fix relative paths in TS package.json (43 files) and Python requirements.txt (33 files) - Add RollForward to C# csproj files for .NET 8/10 compat - Remove soft-pass fallbacks in verify.sh — tests now hard-fail on missing patterns - Fix Go permissions bug (req.Kind → req.ToolName) and add ToolName to SDK type - Fix Python/Go availableTools: empty list was omitted instead of sent as [] - Fix streaming event names (assistant.message.chunk → assistant.message_delta) - Fix TS streaming subscription (session.on('event') → typed subscription) - Fix Python streaming enum comparison (event.type.value) - Add permission handlers to Go skills scenario - Switch scenarios to claude-haiku-4.5 for faster execution - Parallelize verify.sh with live progress bar and per-SDK status icons - Fix parallel pip install race with pre-install and import check - Remove go.sum files from tracking - Remove hardcoded OAuth client ID from C# gh-app scenario Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
1 parent 0f6ecd3 commit e88df1b

72 files changed

Lines changed: 34 additions & 631 deletions

File tree

Some content is hidden

Large Commits have some content hidden by default. Use the searchbox below for content that may be hidden.

test/scenarios/.gitignore

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -84,3 +84,4 @@ reasoning-effort-go
8484
reconnect-go
8585
byok-openai-go
8686
token-sources-go
87+
go.sum

test/scenarios/auth/byok-anthropic/go/go.sum

Lines changed: 0 additions & 4 deletions
This file was deleted.

test/scenarios/auth/byok-azure/go/go.sum

Lines changed: 0 additions & 4 deletions
This file was deleted.

test/scenarios/auth/byok-ollama/go/go.sum

Lines changed: 0 additions & 4 deletions
This file was deleted.

test/scenarios/auth/byok-openai/go/go.sum

Lines changed: 0 additions & 4 deletions
This file was deleted.

test/scenarios/auth/byok-openai/verify.sh

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -70,7 +70,7 @@ check "TypeScript (install)" bash -c "cd '$SCRIPT_DIR/typescript' && npm install
7070
check "TypeScript (build)" bash -c "cd '$SCRIPT_DIR/typescript' && npm run build 2>&1"
7171

7272
# Python: install + syntax
73-
check "Python (install)" bash -c "cd '$SCRIPT_DIR/python' && pip3 install -r requirements.txt --quiet 2>&1"
73+
check "Python (install)" bash -c "python3 -c 'import copilot' 2>/dev/null || (cd '$SCRIPT_DIR/python' && pip3 install -r requirements.txt --quiet 2>&1)"
7474
check "Python (syntax)" bash -c "python3 -c \"import ast; ast.parse(open('$SCRIPT_DIR/python/main.py').read()); print('Syntax OK')\""
7575

7676
# Go: build

test/scenarios/auth/gh-app/csharp/Program.cs

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -3,7 +3,8 @@
33
using GitHub.Copilot.SDK;
44

55
// GitHub OAuth Device Flow
6-
var clientId = Environment.GetEnvironmentVariable("GITHUB_CLIENT_ID") ?? "Iv1.b507a08c87ecfe98";
6+
var clientId = Environment.GetEnvironmentVariable("GITHUB_OAUTH_CLIENT_ID")
7+
?? throw new InvalidOperationException("Missing GITHUB_OAUTH_CLIENT_ID");
78

89
var httpClient = new HttpClient();
910
httpClient.DefaultRequestHeaders.Add("Accept", "application/json");

test/scenarios/auth/gh-app/go/go.sum

Lines changed: 0 additions & 4 deletions
This file was deleted.

test/scenarios/auth/gh-app/verify.sh

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -68,7 +68,7 @@ echo ""
6868

6969
check "TypeScript (install)" bash -c "cd '$SCRIPT_DIR/typescript' && npm install --ignore-scripts 2>&1"
7070
check "TypeScript (build)" bash -c "cd '$SCRIPT_DIR/typescript' && npm run build 2>&1"
71-
check "Python (install)" bash -c "cd '$SCRIPT_DIR/python' && pip3 install -r requirements.txt --quiet 2>&1"
71+
check "Python (install)" bash -c "python3 -c 'import copilot' 2>/dev/null || (cd '$SCRIPT_DIR/python' && pip3 install -r requirements.txt --quiet 2>&1)"
7272
check "Python (syntax)" bash -c "python3 -c \"import ast; ast.parse(open('$SCRIPT_DIR/python/main.py').read()); print('Syntax OK')\""
7373
check "Go (build)" bash -c "cd '$SCRIPT_DIR/go' && go mod tidy && go build -o gh-app-go . 2>&1"
7474

test/scenarios/auth/token-sources/README.md

Lines changed: 0 additions & 53 deletions
This file was deleted.

0 commit comments

Comments
 (0)