[client] Log duplicate client ui pid

[lixmal]

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 enhances the duplicate-process check by returning the PID of any existing client UI process and logging it when detected.

• IsAnotherProcessRunning now returns the PID of a duplicate process alongside the boolean and error.
• Updated the main entrypoint to receive the PID and include it in the warning log.

Reviewed Changes

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

File	Description
client/ui/process/process.go	Changed IsAnotherProcessRunning signature to (int32, bool, error) and updated all return points to include PID.
client/ui/client_ui.go	Updated call to IsAnotherProcessRunning to unpack the PID and log it with Warnf.

Comments suppressed due to low confidence (2)

client/ui/client_ui.go:98

• [nitpick] Consider renaming the local variable pid to something like duplicatePID to clearly distinguish it from the current process's PID and improve readability.

log.Warnf("another process is running with pid %d, exiting", pid)

client/ui/process/process.go:36

• Add or extend unit tests for IsAnotherProcessRunning to cover both the 'no duplicate' case (returning PID=0) and the case where a duplicate process is found, validating the returned PID.

return 0, false, nil