feat(main-layout): generate unique filename for YAML files on save

[tolgaozen]

Summary by CodeRabbit

Release Notes

• Bug Fixes
  • Files uploaded to the system now receive unique identifiers instead of using a fixed filename, ensuring each upload is stored separately.

_{✏️ Tip: You can customize this high-level summary in your review settings.}

[coderabbitai]

📝 Walkthrough

Walkthrough

The playground's file upload mechanism now generates unique timestamps and random IDs to create distinct YAML filenames instead of using a fixed "s.yaml" name. The updated code removes the addRandomSuffix method and deterministic naming, maintaining the existing upload and parsing flow.

Changes

Cohort / File(s)	Summary
Upload filename generation playground/src/layout/main-layout.js	Changed from hardcoded "s.yaml" to dynamically generated filename using timestamp and random value. Generates uniqueId from Date.now() and Math.random(), constructs fileName as "<uniqueId>.yaml", and removes addRandomSuffix function.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~8 minutes

Possibly related PRs

• feat(main-layout): add option to include random suffix in file upload #2725: Both PRs modify the upload filename strategy in the same file's share() method to make filenames non-deterministic, though using different approaches (timestamp-based generation vs. addRandomSuffix option).

Poem

🐰 A timestamp dances, a random seed sings,

No more "s.yaml"—now each upload takes wing!

Unique IDs flutter like carrots so bright,

The playground grows gardens of files, done right! 🥕✨

Pre-merge checks and finishing touches

✅ Passed checks (3 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Title check	✅ Passed	The PR title clearly and concisely summarizes the main change: replacing deterministic 's.yaml' with unique, timestamp-based filenames for YAML file uploads.
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✨ Finishing touches

• 📝 Generate docstrings

---

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

[coderabbitai]

Actionable comments posted: 0

🧹 Nitpick comments (1)

playground/src/layout/main-layout.js (1)

62-65: Consider using crypto.randomUUID() for better uniqueness guarantees.

The current approach using Date.now() and Math.random() provides reasonable uniqueness, but crypto.randomUUID() would offer stronger collision resistance and is now widely supported in modern browsers.

🔎 Optional refactor using crypto.randomUUID()

-// Generate unique filename to avoid conflicts
-const uniqueId = `${Date.now()}-${Math.random().toString(36).substring(2, 9)}`;
-const fileName = `${uniqueId}.yaml`;
+// Generate unique filename to avoid conflicts
+const uniqueId = crypto.randomUUID();
+const fileName = `${uniqueId}.yaml`;

📜 Review details

Configuration used: defaults

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 84020c1 and 3820206.

📒 Files selected for processing (1)

• playground/src/layout/main-layout.js

⏰ 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). (3)

• GitHub Check: Test with Coverage
• GitHub Check: Analyze (go)
• GitHub Check: Test with Coverage

🔇 Additional comments (1)

playground/src/layout/main-layout.js (1)

66-68: No issues found. The put() function signature, access option, and token parameter are all correct for @vercel/blob v2.0.0. The addRandomSuffix option that was removed is confirmed as a valid option in the library, so the removal is appropriate given the manual filename handling in the code.

[codecov]

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 82.58%. Comparing base (84020c1) to head (3820206).
⚠️ Report is 2 commits behind head on master.

Additional details and impacted files

@@            Coverage Diff             @@
##           master    #2726      +/-   ##
==========================================
+ Coverage   82.54%   82.58%   +0.04%     
==========================================
  Files          74       74              
  Lines        8125     8125              
==========================================
+ Hits         6706     6709       +3     
+ Misses        904      902       -2     
+ Partials      515      514       -1     

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

🚀 New features to boost your workflow:

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
• 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.