TeamInklings/OctobotStealth
TeamInklings/OctobotStealth
Archived
1
0
Fork 0
forked from TeamInklings/Octobot
Code Pull requests Wiki Activity Actions
255 commits 1 branch 0 tags 2.8 MiB
Commit graph

120 commits

Author SHA1 Message Date
Macintxsh
83e2c5040e
Remove mctaylors.ru mentions (#251) 
Signed-off-by: mctaylors <cantsendmails@mctaylors.ru>
 2024-01-22 20:47:53 +05:00
Macintxsh
e01fde83c6
Use custom TimeSpanParser (#223) 
Closes #154

---------

Signed-off-by: mctaylors <cantsendmails@mctaylors.ru>
 2023-12-31 12:27:00 +00:00
Octol1ttle
894e819865
Fix newline in LogResult (#245) 
Fixes an issue on Windows that would cause the `ResultErrorMessage` to
be unindented.

Signed-off-by: Octol1ttle <l1ttleofficial@outlook.com>
 2023-12-31 13:42:59 +03:00
Macintxsh
285763d50d
/userinfo: Show if the user was kicked (#242) 
Closes #241 

Updates:
- Show if the user was kicked by [adding "Kicked" parameter to
MemberData](https://github.com/LabsDevelopment/Octobot/issues/241)
- Change `mctaylors-ru`'s `UserInfoBannedPermanently` string to be
different from `UserInfoBanned`
- Finally add `AppendPunishmentsInformation` method to avoid Cognitive
Complexity
- Use MemberData to check if the user was banned
- Rename variable `isMuted` to `wasMuted` to be consistent with other
variable names

---------

Signed-off-by: mctaylors <cantsendmails@mctaylors.ru>
 2023-12-21 20:35:10 +05:00
Octol1ttle
7d9a85d815
Add profiler base (#235) 
This PR adds the base classes required for profiling code inside of
Octobot. The implementation of the profiler is similar to Minecraft,
however it is more detailed and provides per-event logs for each event.
This PR does not change any code to be profiled and this is intentional.
Changes required for profiling will come as separate PRs, one for
commands, one for responders, and one for background services.

Signed-off-by: Octol1ttle <l1ttleofficial@outlook.com>
 2023-12-20 22:59:17 +05:00
Octol1ttle
d4871bb23d
Use AddFromAssembly for responders and command groups (#240) 
now we don't have to explicitly type out command groups woooo

Signed-off-by: Octol1ttle <l1ttleofficial@outlook.com>
 2023-12-20 20:48:32 +03:00
Octol1ttle
21f200c988
Merge BackgroundGuildDataSaverService into GuildDataService (#239) 
Title. idk why I didn't think of this before.
Also, GuildDataService is now properly registered as an IHostedService,
so it receives start & shutdown events. So this PR gets rid of the
workaround that was needed for save-on-shutdown to function

Signed-off-by: Octol1ttle <l1ttleofficial@outlook.com>
 2023-12-20 20:08:56 +03:00
Octol1ttle
bd4c5b26da
Remove "extends IHostedService" from classes where it's not required (#236) 
Originally, these classes were services because I thought that all
DI-resolvable classes need to be services. However, this is not true, so
we can make these classes (notably Utility and GuildDataService) not
extend anything. `UtilityService` was renamed to `Utility` for
simplicity

---------

Signed-off-by: Octol1ttle <l1ttleofficial@outlook.com>
 2023-12-20 19:33:52 +03:00
Macintxsh
6688481093
/about: Show Discord bot username instead of hardcoded one (#230) 
In this PR, I made it so that in the Author field instead of the
hardcoded name was the name of the Discord bot. This was done to match
the icon next to it in the same field.

Replaces #224

---------

Signed-off-by: mctaylors <cantsendmails@mctaylors.ru>
 2023-12-20 16:25:13 +00:00
Octol1ttle
74e32dee9b
Use collection expressions in more places (#238) 
ReSharper inspections have been updated, causing new warnings to appear
in the codebase. This time, the "Use collection expressions" inspection
has been enabled for usecases where the collection is not empty. This PR
fixes the check failures caused by this inspection.
 2023-12-20 19:23:37 +03:00
Macintxsh
f79968fdc2
/about: Use Markdown.Hyperlink instead of custom extension (#229) 
Signed-off-by: mctaylors <cantsendmails@mctaylors.ru>
 2023-12-19 15:32:11 +05:00
Macintxsh
c0b43c6a18
/about: Show link to GitHub profile if Discord member wasn't found (#226) 
In this PR, the behavior of the developer list display in /about has
been changed. Now, if a developer is not on the same server where the
/about command was executed, their username will have a link to their
GitHub profile.

---------

Signed-off-by: mctaylors <cantsendmails@mctaylors.ru>
 2023-12-18 14:26:08 +05:00
Macintxsh
4dc460a269
Add issue button w/ some button updates (#221) 
In this PR, I've added a "Report an issue" button and a few more
button-related changes:
- Add "Report an issue" button
- Add icon for "Octobot's source code"
- Rename `AboutTitleRepository` to `ButtonOpenRepository`
- Rename `OpenEventInfoButton` to `ButtonOpenEventInfo` to be consistent
with other language string names
- Rename `ColorsList.cs` to `Miscellaneous.cs`
- Add public const strings in `Octobot.cs` to get repository & issues links

---------

Signed-off-by: mctaylors <cantsendmails@mctaylors.ru>
 2023-12-17 21:35:09 +03:00
neroduckale
4581b402aa
Do not log messages edited by bots (#202) 
If the author of the edited message is a bot, then Octobot won't log the edit
 2023-12-17 22:02:50 +05:00
Macintxsh
2dc5220f46
/about: Add repository link button (#218) 
In this PR, I moved the repository link from the embed to a button in
/about command for better UI/UX

---------

Signed-off-by: Macintxsh <95250141+mctaylors@users.noreply.github.com>
 2023-12-17 16:49:44 +00:00
Macintxsh
541e18fff0
Add ChannelApiExtensions (#217) 
This PR adds an extension method to make it easier to pass Result<Embed> to CreateMessageAsync

---------

Co-authored-by: Octol1ttle <l1ttleofficial@outlook.com>
 2023-12-17 21:47:52 +05:00
Macintxsh
9a23e1d533
FeedbackServiceExtensions: Add FeedbackMessageOptions support (#219) 
Required for #218
 2023-12-17 20:44:18 +05:00
neroduckale
59c2c5eada
Add link to original message for activated reminders and /listremind (#203) 2023-12-11 23:38:26 +03:00
Macintxsh II
eeaff23fb5
Rewrite music list (#210) 
Changes:
- Use timestamps from
[Splatunes](https://splatoonwiki.org/wiki/Splatune_(Splatoon_Original_Soundtrack))
- Increase track count to 20
- Replace Calamari Inkantation 3MIX with OG Calamari Inkantation due to long
duration

Signed-off-by: mctaylors <cantsendmails@mctaylors.ru>
 2023-12-08 16:15:44 +05:00
Macintxsh II
42ab11d253
Rename "Event details" button to "Open Event Info" (#209) 
The correct phrasing used by Discord is "Event Info"

![image](https://github.com/LabsDevelopment/Octobot/assets/95250141/8165b70d-4c81-4b85-8251-db2de6a7f4ca)

Signed-off-by: mctaylors <cantsendmails@mctaylors.ru>
 2023-12-08 16:06:50 +05:00
neroduckale
bcd1db8c8e
Fix inspection warnings (#208) 
Signed-off-by: neroduckale <100025711+neroduckale@users.noreply.github.com>
 2023-12-06 00:24:55 +05:00
Macintxsh II
4c50bdaff7
Bump .NET version to 8.0 (#207) 
Signed-off-by: mctaylors <cantsendmails@mctaylors.ru>
 2023-12-04 20:18:56 +05:00
Macintxsh II
18cdc63883
Add StringBuilder & Markdown extensions (#206) 
In this PR, I have added StringBuilder extensions to avoid `.Append`
reuse such as `.Append("- ").AppendLine()`

Closes #205

---------

Signed-off-by: mctaylors <cantsendmails@mctaylors.ru>
 2023-12-04 17:09:47 +03:00
dependabot[bot]
7e47c56015
Bump Microsoft.Extensions.Hosting from 7.0.1 to 8.0.0 (#187) 
Bumps [Microsoft.Extensions.Hosting](https://github.com/dotnet/runtime)
from 7.0.1 to 8.0.0.

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Co-authored-by: Octol1ttle <l1ttleofficial@outlook.com>
 2023-11-22 14:27:55 +05:00
Macintosh II
b166300642
Update CDN links for Octobot banner (#192) 
Old CDN links are no longer working and they must be changed to new
links to display Octobot banner properly.
 2023-11-22 08:34:40 +00:00
Octol1ttle
f908919ac9
Put edited message in cache if it's not in cache already (#196) 
Closes #183 

In addition to the fix, now no error will be returned if the message
doesn't exist in the cache as that is a (relatively) normal occurrence
and isn't an indicator of an issue with the bot or its configuration
 2023-11-22 08:29:27 +00:00
Octol1ttle
e65c7a469d
Use TryGetValue instead of ContainsKey + index access to avoid double lookup (#193) 
This PR fixes an issue that is currently causing CI to fail in all pull
requests:
`Notice: "[CA1854] Prefer a 'TryGetValue' call over a Dictionary indexer
access guarded by a 'ContainsKey' check to avoid double lookup" on
/home/runner/work/Octobot/Octobot/src/Services/Update/ScheduledEventUpdateService.cs(107,4168)`
The issue is resolved by following the advice mentioned in the notice.
 2023-11-22 11:19:45 +03:00
Octol1ttle
f785efcbc0
Add more information to guild loaded log message (#179) 
This PR adds additional information in the log about guilds as they are
being loaded. The new information is the owner tag & ID and member count
 2023-11-06 23:39:26 +05:00
Octol1ttle
f12d6d13c5
Check interactions in MemberUpdateService before operating on members (#177) 
This PR fixes an issue that caused REST errors to occur in
MemberUpdateService if the bot tries to interact with a member it can't
interact with. The issue is fixed by returning from TickMemberDataAsync
early if the member cannot be interacted with. An error message was
planned, but it requires adding a lot of services and severely
increasing the complexity. Contributors may feel free to add one if they
deem so necessary.
 2023-11-04 23:33:37 +05:00
Octol1ttle
5f0d806213
Use IFeedbackService interface instead of implementation (#178) 
This PR replaces usages of the `FeedbackService` implementation with the
`IFeedbackService` interface. Using concrete implementations breaks the
whole point of dependency injection, so it doesn't make sense to use
them
 2023-11-04 23:28:22 +05:00
Octol1ttle
a4b009a26f
Disable text commands (#175) 
This PR disables text commands as their presence was not intentional.
 2023-10-30 09:37:16 +05:00
Octol1ttle
0ba357e4c7
Limit string argument length to avoid "embed description too long" errors (#174) 
This PR fixes an error that would appear if a string that's way too long
was passed as a command argument by limiting the string's length

![image](https://github.com/LabsDevelopment/Octobot/assets/61277953/8f8267fd-d382-4a24-b92d-5f9966d7563b)
 2023-10-28 23:10:16 +05:00
Octol1ttle
cf7007f269
Handle guild data load errors better (#172) 
Previously, any errors in guild data load will cause the bot to be
unusable in that guild. It didn't help that the end users had no
information that something was wrong! Now, any errors will be logged
better (with the full path to the file that couldn't be loaded), and the
users will receive a message saying that functionality is degraded

The old way to save objects was to serialize them directly into streams
opened by `File#Create`. This can cause problems if the serialization
isn't completed, because `File#Create` overwrites the file with an empty
one on the spot. Now, objects are first deserialized into a temporary
file, then the original is replaced by the temporary, then the temporary
is deleted.

Errors during guild data load would sometimes cause the bot to replace
the corrupted file with a default one whenever a save is triggered. Now,
guilds with load errors won't have their data saved to aid in debugging

---------

Signed-off-by: Octol1ttle <l1ttleofficial@outlook.com>
Signed-off-by: mctaylors <mctaylxrs@outlook.com>
 2023-10-26 18:14:27 +03:00
Octol1ttle
fb3aebb1e0
Notify user when a command execution error occurs (#171) 
With this PR, whenever command execution fails, the user will get an
error message with details of the error that can be passed on to
developers

An unrelated minor change: errors caused by task cancellations will no
longer be logged

---------

Signed-off-by: Octol1ttle <l1ttleofficial@outlook.com>
Signed-off-by: mctaylors <mctaylxrs@outlook.com>
 2023-10-26 14:54:15 +00:00
Macintosh II
5b84c8d8d0
Add more info output to /delremind (#173) 
_There are times when you want to be sure of what you've destroyed._

Therefore, in this PR I added the output of the text of the deleted
reminder along with its position in the list, because you can make a
mistake with deleting a reminder and forget about what you needed to be
reminded about.

---------

Signed-off-by: mctaylors <mctaylxrs@outlook.com>
 2023-10-26 17:41:47 +03:00
Octol1ttle
fead92129d
Unschedule scheduled event status update only if last update was successful (#168) 
This PR fixes an issue that prevented scheduled event status updates
from running again if they failed. This happened because, before each
update, the events would be synchronized. The `ScheduleOnStatusUpdated`
field would be set even if it's already `true`, which will cause it to
be set to `false` for unsuccessful updates. The issue is fixed by
surrounding the field set call with a condition that will prevent
setting the field from `true` to `false`
 2023-10-17 12:25:15 +00:00
Octol1ttle
580cd24810
Do not try to send messages in empty EventNotificationChannels (#167) 
This PR fixes an error that would occur if an event was created, was
about to start or started and the EventNotificationChannel was empty.
 2023-10-17 12:23:14 +00:00
Octol1ttle
b30d690113
Add filtering by message author to /clear (#169) 
Closes #164 

This PR adds an optional argument to `/clear` - `author` of type User.

If the user is specified, only messages sent by that user will be
cleared. Simple as that.
 2023-10-17 12:20:58 +00:00
Octol1ttle
687883bbf8
Use MemberData to determine a subscriber's role list (#165) 
Closes #163 

Discord's API sucks a lot. You ask it for a member, but it won't give
you a member. This is why this PR updates the
`GetEventNotificationMentions` method used to determine what roles and
users should get pinged for a scheduled event. Previously, the bot asked
Discord to provide the member for each subscriber to determine whether
or not they have the event notification role (to avoid pinging people
personally when the role would already ping them). With this pull
request, the bot uses MemberData, it's own member storage, for that
purpose (if you're wondering why, refer to the first two sentences)
 2023-10-17 12:07:01 +00:00
Octol1ttle
a326adb680
Fix Reminder serialization/deserialization (#166) 
Closes #124 

Before we dive into this PR, let's explain a few rules of serialization
and deserialization in System.Text.Json (often referred as STJ).

The important rule of serialization is that fields are ***not***
serialized unless the serializer gets passed an instance of
`JsonSerializerOptions` that explicitly tells it to include fields.
However, properties are serialized by default.

The important rule of ***de***serialization is that class members are
only deserialized if:
1) In case of properties, they must have a setter
2) If they do not have a setter, the constructor must have an argument,
required or optional, that will act as the setter for that property.

Unfortunately, both of these rules were ignored in some commit that
refactored reminders. This PR is here to fix that issue by:
1) Converting fields in `Reminder.cs` to properties
2) Adding an optional argument to the `MemberData` constructor
 2023-10-17 17:01:24 +05:00
Octol1ttle
67d44ff835
Sanitize input text in message edit log (#162) 
This PR fixes an issue that caused syntax highlighting to be applied to
incorrect lines in the message edit log. The fix is to prepend a
zero-width space before every line of input text. This prevents Discord
from highlighting the line when it wasn't edited
 2023-10-12 21:51:51 +05:00
Octol1ttle
e6f53b13f0
Split extension methods into separate classes (#161) 
This PR splits the extension methods contained in `Extensions.cs` into
separate classes in the `Octobot.Extensions` namespace. This was done
for multiple reasons:
1) The `Extensions.cs` violates SRP (Single Responsibility Principle) -
it takes upon itself every extension method for many types
2) Having a separate class for each extended type is a standard practice
- take a look at
[Remora.Discord](https://github.com/Remora/Remora.Discord/tree/main/Backend/Remora.Discord.Rest/Extensions)
or [osu!](https://github.com/ppy/osu/tree/master/osu.Game/Extensions)
3) Having all extension methods in one file makes it hard to find the
method you want
 2023-10-12 15:37:25 +00:00
Macintosh II
4d526ad32f
Sanitize scheduled event names (#159) 
In this PR, I added `Markdown.Sanitize` for every `scheduledEvent.Name`
and `eventData.Name`. That means, every title in scheduled event embeds
will be sanitized.

I did that because in scheduled event list event title displays only as
sanitized string.

Signed-off-by: Macintosh II <mctaylxrs@outlook.com>
 2023-10-12 10:56:24 +00:00
Macintosh II
e21cefd79d
Use link button for created event message (#158) 
Signed-off-by: Macintosh II <mctaylxrs@outlook.com>
 2023-10-11 22:16:52 +03:00
Macintosh II
8b659a6582
Rename Reminder message to Reminder text (#157) 
Signed-off-by: Macintosh II <mctaylxrs@outlook.com>
 2023-10-06 14:20:18 +00:00
Macintosh II
d1e3558bc6
/remind: Switch away from zero-based numbering and rename index (#155) 
Signed-off-by: Macintosh II <mctaylxrs@outlook.com>
 2023-10-06 14:17:48 +00:00
Macintosh II
a70c228bc4
Change reminders sorting in /listremind (#156) 
Signed-off-by: Macintosh II <mctaylxrs@outlook.com>
 2023-10-06 16:12:36 +03:00
Macintosh II
4748c5de2c
Rename /showinfo to avoid confusion with /guildinfo (#153) 
Depends on #152

Signed-off-by: Macintosh II <mctaylxrs@outlook.com>
Signed-off-by: Macintosh II <95250141+mctaylors@users.noreply.github.com>
 2023-10-06 15:35:08 +03:00
Macintosh II
72f728323e
Add /guildinfo (#152) 
Signed-off-by: Macintosh II <mctaylxrs@outlook.com>
 2023-10-06 15:23:45 +03:00
Octol1ttle
6f1e543edb
Correct illegal nickname characters regex (#151) 
Apparently there are non-letter characters in between A-Z and a-z
 2023-10-04 21:55:39 +05:00