add: record nickname changes on Discord member update

[Colk-tech]

No description provided.

[Copilot]

Pull request overview

This PR implements functionality to track and record nickname changes for Discord server members. When a Discord member's nickname is updated, the bot now captures and persists this change to the database, maintaining a historical record of nickname modifications.

• Adds a new RecordNicknameChangeUsecase that coordinates between messenger, user, and nickname changelog repositories
• Implements Discord bot event handler on_member_update() to detect and record nickname changes
• Extends repository layer with get_by_name() method for messenger lookup and adds dependency injection decorators

Reviewed changes

Copilot reviewed 22 out of 23 changed files in this pull request and generated 2 comments.

Show a summary per file

File	Description
src/system/usecase/nickname/record_nickname_change.py	Core usecase implementation that orchestrates nickname change recording
src/system/usecase/nickname/dto.py	Request/response DTOs for nickname change recording
src/system/usecase/nickname/interface.py	Abstract interface for the nickname change usecase
src/system/ui/discord/bot.py	Discord bot client with on_member_update event handler
src/system/infrastructure/repository/sqlalchemy/crud/messenger.py	Adds get_by_name method and @Inject decorator
src/system/infrastructure/repository/sqlalchemy/crud/user.py	Adds @Inject decorator for dependency injection
src/system/infrastructure/repository/sqlalchemy/crud/nickname.py	Adds @Inject decorator for dependency injection
src/system/domain/interface/repository/messenger.py	Adds abstract get_by_name method to interface
src/system/di/module/usecase/nickname/module.py	DI module binding for nickname usecase
src/system/di/module/ui/discord/bot/module.py	DI module binding for Discord bot
src/system/di/container.py	Registers new DI modules in the container
migration/versions/87e1bd665ac8_create_initial_tables.py	Creates messenger, user, and nickname_changelog tables
migration/versions/b59d27038977_seed_discord_messenger.py	Seeds the discord messenger record
migration/env.py	Enhances database URL conversion for asyncpg compatibility
pyproject.toml	Adds greenlet dependency
src/system/usecase/nickname/test_*.py	Comprehensive test coverage for usecase, DTOs, and interface
src/system/ui/discord/test_bot.py	Tests for Discord bot event handling and edge cases
src/system/infrastructure/repository/sqlalchemy/crud/test_messenger.py	Tests for get_by_name repository method
src/system/di/module/*/test_*.py	Tests for DI module bindings

---

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

[Copilot]

The foreign key constraint on the user_record_id column is missing explicit onupdate and ondelete actions. For consistency with the user table's foreign key constraint (which has onupdate='CASCADE', ondelete='CASCADE'), consider adding explicit cascade behavior here. This ensures nickname changelogs are handled appropriately when the parent user record is updated or deleted.

Suggested change

	sa.ForeignKeyConstraint(['user_record_id'], ['user.record_id'], ),
	sa.ForeignKeyConstraint(['user_record_id'], ['user.record_id'], onupdate='CASCADE', ondelete='CASCADE'),

[Copilot]

The equality check before.nick == after.nick will not properly detect changes when one value is None and the other is an empty string "". Since None is converted to "" in lines 49-50, this creates an edge case: if before.nick is None and after.nick is "", they are treated as the same when checked at line 43 but would be different after conversion. Consider normalizing both values before the comparison, such as: (before.nick or "") == (after.nick or "").

Suggested change

	if before.nick == after.nick:
	if (before.nick or "") == (after.nick or ""):

[codecov-commenter]

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 100.00%. Comparing base (1d01134) to head (fee168d).

Additional details and impacted files

@@            Coverage Diff             @@
##              main       #36    +/-   ##
==========================================
  Coverage   100.00%   100.00%            
==========================================
  Files           47        53     +6     
  Lines          552       674   +122     
==========================================
+ Hits           552       674   +122     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.