Skip to content

fix(firetv): preserve connection errors in status summary - #26

Open
xuyua9 wants to merge 1 commit into
postigodev:mainfrom
xuyua9:fix/firetv-status-connection-error-xuyua9
Open

fix(firetv): preserve connection errors in status summary#26
xuyua9 wants to merge 1 commit into
postigodev:mainfrom
xuyua9:fix/firetv-status-connection-error-xuyua9

Conversation

@xuyua9

@xuyua9 xuyua9 commented Aug 27, 2026

Copy link
Copy Markdown
Contributor

Summary

  • preserve the root cause when get_status cannot connect to a Fire TV
  • keep the existing actionable recovery guidance and successful status behavior
  • add a regression test for the rendered connection error

Context

Issue #9 asked for clearer Fire TV connection errors. This focused follow-up addresses the remaining unwrap_or(false) path that discarded the ADB failure details.

Validation

  • cargo fmt --all -- --check
  • cargo test -p desk_remote_core firetv
  • git diff --check

@vercel

vercel Bot commented Aug 27, 2026

Copy link
Copy Markdown

@ming1523 is attempting to deploy a commit to the postigodev's projects 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.

2 participants