fix: resolve discord investigate link not rendering

[emmanuel-ferdman]

PR Summary

This small PR resolves the discord investigate link not rendering. Fixes #1973.

[coderabbitai]

Walkthrough

A bug fix in the Discord sender module converts a string placeholder to an f-string, enabling dynamic interpolation of the Investigate URI in Discord notifications instead of displaying the placeholder as literal text.

Changes

Cohort / File(s)	Change Summary
Discord sender URI fix src/robusta/integrations/discord/sender.py	Modified Investigate action URI construction to use f-string formatting for dynamic URI interpolation instead of treating the method call as a literal placeholder string

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

🚥 Pre-merge checks | ✅ 4 | ❌ 1

❌ Failed checks (1 warning)

Check name	Status	Explanation	Resolution
Docstring Coverage	⚠️ Warning	Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%.	Write docstrings for the functions missing them to satisfy the coverage threshold.

✅ Passed checks (4 passed)

Check name	Status	Explanation
Title check	✅ Passed	The title clearly describes the main change: fixing the discord investigate link rendering issue, which is the primary objective of this PR.
Description check	✅ Passed	The description is directly related to the changeset, referencing the specific issue (#1973) and explaining the problem being fixed.
Linked Issues check	✅ Passed	The code changes directly address issue #1973 by fixing the discord investigate link rendering through f-string interpolation instead of a literal placeholder.
Out of Scope Changes check	✅ Passed	All changes are focused on resolving the discord investigate link rendering issue; no out-of-scope modifications are present.

_{✏️ Tip: You can configure your own custom pre-merge checks in the settings.}

✨ Finishing touches

• 📝 Generate docstrings

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Post copyable unit tests in a comment

---

📜 Recent review details

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 42ed013 and 1a7437f.

📒 Files selected for processing (1)

• src/robusta/integrations/discord/sender.py

🧰 Additional context used

🧬 Code graph analysis (1)

src/robusta/integrations/discord/sender.py (1)

src/robusta/core/reporting/base.py (1)

• get_investigate_uri (318-341)

⏰ Context from checks skipped due to timeout of 90000ms. You can increase the timeout in your CodeRabbit configuration to a maximum of 15 minutes (900000ms). (1)

• GitHub Check: run_tests

🔇 Additional comments (1)

src/robusta/integrations/discord/sender.py (1)

300-300: LGTM! The f-string prefix correctly fixes the template interpolation.

The addition of the f prefix enables proper interpolation of finding.get_investigate_uri(self.account_id, self.cluster_name), resolving the issue where the literal template string was displayed instead of the actual investigate link. This matches the expected behavior seen in lines 304 and 308, which already use f-strings for similar link constructions.

---

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

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