refactor: extract scheduleTermination helper

[send]

Summary

• Extract duplicated termination scheduling pattern (DispatchWorkItem creation + asyncAfter) into scheduleTermination(after:) helper

Test plan

• ./scripts/test.sh passes (16 tests)

🤖 Generated with Claude Code

[Copilot]

Pull request overview

Refactors main.swift to remove duplication in how the app schedules its self-termination after sending a notification or when launched for a stale click, by centralizing the pattern into a helper.

Changes:

• Extracted duplicated DispatchWorkItem + DispatchQueue.main.asyncAfter termination scheduling into scheduleTermination(after:).
• Updated both call sites to use the new helper.

Comments suppressed due to low confidence (1)

Sources/main.swift:245

• scheduleTermination assigns to global terminationWork and then force-unwraps it when scheduling (terminationWork!). This unwrap is unnecessary and makes the helper more brittle if it’s edited later. Prefer creating a local DispatchWorkItem, assign it to terminationWork, and pass the local variable to asyncAfter (optionally canceling any existing terminationWork before overwriting).

func scheduleTermination(after seconds: TimeInterval) {
    terminationWork = DispatchWorkItem { NSApplication.shared.terminate(nil) }
    DispatchQueue.main.asyncAfter(deadline: .now() + seconds, execute: terminationWork!)
}

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

[Copilot]

Pull request overview

Copilot reviewed 1 out of 1 changed files in this pull request and generated no new comments.

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.