chore(notifications): use uuids for notification sandbox

[gbirman]

No description provided.

[coderabbitai]

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: ASSERTIVE

Plan: Pro

Run ID: 840b67f4-4780-4881-890e-e014c18e110e

📥 Commits

Reviewing files that changed from the base of the PR and between d9f6c3f and ed84328.

📒 Files selected for processing (1)

• rust/cloud-storage/notification_sandbox/src/main.rs

---

📝 Walkthrough

Summary by CodeRabbit

• Bug Fixes
  • Notification identifiers are now generated uniquely for each invocation instead of using fixed values, improving notification isolation and testing accuracy.

Walkthrough

The notification sandbox's main.rs file was modified to generate fresh UUIDv7 strings dynamically in run_notification_cycle. Previously hardcoded UUID values for notification_entity and message_id are now created per invocation instead of using static constants.

Changes

Cohort / File(s)	Summary
Notification ID Generation rust/cloud-storage/notification_sandbox/src/main.rs	Modified run_notification_cycle to generate fresh UUIDv7 strings for notification_entity and message_id on each invocation, replacing previously hardcoded values.

🚥 Pre-merge checks | ✅ 3 | ❌ 1

❌ Failed checks (1 inconclusive)

Check name	Status	Explanation	Resolution
Description check	❓ Inconclusive	No pull request description was provided by the author, making it impossible to assess relevance to the changeset.	Add a description explaining the rationale for using UUIDs for notification identifiers and any relevant context or testing performed.

✅ Passed checks (3 passed)

Check name	Status	Explanation
Title check	✅ Passed	The title follows conventional commits format with 'chore:' prefix, is under 72 characters (56 characters), and accurately describes the main change of using UUIDs in the notification sandbox.
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.

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

---

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.}