Skip to content

Add claude-sounds β€” audio feedback plugin#28

Open
culminationAI wants to merge 2 commits intojmanhype:mainfrom
culminationAI:add-claude-sounds
Open

Add claude-sounds β€” audio feedback plugin#28
culminationAI wants to merge 2 commits intojmanhype:mainfrom
culminationAI:add-claude-sounds

Conversation

@culminationAI
Copy link
Copy Markdown

Adds claude-sounds β€” Your Claude Code shouldn't be silent. Sound effects for all hooks β€” spells, chimes, clicks. Drop mp3s to customize.

10 hook events, 21 sounds, random rotation, fully customizable. macOS (afplay).

Install:

claude plugins marketplace add https://github.com/culminationAI/claude-marketplace
claude plugins install claude-sounds@culmination-marketplace

@qodo-code-review
Copy link
Copy Markdown

Review Summary by Qodo

Add claude-sounds audio feedback plugin to README

πŸ“ Documentation

Grey Divider

Walkthroughs

Description
β€’ Add claude-sounds plugin to README marketplace list
β€’ Plugin provides audio feedback for Claude Code hooks
β€’ Includes 10 hook events, 21 sounds, customizable
Diagram
flowchart LR
  README["README.md"] -- "add plugin entry" --> MARKETPLACE["Marketplace Plugins List"]
  MARKETPLACE -- "references" --> SOUNDS["claude-sounds Plugin"]
  SOUNDS -- "provides" --> AUDIO["Audio Feedback System"]
Loading

Grey Divider

File Changes

1. README.md πŸ“ Documentation +1/-0

Add claude-sounds plugin to marketplace documentation

β€’ Added claude-sounds plugin entry to marketplace plugins section
β€’ Entry includes GitHub repository link and brief description
β€’ Description highlights audio feedback features and customization

README.md


Grey Divider

Qodo Logo

@qodo-code-review
Copy link
Copy Markdown

qodo-code-review bot commented Mar 24, 2026

Code Review by Qodo

🐞 Bugs (0) πŸ“˜ Rule violations (0) πŸ“Ž Requirement gaps (0) πŸ“ Spec deviations (0)

Grey Divider


Action required

1. Missing blank line before heading β˜‘ 🐞 Bug βš™ Maintainability
Description
The new plugin list item is immediately followed by the next '## MCP Servers' heading without a
blank line, which can break Markdown style rules and fail CI linting (awesome-lint).
Code

README.md[R22-23]

+- [claude-sounds](https://github.com/culminationAI/claude-sounds) - Your Claude Code shouldn't be silent. Sound effects for all hooks β€” spells, chimes, clicks. Drop mp3s to customize.
## MCP Servers
Evidence
README.md now has a list item directly followed by a heading, and this repo runs npx awesome-lint
in CI, which commonly enforces Markdown list/heading spacing. The file also previously had a blank
line separating the list from subsequent content, indicating an established formatting pattern.

README.md[20-24]
.github/workflows/ci.yml[1-12]

Agent prompt
The issue below was found during a code review. Follow the provided context and guidance below and implement a solution

## Issue description
The README plugin list item for `claude-sounds` is directly followed by the `## MCP Servers` heading without a blank line, which can violate Markdown style/lint rules and break CI.
### Issue Context
CI runs `npx awesome-lint`, and the surrounding README formatting indicates headings should be separated from lists with a blank line.
### Fix Focus Areas
- README.md[20-24]
- .github/workflows/ci.yml[1-12]
### Suggested change
Add a blank line between the `- [claude-sounds](...) ...` bullet and the `## MCP Servers` heading.

β“˜ Copy this prompt and use it to remediate the issue with your preferred AI generation tools


Grey Divider

β“˜ The new review experience is currently in Beta. Learn more

Grey Divider

Qodo Logo

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.

1 participant