Merge development v4.2.0 #3390

Taaku18 · 2025-10-05T21:17:10Z

See CHANGELOG.md for details.

* Add (truncated) preview to snippets command * Add old view as option with "compact" * Fix black formatting

This pull request updated discord.py to 2.5.2. This also brings a new few features. - snooze - snoozed - unsnooze - clearsnoozed Aswell as a few new config options. - max_snooze_time - snooze_title - snooze_text - unsnooze_tex - unsnooze_notify_channel Signed-off-by: lorenzo132 <[email protected]>

Signed-off-by: lorenzo132 <[email protected]>

Fix jump_url not being displayed

Update pipfile for new dpy version

Update snooze arg

Signed-off-by: lorenzo132 <[email protected]>

* fix: ignore typing failures Make Modmail keep working when typing is disabled/outage * fix: only surpress failures * chore: sync local edits before push

* fix: ignore typing failures Make Modmail keep working when typing is disabled/outage * fix: only surpress failures * chore: sync local edits before push * Fix: closing with timed words/ command in reply. * Fix: typing in changelog command. * Fix: closing with timed words (additional)) * Fix changelog entry for command reply issue Corrected wording in the changelog entry regarding command inclusion in replies. Signed-off-by: lorenzo132 <[email protected]> * Update CHANGELOG for v4.2.0 enhancements Forwarded messages now display correctly in threads. Signed-off-by: lorenzo132 <[email protected]> --------- Signed-off-by: lorenzo132 <[email protected]>

Copilot

Pull Request Overview

This PR merges development version 4.2.0, upgrading discord.py to version 2.6.3 and adding several new features including thread snoozing, forwarded message handling, and improved message processing.

Major discord.py upgrade from 2.3.2 to 2.6.3 with CV2 support
New thread snoozing system with commands to snooze/unsnooze threads
Enhanced forwarded message handling and improved typing reliability

Reviewed Changes

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

Show a summary per file

File	Description
pyproject.toml	Updated version number to 4.2.0
core/utils.py	Added safe typing utilities, forwarded content extraction, and button fixes
core/time.py	Improved time parsing with better handling of vague time expressions
core/thread.py	Major additions for thread snoozing functionality and improved message handling
core/paginator.py	Fixed pagination view handling and footer icon issues
core/config_help.json	Added configuration help for new thread minimum character features
core/config.py	Added default configurations for snoozing and thread minimum characters
core/clients.py	Added null checks for avatar URLs
core/changelog.py	Added null checks for avatar URLs
core/_color_data.py	Removed extra blank line
cogs/utility.py	Added safe typing usage and avatar URL null checks
cogs/plugins.py	Added safe typing usage and avatar URL null checks
cogs/modmail.py	Major additions including snooze commands and improved snippet handling
bot.py	Updated version, enhanced forwarded message processing, and improved error handling
Pipfile	Updated discord.py version and removed uvloop dependency
CHANGELOG.md	Added comprehensive changelog for v4.2.0
.gitattributes	Added text auto-handling configuration
.env.example	Added DISABLE_AUTOUPDATES option

Comments suppressed due to low confidence (1)

core/utils.py:1

The regex character class [$\\-_@.&+] was changed from [$\\-_@.&+]. The hyphen placement change could affect URL matching behavior. Ensure this change is intentional and tested.

import base64

_{Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.}

core/utils.py

Copilot · 2025-10-06T11:31:20Z

core/thread.py

-                message1.embeds[0].author.name.startswith("Note")
-                or message1.embeds[0].author.name.startswith("Persistent Note")
-            ):
+            if message1.embeds[0].footer and "Internal Message" in message1.embeds[0].footer.text:


This condition checks for 'Internal Message' in footer text, but the code elsewhere uses 'Internal Note' and 'Persistent Internal Note'. This inconsistency could cause message identification failures.

Suggested change

if message1.embeds[0].footer and "Internal Message" in message1.embeds[0].footer.text:

if (

message1.embeds[0].footer

and (

"Internal Note" in message1.embeds[0].footer.text

or "Persistent Internal Note" in message1.embeds[0].footer.text

)

):

Copilot · 2025-10-06T11:31:20Z

core/thread.py


        tasks = [self.bot.api.edit_message(message1.id, message), message1.edit(embed=embed1)]
-        if message1.embeds[0].author.name.startswith("Persistent Note"):
+        if message1.embeds[0].footer and "Persistent Internal Message" in message1.embeds[0].footer.text:


Similar to above, this checks for 'Persistent Internal Message' but other code uses 'Persistent Internal Note'. This inconsistency in footer text checks could cause logic errors.

Copilot · 2025-10-06T11:31:21Z

core/thread.py

                tasks += [m2.delete()]

-        if message1.embeds[0].author.name.startswith("Persistent Note"):
+        if message1.embeds[0].footer and "Persistent Internal Message" in message1.embeds[0].footer.text:


