docs: Updated CHANGELOG.

dev: upped version to 5.1.3

CHANGELOG.md

@@ -2,6 +2,28 @@
 
 Mostly based on [keepachangelog](https://keepachangelog.com/en/1.0.0/) except date format. a-c-f-r-o
 
+## [5.1.3] - 06.07.2024
+
+### Added
+
+- Added `.quran` command, which will show the provided ayah in english and arabic, including recitation by Alafasy
+
+### Changed
+
+- Replying to the bot's message in the channel where chatterbot is enabled will also trigger the ai response, as if you pinged the bot. This only works for chatterbot, but not for nadeko ai command prompts
+
+### Fixed
+
+- Fixed `.stickeradd` it now properly supports 300x300 image uploads.
+- Bot should now trim the invalid characters from chatterbot usernames to avoid openai errors
+- Fixed prompt triggering chatterbot responses twice
+
+## [5.1.2] - 29.06.2024
+
+### Fixed
+
+- Fixed `.honeypot` not unbanning and not pruning messages
+
 ## [5.1.1] - 27.06.2024
 
 ### Added

src/NadekoBot/Modules/Administration/Honeypot/HoneyPotService.cs

@@ -71,7 +71,8 @@ public sealed class HoneyPotService : IHoneyPotService, IReadyExecutor, IExecNoC
                 try
                 {
                     Log.Information("Honeypot caught user {User} [{UserId}]", user, user.Id);
-                    await user.BanAsync();
+                    await user.BanAsync(pruneDays: 1);
+                    await user.Guild.RemoveBanAsync(user.Id);
                 }
                 catch (Exception e)
                 {

src/NadekoBot/Modules/Administration/Ticket/TicketCommands.cs

@@ -0,0 +1,14 @@
+// namespace NadekoBot.Modules.Administration;
+//
+// public partial class Administration
+// {
+//     [Group]
+//     public partial class TicketCommands : NadekoModule
+//     {
+//         [Cmd]
+//         public async Task Ticket()
+//         {
+//             
+//         }
+//     }
+// }

src/NadekoBot/Modules/Games/ChatterBot/ChatterBotService.cs

@@ -96,6 +96,8 @@ public class ChatterBotService : IExecOnMessage
             message = msg.Content[normalMention.Length..].Trim();
         else if (msg.Content.StartsWith(nickMention, StringComparison.InvariantCulture))
             message = msg.Content[nickMention.Length..].Trim();
+        else if (msg.ReferencedMessage?.Author.Id == nadekoId)
+            message = msg.Content;
         else
             return null;
 

src/NadekoBot/Modules/Games/ChatterBot/_common/OfficialGptSession.cs

@@ -3,10 +3,12 @@ using Newtonsoft.Json;
 using OneOf.Types;
 using System.Net.Http.Json;
 using SharpToken;
+using System.CodeDom;
+using System.Text.RegularExpressions;
 
 namespace NadekoBot.Modules.Games.Common.ChatterBot;
 
-public class OfficialGptSession : IChatterBotSession
+public partial class OfficialGptSession : IChatterBotSession
 {
     private string Uri
         => $"https://api.openai.com/v1/chat/completions";
@@ -45,7 +47,7 @@ public class OfficialGptSession : IChatterBotSession
         _maxHistory = chatHistory;
         _maxTokens = maxTokens;
         _minTokens = minTokens;
-        _nadekoUsername = nadekoUsername;
+        _nadekoUsername = UsernameCleaner().Replace(nadekoUsername, "");
         _encoding = GptEncoding.GetEncodingForModel(_model);
         messages.Add(new()
         {
@@ -55,14 +57,21 @@ public class OfficialGptSession : IChatterBotSession
         });
     }
 
+
+    [GeneratedRegex("[^a-zA-Z0-9_-]")]
+    private static partial Regex UsernameCleaner();
+    
     public async Task<OneOf.OneOf<ThinkResult, Error<string>>> Think(string input, string username)
     {
+        username = UsernameCleaner().Replace(username, "");
+        
         messages.Add(new()
         {
             Role = "user",
             Content = input,
             Name = username
         });
+        
         while (messages.Count > _maxHistory + 2)
         {
             messages.RemoveAt(1);

src/NadekoBot/Modules/Searches/ReligiousCommands.cs

@@ -0,0 +1,102 @@
+using NadekoBot.Modules.Searches.Common;
+using System.Net.Http.Json;
+using System.Text.Json.Serialization;
+
+namespace NadekoBot.Modules.Searches;
+
+public partial class Searches
+{
+    public partial class ReligiousCommands : NadekoModule
+    {
+        private readonly IHttpClientFactory _httpFactory;
+        
+        public ReligiousCommands(IHttpClientFactory httpFactory)
+        {
+            _httpFactory = httpFactory;
+        }
+        
+        [Cmd]
+        [RequireContext(ContextType.Guild)]
+        public async Task Bible(string book, string chapterAndVerse)
+        {
+            var obj = new BibleVerses();
+            try
+            {
+                using var http = _httpFactory.CreateClient();
+                obj = await http.GetFromJsonAsync<BibleVerses>($"https://bible-api.com/{book} {chapterAndVerse}");
+            }
+            catch
+            {
+            }
+
+            if (obj.Error is not null || obj.Verses is null || obj.Verses.Length == 0)
+                await Response().Error(obj.Error ?? "No verse found.").SendAsync();
+            else
+            {
+                var v = obj.Verses[0];
+                await Response()
+                      .Embed(_sender.CreateEmbed()
+                                    .WithOkColor()
+                                    .WithTitle($"{v.BookName} {v.Chapter}:{v.Verse}")
+                                    .WithDescription(v.Text))
+                      .SendAsync();
+            }
+        }
+
+        [Cmd]
+        [RequireContext(ContextType.Guild)]
+        public async Task Quran(string ayah)
+        {
+            using var http = _httpFactory.CreateClient(); 
+            
+            var obj = await http.GetFromJsonAsync<QuranResponse<QuranAyah>>($"https://api.alquran.cloud/v1/ayah/{Uri.EscapeDataString(ayah)}/editions/en.asad,ar.alafasy");
+            if(obj is null or not { Code: 200 })
+            {
+                await Response().Error("No verse found.").SendAsync();
+                return;
+            }
+
+            var english = obj.Data[0];
+            var arabic = obj.Data[1];
+
+            await using var audio = await http.GetStreamAsync(arabic.Audio);
+
+            await Response()
+                  .Embed(_sender.CreateEmbed()
+                                .WithOkColor()
+                                .AddField("Arabic", arabic.Text)
+                                .AddField("English", english.Text)
+                                .WithFooter(arabic.Number.ToString()))
+                  .File(audio, Uri.EscapeDataString(ayah) + ".mp3")
+                  .SendAsync();
+        }
+    }
+}
+
+public sealed class QuranResponse<T>
+{
+    [JsonPropertyName("code")]
+    public int Code { get; set; }
+    
+    [JsonPropertyName("status")]
+    public string Status { get; set; }
+    
+    [JsonPropertyName("data")]
+    public T[] Data { get; set; }
+}
+
+public sealed class QuranAyah
+{
+    [JsonPropertyName("number")]
+    public int Number { get; set; }
+    
+    [JsonPropertyName("audio")]
+    public string Audio { get; set; }
+    
+    [JsonPropertyName("name")]
+    public string Name { get; set; }
+    
+    [JsonPropertyName("text")]
+    public string Text { get; set; }
+    
+}

src/NadekoBot/Modules/Searches/Searches.cs

@@ -528,34 +528,6 @@ public partial class Searches : NadekoModule<SearchesService>
         }
     }
 
-    [Cmd]
-    [RequireContext(ContextType.Guild)]
-    public async Task Bible(string book, string chapterAndVerse)
-    {
-        var obj = new BibleVerses();
-        try
-        {
-            using var http = _httpFactory.CreateClient();
-            obj = await http.GetFromJsonAsync<BibleVerses>($"https://bible-api.com/{book} {chapterAndVerse}");
-        }
-        catch
-        {
-        }
-
-        if (obj.Error is not null || obj.Verses is null || obj.Verses.Length == 0)
-            await Response().Error(obj.Error ?? "No verse found.").SendAsync();
-        else
-        {
-            var v = obj.Verses[0];
-            await Response()
-                  .Embed(_sender.CreateEmbed()
-                         .WithOkColor()
-                         .WithTitle($"{v.BookName} {v.Chapter}:{v.Verse}")
-                         .WithDescription(v.Text))
-                  .SendAsync();
-        }
-    }
-
     [Cmd]
     public async Task Steam([Leftover] string query)
     {

src/NadekoBot/Modules/Utility/Ai/AiAssistantService.cs

@@ -251,7 +251,7 @@ public sealed class AiAssistantService
                     return false;
 
                 await _cbs.RunChatterBot(sg, msg, channel, sess, query);
-                return false;
+                return true;
             }
 
             var commandString = GetCommandString(model);

src/NadekoBot/Modules/Utility/Utility.cs

@@ -459,47 +459,82 @@ public partial class Utility : NadekoModule
     public async Task StickerAdd(string name = null, string description = null, params string[] tags)
     {
         string format;
-        Stream stream;
-
-        if (ctx.Message.Stickers.Count is 1 && ctx.Message.Stickers.First() is SocketSticker ss)
-        {
-            name ??= ss.Name;
-            description = ss.Description;
-            tags = tags is null or { Length: 0 } ? ss.Tags.ToArray() : tags;
-            format = FormatToExtension(ss.Format);
-
-            using var http = _httpFactory.CreateClient();
-            stream = await http.GetStreamAsync(ss.GetStickerUrl());
-        }
-        else
-        {
-            await Response().Error(strs.sticker_error).SendAsync();
-            return;
-        }
+        Stream stream = null;
 
         try
         {
-            if (tags.Length == 0)
-                tags = [name];
+            if (ctx.Message.Stickers.Count is 1 && ctx.Message.Stickers.First() is SocketSticker ss)
+            {
+                name ??= ss.Name;
+                description = ss.Description;
+                tags = tags is null or { Length: 0 } ? ss.Tags.ToArray() : tags;
+                format = FormatToExtension(ss.Format);
 
-            await ctx.Guild.CreateStickerAsync(
-                name,
-                stream,
-                $"{name}.{format}",
-                tags,
-                string.IsNullOrWhiteSpace(description) ? "Missing description" : description
-            );
+                using var http = _httpFactory.CreateClient();
+                stream = await http.GetStreamAsync(ss.GetStickerUrl());
+            }
+            else if (ctx.Message.Attachments.Count is 1 && name is not null)
+            {
+                if (tags.Length == 0)
+                    tags = [name];
 
-            await ctx.OkAsync();
-        }
-        catch (Exception ex)
-        {
-            Log.Warning(ex, "Error occurred while adding a sticker: {Message}", ex.Message);
-            await Response().Error(strs.error_occured).SendAsync();
+                if (ctx.Message.Attachments.Count != 1)
+                {
+                    await Response().Error(strs.sticker_error).SendAsync();
+                    return;
+                }
+
+                var attach = ctx.Message.Attachments.First();
+
+
+                if (attach.Size > 512_000 || attach.Width != 300 || attach.Height != 300)
+                {
+                    await Response().Error(strs.sticker_error).SendAsync();
+                    return;
+                }
+
+                format = attach.Filename
+                               .Split('.')
+                               .Last()
+                               .ToLowerInvariant();
+
+                if (string.IsNullOrWhiteSpace(format) || (format != "png" && format != "apng"))
+                {
+                    await Response().Error(strs.sticker_error).SendAsync();
+                    return;
+                }
+
+                using var http = _httpFactory.CreateClient();
+                stream = await http.GetStreamAsync(attach.Url);
+            }
+            else
+            {
+                await Response().Error(strs.sticker_error).SendAsync();
+                return;
+            }
+
+            try
+            {
+                await ctx.Guild.CreateStickerAsync(
+                    name,
+                    stream,
+                    $"{name}.{format}",
+                    tags,
+                    string.IsNullOrWhiteSpace(description) ? "Missing description" : description
+                );
+
+                await ctx.OkAsync();
+            }
+            catch
+                (Exception ex)
+            {
+                Log.Warning(ex, "Error occurred while adding a sticker: {Message}", ex.Message);
+                await Response().Error(strs.error_occured).SendAsync();
+            }
         }
         finally
         {
-            await stream.DisposeAsync();
+            await (stream?.DisposeAsync() ?? ValueTask.CompletedTask);
         }
     }
 

src/NadekoBot/NadekoBot.csproj

@@ -4,7 +4,7 @@
         <Nullable>enable</Nullable>
         <ImplicitUsings>true</ImplicitUsings>
         <SatelliteResourceLanguages>en</SatelliteResourceLanguages>
-        <Version>5.1.1</Version>
+        <Version>5.1.3</Version>
 
         <!-- Output/build -->
         <RunWorkingDirectory>$(MSBuildProjectDirectory)</RunWorkingDirectory>

src/NadekoBot/data/aliases.yml

@@ -1192,6 +1192,8 @@ xpreset:
   - xpreset
 bible:
   - bible
+quran:
+  - quran
 edit:
   - edit
 delete:

src/NadekoBot/data/strings/commands/commands.en-US.yml

@@ -2289,7 +2289,11 @@ emojiremove:
     - emotes:
         desc: "The list of emojis to be removed from the server."
 stickeradd:
-  desc: Adds the sticker from your message to this server. Send the sticker along with this command (in the same message).
+  desc: |-
+    Adds the sticker from your message to this server.
+    Send the sticker along with this command (in the same message).
+    Alternatively you can upload an image along with this command but you have to specify the name.
+    The image must be 300x300 in .png or .apng format and up to 512KB in size.
   ex:
     - ''
     - name "description" tag1 tag2 tagN
@@ -4076,6 +4080,16 @@ bible:
         desc: "The name of the biblical book being referenced."
       chapterAndVerse:
         desc: "The reference to a specific passage in the Bible, such as 'Genesis 3:15'"
+quran:
+  desc: |-
+    Shows the text of an ayah of the Quran, as well as the recitation by Alafasy.
+    Supply surah:ayah, or ayah number. For instance, 262 or 2:255 will both get you Ayat Al Kursi
+  ex:
+    - 2:255
+    - 262
+  params:
+    - ayah:
+        desc: "The number of the ayah in the Quran, for example 2:255."
 edit:
   desc: Edits bot's message, you have to specify message ID and new text. You can optionally specify target channel. Supports embeds.
   ex:

src/NadekoBot/data/strings/responses/responses.en-US.json

@@ -1067,8 +1067,7 @@
   "xpshop_already_owned": "You already own this item.",
   "xpshop_item_not_found": "An item with that key doesn't exist.",
   "xpshop_website": "You can see the list of all Xp Shop items here: <https://xpshop.nadeko.bot>",
-  "sticker_invalid_size": "Stickers must be exactly 300x300 pixels.",
-  "sticker_error": "You must either send a sticker along with this command, or upload a 300x300 .png or .apng image.",
+  "sticker_error": "You must either send a sticker along with this command, or upload a 300x300 .png or .apng image. Up to 512KB in size.",
   "sticker_missing_name": "Please specify a name for the sticker.",
   "thread_deleted": "Thread Deleted",
   "thread_created": "Thread Created",