[Q1 Quality] WearOS - Improve login flow

[sztomek]

Description

This PR improves the watch login sync experience by:

1. Proper Error Propagation - Errors now flow from WatchSync through to the ViewModel and UI state
2. Timeout Handling - Adds a 30-second timeout for initial connection to the phone
3. Race Condition Fix - Prevents multiple concurrent sync operations from running simultaneously
4. Retry Mechanism - Provides a retrySync() function for users to manually retry failed syncs
5. Type-Safe State Management - Uses sealed interfaces to represent sync states (Idle, Syncing, Success, Failed)
6. Comprehensive Testing - Adds unit tests for both WatchSync and WearMainActivityViewModel

This is the first part of the job focusing only on the logic pieces. UI will be updated in a subsequent PR.

Fixes PCDROID-408

Testing Instructions

Just review the code and make sure the pipeline is green

Checklist

• If this is a user-facing change, I have added an entry in CHANGELOG.md
• Ensure the linter passes (./gradlew spotlessApply to automatically apply formatting/linting)
• I have considered whether it makes sense to add tests for my changes
• All strings that need to be localized are in modules/services/localization/src/main/res/values/strings.xml
• Any jetpack compose components I added or changed are covered by compose previews
• I have updated (or requested that someone edit) the spreadsheet to reflect any new or changed analytics.

[dangermattic]

	1 Warning
⚠️	This PR is larger than 500 lines of changes. Please consider splitting it into smaller PRs for easier and faster reviews.

Generated by 🚫 Danger

[Copilot]

Pull request overview

This PR improves the WearOS watch login synchronization flow by adding proper error handling, timeout mechanisms, and retry capabilities. The changes focus on the logic layer, with UI updates planned for a subsequent PR.

Changes:

• Adds type-safe state management with sealed interfaces for sync states and errors
• Implements a 30-second timeout for watch-to-phone connection attempts
• Adds race condition prevention for concurrent sync operations
• Provides comprehensive unit test coverage for WatchSync and WearMainActivityViewModel

Reviewed changes

Copilot reviewed 6 out of 6 changed files in this pull request and generated 7 comments.

Show a summary per file

File	Description
wear/src/main/kotlin/au/com/shiftyjelly/pocketcasts/wear/ui/authentication/WatchSyncState.kt	Defines sealed interfaces for sync states (Idle, Syncing, Success, Failed) and error types
wear/src/main/kotlin/au/com/shiftyjelly/pocketcasts/wear/WearMainActivityViewModel.kt	Adds timeout handling, retry mechanism, and proper state management for watch sync
modules/features/account/src/main/java/au/com/shiftyjelly/pocketcasts/account/watchsync/WatchSync.kt	Improves error propagation with Result return type and exception handling
wear/src/test/kotlin/au/com/shiftyjelly/pocketcasts/wear/WearMainActivityViewModelTest.kt	Adds unit tests for ViewModel sync functionality
modules/features/account/src/test/java/au/com/shiftyjelly/pocketcasts/account/watchsync/WatchSyncTest.kt	Adds comprehensive unit tests for WatchSync error handling
modules/services/localization/src/main/res/values/strings.xml	Adds localized strings for sync status messages and error states (for future UI work)

[Copilot]

Pull request overview

Copilot reviewed 6 out of 6 changed files in this pull request and generated 6 comments.

[Copilot]

Pull request overview

Copilot reviewed 8 out of 8 changed files in this pull request and generated 9 comments.

[Copilot]

Pull request overview

Copilot reviewed 10 out of 10 changed files in this pull request and generated 8 comments.

[Copilot]

Pull request overview

Copilot reviewed 11 out of 11 changed files in this pull request and generated 5 comments.