Skip to content

Conversation

@kapishdima
Copy link

@kapishdima kapishdima commented Dec 7, 2025

  • fixed the git repo name.
  • fixed the link to componentsUrl.
  • added excludeItemTypes: ["blocks", "styles", "libs"]. Since I couldn’t find links to the blocks on the website, I’ve temporarily hidden them from the RSS. If you need them, you can just remove them from the array.

Summary by CodeRabbit

  • Updates
    • Modified RSS feed generation to filter specific component types and include components documentation reference.

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

@vercel
Copy link

vercel bot commented Dec 7, 2025

@DimaDevelopment is attempting to deploy a commit to the Retro UI Team on Vercel.

A member of the Team first needs to authorize it.

@coderabbitai
Copy link
Contributor

coderabbitai bot commented Dec 7, 2025

Walkthrough

RSS feed generation configuration updated by adding component documentation URL and item type exclusions, while simplifying the GitHub repository reference to use just the repository name instead of the full URL.

Changes

Cohort / File(s) Summary
RSS Feed Configuration
app/rss.xml/route.ts
Added componentsUrl parameter pointing to /docs/components, added excludeItemTypes array filtering out registry block, style, and library items, and simplified github.repo from full URL to repository name only.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

  • Single file modification with straightforward parameter additions
  • No logic changes or control flow modifications
  • Configuration values are clear and consistent with existing patterns

Poem

🐰 A feed refined, with filters now in place,
Components pointed to their rightful space,
The repo trimmed from URL to name,
RSS flows cleaner, still the very same!

Pre-merge checks and finishing touches

❌ Failed checks (1 warning, 1 inconclusive)
Check name Status Explanation Resolution
Docstring Coverage ⚠️ Warning Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%. You can run @coderabbitai generate docstrings to improve docstring coverage.
Title check ❓ Inconclusive The title 'Fix RSS feed' is vague and generic, using non-specific terminology that doesn't convey the actual changes made (repository name fix, componentsUrl link, or excludeItemTypes array). Consider a more descriptive title like 'Update RSS feed configuration and fix repository reference' to better reflect the specific changes in the changeset.
✅ Passed checks (1 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
✨ 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: CodeRabbit UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 10c7afd and 134b65d.

⛔ Files ignored due to path filters (1)
  • bun.lockb is excluded by !**/bun.lockb
📒 Files selected for processing (1)
  • app/rss.xml/route.ts (3 hunks)
🔇 Additional comments (3)
app/rss.xml/route.ts (3)

11-12: RSS config: componentsUrl and excludeItemTypes align with intent; verify type IDs

componentsUrl: "/docs/components" fits the expected pattern with baseUrl, and the excludeItemTypes list clearly encodes blocks/styles/libs as registry item types. The change matches the PR goal of hiding those item types from the feed while still making it easy to re-enable later.

Please just double‑check that "registry:block" | "registry:style" | "registry:lib" exactly match the item type identifiers used by generateRegistryRssFeed / your registry data, so the filtering behaves as expected.


21-21: GitHub repo now correctly uses the repository name; confirm expected shape

Switching github.repo from the full URL to "RetroUI" matches the usual { owner, repo } pattern and should fix lookups that expect just the repo slug.

Confirm in the @wandry/analytics-sdk docs that github.repo must be "RetroUI" (not the full URL) so that commit metadata and pubDateStrategy: "githubLastEdit" continue to resolve correctly.


40-40: No functional change in closing brace / trailing newline

Only a formatting normalization; behavior of the route handler is unchanged.


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

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

@ariflogs ariflogs merged commit d69ed1d into Logging-Studio:main Dec 8, 2025
1 of 2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants