Skip to content

aaa#9

Merged
breadddevv merged 1 commit into
mainfrom
revamp/codebase
Jun 7, 2026
Merged

aaa#9
breadddevv merged 1 commit into
mainfrom
revamp/codebase

Conversation

@breadddevv

@breadddevv breadddevv commented Jun 7, 2026

Copy link
Copy Markdown
Member

🚀 Summary

Major codebase revamp introducing a guild settings management system with improved build configuration and simplified file loading logic.

✨ Features

  • New config command: Owner-only slash command that opens a modal for configuring boost and logging channels per guild
  • Modal-based channel selection: Guild admins can now select destination channels for boosts and logging through interactive Discord modals

♻️ Improvements

  • Persistent guild settings: Replaced create-only pattern with upsert logic to update existing settings instead of always creating new records
  • Enhanced modal handling: setupResponse event now handles multiple modal types (setupboostifymodal and configboostifymodal) with improved response formatting using ContainerBuilder
  • Simplified file filtering: Removed redundant exclusion checks in command/event file loading logic
  • Build configuration: Added .map file exclusion in tsconfig.json and updated file selection filters to exclude source maps across the codebase

🗑️ Removals

  • Removed example command: Deleted entire example_all.test.ts test/demo slash command file

@coderabbitai

coderabbitai Bot commented Jun 7, 2026

Copy link
Copy Markdown

Review Change Stack

Caution

Review failed

Pull request was closed or merged during review

📝 Walkthrough

Walkthrough

This PR adds a new owner-only /config slash command and corresponding modal response handler to manage guild channel settings for boost announcements and logging. It refactors file-loading filters and build configuration to exclude source maps, and updates modal handling to support both initial setup and configuration changes via database upsert.

Changes

Settings Configuration Feature

Layer / File(s) Summary
Build and file loading configuration
tsconfig.json, src/index.ts, src/libs/loadCommands.ts
TypeScript compiler and command/event loaders now exclude .map source map files consistently via a unified filter predicate.
Configuration command modal
src/commands/settings.ts
New owner-only config slash command displays a "Configuration" modal with required channel-select components for boost channel and logging channel selection.
Modal response handler and database persistence
src/events/setupResponse.ts
Handler accepts both setup and config modal submissions, queries existing guild settings, persists channel selections via prisma.guildSetting.upsert(), and replies with a component-formatted ephemeral message indicating whether configuration was created or changed.

Estimated Code Review Effort

🎯 2 (Simple) | ⏱️ ~12 minutes

Possibly Related PRs

  • teamboostify/boostify#8: Updates to command auto-loading in src/libs/loadCommands.ts and file structure changes are directly related to this PR's refinements in the same modules.

Poem

🐰 A config modal hops into view,
Channel selectors, shiny and new,
Upsert replaces create's old ways,
Settings persist through all of our days,
Maps are excluded, loaders stay clean! 🎉

🚥 Pre-merge checks | ✅ 3 | ❌ 2

❌ Failed checks (1 warning, 1 inconclusive)

Check name Status Explanation Resolution
Docstring Coverage ⚠️ Warning Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%. Write docstrings for the functions missing them to satisfy the coverage threshold.
Title check ❓ Inconclusive The title 'aaa' is vague, non-descriptive, and provides no meaningful information about the changeset's purpose or primary modifications. Use a clear, specific title that describes the main change, such as 'Add guild configuration modal and refactor command/event file loading' or 'Implement settings command with modal-based channel configuration'.
✅ Passed checks (3 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
📝 Generate docstrings
  • Create stacked PR
  • Commit on current branch
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch revamp/codebase

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

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

@breadddevv breadddevv merged commit b84bed6 into main Jun 7, 2026
3 of 4 checks passed
@breadddevv breadddevv deleted the revamp/codebase branch June 7, 2026 21:41
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