Another instance of the footer text inconsistency. The system uses both 'Message' and 'Note' suffixes which could lead to failed message type detection.

cogs/modmail.py

Signed-off-by: Taku <[email protected]>

Copilot

Pull Request Overview

Copilot reviewed 18 out of 20 changed files in this pull request and generated no new comments.

_{Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.}

sebkuip and others added 30 commits April 16, 2025 02:56

Add (truncated) preview to snippets command (#3342)

3ab5161

* Add (truncated) preview to snippets command * Add old view as option with "compact" * Fix black formatting

Fix: Image url regex in thread send method (#3378)

a794ebd

feat minimum character requirement for thread creation. (#3380)

41b3db6

remove: unneeded import

3cfadc2

Signed-off-by: lorenzo132 <[email protected]>

Formatting black

0c73817

Signed-off-by: lorenzo132 <[email protected]>

fix?: internal messages on restoration

6af9bcf

Signed-off-by: lorenzo132 <[email protected]>

formatting

e19968c

Signed-off-by: lorenzo132 <[email protected]>

fix: internal messages.

edc9daa

Signed-off-by: lorenzo132 <[email protected]>

fix: internals

dc1c4af

Signed-off-by: lorenzo132 <[email protected]>

Update thread.py

6164559

Signed-off-by: lorenzo132 <[email protected]>

fix: use same logkey after restoration

8fb8274

Signed-off-by: lorenzo132 <[email protected]>

Add files via upload

934f958

Signed-off-by: lorenzo132 <[email protected]>

Update thread.py

f21b9c0

Signed-off-by: lorenzo132 <[email protected]>

Update thread.py

bd7c4c2

Signed-off-by: lorenzo132 <[email protected]>

Update thread.py

97d0e8d

Signed-off-by: lorenzo132 <[email protected]>

fix: show who send which internal message.

1a5a2bd

Black formatting.

f683023

Update Pipfile

5980f6a

Signed-off-by: lorenzo132 <[email protected]>

Update Pipfile.lock

19ff839

Signed-off-by: lorenzo132 <[email protected]>

fix: unsnooze bug

021a7bd

feat: CV2

dfdc0e7

update: black

d47a30e

fix: duplicates in logs, notes.

050c510

feat: dpy 2.6.3, forwarded messages, bug fixes.

9452017

Merge pull request #2 from LorenzoModmailDev/development

d4fe7ff

Fix jump_url not being displayed

Fix jump_url not being displayed

48e9df3

Update pipfile for new dpy version

69c43fc

fix: bug in note title/color

0223250

Update pipfile for new dpy version

cd7b4cc

Update pipfile for new dpy version

martinbndr and others added 8 commits September 22, 2025 22:31

Update snooze arg

7953813

Merge pull request #4 from LorenzoModmailDev/development

fbc6c01

Update snooze arg

Update Pipfile to include tomli package

6015098

Signed-off-by: lorenzo132 <[email protected]>

auto detect dpy version

2792032

Signed-off-by: lorenzo132 <[email protected]>

Remove crlf terminators

4c7b601

fix: ignore typing failures (#3389)

eebcd7a

* fix: ignore typing failures Make Modmail keep working when typing is disabled/outage * fix: only surpress failures * chore: sync local edits before push

Lock pipenv

cc4d17e

Taaku18 requested a review from Copilot October 6, 2025 11:29

Copilot AI reviewed Oct 6, 2025

View reviewed changes

Taaku18 added 3 commits October 6, 2025 05:05

Remove disutil, undo lowercasing escape seq

7c15387

Add back uvloop

32d9f15

Add config help for snooze configs

177bc30

Signed-off-by: Taku <[email protected]>

Taaku18 requested a review from Copilot October 7, 2025 05:15

Copilot AI reviewed Oct 7, 2025

View reviewed changes

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Uh oh!

Uh oh!

Merge development v4.2.0 #3390

Merge development v4.2.0 #3390

Uh oh!

Taaku18 commented Oct 5, 2025

Uh oh!

Copilot AI left a comment

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Copilot AI Oct 6, 2025

Uh oh!

Copilot AI Oct 6, 2025

Uh oh!

Copilot AI Oct 6, 2025

Uh oh!

Uh oh!

Copilot AI left a comment

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

5 participants

Uh oh!

Merge development v4.2.0 #3390

Are you sure you want to change the base?

Merge development v4.2.0 #3390

Uh oh!

Conversation

Taaku18 commented Oct 5, 2025

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull Request Overview

Reviewed Changes

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Copilot AI Oct 6, 2025

Choose a reason for hiding this comment

Uh oh!

Copilot AI Oct 6, 2025

Choose a reason for hiding this comment

Uh oh!

Copilot AI Oct 6, 2025

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull Request Overview

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

5 participants