Skip to content

Conversation

@sahillllllllll-bit
Copy link

@sahillllllllll-bit sahillllllllll-bit commented Jan 17, 2026

Description

This PR fixes an obvious data issue in the tools configuration where AsyncAPI CLI was listed multiple times across the Tools page.

Specifically, it:

  • Removes the duplicate AsyncAPI CLI card that pointed to a forked GitHub repository
  • Ensures the official AsyncAPI CLI is listed only once
  • Removes the incorrect listing from the Others category, keeping it under CLI only

This change is a pure data correction and does not modify any UI or filtering logic.

Current Behavior

  • The AsyncAPI CLI tool appears multiple times on the Tools page.
  • Two identical AsyncAPI CLI cards are displayed, but one links to an incorrect forked GitHub repository.
  • The AsyncAPI CLI is also listed under the Others category in addition to the CLI category.
  • This results in user confusion about which repository is the official and supported CLI.

Screen Recording of Duplicates and issue

Untitled.video.-.Made.with.Clipchamp.3.mp4

What This PR Fixes

  • Fixes duplicate AsyncAPI CLI cards appearing on the Tools page.
  • Removes the incorrect AsyncAPI CLI entry that linked to a forked GitHub repository.
  • Ensures the official AsyncAPI CLI is listed only once.
  • Removes the AsyncAPI CLI from the Others category, keeping it under the CLI category only.

Summary by CodeRabbit

  • Chores
    • Configuration cleanup and updates removing obsolete tool entries.

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

@netlify
Copy link

netlify bot commented Jan 17, 2026

Deploy Preview for asyncapi-website ready!

Built without sensitive environment variables

Name Link
🔨 Latest commit 560cde0
🔍 Latest deploy log https://app.netlify.com/projects/asyncapi-website/deploys/696b2e554a10a30008c7558e
😎 Deploy Preview https://deploy-preview-4969--asyncapi-website.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@coderabbitai
Copy link
Contributor

coderabbitai bot commented Jan 17, 2026

📝 Walkthrough

Walkthrough

Configuration file updated to remove two AsyncAPI CLI tool entries and their associated metadata from the tools registry. The changes include cleanup of references in the Others section and a minor formatting adjustment to the Others key declaration.

Changes

Cohort / File(s) Summary
Configuration cleanup
config/tools.json
Removed 2 AsyncAPI CLI entries from CLIs section with associated tool objects (title, description, links, filters); cleaned up stray references in Others section; formatted "Others" key declaration

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

Poem

🐰 AsyncAPI tools, once in our config so bright,
Now gently removed from our tools.json sight,
With a hop and a skip, we tidied the space,
Leaving our registry clean and more spare! ✨

🚥 Pre-merge checks | ✅ 3
✅ Passed checks (3 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title accurately summarizes the main change: removing duplicate AsyncAPI CLI entries across categories in the tools configuration file.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✏️ 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

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

@asyncapi-bot
Copy link
Contributor

asyncapi-bot commented Jan 17, 2026

⚡️ Lighthouse report for the changes in this PR:

Category Score
🔴 Performance 36
🟢 Accessibility 98
🟢 Best practices 92
🟢 SEO 100
🔴 PWA 33

Lighthouse ran on https://deploy-preview-4969--asyncapi-website.netlify.app/

@codecov
Copy link

codecov bot commented Jan 17, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 100.00%. Comparing base (41fd41f) to head (560cde0).

Additional details and impacted files
@@            Coverage Diff            @@
##            master     #4969   +/-   ##
=========================================
  Coverage   100.00%   100.00%           
=========================================
  Files           22        22           
  Lines          796       796           
  Branches       146       146           
=========================================
  Hits           796       796           

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

@sahillllllllll-bit
Copy link
Author

Hi @princerajpoot20
just checking whether the proposed approach above looks good to you.
Thanks.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: To Be Triaged

Development

Successfully merging this pull request may close these issues.

2 participants