Skip to content

2.10.1: fix Settings crash on secret params, unset kind condition - #16

Merged
roypadina merged 4 commits into
mainfrom
fix/ntfy-secret-settings-crash
Oct 31, 2026
Merged

roypadina merged 4 commits into
mainfrom
fix/ntfy-secret-settings-crash

Conversation

@roypadina

Copy link
Copy Markdown
Owner

Fixed

  • Settings crash — opening a rule with a password field (e.g. Send to ntfy) in Settings > Actions quit the app. SecretStore.security() used Process.waitUntilExit(), which spins the main run loop; the SecureField binding getter runs it during a SwiftUI update, so the nested layout pass aborted in AttributeGraph (AG::Graph::value_set). It now waits on a terminationHandler semaphore, which blocks without re-entering the run loop.
  • Unset "Value kind" condition — the picker shows "URL" when no kind is saved, but evaluate threw missingParam, so the rule never matched and its actions never appeared in the popup. An unset kind now means URL.

Release

Version 2.10.1 (build 81) + CHANGELOG entry.

Checks

  • Debug and signed Release builds, build-for-testing green.
  • Standalone check: during a security call a 1 ms main-run-loop timer fired 64 times with the old wait, 0 with the new one.
  • Signed local build: rule editor opens without crashing; actions send the clicked item.

🤖 Generated with Claude Code

roypadina and others added 4 commits October 31, 2026 20:51
SecretStore.security() used Process.waitUntilExit(), which spins the main
run loop. The SecureField binding getter calls it during a SwiftUI update,
so the nested layout pass aborted in AttributeGraph (AG::Graph::value_set).
Wait on a terminationHandler semaphore instead: blocks without re-entering
the run loop.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
The kind picker displays "URL" when no kind was saved, but evaluate()
threw missingParam, so such a rule silently never matched and its
actions never showed in the popup.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@roypadina
roypadina merged commit 45e173b into main Oct 31, 2026
1 check passed
@roypadina
roypadina deleted the fix/ntfy-secret-settings-crash branch October 31, 2026 20:10
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