Skip to content

fix(cli): close sessions without starting daemon#1369

Open
EfeDurmaz16 wants to merge 1 commit into
vercel-labs:mainfrom
EfeDurmaz16:fix/clear-state-after-close-failure
Open

fix(cli): close sessions without starting daemon#1369
EfeDurmaz16 wants to merge 1 commit into
vercel-labs:mainfrom
EfeDurmaz16:fix/clear-state-after-close-failure

Conversation

@EfeDurmaz16
Copy link
Copy Markdown

Summary

  • handle single-session close/quit/exit locally using the daemon inventory
  • avoid starting a fresh daemon just to close an inactive session
  • clean up unreachable session sidecars and force-kill the recorded daemon process before the next open

Fixes #1367.

Verification

  • cargo fmt --manifest-path cli/Cargo.toml -- --check
  • cargo test --manifest-path cli/Cargo.toml test_is_transient_error_socket_not_found
  • tmpdir=$(mktemp -d); AGENT_BROWSER_SOCKET_DIR="$tmpdir" cargo run --manifest-path cli/Cargo.toml -- close --json; rmdir "$tmpdir"
  • cargo clippy --manifest-path cli/Cargo.toml --all-targets -- -D warnings
  • git diff --check

@vercel
Copy link
Copy Markdown
Contributor

vercel Bot commented May 15, 2026

@EfeDurmaz16 is attempting to deploy a commit to the Vercel Labs Team on Vercel.

A member of the Team first needs to authorize it.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

After failed close, subsequent open reports success but returns stale content from prior URL

1 participant