Skip to content
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension


Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
6 changes: 2 additions & 4 deletions dotnet/test/E2E/AskUserE2ETests.cs
Original file line number Diff line number Diff line change
Expand Up @@ -93,12 +93,10 @@ public async Task Should_Handle_Freeform_User_Input_Response()
}
});

await session.SendAsync(new MessageOptions
var response = await session.SendAndWaitAsync(new MessageOptions
{
Prompt = "Ask me a question using ask_user and then include my answer in your response. The question should be 'What is your favorite color?'"
});

var response = await TestHelper.GetFinalAssistantMessageAsync(session);
}, TimeSpan.FromSeconds(120));

// Should have received a request
Assert.NotEmpty(userInputRequests);
Expand Down
72 changes: 36 additions & 36 deletions nodejs/package-lock.json

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

2 changes: 1 addition & 1 deletion nodejs/package.json
Original file line number Diff line number Diff line change
Expand Up @@ -56,7 +56,7 @@
"author": "GitHub",
"license": "MIT",
"dependencies": {
"@github/copilot": "^1.0.55-6",
"@github/copilot": "^1.0.55-7",
"vscode-jsonrpc": "^8.2.1",
"zod": "^4.3.6"
},
Expand Down
2 changes: 1 addition & 1 deletion nodejs/samples/package-lock.json

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

7 changes: 3 additions & 4 deletions python/e2e/test_pending_work_resume_e2e.py
Original file line number Diff line number Diff line change
Expand Up @@ -510,11 +510,10 @@ async def blocking_external_tool(args):
)
assert tool_result.success

answer = await get_final_assistant_message(
session2, timeout=PENDING_WORK_TIMEOUT
)
# continue_pending_work=False may interrupt agent continuation before
# a final assistant message, but the pending call should still accept
# an explicit completion.
assert invocation_count == 1
assert "EXTERNAL_RESUMED_BETA" in (answer.data.content or "")

await session2.disconnect()
finally:
Expand Down
28 changes: 13 additions & 15 deletions rust/tests/e2e/mode_handlers.rs
Original file line number Diff line number Diff line change
@@ -1,6 +1,8 @@
use std::sync::Arc;

use async_trait::async_trait;
use github_copilot_sdk::generated::SessionMode;
use github_copilot_sdk::generated::api_types::ModeSetRequest;
use github_copilot_sdk::generated::session_events::{
AutoModeSwitchCompletedData, AutoModeSwitchRequestedData,
AutoModeSwitchResponse as EventAutoModeSwitchResponse, ExitPlanModeAction,
Expand All @@ -11,7 +13,6 @@ use github_copilot_sdk::handler::{
ExitPlanModeHandler, ExitPlanModeResult,
};
use github_copilot_sdk::{ExitPlanModeData, SessionConfig, SessionId};
use serde_json::json;
use tokio::sync::mpsc;

use super::support::{
Expand Down Expand Up @@ -113,22 +114,19 @@ async fn should_invoke_exit_plan_mode_handler_when_model_uses_tool() {
|event| event.parsed_type() == SessionEventType::SessionIdle,
));

let send_result = session
.client()
.call(
"session.send",
Some(json!({
"sessionId": session.id().as_str(),
"prompt": PLAN_PROMPT,
"mode": "plan",
})),
)
session
.rpc()
.mode()
.set(ModeSetRequest {
mode: SessionMode::Plan,
})
.await
.expect("set plan mode");
let message_id = session
.send(PLAN_PROMPT)
.await
.expect("send plan-mode prompt");
assert!(
send_result.get("messageId").is_some(),
"expected messageId in send result"
);
assert!(!message_id.is_empty(), "expected messageId in send result");

let (session_id, request) =
recv_with_timeout(&mut request_rx, "exit-plan-mode request").await;
Expand Down
72 changes: 36 additions & 36 deletions test/harness/package-lock.json

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

2 changes: 1 addition & 1 deletion test/harness/package.json
Original file line number Diff line number Diff line change
Expand Up @@ -11,7 +11,7 @@
"test": "vitest run"
},
"devDependencies": {
"@github/copilot": "^1.0.55-6",
"@github/copilot": "^1.0.55-7",
"@modelcontextprotocol/sdk": "^1.26.0",
"@types/node": "^25.3.3",
"@types/node-forge": "^1.3.14",
Expand Down
Loading
Loading