feat(mail): add the cross-request toolbar tooltip and the missing-file text to MailMessage.

[terabytesoftw]

Pull Request

• Breaking change (fix or feature that would cause existing functionality to change)
• Bugfix (non-breaking change that fixes an issue)
• CI/build configuration
• Documentation update
• New feature (non-breaking change that adds functionality)
• Refactoring (no functional changes)

[coderabbitai]

Navigate logical layers of code changes, visualize relationships, and explore their blast radius.

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

⚙️ Run configuration

Configuration used: Organization UI

Review profile: ASSERTIVE

Plan: Advanced

Run ID: 61244fa6-62de-498b-ba9c-85ca36bf422e

📥 Commits

Reviewing files that changed from the base of the PR and between 8f9a8b7 and ffb9548.

📒 Files selected for processing (1)

• src/Panel/Mail/MailMessage.php

Included review availability: Your plan provides up to 1 included review per hour; 0 remain after this review.

📜 Recent review details

⏰ Context from checks skipped due to timeout. (14)

• GitHub Check: mutation / PHP 8.5-ubuntu-latest
• GitHub Check: phpunit / PHP 8.3-ubuntu-latest
• GitHub Check: phpunit / PHP 8.3-windows-2022
• GitHub Check: phpunit / PHP 8.4-ubuntu-latest
• GitHub Check: phpunit / PHP 8.5-windows-2022
• GitHub Check: phpunit / PHP 8.5-ubuntu-latest
• GitHub Check: phpunit / PHP 8.4-windows-2022
• GitHub Check: composer-require-checker / PHP 8.5-ubuntu-latest
• GitHub Check: easy-coding-standard / PHP 8.5-ubuntu-latest
• GitHub Check: phpstan / PHP 8.5-ubuntu-latest
• GitHub Check: Verify Vite build reproduces dist.
• GitHub Check: Analyze (actions)
• GitHub Check: mutation / PHP 8.5-ubuntu-latest
• GitHub Check: Verify Vite build reproduces dist.

🔇 Additional comments (2)

src/Panel/Mail/MailMessage.php (2)

82-85: LGTM!

---

131-131: 🔒 Security & Privacy | 🛡️ Detected with Advanced Tier

PREVIOUS_REQUEST has no tooltip call site in this repository.

This change only adds an enum value. No code formats it or renders it as HTML, so it does not introduce an escaping issue here.

Likely an incorrect or invalid review comment.

---

📝 Summary

Summary by CodeRabbit

• User Interface
  • Missing email downloads now display a clear “Mail file not found” message.
  • A toolbar tooltip identifies the method and path of the previous request.

Walkthrough

The MailMessage enum adds a response string for a missing .eml file and a tooltip template for identifying a prior request by method and path.

Changes

Mail messages

Layer / File(s)	Summary
Mail message values src/Panel/Mail/MailMessage.php	Adds FILE_NOT_FOUND for a missing mail file response and PREVIOUS_REQUEST with method and path placeholders for a tooltip.

Priority: ⬇️ Low

Estimated code review effort: 1 (Trivial) | ~4 minutes

Change: Feature

Merge Risk: ⚪ Minimal · up to ffb95

The two message values introduce no demonstrated merge-blocking behavior; the change appears ready subject to normal checks.

🚥 Pre-merge checks | ✅ 5

✅ Passed checks (5 passed)

Check name	Status	Explanation
Title check	✅ Passed	The title clearly identifies both changes: the cross-request toolbar tooltip and the missing-file text in MailMessage.
Description check	✅ Passed	The description identifies the pull request as a new, non-breaking feature, which matches the changeset.
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff. Analyzed 0 functions across 1…
Linked Issues check	✅ Passed	Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check	✅ Passed	Check skipped because no linked issues were found for this pull request.

✨ Finishing Touches

📝 Generate docstrings

• Commit to this branch
• Create a new PR

🧪 Generate unit tests (beta)

• Commit to this branch
• Create a new PR

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{A rabbit reads the mail today
A missing file gets words to say
The prior request leaves method, path
A tooltip shows the message’s path
Then hops away, content with its cache}

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

[codecov]

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 100.00%. Comparing base (8f9a8b7) to head (ffb9548).
⚠️ Report is 1 commits behind head on main.
✅ All tests successful. No failed tests found.

Additional details and impacted files

@@             Coverage Diff             @@
##                main       #72   +/-   ##
===========================================
  Coverage     100.00%   100.00%           
  Complexity      2391      2391           
===========================================
  Files            173       173           
  Lines           8723      8723           
===========================================
  Hits            8723      8723           

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

🚀 New features to boost your workflow:

• 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.