feat: タスク完了時の通知機能を追加 - #37
Conversation
- タスク完了時にbell.shスクリプトを実行する仕組みを追加 - CLAUDE.mdにタスク完了通知の詳細な説明を追記 - ターミナルベル音による音声通知機能を実装 🤖 Generated with [Claude Code](https://claude.ai/code) Co-Authored-By: Claude <noreply@anthropic.com>
WalkthroughA new shell script, Changes
Poem
✨ Finishing Touches
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. 🪧 TipsChatThere are 3 ways to chat with CodeRabbit:
SupportNeed help? Create a ticket on our support page for assistance with any issues or questions. Note: Be mindful of the bot's finite context window. It's strongly recommended to break down tasks such as reading entire modules into smaller chunks. For a focused discussion, use review comments to chat about specific files and their changes, instead of using the PR comments. CodeRabbit Commands (Invoked using PR comments)
Other keywords and placeholders
CodeRabbit Configuration File (
|
There was a problem hiding this comment.
Actionable comments posted: 0
🧹 Nitpick comments (2)
.devcontainer/bell.sh (1)
1-3: Add strict-mode flags & portable shebang for safer executionRunning without
set -euo pipefailmakes it easy for the script to silently continue on errors (e.g. ifprintffails in a non-TTY context).
Switching to#!/usr/bin/env bashalso improves portability when/bin/bashis not present.-#!/bin/bash +#!/usr/bin/env bash +set -euo pipefailNo functional change, but the script becomes more robust across environments.
Also double-check that the file is committed with the executable bit (chmod +x).CLAUDE.md (1)
32-34: Hyphenate “to-do list” for correct grammarMinor wording tweak.
- - Task status updates in the todo list + - Task status updates in the to-do list
📜 Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
Plan: Pro
📒 Files selected for processing (2)
.devcontainer/bell.sh(1 hunks)CLAUDE.md(1 hunks)
🧰 Additional context used
🪛 LanguageTool
CLAUDE.md
[grammar] ~33-~33: It appears that a hyphen is missing in the noun “to-do” (= task) or did you mean the verb “to do”?
Context: ...d through: - Task status updates in the todo list - Bell notification script executi...
(TO_DO_HYPHEN)
Summary
Test plan
🤖 Generated with Claude Code
Summary by CodeRabbit
New Features
Documentation