feat: channel-level quiet mode via bot mention (#173)

[BillChirico]

Summary

Implements channel-level quiet mode — moderators can silence the bot in a specific channel by mentioning it with @bot quiet.

Closes #173

Changes

Core Module (src/modules/quietMode.js)

• handleQuietCommand(message, config) — parses quiet/unquiet/status commands from bot mentions
• isQuietMode(guildId, channelId) — check if bot is currently silenced in a channel
• Natural language duration parsing: 30m, 2h, 1 hour, for 30 minutes
• Redis storage with TTL + in-memory Map fallback when Redis is unavailable
• Permission gating via config.quietMode.allowedRoles (any / moderator / admin / specific role IDs)

Commands

Mention	Action
@bot quiet	Silence for default duration (30 min)
@bot quiet for 1 hour	Silence for custom duration
@bot quiet 45m	Silence for 45 minutes
@bot unquiet / resume	Resume immediately
@bot status	Show remaining quiet time

Events (src/modules/events.js)

• On bot mention: check for quiet command first, then suppress AI if quiet mode active
• On regular message accumulation: skip triage buffering if quiet mode active
• Both paths have error isolation so quiet mode failures don't crash the event handler

Config (config.json)

{
  "quietMode": {
    "enabled": false,
    "allowedRoles": ["moderator"],
    "defaultDurationMinutes": 30,
    "maxDurationMinutes": 1440
  }
}

Dashboard (src/api/utils/configAllowlist.js)

Added quietMode to SAFE_CONFIG_KEYS so the web dashboard can edit it.

Files Changed

• src/modules/quietMode.js — new module (345 lines)
• src/modules/events.js — integrate quiet mode checks
• config.json — add quietMode config section (disabled by default)
• src/api/utils/configAllowlist.js — add quietMode to allowlist
• tests/modules/quietMode.test.js — 41 tests

Testing

• 41 tests, all passing
• Covers: duration parsing, permission checks, Redis path, memory fallback, all commands
• Verified pre-existing test failures (17 failures in ai-feedback, triage.coverage, redis) are on main before this branch

Notes

• Quiet mode is disabled by default — opt-in via dashboard or config
• Per-channel scope only — other channels unaffected
• Bot still responds to quiet mode commands even when in quiet mode (so users can unquiet)

[coderabbitai]

Warning

Rate limit exceeded

@BillChirico has exceeded the limit for the number of commits that can be reviewed per hour. Please wait 25 minutes and 38 seconds before requesting another review.

⌛ How to resolve this issue?

After the wait time has elapsed, a review can be triggered using the @coderabbitai review command as a PR comment. Alternatively, push new commits to this PR.

We recommend that you space out your commits to avoid hitting the rate limit.

🚦 How do rate limits work?

CodeRabbit enforces hourly rate limits for each developer per organization.

Our paid plans have higher rate limits than the trial, open-source and free plans. In all cases, we re-allow further reviews after a brief timeout.

Please see our FAQ for further information.

📥 Commits

Reviewing files that changed from the base of the PR and between 44a852e and 4c3f2c7.

📒 Files selected for processing (7)

• config.json
• src/api/utils/configAllowlist.js
• src/api/utils/configValidation.js
• src/modules/events.js
• src/modules/quietMode.js
• tests/modules/quietMode.test.js
• web/src/components/dashboard/ai-feedback-stats.tsx

✨ Finishing Touches

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Post copyable unit tests in a comment
• Commit unit tests in branch feat/issue-173

---

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

• X
• Mastodon
• Reddit
• LinkedIn

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

[greptile-apps]

Greptile Summary

This PR implements channel-level quiet mode, allowing moderators to temporarily silence the bot in specific channels via @bot quiet commands. The implementation is well-structured with proper error handling, comprehensive testing, and follows project conventions.

Key Changes:

• New src/modules/quietMode.js module with Redis storage (memory fallback), natural language duration parsing, and permission checks
• Integration in src/modules/events.js to suppress AI responses when quiet mode is active
• Config schema and allowlist updates for dashboard editing
• 41 comprehensive tests covering all scenarios

Code Quality:

• ✅ Uses Winston logger (no console.log violations)
• ✅ Proper error isolation with try-catch blocks
• ✅ Feature gating (config.quietMode?.enabled)
• ✅ TTL validation prevents invalid Redis operations
• ✅ Disabled by default (opt-in)

Note: The missing defaultDurationMinutes property in configValidation.js schema was already acknowledged in previous review threads.

Confidence Score: 5/5

• This PR is safe to merge with minimal risk
• Clean implementation following all project conventions (Winston logging, ESM, proper error handling), comprehensive test coverage (41 tests), safe defaults (disabled by default), and proper feature gating. The one noted issue (missing validation schema property) was acknowledged in previous review threads and doesn't affect functionality.
• No files require special attention

Important Files Changed

Filename	Overview
src/modules/quietMode.js	New quiet mode module with Redis/memory fallback, natural language duration parsing, permission checks, and comprehensive error handling
src/modules/events.js	Integrated quiet mode checks in message handler with proper error isolation and feature gating
src/api/utils/configValidation.js	Added quietMode validation schema; missing defaultDurationMinutes property (acknowledged in previous review threads)
tests/modules/quietMode.test.js	Comprehensive test suite with 41 tests covering duration parsing, permissions, storage (Redis + memory fallback), and all command scenarios

Sequence Diagram

sequenceDiagram
    participant User
    participant Discord
    participant EventHandler as events.js
    participant QuietMode as quietMode.js
    participant Storage as Redis/Memory
    participant AI as Triage/AI

    User->>Discord: @bot quiet 30m
    Discord->>EventHandler: MessageCreate event
    EventHandler->>QuietMode: handleQuietCommand(message, config)
    QuietMode->>QuietMode: Check permissions
    QuietMode->>QuietMode: Parse duration (30m → 1800s)
    QuietMode->>Storage: setQuiet(guildId, channelId, until, userId)
    Storage-->>QuietMode: OK
    QuietMode-->>EventHandler: return true (was quiet command)
    EventHandler->>User: "Going quiet for 30 minutes"
    
    Note over EventHandler,AI: Bot is now in quiet mode for this channel
    
    User->>Discord: Regular message
    Discord->>EventHandler: MessageCreate event
    EventHandler->>QuietMode: isQuietMode(guildId, channelId)
    QuietMode->>Storage: getQuiet(guildId, channelId)
    Storage-->>QuietMode: {until: timestamp, by: userId}
    QuietMode-->>EventHandler: true (quiet mode active)
    EventHandler->>EventHandler: return early (suppress AI)
    
    Note over EventHandler,AI: AI response suppressed
    
    User->>Discord: @bot unquiet
    Discord->>EventHandler: MessageCreate event
    EventHandler->>QuietMode: handleQuietCommand(message, config)
    QuietMode->>Storage: clearQuiet(guildId, channelId)
    Storage-->>QuietMode: OK
    QuietMode-->>EventHandler: return true
    EventHandler->>User: "Quiet mode lifted — I'm back!"

Loading

_{Last reviewed commit: 4c3f2c7}

[Copilot]

Pull request overview

Adds a per-channel “quiet mode” feature that lets moderators (or other configured roles) temporarily suppress the bot’s AI behavior in a channel via bot-mention commands, with Redis-backed TTL storage and an in-memory fallback.

Changes:

• Introduces src/modules/quietMode.js implementing quiet/unquiet/status commands, duration parsing, and Redis+memory storage.
• Integrates quiet mode checks into the message event flow to suppress AI responses / triage accumulation while active.
• Exposes quietMode to the dashboard config editor allowlist and adds a default quietMode section to config.json, plus comprehensive module tests.

Reviewed changes

Copilot reviewed 5 out of 5 changed files in this pull request and generated 6 comments.

Show a summary per file

File	Description
src/modules/quietMode.js	New module for parsing, permission checks, and Redis/memory-backed quiet state management.
src/modules/events.js	Hooks quiet mode command handling + quiet checks into mention/reply and accumulation paths.
src/api/utils/configAllowlist.js	Adds quietMode to SAFE_CONFIG_KEYS so it can be edited via API/dashboard.
config.json	Adds default quietMode config (disabled), but also includes a full-file reformat.
tests/modules/quietMode.test.js	Adds Vitest coverage for parsing, permissions, storage, and command handling.

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

[Copilot]

Pull request overview

Copilot reviewed 7 out of 7 changed files in this pull request and generated 3 comments.

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

[greptile-apps]

_{7 files reviewed, 1 comment}

_{Edit Code Review Agent Settings | Greptile}