-
-
Notifications
You must be signed in to change notification settings - Fork 983
chore: update tools.json #4439
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
chore: update tools.json #4439
Conversation
WalkthroughRenamed two tag names in config/all-tags.json and performed extensive restructuring of tool catalog data in config/tools-automated.json and targeted repoUrl/description swaps in config/tools.json. No colors or structural schemas changed; updates are confined to JSON content values and categorizations. Changes
Sequence Diagram(s)Estimated code review effort🎯 4 (Complex) | ⏱️ ~45–60 minutes Possibly related PRs
Suggested labels
Suggested reviewers
Poem
Pre-merge checks and finishing touches✅ Passed checks (3 passed)
✨ Finishing touches🧪 Generate unit tests
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. Comment |
✅ Deploy Preview for asyncapi-website ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## master #4439 +/- ##
=========================================
Coverage 100.00% 100.00%
=========================================
Files 22 22
Lines 778 778
Branches 144 144
=========================================
Hits 778 778 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
|
⚡️ Lighthouse report for the changes in this PR:
Lighthouse ran on https://deploy-preview-4439--asyncapi-website.netlify.app/ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Actionable comments posted: 5
📜 Review details
Configuration used: Path: .coderabbit.yaml
Review profile: CHILL
Plan: Pro
📒 Files selected for processing (3)
config/all-tags.json(3 hunks)config/tools-automated.json(21 hunks)config/tools.json(8 hunks)
⏰ Context from checks skipped due to timeout of 180000ms. You can increase the timeout in your CodeRabbit configuration to a maximum of 15 minutes (900000ms). (7)
- GitHub Check: Redirect rules - asyncapi-website
- GitHub Check: Header rules - asyncapi-website
- GitHub Check: Pages changed - asyncapi-website
- GitHub Check: Automerge PR autoapproved by a bot
- GitHub Check: Lighthouse CI
- GitHub Check: Test NodeJS PR - macos-13
- GitHub Check: Test NodeJS PR - windows-latest
🔇 Additional comments (1)
config/tools-automated.json (1)
351-352: Automated catalog: fix AsyncAPI Studio repo regressionLine 351 and Line 370 mirror the same mistaken fork URLs flagged in tools.json. This will publish incorrect data via the automated catalog. Please revert to the official repository.
Apply this diff:
"repoUrl": "https://github.com/Shurtu-gal/action-test-bed" + "repoUrl": "https://github.com/asyncapi/studio" ... "repoUrl": "https://github.com/TahaShahid203/async-api-issue" + "repoUrl": "https://github.com/asyncapi/studio"Also applies to: 370-371
⛔ Skipped due to learnings
Learnt from: anshgoyalevil PR: asyncapi/website#3557 File: tests/fixtures/markdown/check-edit-links-data.js:3-11 Timestamp: 2025-01-19T04:51:41.255Z Learning: In the AsyncAPI website repository, the test data in `tests/fixtures/markdown/check-edit-links-data.js` intentionally includes inconsistent paths (with and without 'docs' prefix) to verify the script's ability to normalize and handle ambiguous path structures.
| "repoUrl": "https://github.com/hkirat/asyncapi-fork" | ||
| }, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Automated catalog: restore AsyncAPI CLI repository URLs
The automated data at Line 794, Line 812, and Line 1058 now points to a fork (hkirat/asyncapi-fork). This diverges from the supported CLI repo and will misinform downstream consumers. Please restore the official link.
Apply this diff:
"repoUrl": "https://github.com/hkirat/asyncapi-fork"
+ "repoUrl": "https://github.com/asyncapi/cli"Also applies to: 812-814, 1058-1059
🤖 Prompt for AI Agents
In config/tools-automated.json around lines 794-795 (also apply same change at
812-814 and 1058-1059), the repoUrl fields currently point to the fork
"https://github.com/hkirat/asyncapi-fork"; update these entries to the official
AsyncAPI CLI repository URL "https://github.com/asyncapi/cli" so downstream
consumers reference the supported source; ensure only the repoUrl values are
changed and keep surrounding JSON structure intact.
| "title": "HTML Template", | ||
| "description": null, | ||
| "links": { | ||
| "repoUrl": "https://github.com/asyncapi/nodejs-template" | ||
| "repoUrl": "https://github.com/dipaksodani/async-gen" | ||
| }, | ||
| "filters": { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Automated catalog: correct HTML template source
Lines 997-1023 list the HTML template under an unrelated repository. That breaks generator automation. Please revert these entries to the maintained asyncapi/html-template URL.
Apply this diff:
"repoUrl": "https://github.com/dipaksodani/async-gen"
+ "repoUrl": "https://github.com/asyncapi/html-template"
...
"repoUrl": "https://github.com/asyncapi/html-template"Also applies to: 1018-1023
🤖 Prompt for AI Agents
In config/tools-automated.json around lines 997-1023 (also adjust the duplicate
at 1018-1023), the HTML template entries point to the wrong repository
(https://github.com/dipaksodani/async-gen); update those entries to the
maintained asyncapi/html-template URL by replacing the repoUrl value with the
correct repository (https://github.com/asyncapi/html-template) for both
occurrences, ensuring the "title" and other metadata remain unchanged.
| "repoUrl": "https://github.com/Shurtu-gal/action-test-bed" | ||
| }, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Restore the official AsyncAPI Studio repository URL
Line 1340 and Line 1367 now link to personal forks instead of the maintained Studio repo, so the public catalog will direct users to unsupported code. Please revert these entries to the official repository to avoid misdirecting traffic.
Apply this diff:
"repoUrl": "https://github.com/Shurtu-gal/action-test-bed"
+ "repoUrl": "https://github.com/asyncapi/studio"
...
"repoUrl": "https://github.com/TahaShahid203/async-api-issue"
+ "repoUrl": "https://github.com/asyncapi/studio"Also applies to: 1367-1368
🤖 Prompt for AI Agents
In config/tools.json around lines 1340-1341 and also lines 1367-1368, two
repoUrl entries point to personal forks
(https://github.com/Shurtu-gal/action-test-bed) instead of the official AsyncAPI
Studio repository; update both repoUrl values to the official AsyncAPI Studio
repo URL (https://github.com/asyncapi/studio) so the public catalog links to the
maintained upstream project, preserving JSON structure and quoting.
| "repoUrl": "https://github.com/hkirat/asyncapi-fork" | ||
| }, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Point AsyncAPI CLI entries back to the official repo
Line 2405 and Line 2428 were swapped to a fork that isn’t the project homepage. That breaks the catalog and risks exposing users to stale binaries. Please restore the official CLI repository URL.
Apply this diff:
"repoUrl": "https://github.com/hkirat/asyncapi-fork"
+ "repoUrl": "https://github.com/asyncapi/cli"
...
"repoUrl": "https://github.com/asyncapi/cli"Also applies to: 2428-2429
🤖 Prompt for AI Agents
In config/tools.json around lines 2405-2406 and 2428-2429, the "repoUrl" fields
were pointed to a personal fork (https://github.com/hkirat/asyncapi-fork);
replace those values with the official AsyncAPI CLI repository URL
(https://github.com/asyncapi/cli) in both locations so the catalog references
the project homepage and avoids stale or unofficial binaries.
| "repoUrl": "https://github.com/dipaksodani/async-gen" | ||
| }, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Keep the HTML template entry aligned with the maintained upstream
Line 2809 now references an unrelated repo, leaving the catalog without the maintained template source. Please revert to the official asyncapi/html-template URL so users get the supported generator template.
Apply this diff:
"repoUrl": "https://github.com/dipaksodani/async-gen"
+ "repoUrl": "https://github.com/asyncapi/html-template"
...
"repoUrl": "https://github.com/asyncapi/html-template"Also applies to: 2836-2837
🤖 Prompt for AI Agents
In config/tools.json around lines 2809 (and also apply same change at
2836-2837), the "repoUrl" entry points to an unrelated fork
(https://github.com/dipaksodani/async-gen); replace it with the official
asyncapi HTML template repository URL
"https://github.com/asyncapi/html-template" so the catalog references the
maintained upstream template. Update each affected "repoUrl" value to the
official URL and ensure JSON formatting/commas remain valid.
Automated changes by create-pull-request GitHub action
Summary by CodeRabbit
New Features
Bug Fixes
Documentation
Chores