Skip to content

chore: update tools.json #4306

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.

Sign up for GitHub

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

Merged
asyncapi-bot merged 5 commits into from
Aug 5, 2025
Merged

chore: update tools.json #4306

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
5 commits
Select commit Hold shift + click to select a range
c12b155
chore: update tools.json
Aug 4, 2025
b5c83a5
Merge master into update-tools/regenerateTools
asyncapi-bot-eve Aug 4, 2025
0396b73
Merge master into update-tools/regenerateTools
asyncapi-bot-eve Aug 4, 2025
6c79703
Merge master into update-tools/regenerateTools
asyncapi-bot-eve Aug 5, 2025
b98fbaf
Merge master into update-tools/regenerateTools
asyncapi-bot-eve Aug 5, 2025
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
13 changes: 9 additions & 4 deletions config/all-tags.json
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -178,12 +178,12 @@
"borderColor": "border-[#B9012b]"
},
{
"name": "Socket.IO",
"name": "TypeScript",
"color": "bg-[#61d0f2]",
"borderColor": "border-[#40ccf7]"
},
{
"name": "TypeScript",
"name": "Socket.IO",
"color": "bg-[#61d0f2]",
"borderColor": "border-[#40ccf7]"
},
Expand Down Expand Up @@ -222,6 +222,11 @@
"color": "bg-[#61d0f2]",
"borderColor": "border-[#40ccf7]"
},
{
"name": "Go",
"color": "bg-[#61d0f2]",
"borderColor": "border-[#40ccf7]"
},
Comment on lines +225 to +229
Copy link
Contributor

@coderabbitai coderabbitai bot Aug 4, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🛠️ Refactor suggestion

Inconsistent naming: “Go” vs “Go/Golang”.

Languages array already contains "Go/Golang" (Line 4). Introducing "Go" as a technology tag will break any consumer code that relies on strict-equality matching of tag names between languages and technologies. Align the name (and optionally unify the colours) to avoid hard-to-track filter mismatches.

🤖 Prompt for AI Agents 
In config/all-tags.json around lines 225 to 229, the tag name "Go" is
inconsistent with the existing "Go/Golang" in the languages array at line 4,
which can cause mismatches in filtering logic. Rename the "Go" tag to
"Go/Golang" to match the existing language tag exactly, and optionally unify the
color and borderColor values to maintain visual consistency.

{
"name": "Markdown",
"color": "bg-[#61d0f2]",
Expand Down Expand Up @@ -273,12 +278,12 @@
"borderColor": "border-[#40ccf7]"
},
{
"name": "SmartPaste",
"name": "JetBrains",
"color": "bg-[#61d0f2]",
"borderColor": "border-[#40ccf7]"
},
{
"name": "JetBrains",
"name": "SmartPaste",
"color": "bg-[#61d0f2]",
"borderColor": "border-[#40ccf7]"
},
Comment on lines +281 to 289
Copy link
Contributor

@coderabbitai coderabbitai bot Aug 4, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

⚠️ Potential issue

Duplicate “SmartPaste” and “JetBrains” tags still present.

Even after the rename, another "SmartPaste" entry exists at Lines 291-294 and another "JetBrains" entry is unchanged earlier in the file. Duplicate tags confuse the filter UI and break CSS classes that depend on deterministic IDs. Please deduplicate.

🤖 Prompt for AI Agents 
In config/all-tags.json around lines 281 to 294, there are duplicate entries for
"SmartPaste" and "JetBrains" tags. To fix this, remove the duplicate tag objects
so each tag name appears only once in the file. Ensure the remaining entries
have consistent color and borderColor values to avoid conflicts in the filter UI
and CSS class generation.

Expand Down
Loading
Loading