Suppress update message if is trigger by desktop

[samuv]

When the thv is bundled with the Desktop app, we don’t need to show the upgrade message because it’s handled by the app’s release cycle.

[codecov]

Codecov Report

❌ Patch coverage is 71.42857% with 2 lines in your changes missing coverage. Please review.
✅ Project coverage is 67.22%. Comparing base (8075728) to head (ee7f859).

Files with missing lines	Patch %	Lines
pkg/desktop/validation.go	60.00%	1 Missing and 1 partial ⚠️

Additional details and impacted files

@@            Coverage Diff             @@
##             main    #3885      +/-   ##
==========================================
+ Coverage   67.17%   67.22%   +0.04%     
==========================================
  Files         449      449              
  Lines       45271    45278       +7     
==========================================
+ Hits        30413    30439      +26     
+ Misses      12492    12467      -25     
- Partials     2366     2372       +6     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

[eleftherias]

Because we bundle the update check with metrics, I wonder if this would cause us to lose CLI usage metrics. @lujunsan Could you take a look?

[lujunsan]

Thanks for the heads up @eleftherias .

@samuv added a comment, as what Ria pointed out is correct.

[samuv]

thanks @eleftherias and @lujunsan. I’m moving the check to where the message-printing logic lives, as suggested, so we can leave the rest untouched