Creepso/agent-desktop
1
0
Fork 0
mirror of https://github.com/lahfir/agent-desktop.git synced 2026-08-18 21:17:29 +00:00
Code Issues Projects Releases Packages Wiki Activity Actions

fix: report the probe budget the endpoint wait actually received

Browse source 
The failure details carried the pre-launch reserve, but the probe runs
against the overall deadline, so a fast launch grants it far more time
than the reserve names. The reported probe_budget_ms is now the
deadline's remainder at probe start, which is the number someone
debugging a timeout actually needs.
This commit is contained in:
Lahfir 2026-08-13 22:23:42 -07:00
parent e68c585163
commit a2ffaa6787
1 changed files with 5 additions and 2 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

7
crates/core/src/commands/launch.rs
View file

@ -43,8 +43,8 @@ pub fn execute(args: LaunchArgs, adapter: &dyn PlatformAdapter) -> Result<Value,
let lease = adapter.acquire_interaction_lease(launch_deadline)?;
let mut launched = adapter.launch_app(&app, &options, &lease)?;
if let Some(port) = options.cdp_port {
let reserve = probe_budget.unwrap_or_default();
launched.cdp = Some(verify_cdp_endpoint(port, deadline, reserve, &launched)?);
let granted = deadline.remaining();
launched.cdp = Some(verify_cdp_endpoint(port, deadline, granted, &launched)?);
}
launched.suggestion = launch_suggestion(requested_cdp_port.is_some(), &launched);
Ok(serde_json::to_value(launched)?)
@ -173,6 +173,9 @@ fn resolve_cdp_port(requested: u16) -> Result<u16, AppError> {
/// (pid, process instance, the budget the probe was given) layered onto the
/// probe's own message and evidence rather than a second, separately worded
/// error that could drift from what was actually observed.
/// `probe_budget` is the time the probe actually received: the deadline's
/// remainder at probe start, not the pre-launch reserve, which a fast launch
/// leaves far behind.
fn verify_cdp_endpoint(
port: u16,
deadline: crate::Deadline,