fix(stats): address success rate review comments from #2988

[riderx]

Summary (AI generated)

• getUpdateStatsCF and get_update_stats() now include apps with install/fail outcomes even when no get event occurred in the snapshot window.
• Global total is aggregated from all apps in the query result, not only post-filtered rows.
• Public /data headline + daily success rate now uses the same app_log set/failure event stream as platform/country/version breakdowns.

Motivation (AI generated)

Follow-up to #2988 review comments: the global success rate still dropped install/fail-only apps due to a get > 0 filter, and public headline metrics mixed version_usage with app_log breakdowns.

Business Impact (AI generated)

More accurate global_stats snapshots and internally consistent public success rate numbers across headline and dimensional views.

Test Plan (AI generated)

• bunx vitest run tests/public-live-update-metrics.unit.test.ts tests/public-stats.unit.test.ts
• Compare /private/website_stats/live_updates headline vs platform breakdown success rates after deploy
• Verify next global_stats cron stores success_rate including install/fail-only apps

Generated with AI

  

---

^{Need help on this PR? Tag @codesmith-bot with what you need. Autofix is disabled.}

[coderabbitai]

Important

Review skipped

Draft detected.

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: Organization UI

Review profile: ASSERTIVE

Plan: Pro

Run ID: ef4f36c1-556f-490d-96b2-7065634b8057

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

• 🔍 Trigger review

---

_{Comment @coderabbitai help to get the list of available commands.}

[codspeed-hq]

Merging this PR will not alter performance

✅ 43 untouched benchmarks
⏩ 2 skipped benchmarks¹

---

_{Comparing cursor/fix-success-rate-review-04fe (997034b) with main (bc40030)}

Footnotes

1. 2 benchmarks were skipped, so the baseline results were used instead. If they were deleted from the codebase, click here and archive them to remove them from the performance reports. ↩

[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