Add report_error log frequency mode

[alokedesai]

Description

Adds a ReportErrorLogMode enum for report_error! so call sites can choose between the existing default EveryTime behavior and a new OncePerRun mode.

The OncePerRun mode uses a per-macro-invocation static AtomicBool, so each call site logs the first time it is reached during the app run and suppresses later hits. The default one-argument report_error!(err) form remains backwards-compatible and logs every time.

Also threads the same option through report_if_error!(result, log_mode) for callers that use the result helper.

Linked Issue

N/A — Slack request.

• The linked issue is labeled ready-to-spec or ready-to-implement.
• Where appropriate, screenshots or a short video of the implementation are included below (especially for user-visible or UI changes).

Testing

Added warp_core::errors::tests::report_error_log_mode_controls_log_frequency, which installs a test logger and verifies:

• default report_error!(err) logs twice for two hits
• ReportErrorLogMode::EveryTime logs twice for two hits
• ReportErrorLogMode::OncePerRun logs once for two hits of the same macro callsite

Validation run:

• ./script/format
• ./script/format --check && cargo clippy -p warp_core --all-targets --tests -- -D warnings
• cargo nextest run -p warp_core -E 'test(report_error_log_mode_controls_log_frequency)' — 1 passed
• cargo nextest run -p warp_core — 44 passed
• cargo clippy --workspace --exclude warp_completer --all-targets --tests -- -D warnings
• cargo clippy -p warp_completer --all-targets --tests -- -D warnings

I also tried cargo clippy --workspace --all-targets --all-features --tests -- -D warnings; it failed before reaching this change because release-bundle app binaries expected generated channel config files such as stable_config.json, dev_config.json, local_config.json, and preview_config.json under OUT_DIR.

• I have manually tested my changes locally with ./script/run

Screenshots / Videos

N/A — no UI change.

Agent Mode

• Warp Agent Mode - This PR was created via Warp's AI Agent Mode

CHANGELOG-NONE

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

Conversation: https://staging.warp.dev/conversation/b3b97505-1e9f-427c-b87d-7f8757773bd3
Run: https://oz.staging.warp.dev/runs/019e7f0b-9f95-72e6-bb15-5b5b3ad074db

This PR was generated with Oz.

[oz-for-oss]

@alokedesai

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

Adds ReportErrorLogMode so report_error! and report_if_error! callers can opt into per-run log suppression while preserving the default every-time behavior. The implementation is small, localized to warp_core error reporting, and includes a focused logger-based test for the direct report_error! behavior.

Concerns

• No blocking correctness, security, or spec-alignment concerns found.
• One non-blocking test coverage suggestion is noted inline for the new macro surface.

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