Move showProgressNotification() to be the first thing in Workers' doW…

[BurningAXE]

JIRA ticket
Will be released in: 2025.3.0

Root cause analysis (for bugfixes only)

First known affected version: 2025.3.0-dev

• In all workers the first thing we do is logCrashlytics() and then we call showProgressNotification()
• logCrashlytics() is a little misleading because apart from logging in Crashlytics, it saves logs in a local file. This can be time consuming
• In rare cases when device's CPU is overwhelmed this can delay the call to showProgressNotification() (and consequently the required startForeground()) beyond the timeout that triggers a ForegroundServiceDidNotStartInTimeException()

Notable changes

• swaps the order so that showProgressNotification() is the first thing that each Worker does in its doWork()

Testing guidance

• Open sync screen and tap sync buttons on events and images like crazy for a minute or two - if no crash - celebrate!

Additional work checklist

• Effect on other features and security has been considered
• Design document marked as "In development" (if applicable)
• External (Gitbook) and internal (Confluence) Documentation is up to date (or ticket created)
• Test cases in Testiny are up to date (or ticket created)
• Other teams notified about the changes (if applicable)

[Copilot]

Pull Request Overview

This PR moves the showProgressNotification() call to be the first statement in each Worker's doWork() method, appearing before crashlyticsLog("Started"), to prevent timeout crashes.

• Reorders initialization sequence in worker classes to show progress notification first
• Ensures progress notification displays immediately when workers start
• Addresses a specific timeout crash issue by prioritizing user-visible feedback

Reviewed Changes

Copilot reviewed 9 out of 9 changed files in this pull request and generated 1 comment.

Show a summary per file

File	Description
ProjectConfigDownSyncWorker.kt	Moves showProgressNotification() before crashlyticsLog()
DeviceConfigDownSyncWorker.kt	Moves showProgressNotification() before crashlyticsLog()
EventUpSyncUploaderWorker.kt	Moves showProgressNotification() before crashlyticsLog()
EventSyncMasterWorker.kt	Moves showProgressNotification() before crashlyticsLog()
EventStartSyncReporterWorker.kt	Moves showProgressNotification() before crashlyticsLog()
EventEndSyncReporterWorker.kt	Moves showProgressNotification() before crashlyticsLog()
BaseEventDownSyncDownloaderWorker.kt	Moves showProgressNotification() before crashlyticsLog()
RealmToRoomMigrationWorker.kt	Reorders multiple initialization calls including showProgressNotification()
StoreUserLocationIntoCurrentSessionWorker.kt	Moves showProgressNotification() before crashlyticsLog()

---

_{Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.}

[sonarqubecloud]

Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
100.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud