Skip to content

test(wallpaper): catch the daemon query/restore hang in awww/swww - #14

Open
Delcado19 wants to merge 1 commit into
HyDE-Project:mainfrom
Delcado19:fix/wallpaper-backend-query-hang
Open

Delcado19 wants to merge 1 commit into
HyDE-Project:mainfrom
Delcado19:fix/wallpaper-backend-query-hang

Conversation

@Delcado19

Copy link
Copy Markdown
Contributor

Summary

  • Reproduces (with a daemon stub that never returns) the hang in wallpaper.awww.sh/wallpaper.swww.sh when the awww/swww daemon is running but unresponsive: the health-check (<backend> query, then <backend> restore after a restart) runs as a plain foreground call with no bound, unlike the apply command it precedes.
  • Static-checks that every query/restore call in both backends stays wrapped in timeout.
  • Companion to the HyDE-side fix (opened as a separate PR against HyDE-Project/HyDE).

Test plan

  • New test_wallpaper_backend_query_hang case added to the suite

Reproduced live: a wallpaper daemon that never returns a status makes
the unguarded "<backend> query"/"<backend> restore" calls in
wallpaper.awww.sh and wallpaper.swww.sh block theme.switch.sh forever,
the same class of bug 137eda17 fixed for the apply command but left
open for the daemon health-check calls. Static-checks that every
query/restore call in both backends stays wrapped in timeout.
@coderabbitai

coderabbitai Bot commented Sep 21, 2026

Copy link
Copy Markdown
Contributor

Important

  • 🔍 Trigger review

This repository does not receive automatic reviews because it has fewer than 10 stars.

⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Advanced

Run ID: 526b9b5a-85d2-42b7-b9e4-c01894f9d362


Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

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.

1 participant