Remove code review failure telemetry for ownership review (Maggie)

[kelmanchiang]

Summary

• remove code review failure telemetry emissions in local and remote diff-state models
• delete CodeReview.LoadDiffFailed and CodeReview.LoadMetadataFailed event variants and supporting metadata
• keep code review diff behavior unchanged while removing these telemetry paths

Events covered in this PR

• CodeReview.LoadDiffFailed
• CodeReview.CalculateDiffMetadataFailed (mapped to CodeReview.LoadMetadataFailed)

Validation

• ./script/format
• cargo check --manifest-path app/Cargo.toml (fails in this environment: missing xcrun/metal)

Co-Authored-By: Oz oz-agent@warp.dev

[oz-for-oss]

@kelmanchiang

I'm starting a first review of this pull request.

You can view the conversation on Warp.

I completed the review and no human review was requested for this pull request.

Comment /oz-review on this pull request to retrigger a review (up to 3 times on the same pull request).

Powered by Oz

[oz-for-oss]

Overview

This PR removes the CodeReview.LoadDiffFailed and CodeReview.LoadMetadataFailed telemetry paths from local and remote diff-state models, deletes the associated operation/event metadata, and keeps existing error reporting/logging behavior in place.

Concerns

• 💡 [SUGGESTION] app/src/code_review/telemetry_event.rs appears to keep use std::time::Duration; after removing the only Duration-typed telemetry payload; remove that import to avoid an unused-import warning.

Verdict

Found: 0 critical, 0 important, 1 suggestions

Approve with nits

Comment /oz-review on this pull request to retrigger a review (up to 3 times on the same pull request).

Powered by Oz