[client] avoid overwriting state manager on iOS

[pascal-fischer]

Describe your changes

Issue ticket number and link

Stack

Checklist

• Is it a bug fix
• Is a typo/documentation fix
• Is a feature enhancement
• It is a refactor
• Created tests that fail without the change (if possible)
• Extended the README / documentation, if necessary

By submitting this pull request, you confirm that you have read and agree to the terms of the Contributor License Agreement.

[sonarqubecloud]

Quality Gate passed

Issues
0 New issues
0 Accepted issues

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

See analysis details on SonarQube Cloud

[Copilot]

Pull Request Overview

This PR centralizes how the engine’s state manager path is determined, ensuring that on iOS the mobile-specific path isn’t subsequently overwritten by the default.

• Introduced a path variable set to the default state path before the iOS check
• Overrode path with mobileDep.StateFilePath when runtime.GOOS == "ios"
• Consolidated engine.stateManager = statemanager.New(path) into one call

Comments suppressed due to low confidence (3)

client/internal/engine.go:245

• [nitpick] Consider renaming the local variable path to statePath to avoid confusion with the Go path package and improve readability.

path := statemanager.GetDefaultStatePath()

client/internal/engine.go:246

• Errors from createFile are currently ignored. You may want to return or log the error to make failures creating the state file visible.

if runtime.GOOS == "ios" {

client/internal/engine.go:257

• Add unit tests to verify that on iOS mobileDep.StateFilePath is used, and on other platforms the default path is used for the state manager.

engine.stateManager = statemanager.New(path)