.github/workflows/clang-tidy.yml

@@ -10,7 +10,6 @@ concurrency:
 
 jobs:
   review:
-    if: ${{ !contains(github.event.pull_request.labels.*.name, 'skip-clang-tidy') }}
     name: "clang-tidy ${{ matrix.os }}, Qt ${{ matrix.qt-version }})"
     runs-on: ${{ matrix.os }}
     strategy:

.github/workflows/test-macos.yml

@@ -8,7 +8,6 @@ on:
 
 env:
   TWITCH_PUBSUB_SERVER_TAG: v1.0.7
-  HTTPBOX_TAG: v0.2.1
   QT_QPA_PLATFORM: minimal
   HOMEBREW_NO_AUTO_UPDATE: 1
   HOMEBREW_NO_INSTALLED_DEPENDENTS_CHECK: 1
@@ -25,7 +24,7 @@ jobs:
       matrix:
         os: [macos-13]
         qt-version: [5.15.2, 6.7.1]
-        plugins: [true]
+        plugins: [false]
       fail-fast: false
     env:
       C2_BUILD_WITH_QT6: ${{ startsWith(matrix.qt-version, '6.') && 'ON' || 'OFF' }}
@@ -59,13 +58,6 @@ jobs:
         run: |
           brew install boost openssl rapidjson p7zip create-dmg cmake
 
-      - name: Install httpbox
-        run: |
-          curl -L -o httpbox.tar.xz "https://github.com/Chatterino/httpbox/releases/download/${{ env.HTTPBOX_TAG }}/httpbox-x86_64-apple-darwin.tar.xz"
-          tar -xJf httpbox.tar.xz
-          mv ./httpbox-x86_64-apple-darwin/httpbox /usr/local/bin
-        working-directory: /tmp
-
       - name: Build
         run: |
           mkdir build-test
@@ -91,6 +83,10 @@ jobs:
           curl -L -o server.key "https://github.com/Chatterino/twitch-pubsub-server-test/raw/${{ env.TWITCH_PUBSUB_SERVER_TAG }}/cmd/server/server.key"
           cd ..
 
+      - name: Cargo Install httpbox
+        run: |
+          cargo install --git https://github.com/kevinastone/httpbox --rev 89b971f
+
       - name: Test
         timeout-minutes: 30
         run: |

.github/workflows/test-windows.yml

@@ -8,7 +8,6 @@ on:
 
 env:
   TWITCH_PUBSUB_SERVER_TAG: v1.0.7
-  HTTPBOX_TAG: v0.2.1
   QT_QPA_PLATFORM: minimal
   # Last known good conan version
   # 2.0.3 has a bug on Windows (conan-io/conan#13606)
@@ -26,7 +25,7 @@ jobs:
       matrix:
         os: [windows-latest]
         qt-version: [5.15.2, 6.7.1]
-        plugins: [true]
+        plugins: [false]
         skip-artifact: [false]
         skip-crashpad: [false]
       fail-fast: false
@@ -112,13 +111,6 @@ jobs:
           mkdir -Force build-test/bin
           cp "$((ls $Env:VCToolsRedistDir/onecore/x64 -Filter '*.CRT')[0].FullName)/*" build-test/bin
 
-      - name: Install httpbox
-        run: |
-          mkdir httpbox
-          Invoke-WebRequest -Uri "https://github.com/Chatterino/httpbox/releases/download/${{ env.HTTPBOX_TAG }}/httpbox-x86_64-pc-windows-msvc.zip" -outfile "httpbox.zip"
-          Expand-Archive httpbox.zip -DestinationPath httpbox
-          rm httpbox.zip
-
       - name: Build
         run: |
           cmake `
@@ -147,10 +139,14 @@ jobs:
           Invoke-WebRequest -Uri "https://github.com/Chatterino/twitch-pubsub-server-test/raw/${{ env.TWITCH_PUBSUB_SERVER_TAG }}/cmd/server/server.key" -outfile "server.key"
           cd ..
 
+      - name: Cargo Install httpbox
+        run: |
+          cargo install --git https://github.com/kevinastone/httpbox --rev 89b971f
+
       - name: Test
         timeout-minutes: 30
         run: |
-          ..\httpbox\httpbox.exe --port 9051 &
+          httpbox --port 9051 &
           cd ..\pubsub-server-test
           .\server.exe 127.0.0.1:9050 &
           cd ..\build-test

.gitmodules

@@ -44,6 +44,3 @@
 [submodule "lib/expected-lite"]
 	path = lib/expected-lite
 	url = https://github.com/martinmoene/expected-lite
-[submodule "lib/sol2"]
-	path = lib/sol2
-	url = https://github.com/ThePhD/sol2.git

.prettierignore

@@ -2,7 +2,6 @@
 resources/*.json
 benchmarks/resources/*.json
 tests/resources/*.json
-tests/snapshots/**/*.json
 # ...themes should be prettified for readability.
 !resources/themes/*.json
 

BUILDING_ON_WINDOWS.md

@@ -170,10 +170,9 @@ To automatically format your code, do the following:
 
 1. Download [LLVM 16.0.6](https://github.com/llvm/llvm-project/releases/download/llvmorg-16.0.6/LLVM-16.0.6-win64.exe)
 2. During the installation, make sure to add it to your path
-3. Enable Beautifier under `Extensions` on the left (check "Load on start" and restart)
-4. In Qt Creator, Select `Tools` > `Options` > `Beautifier`
-5. Under `General` select `Tool: ClangFormat` and enable `Automatic Formatting on File Save`
-6. Under `Clang Format` select `Use predefined style: File` and `Fallback style: None`
+3. In Qt Creator, Select `Tools` > `Options` > `Beautifier`
+4. Under `General` select `Tool: ClangFormat` and enable `Automatic Formatting on File Save`
+5. Under `Clang Format` select `Use predefined style: File` and `Fallback style: None`
 
 ### Building on MSVC with AddressSanitizer
 

CHANGELOG.md

@@ -4,10 +4,10 @@
 
 - Major: Add option to show pronouns in user card. (#5442, #5583)
 - Major: Release plugins alpha. (#5288)
-- Major: Improve high-DPI support on Windows. (#4868, #5391, #5664, #5666)
-- Major: Added transparent overlay window (default keybind: <kbd>CTRL</kbd> + <kbd>ALT</kbd> + <kbd>N</kbd>). (#4746, #5643, #5659)
+- Major: Improve high-DPI support on Windows. (#4868, #5391)
+- Major: Added transparent overlay window (default keybind: <kbd>CTRL</kbd> + <kbd>ALT</kbd> + <kbd>N</kbd>). (#4746)
 - Minor: Removed the Ctrl+Shift+L hotkey for toggling the "live only" tab visibility state. (#5530)
-- Minor: Add support for Shared Chat messages. Shared chat messages can be filtered with the `flags.shared` filter variable, or with search using `is:shared`. Some messages like subscriptions are filtered on purpose to avoid confusion for the broadcaster. If you have both channels participating in Shared Chat open, only one of the message triggering your highlight will trigger. (#5606, #5625, #5661)
+- Minor: Add support for Shared Chat messages. Shared chat messages can be filtered with the `flags.shared` filter variable, or with search using `is:shared`. Some messages like subscriptions are filtered on purpose to avoid confusion for the broadcaster. If you have both channels participating in Shared Chat open, only one of the message triggering your highlight will trigger. (#5606, #5625)
 - Minor: Moved tab visibility control to a submenu, without any toggle actions. (#5530)
 - Minor: Add option to customise Moderation buttons with images. (#5369)
 - Minor: Colored usernames now update on the fly when changing the "Color @usernames" setting. (#5300)
@@ -30,16 +30,10 @@
 - Minor: Links can now have prefixes and suffixes such as parentheses. (#5486, #5515)
 - Minor: Added support for scrolling in splits with touchscreen panning gestures. (#5524)
 - Minor: Removed experimental IRC support. (#5547)
-- Minor: Remember last popup size for next popup. (#5635)
 - Minor: Moderators can now see which mods start and cancel raids. (#5563)
 - Minor: The emote popup now reloads when Twitch emotes are reloaded. (#5580)
 - Minor: Added `--login <username>` CLI argument to specify which account to start logged in as. (#5626)
 - Minor: When blocking a channel, Chatterino will now warn you about that action. (#5615)
-- Minor: Indicate when subscriptions and resubscriptions are for multiple months. (#5642)
-- Minor: Proxy URL information is now included in the `/debug-env` command. (#5648)
-- Minor: Make raid entry message usernames clickable. (#5651)
-- Minor: Tabs unhighlight when their content is read in other tabs. (#5649)
-- Minor: Made usernames in bits and sub messages clickable. (#5686)
 - Bugfix: Fixed tab move animation occasionally failing to start after closing a tab. (#5426, #5612)
 - Bugfix: If a network request errors with 200 OK, Qt's error code is now reported instead of the HTTP status. (#5378)
 - Bugfix: Fixed restricted users usernames not being clickable. (#5405)
@@ -51,14 +45,12 @@
 - Bugfix: Fixed tooltips and input completion popups not working after moving a split. (#5541, #5576)
 - Bugfix: Fixed rare issue on shutdown where the client would hang. (#5557)
 - Bugfix: Fixed `/clearmessages` not working with more than one window. (#5489)
-- Bugfix: Fixed splits staying paused after unfocusing Chatterino in certain configurations. (#5504, #5637)
+- Bugfix: Fixed splits staying paused after unfocusing Chatterino in certain configurations. (#5504)
 - Bugfix: Links with invalid characters in the domain are no longer detected. (#5509)
 - Bugfix: Fixed janky selection for messages with RTL segments (selection is still wrong, but consistently wrong). (#5525)
-- Bugfix: Fixed event emotes not showing up in autocomplete and popups. (#5239, #5580, #5582, #5632)
+- Bugfix: Fixed event emotes not showing up in autocomplete and popups. (#5239, #5580, #5582)
 - Bugfix: Fixed tab visibility being controllable in the emote popup. (#5530)
 - Bugfix: Fixed account switch not being saved if no other settings were changed. (#5558)
-- Bugfix: Fixed a crash that could occur when handling the quick switcher popup really quickly. (#5687)
-- Bugfix: Fixed 7TV badges being inadvertently animated. (#5674)
 - Bugfix: Fixed some tooltips not being readable. (#5578)
 - Bugfix: Fixed log files being locked longer than needed. (#5592)
 - Bugfix: Fixed global badges not showing in anonymous mode. (#5599)
@@ -66,8 +58,6 @@
 - Bugfix: Fixed incorrect message being disabled in some cases upon approving or denying an automod caught message. (#5611)
 - Bugfix: Fixed double-click selection not working when clicking outside a message. (#5617)
 - Bugfix: Fixed emotes starting with ":" not tab-completing. (#5603)
-- Bugfix: Fixed 7TV emotes messing with Qt's HTML. (#5677)
-- Bugfix: Fixed incorrect messages getting replaced visually. (#5683)
 - Dev: Update Windows build from Qt 6.5.0 to Qt 6.7.1. (#5420)
 - Dev: Update vcpkg build Qt from 6.5.0 to 6.7.0, boost from 1.83.0 to 1.85.0, openssl from 3.1.3 to 3.3.0. (#5422)
 - Dev: Unsingletonize `ISoundController`. (#5462)
@@ -80,7 +70,7 @@
 - Dev: Removed unused timegate settings. (#5361)
 - Dev: Add `Channel::addSystemMessage` helper function, allowing us to avoid the common `channel->addMessage(makeSystemMessage(...));` pattern. (#5500)
 - Dev: Unsingletonize `Resources2`. (#5460)
-- Dev: All Lua globals now show in the `c2` global in the LuaLS metadata. (#5385, #5682)
+- Dev: All Lua globals now show in the `c2` global in the LuaLS metadata. (#5385)
 - Dev: Images are now loaded in worker threads. (#5431)
 - Dev: Fixed broken `SignalVector::operator[]` implementation. (#5556)
 - Dev: Qt Creator now auto-configures Conan when loading the project and skips vcpkg. (#5305)
@@ -111,17 +101,9 @@
 - Dev: Added more tests for input completion. (#5604)
 - Dev: Refactored legacy Unicode zero-width-joiner replacement. (#5594)
 - Dev: The JSON output when copying a message (<kbd>SHIFT</kbd> + right-click) is now more extensive. (#5600)
-- Dev: Added more tests for message building. (#5598, #5654, #5656, #5671)
 - Dev: Twitch messages are now sent using Twitch's Helix API instead of IRC by default. (#5607)
 - Dev: `GIFTimer` is no longer initialized in tests. (#5608)
 - Dev: Emojis now use flags instead of a set of strings for capabilities. (#5616)
-- Dev: Move plugins to Sol2. (#5622, #5682)
-- Dev: Refactored static `MessageBuilder` helpers to standalone functions. (#5652)
-- Dev: Decoupled reply parsing from `MessageBuilder`. (#5660, #5668)
-- Dev: Refactored IRC message building. (#5663)
-- Dev: Fixed some compiler warnings. (#5672)
-- Dev: Unified parsing of historic and live IRC messages. (#5678)
-- Dev: 7TV's `entitlement.reset` is now explicitly ignored. (#5685)
 
 ## 2.5.1
 

CMakeLists.txt

@@ -1,9 +1,6 @@
 cmake_minimum_required(VERSION 3.15)
 cmake_policy(SET CMP0087 NEW) # evaluates generator expressions in `install(CODE/SCRIPT)`
 cmake_policy(SET CMP0091 NEW) # select MSVC runtime library through `CMAKE_MSVC_RUNTIME_LIBRARY`
-if (POLICY CMP0167)
-    cmake_policy(SET CMP0167 NEW) # find Boost's own CMake config file
-endif ()
 include(FeatureSummary)
 
 list(APPEND CMAKE_MODULE_PATH
@@ -215,8 +212,6 @@ endif()
 if (CHATTERINO_PLUGINS)
     set(LUA_INCLUDE_DIRS "${CMAKE_SOURCE_DIR}/lib/lua/src")
     add_subdirectory(lib/lua)
-
-    find_package(Sol2 REQUIRED)
 endif()
 
 if (BUILD_WITH_CRASHPAD)

benchmarks/CMakeLists.txt

@@ -5,6 +5,7 @@ set(benchmark_SOURCES
     resources/bench.qrc
 
     src/Emojis.cpp
+    src/Highlights.cpp
     src/FormatTime.cpp
     src/Helpers.cpp
     src/LimitedQueue.cpp

benchmarks/src/Highlights.cpp

@@ -0,0 +1,102 @@
+#include "Application.hpp"
+#include "common/Channel.hpp"
+#include "controllers/accounts/AccountController.hpp"
+#include "controllers/highlights/HighlightController.hpp"
+#include "controllers/highlights/HighlightPhrase.hpp"
+#include "messages/Message.hpp"
+#include "messages/MessageBuilder.hpp"
+#include "mocks/BaseApplication.hpp"
+#include "mocks/UserData.hpp"
+#include "util/Helpers.hpp"
+
+#include <benchmark/benchmark.h>
+#include <QDebug>
+#include <QString>
+#include <QTemporaryDir>
+
+using namespace chatterino;
+
+class BenchmarkMessageBuilder : public MessageBuilder
+{
+public:
+    explicit BenchmarkMessageBuilder(
+        Channel *_channel, const Communi::IrcPrivateMessage *_ircMessage,
+        const MessageParseArgs &_args)
+        : MessageBuilder(_channel, _ircMessage, _args)
+    {
+    }
+
+    virtual MessagePtr build()
+    {
+        // PARSE
+        this->parse();
+        this->usernameColor_ = getRandomColor(this->ircMessage->nick());
+
+        // words
+        // this->addWords(this->originalMessage_.split(' '));
+
+        this->message().messageText = this->originalMessage_;
+        this->message().searchText = this->message().localizedName + " " +
+                                     this->userName + ": " +
+                                     this->originalMessage_;
+        return nullptr;
+    }
+
+    void bench()
+    {
+        this->parseHighlights();
+    }
+};
+
+class MockApplication : public mock::BaseApplication
+{
+public:
+    MockApplication()
+        : highlights(this->settings, &this->accounts)
+    {
+    }
+
+    AccountController *getAccounts() override
+    {
+        return &this->accounts;
+    }
+    HighlightController *getHighlights() override
+    {
+        return &this->highlights;
+    }
+
+    IUserDataController *getUserData() override
+    {
+        return &this->userData;
+    }
+
+    AccountController accounts;
+    HighlightController highlights;
+    mock::UserDataController userData;
+};
+
+static void BM_HighlightTest(benchmark::State &state)
+{
+    MockApplication mockApplication;
+
+    std::string message =
+        R"(@badge-info=subscriber/34;badges=moderator/1,subscriber/24;color=#FF0000;display-name=테스트계정420;emotes=41:6-13,15-22;flags=;id=a3196c7e-be4c-4b49-9c5a-8b8302b50c2a;mod=1;room-id=11148817;subscriber=1;tmi-sent-ts=1590922213730;turbo=0;user-id=117166826;user-type=mod :testaccount_420!testaccount_420@testaccount_420.tmi.twitch.tv PRIVMSG #pajlada :-tags Kreygasm,Kreygasm (no space))";
+    auto ircMessage = Communi::IrcMessage::fromData(message.c_str(), nullptr);
+    auto privMsg = dynamic_cast<Communi::IrcPrivateMessage *>(ircMessage);
+    assert(privMsg != nullptr);
+    MessageParseArgs args;
+    auto emptyChannel = Channel::getEmpty();
+
+    for (auto _ : state)
+    {
+        state.PauseTiming();
+        BenchmarkMessageBuilder b(emptyChannel.get(), privMsg, args);
+
+        b.build();
+        state.ResumeTiming();
+
+        b.bench();
+    }
+}
+
+BENCHMARK(BM_HighlightTest);

cmake/FindSol2.cmake

@@ -1,21 +0,0 @@
-include(FindPackageHandleStandardArgs)
-
-find_path(Sol2_INCLUDE_DIR sol/sol.hpp HINTS ${CMAKE_SOURCE_DIR}/lib/sol2/include)
-
-find_package_handle_standard_args(Sol2 DEFAULT_MSG Sol2_INCLUDE_DIR)
-
-if (Sol2_FOUND)
-    add_library(Sol2 INTERFACE IMPORTED)
-    set_target_properties(Sol2 PROPERTIES
-        INTERFACE_INCLUDE_DIRECTORIES "${Sol2_INCLUDE_DIR}"
-    )
-    target_compile_definitions(Sol2 INTERFACE 
-        SOL_ALL_SAFETIES_ON=1
-        SOL_USING_CXX_LUA=1
-        SOL_NO_NIL=0
-    )
-    target_link_libraries(Sol2 INTERFACE lua)
-    add_library(sol2::sol2 ALIAS Sol2)
-endif ()
-
-mark_as_advanced(Sol2_INCLUDE_DIR)

cmake/sanitizers-cmake

@@ -1 +1 @@
-Subproject commit 0573e2ea8651b9bb3083f193c41eb086497cc80a
+Subproject commit 3f0542e4e034aab417c51b2b22c94f83355dee15

docs/plugin-meta.lua

@@ -5,22 +5,13 @@
 -- Add the folder this file is in to "Lua.workspace.library".
 
 c2 = {}
----@enum c2.LogLevel
-c2.LogLevel = {
-    Debug = {}, ---@type c2.LogLevel.Debug
-    Info = {}, ---@type c2.LogLevel.Info
-    Warning = {}, ---@type c2.LogLevel.Warning
-    Critical = {}, ---@type c2.LogLevel.Critical
-}
+---@alias c2.LogLevel integer
+---@type { Debug: c2.LogLevel, Info: c2.LogLevel, Warning: c2.LogLevel, Critical: c2.LogLevel }
+c2.LogLevel = {}
 
--- Begin src/controllers/plugins/api/EventType.hpp
-
----@enum c2.EventType
-c2.EventType = {
-    CompletionRequested = {}, ---@type c2.EventType.CompletionRequested
-}
-
--- End src/controllers/plugins/api/EventType.hpp
+---@alias c2.EventType integer
+---@type { CompletionRequested: c2.EventType }
+c2.EventType = {}
 
 ---@class CommandContext
 ---@field words string[] The words typed when executing the command. For example `/foo bar baz` will result in `{"/foo", "bar", "baz"}`.
@@ -38,40 +29,19 @@ c2.EventType = {
 
 -- Begin src/common/Channel.hpp
 
----@enum c2.ChannelType
-c2.ChannelType = {
-    None = {}, ---@type c2.ChannelType.None
-    Direct = {}, ---@type c2.ChannelType.Direct
-    Twitch = {}, ---@type c2.ChannelType.Twitch
-    TwitchWhispers = {}, ---@type c2.ChannelType.TwitchWhispers
-    TwitchWatching = {}, ---@type c2.ChannelType.TwitchWatching
-    TwitchMentions = {}, ---@type c2.ChannelType.TwitchMentions
-    TwitchLive = {}, ---@type c2.ChannelType.TwitchLive
-    TwitchAutomod = {}, ---@type c2.ChannelType.TwitchAutomod
-    TwitchEnd = {}, ---@type c2.ChannelType.TwitchEnd
-    Misc = {}, ---@type c2.ChannelType.Misc
-}
+---@alias c2.ChannelType integer
+---@type { None: c2.ChannelType, Direct: c2.ChannelType, Twitch: c2.ChannelType, TwitchWhispers: c2.ChannelType, TwitchWatching: c2.ChannelType, TwitchMentions: c2.ChannelType, TwitchLive: c2.ChannelType, TwitchAutomod: c2.ChannelType, TwitchEnd: c2.ChannelType, Irc: c2.ChannelType, Misc: c2.ChannelType }
+c2.ChannelType = {}
 
 -- End src/common/Channel.hpp
 
 -- Begin src/controllers/plugins/api/ChannelRef.hpp
 
--- Begin src/providers/twitch/TwitchChannel.hpp
-
----@class StreamStatus
----@field live boolean
----@field viewer_count number
----@field title string Stream title or last stream title
----@field game_name string
----@field game_id string
----@field uptime number Seconds since the stream started.
-
----@class RoomModes
----@field subscriber_only boolean
----@field unique_chat boolean You might know this as r9kbeta or robot9000.
----@field emotes_only boolean Whether or not text is allowed in messages. Note that "emotes" here only means Twitch emotes, not Unicode emoji, nor 3rd party text-based emotes
-
--- End src/providers/twitch/TwitchChannel.hpp
+---@alias c2.Platform integer
+--- This enum describes a platform for the purpose of searching for a channel.
+--- Currently only Twitch is supported because identifying IRC channels is tricky.
+---@type { Twitch: c2.Platform }
+c2.Platform = {}
 
 ---@class c2.Channel
 c2.Channel = {}
@@ -102,7 +72,7 @@ function c2.Channel:get_display_name() end
 --- Note that this does not execute client-commands.
 ---
 ---@param message string
----@param execute_commands? boolean Should commands be run on the text?
+---@param execute_commands boolean Should commands be run on the text?
 function c2.Channel:send_message(message, execute_commands) end
 
 --- Adds a system message client-side
@@ -161,8 +131,9 @@ function c2.Channel:__tostring() end
 --- - /automod
 ---
 ---@param name string Which channel are you looking for?
+---@param platform c2.Platform Where to search for the channel?
 ---@return c2.Channel?
-function c2.Channel.by_name(name) end
+function c2.Channel.by_name(name, platform) end
 
 --- Finds a channel by the Twitch user ID of its owner.
 ---
@@ -170,101 +141,98 @@ function c2.Channel.by_name(name) end
 ---@return c2.Channel?
 function c2.Channel.by_twitch_id(id) end
 
+---@class RoomModes
+---@field unique_chat boolean You might know this as r9kbeta or robot9000.
+---@field subscriber_only boolean
+---@field emotes_only boolean Whether or not text is allowed in messages. Note that "emotes" here only means Twitch emotes, not Unicode emoji, nor 3rd party text-based emotes
+---@field follower_only number? Time in minutes you need to follow to chat or nil.
+---@field slow_mode number? Time in seconds you need to wait before sending messages or nil.
+
+---@class StreamStatus
+---@field live boolean
+---@field viewer_count number
+---@field uptime number Seconds since the stream started.
+---@field title string Stream title or last stream title
+---@field game_name string
+---@field game_id string
+
 -- End src/controllers/plugins/api/ChannelRef.hpp
 
 -- Begin src/controllers/plugins/api/HTTPResponse.hpp
 
----@class c2.HTTPResponse
-c2.HTTPResponse = {}
+---@class HTTPResponse
+HTTPResponse = {}
 
 --- Returns the data. This is not guaranteed to be encoded using any
 --- particular encoding scheme. It's just the bytes the server returned.
 ---
----@return string
----@nodiscard
-function c2.HTTPResponse:data() end
+function HTTPResponse:data() end
 
 --- Returns the status code.
 ---
----@return number|nil
----@nodiscard
-function c2.HTTPResponse:status() end
+function HTTPResponse:status() end
 
 --- A somewhat human readable description of an error if such happened
 ---
----@return string
----@nodiscard
-function c2.HTTPResponse:error() end
-
----@return string
----@nodiscard
-function c2.HTTPResponse:__tostring() end
+function HTTPResponse:error() end
 
 -- End src/controllers/plugins/api/HTTPResponse.hpp
 
 -- Begin src/controllers/plugins/api/HTTPRequest.hpp
 
----@alias c2.HTTPCallback fun(result: c2.HTTPResponse): nil
----@class c2.HTTPRequest
-c2.HTTPRequest = {}
+---@alias HTTPCallback fun(result: HTTPResponse): nil
+---@class HTTPRequest
+HTTPRequest = {}
 
 --- Sets the success callback
 ---
----@param callback c2.HTTPCallback Function to call when the HTTP request succeeds
-function c2.HTTPRequest:on_success(callback) end
+---@param callback HTTPCallback Function to call when the HTTP request succeeds
+function HTTPRequest:on_success(callback) end
 
 --- Sets the failure callback
 ---
----@param callback c2.HTTPCallback Function to call when the HTTP request fails or returns a non-ok status
-function c2.HTTPRequest:on_error(callback) end
+---@param callback HTTPCallback Function to call when the HTTP request fails or returns a non-ok status
+function HTTPRequest:on_error(callback) end
 
 --- Sets the finally callback
 ---
 ---@param callback fun(): nil Function to call when the HTTP request finishes
-function c2.HTTPRequest:finally(callback) end
+function HTTPRequest:finally(callback) end
 
 --- Sets the timeout
 ---
 ---@param timeout integer How long in milliseconds until the times out
-function c2.HTTPRequest:set_timeout(timeout) end
+function HTTPRequest:set_timeout(timeout) end
 
 --- Sets the request payload
 ---
 ---@param data string
-function c2.HTTPRequest:set_payload(data) end
+function HTTPRequest:set_payload(data) end
 
 --- Sets a header in the request
 ---
 ---@param name string
 ---@param value string
-function c2.HTTPRequest:set_header(name, value) end
+function HTTPRequest:set_header(name, value) end
 
 --- Executes the HTTP request
 ---
-function c2.HTTPRequest:execute() end
-
----@return string
-function c2.HTTPRequest:__tostring() end
+function HTTPRequest:execute() end
 
 --- Creates a new HTTPRequest
 ---
----@param method c2.HTTPMethod Method to use
+---@param method HTTPMethod Method to use
 ---@param url string Where to send the request to
----@return c2.HTTPRequest
-function c2.HTTPRequest.create(method, url) end
+---@return HTTPRequest
+function HTTPRequest.create(method, url) end
 
 -- End src/controllers/plugins/api/HTTPRequest.hpp
 
 -- Begin src/common/network/NetworkCommon.hpp
 
----@enum c2.HTTPMethod
-c2.HTTPMethod = {
-    Get = {}, ---@type c2.HTTPMethod.Get
-    Post = {}, ---@type c2.HTTPMethod.Post
-    Put = {}, ---@type c2.HTTPMethod.Put
-    Delete = {}, ---@type c2.HTTPMethod.Delete
-    Patch = {}, ---@type c2.HTTPMethod.Patch
-}
+---@alias HTTPMethod integer
+---@type { Get: HTTPMethod, Post: HTTPMethod, Put: HTTPMethod, Delete: HTTPMethod, Patch: HTTPMethod }
+HTTPMethod = {}
 
 -- End src/common/network/NetworkCommon.hpp
 
@@ -277,7 +245,7 @@ function c2.register_command(name, handler) end
 
 --- Registers a callback to be invoked when completions for a term are requested.
 ---
----@param type c2.EventType.CompletionRequested
+---@param type "CompletionRequested"
 ---@param func fun(event: CompletionEvent): CompletionList The callback to be invoked.
 function c2.register_callback(type, func) end
 

docs/wip-plugins.md

@@ -171,7 +171,7 @@ function cmd_words(ctx)
     -- ctx contains:
     -- words - table of words supplied to the command including the trigger
     -- channel - the channel the command is being run in
-    ctx.channel:add_system_message("Words are: " .. table.concat(ctx.words, " "))
+    channel:add_system_message("Words are: " .. table.concat(ctx.words, " "))
 end
 
 c2.register_command("/words", cmd_words)
@@ -183,7 +183,7 @@ Limitations/known issues:
   rebuilding the window content caused by reloading another plugin will solve this.
 - Spaces in command names aren't handled very well (https://github.com/Chatterino/chatterino2/issues/1517).
 
-#### `register_callback(c2.EventType.CompletionRequested, handler)`
+#### `register_callback("CompletionRequested", handler)`
 
 Registers a callback (`handler`) to process completions. The callback takes a single table with the following entries:
 
@@ -207,7 +207,7 @@ function string.startswith(s, other)
 end
 
 c2.register_callback(
-    c2.EventType.CompletionRequested,
+    "CompletionRequested",
     function(event)
         if ("!join"):startswith(event.query) then
             ---@type CompletionList
@@ -219,6 +219,15 @@ c2.register_callback(
 )
 ```
 
+#### `Platform` enum
+
+This table describes platforms that can be accessed. Chatterino supports IRC
+however plugins do not yet have explicit access to get IRC channels objects.
+The values behind the names may change, do not count on them. It has the
+following keys:
+
+- `Twitch`
+
 #### `ChannelType` enum
 
 This table describes channel types Chatterino supports. The values behind the
@@ -251,9 +260,9 @@ used on non-Twitch channels. Special channels while marked as
 is an actual Twitch chatroom use `Channel:get_type()` instead of
 `Channel:is_twitch_channel()`.
 
-##### `Channel:by_name(name)`
+##### `Channel:by_name(name, platform)`
 
-Finds a channel given by `name`. Returns the channel or `nil` if not open.
+Finds a channel given by `name` on `platform` (see [`Platform` enum](#Platform-enum)). Returns the channel or `nil` if not open.
 
 Some miscellaneous channels are marked as if they are specifically Twitch channels:
 
@@ -266,7 +275,7 @@ Some miscellaneous channels are marked as if they are specifically Twitch channe
 Example:
 
 ```lua
-local pajladas = c2.Channel.by_name("pajlada")
+local pajladas = c2.Channel.by_name("pajlada", c2.Platform.Twitch)
 ```
 
 ##### `Channel:by_twitch_id(id)`
@@ -354,7 +363,7 @@ pajladas:add_system_message("Hello, world!")
 
 Returns `true` if the channel is a Twitch channel, that is its type name has
 the `Twitch` prefix. This returns `true` for special channels like Mentions.
-You might want `Channel:get_type() == c2.ChannelType.Twitch` if you want to use
+You might want `Channel:get_type() == "Twitch"` if you want to use
 Twitch-specific functions.
 
 ##### `Channel:get_twitch_id()`

lib/expected-lite

@@ -1 +1 @@
-Subproject commit 5b5caad7cd57d5ba3ca796bf1521b131d73ca405
+Subproject commit f339d2f73730f8fee4412f5e4938717866ecef48

lib/lua/CMakeLists.txt

@@ -1,44 +1,48 @@
 project(lua CXX)
 
 #[====[
-This list contains all .c files except lua.c and onelua.c
-Use the following command from the repository root to get these file:
-perl -e 'print s/^lib\/lua\///r . "\n" for grep { /\.c$/ && !/(lua|onelua)\.c$/ } glob "lib/lua/src/*.c"'
+Updating this list:
+remove all listed files
+go to line below, ^y2j4j$@" and then reindent the file names
+/LUA_SRC
+:r!ls lib/lua/src | grep '\.c' | grep -Ev 'lua\.c|onelua\.c' | sed 's#^#src/#'
+
 #]====]
 set(LUA_SRC 
-    src/lapi.c
-    src/lauxlib.c
-    src/lbaselib.c
-    src/lcode.c
-    src/lcorolib.c
-    src/lctype.c
-    src/ldblib.c
-    src/ldebug.c
-    src/ldo.c
-    src/ldump.c
-    src/lfunc.c
-    src/lgc.c
-    src/linit.c
-    src/liolib.c
-    src/llex.c
-    src/lmathlib.c
-    src/lmem.c
-    src/loadlib.c
-    src/lobject.c
-    src/lopcodes.c
-    src/loslib.c
-    src/lparser.c
-    src/lstate.c
-    src/lstring.c
-    src/lstrlib.c
-    src/ltable.c
-    src/ltablib.c
-    src/ltests.c
-    src/ltm.c
-    src/lundump.c
-    src/lutf8lib.c
-    src/lvm.c
-    src/lzio.c
+    "src/lapi.c"
+    "src/lauxlib.c"
+    "src/lbaselib.c"
+    "src/lcode.c"
+    "src/lcorolib.c"
+    "src/lctype.c"
+    "src/ldblib.c"
+    "src/ldebug.c"
+    "src/ldo.c"
+    "src/ldump.c"
+    "src/lfunc.c"
+    "src/lgc.c"
+    "src/linit.c"
+    "src/liolib.c"
+    "src/llex.c"
+    "src/lmathlib.c"
+    "src/lmem.c"
+    "src/loadlib.c"
+    "src/lobject.c"
+    "src/lopcodes.c"
+    "src/loslib.c"
+    "src/lparser.c"
+    "src/lstate.c"
+    "src/lstring.c"
+    "src/lstrlib.c"
+    "src/ltable.c"
+    "src/ltablib.c"
+    "src/ltests.c"
+    "src/ltm.c"
+    "src/lua.c"
+    "src/lundump.c"
+    "src/lutf8lib.c"
+    "src/lvm.c"
+    "src/lzio.c"
 )
 
 add_library(lua STATIC ${LUA_SRC})
@@ -46,14 +50,4 @@ target_include_directories(lua
     PUBLIC
     ${LUA_INCLUDE_DIRS}
 )
-set_target_properties(${liblua} PROPERTIES
-    LANGUAGE CXX
-    LINKER_LANGUAGE CXX
-    CXX_STANDARD 98
-    CXX_EXTENSIONS TRUE
-)
-target_compile_options(lua PRIVATE
-    -w # this makes clang shut up about c-as-c++
-    $<$<AND:$<BOOL:${MSVC}>,$<CXX_COMPILER_ID:Clang>>:/EHsc> # enable exceptions in clang-cl
-)
-set_source_files_properties(${LUA_SRC} PROPERTIES LANGUAGE CXX)
+set_source_files_properties(${LUA_SRC} PROPERTIES LANGUAGE C)

lib/lua/src

@@ -1 +1 @@
-Subproject commit 1ab3208a1fceb12fca8f24ba57d6e13c5bff15e3
+Subproject commit 0897c0a4289ef3a8d45761266124613f364bef60

lib/settings

@@ -1 +1 @@
-Subproject commit 4a0a1e599377cdcdc91b0fbbefc312936b48730c
+Subproject commit c58874c1aa5d0619df2c975bcb87433941b46920

lib/sol2

@@ -1 +0,0 @@
-Subproject commit 2b0d2fe8ba0074e16b499940c4f3126b9c7d3471

mocks/include/mocks/BaseApplication.hpp

@@ -3,7 +3,6 @@
 #include "common/Args.hpp"
 #include "mocks/DisabledStreamerMode.hpp"
 #include "mocks/EmptyApplication.hpp"
-#include "mocks/TwitchUsers.hpp"
 #include "providers/bttv/BttvLiveUpdates.hpp"
 #include "singletons/Fonts.hpp"
 #include "singletons/Settings.hpp"
@@ -56,11 +55,6 @@ public:
         return &this->fonts;
     }
 
-    ITwitchUsers *getTwitchUsers() override
-    {
-        return &this->twitchUsers;
-    }
-
     BttvLiveUpdates *getBttvLiveUpdates() override
     {
         return nullptr;
@@ -77,7 +71,6 @@ public:
     DisabledStreamerMode streamerMode;
     Theme theme;
     Fonts fonts;
-    TwitchUsers twitchUsers;
 };
 
 }  // namespace chatterino::mock

mocks/include/mocks/ChatterinoBadges.hpp

@@ -2,8 +2,6 @@
 
 #include "providers/chatterino/ChatterinoBadges.hpp"
 
-#include <unordered_map>
-
 namespace chatterino::mock {
 
 class ChatterinoBadges : public IChatterinoBadges
@@ -11,21 +9,9 @@ class ChatterinoBadges : public IChatterinoBadges
 public:
     std::optional<EmotePtr> getBadge(const UserId &id) override
     {
-        auto it = this->users.find(id);
-        if (it != this->users.end())
-        {
-            return it->second;
-        }
+        (void)id;
         return std::nullopt;
     }
-
-    void setBadge(UserId id, EmotePtr emote)
-    {
-        this->users.emplace(std::move(id), std::move(emote));
-    }
-
-private:
-    std::unordered_map<UserId, EmotePtr> users;
 };
 
 }  // namespace chatterino::mock

mocks/include/mocks/Helix.hpp

@@ -350,7 +350,7 @@ public:
     // contains a comma
     MOCK_METHOD(
         void, getChatters,
-        (QString broadcasterID, QString moderatorID, size_t maxChattersToFetch,
+        (QString broadcasterID, QString moderatorID, int maxChattersToFetch,
          ResultCallback<HelixChatters> successCallback,
          (FailureCallback<HelixGetChattersError, QString> failureCallback)),
         (override));  // getChatters

mocks/include/mocks/TwitchIrcServer.hpp

@@ -7,11 +7,8 @@
 #include "providers/seventv/eventapi/Dispatch.hpp"
 #include "providers/seventv/eventapi/Message.hpp"
 #include "providers/seventv/SeventvEmotes.hpp"
-#include "providers/twitch/TwitchChannel.hpp"
 #include "providers/twitch/TwitchIrcServer.hpp"
 
-#include <unordered_map>
-
 namespace chatterino::mock {
 
 class MockTwitchIrcServer : public ITwitchIrcServer
@@ -70,30 +67,7 @@ public:
     std::shared_ptr<Channel> getChannelOrEmptyByID(
         const QString &channelID) override
     {
-        // XXX: this is the same as in TwitchIrcServer::getChannelOrEmptyByID
-        for (const auto &[name, weakChannel] : this->mockChannels)
-        {
-            auto channel = weakChannel.lock();
-            if (!channel)
-            {
-                continue;
-            }
-
-            auto twitchChannel =
-                std::dynamic_pointer_cast<TwitchChannel>(channel);
-            if (!twitchChannel)
-            {
-                continue;
-            }
-
-            if (twitchChannel->roomId() == channelID &&
-                twitchChannel->getName().count(':') < 2)
-            {
-                return channel;
-            }
-        }
-
-        return Channel::getEmpty();
+        return {};
     }
 
     void dropSeventvChannel(const QString &userID,
@@ -149,8 +123,6 @@ public:
     ChannelPtr liveChannel;
     ChannelPtr automodChannel;
     QString lastUserThatWhisperedMe{"forsen"};
-
-    std::unordered_map<QString, std::weak_ptr<Channel>> mockChannels;
 };
 
 }  // namespace chatterino::mock

mocks/include/mocks/TwitchUsers.hpp

@@ -1,24 +0,0 @@
-#pragma once
-
-#include "providers/twitch/TwitchUser.hpp"
-#include "providers/twitch/TwitchUsers.hpp"
-
-namespace chatterino::mock {
-
-class TwitchUsers : public ITwitchUsers
-{
-public:
-    TwitchUsers() = default;
-
-    std::shared_ptr<TwitchUser> resolveID(const UserId &id)
-    {
-        TwitchUser u = {
-            .id = id.string,
-            .name = {},
-            .displayName = {},
-        };
-        return std::make_shared<TwitchUser>(u);
-    }
-};
-
-}  // namespace chatterino::mock

mocks/include/mocks/UserData.hpp

@@ -2,8 +2,6 @@
 
 #include "controllers/userdata/UserDataController.hpp"
 
-#include <unordered_map>
-
 namespace chatterino::mock {
 
 class UserDataController : public IUserDataController
@@ -15,11 +13,6 @@ public:
     // If the user does not have any extra data, return none
     std::optional<UserData> getUser(const QString &userID) const override
     {
-        auto it = this->userMap.find(userID);
-        if (it != this->userMap.end())
-        {
-            return it->second;
-        }
         return std::nullopt;
     }
 
@@ -27,21 +20,8 @@ public:
     void setUserColor(const QString &userID,
                       const QString &colorString) override
     {
-        auto it = this->userMap.find(userID);
-        if (it != this->userMap.end())
-        {
-            it->second.color = QColor(colorString);
-        }
-        else
-        {
-            this->userMap.emplace(userID, UserData{
-                                              .color = QColor(colorString),
-                                          });
-        }
+        // do nothing
     }
-
-private:
-    std::unordered_map<QString, UserData> userMap;
 };
 
 }  // namespace chatterino::mock

resources/contributors.txt

@@ -35,7 +35,7 @@ Confuseh | https://github.com/Confuseh |
 ch-ems | https://github.com/ch-ems |
 Bur0k | https://github.com/Bur0k |
 nuuls | https://github.com/nuuls |
-Chronophylos | https://github.com/Chronophylos |
+Chronophylos | https://github.com/Chronophylos | 
 Ckath | https://github.com/Ckath |
 matijakevic | https://github.com/matijakevic |
 nforro | https://github.com/nforro |
@@ -79,7 +79,6 @@ KleberPF | https://github.com/KleberPF |
 nealxm | https://github.com/nealxm | :/avatars/nealxm.png
 Niller2005 | https://github.com/Niller2005 | :/avatars/niller2005.png
 JakeRYW | https://github.com/JakeRYW | :/avatars/jakeryw.png
-maliByatzes | https://github.com/maliByatzes | :/avatars/maliByatzes.png
 
 # If you are a contributor add yourself above this line
 

scripts/make_luals_meta.py

@@ -1,4 +1,3 @@
-#!/usr/bin/env python3
 """
 This script generates docs/plugin-meta.lua. It accepts no arguments
 
@@ -197,7 +196,7 @@ def write_func(path: Path, line: int, comments: list[str], out: TextIOWrapper):
     if not comments[0].startswith("@"):
         out.write(f"--- {comments[0]}\n---\n")
         comments = comments[1:]
-    params: list[str] = []
+    params = []
     for comment in comments[:-1]:
         if not comment.startswith("@lua"):
             panic(path, line, f"Invalid function specification - got '{comment}'")
@@ -210,7 +209,7 @@ def write_func(path: Path, line: int, comments: list[str], out: TextIOWrapper):
         panic(path, line, f"Invalid function exposure - got '{comments[-1]}'")
     name = comments[-1].split(" ", 1)[1]
     printmsg(path, line, f"function {name}")
-    lua_params = ", ".join(p.removesuffix("?") for p in params)
+    lua_params = ", ".join(params)
     out.write(f"function {name}({lua_params}) end\n\n")
 
 
@@ -243,19 +242,17 @@ def read_file(path: Path, out: TextIOWrapper):
                 )
             name = header[0].split(" ", 1)[1]
             printmsg(path, reader.line_no(), f"enum {name}")
+            out.write(f"---@alias {name} integer\n")
             if header_comment:
                 out.write(f"--- {header_comment}\n")
-            out.write(f"---@enum {name}\n")
-            out.write(f"{name} = {{\n")
+            out.write("---@type { ")
             out.write(
-                "\n".join(
-                    [
-                        f"    {variant} = {{}}, ---@type {name}.{variant}"
-                        for variant in reader.read_enum_variants()
-                    ]
+                ", ".join(
+                    [f"{variant}: {name}" for variant in reader.read_enum_variants()]
                 )
             )
-            out.write("\n}\n\n")
+            out.write(" }\n")
+            out.write(f"{name} = {{}}\n\n")
             continue
 
         # class

src/CMakeLists.txt

@@ -2,7 +2,6 @@ set(LIBRARY_PROJECT "${PROJECT_NAME}-lib")
 set(VERSION_PROJECT "${LIBRARY_PROJECT}-version")
 set(EXECUTABLE_PROJECT "${PROJECT_NAME}")
 add_compile_definitions(QT_DISABLE_DEPRECATED_BEFORE=0x050F00)
-add_compile_definitions(QT_WARN_DEPRECATED_UP_TO=0x050F00)
 
 # registers the native messageing host
 option(CHATTERINO_DEBUG_NATIVE_MESSAGES "Debug native messages" OFF)
@@ -40,7 +39,6 @@ set(SOURCE_FILES
         common/WindowDescriptors.cpp
         common/WindowDescriptors.hpp
 
-        common/enums/MessageContext.hpp
         common/enums/MessageOverflow.hpp
 
         common/network/NetworkCommon.cpp
@@ -227,28 +225,24 @@ set(SOURCE_FILES
         controllers/pings/MutedChannelModel.cpp
         controllers/pings/MutedChannelModel.hpp
 
-
         controllers/plugins/api/ChannelRef.cpp
         controllers/plugins/api/ChannelRef.hpp
-        controllers/plugins/api/EventType.hpp
+        controllers/plugins/api/IOWrapper.cpp
+        controllers/plugins/api/IOWrapper.hpp
         controllers/plugins/api/HTTPRequest.cpp
         controllers/plugins/api/HTTPRequest.hpp
         controllers/plugins/api/HTTPResponse.cpp
         controllers/plugins/api/HTTPResponse.hpp
-        controllers/plugins/api/IOWrapper.cpp
-        controllers/plugins/api/IOWrapper.hpp
         controllers/plugins/LuaAPI.cpp
         controllers/plugins/LuaAPI.hpp
-        controllers/plugins/LuaUtilities.cpp
-        controllers/plugins/LuaUtilities.hpp
-        controllers/plugins/PluginController.cpp
-        controllers/plugins/PluginController.hpp
-        controllers/plugins/Plugin.cpp
-        controllers/plugins/Plugin.hpp
         controllers/plugins/PluginPermission.cpp
         controllers/plugins/PluginPermission.hpp
-        controllers/plugins/SolTypes.cpp
-        controllers/plugins/SolTypes.hpp
+        controllers/plugins/Plugin.cpp
+        controllers/plugins/Plugin.hpp
+        controllers/plugins/PluginController.hpp
+        controllers/plugins/PluginController.cpp
+        controllers/plugins/LuaUtilities.cpp
+        controllers/plugins/LuaUtilities.hpp
 
         controllers/sound/ISoundController.hpp
         controllers/sound/MiniaudioBackend.cpp
@@ -283,9 +277,6 @@ set(SOURCE_FILES
         messages/MessageElement.cpp
         messages/MessageElement.hpp
         messages/MessageFlag.hpp
-        messages/MessageSimilarity.cpp
-        messages/MessageSimilarity.hpp
-        messages/MessageSink.hpp
         messages/MessageThread.cpp
         messages/MessageThread.hpp
 
@@ -418,8 +409,6 @@ set(SOURCE_FILES
         providers/twitch/TwitchEmotes.hpp
         providers/twitch/TwitchHelpers.cpp
         providers/twitch/TwitchHelpers.hpp
-        providers/twitch/TwitchIrc.cpp
-        providers/twitch/TwitchIrc.hpp
         providers/twitch/TwitchIrcServer.cpp
         providers/twitch/TwitchIrcServer.hpp
         providers/twitch/TwitchUser.cpp
@@ -508,8 +497,6 @@ set(SOURCE_FILES
         util/InitUpdateButton.hpp
         util/IpcQueue.cpp
         util/IpcQueue.hpp
-        util/IrcHelpers.cpp
-        util/IrcHelpers.hpp
         util/LayoutHelper.cpp
         util/LayoutHelper.hpp
         util/LoadPixmap.cpp
@@ -531,8 +518,6 @@ set(SOURCE_FILES
         util/Twitch.hpp
         util/TypeName.hpp
         util/Variant.hpp
-        util/VectorMessageSink.cpp
-        util/VectorMessageSink.hpp
         util/WidgetHelpers.cpp
         util/WidgetHelpers.hpp
         util/WindowsHelper.cpp
@@ -802,7 +787,7 @@ target_link_libraries(${LIBRARY_PROJECT}
         $<$<BOOL:${WIN32}>:Wtsapi32>
         )
 if (CHATTERINO_PLUGINS)
-    target_link_libraries(${LIBRARY_PROJECT} PUBLIC lua sol2::sol2)
+    target_link_libraries(${LIBRARY_PROJECT} PUBLIC lua)
 endif()
 
 if (BUILD_WITH_QTKEYCHAIN)
@@ -984,7 +969,6 @@ target_compile_definitions(${LIBRARY_PROJECT} PUBLIC
     IRC_STATIC
     IRC_NAMESPACE=Communi
     $<$<BOOL:${WIN32}>:_WIN32_WINNT=0x0A00> # Windows 10
-    $<$<BOOL:${BUILD_TESTS}>:CHATTERINO_WITH_TESTS>
     )
 
 if (USE_SYSTEM_QTKEYCHAIN)

src/PrecompiledHeader.hpp

@@ -129,10 +129,6 @@
 #    include <unordered_set>
 #    include <vector>
 
-#    ifdef CHATTERINO_HAVE_PLUGINS
-#        include <sol/sol.hpp>
-#    endif
-
 #    ifndef UNUSED
 #        define UNUSED(x) (void)(x)
 #    endif

src/common/Channel.cpp

@@ -3,9 +3,7 @@
 #include "Application.hpp"
 #include "messages/Message.hpp"
 #include "messages/MessageBuilder.hpp"
-#include "messages/MessageSimilarity.hpp"
 #include "providers/twitch/IrcMessageHandler.hpp"
-#include "providers/twitch/TwitchIrcServer.hpp"
 #include "singletons/Emotes.hpp"
 #include "singletons/Logging.hpp"
 #include "singletons/Settings.hpp"
@@ -123,10 +121,10 @@ void Channel::addSystemMessage(const QString &contents)
     this->addMessage(msg, MessageContext::Original);
 }
 
-void Channel::addOrReplaceTimeout(MessagePtr message, QTime now)
+void Channel::addOrReplaceTimeout(MessagePtr message)
 {
     addOrReplaceChannelTimeout(
-        this->getMessageSnapshot(), std::move(message), now,
+        this->getMessageSnapshot(), std::move(message), QTime::currentTime(),
         [this](auto /*idx*/, auto msg, auto replacement) {
             this->replaceMessage(msg, replacement);
         },
@@ -255,33 +253,21 @@ void Channel::fillInMissingMessages(const std::vector<MessagePtr> &messages)
     }
 }
 
-void Channel::replaceMessage(const MessagePtr &message,
-                             const MessagePtr &replacement)
+void Channel::replaceMessage(MessagePtr message, MessagePtr replacement)
 {
     int index = this->messages_.replaceItem(message, replacement);
 
     if (index >= 0)
     {
-        this->messageReplaced.invoke((size_t)index, message, replacement);
+        this->messageReplaced.invoke((size_t)index, replacement);
     }
 }
 
-void Channel::replaceMessage(size_t index, const MessagePtr &replacement)
+void Channel::replaceMessage(size_t index, MessagePtr replacement)
 {
-    MessagePtr prev;
-    if (this->messages_.replaceItem(index, replacement, &prev))
+    if (this->messages_.replaceItem(index, replacement))
     {
-        this->messageReplaced.invoke(index, prev, replacement);
-    }
-}
-
-void Channel::replaceMessage(size_t hint, const MessagePtr &message,
-                             const MessagePtr &replacement)
-{
-    auto index = this->messages_.replaceItem(hint, message, replacement);
-    if (index >= 0)
-    {
-        this->messageReplaced.invoke(hint, message, replacement);
+        this->messageReplaced.invoke(index, replacement);
     }
 }
 
@@ -301,15 +287,10 @@ void Channel::clearMessages()
 }
 
 MessagePtr Channel::findMessage(QString messageID)
-{
-    return this->findMessageByID(messageID);
-}
-
-MessagePtr Channel::findMessageByID(QStringView messageID)
 {
     MessagePtr res;
 
-    if (auto msg = this->messages_.rfind([messageID](const MessagePtr &msg) {
+    if (auto msg = this->messages_.rfind([&messageID](const MessagePtr &msg) {
             return msg->id == messageID;
         });
         msg)
@@ -320,19 +301,6 @@ MessagePtr Channel::findMessageByID(QStringView messageID)
     return res;
 }
 
-void Channel::applySimilarityFilters(const MessagePtr &message) const
-{
-    setSimilarityFlags(message, this->messages_.getSnapshot());
-}
-
-MessageSinkTraits Channel::sinkTraits() const
-{
-    return {
-        MessageSinkTrait::AddMentionsToGlobalChannel,
-        MessageSinkTrait::RequiresKnownChannelPointReward,
-    };
-}
-
 bool Channel::canSendMessage() const
 {
     return false;

src/common/Channel.hpp

@@ -1,10 +1,8 @@
 #pragma once
 
-#include "common/enums/MessageContext.hpp"
 #include "controllers/completion/TabCompletionModel.hpp"
 #include "messages/LimitedQueue.hpp"
 #include "messages/MessageFlag.hpp"
-#include "messages/MessageSink.hpp"
 
 #include <magic_enum/magic_enum.hpp>
 #include <pajlada/signals/signal.hpp>
@@ -28,7 +26,15 @@ enum class TimeoutStackStyle : int {
     Default = DontStackBeyondUserMessage,
 };
 
-class Channel : public std::enable_shared_from_this<Channel>, public MessageSink
+/// Context of the message being added to a channel
+enum class MessageContext {
+    /// This message is the original
+    Original,
+    /// This message is a repost of a message that has already been added in a channel
+    Repost,
+};
+
+class Channel : public std::enable_shared_from_this<Channel>
 {
 public:
     // This is for Lua. See scripts/make_luals_meta.py
@@ -49,7 +55,7 @@ public:
     };
 
     explicit Channel(const QString &name, Type type);
-    ~Channel() override;
+    virtual ~Channel();
 
     // SIGNALS
     pajlada::Signals::Signal<const QString &, const QString &, bool &>
@@ -60,9 +66,7 @@ public:
     pajlada::Signals::Signal<MessagePtr &, std::optional<MessageFlags>>
         messageAppended;
     pajlada::Signals::Signal<std::vector<MessagePtr> &> messagesAddedAtStart;
-    /// (index, prev-message, replacement)
-    pajlada::Signals::Signal<size_t, const MessagePtr &, const MessagePtr &>
-        messageReplaced;
+    pajlada::Signals::Signal<size_t, MessagePtr &> messageReplaced;
     /// Invoked when some number of messages were filled in using time received
     pajlada::Signals::Signal<const std::vector<MessagePtr> &> filledInMessages;
     pajlada::Signals::NoArgSignal destroyed;
@@ -81,9 +85,8 @@ public:
     // overridingFlags can be filled in with flags that should be used instead
     // of the message's flags. This is useful in case a flag is specific to a
     // type of split
-    void addMessage(
-        MessagePtr message, MessageContext context,
-        std::optional<MessageFlags> overridingFlags = std::nullopt) final;
+    void addMessage(MessagePtr message, MessageContext context,
+                    std::optional<MessageFlags> overridingFlags = std::nullopt);
     void addMessagesAtStart(const std::vector<MessagePtr> &messages_);
 
     void addSystemMessage(const QString &contents);
@@ -91,28 +94,19 @@ public:
     /// Inserts the given messages in order by Message::serverReceivedTime.
     void fillInMissingMessages(const std::vector<MessagePtr> &messages);
 
-    void addOrReplaceTimeout(MessagePtr message, QTime now) final;
-    void disableAllMessages() final;
-    void replaceMessage(const MessagePtr &message,
-                        const MessagePtr &replacement);
-    void replaceMessage(size_t index, const MessagePtr &replacement);
-    void replaceMessage(size_t hint, const MessagePtr &message,
-                        const MessagePtr &replacement);
+    void addOrReplaceTimeout(MessagePtr message);
+    void disableAllMessages();
+    void replaceMessage(MessagePtr message, MessagePtr replacement);
+    void replaceMessage(size_t index, MessagePtr replacement);
     void deleteMessage(QString messageID);
 
     /// Removes all messages from this channel and invokes #messagesCleared
     void clearMessages();
 
-    [[deprecated("Use findMessageByID instead")]] MessagePtr findMessage(
-        QString messageID);
-    MessagePtr findMessageByID(QStringView messageID) final;
+    MessagePtr findMessage(QString messageID);
 
     bool hasMessages() const;
 
-    void applySimilarityFilters(const MessagePtr &message) const final;
-
-    MessageSinkTraits sinkTraits() const final;
-
     // CHANNEL INFO
     virtual bool canSendMessage() const;
     virtual bool isWritable() const;  // whether split input will be usable
@@ -171,3 +165,30 @@ private:
 };
 
 }  // namespace chatterino
+
+template <>
+constexpr magic_enum::customize::customize_t
+    magic_enum::customize::enum_name<chatterino::Channel::Type>(
+        chatterino::Channel::Type value) noexcept
+{
+    using Type = chatterino::Channel::Type;
+    switch (value)
+    {
+        case Type::Twitch:
+            return "twitch";
+        case Type::TwitchWhispers:
+            return "whispers";
+        case Type::TwitchWatching:
+            return "watching";
+        case Type::TwitchMentions:
+            return "mentions";
+        case Type::TwitchLive:
+            return "live";
+        case Type::TwitchAutomod:
+            return "automod";
+        case Type::Misc:
+            return "misc";
+        default:
+            return default_tag;
+    }
+}

src/common/ChatterinoSetting.hpp

@@ -3,7 +3,6 @@
 #include "util/QMagicEnum.hpp"
 
 #include <pajlada/settings.hpp>
-#include <QSize>
 #include <QString>
 
 namespace chatterino {
@@ -56,7 +55,6 @@ using DoubleSetting = ChatterinoSetting<double>;
 using IntSetting = ChatterinoSetting<int>;
 using StringSetting = ChatterinoSetting<std::string>;
 using QStringSetting = ChatterinoSetting<QString>;
-using QSizeSetting = ChatterinoSetting<QSize>;
 
 template <typename Enum>
 class EnumSetting
@@ -73,6 +71,7 @@ public:
         _registerSetting(this->getData());
     }
 
+    template <typename T2>
     EnumSetting<Enum> &operator=(Enum newValue)
     {
         this->setValue(Underlying(newValue));

src/common/ProviderId.hpp

@@ -2,8 +2,6 @@
 
 namespace chatterino {
 
-enum class ProviderId {  // NOLINT(performance-enum-size)
-    Twitch,
-};
+enum class ProviderId { Twitch, Irc };
 //
 }  // namespace chatterino

src/common/SignalVector.hpp

@@ -87,8 +87,7 @@ public:
             }
             else
             {
-                assert(index >= 0 &&
-                       index <= static_cast<int>(this->items_.size()));
+                assert(index >= 0 && index <= this->items_.size());
             }
 
             this->items_.insert(this->items_.begin() + index, item);
@@ -117,7 +116,7 @@ public:
     void removeAt(int index, void *caller = nullptr)
     {
         assertInGuiThread();
-        assert(index >= 0 && index < static_cast<int>(this->items_.size()));
+        assert(index >= 0 && index < int(this->items_.size()));
 
         T item = this->items_[index];
         this->items_.erase(this->items_.begin() + index);
@@ -133,14 +132,13 @@ public:
     {
         assertInGuiThread();
 
-        for (size_t index = 0; index < this->items_.size(); ++index)
+        for (int index = 0; index < this->items_.size(); ++index)
         {
             T item = this->items_[index];
             if (matcher(item))
             {
                 this->items_.erase(this->items_.begin() + index);
-                SignalVectorItemEvent<T> args{item, static_cast<int>(index),
-                                              caller};
+                SignalVectorItemEvent<T> args{item, index, caller};
                 this->itemRemoved.invoke(args);
                 this->itemsChanged_();
                 return true;

src/common/SignalVectorModel.hpp

@@ -43,7 +43,7 @@ public:
             }
             // get row index
             int index = this->getModelIndexFromVectorIndex(args.index);
-            assert(index >= 0 && index <= static_cast<int>(this->rows_.size()));
+            assert(index >= 0 && index <= this->rows_.size());
 
             // get row items
             std::vector<QStandardItem *> row = this->createRow();
@@ -75,7 +75,7 @@ public:
             }
 
             int row = this->getModelIndexFromVectorIndex(args.index);
-            assert(row >= 0 && row <= static_cast<int>(this->rows_.size()));
+            assert(row >= 0 && row <= this->rows_.size());
 
             // remove row
             std::vector<QStandardItem *> items = this->rows_[row].items;
@@ -130,8 +130,7 @@ public:
     {
         int row = index.row();
         int column = index.column();
-        if (row < 0 || column < 0 ||
-            row >= static_cast<int>(this->rows_.size()) ||
+        if (row < 0 || column < 0 || row >= this->rows_.size() ||
             column >= this->columnCount_)
         {
             return QVariant();
@@ -145,8 +144,7 @@ public:
     {
         int row = index.row();
         int column = index.column();
-        if (row < 0 || column < 0 ||
-            row >= static_cast<int>(this->rows_.size()) ||
+        if (row < 0 || column < 0 || row >= this->rows_.size() ||
             column >= this->columnCount_)
         {
             return false;
@@ -154,7 +152,7 @@ public:
 
         Row &rowItem = this->rows_[row];
 
-        assert(this->columnCount_ == static_cast<int>(rowItem.items.size()));
+        assert(this->columnCount_ == rowItem.items.size());
 
         auto &cell = rowItem.items[column];
 
@@ -169,7 +167,7 @@ public:
             int vecRow = this->getVectorIndexFromModelIndex(row);
             // TODO: This is only a safety-thing for when we modify data that's being modified right now.
             // It should not be necessary, but it would require some rethinking about this surrounding logic
-            if (vecRow >= static_cast<int>(this->vector_->readOnly()->size()))
+            if (vecRow >= this->vector_->readOnly()->size())
             {
                 return false;
             }
@@ -226,19 +224,18 @@ public:
     {
         int row = index.row(), column = index.column();
 
-        if (row < 0 || column < 0 ||
-            row >= static_cast<int>(this->rows_.size()) ||
+        if (row < 0 || column < 0 || row >= this->rows_.size() ||
             column >= this->columnCount_)
         {
             return Qt::NoItemFlags;
         }
 
-        assert(row >= 0 && row < static_cast<int>(this->rows_.size()) &&
-               column >= 0 && column < this->columnCount_);
+        assert(row >= 0 && row < this->rows_.size() && column >= 0 &&
+               column < this->columnCount_);
 
         const auto &rowItem = this->rows_[row];
 
-        assert(this->columnCount_ == static_cast<int>(rowItem.items.size()));
+        assert(this->columnCount_ == rowItem.items.size());
 
         return rowItem.items[column]->flags();
     }
@@ -270,8 +267,7 @@ public:
             return false;
         }
 
-        assert(sourceRow >= 0 &&
-               sourceRow < static_cast<int>(this->rows_.size()));
+        assert(sourceRow >= 0 && sourceRow < this->rows_.size());
 
         int signalVectorRow = this->getVectorIndexFromModelIndex(sourceRow);
         this->beginMoveRows(sourceParent, sourceRow, sourceRow,
@@ -298,7 +294,7 @@ public:
             return false;
         }
 
-        assert(row >= 0 && row < static_cast<int>(this->rows_.size()));
+        assert(row >= 0 && row < this->rows_.size());
 
         int signalVectorRow = this->getVectorIndexFromModelIndex(row);
         this->vector_->removeAt(signalVectorRow);
@@ -341,10 +337,8 @@ public:
             int from = data->data("chatterino_row_id").toInt();
             int to = parent.row();
 
-            auto vectorFrom =
-                static_cast<size_t>(this->getVectorIndexFromModelIndex(from));
-            auto vectorTo =
-                static_cast<size_t>(this->getVectorIndexFromModelIndex(to));
+            int vectorFrom = this->getVectorIndexFromModelIndex(from);
+            int vectorTo = this->getVectorIndexFromModelIndex(to);
 
             if (vectorFrom < 0 || vectorFrom > this->vector_->raw().size() ||
                 vectorTo < 0 || vectorTo > this->vector_->raw().size())
@@ -408,7 +402,7 @@ protected:
 
     void insertCustomRow(std::vector<QStandardItem *> row, int index)
     {
-        assert(index >= 0 && index <= static_cast<int>(this->rows_.size()));
+        assert(index >= 0 && index <= this->rows_.size());
 
         this->beginInsertRows(QModelIndex(), index, index);
         this->rows_.insert(this->rows_.begin() + index,
@@ -418,7 +412,7 @@ protected:
 
     void removeCustomRow(int index)
     {
-        assert(index >= 0 && index <= static_cast<int>(this->rows_.size()));
+        assert(index >= 0 && index <= this->rows_.size());
         assert(this->rows_[index].isCustomRow);
 
         this->beginRemoveRows(QModelIndex(), index, index);

src/common/enums/MessageContext.hpp

@@ -1,13 +0,0 @@
-#pragma once
-
-namespace chatterino {
-
-/// Context of the message being added to a channel
-enum class MessageContext {
-    /// This message is the original
-    Original,
-    /// This message is a repost of a message that has already been added in a channel
-    Repost,
-};
-
-}  // namespace chatterino

src/common/network/NetworkCommon.hpp

@@ -16,7 +16,7 @@ using NetworkErrorCallback = std::function<void(NetworkResult)>;
 using NetworkFinallyCallback = std::function<void()>;
 
 /**
- * @exposeenum c2.HTTPMethod
+ * @exposeenum HTTPMethod
  */
 enum class NetworkRequestType {
     Get,

src/controllers/commands/builtin/chatterino/Debugging.cpp

@@ -79,7 +79,6 @@ QString listEnvironmentVariables(const CommandContext &ctx)
     QStringList debugMessages{
         "recentMessagesApiUrl: " + env.recentMessagesApiUrl,
         "linkResolverUrl: " + env.linkResolverUrl,
-        "proxyUrl: " + env.proxyUrl.value_or("N/A"),
         "twitchServerHost: " + env.twitchServerHost,
         "twitchServerPort: " + QString::number(env.twitchServerPort),
         "twitchServerSecure: " + QString::number(env.twitchServerSecure),

src/controllers/completion/sources/UnifiedSource.cpp

@@ -37,7 +37,7 @@ void UnifiedSource::addToListModel(GenericListModel &model,
         source->addToListModel(model, maxCount - used);
         // Calculate how many items have been added so far
         used = model.rowCount() - startingSize;
-        if (used >= static_cast<int>(maxCount))
+        if (used >= maxCount)
         {
             // Used up all of limit
             break;
@@ -58,15 +58,15 @@ void UnifiedSource::addToStringList(QStringList &list, size_t maxCount,
     }
 
     // Make sure to only add maxCount elements in total.
-    auto startingSize = list.size();
-    QStringList::size_type used = 0;
+    int startingSize = list.size();
+    int used = 0;
 
     for (const auto &source : this->sources_)
     {
         source->addToStringList(list, maxCount - used, isFirstWord);
         // Calculate how many items have been added so far
         used = list.size() - startingSize;
-        if (used >= static_cast<QStringList::size_type>(maxCount))
+        if (used >= maxCount)
         {
             // Used up all of limit
             break;

src/controllers/ignores/IgnoreController.cpp

@@ -1,134 +1,12 @@
 #include "controllers/ignores/IgnoreController.hpp"
 
 #include "Application.hpp"
-#include "common/Literals.hpp"
 #include "common/QLogging.hpp"
 #include "controllers/accounts/AccountController.hpp"
 #include "controllers/ignores/IgnorePhrase.hpp"
 #include "providers/twitch/TwitchAccount.hpp"
-#include "providers/twitch/TwitchIrc.hpp"
 #include "singletons/Settings.hpp"
 
-namespace {
-
-using namespace chatterino::literals;
-
-/**
-  * Computes (only) the replacement of @a match in @a source.
-  * The parts before and after the match in @a source are ignored.
-  *
-  * Occurrences of \b{\\1}, \b{\\2}, ..., in @a replacement are replaced
-  * with the string captured by the corresponding capturing group.
-  * This function should only be used if the regex contains capturing groups.
-  * 
-  * Since Qt doesn't provide a way of replacing a single match with some replacement
-  * while supporting both capturing groups and lookahead/-behind in the regex,
-  * this is included here. It's essentially the implementation of 
-  * QString::replace(const QRegularExpression &, const QString &).
-  * @see https://github.com/qt/qtbase/blob/97bb0ecfe628b5bb78e798563212adf02129c6f6/src/corelib/text/qstring.cpp#L4594-L4703
-  */
-QString makeRegexReplacement(QStringView source,
-                             const QRegularExpression &regex,
-                             const QRegularExpressionMatch &match,
-                             const QString &replacement)
-{
-    using SizeType = QString::size_type;
-    struct QStringCapture {
-        SizeType pos;
-        SizeType len;
-        int captureNumber;
-    };
-
-    qsizetype numCaptures = regex.captureCount();
-
-    // 1. build the backreferences list, holding where the backreferences
-    //    are in the replacement string
-    QVarLengthArray<QStringCapture> backReferences;
-
-    SizeType replacementLength = replacement.size();
-    for (SizeType i = 0; i < replacementLength - 1; i++)
-    {
-        if (replacement[i] != u'\\')
-        {
-            continue;
-        }
-
-        int no = replacement[i + 1].digitValue();
-        if (no <= 0 || no > numCaptures)
-        {
-            continue;
-        }
-
-        QStringCapture backReference{.pos = i, .len = 2};
-
-        if (i < replacementLength - 2)
-        {
-            int secondDigit = replacement[i + 2].digitValue();
-            if (secondDigit != -1 && ((no * 10) + secondDigit) <= numCaptures)
-            {
-                no = (no * 10) + secondDigit;
-                ++backReference.len;
-            }
-        }
-
-        backReference.captureNumber = no;
-        backReferences.append(backReference);
-    }
-
-    // 2. iterate on the matches.
-    //    For every match, copy the replacement string in chunks
-    //    with the proper replacements for the backreferences
-
-    // length of the new string, with all the replacements
-    SizeType newLength = 0;
-    QVarLengthArray<QStringView> chunks;
-    QStringView replacementView{replacement};
-
-    // Initially: empty, as we only care about the replacement
-    SizeType len = 0;
-    SizeType lastEnd = 0;
-    for (const QStringCapture &backReference : std::as_const(backReferences))
-    {
-        // part of "replacement" before the backreference
-        len = backReference.pos - lastEnd;
-        if (len > 0)
-        {
-            chunks << replacementView.mid(lastEnd, len);
-            newLength += len;
-        }
-
-        // backreference itself
-        len = match.capturedLength(backReference.captureNumber);
-        if (len > 0)
-        {
-            chunks << source.mid(
-                match.capturedStart(backReference.captureNumber), len);
-            newLength += len;
-        }
-
-        lastEnd = backReference.pos + backReference.len;
-    }
-
-    // add the last part of the replacement string
-    len = replacementView.size() - lastEnd;
-    if (len > 0)
-    {
-        chunks << replacementView.mid(lastEnd, len);
-        newLength += len;
-    }
-
-    // 3. assemble the chunks together
-    QString dst;
-    dst.reserve(newLength);
-    for (const QStringView &chunk : std::as_const(chunks))
-    {
-        dst += chunk;
-    }
-    return dst;
-}
-
-}  // namespace
-
 namespace chatterino {
 
 bool isIgnoredMessage(IgnoredMessageParameters &&params)
@@ -187,187 +65,4 @@ bool isIgnoredMessage(IgnoredMessageParameters &&params)
     return false;
 }
 
-void processIgnorePhrases(const std::vector<IgnorePhrase> &phrases,
-                          QString &content,
-                          std::vector<TwitchEmoteOccurrence> &twitchEmotes)
-{
-    using SizeType = QString::size_type;
-
-    auto removeEmotesInRange = [&twitchEmotes](SizeType pos, SizeType len) {
-        // all emotes outside the range come before `it`
-        // all emotes in the range start at `it`
-        auto it = std::partition(
-            twitchEmotes.begin(), twitchEmotes.end(),
-            [pos, len](const auto &item) {
-                // returns true for emotes outside the range
-                return !((item.start >= pos) && item.start < (pos + len));
-            });
-        std::vector<TwitchEmoteOccurrence> emotesInRange(it,
-                                                         twitchEmotes.end());
-        twitchEmotes.erase(it, twitchEmotes.end());
-        return emotesInRange;
-    };
-
-    auto shiftIndicesAfter = [&twitchEmotes](int pos, int by) {
-        for (auto &item : twitchEmotes)
-        {
-            auto &index = item.start;
-            if (index >= pos)
-            {
-                index += by;
-                item.end += by;
-            }
-        }
-    };
-
-    auto addReplEmotes = [&twitchEmotes](const IgnorePhrase &phrase,
-                                         const auto &midrepl,
-                                         SizeType startIndex) {
-        if (!phrase.containsEmote())
-        {
-            return;
-        }
-
-#if QT_VERSION >= QT_VERSION_CHECK(6, 0, 0)
-        auto words = midrepl.tokenize(u' ');
-#else
-        auto words = midrepl.split(' ');
-#endif
-        SizeType pos = 0;
-        for (const auto &word : words)
-        {
-            for (const auto &emote : phrase.getEmotes())
-            {
-                if (word == emote.first.string)
-                {
-                    if (emote.second == nullptr)
-                    {
-                        qCDebug(chatterinoTwitch)
-                            << "emote null" << emote.first.string;
-                    }
-                    twitchEmotes.push_back(TwitchEmoteOccurrence{
-                        static_cast<int>(startIndex + pos),
-                        static_cast<int>(startIndex + pos +
-                                         emote.first.string.length()),
-                        emote.second,
-                        emote.first,
-                    });
-                }
-            }
-            pos += word.length() + 1;
-        }
-    };
-
-    auto replaceMessageAt = [&](const IgnorePhrase &phrase, SizeType from,
-                                SizeType length, const QString &replacement) {
-        auto removedEmotes = removeEmotesInRange(from, length);
-        content.replace(from, length, replacement);
-        auto wordStart = from;
-        while (wordStart > 0)
-        {
-            if (content[wordStart - 1] == ' ')
-            {
-                break;
-            }
-            --wordStart;
-        }
-        auto wordEnd = from + replacement.length();
-        while (wordEnd < content.length())
-        {
-            if (content[wordEnd] == ' ')
-            {
-                break;
-            }
-            ++wordEnd;
-        }
-
-        shiftIndicesAfter(static_cast<int>(from + length),
-                          static_cast<int>(replacement.length() - length));
-
-        auto midExtendedRef =
-            QStringView{content}.mid(wordStart, wordEnd - wordStart);
-
-        for (auto &emote : removedEmotes)
-        {
-            if (emote.ptr == nullptr)
-            {
-                qCDebug(chatterinoTwitch)
-                    << "Invalid emote occurrence" << emote.name.string;
-                continue;
-            }
-            QRegularExpression emoteregex(
-                "\\b" + emote.name.string + "\\b",
-                QRegularExpression::UseUnicodePropertiesOption);
-#if QT_VERSION >= QT_VERSION_CHECK(6, 5, 0)
-            auto match = emoteregex.matchView(midExtendedRef);
-#else
-            auto match = emoteregex.match(midExtendedRef);
-#endif
-            if (match.hasMatch())
-            {
-                emote.start = static_cast<int>(from + match.capturedStart());
-                emote.end = static_cast<int>(from + match.capturedEnd());
-                twitchEmotes.push_back(std::move(emote));
-            }
-        }
-
-        addReplEmotes(phrase, midExtendedRef, wordStart);
-    };
-
-    for (const auto &phrase : phrases)
-    {
-        if (phrase.isBlock())
-        {
-            continue;
-        }
-        const auto &pattern = phrase.getPattern();
-        if (pattern.isEmpty())
-        {
-            continue;
-        }
-        if (phrase.isRegex())
-        {
-            const auto &regex = phrase.getRegex();
-            if (!regex.isValid())
-            {
-                continue;
-            }
-
-            QRegularExpressionMatch match;
-            size_t iterations = 0;
-            SizeType from = 0;
-            while ((from = content.indexOf(regex, from, &match)) != -1)
-            {
-                auto replacement = phrase.getReplace();
-                if (regex.captureCount() > 0)
-                {
-                    replacement = makeRegexReplacement(content, regex, match,
-                                                       replacement);
-                }
-
-                replaceMessageAt(phrase, from, match.capturedLength(),
-                                 replacement);
-                from += phrase.getReplace().length();
-                iterations++;
-                if (iterations >= 128)
-                {
-                    content = u"Too many replacements - check your ignores!"_s;
-                    return;
-                }
-            }
-
-            continue;
-        }
-
-        SizeType from = 0;
-        while ((from = content.indexOf(pattern, from,
-                                       phrase.caseSensitivity())) != -1)
-        {
-            replaceMessageAt(phrase, from, pattern.length(),
-                             phrase.getReplace());
-            from += phrase.getReplace().length();
-        }
-    }
-}
-
 }  // namespace chatterino

src/controllers/ignores/IgnoreController.hpp

@@ -2,13 +2,8 @@
 
 #include <QString>
 
-#include <vector>
-
 namespace chatterino {
 
-class IgnorePhrase;
-struct TwitchEmoteOccurrence;
-
 enum class ShowIgnoredUsersMessages { Never, IfModerator, IfBroadcaster };
 
 struct IgnoredMessageParameters {
@@ -21,17 +16,4 @@ struct IgnoredMessageParameters {
 
 bool isIgnoredMessage(IgnoredMessageParameters &&params);
 
-/// @brief Processes replacement ignore-phrases for a message
-///
-/// @param phrases A list of IgnorePhrases to process. Block phrases as well as
-/// 	           invalid phrases are ignored.
-/// @param content The message text. This gets altered by replacements.
-/// @param twitchEmotes A list of emotes present in the message. Occurrences
-///                     that have been removed from the message will also be
-///                     removed in this list. Similarly, if new emotes are added
-///                     from a replacement, this list gets updated as well.
-void processIgnorePhrases(const std::vector<IgnorePhrase> &phrases,
-                          QString &content,
-                          std::vector<TwitchEmoteOccurrence> &twitchEmotes);
-
 }  // namespace chatterino

src/controllers/ignores/IgnorePhrase.cpp

@@ -95,11 +95,11 @@ bool IgnorePhrase::containsEmote() const
 {
     if (!this->emotesChecked_)
     {
-        auto accemotes =
-            getApp()->getAccounts()->twitch.getCurrent()->accessEmotes();
-        if (*accemotes)
+        const auto &accvec = getApp()->getAccounts()->twitch.accounts;
+        for (const auto &acc : accvec)
         {
-            for (const auto &emote : **accemotes)
+            const auto &accemotes = *acc->accessEmotes();
+            for (const auto &emote : *accemotes)
             {
                 if (this->replace_.contains(emote.first.string,
                                             Qt::CaseSensitive))

src/controllers/plugins/LuaAPI.cpp

@@ -3,45 +3,34 @@
 
 #    include "Application.hpp"
 #    include "common/QLogging.hpp"
+#    include "controllers/commands/CommandController.hpp"
 #    include "controllers/plugins/LuaUtilities.hpp"
 #    include "controllers/plugins/PluginController.hpp"
-#    include "controllers/plugins/SolTypes.hpp"  // for lua operations on QString{,List} for CompletionList
+#    include "messages/MessageBuilder.hpp"
+#    include "providers/twitch/TwitchIrcServer.hpp"
 
+extern "C" {
 #    include <lauxlib.h>
 #    include <lua.h>
 #    include <lualib.h>
+}
 #    include <QFileInfo>
-#    include <QList>
 #    include <QLoggingCategory>
 #    include <QTextCodec>
 #    include <QUrl>
-#    include <sol/forward.hpp>
-#    include <sol/protected_function_result.hpp>
-#    include <sol/reference.hpp>
-#    include <sol/stack.hpp>
-#    include <sol/state_view.hpp>
-#    include <sol/types.hpp>
-#    include <sol/variadic_args.hpp>
-#    include <sol/variadic_results.hpp>
-
-#    include <stdexcept>
-#    include <string>
-#    include <utility>
 
 namespace {
 using namespace chatterino;
 
-void logHelper(lua_State *L, Plugin *pl, QDebug stream,
-               const sol::variadic_args &args)
+void logHelper(lua_State *L, Plugin *pl, QDebug stream, int argc)
 {
     stream.noquote();
     stream << "[" + pl->id + ":" + pl->meta.name + "]";
-    for (const auto &arg : args)
+    for (int i = 1; i <= argc; i++)
     {
-        stream << lua::toString(L, arg.stack_index());
-        // Remove this from our stack
-        lua_pop(L, 1);
+        stream << lua::toString(L, i);
     }
+    lua_pop(L, argc);
 }
 
 QDebug qdebugStreamForLogLevel(lua::api::LogLevel lvl)
@@ -74,92 +63,195 @@ QDebug qdebugStreamForLogLevel(lua::api::LogLevel lvl)
 // luaL_error is a c-style vararg function, this makes clang-tidy not dislike it so much
 namespace chatterino::lua::api {
 
-CompletionList::CompletionList(const sol::table &table)
-    : values(table.get<QStringList>("values"))
-    , hideOthers(table["hide_others"])
+int c2_register_command(lua_State *L)
 {
-}
-
-sol::table toTable(lua_State *L, const CompletionEvent &ev)
-{
-    return sol::state_view(L).create_table_with(
-        "query", ev.query,                          //
-        "full_text_content", ev.full_text_content,  //
-        "cursor_position", ev.cursor_position,      //
-        "is_first_word", ev.is_first_word           //
-    );
-}
-
-void c2_register_callback(ThisPluginState L, EventType evtType,
-                          sol::protected_function callback)
-{
-    L.plugin()->callbacks[evtType] = std::move(callback);
-}
-
-void c2_log(ThisPluginState L, LogLevel lvl, sol::variadic_args args)
-{
-    lua::StackGuard guard(L);
+    auto *pl = getApp()->getPlugins()->getPluginByStatePtr(L);
+    if (pl == nullptr)
     {
-        QDebug stream = qdebugStreamForLogLevel(lvl);
-        logHelper(L, L.plugin(), stream, args);
+        luaL_error(L, "internal error: no plugin");
+        return 0;
     }
+
+    QString name;
+    if (!lua::peek(L, &name, 1))
+    {
+        luaL_error(L, "cannot get command name (1st arg of register_command, "
+                      "expected a string)");
+        return 0;
+    }
+    if (lua_isnoneornil(L, 2))
+    {
+        luaL_error(L, "missing argument for register_command: function "
+                      "\"pointer\"");
+        return 0;
+    }
+
+    auto callbackSavedName = QString("c2commandcb-%1").arg(name);
+    lua_setfield(L, LUA_REGISTRYINDEX, callbackSavedName.toStdString().c_str());
+    auto ok = pl->registerCommand(name, callbackSavedName);
+
+    // delete both name and callback
+    lua_pop(L, 2);
+
+    lua::push(L, ok);
+    return 1;
 }
 
-void c2_later(ThisPluginState L, sol::protected_function callback, int time)
+int c2_register_callback(lua_State *L)
 {
-    if (time <= 0)
+    auto *pl = getApp()->getPlugins()->getPluginByStatePtr(L);
+    if (pl == nullptr)
     {
-        throw std::runtime_error(
-            "c2.later time must be strictly greater than zero.");
+        luaL_error(L, "internal error: no plugin");
+        return 0;
+    }
+    EventType evtType{};
+    if (!lua::peek(L, &evtType, 1))
+    {
+        luaL_error(L, "cannot get event name (1st arg of register_callback, "
+                      "expected a string)");
+        return 0;
+    }
+    if (lua_isnoneornil(L, 2))
+    {
+        luaL_error(L, "missing argument for register_callback: function "
+                      "\"pointer\"");
+        return 0;
+    }
+
+    auto typeName = magic_enum::enum_name(evtType);
+    std::string callbackSavedName;
+    callbackSavedName.reserve(5 + typeName.size());
+    callbackSavedName += "c2cb-";
+    callbackSavedName += typeName;
+    lua_setfield(L, LUA_REGISTRYINDEX, callbackSavedName.c_str());
+
+    lua_pop(L, 2);
+
+    return 0;
+}
+
+int c2_log(lua_State *L)
+{
+    auto *pl = getApp()->getPlugins()->getPluginByStatePtr(L);
+    if (pl == nullptr)
+    {
+        luaL_error(L, "c2_log: internal error: no plugin?");
+        return 0;
+    }
+    auto logc = lua_gettop(L) - 1;
+    // This is almost the expansion of qCDebug() macro, actual thing is wrapped in a for loop
+    LogLevel lvl{};
+    if (!lua::pop(L, &lvl, 1))
+    {
+        luaL_error(L, "Invalid log level, use one from c2.LogLevel.");
+        return 0;
+    }
+    QDebug stream = qdebugStreamForLogLevel(lvl);
+    logHelper(L, pl, stream, logc);
+    return 0;
+}
+
+int c2_later(lua_State *L)
+{
+    auto *pl = getApp()->getPlugins()->getPluginByStatePtr(L);
+    if (pl == nullptr)
+    {
+        return luaL_error(L, "c2.later: internal error: no plugin?");
+    }
+    if (lua_gettop(L) != 2)
+    {
+        return luaL_error(
+            L, "c2.later expects two arguments (a callback that takes no "
+               "arguments and returns nothing and a number the time in "
+               "milliseconds to wait)\n");
+    }
+    int time{};
+    if (!lua::pop(L, &time))
+    {
+        return luaL_error(L, "cannot get time (2nd arg of c2.later, "
+                             "expected a number)");
+    }
+
+    if (!lua_isfunction(L, lua_gettop(L)))
+    {
+        return luaL_error(L, "cannot get callback (1st arg of c2.later, "
+                             "expected a function)");
     }
-    sol::state_view lua(L);
 
     auto *timer = new QTimer();
     timer->setInterval(time);
-    timer->setSingleShot(true);
-    auto id = L.plugin()->addTimeout(timer);
+    auto id = pl->addTimeout(timer);
     auto name = QString("timeout_%1").arg(id);
+    auto *coro = lua_newthread(L);
 
-    sol::state_view main = sol::main_thread(L);
+    QObject::connect(timer, &QTimer::timeout, [pl, coro, name, timer]() {
+        timer->deleteLater();
+        pl->removeTimeout(timer);
+        int nres{};
+        lua_resume(coro, nullptr, 0, &nres);
 
-    sol::thread thread = sol::thread::create(main);
-    sol::protected_function cb(thread.state(), callback);
-    main.registry()[name.toStdString()] = thread;
-
-    QObject::connect(
-        timer, &QTimer::timeout,
-        [pl = L.plugin(), name, timer, cb, thread, main]() {
-            timer->deleteLater();
-            pl->removeTimeout(timer);
-            sol::protected_function_result res = cb();
-
-            if (res.return_count() != 0)
-            {
-                stackDump(thread.lua_state(),
-                          pl->id +
-                              ": timer returned a value, this shouldn't happen "
-                              "and is probably a plugin bug");
-            }
-            main.registry()[name.toStdString()] = sol::nil;
-        });
+        lua_pushnil(coro);
+        lua_setfield(coro, LUA_REGISTRYINDEX, name.toStdString().c_str());
+        if (lua_gettop(coro) != 0)
+        {
+            stackDump(coro,
+                      pl->id +
+                          ": timer returned a value, this shouldn't happen "
+                          "and is probably a plugin bug");
+        }
+    });
+    stackDump(L, "before setfield");
+    lua_setfield(L, LUA_REGISTRYINDEX, name.toStdString().c_str());
+    lua_xmove(L, coro, 1);  // move function to thread
     timer->start();
+
+    return 0;
 }
 
-// TODO: Add tests for this once we run tests in debug mode
-sol::variadic_results g_load(ThisPluginState s, sol::object data)
+int g_load(lua_State *L)
 {
 #    ifdef NDEBUG
-    (void)data;
-    (void)s;
-    throw std::runtime_error("load() is only usable in debug mode");
+    luaL_error(L, "load() is only usable in debug mode");
+    return 0;
 #    else
+    auto countArgs = lua_gettop(L);
+    QByteArray data;
+    if (lua::peek(L, &data, 1))
+    {
+        auto *utf8 = QTextCodec::codecForName("UTF-8");
+        QTextCodec::ConverterState state;
+        utf8->toUnicode(data.constData(), data.size(), &state);
+        if (state.invalidChars != 0)
+        {
+            luaL_error(L, "invalid utf-8 in load() is not allowed");
+            return 0;
+        }
+    }
+    else
+    {
+        luaL_error(L, "using reader function in load() is not allowed");
+        return 0;
+    }
 
-    // If you're modifying this PLEASE verify it works, Sol is very annoying about serialization
-    // - Mm2PL
-    sol::state_view lua(s);
-    auto load = lua.registry()["real_load"];
-    sol::protected_function_result ret = load(data, "=(load)", "t");
-    return ret;
+    for (int i = 0; i < countArgs; i++)
+    {
+        lua_seti(L, LUA_REGISTRYINDEX, i);
+    }
+
+    // fetch load and call it
+    lua_getfield(L, LUA_REGISTRYINDEX, "real_load");
+
+    for (int i = 0; i < countArgs; i++)
+    {
+        lua_geti(L, LUA_REGISTRYINDEX, i);
+        lua_pushnil(L);
+        lua_seti(L, LUA_REGISTRYINDEX, i);
+    }
+
+    lua_call(L, countArgs, LUA_MULTRET);
+
+    return lua_gettop(L);
 #    endif
 }
 
@@ -228,7 +320,7 @@ int searcherAbsolute(lua_State *L)
 int searcherRelative(lua_State *L)
 {
     lua_Debug dbg;
-    lua_getstack(L, 2, &dbg);
+    lua_getstack(L, 1, &dbg);
     lua_getinfo(L, "S", &dbg);
     auto currentFile = QString::fromUtf8(dbg.source, dbg.srclen);
     if (currentFile.startsWith("@"))
@@ -254,14 +346,22 @@ int searcherRelative(lua_State *L)
     return loadfile(L, filename);
 }
 
-void g_print(ThisPluginState L, sol::variadic_args args)
+int g_print(lua_State *L)
 {
+    auto *pl = getApp()->getPlugins()->getPluginByStatePtr(L);
+    if (pl == nullptr)
+    {
+        luaL_error(L, "c2_print: internal error: no plugin?");
+        return 0;
+    }
+    auto argc = lua_gettop(L);
     // This is almost the expansion of qCDebug() macro, actual thing is wrapped in a for loop
     auto stream =
         (QMessageLogger(QT_MESSAGELOG_FILE, QT_MESSAGELOG_LINE,
                         QT_MESSAGELOG_FUNC, chatterinoLua().categoryName())
              .debug());
-    logHelper(L, L.plugin(), stream, args);
+    logHelper(L, pl, stream, argc);
+    return 0;
 }
 
 }  // namespace chatterino::lua::api

src/controllers/plugins/LuaAPI.hpp

@@ -1,17 +1,17 @@
 #pragma once
 
 #ifdef CHATTERINO_HAVE_PLUGINS
-#    include "controllers/plugins/api/ChannelRef.hpp"
-#    include "controllers/plugins/Plugin.hpp"
-#    include "controllers/plugins/SolTypes.hpp"
 
+extern "C" {
 #    include <lua.h>
-#    include <QList>
+}
+#    include "controllers/plugins/LuaUtilities.hpp"
+
 #    include <QString>
-#    include <sol/table.hpp>
 
 #    include <cassert>
 #    include <memory>
+#    include <vector>
 
 struct lua_State;
 namespace chatterino::lua::api {
@@ -30,8 +30,11 @@ namespace chatterino::lua::api {
 enum class LogLevel { Debug, Info, Warning, Critical };
 
 /**
- * @includefile controllers/plugins/api/EventType.hpp
+ * @exposeenum c2.EventType
  */
+enum class EventType {
+    CompletionRequested,
+};
 
 /**
  * @lua@class CommandContext
@@ -43,12 +46,10 @@ enum class LogLevel { Debug, Info, Warning, Critical };
  * @lua@class CompletionList
  */
 struct CompletionList {
-    CompletionList(const sol::table &);
-
     /**
      * @lua@field values string[] The completions
      */
-    QStringList values;
+    std::vector<QString> values{};
 
     /**
      * @lua@field hide_others boolean Whether other completions from Chatterino should be hidden/ignored.
@@ -78,8 +79,6 @@ struct CompletionEvent {
     bool is_first_word{};
 };
 
-sol::table toTable(lua_State *L, const CompletionEvent &ev);
-
 /**
  * @includefile common/Channel.hpp
  * @includefile controllers/plugins/api/ChannelRef.hpp
@@ -96,16 +95,16 @@ sol::table toTable(lua_State *L, const CompletionEvent &ev);
  * @lua@return boolean ok  Returns `true` if everything went ok, `false` if a command with this name exists.
  * @exposed c2.register_command
  */
+int c2_register_command(lua_State *L);
 
 /**
  * Registers a callback to be invoked when completions for a term are requested.
  *
- * @lua@param type c2.EventType.CompletionRequested
+ * @lua@param type "CompletionRequested"
  * @lua@param func fun(event: CompletionEvent): CompletionList The callback to be invoked.
  * @exposed c2.register_callback
  */
-void c2_register_callback(ThisPluginState L, EventType evtType,
-                          sol::protected_function callback);
+int c2_register_callback(lua_State *L);
 
 /**
  * Writes a message to the Chatterino log.
@@ -114,7 +113,7 @@ void c2_register_callback(ThisPluginState L, EventType evtType,
  * @lua@param ... any Values to log. Should be convertible to a string with `tostring()`.
  * @exposed c2.log
  */
-void c2_log(ThisPluginState L, LogLevel lvl, sol::variadic_args args);
+int c2_log(lua_State *L);
 
 /**
  * Calls callback around msec milliseconds later. Does not freeze Chatterino.
@@ -123,11 +122,11 @@ void c2_log(ThisPluginState L, LogLevel lvl, sol::variadic_args args);
  * @lua@param msec number How long to wait.
  * @exposed c2.later
  */
-void c2_later(ThisPluginState L, sol::protected_function callback, int time);
+int c2_later(lua_State *L);
 
 // These ones are global
-sol::variadic_results g_load(ThisPluginState s, sol::object data);
-void g_print(ThisPluginState L, sol::variadic_args args);
+int g_load(lua_State *L);
+int g_print(lua_State *L);
 // NOLINTEND(readability-identifier-naming)
 
 // This is for require() exposed as an element of package.searchers

src/controllers/plugins/LuaUtilities.cpp

@@ -1,10 +1,16 @@
 #ifdef CHATTERINO_HAVE_PLUGINS
 #    include "controllers/plugins/LuaUtilities.hpp"
 
+#    include "common/Channel.hpp"
 #    include "common/QLogging.hpp"
+#    include "controllers/commands/CommandContext.hpp"
+#    include "controllers/plugins/api/ChannelRef.hpp"
+#    include "controllers/plugins/LuaAPI.hpp"
 
+extern "C" {
 #    include <lauxlib.h>
 #    include <lua.h>
+}
 
 #    include <climits>
 #    include <cstdlib>
@@ -73,6 +79,9 @@ QString humanErrorText(lua_State *L, int errCode)
         case LUA_ERRFILE:
             errName = "(file error)";
             break;
+        case ERROR_BAD_PEEK:
+            errName = "(unable to convert value to c++)";
+            break;
         default:
             errName = "(unknown error type)";
     }
@@ -84,6 +93,18 @@ QString humanErrorText(lua_State *L, int errCode)
     return errName;
 }
 
+StackIdx pushEmptyArray(lua_State *L, int countArray)
+{
+    lua_createtable(L, countArray, 0);
+    return lua_gettop(L);
+}
+
+StackIdx pushEmptyTable(lua_State *L, int countProperties)
+{
+    lua_createtable(L, 0, countProperties);
+    return lua_gettop(L);
+}
+
 StackIdx push(lua_State *L, const QString &str)
 {
     return lua::push(L, str.toStdString());
@@ -95,6 +116,82 @@ StackIdx push(lua_State *L, const std::string &str)
     return lua_gettop(L);
 }
 
+StackIdx push(lua_State *L, const CommandContext &ctx)
+{
+    StackGuard guard(L, 1);
+    auto outIdx = pushEmptyTable(L, 2);
+
+    push(L, ctx.words);
+    lua_setfield(L, outIdx, "words");
+
+    push(L, ctx.channel);
+    lua_setfield(L, outIdx, "channel");
+
+    return outIdx;
+}
+
+StackIdx push(lua_State *L, const bool &b)
+{
+    lua_pushboolean(L, int(b));
+    return lua_gettop(L);
+}
+
+StackIdx push(lua_State *L, const int &b)
+{
+    lua_pushinteger(L, b);
+    return lua_gettop(L);
+}
+
+StackIdx push(lua_State *L, const api::CompletionEvent &ev)
+{
+    auto idx = pushEmptyTable(L, 4);
+#    define PUSH(field)         \
+        lua::push(L, ev.field); \
+        lua_setfield(L, idx, #field)
+    PUSH(query);
+    PUSH(full_text_content);
+    PUSH(cursor_position);
+    PUSH(is_first_word);
+#    undef PUSH
+    return idx;
+}
+
+bool peek(lua_State *L, int *out, StackIdx idx)
+{
+    StackGuard guard(L);
+    if (lua_isnumber(L, idx) == 0)
+    {
+        return false;
+    }
+
+    *out = lua_tointeger(L, idx);
+    return true;
+}
+
+bool peek(lua_State *L, bool *out, StackIdx idx)
+{
+    StackGuard guard(L);
+    if (!lua_isboolean(L, idx))
+    {
+        return false;
+    }
+
+    *out = bool(lua_toboolean(L, idx));
+    return true;
+}
+
+bool peek(lua_State *L, double *out, StackIdx idx)
+{
+    StackGuard guard(L);
+    int ok{0};
+    auto v = lua_tonumberx(L, idx, &ok);
+    if (ok != 0)
+    {
+        *out = v;
+    }
+    return ok != 0;
+}
+
 bool peek(lua_State *L, QString *out, StackIdx idx)
 {
     StackGuard guard(L);
@@ -112,6 +209,57 @@ bool peek(lua_State *L, QString *out, StackIdx idx)
     return true;
 }
 
+bool peek(lua_State *L, QByteArray *out, StackIdx idx)
+{
+    StackGuard guard(L);
+    size_t len{0};
+    const char *str = lua_tolstring(L, idx, &len);
+    if (str == nullptr)
+    {
+        return false;
+    }
+    if (len >= INT_MAX)
+    {
+        assert(false && "string longer than INT_MAX, shit's fucked, yo");
+    }
+    *out = QByteArray(str, int(len));
+    return true;
+}
+
+bool peek(lua_State *L, std::string *out, StackIdx idx)
+{
+    StackGuard guard(L);
+    size_t len{0};
+    const char *str = lua_tolstring(L, idx, &len);
+    if (str == nullptr)
+    {
+        return false;
+    }
+    if (len >= INT_MAX)
+    {
+        assert(false && "string longer than INT_MAX, shit's fucked, yo");
+    }
+    *out = std::string(str, len);
+    return true;
+}
+
+bool peek(lua_State *L, api::CompletionList *out, StackIdx idx)
+{
+    StackGuard guard(L);
+    int typ = lua_getfield(L, idx, "values");
+    if (typ != LUA_TTABLE)
+    {
+        lua_pop(L, 1);
+        return false;
+    }
+    if (!lua::pop(L, &out->values, -1))
+    {
+        return false;
+    }
+    lua_getfield(L, idx, "hide_others");
+    return lua::pop(L, &out->hideOthers);
+}
+
 QString toString(lua_State *L, StackIdx idx)
 {
     size_t len{};

src/controllers/plugins/LuaUtilities.hpp

@@ -2,20 +2,37 @@
 
 #ifdef CHATTERINO_HAVE_PLUGINS
 
+#    include "common/QLogging.hpp"
+
+extern "C" {
 #    include <lua.h>
 #    include <lualib.h>
+}
 #    include <magic_enum/magic_enum.hpp>
 #    include <QList>
-#    include <sol/state_view.hpp>
 
 #    include <cassert>
+#    include <optional>
 #    include <string>
 #    include <string_view>
 #    include <type_traits>
+#    include <variant>
+#    include <vector>
 struct lua_State;
+class QJsonObject;
+namespace chatterino {
+struct CommandContext;
+}  // namespace chatterino
 
 namespace chatterino::lua {
 
+namespace api {
+    struct CompletionList;
+    struct CompletionEvent;
+}  // namespace api
+
+constexpr int ERROR_BAD_PEEK = LUA_OK - 1;
+
 /**
  * @brief Dumps the Lua stack into qCDebug(chatterinoLua)
  *
@@ -23,9 +40,6 @@ namespace chatterino::lua {
  */
 void stackDump(lua_State *L, const QString &tag);
 
-// This is for calling stackDump out of gdb as it's not easy to create a QString there
-const QString GDB_DUMMY = "GDB_DUMMY";
-
 /**
  * @brief Converts a lua error code and potentially string on top of the stack into a human readable message
  */
@@ -36,11 +50,33 @@ QString humanErrorText(lua_State *L, int errCode);
  */
 using StackIdx = int;
 
+/**
+ * @brief Creates a table with countArray array properties on the Lua stack
+ * @return stack index of the newly created table
+ */
+StackIdx pushEmptyArray(lua_State *L, int countArray);
+
+/**
+ * @brief Creates a table with countProperties named properties on the Lua stack
+ * @return stack index of the newly created table
+ */
+StackIdx pushEmptyTable(lua_State *L, int countProperties);
+
+StackIdx push(lua_State *L, const CommandContext &ctx);
 StackIdx push(lua_State *L, const QString &str);
 StackIdx push(lua_State *L, const std::string &str);
+StackIdx push(lua_State *L, const bool &b);
+StackIdx push(lua_State *L, const int &b);
+StackIdx push(lua_State *L, const api::CompletionEvent &ev);
 
 // returns OK?
+bool peek(lua_State *L, int *out, StackIdx idx = -1);
+bool peek(lua_State *L, bool *out, StackIdx idx = -1);
+bool peek(lua_State *L, double *out, StackIdx idx = -1);
 bool peek(lua_State *L, QString *out, StackIdx idx = -1);
+bool peek(lua_State *L, QByteArray *out, StackIdx idx = -1);
+bool peek(lua_State *L, std::string *out, StackIdx idx = -1);
+bool peek(lua_State *L, api::CompletionList *out, StackIdx idx = -1);
 
 /**
  * @brief Converts Lua object at stack index idx to a string.
@@ -104,29 +140,246 @@ public:
     }
 };
 
+/// TEMPLATES
+
+template <typename T>
+StackIdx push(lua_State *L, std::optional<T> val)
+{
+    if (val.has_value())
+    {
+        return lua::push(L, *val);
+    }
+    lua_pushnil(L);
+    return lua_gettop(L);
+}
+
+template <typename T>
+bool peek(lua_State *L, std::optional<T> *out, StackIdx idx = -1)
+{
+    if (lua_isnil(L, idx))
+    {
+        *out = std::nullopt;
+        return true;
+    }
+
+    *out = T();
+    return peek(L, out->operator->(), idx);
+}
+
+template <typename T>
+bool peek(lua_State *L, std::vector<T> *vec, StackIdx idx = -1)
+{
+    StackGuard guard(L);
+
+    if (!lua_istable(L, idx))
+    {
+        lua::stackDump(L, "!table");
+        qCDebug(chatterinoLua)
+            << "value is not a table, type is" << lua_type(L, idx);
+        return false;
+    }
+    auto len = lua_rawlen(L, idx);
+    if (len == 0)
+    {
+        qCDebug(chatterinoLua) << "value has 0 length";
+        return true;
+    }
+    if (len > 1'000'000)
+    {
+        qCDebug(chatterinoLua) << "value is too long";
+        return false;
+    }
+    // count like lua
+    for (int i = 1; i <= len; i++)
+    {
+        lua_geti(L, idx, i);
+        std::optional<T> obj;
+        if (!lua::peek(L, &obj))
+        {
+            //lua_seti(L, LUA_REGISTRYINDEX, 1);  // lazy
+            qCDebug(chatterinoLua)
+                << "Failed to convert lua object into c++: at array index " << i
+                << ":";
+            stackDump(L, "bad conversion into string");
+            return false;
+        }
+        lua_pop(L, 1);
+        vec->push_back(obj.value());
+    }
+    return true;
+}
+
+/**
+ * @brief Converts object at stack index idx to enum given by template parameter T
+ */
+template <typename T,
+          typename std::enable_if<std::is_enum_v<T>, bool>::type = true>
+bool peek(lua_State *L, T *out, StackIdx idx = -1)
+{
+    std::string tmp;
+    if (!lua::peek(L, &tmp, idx))
+    {
+        return false;
+    }
+    std::optional<T> opt = magic_enum::enum_cast<T>(tmp);
+    if (opt.has_value())
+    {
+        *out = opt.value();
+        return true;
+    }
+
+    return false;
+}
+
+/**
+ * @brief Converts a vector<T> to Lua and pushes it onto the stack.
+ *
+ * Needs StackIdx push(lua_State*, T); to work.
+ *
+ * @return Stack index of newly created table.
+ */
+template <typename T>
+StackIdx push(lua_State *L, std::vector<T> vec)
+{
+    auto out = pushEmptyArray(L, vec.size());
+    int i = 1;
+    for (const auto &el : vec)
+    {
+        push(L, el);
+        lua_seti(L, out, i);
+        i += 1;
+    }
+    return out;
+}
+
+/**
+ * @brief Converts a QList<T> to Lua and pushes it onto the stack.
+ *
+ * Needs StackIdx push(lua_State*, T); to work.
+ *
+ * @return Stack index of newly created table.
+ */
+template <typename T>
+StackIdx push(lua_State *L, QList<T> vec)
+{
+    auto out = pushEmptyArray(L, vec.size());
+    int i = 1;
+    for (const auto &el : vec)
+    {
+        push(L, el);
+        lua_seti(L, out, i);
+        i += 1;
+    }
+    return out;
+}
+
+/**
+ * @brief Converts an enum given by T to Lua (into a string) and pushes it onto the stack.
+ *
+ * @return Stack index of newly created string.
+ */
+template <typename T, typename std::enable_if_t<std::is_enum_v<T>, bool> = true>
+StackIdx push(lua_State *L, T inp)
+{
+    std::string_view name = magic_enum::enum_name<T>(inp);
+    return lua::push(L, std::string(name));
+}
+
+/**
+ * @brief Converts a Lua object into c++ and removes it from the stack.
+ * If peek fails, the object is still removed from the stack.
+ *
+ * Relies on bool peek(lua_State*, T*, StackIdx) existing.
+ */
+template <typename T>
+bool pop(lua_State *L, T *out, StackIdx idx = -1)
+{
+    StackGuard guard(L, -1);
+    auto ok = peek(L, out, idx);
+    if (idx < 0)
+    {
+        idx = lua_gettop(L) + idx + 1;
+    }
+    lua_remove(L, idx);
+    return ok;
+}
+
 /**
  * @brief Creates a table mapping enum names to unique values.
  *
  * Values in this table may change.
  *
- * @returns Sol reference to the table
+ * @returns stack index of newly created table
  */
 template <typename T>
-    requires std::is_enum_v<T>
-sol::table createEnumTable(sol::state_view &lua)
+StackIdx pushEnumTable(lua_State *L)
 {
-    constexpr auto values = magic_enum::enum_values<T>();
-    auto out = lua.create_table(0, values.size());
+    // std::array<T, _>
+    auto values = magic_enum::enum_values<T>();
+    StackIdx out = lua::pushEmptyTable(L, values.size());
     for (const T v : values)
     {
         std::string_view name = magic_enum::enum_name<T>(v);
         std::string str(name);
 
-        out.raw_set(str, v);
+        lua::push(L, str);
+        lua_setfield(L, out, str.c_str());
     }
     return out;
 }
 
+// Represents a Lua function on the stack
+template <typename ReturnType, typename... Args>
+class CallbackFunction
+{
+    StackIdx stackIdx_;
+    lua_State *L;
+
+public:
+    CallbackFunction(lua_State *L, StackIdx stackIdx)
+        : stackIdx_(stackIdx)
+        , L(L)
+    {
+    }
+
+    // this type owns the stackidx, it must not be trivially copiable
+    CallbackFunction operator=(CallbackFunction &) = delete;
+    CallbackFunction(CallbackFunction &) = delete;
+
+    // Permit only move
+    CallbackFunction &operator=(CallbackFunction &&) = default;
+    CallbackFunction(CallbackFunction &&) = default;
+
+    ~CallbackFunction()
+    {
+        lua_remove(L, this->stackIdx_);
+    }
+
+    std::variant<int, ReturnType> operator()(Args... arguments)
+    {
+        lua_pushvalue(this->L, this->stackIdx_);
+        (  // apparently this calls lua::push() for every Arg
+            [this, &arguments] {
+                lua::push(this->L, arguments);
+            }(),
+            ...);
+
+        int res = lua_pcall(L, sizeof...(Args), 1, 0);
+        if (res != LUA_OK)
+        {
+            qCDebug(chatterinoLua) << "error is: " << res;
+            return {res};
+        }
+
+        ReturnType val;
+        if (!lua::pop(L, &val))
+        {
+            return {ERROR_BAD_PEEK};
+        }
+        return {val};
+    }
+};
+
 }  // namespace chatterino::lua
 
 #endif

src/controllers/plugins/Plugin.cpp

@@ -7,13 +7,14 @@
 #    include "controllers/plugins/PluginPermission.hpp"
 #    include "util/QMagicEnum.hpp"
 
+extern "C" {
 #    include <lua.h>
+}
 #    include <magic_enum/magic_enum.hpp>
 #    include <QJsonArray>
 #    include <QJsonObject>
 #    include <QLoggingCategory>
 #    include <QUrl>
-#    include <sol/sol.hpp>
 
 #    include <algorithm>
 #    include <unordered_map>
@@ -189,8 +190,7 @@ PluginMeta::PluginMeta(const QJsonObject &obj)
     }
 }
 
-bool Plugin::registerCommand(const QString &name,
-                             sol::protected_function function)
+bool Plugin::registerCommand(const QString &name, const QString &functionName)
 {
     if (this->ownedCommands.find(name) != this->ownedCommands.end())
     {
@@ -202,7 +202,7 @@ bool Plugin::registerCommand(const QString &name,
     {
         return false;
     }
-    this->ownedCommands.emplace(name, std::move(function));
+    this->ownedCommands.insert({name, functionName});
     return true;
 }
 
@@ -223,24 +223,14 @@ Plugin::~Plugin()
         QObject::disconnect(timer, nullptr, nullptr, nullptr);
         timer->deleteLater();
     }
-    this->httpRequests.clear();
     qCDebug(chatterinoLua) << "Destroyed" << this->activeTimeouts.size()
                            << "timers for plugin" << this->id
                            << "while destroying the object";
     this->activeTimeouts.clear();
     if (this->state_ != nullptr)
     {
-        // clearing this after the state is gone is not safe to do
-        this->ownedCommands.clear();
-        this->callbacks.clear();
         lua_close(this->state_);
     }
-    assert(this->ownedCommands.empty() &&
-           "This must be empty or destructor of sol::protected_function would "
-           "explode malloc structures later");
-    assert(this->callbacks.empty() &&
-           "This must be empty or destructor of sol::protected_function would "
-           "explode malloc structures later");
 }
 int Plugin::addTimeout(QTimer *timer)
 {

src/controllers/plugins/Plugin.hpp

@@ -2,8 +2,8 @@
 
 #ifdef CHATTERINO_HAVE_PLUGINS
 #    include "Application.hpp"
-#    include "controllers/plugins/api/EventType.hpp"
-#    include "controllers/plugins/api/HTTPRequest.hpp"
+#    include "common/network/NetworkCommon.hpp"
+#    include "controllers/plugins/LuaAPI.hpp"
 #    include "controllers/plugins/LuaUtilities.hpp"
 #    include "controllers/plugins/PluginPermission.hpp"
 
@@ -11,10 +11,7 @@
 #    include <QString>
 #    include <QUrl>
 #    include <semver/semver.hpp>
-#    include <sol/forward.hpp>
 
-#    include <memory>
-#    include <optional>
 #    include <unordered_map>
 #    include <unordered_set>
 #    include <vector>
@@ -59,8 +56,6 @@ struct PluginMeta {
     }
 
     explicit PluginMeta(const QJsonObject &obj);
-    // This is for tests
-    PluginMeta() = default;
 };
 
 class Plugin
@@ -80,18 +75,13 @@ public:
 
     ~Plugin();
 
-    Plugin(const Plugin &) = delete;
-    Plugin(Plugin &&) = delete;
-    Plugin &operator=(const Plugin &) = delete;
-    Plugin &operator=(Plugin &&) = delete;
-
     /**
      * @brief Perform all necessary tasks to bind a command name to this plugin
      * @param name name of the command to create
-     * @param function the function that should be called when the command is executed
+     * @param functionName name of the function that should be called when the command is executed
      * @return true if addition succeeded, false otherwise (for example because the command name is already taken)
      */
-    bool registerCommand(const QString &name, sol::protected_function function);
+    bool registerCommand(const QString &name, const QString &functionName);
 
     /**
      * @brief Get names of all commands belonging to this plugin
@@ -108,19 +98,35 @@ public:
         return this->loadDirectory_.absoluteFilePath("data");
     }
 
-    std::optional<sol::protected_function> getCompletionCallback()
+    // Note: The CallbackFunction object's destructor will remove the function from the lua stack
+    using LuaCompletionCallback =
+        lua::CallbackFunction<lua::api::CompletionList,
+                              lua::api::CompletionEvent>;
+    std::optional<LuaCompletionCallback> getCompletionCallback()
     {
         if (this->state_ == nullptr || !this->error_.isNull())
         {
             return {};
         }
-        auto it =
-            this->callbacks.find(lua::api::EventType::CompletionRequested);
-        if (it == this->callbacks.end())
+        // this uses magic enum to help automatic tooling find usages
+        auto typeName =
+            magic_enum::enum_name(lua::api::EventType::CompletionRequested);
+        std::string cbName;
+        cbName.reserve(5 + typeName.size());
+        cbName += "c2cb-";
+        cbName += typeName;
+        auto typ =
+            lua_getfield(this->state_, LUA_REGISTRYINDEX, cbName.c_str());
+        if (typ != LUA_TFUNCTION)
         {
+            lua_pop(this->state_, 1);
             return {};
         }
-        return it->second;
+
+        // move
+        return std::make_optional<lua::CallbackFunction<
+            lua::api::CompletionList, lua::api::CompletionEvent>>(
+            this->state_, lua_gettop(this->state_));
     }
 
     /**
@@ -137,25 +143,18 @@ public:
     bool hasFSPermissionFor(bool write, const QString &path);
     bool hasHTTPPermissionFor(const QUrl &url);
 
-    std::map<lua::api::EventType, sol::protected_function> callbacks;
-
-    // In-flight HTTP Requests
-    // This is a lifetime hack to ensure they get deleted with the plugin. This relies on the Plugin getting deleted on reload!
-    std::vector<std::shared_ptr<lua::api::HTTPRequest>> httpRequests;
-
 private:
     QDir loadDirectory_;
     lua_State *state_;
 
     QString error_;
 
-    // maps command name -> function
-    std::unordered_map<QString, sol::protected_function> ownedCommands;
+    // maps command name -> function name
+    std::unordered_map<QString, QString> ownedCommands;
     std::vector<QTimer *> activeTimeouts;
     int lastTimerId = 0;
 
     friend class PluginController;
-    friend class PluginControllerAccess;  // this is for tests
 };
 }  // namespace chatterino
 #endif

src/controllers/plugins/PluginController.cpp

@@ -13,20 +13,16 @@
 #    include "controllers/plugins/api/IOWrapper.hpp"
 #    include "controllers/plugins/LuaAPI.hpp"
 #    include "controllers/plugins/LuaUtilities.hpp"
-#    include "controllers/plugins/SolTypes.hpp"
 #    include "messages/MessageBuilder.hpp"
 #    include "singletons/Paths.hpp"
 #    include "singletons/Settings.hpp"
 
+extern "C" {
 #    include <lauxlib.h>
 #    include <lua.h>
 #    include <lualib.h>
+}
 #    include <QJsonDocument>
-#    include <sol/overload.hpp>
-#    include <sol/sol.hpp>
-#    include <sol/types.hpp>
-#    include <sol/variadic_args.hpp>
-#    include <sol/variadic_results.hpp>
 
 #    include <memory>
 #    include <utility>
@@ -117,11 +113,10 @@ bool PluginController::tryLoadFromDir(const QDir &pluginDir)
     return true;
 }
 
-void PluginController::openLibrariesFor(Plugin *plugin)
+void PluginController::openLibrariesFor(lua_State *L, const PluginMeta &meta,
+                                        const QDir &pluginDir)
 {
-    auto *L = plugin->state_;
     lua::StackGuard guard(L);
-    sol::state_view lua(L);
     // Stuff to change, remove or hide behind a permission system:
     static const std::vector<luaL_Reg> loadedlibs = {
         luaL_Reg{LUA_GNAME, luaopen_base},
@@ -129,6 +124,8 @@ void PluginController::openLibrariesFor(Plugin *plugin)
 
         luaL_Reg{LUA_COLIBNAME, luaopen_coroutine},
         luaL_Reg{LUA_TABLIBNAME, luaopen_table},
+        // luaL_Reg{LUA_IOLIBNAME, luaopen_io},
+        // - explicit fs access, needs wrapper with permissions, no usage ideas yet
         // luaL_Reg{LUA_OSLIBNAME, luaopen_os},
         // - fs access
         // - environ access
@@ -150,100 +147,155 @@ void PluginController::openLibrariesFor(Plugin *plugin)
     luaL_requiref(L, LUA_IOLIBNAME, luaopen_io, int(false));
     lua_setfield(L, LUA_REGISTRYINDEX, lua::api::REG_REAL_IO_NAME);
 
-    auto r = lua.registry();
-    auto g = lua.globals();
-    auto c2 = lua.create_table();
-    g["c2"] = c2;
+    // NOLINTNEXTLINE(*-avoid-c-arrays)
+    static const luaL_Reg c2Lib[] = {
+        {"register_command", lua::api::c2_register_command},
+        {"register_callback", lua::api::c2_register_callback},
+        {"log", lua::api::c2_log},
+        {"later", lua::api::c2_later},
+        {nullptr, nullptr},
+    };
+    lua_pushglobaltable(L);
+    auto gtable = lua_gettop(L);
+
+    // count of elements in C2LIB + LogLevel + EventType
+    auto c2libIdx = lua::pushEmptyTable(L, 8);
+
+    luaL_setfuncs(L, c2Lib, 0);
+
+    lua::pushEnumTable<lua::api::LogLevel>(L);
+    lua_setfield(L, c2libIdx, "LogLevel");
+
+    lua::pushEnumTable<lua::api::EventType>(L);
+    lua_setfield(L, c2libIdx, "EventType");
+
+    lua::pushEnumTable<lua::api::LPlatform>(L);
+    lua_setfield(L, c2libIdx, "Platform");
+
+    lua::pushEnumTable<Channel::Type>(L);
+    lua_setfield(L, c2libIdx, "ChannelType");
+
+    lua::pushEnumTable<NetworkRequestType>(L);
+    lua_setfield(L, c2libIdx, "HTTPMethod");
+
+    // Initialize metatables for objects
+    lua::api::ChannelRef::createMetatable(L);
+    lua_setfield(L, c2libIdx, "Channel");
+
+    lua::api::HTTPRequest::createMetatable(L);
+    lua_setfield(L, c2libIdx, "HTTPRequest");
+
+    lua::api::HTTPResponse::createMetatable(L);
+    lua_setfield(L, c2libIdx, "HTTPResponse");
+
+    lua_setfield(L, gtable, "c2");
 
     // ban functions
     // Note: this might not be fully secure? some kind of metatable fuckery might come up?
 
-#    ifndef NDEBUG
-    lua.registry()["real_load"] = lua.globals()["load"];
-#    endif
-    // See chatterino::lua::api::g_load implementation
+    // possibly randomize this name at runtime to prevent some attacks?
 
-    g["loadfile"] = sol::nil;
-    g["dofile"] = sol::nil;
+#    ifndef NDEBUG
+    lua_getfield(L, gtable, "load");
+    lua_setfield(L, LUA_REGISTRYINDEX, "real_load");
+#    endif
+
+    // NOLINTNEXTLINE(*-avoid-c-arrays)
+    static const luaL_Reg replacementFuncs[] = {
+        {"load", lua::api::g_load},
+        {"print", lua::api::g_print},
+        {nullptr, nullptr},
+    };
+    luaL_setfuncs(L, replacementFuncs, 0);
+
+    lua_pushnil(L);
+    lua_setfield(L, gtable, "loadfile");
+
+    lua_pushnil(L);
+    lua_setfield(L, gtable, "dofile");
 
     // set up package lib
+    lua_getfield(L, gtable, "package");
+
+    auto package = lua_gettop(L);
+    lua_pushstring(L, "");
+    lua_setfield(L, package, "cpath");
+
+    // we don't use path
+    lua_pushstring(L, "");
+    lua_setfield(L, package, "path");
+
     {
-        auto package = g["package"];
-        package["cpath"] = "";
-        package["path"] = "";
-
-        sol::protected_function tbremove = g["table"]["remove"];
-
-        // remove searcher_Croot, searcher_C and searcher_Lua leaving only searcher_preload
-        sol::table searchers = package["searchers"];
-        for (int i = 0; i < 3; i++)
-        {
-            tbremove(searchers);
-        }
-        searchers.add(&lua::api::searcherRelative);
-        searchers.add(&lua::api::searcherAbsolute);
+        lua_getfield(L, gtable, "table");
+        auto table = lua_gettop(L);
+        lua_getfield(L, -1, "remove");
+        lua_remove(L, table);
     }
-    // set up io lib
+    auto remove = lua_gettop(L);
+
+    // remove searcher_Croot, searcher_C and searcher_Lua leaving only searcher_preload
+    for (int i = 0; i < 3; i++)
     {
-        auto c2io = lua.create_table();
-        auto realio = r[lua::api::REG_REAL_IO_NAME];
-        c2io["type"] = realio["type"];
-        g["io"] = c2io;
-        // prevent plugins getting direct access to realio
-        r[LUA_LOADED_TABLE]["io"] = c2io;
-
-        // Don't give plugins the option to shit into our stdio
-        r["_IO_input"] = sol::nil;
-        r["_IO_output"] = sol::nil;
+        lua_pushvalue(L, remove);
+        lua_getfield(L, package, "searchers");
+        lua_pcall(L, 1, 0, 0);
     }
-    PluginController::initSol(lua, plugin);
-}
+    lua_pop(L, 1);  // get rid of remove
 
-// TODO: investigate if `plugin` can ever point to an invalid plugin,
-// especially in cases when the plugin is errored.
-void PluginController::initSol(sol::state_view &lua, Plugin *plugin)
-{
-    auto g = lua.globals();
-    // Do not capture plugin->state_ in lambdas, this makes the functions unusable in callbacks
-    g.set_function("print", &lua::api::g_print);
-    g.set_function("load", &lua::api::g_load);
+    lua_getfield(L, package, "searchers");
+    lua_pushcclosure(L, lua::api::searcherRelative, 0);
+    lua_seti(L, -2, 2);
 
-    sol::table c2 = g["c2"];
-    c2.set_function("register_command",
-                    [plugin](const QString &name, sol::protected_function cb) {
-                        return plugin->registerCommand(name, std::move(cb));
-                    });
-    c2.set_function("register_callback", &lua::api::c2_register_callback);
-    c2.set_function("log", &lua::api::c2_log);
-    c2.set_function("later", &lua::api::c2_later);
+    lua::push(L, QString(pluginDir.absolutePath()));
+    lua_pushcclosure(L, lua::api::searcherAbsolute, 1);
+    lua_seti(L, -2, 3);
+    lua_pop(L, 2);  // remove package, package.searchers
 
-    lua::api::ChannelRef::createUserType(c2);
-    lua::api::HTTPResponse::createUserType(c2);
-    lua::api::HTTPRequest::createUserType(c2);
-    c2["ChannelType"] = lua::createEnumTable<Channel::Type>(lua);
-    c2["HTTPMethod"] = lua::createEnumTable<NetworkRequestType>(lua);
-    c2["EventType"] = lua::createEnumTable<lua::api::EventType>(lua);
-    c2["LogLevel"] = lua::createEnumTable<lua::api::LogLevel>(lua);
+    // NOLINTNEXTLINE(*-avoid-c-arrays)
+    static const luaL_Reg ioLib[] = {
+        {"close", lua::api::io_close},
+        {"flush", lua::api::io_flush},
+        {"input", lua::api::io_input},
+        {"lines", lua::api::io_lines},
+        {"open", lua::api::io_open},
+        {"output", lua::api::io_output},
+        {"popen", lua::api::io_popen},  // stub
+        {"read", lua::api::io_read},
+        {"tmpfile", lua::api::io_tmpfile},  // stub
+        {"write", lua::api::io_write},
+        // type = realio.type
+        {nullptr, nullptr},
+    };
+    // TODO: io.popen stub
+    auto iolibIdx = lua::pushEmptyTable(L, 1);
+    luaL_setfuncs(L, ioLib, 0);
 
-    sol::table io = g["io"];
-    io.set_function(
-        "open", sol::overload(&lua::api::io_open, &lua::api::io_open_modeless));
-    io.set_function("lines", sol::overload(&lua::api::io_lines,
-                                           &lua::api::io_lines_noargs));
-    io.set_function("input", sol::overload(&lua::api::io_input_argless,
-                                           &lua::api::io_input_name,
-                                           &lua::api::io_input_file));
-    io.set_function("output", sol::overload(&lua::api::io_output_argless,
-                                            &lua::api::io_output_name,
-                                            &lua::api::io_output_file));
-    io.set_function("close", sol::overload(&lua::api::io_close_argless,
-                                           &lua::api::io_close_file));
-    io.set_function("flush", sol::overload(&lua::api::io_flush_argless,
-                                           &lua::api::io_flush_file));
-    io.set_function("read", &lua::api::io_read);
-    io.set_function("write", &lua::api::io_write);
-    io.set_function("popen", &lua::api::io_popen);
-    io.set_function("tmpfile", &lua::api::io_tmpfile);
+    // set ourio.type = realio.type
+    lua_pushvalue(L, iolibIdx);
+    lua_getfield(L, LUA_REGISTRYINDEX, lua::api::REG_REAL_IO_NAME);
+    lua_getfield(L, -1, "type");
+    lua_remove(L, -2);  // remove realio
+    lua_setfield(L, iolibIdx, "type");
+    lua_pop(L, 1);  // still have iolib on top of stack
+
+    lua_pushvalue(L, iolibIdx);
+    lua_setfield(L, gtable, "io");
+
+    lua_pushvalue(L, iolibIdx);
+    lua_setfield(L, LUA_REGISTRYINDEX, lua::api::REG_C2_IO_NAME);
+
+    luaL_getsubtable(L, LUA_REGISTRYINDEX, LUA_LOADED_TABLE);
+    lua_pushvalue(L, iolibIdx);
+    lua_setfield(L, -2, "io");
+
+    lua_pop(L, 3);  // remove gtable, iolib, LOADED
+
+    // Don't give plugins the option to shit into our stdio
+    lua_pushnil(L);
+    lua_setfield(L, LUA_REGISTRYINDEX, "_IO_input");
+
+    lua_pushnil(L);
+    lua_setfield(L, LUA_REGISTRYINDEX, "_IO_output");
 }
 
 void PluginController::load(const QFileInfo &index, const QDir &pluginDir,
@@ -262,7 +314,7 @@ void PluginController::load(const QFileInfo &index, const QDir &pluginDir,
                                  << " because safe mode is enabled.";
         return;
     }
-    PluginController::openLibrariesFor(temp);
+    PluginController::openLibrariesFor(l, meta, pluginDir);
 
     if (!PluginController::isPluginEnabled(pluginName) ||
         !getSettings()->pluginsEnabled)
@@ -293,13 +345,17 @@ bool PluginController::reload(const QString &id)
     {
         return false;
     }
-
+    if (it->second->state_ != nullptr)
+    {
+        lua_close(it->second->state_);
+        it->second->state_ = nullptr;
+    }
     for (const auto &[cmd, _] : it->second->ownedCommands)
     {
         getApp()->getCommands()->unregisterPluginCommand(cmd);
     }
+    it->second->ownedCommands.clear();
     QDir loadDir = it->second->loadDirectory_;
-    // Since Plugin owns the state, it will clean up everything related to it
     this->plugins_.erase(id);
     this->tryLoadFromDir(loadDir);
     return true;
@@ -313,36 +369,27 @@ QString PluginController::tryExecPluginCommand(const QString &commandName,
         if (auto it = plugin->ownedCommands.find(commandName);
             it != plugin->ownedCommands.end())
         {
-            sol::state_view lua(plugin->state_);
-            sol::table args = lua.create_table_with(
-                "words", ctx.words,                           //
-                "channel", lua::api::ChannelRef(ctx.channel)  //
-            );
+            const auto &funcName = it->second;
 
-            auto result =
-                lua::tryCall<std::optional<QString>>(it->second, args);
-            if (!result)
-            {
-                ctx.channel->addSystemMessage(
-                    QStringView(
-                        u"Failed to evaluate command from plugin %1: %2")
-                        .arg(plugin->meta.name, result.error()));
-                return {};
-            }
+            auto *L = plugin->state_;
+            lua_getfield(L, LUA_REGISTRYINDEX, funcName.toStdString().c_str());
+            lua::push(L, ctx);
 
-            auto opt = result.value();
-            if (!opt)
+            auto res = lua_pcall(L, 1, 0, 0);
+            if (res != LUA_OK)
             {
-                return {};
+                ctx.channel->addSystemMessage("Lua error: " +
+                                              lua::humanErrorText(L, res));
+                return "";
             }
-            return *opt;
+            return "";
         }
     }
     qCCritical(chatterinoLua)
         << "Something's seriously up, no plugin owns command" << commandName
         << "yet a call to execute it came in";
     assert(false && "missing plugin command owner");
-    return {};
+    return "";
 }
 
 bool PluginController::isPluginEnabled(const QString &id)
@@ -388,31 +435,32 @@ std::pair<bool, QStringList> PluginController::updateCustomCompletions(
             continue;
         }
 
+        lua::StackGuard guard(pl->state_);
+
         auto opt = pl->getCompletionCallback();
         if (opt)
         {
             qCDebug(chatterinoLua)
                 << "Processing custom completions from plugin" << name;
             auto &cb = *opt;
-            sol::state_view view(pl->state_);
-            auto errOrList = lua::tryCall<sol::table>(
-                cb,
-                toTable(pl->state_, lua::api::CompletionEvent{
-                                        .query = query,
-                                        .full_text_content = fullTextContent,
-                                        .cursor_position = cursorPosition,
-                                        .is_first_word = isFirstWord,
-                                    }));
-            if (!errOrList.has_value())
+            auto errOrList = cb(lua::api::CompletionEvent{
+                .query = query,
+                .full_text_content = fullTextContent,
+                .cursor_position = cursorPosition,
+                .is_first_word = isFirstWord,
+            });
+            if (std::holds_alternative<int>(errOrList))
             {
+                guard.handled();
+                int err = std::get<int>(errOrList);
                 qCDebug(chatterinoLua)
                     << "Got error from plugin " << pl->meta.name
                     << " while refreshing tab completion: "
-                    << errOrList.get_unexpected().error();
+                    << lua::humanErrorText(pl->state_, err);
                 continue;
             }
 
-            auto list = lua::api::CompletionList(*errOrList);
+            auto list = std::get<lua::api::CompletionList>(errOrList);
             if (list.hideOthers)
             {
                 results = QStringList(list.values.begin(), list.values.end());

src/controllers/plugins/PluginController.hpp

@@ -10,7 +10,6 @@
 #    include <QJsonArray>
 #    include <QJsonObject>
 #    include <QString>
-#    include <sol/forward.hpp>
 
 #    include <algorithm>
 #    include <map>
@@ -67,16 +66,11 @@ private:
               const PluginMeta &meta);
 
     // This function adds lua standard libraries into the state
-    static void openLibrariesFor(Plugin *plugin);
-
-    static void initSol(sol::state_view &lua, Plugin *plugin);
-
+    static void openLibrariesFor(lua_State *L, const PluginMeta & /*meta*/,
+                                 const QDir &pluginDir);
     static void loadChatterinoLib(lua_State *l);
     bool tryLoadFromDir(const QDir &pluginDir);
     std::map<QString, std::unique_ptr<Plugin>> plugins_;
-
-    // This is for tests, pay no attention
-    friend class PluginControllerAccess;
 };
 
 }  // namespace chatterino

src/controllers/plugins/PluginPermission.hpp

@@ -10,8 +10,6 @@ namespace chatterino {
 
 struct PluginPermission {
     explicit PluginPermission(const QJsonObject &obj);
-    // This is for tests
-    PluginPermission() = default;
 
     enum class Type {
         FilesystemRead,

src/controllers/plugins/SolTypes.cpp

@@ -1,131 +0,0 @@
-#ifdef CHATTERINO_HAVE_PLUGINS
-#    include "controllers/plugins/SolTypes.hpp"
-
-#    include "controllers/plugins/PluginController.hpp"
-
-#    include <QObject>
-#    include <sol/thread.hpp>
-namespace chatterino::lua {
-
-Plugin *ThisPluginState::plugin()
-{
-    if (this->plugptr_ != nullptr)
-    {
-        return this->plugptr_;
-    }
-    auto *pl = getApp()->getPlugins()->getPluginByStatePtr(this->state_);
-    if (pl == nullptr)
-    {
-        throw std::runtime_error("internal error: missing plugin");
-    }
-    this->plugptr_ = pl;
-    return pl;
-}
-
-}  // namespace chatterino::lua
-
-// NOLINTBEGIN(readability-named-parameter)
-// QString
-bool sol_lua_check(sol::types<QString>, lua_State *L, int index,
-                   std::function<sol::check_handler_type> handler,
-                   sol::stack::record &tracking)
-{
-    return sol::stack::check<const char *>(L, index, handler, tracking);
-}
-
-QString sol_lua_get(sol::types<QString>, lua_State *L, int index,
-                    sol::stack::record &tracking)
-{
-    auto str = sol::stack::get<std::string_view>(L, index, tracking);
-    return QString::fromUtf8(str.data(), static_cast<qsizetype>(str.length()));
-}
-
-int sol_lua_push(sol::types<QString>, lua_State *L, const QString &value)
-{
-    return sol::stack::push(L, value.toUtf8().data());
-}
-
-// QStringList
-bool sol_lua_check(sol::types<QStringList>, lua_State *L, int index,
-                   std::function<sol::check_handler_type> handler,
-                   sol::stack::record &tracking)
-{
-    return sol::stack::check<sol::table>(L, index, handler, tracking);
-}
-
-QStringList sol_lua_get(sol::types<QStringList>, lua_State *L, int index,
-                        sol::stack::record &tracking)
-{
-    sol::table table = sol::stack::get<sol::table>(L, index, tracking);
-    QStringList result;
-    result.reserve(static_cast<qsizetype>(table.size()));
-    for (size_t i = 1; i < table.size() + 1; i++)
-    {
-        result.append(table.get<QString>(i));
-    }
-    return result;
-}
-
-int sol_lua_push(sol::types<QStringList>, lua_State *L,
-                 const QStringList &value)
-{
-    sol::table table = sol::table::create(L, static_cast<int>(value.size()));
-    for (const QString &str : value)
-    {
-        table.add(str);
-    }
-    return sol::stack::push(L, table);
-}
-
-// QByteArray
-bool sol_lua_check(sol::types<QByteArray>, lua_State *L, int index,
-                   std::function<sol::check_handler_type> handler,
-                   sol::stack::record &tracking)
-{
-    return sol::stack::check<const char *>(L, index, handler, tracking);
-}
-
-QByteArray sol_lua_get(sol::types<QByteArray>, lua_State *L, int index,
-                       sol::stack::record &tracking)
-{
-    auto str = sol::stack::get<std::string_view>(L, index, tracking);
-    return QByteArray::fromRawData(str.data(), str.length());
-}
-
-int sol_lua_push(sol::types<QByteArray>, lua_State *L, const QByteArray &value)
-{
-    return sol::stack::push(L,
-                            std::string_view(value.constData(), value.size()));
-}
-
-namespace chatterino::lua {
-
-// ThisPluginState
-
-bool sol_lua_check(sol::types<chatterino::lua::ThisPluginState>,
-                   lua_State * /*L*/, int /* index*/,
-                   std::function<sol::check_handler_type> /* handler*/,
-                   sol::stack::record & /*tracking*/)
-{
-    return true;
-}
-
-chatterino::lua::ThisPluginState sol_lua_get(
-    sol::types<chatterino::lua::ThisPluginState>, lua_State *L, int /*index*/,
-    sol::stack::record &tracking)
-{
-    tracking.use(0);
-    return {L};
-}
-
-int sol_lua_push(sol::types<chatterino::lua::ThisPluginState>, lua_State *L,
-                 const chatterino::lua::ThisPluginState &value)
-{
-    return sol::stack::push(L, sol::thread(L, value));
-}
-
-}  // namespace chatterino::lua
-
-// NOLINTEND(readability-named-parameter)
-
-#endif

src/controllers/plugins/SolTypes.hpp

@@ -1,170 +0,0 @@
-#pragma once
-#ifdef CHATTERINO_HAVE_PLUGINS
-#    include "util/QMagicEnum.hpp"
-#    include "util/TypeName.hpp"
-
-#    include <nonstd/expected.hpp>
-#    include <QObject>
-#    include <QString>
-#    include <QStringBuilder>
-#    include <QStringList>
-#    include <sol/sol.hpp>
-
-namespace chatterino::detail {
-
-// NOLINTBEGIN(readability-identifier-naming)
-template <typename T>
-constexpr bool IsOptional = false;
-template <typename T>
-constexpr bool IsOptional<std::optional<T>> = true;
-// NOLINTEND(readability-identifier-naming)
-
-}  // namespace chatterino::detail
-
-namespace chatterino {
-
-class Plugin;
-
-}  // namespace chatterino
-
-namespace chatterino::lua {
-
-class ThisPluginState
-{
-public:
-    ThisPluginState(lua_State *Ls)
-        : plugptr_(nullptr)
-        , state_(Ls)
-    {
-    }
-
-    operator lua_State *() const noexcept
-    {
-        return this->state_;
-    }
-
-    lua_State *operator->() const noexcept
-    {
-        return this->state_;
-    }
-    lua_State *state() const noexcept
-    {
-        return this->state_;
-    }
-
-    Plugin *plugin();
-
-private:
-    Plugin *plugptr_;
-    lua_State *state_;
-};
-
-/// @brief Attempts to call @a function with @a args
-///
-/// @a T is expected to be returned.
-/// If `void` is specified, the returned values
-/// are ignored.
-/// `std::optional<T>` means nil|LuaEquiv<T> (or zero returns)
-/// A return type that doesn't match returns an error
-template <typename T, typename... Args>
-inline nonstd::expected_lite::expected<T, QString> tryCall(
-    const sol::protected_function &function, Args &&...args)
-{
-    sol::protected_function_result result =
-        function(std::forward<Args>(args)...);
-    if (!result.valid())
-    {
-        sol::error err = result;
-        return nonstd::expected_lite::make_unexpected(
-            QString::fromUtf8(err.what()));
-    }
-
-    if constexpr (std::is_same_v<T, void>)
-    {
-        return {};
-    }
-    else
-    {
-        if constexpr (detail::IsOptional<T>)
-        {
-            if (result.return_count() == 0)
-            {
-                return {};
-            }
-        }
-        if (result.return_count() > 1)
-        {
-            return nonstd::expected_lite::make_unexpected(
-                u"Expected one value to be returned but " %
-                QString::number(result.return_count()) %
-                u" values were returned");
-        }
-
-        try
-        {
-            if constexpr (detail::IsOptional<T>)
-            {
-                // we want to error on anything that is not nil|T,
-                // std::optional<T> in sol means "give me a T or if it does not match nullopt"
-                if (result.get_type() == sol::type::nil)
-                {
-                    return {};
-                }
-                auto ret = result.get<T>();
-
-                if (!ret)
-                {
-                    auto t = type_name<T>();
-                    return nonstd::expected_lite::make_unexpected(
-                        u"Expected " % QLatin1String(t.data(), t.size()) %
-                        u" to be returned but " %
-                        qmagicenum::enumName(result.get_type()) %
-                        u" was returned");
-                }
-                return *ret;
-            }
-            else
-            {
-                auto ret = result.get<std::optional<T>>();
-
-                if (!ret)
-                {
-                    auto t = type_name<T>();
-                    return nonstd::expected_lite::make_unexpected(
-                        u"Expected " % QLatin1String(t.data(), t.size()) %
-                        u" to be returned but " %
-                        qmagicenum::enumName(result.get_type()) %
-                        u" was returned");
-                }
-                return *ret;
-            }
-        }
-        catch (std::runtime_error &e)
-        {
-            return nonstd::expected_lite::make_unexpected(
-                QString::fromUtf8(e.what()));
-        }
-        // non other exceptions we let it explode
-    }
-}
-
-// NOLINTNEXTLINE(cppcoreguidelines-macro-usage)
-#    define SOL_STACK_FUNCTIONS(TYPE)                                      \
-        bool sol_lua_check(sol::types<TYPE>, lua_State *L, int index,      \
-                           std::function<sol::check_handler_type> handler, \
-                           sol::stack::record &tracking);                  \
-        TYPE sol_lua_get(sol::types<TYPE>, lua_State *L, int index,        \
-                         sol::stack::record &tracking);                    \
-        int sol_lua_push(sol::types<TYPE>, lua_State *L, const TYPE &value);
-
-SOL_STACK_FUNCTIONS(chatterino::lua::ThisPluginState)
-
-}  // namespace chatterino::lua
-
-SOL_STACK_FUNCTIONS(QString)
-SOL_STACK_FUNCTIONS(QStringList)
-SOL_STACK_FUNCTIONS(QByteArray)
-
-#    undef SOL_STACK_FUNCTIONS
-
-#endif

src/controllers/plugins/api/ChannelRef.cpp

@@ -1,224 +1,397 @@
 #ifdef CHATTERINO_HAVE_PLUGINS
 #    include "controllers/plugins/api/ChannelRef.hpp"
 
-#    include "Application.hpp"
 #    include "common/Channel.hpp"
 #    include "controllers/commands/CommandController.hpp"
-#    include "controllers/plugins/SolTypes.hpp"
+#    include "controllers/plugins/LuaAPI.hpp"
+#    include "controllers/plugins/LuaUtilities.hpp"
+#    include "messages/MessageBuilder.hpp"
 #    include "providers/twitch/TwitchChannel.hpp"
 #    include "providers/twitch/TwitchIrcServer.hpp"
 
-#    include <sol/sol.hpp>
+extern "C" {
+#    include <lauxlib.h>
+#    include <lua.h>
+}
 
+#    include <cassert>
 #    include <memory>
 #    include <optional>
 
 namespace chatterino::lua::api {
+// NOLINTBEGIN(*vararg)
 
-ChannelRef::ChannelRef(const std::shared_ptr<Channel> &chan)
-    : weak(chan)
+// NOLINTNEXTLINE(*-avoid-c-arrays)
+static const luaL_Reg CHANNEL_REF_METHODS[] = {
+    {"is_valid", &ChannelRef::is_valid},
+    {"get_name", &ChannelRef::get_name},
+    {"get_type", &ChannelRef::get_type},
+    {"get_display_name", &ChannelRef::get_display_name},
+    {"send_message", &ChannelRef::send_message},
+    {"add_system_message", &ChannelRef::add_system_message},
+    {"is_twitch_channel", &ChannelRef::is_twitch_channel},
+
+    // Twitch
+    {"get_room_modes", &ChannelRef::get_room_modes},
+    {"get_stream_status", &ChannelRef::get_stream_status},
+    {"get_twitch_id", &ChannelRef::get_twitch_id},
+    {"is_broadcaster", &ChannelRef::is_broadcaster},
+    {"is_mod", &ChannelRef::is_mod},
+    {"is_vip", &ChannelRef::is_vip},
+
+    // misc
+    {"__tostring", &ChannelRef::to_string},
+
+    // static
+    {"by_name", &ChannelRef::get_by_name},
+    {"by_twitch_id", &ChannelRef::get_by_twitch_id},
+    {nullptr, nullptr},
+};
+
+void ChannelRef::createMetatable(lua_State *L)
 {
+    lua::StackGuard guard(L, 1);
+
+    luaL_newmetatable(L, "c2.Channel");
+    lua_pushstring(L, "__index");
+    lua_pushvalue(L, -2);  // clone metatable
+    lua_settable(L, -3);   // metatable.__index = metatable
+
+    // Generic IWeakResource stuff
+    lua_pushstring(L, "__gc");
+    lua_pushcfunction(
+        L, (&WeakPtrUserData<UserData::Type::Channel, ChannelRef>::destroy));
+    lua_settable(L, -3);  // metatable.__gc = WeakPtrUserData<...>::destroy
+
+    luaL_setfuncs(L, CHANNEL_REF_METHODS, 0);
 }
 
-std::shared_ptr<Channel> ChannelRef::strong()
+ChannelPtr ChannelRef::getOrError(lua_State *L, bool expiredOk)
 {
-    auto c = this->weak.lock();
-    if (!c)
+    if (lua_gettop(L) < 1)
     {
-        throw std::runtime_error(
-            "Expired c2.Channel used - use c2.Channel:is_valid() to "
-            "check validity");
+        luaL_error(L, "Called c2.Channel method without a channel object");
+        return nullptr;
     }
-    return c;
-}
-
-std::shared_ptr<TwitchChannel> ChannelRef::twitch()
-{
-    auto c = std::dynamic_pointer_cast<TwitchChannel>(this->weak.lock());
-    if (!c)
+    if (lua_isuserdata(L, lua_gettop(L)) == 0)
     {
-        throw std::runtime_error(
-            "Expired or non-twitch c2.Channel used - use "
-            "c2.Channel:is_valid() and c2.Channe:is_twitch_channel()");
+        luaL_error(
+            L, "Called c2.Channel method with a non-userdata 'self' argument");
+        return nullptr;
     }
-    return c;
-}
-
-bool ChannelRef::is_valid()
-{
-    return !this->weak.expired();
-}
-
-QString ChannelRef::get_name()
-{
-    return this->strong()->getName();
-}
-
-Channel::Type ChannelRef::get_type()
-{
-    return this->strong()->getType();
-}
-
-QString ChannelRef::get_display_name()
-{
-    return this->strong()->getDisplayName();
-}
-
-void ChannelRef::send_message(QString text, sol::variadic_args va)
-{
-    bool execCommands = [&] {
-        if (va.size() >= 1)
+    // luaL_checkudata is no-return if check fails
+    auto *checked = luaL_checkudata(L, lua_gettop(L), "c2.Channel");
+    auto *data =
+        WeakPtrUserData<UserData::Type::Channel, Channel>::from(checked);
+    if (data == nullptr)
+    {
+        luaL_error(L,
+                   "Called c2.Channel method with an invalid channel pointer");
+        return nullptr;
+    }
+    lua_pop(L, 1);
+    if (data->target.expired())
+    {
+        if (!expiredOk)
         {
-            return va.get<bool>();
+            luaL_error(L,
+                       "Usage of expired c2.Channel object. Underlying "
+                       "resource was freed. Use Channel:is_valid() to check");
         }
-        return false;
-    }();
-    text = text.replace('\n', ' ');
-    auto chan = this->strong();
-    if (execCommands)
-    {
-        text = getApp()->getCommands()->execCommand(text, chan, false);
+        return nullptr;
     }
-    chan->sendMessage(text);
+    return data->target.lock();
 }
 
-void ChannelRef::add_system_message(QString text)
+std::shared_ptr<TwitchChannel> ChannelRef::getTwitchOrError(lua_State *L)
 {
-    text = text.replace('\n', ' ');
-    this->strong()->addSystemMessage(text);
-}
-
-bool ChannelRef::is_twitch_channel()
-{
-    return this->strong()->isTwitchChannel();
-}
-
-sol::table ChannelRef::get_room_modes(sol::this_state state)
-{
-    return toTable(state.L, *this->twitch()->accessRoomModes());
-}
-
-sol::table ChannelRef::get_stream_status(sol::this_state state)
-{
-    return toTable(state.L, *this->twitch()->accessStreamStatus());
-}
-
-QString ChannelRef::get_twitch_id()
-{
-    return this->twitch()->roomId();
-}
-
-bool ChannelRef::is_broadcaster()
-{
-    return this->twitch()->isBroadcaster();
-}
-
-bool ChannelRef::is_mod()
-{
-    return this->twitch()->isMod();
-}
-
-bool ChannelRef::is_vip()
-{
-    return this->twitch()->isVip();
-}
-
-QString ChannelRef::to_string()
-{
-    auto chan = this->weak.lock();
-    if (!chan)
+    auto ref = ChannelRef::getOrError(L);
+    auto ptr = dynamic_pointer_cast<TwitchChannel>(ref);
+    if (ptr == nullptr)
     {
-        return "<c2.Channel expired>";
+        luaL_error(L,
+                   "c2.Channel Twitch-only operation on non-Twitch channel.");
     }
-    return QStringView(u"<c2.Channel %1>").arg(chan->getName());
+    return ptr;
 }
 
-std::optional<ChannelRef> ChannelRef::get_by_name(const QString &name)
+int ChannelRef::is_valid(lua_State *L)
 {
-    auto chan = getApp()->getTwitch()->getChannelOrEmpty(name);
-    if (chan->isEmpty())
+    ChannelPtr that = ChannelRef::getOrError(L, true);
+    lua::push(L, that != nullptr);
+    return 1;
+}
+
+int ChannelRef::get_name(lua_State *L)
+{
+    ChannelPtr that = ChannelRef::getOrError(L);
+    lua::push(L, that->getName());
+    return 1;
+}
+
+int ChannelRef::get_type(lua_State *L)
+{
+    ChannelPtr that = ChannelRef::getOrError(L);
+    lua::push(L, that->getType());
+    return 1;
+}
+
+int ChannelRef::get_display_name(lua_State *L)
+{
+    ChannelPtr that = ChannelRef::getOrError(L);
+    lua::push(L, that->getDisplayName());
+    return 1;
+}
+
+int ChannelRef::send_message(lua_State *L)
+{
+    if (lua_gettop(L) != 2 && lua_gettop(L) != 3)
     {
-        return std::nullopt;
+        luaL_error(L, "Channel:send_message needs 1 or 2 arguments (message "
+                      "text and optionally execute_commands flag)");
+        return 0;
     }
-    return chan;
-}
-
-std::optional<ChannelRef> ChannelRef::get_by_twitch_id(const QString &id)
-{
-    auto chan = getApp()->getTwitch()->getChannelOrEmptyByID(id);
-    if (chan->isEmpty())
+    bool execcmds = false;
+    if (lua_gettop(L) == 3)
     {
-        return std::nullopt;
-    }
-    return chan;
-}
-
-void ChannelRef::createUserType(sol::table &c2)
-{
-    // clang-format off
-    c2.new_usertype<ChannelRef>(
-        "Channel", sol::no_constructor, 
-        // meta methods
-        sol::meta_method::to_string, &ChannelRef::to_string,
-
-        // Channel
-        "is_valid", &ChannelRef::is_valid,
-        "get_name",&ChannelRef::get_name,
-        "get_type", &ChannelRef::get_type,
-        "get_display_name", &ChannelRef::get_display_name,
-        "send_message", &ChannelRef::send_message,
-        "add_system_message", &ChannelRef::add_system_message,
-        "is_twitch_channel", &ChannelRef::is_twitch_channel,
-
-        // TwitchChannel
-        "get_room_modes", &ChannelRef::get_room_modes, 
-        "get_stream_status", &ChannelRef::get_stream_status,
-        "get_twitch_id", &ChannelRef::get_twitch_id,
-        "is_broadcaster", &ChannelRef::is_broadcaster,
-        "is_mod", &ChannelRef::is_mod,
-        "is_vip", &ChannelRef::is_vip,
-
-        // static
-        "by_name", &ChannelRef::get_by_name,
-        "by_twitch_id", &ChannelRef::get_by_twitch_id
-    );
-    // clang-format on
-}
-
-sol::table toTable(lua_State *L, const TwitchChannel::RoomModes &modes)
-{
-    auto maybe = [](int value) {
-        if (value >= 0)
+        if (!lua::pop(L, &execcmds))
         {
-            return std::optional{value};
+            luaL_error(L, "cannot get execute_commands (2nd argument of "
+                          "Channel:send_message)");
+            return 0;
         }
-        return std::optional<int>{};
+    }
+
+    QString text;
+    if (!lua::pop(L, &text))
+    {
+        luaL_error(L, "cannot get text (1st argument of Channel:send_message)");
+        return 0;
+    }
+
+    ChannelPtr that = ChannelRef::getOrError(L);
+
+    text = text.replace('\n', ' ');
+    if (execcmds)
+    {
+        text = getApp()->getCommands()->execCommand(text, that, false);
+    }
+    that->sendMessage(text);
+    return 0;
+}
+
+int ChannelRef::add_system_message(lua_State *L)
+{
+    // needs to account for the hidden self argument
+    if (lua_gettop(L) != 2)
+    {
+        luaL_error(
+            L, "Channel:add_system_message needs exactly 1 argument (message "
+               "text)");
+        return 0;
+    }
+
+    QString text;
+    if (!lua::pop(L, &text))
+    {
+        luaL_error(
+            L, "cannot get text (1st argument of Channel:add_system_message)");
+        return 0;
+    }
+    ChannelPtr that = ChannelRef::getOrError(L);
+    text = text.replace('\n', ' ');
+    that->addSystemMessage(text);
+    return 0;
+}
+
+int ChannelRef::is_twitch_channel(lua_State *L)
+{
+    ChannelPtr that = ChannelRef::getOrError(L);
+    lua::push(L, that->isTwitchChannel());
+    return 1;
+}
+
+int ChannelRef::get_room_modes(lua_State *L)
+{
+    auto tc = ChannelRef::getTwitchOrError(L);
+    const auto m = tc->accessRoomModes();
+    const auto modes = LuaRoomModes{
+        .unique_chat = m->r9k,
+        .subscriber_only = m->submode,
+        .emotes_only = m->emoteOnly,
+        .follower_only = (m->followerOnly == -1)
+                             ? std::nullopt
+                             : std::optional(m->followerOnly),
+        .slow_mode =
+            (m->slowMode == 0) ? std::nullopt : std::optional(m->slowMode),
+
     };
-    // clang-format off
-    return sol::table::create_with(L,
-        "subscriber_only", modes.submode,
-        "unique_chat", modes.r9k,
-        "emotes_only", modes.emoteOnly,
-        "follower_only", maybe(modes.followerOnly),
-        "slow_mode", maybe(modes.slowMode)
-    );
-    // clang-format on
+    lua::push(L, modes);
+    return 1;
 }
 
-sol::table toTable(lua_State *L, const TwitchChannel::StreamStatus &status)
+int ChannelRef::get_stream_status(lua_State *L)
 {
-    // clang-format off
-    return sol::table::create_with(L,
-        "live", status.live,
-        "viewer_count", status.viewerCount,
-        "title", status.title,
-        "game_name", status.game,
-        "game_id", status.gameId,
-        "uptime", status.uptimeSeconds
-    );
-    // clang-format on
+    auto tc = ChannelRef::getTwitchOrError(L);
+    const auto s = tc->accessStreamStatus();
+    const auto status = LuaStreamStatus{
+        .live = s->live,
+        .viewer_count = static_cast<int>(s->viewerCount),
+        .uptime = s->uptimeSeconds,
+        .title = s->title,
+        .game_name = s->game,
+        .game_id = s->gameId,
+    };
+    lua::push(L, status);
+    return 1;
 }
 
-}  // namespace chatterino::lua::api
+int ChannelRef::get_twitch_id(lua_State *L)
+{
+    auto tc = ChannelRef::getTwitchOrError(L);
+    lua::push(L, tc->roomId());
+    return 1;
+}
 
+int ChannelRef::is_broadcaster(lua_State *L)
+{
+    auto tc = ChannelRef::getTwitchOrError(L);
+    lua::push(L, tc->isBroadcaster());
+    return 1;
+}
+
+int ChannelRef::is_mod(lua_State *L)
+{
+    auto tc = ChannelRef::getTwitchOrError(L);
+    lua::push(L, tc->isMod());
+    return 1;
+}
+
+int ChannelRef::is_vip(lua_State *L)
+{
+    auto tc = ChannelRef::getTwitchOrError(L);
+    lua::push(L, tc->isVip());
+    return 1;
+}
+
+int ChannelRef::get_by_name(lua_State *L)
+{
+    if (lua_gettop(L) != 2)
+    {
+        luaL_error(L, "Channel.by_name needs exactly 2 arguments (channel "
+                      "name and platform)");
+        lua_pushnil(L);
+        return 1;
+    }
+    LPlatform platform{};
+    if (!lua::pop(L, &platform))
+    {
+        luaL_error(L, "cannot get platform (2nd argument of Channel.by_name, "
+                      "expected a string)");
+        lua_pushnil(L);
+        return 1;
+    }
+    QString name;
+    if (!lua::pop(L, &name))
+    {
+        luaL_error(L,
+                   "cannot get channel name (1st argument of Channel.by_name, "
+                   "expected a string)");
+        lua_pushnil(L);
+        return 1;
+    }
+    auto chn = getApp()->getTwitch()->getChannelOrEmpty(name);
+    lua::push(L, chn);
+    return 1;
+}
+
+int ChannelRef::get_by_twitch_id(lua_State *L)
+{
+    if (lua_gettop(L) != 1)
+    {
+        luaL_error(
+            L, "Channel.by_twitch_id needs exactly 1 arguments (channel owner "
+               "id)");
+        lua_pushnil(L);
+        return 1;
+    }
+    QString id;
+    if (!lua::pop(L, &id))
+    {
+        luaL_error(L,
+                   "cannot get channel name (1st argument of Channel.by_name, "
+                   "expected a string)");
+        lua_pushnil(L);
+        return 1;
+    }
+    auto chn = getApp()->getTwitch()->getChannelOrEmptyByID(id);
+
+    lua::push(L, chn);
+    return 1;
+}
+
+int ChannelRef::to_string(lua_State *L)
+{
+    ChannelPtr that = ChannelRef::getOrError(L, true);
+    if (that == nullptr)
+    {
+        lua_pushstring(L, "<c2.Channel expired>");
+        return 1;
+    }
+    QString formated = QString("<c2.Channel %1>").arg(that->getName());
+    lua::push(L, formated);
+    return 1;
+}
+}  // namespace chatterino::lua::api
+// NOLINTEND(*vararg)
+//
+namespace chatterino::lua {
+StackIdx push(lua_State *L, const api::LuaRoomModes &modes)
+{
+    auto out = lua::pushEmptyTable(L, 6);
+#    define PUSH(field)            \
+        lua::push(L, modes.field); \
+        lua_setfield(L, out, #field)
+    PUSH(unique_chat);
+    PUSH(subscriber_only);
+    PUSH(emotes_only);
+    PUSH(follower_only);
+    PUSH(slow_mode);
+#    undef PUSH
+    return out;
+}
+
+StackIdx push(lua_State *L, const api::LuaStreamStatus &status)
+{
+    auto out = lua::pushEmptyTable(L, 6);
+#    define PUSH(field)             \
+        lua::push(L, status.field); \
+        lua_setfield(L, out, #field)
+    PUSH(live);
+    PUSH(viewer_count);
+    PUSH(uptime);
+    PUSH(title);
+    PUSH(game_name);
+    PUSH(game_id);
+#    undef PUSH
+    return out;
+}
+
+StackIdx push(lua_State *L, ChannelPtr chn)
+{
+    using namespace chatterino::lua::api;
+
+    if (chn->isEmpty())
+    {
+        lua_pushnil(L);
+        return lua_gettop(L);
+    }
+    WeakPtrUserData<UserData::Type::Channel, Channel>::create(
+        L, chn->weak_from_this());
+    luaL_getmetatable(L, "c2.Channel");
+    lua_setmetatable(L, -2);
+    return lua_gettop(L);
+}
+
+}  // namespace chatterino::lua
 #endif

src/controllers/plugins/api/ChannelRef.hpp

@@ -1,24 +1,48 @@
 #pragma once
 #ifdef CHATTERINO_HAVE_PLUGINS
 #    include "common/Channel.hpp"
+#    include "controllers/plugins/LuaUtilities.hpp"
+#    include "controllers/plugins/PluginController.hpp"
 #    include "providers/twitch/TwitchChannel.hpp"
 
-#    include <sol/forward.hpp>
+#    include <optional>
 
 namespace chatterino::lua::api {
 // NOLINTBEGIN(readability-identifier-naming)
 
 /**
- * @includefile providers/twitch/TwitchChannel.hpp
+ * This enum describes a platform for the purpose of searching for a channel.
+ * Currently only Twitch is supported because identifying IRC channels is tricky.
+ * @exposeenum c2.Platform
  */
+enum class LPlatform {
+    Twitch,
+    //IRC,
+};
 
 /**
  * @lua@class c2.Channel
  */
 struct ChannelRef {
-public:
-    ChannelRef(const std::shared_ptr<Channel> &chan);
+    static void createMetatable(lua_State *L);
+    friend class chatterino::PluginController;
 
+    /**
+     * @brief Get the content of the top object on Lua stack, usually first argument to function as a ChannelPtr.
+     * If the object given is not a userdatum or the pointer inside that
+     * userdatum doesn't point to a Channel, a lua error is thrown.
+     *
+     * @param expiredOk Should an expired return nullptr instead of erroring
+     */
+    static ChannelPtr getOrError(lua_State *L, bool expiredOk = false);
+
+    /**
+     * @brief Casts the result of getOrError to std::shared_ptr<TwitchChannel>
+     * if that fails thows a lua error.
+     */
+    static std::shared_ptr<TwitchChannel> getTwitchOrError(lua_State *L);
+
+public:
     /**
      * Returns true if the channel this object points to is valid.
      * If the object expired, returns false
@@ -27,7 +51,7 @@ public:
      * @lua@return boolean success
      * @exposed c2.Channel:is_valid
      */
-    bool is_valid();
+    static int is_valid(lua_State *L);
 
     /**
      * Gets the channel's name. This is the lowercase login name.
@@ -35,7 +59,7 @@ public:
      * @lua@return string name
      * @exposed c2.Channel:get_name
      */
-    QString get_name();
+    static int get_name(lua_State *L);
 
     /**
      * Gets the channel's type
@@ -43,7 +67,7 @@ public:
      * @lua@return c2.ChannelType
      * @exposed c2.Channel:get_type
      */
-    Channel::Type get_type();
+    static int get_type(lua_State *L);
 
     /**
      * Get the channel owner's display name. This may contain non-lowercase ascii characters.
@@ -51,17 +75,17 @@ public:
      * @lua@return string name
      * @exposed c2.Channel:get_display_name
      */
-    QString get_display_name();
+    static int get_display_name(lua_State *L);
 
     /**
      * Sends a message to the target channel.
      * Note that this does not execute client-commands.
      *
      * @lua@param message string
-     * @lua@param execute_commands? boolean Should commands be run on the text?
+     * @lua@param execute_commands boolean Should commands be run on the text?
      * @exposed c2.Channel:send_message
      */
-    void send_message(QString text, sol::variadic_args va);
+    static int send_message(lua_State *L);
 
     /**
      * Adds a system message client-side
@@ -69,7 +93,7 @@ public:
      * @lua@param message string
      * @exposed c2.Channel:add_system_message
      */
-    void add_system_message(QString text);
+    static int add_system_message(lua_State *L);
 
     /**
      * Returns true for twitch channels.
@@ -79,7 +103,7 @@ public:
      * @lua@return boolean
      * @exposed c2.Channel:is_twitch_channel
      */
-    bool is_twitch_channel();
+    static int is_twitch_channel(lua_State *L);
 
     /**
      * Twitch Channel specific functions
@@ -91,7 +115,7 @@ public:
      * @lua@return RoomModes
      * @exposed c2.Channel:get_room_modes
      */
-    sol::table get_room_modes(sol::this_state state);
+    static int get_room_modes(lua_State *L);
 
     /**
      * Returns a copy of the stream status.
@@ -99,7 +123,7 @@ public:
      * @lua@return StreamStatus
      * @exposed c2.Channel:get_stream_status
      */
-    sol::table get_stream_status(sol::this_state state);
+    static int get_stream_status(lua_State *L);
 
     /**
      * Returns the Twitch user ID of the owner of the channel.
@@ -107,7 +131,7 @@ public:
      * @lua@return string
      * @exposed c2.Channel:get_twitch_id
      */
-    QString get_twitch_id();
+    static int get_twitch_id(lua_State *L);
 
     /**
      * Returns true if the channel is a Twitch channel and the user owns it
@@ -115,7 +139,7 @@ public:
      * @lua@return boolean
      * @exposed c2.Channel:is_broadcaster
      */
-    bool is_broadcaster();
+    static int is_broadcaster(lua_State *L);
 
     /**
      * Returns true if the channel is a Twitch channel and the user is a moderator in the channel
@@ -124,7 +148,7 @@ public:
      * @lua@return boolean
      * @exposed c2.Channel:is_mod
      */
-    bool is_mod();
+    static int is_mod(lua_State *L);
 
     /**
      * Returns true if the channel is a Twitch channel and the user is a VIP in the channel
@@ -133,7 +157,7 @@ public:
      * @lua@return boolean
      * @exposed c2.Channel:is_vip
      */
-    bool is_vip();
+    static int is_vip(lua_State *L);
 
     /**
       * Misc
@@ -143,7 +167,7 @@ public:
      * @lua@return string
      * @exposed c2.Channel:__tostring
      */
-    QString to_string();
+    static int to_string(lua_State *L);
 
     /**
      * Static functions
@@ -160,10 +184,11 @@ public:
      *  - /automod
      *
      * @lua@param name string Which channel are you looking for?
+     * @lua@param platform c2.Platform Where to search for the channel?
      * @lua@return c2.Channel?
      * @exposed c2.Channel.by_name
      */
-    static std::optional<ChannelRef> get_by_name(const QString &name);
+    static int get_by_name(lua_State *L);
 
     /**
      * Finds a channel by the Twitch user ID of its owner.
@@ -172,24 +197,79 @@ public:
      * @lua@return c2.Channel?
      * @exposed c2.Channel.by_twitch_id
      */
-    static std::optional<ChannelRef> get_by_twitch_id(const QString &id);
+    static int get_by_twitch_id(lua_State *L);
+};
 
-    static void createUserType(sol::table &c2);
+// This is a copy of the TwitchChannel::RoomModes structure, except it uses nicer optionals
+/**
+ * @lua@class RoomModes
+ */
+struct LuaRoomModes {
+    /**
+     * @lua@field unique_chat boolean You might know this as r9kbeta or robot9000.
+     */
+    bool unique_chat = false;
 
-private:
-    std::weak_ptr<Channel> weak;
+    /**
+     * @lua@field subscriber_only boolean
+     */
+    bool subscriber_only = false;
 
-    /// Locks the weak pointer and throws if the pointer expired
-    std::shared_ptr<Channel> strong();
+    /**
+     * @lua@field emotes_only boolean Whether or not text is allowed in messages. Note that "emotes" here only means Twitch emotes, not Unicode emoji, nor 3rd party text-based emotes
+     */
+    bool emotes_only = false;
 
-    /// Locks the weak pointer and throws if the pointer is invalid
-    std::shared_ptr<TwitchChannel> twitch();
+    /**
+     * @lua@field follower_only number? Time in minutes you need to follow to chat or nil.
+     */
+    std::optional<int> follower_only;
+    /**
+     * @lua@field slow_mode number? Time in seconds you need to wait before sending messages or nil.
+     */
+    std::optional<int> slow_mode;
+};
+
+/**
+ * @lua@class StreamStatus
+ */
+struct LuaStreamStatus {
+    /**
+     * @lua@field live boolean
+     */
+    bool live = false;
+
+    /**
+     * @lua@field viewer_count number
+     */
+    int viewer_count = 0;
+
+    /**
+     * @lua@field uptime number Seconds since the stream started.
+     */
+    int uptime = 0;
+
+    /**
+     * @lua@field title string Stream title or last stream title
+     */
+    QString title;
+
+    /**
+     * @lua@field game_name string
+     */
+    QString game_name;
+
+    /**
+     * @lua@field game_id string
+     */
+    QString game_id;
 };
 
 // NOLINTEND(readability-identifier-naming)
-
-sol::table toTable(lua_State *L, const TwitchChannel::RoomModes &modes);
-sol::table toTable(lua_State *L, const TwitchChannel::StreamStatus &status);
-
 }  // namespace chatterino::lua::api
+namespace chatterino::lua {
+StackIdx push(lua_State *L, const api::LuaRoomModes &modes);
+StackIdx push(lua_State *L, const api::LuaStreamStatus &status);
+StackIdx push(lua_State *L, ChannelPtr chn);
+}  // namespace chatterino::lua
 #endif

src/controllers/plugins/api/EventType.hpp

@@ -1,14 +0,0 @@
-#pragma once
-#ifdef CHATTERINO_HAVE_PLUGINS
-
-namespace chatterino::lua::api {
-
-/**
- * @exposeenum c2.EventType
- */
-enum class EventType {
-    CompletionRequested,
-};
-
-}  // namespace chatterino::lua::api
-#endif

src/controllers/plugins/api/HTTPRequest.cpp

@@ -6,173 +6,402 @@
 #    include "common/network/NetworkRequest.hpp"
 #    include "common/network/NetworkResult.hpp"
 #    include "controllers/plugins/api/HTTPResponse.hpp"
+#    include "controllers/plugins/LuaAPI.hpp"
 #    include "controllers/plugins/LuaUtilities.hpp"
-#    include "controllers/plugins/PluginController.hpp"
-#    include "controllers/plugins/SolTypes.hpp"
 #    include "util/DebugCount.hpp"
 
+extern "C" {
 #    include <lauxlib.h>
 #    include <lua.h>
-#    include <QChar>
-#    include <QLoggingCategory>
+}
 #    include <QRandomGenerator>
 #    include <QUrl>
-#    include <sol/forward.hpp>
-#    include <sol/raii.hpp>
-#    include <sol/state_view.hpp>
-#    include <sol/table.hpp>
-#    include <sol/types.hpp>
 
-#    include <optional>
-#    include <stdexcept>
+#    include <memory>
 #    include <utility>
-#    include <vector>
 
 namespace chatterino::lua::api {
+// NOLINTBEGIN(*vararg)
+// NOLINTNEXTLINE(*-avoid-c-arrays)
+static const luaL_Reg HTTP_REQUEST_METHODS[] = {
+    {"on_success", &HTTPRequest::on_success_wrap},
+    {"on_error", &HTTPRequest::on_error_wrap},
+    {"finally", &HTTPRequest::finally_wrap},
 
-void HTTPRequest::createUserType(sol::table &c2)
+    {"execute", &HTTPRequest::execute_wrap},
+    {"set_timeout", &HTTPRequest::set_timeout_wrap},
+    {"set_payload", &HTTPRequest::set_payload_wrap},
+    {"set_header", &HTTPRequest::set_header_wrap},
+    // static
+    {"create", &HTTPRequest::create},
+    {nullptr, nullptr},
+};
+
+std::shared_ptr<HTTPRequest> HTTPRequest::getOrError(lua_State *L,
+                                                     StackIdx where)
 {
-    c2.new_usertype<HTTPRequest>(                              //
-        "HTTPRequest", sol::no_constructor,                    //
-        sol::meta_method::to_string, &HTTPRequest::to_string,  //
-
-        "on_success", &HTTPRequest::on_success,  //
-        "on_error", &HTTPRequest::on_error,      //
-        "finally", &HTTPRequest::finally,        //
-
-        "set_timeout", &HTTPRequest::set_timeout,  //
-        "set_payload", &HTTPRequest::set_payload,  //
-        "set_header", &HTTPRequest::set_header,    //
-        "execute", &HTTPRequest::execute,          //
-
-        "create", &HTTPRequest::create  //
-    );
+    if (lua_gettop(L) < 1)
+    {
+        // The nullptr is there just to appease the compiler, luaL_error is no return
+        luaL_error(L, "Called c2.HTTPRequest method without a request object");
+        return nullptr;
+    }
+    if (lua_isuserdata(L, where) == 0)
+    {
+        luaL_error(
+            L,
+            "Called c2.HTTPRequest method with a non-userdata 'self' argument");
+        return nullptr;
+    }
+    // luaL_checkudata is no-return if check fails
+    auto *checked = luaL_checkudata(L, where, "c2.HTTPRequest");
+    auto *data =
+        SharedPtrUserData<UserData::Type::HTTPRequest, HTTPRequest>::from(
+            checked);
+    if (data == nullptr)
+    {
+        luaL_error(L, "Called c2.HTTPRequest method with an invalid pointer");
+        return nullptr;
+    }
+    lua_remove(L, where);
+    if (data->target == nullptr)
+    {
+        luaL_error(
+            L, "Internal error: SharedPtrUserData<UserData::Type::HTTPRequest, "
+               "HTTPRequest>::target was null. This is a Chatterino bug!");
+        return nullptr;
+    }
+    if (data->target->done)
+    {
+        luaL_error(L, "This c2.HTTPRequest has already been executed!");
+        return nullptr;
+    }
+    return data->target;
 }
 
-void HTTPRequest::on_success(sol::protected_function func)
+void HTTPRequest::createMetatable(lua_State *L)
 {
-    this->cbSuccess = std::make_optional(func);
+    lua::StackGuard guard(L, 1);
+
+    luaL_newmetatable(L, "c2.HTTPRequest");
+    lua_pushstring(L, "__index");
+    lua_pushvalue(L, -2);  // clone metatable
+    lua_settable(L, -3);   // metatable.__index = metatable
+
+    // Generic ISharedResource stuff
+    lua_pushstring(L, "__gc");
+    lua_pushcfunction(L, (&SharedPtrUserData<UserData::Type::HTTPRequest,
+                                             HTTPRequest>::destroy));
+    lua_settable(L, -3);  // metatable.__gc = SharedPtrUserData<...>::destroy
+
+    luaL_setfuncs(L, HTTP_REQUEST_METHODS, 0);
 }
 
-void HTTPRequest::on_error(sol::protected_function func)
+int HTTPRequest::on_success_wrap(lua_State *L)
 {
-    this->cbError = std::make_optional(func);
+    lua::StackGuard guard(L, -2);
+    auto ptr = HTTPRequest::getOrError(L, 1);
+    return ptr->on_success(L);
 }
 
-void HTTPRequest::set_timeout(int timeout)
+int HTTPRequest::on_success(lua_State *L)
 {
-    this->timeout_ = timeout;
+    auto top = lua_gettop(L);
+    if (top != 1)
+    {
+        return luaL_error(
+            L, "HTTPRequest:on_success needs 1 argument (a callback "
+               "that takes an HTTPResult and doesn't return anything)");
+    }
+    if (!lua_isfunction(L, top))
+    {
+        return luaL_error(
+            L, "HTTPRequest:on_success needs 1 argument (a callback "
+               "that takes an HTTPResult and doesn't return anything)");
+    }
+    auto shared = this->pushPrivate(L);
+    lua_pushvalue(L, -2);
+    lua_setfield(L, shared, "success");  // this deletes the function copy
+    lua_pop(L, 2);  // delete the table and function original
+    return 0;
 }
 
-void HTTPRequest::finally(sol::protected_function func)
+int HTTPRequest::on_error_wrap(lua_State *L)
 {
-    this->cbFinally = std::make_optional(func);
+    lua::StackGuard guard(L, -2);
+    auto ptr = HTTPRequest::getOrError(L, 1);
+    return ptr->on_error(L);
 }
 
-void HTTPRequest::set_payload(QByteArray payload)
+int HTTPRequest::on_error(lua_State *L)
 {
-    this->req_ = std::move(this->req_).payload(payload);
+    auto top = lua_gettop(L);
+    if (top != 1)
+    {
+        return luaL_error(
+            L, "HTTPRequest:on_error needs 1 argument (a callback "
+               "that takes an HTTPResult and doesn't return anything)");
+    }
+    if (!lua_isfunction(L, top))
+    {
+        return luaL_error(
+            L, "HTTPRequest:on_error needs 1 argument (a callback "
+               "that takes an HTTPResult and doesn't return anything)");
+    }
+    auto shared = this->pushPrivate(L);
+    lua_pushvalue(L, -2);
+    lua_setfield(L, shared, "error");  // this deletes the function copy
+    lua_pop(L, 2);                     // delete the table and function original
+    return 0;
 }
 
-// name and value may be random bytes
-void HTTPRequest::set_header(QByteArray name, QByteArray value)
+int HTTPRequest::set_timeout_wrap(lua_State *L)
 {
-    this->req_ = std::move(this->req_).header(name, value);
+    lua::StackGuard guard(L, -2);
+    auto ptr = HTTPRequest::getOrError(L, 1);
+    return ptr->set_timeout(L);
 }
 
-std::shared_ptr<HTTPRequest> HTTPRequest::create(sol::this_state L,
-                                                 NetworkRequestType method,
-                                                 QString url)
+int HTTPRequest::set_timeout(lua_State *L)
 {
+    auto top = lua_gettop(L);
+    if (top != 1)
+    {
+        return luaL_error(
+            L, "HTTPRequest:set_timeout needs 1 argument (a number of "
+               "milliseconds after which the request will time out)");
+    }
+
+    int temporary = -1;
+    if (!lua::pop(L, &temporary))
+    {
+        return luaL_error(
+            L, "HTTPRequest:set_timeout failed to get timeout, expected a "
+               "positive integer");
+    }
+    if (temporary <= 0)
+    {
+        return luaL_error(
+            L, "HTTPRequest:set_timeout failed to get timeout, expected a "
+               "positive integer");
+    }
+    this->timeout_ = temporary;
+    return 0;
+}
+
+int HTTPRequest::finally_wrap(lua_State *L)
+{
+    lua::StackGuard guard(L, -2);
+    auto ptr = HTTPRequest::getOrError(L, 1);
+    return ptr->finally(L);
+}
+
+int HTTPRequest::finally(lua_State *L)
+{
+    auto top = lua_gettop(L);
+    if (top != 1)
+    {
+        return luaL_error(L, "HTTPRequest:finally needs 1 argument (a callback "
+                             "that takes nothing and doesn't return anything)");
+    }
+    if (!lua_isfunction(L, top))
+    {
+        return luaL_error(L, "HTTPRequest:finally needs 1 argument (a callback "
+                             "that takes nothing and doesn't return anything)");
+    }
+    auto shared = this->pushPrivate(L);
+    lua_pushvalue(L, -2);
+    lua_setfield(L, shared, "finally");  // this deletes the function copy
+    lua_pop(L, 2);  // delete the table and function original
+    return 0;
+}
+
+int HTTPRequest::set_payload_wrap(lua_State *L)
+{
+    lua::StackGuard guard(L, -2);
+    auto ptr = HTTPRequest::getOrError(L, 1);
+    return ptr->set_payload(L);
+}
+
+int HTTPRequest::set_payload(lua_State *L)
+{
+    auto top = lua_gettop(L);
+    if (top != 1)
+    {
+        return luaL_error(
+            L, "HTTPRequest:set_payload needs 1 argument (a string payload)");
+    }
+
+    std::string temporary;
+    if (!lua::pop(L, &temporary))
+    {
+        return luaL_error(
+            L, "HTTPRequest:set_payload failed to get payload, expected a "
+               "string");
+    }
+    this->req_ =
+        std::move(this->req_).payload(QByteArray::fromStdString(temporary));
+    return 0;
+}
+
+int HTTPRequest::set_header_wrap(lua_State *L)
+{
+    lua::StackGuard guard(L, -3);
+    auto ptr = HTTPRequest::getOrError(L, 1);
+    return ptr->set_header(L);
+}
+
+int HTTPRequest::set_header(lua_State *L)
+{
+    auto top = lua_gettop(L);
+    if (top != 2)
+    {
+        return luaL_error(
+            L, "HTTPRequest:set_header needs 2 arguments (a header name "
+               "and a value)");
+    }
+
+    std::string value;
+    if (!lua::pop(L, &value))
+    {
+        return luaL_error(
+            L, "cannot get value (2nd argument of HTTPRequest:set_header)");
+    }
+    std::string name;
+    if (!lua::pop(L, &name))
+    {
+        return luaL_error(
+            L, "cannot get name (1st argument of HTTPRequest:set_header)");
+    }
+    this->req_ = std::move(this->req_)
+                     .header(QByteArray::fromStdString(name),
+                             QByteArray::fromStdString(value));
+    return 0;
+}
+
+int HTTPRequest::create(lua_State *L)
+{
+    lua::StackGuard guard(L, -1);
+    if (lua_gettop(L) != 2)
+    {
+        return luaL_error(
+            L, "HTTPRequest.create needs exactly 2 arguments (method "
+               "and url)");
+    }
+    QString url;
+    if (!lua::pop(L, &url))
+    {
+        return luaL_error(L,
+                          "cannot get url (2nd argument of HTTPRequest.create, "
+                          "expected a string)");
+    }
     auto parsedurl = QUrl(url);
     if (!parsedurl.isValid())
     {
-        throw std::runtime_error(
-            "cannot parse url (2nd argument of HTTPRequest.create, "
-            "got invalid url in argument)");
+        return luaL_error(
+            L, "cannot parse url (2nd argument of HTTPRequest.create, "
+               "got invalid url in argument)");
+    }
+    NetworkRequestType method{};
+    if (!lua::pop(L, &method))
+    {
+        return luaL_error(
+            L, "cannot get method (1st argument of HTTPRequest.create, "
+               "expected a string)");
     }
     auto *pl = getApp()->getPlugins()->getPluginByStatePtr(L);
     if (!pl->hasHTTPPermissionFor(parsedurl))
     {
-        throw std::runtime_error(
-            "Plugin does not have permission to send HTTP requests "
-            "to this URL");
+        return luaL_error(
+            L, "Plugin does not have permission to send HTTP requests "
+               "to this URL");
     }
     NetworkRequest r(parsedurl, method);
-    return std::make_shared<HTTPRequest>(ConstructorAccessTag{}, std::move(r));
+    lua::push(
+        L, std::make_shared<HTTPRequest>(ConstructorAccessTag{}, std::move(r)));
+    return 1;
 }
 
-void HTTPRequest::execute(sol::this_state L)
+int HTTPRequest::execute_wrap(lua_State *L)
 {
-    if (this->done)
-    {
-        throw std::runtime_error(
-            "Cannot execute this c2.HTTPRequest, it was executed already!");
-    }
+    auto ptr = HTTPRequest::getOrError(L, 1);
+    return ptr->execute(L);
+}
+
+int HTTPRequest::execute(lua_State *L)
+{
+    auto shared = this->shared_from_this();
     this->done = true;
-
-    // this keeps the object alive even if Lua were to forget about it,
-    auto hack = this->weak_from_this();
-    auto *pl = getApp()->getPlugins()->getPluginByStatePtr(L);
-    pl->httpRequests.push_back(this->shared_from_this());
-
     std::move(this->req_)
-        .onSuccess([L, hack](const NetworkResult &res) {
-            auto self = hack.lock();
-            if (!self)
-            {
-                return;
-            }
-            if (!self->cbSuccess.has_value())
-            {
-                return;
-            }
+        .onSuccess([shared, L](const NetworkResult &res) {
             lua::StackGuard guard(L);
-            (*self->cbSuccess)(HTTPResponse(res));
-            self->cbSuccess = std::nullopt;
-        })
-        .onError([L, hack](const NetworkResult &res) {
-            auto self = hack.lock();
-            if (!self)
-            {
-                return;
-            }
-            if (!self->cbError.has_value())
-            {
-                return;
-            }
-            lua::StackGuard guard(L);
-            (*self->cbError)(HTTPResponse(res));
-            self->cbError = std::nullopt;
-        })
-        .finally([L, hack]() {
-            auto self = hack.lock();
-            if (!self)
-            {
-                // this could happen if the plugin was deleted
-                return;
-            }
-            auto *pl = getApp()->getPlugins()->getPluginByStatePtr(L);
-            for (auto it = pl->httpRequests.begin();
-                 it < pl->httpRequests.end(); it++)
-            {
-                if (*it == self)
-                {
-                    pl->httpRequests.erase(it);
-                    break;
-                }
-            }
+            auto *thread = lua_newthread(L);
 
-            if (!self->cbFinally.has_value())
+            auto priv = shared->pushPrivate(thread);
+            lua_getfield(thread, priv, "success");
+            auto cb = lua_gettop(thread);
+            if (lua_isfunction(thread, cb))
             {
-                return;
+                lua::push(thread, std::make_shared<HTTPResponse>(res));
+                // one arg, no return, no msgh
+                lua_pcall(thread, 1, 0, 0);
             }
+            else
+            {
+                lua_pop(thread, 1);  // remove callback
+            }
+            lua_closethread(thread, nullptr);
+            lua_pop(L, 1);  // remove thread from L
+        })
+        .onError([shared, L](const NetworkResult &res) {
             lua::StackGuard guard(L);
-            (*self->cbFinally)();
-            self->cbFinally = std::nullopt;
+            auto *thread = lua_newthread(L);
+
+            auto priv = shared->pushPrivate(thread);
+            lua_getfield(thread, priv, "error");
+            auto cb = lua_gettop(thread);
+            if (lua_isfunction(thread, cb))
+            {
+                lua::push(thread, std::make_shared<HTTPResponse>(res));
+                // one arg, no return, no msgh
+                lua_pcall(thread, 1, 0, 0);
+            }
+            else
+            {
+                lua_pop(thread, 1);  // remove callback
+            }
+            lua_closethread(thread, nullptr);
+            lua_pop(L, 1);  // remove thread from L
+        })
+        .finally([shared, L]() {
+            lua::StackGuard guard(L);
+            auto *thread = lua_newthread(L);
+
+            auto priv = shared->pushPrivate(thread);
+            lua_getfield(thread, priv, "finally");
+            auto cb = lua_gettop(thread);
+            if (lua_isfunction(thread, cb))
+            {
+                // no args, no return, no msgh
+                lua_pcall(thread, 0, 0, 0);
+            }
+            else
+            {
+                lua_pop(thread, 1);  // remove callback
+            }
+            // remove our private data
+            lua_pushnil(thread);
+            lua_setfield(thread, LUA_REGISTRYINDEX,
+                         shared->privateKey.toStdString().c_str());
+            lua_closethread(thread, nullptr);
+            lua_pop(L, 1);  // remove thread from L
+
+            // we removed our private table, forget the key for it
+            shared->privateKey = QString();
         })
         .timeout(this->timeout_)
         .execute();
+    return 0;
 }
 
 HTTPRequest::HTTPRequest(HTTPRequest::ConstructorAccessTag /*ignored*/,
@@ -189,10 +418,34 @@ HTTPRequest::~HTTPRequest()
     // but that's better than accessing a possibly invalid lua_State pointer.
 }
 
-QString HTTPRequest::to_string()
+StackIdx HTTPRequest::pushPrivate(lua_State *L)
 {
-    return "<HTTPRequest>";
+    if (this->privateKey.isEmpty())
+    {
+        this->privateKey = QString("HTTPRequestPrivate%1")
+                               .arg(QRandomGenerator::system()->generate());
+        pushEmptyTable(L, 4);
+        lua_setfield(L, LUA_REGISTRYINDEX,
+                     this->privateKey.toStdString().c_str());
+    }
+    lua_getfield(L, LUA_REGISTRYINDEX, this->privateKey.toStdString().c_str());
+    return lua_gettop(L);
 }
 
+// NOLINTEND(*vararg)
 }  // namespace chatterino::lua::api
+
+namespace chatterino::lua {
+
+StackIdx push(lua_State *L, std::shared_ptr<api::HTTPRequest> request)
+{
+    using namespace chatterino::lua::api;
+
+    SharedPtrUserData<UserData::Type::HTTPRequest, HTTPRequest>::create(
+        L, std::move(request));
+    luaL_getmetatable(L, "c2.HTTPRequest");
+    lua_setmetatable(L, -2);
+    return lua_gettop(L);
+}
+}  // namespace chatterino::lua
 #endif

src/controllers/plugins/api/HTTPRequest.hpp

@@ -2,25 +2,19 @@
 #ifdef CHATTERINO_HAVE_PLUGINS
 #    include "common/network/NetworkRequest.hpp"
 #    include "controllers/plugins/LuaUtilities.hpp"
-
-#    include <sol/forward.hpp>
-#    include <sol/types.hpp>
+#    include "controllers/plugins/PluginController.hpp"
 
 #    include <memory>
 
-namespace chatterino {
-class PluginController;
-}  // namespace chatterino
-
 namespace chatterino::lua::api {
 // NOLINTBEGIN(readability-identifier-naming)
 
 /**
- * @lua@alias c2.HTTPCallback fun(result: c2.HTTPResponse): nil
+ * @lua@alias HTTPCallback fun(result: HTTPResponse): nil
  */
 
 /**
- * @lua@class c2.HTTPRequest
+ * @lua@class HTTPRequest
  */
 class HTTPRequest : public std::enable_shared_from_this<HTTPRequest>
 {
@@ -39,19 +33,33 @@ public:
 private:
     NetworkRequest req_;
 
-    static void createUserType(sol::table &c2);
+    static void createMetatable(lua_State *L);
     friend class chatterino::PluginController;
 
+    /**
+     * @brief Get the content of the top object on Lua stack, usually the first argument as an HTTPRequest
+     *
+     * If the object given is not a userdatum or the pointer inside that
+     * userdatum doesn't point to a HTTPRequest, a lua error is thrown.
+     *
+     * This function always returns a non-null pointer.
+     */
+    static std::shared_ptr<HTTPRequest> getOrError(lua_State *L,
+                                                   StackIdx where = -1);
+    /**
+     * Pushes the private table onto the lua stack.
+     *
+     * This might create it if it doesn't exist.
+     */
+    StackIdx pushPrivate(lua_State *L);
+
     // This is the key in the registry the private table it held at (if it exists)
     // This might be a null QString if the request has already been executed or
     // the table wasn't created yet.
+    QString privateKey;
     int timeout_ = 10'000;
     bool done = false;
 
-    std::optional<sol::protected_function> cbSuccess;
-    std::optional<sol::protected_function> cbError;
-    std::optional<sol::protected_function> cbFinally;
-
 public:
     // These functions are wrapped so data can be accessed more easily. When a call from Lua comes in:
     //  - the static wrapper function is called
@@ -61,63 +69,65 @@ public:
     /**
      * Sets the success callback
      *
-     * @lua@param callback c2.HTTPCallback Function to call when the HTTP request succeeds
-     * @exposed c2.HTTPRequest:on_success
+     * @lua@param callback HTTPCallback Function to call when the HTTP request succeeds
+     * @exposed HTTPRequest:on_success
      */
-    void on_success(sol::protected_function func);
+    static int on_success_wrap(lua_State *L);
+    int on_success(lua_State *L);
 
     /**
      * Sets the failure callback
      *
-     * @lua@param callback c2.HTTPCallback Function to call when the HTTP request fails or returns a non-ok status
-     * @exposed c2.HTTPRequest:on_error
+     * @lua@param callback HTTPCallback Function to call when the HTTP request fails or returns a non-ok status
+     * @exposed HTTPRequest:on_error
      */
-    void on_error(sol::protected_function func);
+    static int on_error_wrap(lua_State *L);
+    int on_error(lua_State *L);
 
     /**
      * Sets the finally callback
      *
      * @lua@param callback fun(): nil Function to call when the HTTP request finishes
-     * @exposed c2.HTTPRequest:finally
+     * @exposed HTTPRequest:finally
      */
-    void finally(sol::protected_function func);
+    static int finally_wrap(lua_State *L);
+    int finally(lua_State *L);
 
     /**
      * Sets the timeout
      *
      * @lua@param timeout integer How long in milliseconds until the times out
-     * @exposed c2.HTTPRequest:set_timeout
+     * @exposed HTTPRequest:set_timeout
      */
-    void set_timeout(int timeout);
+    static int set_timeout_wrap(lua_State *L);
+    int set_timeout(lua_State *L);
 
     /**
      * Sets the request payload
      *
      * @lua@param data string
-     * @exposed c2.HTTPRequest:set_payload
+     * @exposed HTTPRequest:set_payload
      */
-    void set_payload(QByteArray payload);
+    static int set_payload_wrap(lua_State *L);
+    int set_payload(lua_State *L);
 
     /**
      * Sets a header in the request
      *
      * @lua@param name string
      * @lua@param value string
-     * @exposed c2.HTTPRequest:set_header
+     * @exposed HTTPRequest:set_header
      */
-    void set_header(QByteArray name, QByteArray value);
+    static int set_header_wrap(lua_State *L);
+    int set_header(lua_State *L);
 
     /**
      * Executes the HTTP request
      *
-     * @exposed c2.HTTPRequest:execute
+     * @exposed HTTPRequest:execute
      */
-    void execute(sol::this_state L);
-    /**
-     * @lua@return string
-     * @exposed c2.HTTPRequest:__tostring
-     */
-    QString to_string();
+    static int execute_wrap(lua_State *L);
+    int execute(lua_State *L);
 
     /**
      * Static functions
@@ -126,15 +136,13 @@ public:
     /**
      * Creates a new HTTPRequest
      *
-     * @lua@param method c2.HTTPMethod Method to use
+     * @lua@param method HTTPMethod Method to use
      * @lua@param url string Where to send the request to
      *
-     * @lua@return c2.HTTPRequest
-     * @exposed c2.HTTPRequest.create
+     * @lua@return HTTPRequest
+     * @exposed HTTPRequest.create
      */
-    static std::shared_ptr<HTTPRequest> create(sol::this_state L,
-                                               NetworkRequestType method,
-                                               QString url);
+    static int create(lua_State *L);
 };
 
 // NOLINTEND(readability-identifier-naming)

src/controllers/plugins/api/HTTPResponse.cpp

@@ -2,28 +2,77 @@
 #    include "controllers/plugins/api/HTTPResponse.hpp"
 
 #    include "common/network/NetworkResult.hpp"
-#    include "controllers/plugins/SolTypes.hpp"
+#    include "controllers/plugins/LuaAPI.hpp"
 #    include "util/DebugCount.hpp"
 
+extern "C" {
 #    include <lauxlib.h>
-#    include <sol/raii.hpp>
-#    include <sol/types.hpp>
-
+}
 #    include <utility>
 
 namespace chatterino::lua::api {
+// NOLINTBEGIN(*vararg)
+// NOLINTNEXTLINE(*-avoid-c-arrays)
+static const luaL_Reg HTTP_RESPONSE_METHODS[] = {
+    {"data", &HTTPResponse::data_wrap},
+    {"status", &HTTPResponse::status_wrap},
+    {"error", &HTTPResponse::error_wrap},
+    {nullptr, nullptr},
+};
 
-void HTTPResponse::createUserType(sol::table &c2)
+void HTTPResponse::createMetatable(lua_State *L)
 {
-    c2.new_usertype<HTTPResponse>(  //
-        "HTTPResponse", sol::no_constructor,
-        // metamethods
-        sol::meta_method::to_string, &HTTPResponse::to_string,  //
+    lua::StackGuard guard(L, 1);
 
-        "data", &HTTPResponse::data,      //
-        "status", &HTTPResponse::status,  //
-        "error", &HTTPResponse::error     //
-    );
+    luaL_newmetatable(L, "c2.HTTPResponse");
+    lua_pushstring(L, "__index");
+    lua_pushvalue(L, -2);  // clone metatable
+    lua_settable(L, -3);   // metatable.__index = metatable
+
+    // Generic ISharedResource stuff
+    lua_pushstring(L, "__gc");
+    lua_pushcfunction(L, (&SharedPtrUserData<UserData::Type::HTTPResponse,
+                                             HTTPResponse>::destroy));
+    lua_settable(L, -3);  // metatable.__gc = SharedPtrUserData<...>::destroy
+
+    luaL_setfuncs(L, HTTP_RESPONSE_METHODS, 0);
+}
+
+std::shared_ptr<HTTPResponse> HTTPResponse::getOrError(lua_State *L,
+                                                       StackIdx where)
+{
+    if (lua_gettop(L) < 1)
+    {
+        // The nullptr is there just to appease the compiler, luaL_error is no return
+        luaL_error(L, "Called c2.HTTPResponse method without a request object");
+        return nullptr;
+    }
+    if (lua_isuserdata(L, where) == 0)
+    {
+        luaL_error(L, "Called c2.HTTPResponse method with a non-userdata "
+                      "'self' argument");
+        return nullptr;
+    }
+    // luaL_checkudata is no-return if check fails
+    auto *checked = luaL_checkudata(L, where, "c2.HTTPResponse");
+    auto *data =
+        SharedPtrUserData<UserData::Type::HTTPResponse, HTTPResponse>::from(
+            checked);
+    if (data == nullptr)
+    {
+        luaL_error(L, "Called c2.HTTPResponse method with an invalid pointer");
+        return nullptr;
+    }
+    lua_remove(L, where);
+    if (data->target == nullptr)
+    {
+        luaL_error(
+            L,
+            "Internal error: SharedPtrUserData<UserData::Type::HTTPResponse, "
+            "HTTPResponse>::target was null. This is a Chatterino bug!");
+        return nullptr;
+    }
+    return data->target;
 }
 
 HTTPResponse::HTTPResponse(NetworkResult res)
@@ -36,30 +85,60 @@ HTTPResponse::~HTTPResponse()
     DebugCount::decrease("lua::api::HTTPResponse");
 }
 
-QByteArray HTTPResponse::data()
+int HTTPResponse::data_wrap(lua_State *L)
 {
-    return this->result_.getData();
+    lua::StackGuard guard(L, 0);  // 1 in, 1 out
+    auto ptr = HTTPResponse::getOrError(L, 1);
+    return ptr->data(L);
 }
 
-std::optional<int> HTTPResponse::status()
+int HTTPResponse::data(lua_State *L)
 {
-    return this->result_.status();
+    lua::push(L, this->result_.getData().toStdString());
+    return 1;
 }
 
-QString HTTPResponse::error()
+int HTTPResponse::status_wrap(lua_State *L)
 {
-    return this->result_.formatError();
+    lua::StackGuard guard(L, 0);  // 1 in, 1 out
+    auto ptr = HTTPResponse::getOrError(L, 1);
+    return ptr->status(L);
 }
 
-QString HTTPResponse::to_string()
+int HTTPResponse::status(lua_State *L)
 {
-    if (this->status().has_value())
-    {
-        return QStringView(u"<c2.HTTPResponse status %1>")
-            .arg(QString::number(*this->status()));
-    }
-    return "<c2.HTTPResponse no status>";
+    lua::push(L, this->result_.status());
+    return 1;
 }
 
+int HTTPResponse::error_wrap(lua_State *L)
+{
+    lua::StackGuard guard(L, 0);  // 1 in, 1 out
+    auto ptr = HTTPResponse::getOrError(L, 1);
+    return ptr->error(L);
+}
+
+int HTTPResponse::error(lua_State *L)
+{
+    lua::push(L, this->result_.formatError());
+    return 1;
+}
+
+// NOLINTEND(*vararg)
 }  // namespace chatterino::lua::api
+
+namespace chatterino::lua {
+StackIdx push(lua_State *L, std::shared_ptr<api::HTTPResponse> request)
+{
+    using namespace chatterino::lua::api;
+
+    // Prepare table
+    SharedPtrUserData<UserData::Type::HTTPResponse, HTTPResponse>::create(
+        L, std::move(request));
+    luaL_getmetatable(L, "c2.HTTPResponse");
+    lua_setmetatable(L, -2);
+
+    return lua_gettop(L);
+}
+}  // namespace chatterino::lua
 #endif

src/controllers/plugins/api/HTTPResponse.hpp

@@ -1,11 +1,12 @@
 #pragma once
 #ifdef CHATTERINO_HAVE_PLUGINS
 #    include "common/network/NetworkResult.hpp"
-
-#    include <lua.h>
-#    include <sol/sol.hpp>
+#    include "controllers/plugins/LuaUtilities.hpp"
 
 #    include <memory>
+extern "C" {
+#    include <lua.h>
+}
 
 namespace chatterino {
 class PluginController;
@@ -15,9 +16,9 @@ namespace chatterino::lua::api {
 // NOLINTBEGIN(readability-identifier-naming)
 
 /**
- * @lua@class c2.HTTPResponse
+ * @lua@class HTTPResponse
  */
-class HTTPResponse
+class HTTPResponse : public std::enable_shared_from_this<HTTPResponse>
 {
     NetworkResult result_;
 
@@ -30,46 +31,50 @@ public:
     ~HTTPResponse();
 
 private:
-    static void createUserType(sol::table &c2);
+    static void createMetatable(lua_State *L);
     friend class chatterino::PluginController;
 
+    /**
+     * @brief Get the content of the top object on Lua stack, usually the first argument as an HTTPResponse
+     *
+     * If the object given is not a userdatum or the pointer inside that
+     * userdatum doesn't point to a HTTPResponse, a lua error is thrown.
+     *
+     * This function always returns a non-null pointer.
+     */
+    static std::shared_ptr<HTTPResponse> getOrError(lua_State *L,
+                                                    StackIdx where = -1);
+
 public:
     /**
      * Returns the data. This is not guaranteed to be encoded using any
      * particular encoding scheme. It's just the bytes the server returned.
      * 
-     * @lua@return string
-     * @lua@nodiscard
-     * @exposed c2.HTTPResponse:data
+     * @exposed HTTPResponse:data
      */
-    QByteArray data();
+    static int data_wrap(lua_State *L);
+    int data(lua_State *L);
 
     /**
      * Returns the status code.
      *
-     * @lua@return number|nil
-     * @lua@nodiscard
-     * @exposed c2.HTTPResponse:status
+     * @exposed HTTPResponse:status
      */
-    std::optional<int> status();
+    static int status_wrap(lua_State *L);
+    int status(lua_State *L);
 
     /**
      * A somewhat human readable description of an error if such happened
-     *
-     * @lua@return string
-     * @lua@nodiscard
-     * @exposed c2.HTTPResponse:error
+     * @exposed HTTPResponse:error
      */
-    QString error();
 
-    /**
-     * @lua@return string
-     * @lua@nodiscard
-     * @exposed c2.HTTPResponse:__tostring
-     */
-    QString to_string();
+    static int error_wrap(lua_State *L);
+    int error(lua_State *L);
 };
 
 // NOLINTEND(readability-identifier-naming)
 }  // namespace chatterino::lua::api
+namespace chatterino::lua {
+StackIdx push(lua_State *L, std::shared_ptr<api::HTTPResponse> request);
+}  // namespace chatterino::lua
 #endif

src/controllers/plugins/api/IOWrapper.cpp

@@ -2,17 +2,15 @@
 #    include "controllers/plugins/api/IOWrapper.hpp"
 
 #    include "Application.hpp"
-#    include "common/QLogging.hpp"
+#    include "controllers/plugins/LuaUtilities.hpp"
 #    include "controllers/plugins/PluginController.hpp"
 
+extern "C" {
 #    include <lauxlib.h>
 #    include <lua.h>
-#    include <QString>
-#    include <sol/sol.hpp>
+}
 
 #    include <cerrno>
-#    include <stdexcept>
-#    include <utility>
 
 namespace chatterino::lua::api {
 
@@ -93,28 +91,45 @@ struct LuaFileMode {
     }
 };
 
-sol::variadic_results ioError(lua_State *L, const QString &value,
-                              int errnoequiv)
+int ioError(lua_State *L, const QString &value, int errnoequiv)
 {
-    sol::variadic_results out;
-    out.push_back(sol::nil);
-    out.push_back(sol::make_object(L, value.toStdString()));
-    out.push_back({L, sol::in_place_type<int>, errnoequiv});
-    return out;
+    lua_pushnil(L);
+    lua::push(L, value);
+    lua::push(L, errnoequiv);
+    return 3;
 }
 
-sol::variadic_results io_open(sol::this_state L, QString filename,
-                              QString strmode)
+// NOLINTBEGIN(*vararg)
+int io_open(lua_State *L)
 {
     auto *pl = getApp()->getPlugins()->getPluginByStatePtr(L);
     if (pl == nullptr)
     {
-        throw std::runtime_error("internal error: no plugin");
+        luaL_error(L, "internal error: no plugin");
+        return 0;
     }
-    LuaFileMode mode(strmode);
-    if (!mode.error.isEmpty())
+    LuaFileMode mode;
+    if (lua_gettop(L) == 2)
     {
-        throw std::runtime_error(mode.error.toStdString());
+        // we have a mode
+        QString smode;
+        if (!lua::pop(L, &smode))
+        {
+            return luaL_error(
+                L,
+                "io.open mode (2nd argument) must be a string or not present");
+        }
+        mode = LuaFileMode(smode);
+        if (!mode.error.isEmpty())
+        {
+            return luaL_error(L, mode.error.toStdString().c_str());
+        }
+    }
+    QString filename;
+    if (!lua::pop(L, &filename))
+    {
+        return luaL_error(L,
+                          "io.open filename (1st argument) must be a string");
     }
     QFileInfo file(pl->dataDirectory().filePath(filename));
     auto abs = file.absoluteFilePath();
@@ -129,35 +144,39 @@ sol::variadic_results io_open(sol::this_state L, QString filename,
                        "Plugin does not have permissions to access given file.",
                        EACCES);
     }
-
-    sol::state_view lua(L);
-    auto open = lua.registry()[REG_REAL_IO_NAME]["open"];
-    sol::protected_function_result res =
-        open(abs.toStdString(), mode.toString().toStdString());
-    return res;
-}
-sol::variadic_results io_open_modeless(sol::this_state L, QString filename)
-{
-    return io_open(L, std::move(filename), "r");
+    lua_getfield(L, LUA_REGISTRYINDEX, REG_REAL_IO_NAME);
+    lua_getfield(L, -1, "open");
+    lua_remove(L, -2);  // remove LUA_REGISTRYINDEX[REAL_IO_NAME]
+    lua::push(L, abs);
+    lua::push(L, mode.toString());
+    lua_call(L, 2, 3);
+    return 3;
 }
 
-sol::variadic_results io_lines_noargs(sol::this_state L)
-{
-    sol::state_view lua(L);
-    auto lines = lua.registry()[REG_REAL_IO_NAME]["lines"];
-    sol::protected_function_result res = lines();
-    return res;
-}
-
-sol::variadic_results io_lines(sol::this_state L, QString filename,
-                               sol::variadic_args args)
+int io_lines(lua_State *L)
 {
     auto *pl = getApp()->getPlugins()->getPluginByStatePtr(L);
     if (pl == nullptr)
     {
-        throw std::runtime_error("internal error: no plugin");
+        luaL_error(L, "internal error: no plugin");
+        return 0;
+    }
+    if (lua_gettop(L) == 0)
+    {
+        // io.lines() case, just call realio.lines
+        lua_getfield(L, LUA_REGISTRYINDEX, REG_REAL_IO_NAME);
+        lua_getfield(L, -1, "lines");
+        lua_remove(L, -2);  // remove LUA_REGISTRYINDEX[REAL_IO_NAME]
+        lua_call(L, 0, 1);
+        return 1;
+    }
+    QString filename;
+    if (!lua::pop(L, &filename))
+    {
+        return luaL_error(
+            L,
+            "io.lines filename (1st argument) must be a string or not present");
     }
-    sol::state_view lua(L);
     QFileInfo file(pl->dataDirectory().filePath(filename));
     auto abs = file.absoluteFilePath();
     qCDebug(chatterinoLua) << "[" << pl->id << ":" << pl->meta.name
@@ -166,168 +185,191 @@ sol::variadic_results io_lines(sol::this_state L, QString filename,
     bool ok = pl->hasFSPermissionFor(false, abs);
     if (!ok)
     {
-        throw std::runtime_error(
-            "Plugin does not have permissions to access given file.");
+        return ioError(L,
+                       "Plugin does not have permissions to access given file.",
+                       EACCES);
     }
+    // Our stack looks like this:
+    // - {...}[1]
+    // - {...}[2]
+    // ...
+    // We want:
+    // - REG[REG_REAL_IO_NAME].lines
+    // - absolute file path
+    // - {...}[1]
+    // - {...}[2]
+    // ...
 
-    auto lines = lua.registry()[REG_REAL_IO_NAME]["lines"];
-    sol::protected_function_result res = lines(abs.toStdString(), args);
-    return res;
+    lua_getfield(L, LUA_REGISTRYINDEX, REG_REAL_IO_NAME);
+    lua_getfield(L, -1, "lines");
+    lua_remove(L, -2);  // remove LUA_REGISTRYINDEX[REAL_IO_NAME]
+    lua_insert(L, 1);   // move function to start of stack
+    lua::push(L, abs);
+    lua_insert(L, 2);  // move file name just after the function
+    lua_call(L, lua_gettop(L) - 1, LUA_MULTRET);
+    return lua_gettop(L);
 }
 
-sol::variadic_results io_input_argless(sol::this_state L)
-{
-    auto *pl = getApp()->getPlugins()->getPluginByStatePtr(L);
-    if (pl == nullptr)
+namespace {
+
+    // This is the code for both io.input and io.output
+    int globalFileCommon(lua_State *L, bool output)
     {
-        throw std::runtime_error("internal error: no plugin");
-    }
-    sol::state_view lua(L);
+        auto *pl = getApp()->getPlugins()->getPluginByStatePtr(L);
+        if (pl == nullptr)
+        {
+            luaL_error(L, "internal error: no plugin");
+            return 0;
+        }
+        // Three signature cases:
+        // io.input()
+        // io.input(file)
+        // io.input(name)
+        if (lua_gettop(L) == 0)
+        {
+            // We have no arguments, call realio.input()
+            lua_getfield(L, LUA_REGISTRYINDEX, REG_REAL_IO_NAME);
+            if (output)
+            {
+                lua_getfield(L, -1, "output");
+            }
+            else
+            {
+                lua_getfield(L, -1, "input");
+            }
+            lua_remove(L, -2);  // remove LUA_REGISTRYINDEX[REAL_IO_NAME]
+            lua_call(L, 0, 1);
+            return 1;
+        }
+        if (lua_gettop(L) != 1)
+        {
+            return luaL_error(L, "Too many arguments given to io.input().");
+        }
+        // Now check if we have a file or name
+        auto *p = luaL_testudata(L, 1, LUA_FILEHANDLE);
+        if (p == nullptr)
+        {
+            // this is not a file handle, send it to open
+            luaL_getsubtable(L, LUA_REGISTRYINDEX, REG_C2_IO_NAME);
+            lua_getfield(L, -1, "open");
+            lua_remove(L, -2);  // remove io
 
-    auto func = lua.registry()[REG_REAL_IO_NAME]["input"];
-    sol::protected_function_result res = func();
-    return res;
-}
-sol::variadic_results io_input_file(sol::this_state L, sol::userdata file)
+            lua_pushvalue(L, 1);  // dupe arg
+            if (output)
+            {
+                lua_pushstring(L, "w");
+            }
+            else
+            {
+                lua_pushstring(L, "r");
+            }
+            lua_call(L, 2, 1);  // call ourio.open(arg1, 'r'|'w')
+            // if this isn't a string ourio.open errors
+
+            // this leaves us with:
+            // 1. arg
+            // 2. new_file
+            lua_remove(L, 1);  // remove arg, replacing it with new_file
+        }
+
+        // file handle, pass it off to realio.input
+        lua_getfield(L, LUA_REGISTRYINDEX, REG_REAL_IO_NAME);
+        if (output)
+        {
+            lua_getfield(L, -1, "output");
+        }
+        else
+        {
+            lua_getfield(L, -1, "input");
+        }
+        lua_remove(L, -2);    // remove LUA_REGISTRYINDEX[REAL_IO_NAME]
+        lua_pushvalue(L, 1);  // duplicate arg
+        lua_call(L, 1, 1);
+        return 1;
+    }
+
+}  // namespace
+
+int io_input(lua_State *L)
 {
-    auto *pl = getApp()->getPlugins()->getPluginByStatePtr(L);
-    if (pl == nullptr)
+    return globalFileCommon(L, false);
+}
+
+int io_output(lua_State *L)
+{
+    return globalFileCommon(L, true);
+}
+
+int io_close(lua_State *L)
+{
+    if (lua_gettop(L) > 1)
     {
-        throw std::runtime_error("internal error: no plugin");
+        return luaL_error(
+            L, "Too many arguments for io.close. Expected one or zero.");
     }
-    sol::state_view lua(L);
-
-    auto func = lua.registry()[REG_REAL_IO_NAME]["input"];
-    sol::protected_function_result res = func(file);
-    return res;
-}
-sol::variadic_results io_input_name(sol::this_state L, QString filename)
-{
-    auto *pl = getApp()->getPlugins()->getPluginByStatePtr(L);
-    if (pl == nullptr)
+    if (lua_gettop(L) == 0)
     {
-        throw std::runtime_error("internal error: no plugin");
+        lua_getfield(L, LUA_REGISTRYINDEX, "_IO_output");
     }
-    sol::state_view lua(L);
-    auto res = io_open(L, std::move(filename), "r");
-    if (res.size() != 1)
+    lua_getfield(L, -1, "close");
+    lua_pushvalue(L, -2);
+    lua_call(L, 1, 0);
+    return 0;
+}
+
+int io_flush(lua_State *L)
+{
+    if (lua_gettop(L) > 1)
     {
-        throw std::runtime_error(res.at(1).as<std::string>());
+        return luaL_error(
+            L, "Too many arguments for io.flush. Expected one or zero.");
     }
-    auto obj = res.at(0);
-    if (obj.get_type() != sol::type::userdata)
+    lua_getfield(L, LUA_REGISTRYINDEX, "_IO_output");
+    lua_getfield(L, -1, "flush");
+    lua_pushvalue(L, -2);
+    lua_call(L, 1, 0);
+    return 0;
+}
+
+int io_read(lua_State *L)
+{
+    if (lua_gettop(L) > 1)
     {
-        throw std::runtime_error("a file must be a userdata.");
+        return luaL_error(
+            L, "Too many arguments for io.read. Expected one or zero.");
     }
-    return io_input_file(L, obj);
+    lua_getfield(L, LUA_REGISTRYINDEX, "_IO_input");
+    lua_getfield(L, -1, "read");
+    lua_insert(L, 1);
+    lua_insert(L, 2);
+    lua_call(L, lua_gettop(L) - 1, 1);
+    return 1;
 }
 
-sol::variadic_results io_output_argless(sol::this_state L)
+int io_write(lua_State *L)
 {
-    auto *pl = getApp()->getPlugins()->getPluginByStatePtr(L);
-    if (pl == nullptr)
-    {
-        throw std::runtime_error("internal error: no plugin");
-    }
-    sol::state_view lua(L);
-
-    auto func = lua.registry()[REG_REAL_IO_NAME]["output"];
-    sol::protected_function_result res = func();
-    return res;
-}
-sol::variadic_results io_output_file(sol::this_state L, sol::userdata file)
-{
-    auto *pl = getApp()->getPlugins()->getPluginByStatePtr(L);
-    if (pl == nullptr)
-    {
-        throw std::runtime_error("internal error: no plugin");
-    }
-    sol::state_view lua(L);
-
-    auto func = lua.registry()[REG_REAL_IO_NAME]["output"];
-    sol::protected_function_result res = func(file);
-    return res;
-}
-sol::variadic_results io_output_name(sol::this_state L, QString filename)
-{
-    auto *pl = getApp()->getPlugins()->getPluginByStatePtr(L);
-    if (pl == nullptr)
-    {
-        throw std::runtime_error("internal error: no plugin");
-    }
-    sol::state_view lua(L);
-    auto res = io_open(L, std::move(filename), "w");
-    if (res.size() != 1)
-    {
-        throw std::runtime_error(res.at(1).as<std::string>());
-    }
-    auto obj = res.at(0);
-    if (obj.get_type() != sol::type::userdata)
-    {
-        throw std::runtime_error("internal error: a file must be a userdata.");
-    }
-    return io_output_file(L, obj);
+    lua_getfield(L, LUA_REGISTRYINDEX, "_IO_output");
+    lua_getfield(L, -1, "write");
+    lua_insert(L, 1);
+    lua_insert(L, 2);
+    // (input)
+    // (input).read
+    // args
+    lua_call(L, lua_gettop(L) - 1, 1);
+    return 1;
 }
 
-bool io_close_argless(sol::this_state L)
+int io_popen(lua_State *L)
 {
-    sol::state_view lua(L);
-    auto out = lua.registry()["_IO_output"];
-    return io_close_file(L, out);
+    return luaL_error(L, "io.popen: This function is a stub!");
 }
 
-bool io_close_file(sol::this_state L, sol::userdata file)
+int io_tmpfile(lua_State *L)
 {
-    sol::state_view lua(L);
-    return file["close"](file);
+    return luaL_error(L, "io.tmpfile: This function is a stub!");
 }
 
-void io_flush_argless(sol::this_state L)
-{
-    sol::state_view lua(L);
-    auto out = lua.registry()["_IO_output"];
-    io_flush_file(L, out);
-}
-
-void io_flush_file(sol::this_state L, sol::userdata file)
-{
-    sol::state_view lua(L);
-    file["flush"](file);
-}
-
-sol::variadic_results io_read(sol::this_state L, sol::variadic_args args)
-{
-    sol::state_view lua(L);
-    auto inp = lua.registry()["_IO_input"];
-    if (!inp.is<sol::userdata>())
-    {
-        throw std::runtime_error("Input not set to a file");
-    }
-    sol::protected_function read = inp["read"];
-    return read(inp, args);
-}
-
-sol::variadic_results io_write(sol::this_state L, sol::variadic_args args)
-{
-    sol::state_view lua(L);
-    auto out = lua.registry()["_IO_output"];
-    if (!out.is<sol::userdata>())
-    {
-        throw std::runtime_error("Output not set to a file");
-    }
-    sol::protected_function write = out["write"];
-    return write(out, args);
-}
-
-void io_popen()
-{
-    throw std::runtime_error("io.popen: This function is a stub!");
-}
-
-void io_tmpfile()
-{
-    throw std::runtime_error("io.tmpfile: This function is a stub!");
-}
+// NOLINTEND(*vararg)
 
 }  // namespace chatterino::lua::api
 #endif

src/controllers/plugins/api/IOWrapper.hpp

@@ -1,9 +1,5 @@
 #pragma once
 #ifdef CHATTERINO_HAVE_PLUGINS
-#    include <QString>
-#    include <sol/types.hpp>
-#    include <sol/variadic_args.hpp>
-#    include <sol/variadic_results.hpp>
 
 struct lua_State;
 
@@ -12,6 +8,7 @@ namespace chatterino::lua::api {
 // These functions are exposed as `_G.io`, they are wrappers for native Lua functionality.
 
 const char *const REG_REAL_IO_NAME = "real_lua_io_lib";
+const char *const REG_C2_IO_NAME = "c2io";
 
 /**
  * Opens a file.
@@ -23,9 +20,7 @@ const char *const REG_REAL_IO_NAME = "real_lua_io_lib";
  * @lua@param mode nil|"r"|"w"|"a"|"r+"|"w+"|"a+"
  * @exposed io.open
  */
-sol::variadic_results io_open(sol::this_state L, QString filename,
-                              QString strmode);
-sol::variadic_results io_open_modeless(sol::this_state L, QString filename);
+int io_open(lua_State *L);
 
 /**
  * Equivalent to io.input():lines("l") or a specific iterator over given file
@@ -37,9 +32,7 @@ sol::variadic_results io_open_modeless(sol::this_state L, QString filename);
  * @lua@param ...
  * @exposed io.lines
  */
-sol::variadic_results io_lines(sol::this_state L, QString filename,
-                               sol::variadic_args args);
-sol::variadic_results io_lines_noargs(sol::this_state L);
+int io_lines(lua_State *L);
 
 /**
  * Opens a file and sets it as default input or if given no arguments returns the default input.
@@ -49,9 +42,7 @@ sol::variadic_results io_lines_noargs(sol::this_state L);
  * @lua@return nil|FILE*
  * @exposed io.input
  */
-sol::variadic_results io_input_argless(sol::this_state L);
-sol::variadic_results io_input_file(sol::this_state L, sol::userdata file);
-sol::variadic_results io_input_name(sol::this_state L, QString filename);
+int io_input(lua_State *L);
 
 /**
  * Opens a file and sets it as default output or if given no arguments returns the default output
@@ -61,9 +52,7 @@ sol::variadic_results io_input_name(sol::this_state L, QString filename);
  * @lua@return nil|FILE*
  * @exposed io.output
  */
-sol::variadic_results io_output_argless(sol::this_state L);
-sol::variadic_results io_output_file(sol::this_state L, sol::userdata file);
-sol::variadic_results io_output_name(sol::this_state L, QString filename);
+int io_output(lua_State *L);
 
 /**
  * Closes given file or io.output() if not given.
@@ -72,8 +61,7 @@ sol::variadic_results io_output_name(sol::this_state L, QString filename);
  * @lua@param nil|FILE*
  * @exposed io.close
  */
-bool io_close_argless(sol::this_state L);
-bool io_close_file(sol::this_state L, sol::userdata file);
+int io_close(lua_State *L);
 
 /**
  * Flushes the buffer for given file or io.output() if not given.
@@ -82,8 +70,7 @@ bool io_close_file(sol::this_state L, sol::userdata file);
  * @lua@param nil|FILE*
  * @exposed io.flush
  */
-void io_flush_argless(sol::this_state L);
-void io_flush_file(sol::this_state L, sol::userdata file);
+int io_flush(lua_State *L);
 
 /**
  * Reads some data from the default input file
@@ -92,7 +79,7 @@ void io_flush_file(sol::this_state L, sol::userdata file);
  * @lua@param nil|string
  * @exposed io.read
  */
-sol::variadic_results io_read(sol::this_state L, sol::variadic_args args);
+int io_read(lua_State *L);
 
 /**
  * Writes some data to the default output file
@@ -101,10 +88,10 @@ sol::variadic_results io_read(sol::this_state L, sol::variadic_args args);
  * @lua@param nil|string
  * @exposed io.write
  */
-sol::variadic_results io_write(sol::this_state L, sol::variadic_args args);
+int io_write(lua_State *L);
 
-void io_popen();
-void io_tmpfile();
+int io_popen(lua_State *L);
+int io_tmpfile(lua_State *L);
 
 // NOLINTEND(readability-identifier-naming)
 }  // namespace chatterino::lua::api

src/messages/LimitedQueue.hpp

@@ -8,7 +8,6 @@
 #include <mutex>
 #include <optional>
 #include <shared_mutex>
-#include <utility>
 #include <vector>
 
 namespace chatterino {
@@ -197,12 +196,12 @@ public:
         std::unique_lock lock(this->mutex_);
 
         Equals eq;
-        for (size_t i = 0; i < this->buffer_.size(); ++i)
+        for (int i = 0; i < this->buffer_.size(); ++i)
         {
             if (eq(this->buffer_[i], needle))
             {
                 this->buffer_[i] = replacement;
-                return static_cast<int>(i);
+                return i;
             }
         }
         return -1;
@@ -213,10 +212,9 @@ public:
      *
      * @param[in] index the index of the item to replace
      * @param[in] replacement the item to put in place of the item at index
-     * @param[out] prev (optional) the item located at @a index before replacing
      * @return true if a replacement took place
      */
-    bool replaceItem(size_t index, const T &replacement, T *prev = nullptr)
+    bool replaceItem(size_t index, const T &replacement)
     {
         std::unique_lock lock(this->mutex_);
 
@@ -225,46 +223,10 @@ public:
             return false;
         }
 
-        if (prev)
-        {
-            *prev = std::exchange(this->buffer_[index], replacement);
-        }
-        else
-        {
-            this->buffer_[index] = replacement;
-        }
+        this->buffer_[index] = replacement;
         return true;
     }
 
-    /**
-     * @brief Replace the needle with the given item
-     *
-     * @param hint A hint on where the needle _might_ be
-     * @param[in] needle the item to search for
-     * @param[in] replacement the item to replace needle with
-     * @return the index of the replaced item, or -1 if no replacement took place
-     */
-    int replaceItem(size_t hint, const T &needle, const T &replacement)
-    {
-        std::unique_lock lock(this->mutex_);
-
-        if (hint < this->buffer_.size() && this->buffer_[hint] == needle)
-        {
-            this->buffer_[hint] = replacement;
-            return static_cast<int>(hint);
-        }
-
-        for (size_t i = 0; i < this->buffer_.size(); ++i)
-        {
-            if (this->buffer_[i] == needle)
-            {
-                this->buffer_[i] = replacement;
-                return static_cast<int>(i);
-            }
-        }
-        return -1;
-    }
-
     /**
      * @brief Inserts the given item before another item
      * 
@@ -353,32 +315,6 @@ public:
         return std::nullopt;
     }
 
-    /**
-     * @brief Find an item with a hint
-     *
-     * @param hint A hint on where the needle _might_ be
-     * @param predicate that will used to find the item
-     * @return the item and its index or none if it's not found
-     */
-    std::optional<std::pair<size_t, T>> find(size_t hint, auto &&predicate)
-    {
-        std::unique_lock lock(this->mutex_);
-
-        if (hint < this->buffer_.size() && predicate(this->buffer_[hint]))
-        {
-            return std::pair{hint, this->buffer_[hint]};
-        };
-
-        for (size_t i = 0; i < this->buffer_.size(); i++)
-        {
-            if (predicate(this->buffer_[i]))
-            {
-                return std::pair{i, this->buffer_[i]};
-            }
-        }
-        return std::nullopt;
-    }
-
     /**
      * @brief Returns the first item matching a predicate, checking in reverse
      * 

src/messages/Message.hpp

@@ -22,7 +22,6 @@ class ScrollbarHighlight;
 
 struct Message;
 using MessagePtr = std::shared_ptr<const Message>;
-using MessagePtrMut = std::shared_ptr<Message>;
 struct Message {
     Message();
     ~Message();

src/messages/MessageBuilder.hpp

@@ -14,6 +14,8 @@
 
 #include <ctime>
 #include <memory>
+#include <optional>
+#include <tuple>
 #include <unordered_map>
 #include <utility>
 
@@ -29,7 +31,6 @@ struct AutomodUserAction;
 struct AutomodInfoAction;
 struct Message;
 using MessagePtr = std::shared_ptr<const Message>;
-using MessagePtrMut = std::shared_ptr<Message>;
 
 class MessageElement;
 class TextElement;
@@ -44,7 +45,6 @@ struct HelixVip;
 using HelixModerator = HelixVip;
 struct ChannelPointReward;
 struct DeleteAction;
-struct TwitchEmoteOccurrence;
 
 namespace linkparser {
     struct Parsed;
@@ -65,7 +65,6 @@ struct LiveUpdatesUpdateEmoteSetMessageTag {
 struct ImageUploaderResultTag {
 };
 
-// NOLINTBEGIN(readability-identifier-naming)
 const SystemMessageTag systemMessage{};
 const TimeoutMessageTag timeoutMessage{};
 const LiveUpdatesUpdateEmoteMessageTag liveUpdatesUpdateEmoteMessage{};
@@ -76,7 +75,6 @@ const LiveUpdatesUpdateEmoteSetMessageTag liveUpdatesUpdateEmoteSetMessage{};
 // This signifies that you want to construct a message containing the result of
 // a successful image upload.
 const ImageUploaderResultTag imageUploaderResultMessage{};
-// NOLINTEND(readability-identifier-naming)
 
 MessagePtr makeSystemMessage(const QString &text);
 MessagePtr makeSystemMessage(const QString &text, const QTime &time);
@@ -88,22 +86,39 @@ struct MessageParseArgs {
     bool trimSubscriberUsername = false;
     bool isStaffOrBroadcaster = false;
     bool isSubscriptionMessage = false;
-    bool allowIgnore = true;
-    bool isAction = false;
     QString channelPointRewardId = "";
 };
 
-struct HighlightAlert {
-    QUrl customSound;
-    bool playSound = false;
-    bool windowAlert = false;
+struct TwitchEmoteOccurrence {
+    int start;
+    int end;
+    EmotePtr ptr;
+    EmoteName name;
+
+    bool operator==(const TwitchEmoteOccurrence &other) const
+    {
+        return std::tie(this->start, this->end, this->ptr, this->name) ==
+               std::tie(other.start, other.end, other.ptr, other.name);
+    }
 };
+
 class MessageBuilder
 {
 public:
     /// Build a message without a base IRC message.
     MessageBuilder();
 
+    /// Build a message based on an incoming IRC PRIVMSG
+    explicit MessageBuilder(Channel *_channel,
+                            const Communi::IrcPrivateMessage *_ircMessage,
+                            const MessageParseArgs &_args);
+
+    /// Build a message based on an incoming IRC message (e.g. notice)
+    explicit MessageBuilder(Channel *_channel,
+                            const Communi::IrcMessage *_ircMessage,
+                            const MessageParseArgs &_args, QString content,
+                            bool isAction);
+
     MessageBuilder(SystemMessageTag, const QString &text,
                    const QTime &time = QTime::currentTime());
     MessageBuilder(TimeoutMessageTag, const QString &timeoutUser,
@@ -148,10 +163,17 @@ public:
 
     ~MessageBuilder() = default;
 
+    QString userName;
+
+    /// The Twitch Channel the message was received in
+    TwitchChannel *twitchChannel = nullptr;
+    /// The Twitch Channel the message was sent in, according to the Shared Chat feature
+    TwitchChannel *sourceChannel = nullptr;
+
     Message *operator->();
     Message &message();
-    MessagePtrMut release();
-    std::weak_ptr<const Message> weakOf();
+    MessagePtr release();
+    std::weak_ptr<Message> weakOf();
 
     void append(std::unique_ptr<MessageElement> element);
     void addLink(const linkparser::Parsed &parsedLink, const QString &source);
@@ -168,8 +190,14 @@ public:
         return pointer;
     }
 
-    static void triggerHighlights(const Channel *channel,
-                                  const HighlightAlert &alert);
+    [[nodiscard]] bool isIgnored() const;
+    bool isIgnoredReply() const;
+    void triggerHighlights();
+    MessagePtr build();
+
+    void setThread(std::shared_ptr<MessageThread> thread);
+    void setParent(MessagePtr parent);
+    void setMessageOffset(int offset);
 
     void appendChannelPointRewardMessage(const ChannelPointReward &reward,
                                          bool isMod, bool isBroadcaster);
@@ -209,133 +237,110 @@ public:
     static MessagePtr makeLowTrustUpdateMessage(
         const PubSubLowTrustUsersMessage &action);
 
-    /// @brief Builds a message out of an `ircMessage`.
-    ///
-    /// Building a message won't cause highlights to be triggered. They will
-    /// only be parsed. To trigger highlights (play sound etc.), use
-    /// triggerHighlights().
-    ///
-    /// @param channel The channel this message was sent to. Must not be
-    ///                `nullptr`.
-    /// @param ircMessage The original message. This can be any message
-    ///                   (PRIVMSG, USERNOTICE, etc.). Its content is not
-    ///                   accessed through this parameter but through `content`,
-    ///                   as the content might be inside a tag (e.g. gifts in a
-    ///                   USERNOTICE).
-    /// @param args Arguments from parsing a chat message.
-    /// @param content The message text. This isn't always the entire text. In
-    ///                replies, the leading mention can be cut off.
-    ///                See `messageOffset`.
-    /// @param messageOffset Starting offset to be used on index-based
-    ///                      operations on `content` such as parsing emotes.
-    ///                      For example:
-    ///                         ircMessage = "@hi there"
-    ///                         content = "there"
-    ///                         messageOffset_ = 4
-    ///                      The index 6 would resolve to 6 - 4 = 2 => 'e'
-    /// @param thread The reply thread this message is part of. If there's no
-    ///               thread, this is an empty `shared_ptr`.
-    /// @param parent The direct parent this message is replying to. This does
-    ///               not need to be the `thread`s root. If this message isn't
-    ///               replying to anything, this is an empty `shared_ptr`.
-    ///
-    /// @returns The built message and a highlight result. If the message is
-    ///          ignored (e.g. from a blocked user), then the returned pointer
-    ///          will be en empty `shared_ptr`.
-    static std::pair<MessagePtrMut, HighlightAlert> makeIrcMessage(
-        Channel *channel, const Communi::IrcMessage *ircMessage,
-        const MessageParseArgs &args, QString content,
-        QString::size_type messageOffset,
-        const std::shared_ptr<MessageThread> &thread = {},
-        const MessagePtr &parent = {});
+    static std::unordered_map<QString, QString> parseBadgeInfoTag(
+        const QVariantMap &tags);
 
-    static MessagePtrMut makeSystemMessageWithUser(
-        const QString &text, const QString &loginName,
-        const QString &displayName, const MessageColor &userColor,
-        const QTime &time);
+    // Parses "badges" tag which contains a comma separated list of key-value elements
+    static std::vector<Badge> parseBadgeTag(const QVariantMap &tags);
 
-    static MessagePtrMut makeSubgiftMessage(const QString &text,
-                                            const QVariantMap &tags,
-                                            const QTime &time);
+    static std::vector<TwitchEmoteOccurrence> parseTwitchEmotes(
+        const QVariantMap &tags, const QString &originalMessage,
+        int messageOffset);
 
-private:
-    struct TextState {
-        TwitchChannel *twitchChannel = nullptr;
-        bool hasBits = false;
-        bool bitsStacked = false;
-        int bitsLeft = 0;
-    };
-    void addEmoji(const EmotePtr &emote);
-    void addTextOrEmote(TextState &state, QString string);
+    static void processIgnorePhrases(
+        const std::vector<IgnorePhrase> &phrases, QString &originalMessage,
+        std::vector<TwitchEmoteOccurrence> &twitchEmotes);
 
-    Outcome tryAppendCheermote(TextState &state, const QString &string);
-    Outcome tryAppendEmote(TwitchChannel *twitchChannel, const EmoteName &name);
+protected:
+    void addTextOrEmoji(EmotePtr emote);
+    void addTextOrEmoji(const QString &string_);
 
     bool isEmpty() const;
     MessageElement &back();
     std::unique_ptr<MessageElement> releaseBack();
 
+    MessageColor textColor_ = MessageColor::Text;
+
     // Helper method that emplaces some text stylized as system text
     // and then appends that text to the QString parameter "toUpdate".
     // Returns the TextElement that was emplaced.
     TextElement *emplaceSystemTextAndUpdate(const QString &text,
                                             QString &toUpdate);
 
+    std::shared_ptr<Message> message_;
+
     void parse();
-    void parseUsernameColor(const QVariantMap &tags, const QString &userID);
-    void parseUsername(const Communi::IrcMessage *ircMessage,
-                       TwitchChannel *twitchChannel,
-                       bool trimSubscriberUsername);
-    void parseMessageID(const QVariantMap &tags);
-
-    /// Parses the room-ID this message was received in
-    ///
-    /// @returns The room-ID
-    static QString parseRoomID(const QVariantMap &tags,
-                               TwitchChannel *twitchChannel);
-
-    /// Parses the shared-chat information from this message.
-    ///
-    /// @param tags The tags of the received message
-    /// @param twitchChannel The channel this message was received in
-    /// @returns The source channel - the channel this message originated from.
-    ///          If there's no channel currently open, @a twitchChannel is
-    ///          returned.
-    TwitchChannel *parseSharedChatInfo(const QVariantMap &tags,
-                                       TwitchChannel *twitchChannel);
-
+    void parseUsernameColor();
+    void parseUsername();
+    void parseMessageID();
+    void parseRoomID();
     // Parse & build thread information into the message
     // Will read information from thread_ or from IRC tags
-    void parseThread(const QString &messageContent, const QVariantMap &tags,
-                     const Channel *channel,
-                     const std::shared_ptr<MessageThread> &thread,
-                     const MessagePtr &parent);
+    void parseThread();
     // parseHighlights only updates the visual state of the message, but leaves the playing of alerts and sounds to the triggerHighlights function
-    HighlightAlert parseHighlights(const QVariantMap &tags,
-                                   const QString &originalMessage,
-                                   const MessageParseArgs &args);
+    void parseHighlights();
+    void appendChannelName();
+    void appendUsername();
 
-    void appendChannelName(const Channel *channel);
-    void appendUsername(const QVariantMap &tags, const MessageParseArgs &args);
+    /// Return the Twitch Channel this message originated from
+    ///
+    /// Useful to handle messages from the "Shared Chat" feature
+    ///
+    /// Can return nullptr
+    const TwitchChannel *getSourceChannel() const;
+
+    std::tuple<std::optional<EmotePtr>, MessageElementFlags, bool> parseEmote(
+        const EmoteName &name) const;
+    Outcome tryAppendEmote(const EmoteName &name);
 
     void addWords(const QStringList &words,
-                  const std::vector<TwitchEmoteOccurrence> &twitchEmotes,
-                  TextState &state);
+                  const std::vector<TwitchEmoteOccurrence> &twitchEmotes);
 
-    void appendTwitchBadges(const QVariantMap &tags,
-                            TwitchChannel *twitchChannel);
-    void appendChatterinoBadges(const QString &userID);
-    void appendFfzBadges(TwitchChannel *twitchChannel, const QString &userID);
-    void appendSeventvBadges(const QString &userID);
+    void appendTwitchBadges();
+    void appendChatterinoBadges();
+    void appendFfzBadges();
+    void appendSeventvBadges();
+    Outcome tryParseCheermote(const QString &string);
 
-    [[nodiscard]] static bool isIgnored(const QString &originalMessage,
-                                        const QString &userID,
-                                        const Channel *channel);
+    bool shouldAddModerationElements() const;
 
-    std::shared_ptr<Message> message_;
-    MessageColor textColor_ = MessageColor::Text;
+    QString roomID_;
+    bool hasBits_ = false;
+    QString bits;
+    int bitsLeft{};
+    bool bitsStacked = false;
+    bool historicalMessage_ = false;
+    std::shared_ptr<MessageThread> thread_;
+    MessagePtr parent_;
+
+    /**
+     * Starting offset to be used on index-based operations on `originalMessage_`.
+     *
+     * For example:
+     * originalMessage_ = "there"
+     * messageOffset_ = 4
+     * (the irc message is "hey there")
+     *
+     * then the index 6 would resolve to 6 - 4 = 2 => 'e'
+     */
+    int messageOffset_ = 0;
+
+    QString userId_;
+    bool senderIsBroadcaster{};
+
+    Channel *channel = nullptr;
+    const Communi::IrcMessage *ircMessage;
+    MessageParseArgs args;
+    const QVariantMap tags;
+    QString originalMessage_;
+
+    const bool action_{};
 
     QColor usernameColor_ = {153, 153, 153};
+
+    bool highlightAlert_ = false;
+    bool highlightSound_ = false;
+    std::optional<QUrl> highlightSoundCustomUrl_{};
 };
 
 }  // namespace chatterino

src/messages/MessageElement.hpp

@@ -66,10 +66,6 @@ enum class MessageElementFlag : int64_t {
     BitsStatic = (1LL << 11),
     BitsAnimated = (1LL << 12),
 
-    // Slot 0: Twitch
-    // - Shared Channel indicator badge
-    BadgeSharedChannel = (1LL << 37),
-
     // Slot 1: Twitch
     // - Staff badge
     // - Admin badge
@@ -123,7 +119,7 @@ enum class MessageElementFlag : int64_t {
 
     Badges = BadgeGlobalAuthority | BadgePredictions | BadgeChannelAuthority |
              BadgeSubscription | BadgeVanity | BadgeChatterino | BadgeSevenTV |
-             BadgeFfz | BadgeSharedChannel,
+             BadgeFfz,
 
     ChannelName = (1LL << 20),
 

src/messages/MessageSimilarity.cpp

@@ -1,121 +0,0 @@
-#include "messages/MessageSimilarity.hpp"
-
-#include "Application.hpp"
-#include "controllers/accounts/AccountController.hpp"
-#include "messages/LimitedQueueSnapshot.hpp"  // IWYU pragma: keep
-#include "providers/twitch/TwitchAccount.hpp"
-#include "singletons/Settings.hpp"
-
-#include <algorithm>
-#include <vector>
-
-namespace {
-
-using namespace chatterino;
-
-float relativeSimilarity(QStringView str1, QStringView str2)
-{
-    using SizeType = QStringView::size_type;
-
-    // Longest Common Substring Problem
-    std::vector<std::vector<int>> tree(str1.size(),
-                                       std::vector<int>(str2.size(), 0));
-    int z = 0;
-
-    for (SizeType i = 0; i < str1.size(); ++i)
-    {
-        for (SizeType j = 0; j < str2.size(); ++j)
-        {
-            if (str1[i] == str2[j])
-            {
-                if (i == 0 || j == 0)
-                {
-                    tree[i][j] = 1;
-                }
-                else
-                {
-                    tree[i][j] = tree[i - 1][j - 1] + 1;
-                }
-                z = std::max(tree[i][j], z);
-            }
-            else
-            {
-                tree[i][j] = 0;
-            }
-        }
-    }
-
-    // ensure that no div by 0
-    if (z == 0)
-    {
-        return 0.F;
-    }
-
-    auto div = std::max<>({static_cast<SizeType>(1), str1.size(), str2.size()});
-
-    return float(z) / float(div);
-}
-
-template <std::ranges::bidirectional_range T>
-float inMessages(const MessagePtr &msg, const T &messages)
-{
-    float similarityPercent = 0.0F;
-
-    for (const auto &prevMsg :
-         messages | std::views::reverse |
-             std::views::take(getSettings()->hideSimilarMaxMessagesToCheck))
-    {
-        if (prevMsg->parseTime.secsTo(QTime::currentTime()) >=
-            getSettings()->hideSimilarMaxDelay)
-        {
-            break;
-        }
-        if (getSettings()->hideSimilarBySameUser &&
-            msg->loginName != prevMsg->loginName)
-        {
-            continue;
-        }
-        similarityPercent = std::max(
-            similarityPercent,
-            relativeSimilarity(msg->messageText, prevMsg->messageText));
-    }
-
-    return similarityPercent;
-}
-
-}  // namespace
-
-namespace chatterino {
-
-template <std::ranges::bidirectional_range T>
-void setSimilarityFlags(const MessagePtr &message, const T &messages)
-{
-    if (getSettings()->similarityEnabled)
-    {
-        bool isMyself =
-            message->loginName ==
-            getApp()->getAccounts()->twitch.getCurrent()->getUserName();
-        bool hideMyself = getSettings()->hideSimilarMyself;
-
-        if (isMyself && !hideMyself)
-        {
-            return;
-        }
-
-        if (inMessages(message, messages) > getSettings()->similarityPercentage)
-        {
-            message->flags.set(MessageFlag::Similar);
-            if (getSettings()->colorSimilarDisabled)
-            {
-                message->flags.set(MessageFlag::Disabled);
-            }
-        }
-    }
-}
-
-template void setSimilarityFlags<std::vector<MessagePtr>>(
-    const MessagePtr &msg, const std::vector<MessagePtr> &messages);
-template void setSimilarityFlags<LimitedQueueSnapshot<MessagePtr>>(
-    const MessagePtr &msg, const LimitedQueueSnapshot<MessagePtr> &messages);
-
-}  // namespace chatterino

src/messages/MessageSimilarity.hpp

@@ -1,11 +0,0 @@
-#pragma once
-
-#include "messages/Message.hpp"
-
-#include <ranges>
-namespace chatterino {
-
-template <std::ranges::bidirectional_range T>
-void setSimilarityFlags(const MessagePtr &message, const T &messages);
-
-}  // namespace chatterino

src/messages/MessageSink.hpp

@@ -1,67 +0,0 @@
-#pragma once
-
-#include "common/enums/MessageContext.hpp"
-#include "common/FlagsEnum.hpp"
-#include "messages/MessageFlag.hpp"
-
-#include <memory>
-#include <optional>
-
-class QStringView;
-class QTime;
-
-namespace chatterino {
-
-struct Message;
-using MessagePtr = std::shared_ptr<const Message>;
-
-enum class MessageSinkTrait : uint8_t {
-    None = 0,
-
-    /// Messages with the `Highlighted` and `ShowInMentions` flags should be
-    /// added to the global mentions channel when encountered.
-    AddMentionsToGlobalChannel = 1 << 0,
-
-    /// A channel-point redemption whose reward is not yet known should not be
-    /// added to this sink, but queued in the corresponding TwitchChannel
-    /// (`addQueuedRedemption`).
-    RequiresKnownChannelPointReward = 1 << 1,
-};
-using MessageSinkTraits = FlagsEnum<MessageSinkTrait>;
-
-/// A generic interface for a managed buffer of `Message`s
-class MessageSink
-{
-public:
-    virtual ~MessageSink() = default;
-
-    /// Add a message to this sink
-    ///
-    /// @param message The message to add (non-null)
-    /// @param ctx The context in which this message is being added.
-    /// @param overridingFlags
-    virtual void addMessage(
-        MessagePtr message, MessageContext ctx,
-        std::optional<MessageFlags> overridingFlags = std::nullopt) = 0;
-
-    /// Adds a timeout message or merges it into an existing one
-    virtual void addOrReplaceTimeout(MessagePtr clearchatMessage,
-                                     QTime now) = 0;
-
-    /// Flags all messages as `Disabled`
-    virtual void disableAllMessages() = 0;
-
-    /// Searches for similar messages and flags this message as similar
-    /// (based on the current settings).
-    virtual void applySimilarityFilters(const MessagePtr &message) const = 0;
-
-    /// @brief Searches for a message by an ID
-    ///
-    /// If there is no message found, an empty shared-pointer is returned.
-    virtual MessagePtr findMessageByID(QStringView id) = 0;
-
-    /// Behaviour to be exercised when parsing/building messages for this sink.
-    virtual MessageSinkTraits sinkTraits() const = 0;
-};
-
-}  // namespace chatterino

src/messages/layouts/MessageLayoutContainer.cpp

@@ -212,8 +212,7 @@ void MessageLayoutContainer::breakLine()
     this->lineStart_ = this->elements_.size();
     //    this->currentX = (int)(this->scale * 8);
 
-    if (this->canCollapse() &&
-        static_cast<int>(this->line_ + 1) >= maxUncollapsedLines())
+    if (this->canCollapse() && this->line_ + 1 >= maxUncollapsedLines())
     {
         this->canAddMessages_ = false;
         return;

src/messages/layouts/MessageLayoutElement.cpp

@@ -514,10 +514,9 @@ int TextLayoutElement::getXFromIndex(size_t index)
     else if (index < static_cast<size_t>(this->getText().size()))
     {
         int x = 0;
-        for (size_t i = 0; i < index; i++)
+        for (int i = 0; i < index; i++)
         {
-            x += metrics.horizontalAdvance(
-                this->getText()[static_cast<QString::size_type>(i)]);
+            x += metrics.horizontalAdvance(this->getText()[i]);
         }
         return x + this->getRect().left();
     }

src/providers/ffz/FfzBadges.cpp

@@ -1,6 +1,5 @@
 #include "providers/ffz/FfzBadges.hpp"
 
-#include "Application.hpp"
 #include "common/network/NetworkRequest.hpp"
 #include "common/network/NetworkResult.hpp"
 #include "messages/Emote.hpp"
@@ -110,30 +109,4 @@ void FfzBadges::load()
         .execute();
 }
 
-void FfzBadges::registerBadge(int badgeID, Badge badge)
-{
-    assert(getApp()->isTest());
-
-    std::unique_lock lock(this->mutex_);
-
-    this->badges.emplace(badgeID, std::move(badge));
-}
-
-void FfzBadges::assignBadgeToUser(const UserId &userID, int badgeID)
-{
-    assert(getApp()->isTest());
-
-    std::unique_lock lock(this->mutex_);
-
-    auto it = this->userBadges.find(userID.string);
-    if (it != this->userBadges.end())
-    {
-        it->second.emplace(badgeID);
-    }
-    else
-    {
-        this->userBadges.emplace(userID.string, std::set{badgeID});
-    }
-}
-
 }  // namespace chatterino

src/providers/ffz/FfzBadges.hpp

@@ -30,9 +30,6 @@ public:
     std::vector<Badge> getUserBadges(const UserId &id);
     std::optional<Badge> getBadge(int badgeID) const;
 
-    void registerBadge(int badgeID, Badge badge);
-    void assignBadgeToUser(const UserId &userID, int badgeID);
-
     void load();
 
 private:

src/providers/recentmessages/Impl.cpp

@@ -3,9 +3,7 @@
 #include "common/Env.hpp"
 #include "messages/MessageBuilder.hpp"
 #include "providers/twitch/IrcMessageHandler.hpp"
-#include "providers/twitch/TwitchChannel.hpp"
 #include "util/Helpers.hpp"
-#include "util/VectorMessageSink.hpp"
 
 #include <QJsonArray>
 #include <QUrlQuery>
@@ -42,13 +40,7 @@ std::vector<Communi::IrcMessage *> parseRecentMessages(
 std::vector<MessagePtr> buildRecentMessages(
     std::vector<Communi::IrcMessage *> &messages, Channel *channel)
 {
-    VectorMessageSink sink({}, MessageFlag::RecentMessage);
-
-    auto *twitchChannel = dynamic_cast<TwitchChannel *>(channel);
-    if (!twitchChannel)
-    {
-        return {};
-    }
+    std::vector<MessagePtr> allBuiltMessages;
 
     for (auto *message : messages)
     {
@@ -66,16 +58,24 @@ std::vector<MessagePtr> buildRecentMessages(
                 auto msg = makeSystemMessage(
                     QLocale().toString(msgDate, QLocale::LongFormat),
                     QTime(0, 0));
-                sink.addMessage(msg, MessageContext::Original);
+                msg->flags.set(MessageFlag::RecentMessage);
+                allBuiltMessages.emplace_back(msg);
             }
         }
 
-        IrcMessageHandler::parseMessageInto(message, sink, twitchChannel);
+        auto builtMessages = IrcMessageHandler::parseMessageWithReply(
+            channel, message, allBuiltMessages);
+
+        for (const auto &builtMessage : builtMessages)
+        {
+            builtMessage->flags.set(MessageFlag::RecentMessage);
+            allBuiltMessages.emplace_back(builtMessage);
+        }
 
         message->deleteLater();
     }
 
-    return std::move(sink).takeMessages();
+    return allBuiltMessages;
 }
 
 // Returns the URL to be used for querying the Recent Messages API for the

src/providers/seventv/SeventvBadges.cpp

@@ -59,7 +59,7 @@ void SeventvBadges::registerBadge(const QJsonObject &badgeJson)
 
     auto emote = Emote{
         .name = EmoteName{},
-        .images = SeventvEmotes::createImageSet(badgeJson, true),
+        .images = SeventvEmotes::createImageSet(badgeJson),
         .tooltip = Tooltip{badgeJson["tooltip"].toString()},
         .homePage = Url{},
         .id = EmoteId{badgeID},

src/providers/seventv/SeventvEmotes.cpp

@@ -79,8 +79,7 @@ bool isZeroWidthRecommended(const QJsonObject &emoteData)
 Tooltip createTooltip(const QString &name, const QString &author, bool isGlobal)
 {
     return Tooltip{QString("%1<br>%2 7TV Emote<br>By: %3")
-                       .arg(name.toHtmlEscaped(),
-                            isGlobal ? "Global" : "Channel",
+                       .arg(name, isGlobal ? "Global" : "Channel",
                             author.isEmpty() ? "<deleted>" : author)};
 }
 
@@ -88,8 +87,7 @@ Tooltip createAliasedTooltip(const QString &name, const QString &baseName,
                              const QString &author, bool isGlobal)
 {
     return Tooltip{QString("%1<br>Alias of %2<br>%3 7TV Emote<br>By: %4")
-                       .arg(name.toHtmlEscaped(), baseName.toHtmlEscaped(),
-                            isGlobal ? "Global" : "Channel",
+                       .arg(name, baseName, isGlobal ? "Global" : "Channel",
                             author.isEmpty() ? "<deleted>" : author)};
 }
 
@@ -108,18 +106,12 @@ CreateEmoteResult createEmote(const QJsonObject &activeEmote,
             ? createAliasedTooltip(emoteName.string, baseEmoteName.string,
                                    author.string, isGlobal)
             : createTooltip(emoteName.string, author.string, isGlobal);
-    auto imageSet = SeventvEmotes::createImageSet(emoteData, false);
+    auto imageSet = SeventvEmotes::createImageSet(emoteData);
 
-    auto emote = Emote({
-        emoteName,
-        imageSet,
-        tooltip,
-        Url{EMOTE_LINK_FORMAT.arg(emoteId.string)},
-        zeroWidth,
-        emoteId,
-        author,
-        makeConditionedOptional(aliasedName, baseEmoteName),
-    });
+    auto emote =
+        Emote({emoteName, imageSet, tooltip,
+               Url{EMOTE_LINK_FORMAT.arg(emoteId.string)}, zeroWidth, emoteId,
+               author, makeConditionedOptional(aliasedName, baseEmoteName)});
 
     return {emote, emoteId, emoteName, !emote.images.getImage1()->isEmpty()};
 }
@@ -435,8 +427,7 @@ void SeventvEmotes::getEmoteSet(
         });
 }
 
-ImageSet SeventvEmotes::createImageSet(const QJsonObject &emoteData,
-                                       bool useStatic)
+ImageSet SeventvEmotes::createImageSet(const QJsonObject &emoteData)
 {
     auto host = emoteData["host"].toObject();
     // "//cdn.7tv[...]"
@@ -472,21 +463,9 @@ ImageSet SeventvEmotes::createImageSet(const QJsonObject &emoteData,
             baseWidth = width;
         }
 
-        auto name = [&] {
-            if (useStatic)
-            {
-                auto staticName = file["static_name"].toString();
-                if (!staticName.isEmpty())
-                {
-                    return staticName;
-                }
-            }
-            return file["name"].toString();
-        }();
-
-        auto image =
-            Image::fromUrl({QString("https:%1/%2").arg(baseUrl, name)}, scale,
-                           {static_cast<int>(width), file["height"].toInt(16)});
+        auto image = Image::fromUrl(
+            {QString("https:%1/%2").arg(baseUrl, file["name"].toString())},
+            scale, {static_cast<int>(width), file["height"].toInt(16)});
 
         sizes.at(nextSize) = image;
         nextSize++;

src/providers/seventv/SeventvEmotes.hpp

@@ -153,10 +153,8 @@ public:
      * Creates an image set from a 7TV emote or badge.
      *
      * @param emoteData { host: { files: [], url } }
-     * @param useStatic use static version if possible
      */
-    static ImageSet createImageSet(const QJsonObject &emoteData,
-                                   bool useStatic);
+    static ImageSet createImageSet(const QJsonObject &emoteData);
 
 private:
     Atomic<std::shared_ptr<const EmoteMap>> global_;

src/providers/seventv/SeventvEventAPI.cpp

@@ -233,10 +233,6 @@ void SeventvEventAPI::handleDispatch(const Dispatch &dispatch)
             }
         }
         break;
-        case SubscriptionType::ResetEntitlement: {
-            // unhandled (not clear what we'd do here yet)
-        }
-        break;
         default: {
             qCDebug(chatterinoSeventvEventAPI)
                 << "Unknown subscription type:"

src/providers/seventv/eventapi/Subscription.hpp

@@ -27,7 +27,6 @@ enum class SubscriptionType {
     CreateEntitlement,
     UpdateEntitlement,
     DeleteEntitlement,
-    ResetEntitlement,
 
     INVALID,
 };
@@ -120,8 +119,6 @@ constexpr magic_enum::customize::customize_t magic_enum::customize::enum_name<
             return "entitlement.update";
         case SubscriptionType::DeleteEntitlement:
             return "entitlement.delete";
-        case SubscriptionType::ResetEntitlement:
-            return "entitlement.reset";
 
         default:
             return default_tag;

src/providers/twitch/IrcMessageHandler.hpp

@@ -16,7 +16,6 @@ struct Message;
 using MessagePtr = std::shared_ptr<const Message>;
 class TwitchChannel;
 class TwitchMessageBuilder;
-class MessageSink;
 
 struct ClearChatMessage {
     MessagePtr message;
@@ -34,34 +33,30 @@ public:
      * Parse an IRC message into 0 or more Chatterino messages
      * Takes previously loaded messages into consideration to add reply contexts
      **/
-    static void parseMessageInto(Communi::IrcMessage *message,
-                                 MessageSink &sink, TwitchChannel *channel);
+    static std::vector<MessagePtr> parseMessageWithReply(
+        Channel *channel, Communi::IrcMessage *message,
+        std::vector<MessagePtr> &otherLoaded);
 
     void handlePrivMessage(Communi::IrcPrivateMessage *message,
                            ITwitchIrcServer &twitchServer);
-    static void parsePrivMessageInto(Communi::IrcPrivateMessage *message,
-                                     MessageSink &sink, TwitchChannel *channel);
 
     void handleRoomStateMessage(Communi::IrcMessage *message);
     void handleClearChatMessage(Communi::IrcMessage *message);
     void handleClearMessageMessage(Communi::IrcMessage *message);
     void handleUserStateMessage(Communi::IrcMessage *message);
-
     void handleWhisperMessage(Communi::IrcMessage *ircMessage);
+
     void handleUserNoticeMessage(Communi::IrcMessage *message,
                                  ITwitchIrcServer &twitchServer);
-    static void parseUserNoticeMessageInto(Communi::IrcMessage *message,
-                                           MessageSink &sink,
-                                           TwitchChannel *channel);
 
     void handleNoticeMessage(Communi::IrcNoticeMessage *message);
 
     void handleJoinMessage(Communi::IrcMessage *message);
     void handlePartMessage(Communi::IrcMessage *message);
 
-    static void addMessage(Communi::IrcMessage *message, MessageSink &sink,
-                           TwitchChannel *chan, const QString &originalContent,
-                           ITwitchIrcServer &twitch, bool isSub, bool isAction);
+    void addMessage(Communi::IrcMessage *message, const ChannelPtr &chan,
+                    const QString &originalContent, ITwitchIrcServer &server,
+                    bool isSub, bool isAction);
 
 private:
     static float similarity(const MessagePtr &msg,

src/providers/twitch/TwitchAccount.cpp

@@ -174,17 +174,6 @@ void TwitchAccount::unblockUser(const QString &userId, const QObject *caller,
         std::move(onFailure));
 }
 
-void TwitchAccount::blockUserLocally(const QString &userID)
-{
-    assertInGuiThread();
-    assert(getApp()->isTest());
-
-    TwitchUser blockedUser;
-    blockedUser.id = userID;
-    this->ignores_.insert(blockedUser);
-    this->ignoresUserIds_.insert(blockedUser.id);
-}
-
 const std::unordered_set<TwitchUser> &TwitchAccount::blocks() const
 {
     assertInGuiThread();
@@ -347,12 +336,6 @@ SharedAccessGuard<std::shared_ptr<const EmoteMap>> TwitchAccount::accessEmotes()
     return this->emotes_.accessConst();
 }
 
-void TwitchAccount::setEmotes(std::shared_ptr<const EmoteMap> emotes)
-{
-    assert(getApp()->isTest());
-    *this->emotes_.access() = std::move(emotes);
-}
-
 std::optional<EmotePtr> TwitchAccount::twitchEmote(const EmoteName &name) const
 {
     auto emotes = this->emotes_.accessConst();

src/providers/twitch/TwitchAccount.hpp

@@ -71,8 +71,6 @@ public:
                      std::function<void()> onSuccess,
                      std::function<void()> onFailure);
 
-    void blockUserLocally(const QString &userID);
-
     [[nodiscard]] const std::unordered_set<TwitchUser> &blocks() const;
     [[nodiscard]] const std::unordered_set<QString> &blockedUserIds() const;
 
@@ -85,21 +83,16 @@ public:
     /// Returns true if the account has access to the given emote set
     bool hasEmoteSet(const EmoteSetId &id) const;
 
-    /// Returns a map of emote sets the account has access to
+    /// Return a map of emote sets the account has access to
     ///
     /// Key being the emote set ID, and contents being information about the emote set
     /// and the emotes contained in the emote set
     SharedAccessGuard<std::shared_ptr<const TwitchEmoteSetMap>>
         accessEmoteSets() const;
 
-    /// Returns a map of emotes the account has access to
+    /// Return a map of emotes the account has access to
     SharedAccessGuard<std::shared_ptr<const EmoteMap>> accessEmotes() const;
 
-    /// Sets the emotes this account has access to
-    ///
-    /// This should only be used in tests.
-    void setEmotes(std::shared_ptr<const EmoteMap> emotes);
-
     /// Return the emote by emote name if the account has access to the emote
     std::optional<EmotePtr> twitchEmote(const EmoteName &name) const;
 

src/providers/twitch/TwitchBadges.hpp

@@ -45,10 +45,10 @@ public:
 
     void loadTwitchBadges();
 
+private:
     /// Loads the badges shipped with Chatterino (twitch-badges.json)
     void loadLocalBadges();
 
-private:
     void loaded();
     void loadEmoteImage(const QString &name, const ImagePtr &image,
                         BadgeIconCallback &&callback);

src/providers/twitch/TwitchChannel.cpp

@@ -452,8 +452,8 @@ void TwitchChannel::addChannelPointReward(const ChannelPointReward &reward)
                 if (reward.id == msg.rewardID)
                 {
                     IrcMessageHandler::instance().addMessage(
-                        msg.message.get(), *this, this, msg.originalContent,
-                        *server, false, false);
+                        msg.message.get(), shared_from_this(),
+                        msg.originalContent, *server, false, false);
                     return true;
                 }
                 return false;
@@ -462,14 +462,6 @@ void TwitchChannel::addChannelPointReward(const ChannelPointReward &reward)
     }
 }
 
-void TwitchChannel::addKnownChannelPointReward(const ChannelPointReward &reward)
-{
-    assert(getApp()->isTest());
-
-    auto channelPointRewards = this->channelPointRewards_.access();
-    channelPointRewards->try_emplace(reward.id, reward);
-}
-
 bool TwitchChannel::isChannelPointRewardKnown(const QString &rewardId)
 {
     const auto &pointRewards = this->channelPointRewards_.accessConst();
@@ -1356,6 +1348,8 @@ void TwitchChannel::loadRecentMessages()
                 {
                     msgs.push_back(msg);
                 }
+
+                tc->addRecentChatter(msg->displayName);
             }
 
             getApp()->getTwitch()->getMentionsChannel()->fillInMissingMessages(
@@ -1566,7 +1560,7 @@ void TwitchChannel::refreshBadges()
     getHelix()->getChannelBadges(
         this->roomId(),
         // successCallback
-        [this, weak = weakOf<Channel>(this)](const auto &channelBadges) {
+        [this, weak = weakOf<Channel>(this)](auto channelBadges) {
             auto shared = weak.lock();
             if (!shared)
             {
@@ -1574,7 +1568,31 @@ void TwitchChannel::refreshBadges()
                 return;
             }
 
-            this->addTwitchBadgeSets(channelBadges);
+            auto badgeSets = this->badgeSets_.access();
+
+            for (const auto &badgeSet : channelBadges.badgeSets)
+            {
+                const auto &setID = badgeSet.setID;
+                for (const auto &version : badgeSet.versions)
+                {
+                    auto emote = Emote{
+                        .name = EmoteName{},
+                        .images =
+                            ImageSet{
+                                Image::fromUrl(version.imageURL1x, 1,
+                                               BASE_BADGE_SIZE),
+                                Image::fromUrl(version.imageURL2x, .5,
+                                               BASE_BADGE_SIZE * 2),
+                                Image::fromUrl(version.imageURL4x, .25,
+                                               BASE_BADGE_SIZE * 4),
+                            },
+                        .tooltip = Tooltip{version.title},
+                        .homePage = version.clickURL,
+                    };
+                    (*badgeSets)[setID][version.id] =
+                        std::make_shared<Emote>(emote);
+                }
+            }
         },
         // failureCallback
         [this, weak = weakOf<Channel>(this)](auto error, auto message) {
@@ -1605,33 +1623,6 @@ void TwitchChannel::refreshBadges()
         });
 }
 
-void TwitchChannel::addTwitchBadgeSets(const HelixChannelBadges &channelBadges)
-{
-    auto badgeSets = this->badgeSets_.access();
-
-    for (const auto &badgeSet : channelBadges.badgeSets)
-    {
-        const auto &setID = badgeSet.setID;
-        for (const auto &version : badgeSet.versions)
-        {
-            auto emote = Emote{
-                .name = EmoteName{},
-                .images =
-                    ImageSet{
-                        Image::fromUrl(version.imageURL1x, 1, BASE_BADGE_SIZE),
-                        Image::fromUrl(version.imageURL2x, .5,
-                                       BASE_BADGE_SIZE * 2),
-                        Image::fromUrl(version.imageURL4x, .25,
-                                       BASE_BADGE_SIZE * 4),
-                    },
-                .tooltip = Tooltip{version.title},
-                .homePage = version.clickURL,
-            };
-            (*badgeSets)[setID][version.id] = std::make_shared<Emote>(emote);
-        }
-    }
-}
-
 void TwitchChannel::refreshCheerEmotes()
 {
     getHelix()->getCheermotes(
@@ -1644,75 +1635,74 @@ void TwitchChannel::refreshCheerEmotes()
                 return;
             }
 
-            this->setCheerEmoteSets(cheermoteSets);
+            std::vector<CheerEmoteSet> emoteSets;
+
+            for (const auto &set : cheermoteSets)
+            {
+                auto cheerEmoteSet = CheerEmoteSet();
+                cheerEmoteSet.regex = QRegularExpression(
+                    "^" + set.prefix + "([1-9][0-9]*)$",
+                    QRegularExpression::CaseInsensitiveOption);
+
+                for (const auto &tier : set.tiers)
+                {
+                    CheerEmote cheerEmote;
+
+                    cheerEmote.color = QColor(tier.color);
+                    cheerEmote.minBits = tier.minBits;
+                    cheerEmote.regex = cheerEmoteSet.regex;
+
+                    // TODO(pajlada): We currently hardcode dark here :|
+                    // We will continue to do so for now since we haven't had to
+                    // solve that anywhere else
+
+                    // Combine the prefix (e.g. BibleThump) with the tier (1, 100 etc.)
+                    auto emoteTooltip =
+                        set.prefix + tier.id + "<br>Twitch Cheer Emote";
+                    auto makeImageSet = [](const HelixCheermoteImage &image) {
+                        return ImageSet{
+                            Image::fromUrl(image.imageURL1x, 1.0,
+                                           BASE_BADGE_SIZE),
+                            Image::fromUrl(image.imageURL2x, 0.5,
+                                           BASE_BADGE_SIZE * 2),
+                            Image::fromUrl(image.imageURL4x, 0.25,
+                                           BASE_BADGE_SIZE * 4),
+                        };
+                    };
+                    cheerEmote.animatedEmote = std::make_shared<Emote>(Emote{
+                        .name = EmoteName{"cheer emote"},
+                        .images = makeImageSet(tier.darkAnimated),
+                        .tooltip = Tooltip{emoteTooltip},
+                        .homePage = Url{},
+                    });
+                    cheerEmote.staticEmote = std::make_shared<Emote>(Emote{
+                        .name = EmoteName{"cheer emote"},
+                        .images = makeImageSet(tier.darkStatic),
+                        .tooltip = Tooltip{emoteTooltip},
+                        .homePage = Url{},
+                    });
+
+                    cheerEmoteSet.cheerEmotes.emplace_back(
+                        std::move(cheerEmote));
+                }
+
+                // Sort cheermotes by cost
+                std::sort(cheerEmoteSet.cheerEmotes.begin(),
+                          cheerEmoteSet.cheerEmotes.end(),
+                          [](const auto &lhs, const auto &rhs) {
+                              return lhs.minBits > rhs.minBits;
+                          });
+
+                emoteSets.emplace_back(std::move(cheerEmoteSet));
+            }
+
+            *this->cheerEmoteSets_.access() = std::move(emoteSets);
         },
         [] {
             // Failure
         });
 }
 
-void TwitchChannel::setCheerEmoteSets(
-    const std::vector<HelixCheermoteSet> &cheermoteSets)
-{
-    std::vector<CheerEmoteSet> emoteSets;
-
-    for (const auto &set : cheermoteSets)
-    {
-        auto cheerEmoteSet = CheerEmoteSet();
-        cheerEmoteSet.regex =
-            QRegularExpression("^" + set.prefix + "([1-9][0-9]*)$",
-                               QRegularExpression::CaseInsensitiveOption);
-
-        for (const auto &tier : set.tiers)
-        {
-            CheerEmote cheerEmote;
-
-            cheerEmote.color = QColor(tier.color);
-            cheerEmote.minBits = tier.minBits;
-            cheerEmote.regex = cheerEmoteSet.regex;
-
-            // TODO(pajlada): We currently hardcode dark here :|
-            // We will continue to do so for now since we haven't had to
-            // solve that anywhere else
-
-            // Combine the prefix (e.g. BibleThump) with the tier (1, 100 etc.)
-            auto emoteTooltip = set.prefix + tier.id + "<br>Twitch Cheer Emote";
-            auto makeImageSet = [](const HelixCheermoteImage &image) {
-                return ImageSet{
-                    Image::fromUrl(image.imageURL1x, 1.0, BASE_BADGE_SIZE),
-                    Image::fromUrl(image.imageURL2x, 0.5, BASE_BADGE_SIZE * 2),
-                    Image::fromUrl(image.imageURL4x, 0.25, BASE_BADGE_SIZE * 4),
-                };
-            };
-            cheerEmote.animatedEmote = std::make_shared<Emote>(Emote{
-                .name = EmoteName{u"cheer emote"_s},
-                .images = makeImageSet(tier.darkAnimated),
-                .tooltip = Tooltip{emoteTooltip},
-                .homePage = Url{},
-            });
-            cheerEmote.staticEmote = std::make_shared<Emote>(Emote{
-                .name = EmoteName{u"cheer emote"_s},
-                .images = makeImageSet(tier.darkStatic),
-                .tooltip = Tooltip{emoteTooltip},
-                .homePage = Url{},
-            });
-
-            cheerEmoteSet.cheerEmotes.emplace_back(std::move(cheerEmote));
-        }
-
-        // Sort cheermotes by cost
-        std::sort(cheerEmoteSet.cheerEmotes.begin(),
-                  cheerEmoteSet.cheerEmotes.end(),
-                  [](const auto &lhs, const auto &rhs) {
-                      return lhs.minBits > rhs.minBits;
-                  });
-
-        emoteSets.emplace_back(std::move(cheerEmoteSet));
-    }
-
-    *this->cheerEmoteSets_.access() = std::move(emoteSets);
-}
-
 void TwitchChannel::createClip()
 {
     if (!this->isLive())
@@ -1869,12 +1859,6 @@ std::vector<FfzBadges::Badge> TwitchChannel::ffzChannelBadges(
     return badges;
 }
 
-void TwitchChannel::setFfzChannelBadges(FfzChannelBadgeMap map)
-{
-    this->tgFfzChannelBadges_.guard();
-    this->ffzChannelBadges_ = std::move(map);
-}
-
 std::optional<EmotePtr> TwitchChannel::ffzCustomModBadge() const
 {
     return this->ffzCustomModBadge_.get();
@@ -1885,16 +1869,6 @@ std::optional<EmotePtr> TwitchChannel::ffzCustomVipBadge() const
     return this->ffzCustomVipBadge_.get();
 }
 
-void TwitchChannel::setFfzCustomModBadge(std::optional<EmotePtr> badge)
-{
-    this->ffzCustomModBadge_.set(std::move(badge));
-}
-
-void TwitchChannel::setFfzCustomVipBadge(std::optional<EmotePtr> badge)
-{
-    this->ffzCustomVipBadge_.set(std::move(badge));
-}
-
 std::optional<CheerEmote> TwitchChannel::cheerEmote(const QString &string) const
 {
     auto sets = this->cheerEmoteSets_.access();

src/providers/twitch/TwitchChannel.hpp

@@ -25,8 +25,6 @@
 #include <optional>
 #include <unordered_map>
 
-class TestIrcMessageHandlerP;
-
 namespace chatterino {
 
 enum class HighlightState;
@@ -53,9 +51,6 @@ struct ChannelPointReward;
 class MessageThread;
 struct CheerEmoteSet;
 struct HelixStream;
-struct HelixCheermoteSet;
-struct HelixGlobalBadges;
-using HelixChannelBadges = HelixGlobalBadges;
 
 class TwitchIrcServer;
 
@@ -64,55 +59,22 @@ const int MAX_QUEUED_REDEMPTIONS = 16;
 class TwitchChannel final : public Channel, public ChannelChatters
 {
 public:
-    /**
-     * @lua@class StreamStatus
-     */
     struct StreamStatus {
-        /**
-         * @lua@field live boolean
-         */
         bool live = false;
         bool rerun = false;
-        /**
-         * @lua@field viewer_count number
-         */
         unsigned viewerCount = 0;
-        /**
-         * @lua@field title string Stream title or last stream title
-         */
         QString title;
-        /**
-         * @lua@field game_name string
-         */
         QString game;
-        /**
-         * @lua@field game_id string
-         */
         QString gameId;
         QString uptime;
-        /**
-         * @lua@field uptime number Seconds since the stream started.
-         */
         int uptimeSeconds = 0;
         QString streamType;
         QString streamId;
     };
 
-    /**
-     * @lua@class RoomModes
-     */
     struct RoomModes {
-        /**
-         * @lua@field subscriber_only boolean
-         */
         bool submode = false;
-        /**
-         * @lua@field unique_chat boolean You might know this as r9kbeta or robot9000.
-         */
         bool r9k = false;
-        /**
-         * @lua@field emotes_only boolean Whether or not text is allowed in messages. Note that "emotes" here only means Twitch emotes, not Unicode emoji, nor 3rd party text-based emotes
-         */
         bool emoteOnly = false;
 
         /**
@@ -121,8 +83,6 @@ public:
          * Special cases:
          * -1 = follower mode off
          *  0 = follower mode on, no time requirement
-         *
-         * @lua@field follower_only number? Time in minutes you need to follow to chat or nil.
          **/
         int followerOnly = -1;
 
@@ -130,8 +90,6 @@ public:
          * @brief Number of seconds required to wait before typing emotes
          *
          * 0 = slow mode off
-         *
-         * @lua@field slow_mode number? Time in seconds you need to wait before sending messages or nil.
          **/
         int slowMode = 0;
     };
@@ -237,15 +195,9 @@ public:
      * Returns a list of channel-specific FrankerFaceZ badges for the given user
      */
     std::vector<FfzBadges::Badge> ffzChannelBadges(const QString &userID) const;
-    void setFfzChannelBadges(FfzChannelBadgeMap map);
-    void setFfzCustomModBadge(std::optional<EmotePtr> badge);
-    void setFfzCustomVipBadge(std::optional<EmotePtr> badge);
-
-    void addTwitchBadgeSets(const HelixChannelBadges &channelBadges);
 
     // Cheers
     std::optional<CheerEmote> cheerEmote(const QString &string) const;
-    void setCheerEmoteSets(const std::vector<HelixCheermoteSet> &cheermoteSets);
 
     // Replies
     /**
@@ -291,10 +243,6 @@ public:
      * This will look at queued up partial messages, and if one is found it will add the queued up partial messages fully hydrated.
      **/
     void addChannelPointReward(const ChannelPointReward &reward);
-    /// Adds @a reward to the known rewards
-    ///
-    /// Unlike in #addChannelPointReward(), no message will be sent.
-    void addKnownChannelPointReward(const ChannelPointReward &reward);
     bool isChannelPointRewardKnown(const QString &rewardId);
     std::optional<ChannelPointReward> channelPointReward(
         const QString &rewardId) const;
@@ -501,7 +449,6 @@ private:
     friend class MessageBuilder;
     friend class IrcMessageHandler;
     friend class Commands_E2E_Test;
-    friend class ::TestIrcMessageHandlerP;
 };
 
 }  // namespace chatterino

src/providers/twitch/TwitchEmotes.cpp

@@ -494,14 +494,17 @@ TwitchEmoteSetMeta getTwitchEmoteSetMeta(const HelixChannelEmote &emote)
             return u"x-c2-globals"_s;
         }
 
-        // some bit emote-sets have an id, but we want to combine them into a
-        // single set
-        if (isBits)
+        if (!emote.setID.isEmpty())
         {
-            return TWITCH_BIT_EMOTE_SET_PREFIX % emote.ownerID;
+            return emote.setID;
         }
-        // isSub
-        return TWITCH_SUB_EMOTE_SET_PREFIX % emote.ownerID;
+
+        if (isSub)
+        {
+            return TWITCH_SUB_EMOTE_SET_PREFIX % emote.ownerID;
+        }
+        // isBits
+        return TWITCH_BIT_EMOTE_SET_PREFIX % emote.ownerID;
     }();
 
     return {

src/providers/twitch/TwitchIrc.cpp

@@ -1,166 +0,0 @@
-#include "providers/twitch/TwitchIrc.hpp"
-
-#include "Application.hpp"
-#include "common/Aliases.hpp"
-#include "common/QLogging.hpp"
-#include "singletons/Emotes.hpp"
-#include "util/IrcHelpers.hpp"
-
-namespace {
-
-using namespace chatterino;
-
-void appendTwitchEmoteOccurrences(const QString &emote,
-                                  std::vector<TwitchEmoteOccurrence> &vec,
-                                  const std::vector<int> &correctPositions,
-                                  const QString &originalMessage,
-                                  int messageOffset)
-{
-    auto *app = getApp();
-    if (!emote.contains(':'))
-    {
-        return;
-    }
-
-    auto parameters = emote.split(':');
-
-    if (parameters.length() < 2)
-    {
-        return;
-    }
-
-    auto id = EmoteId{parameters.at(0)};
-
-    auto occurrences = parameters.at(1).split(',');
-
-    for (const QString &occurrence : occurrences)
-    {
-        auto coords = occurrence.split('-');
-
-        if (coords.length() < 2)
-        {
-            return;
-        }
-
-        auto from = coords.at(0).toUInt() - messageOffset;
-        auto to = coords.at(1).toUInt() - messageOffset;
-        auto maxPositions = correctPositions.size();
-        if (from > to || to >= maxPositions)
-        {
-            // Emote coords are out of range
-            qCDebug(chatterinoTwitch)
-                << "Emote coords" << from << "-" << to << "are out of range ("
-                << maxPositions << ")";
-            return;
-        }
-
-        auto start = correctPositions[from];
-        auto end = correctPositions[to];
-        if (start > end || start < 0 || end > originalMessage.length())
-        {
-            // Emote coords are out of range from the modified character positions
-            qCDebug(chatterinoTwitch) << "Emote coords" << from << "-" << to
-                                      << "are out of range after offsets ("
-                                      << originalMessage.length() << ")";
-            return;
-        }
-
-        auto name = EmoteName{originalMessage.mid(start, end - start + 1)};
-        TwitchEmoteOccurrence emoteOccurrence{
-            start,
-            end,
-            app->getEmotes()->getTwitchEmotes()->getOrCreateEmote(id, name),
-            name,
-        };
-        if (emoteOccurrence.ptr == nullptr)
-        {
-            qCDebug(chatterinoTwitch)
-                << "nullptr" << emoteOccurrence.name.string;
-        }
-        vec.push_back(std::move(emoteOccurrence));
-    }
-}
-
-}  // namespace
-
-namespace chatterino {
-
-std::unordered_map<QString, QString> parseBadgeInfoTag(const QVariantMap &tags)
-{
-    std::unordered_map<QString, QString> infoMap;
-
-    auto infoIt = tags.constFind("badge-info");
-    if (infoIt == tags.end())
-    {
-        return infoMap;
-    }
-
-    auto info = infoIt.value().toString().split(',', Qt::SkipEmptyParts);
-
-    for (const QString &badge : info)
-    {
-        infoMap.emplace(slashKeyValue(badge));
-    }
-
-    return infoMap;
-}
-
-std::vector<Badge> parseBadgeTag(const QVariantMap &tags)
-{
-    std::vector<Badge> b;
-
-    auto badgesIt = tags.constFind("badges");
-    if (badgesIt == tags.end())
-    {
-        return b;
-    }
-
-    auto badges = badgesIt.value().toString().split(',', Qt::SkipEmptyParts);
-
-    for (const QString &badge : badges)
-    {
-        if (!badge.contains('/'))
-        {
-            continue;
-        }
-
-        auto pair = slashKeyValue(badge);
-        b.emplace_back(Badge{pair.first, pair.second});
-    }
-
-    return b;
-}
-
-std::vector<TwitchEmoteOccurrence> parseTwitchEmotes(const QVariantMap &tags,
-                                                     const QString &content,
-                                                     int messageOffset)
-{
-    // Twitch emotes
-    std::vector<TwitchEmoteOccurrence> twitchEmotes;
-
-    auto emotesTag = tags.find("emotes");
-
-    if (emotesTag == tags.end())
-    {
-        return twitchEmotes;
-    }
-
-    QStringList emoteString = emotesTag.value().toString().split('/');
-    std::vector<int> correctPositions;
-    for (int i = 0; i < content.size(); ++i)
-    {
-        if (!content.at(i).isLowSurrogate())
-        {
-            correctPositions.push_back(i);
-        }
-    }
-    for (const QString &emote : emoteString)
-    {
-        appendTwitchEmoteOccurrences(emote, twitchEmotes, correctPositions,
-                                     content, messageOffset);
-    }
-
-    return twitchEmotes;
-}
-
-}  // namespace chatterino

src/providers/twitch/TwitchIrc.hpp

@@ -1,67 +0,0 @@
-#pragma once
-
-#include "messages/Emote.hpp"
-#include "providers/twitch/TwitchBadge.hpp"
-
-#include <QString>
-#include <QVariantMap>
-
-#include <unordered_map>
-
-namespace chatterino {
-
-struct TwitchEmoteOccurrence {
-    int start;
-    int end;
-    EmotePtr ptr;
-    EmoteName name;
-
-    bool operator==(const TwitchEmoteOccurrence &other) const
-    {
-        return std::tie(this->start, this->end, this->ptr, this->name) ==
-               std::tie(other.start, other.end, other.ptr, other.name);
-    }
-};
-
-/// @brief Parses the `badge-info` tag of an IRC message
-///
-/// The `badge-info` tag maps badge-names to a value. Subscriber badges, for
-/// example, are mapped to the number of months the chatter is subscribed for.
-///
-/// **Example**:
-/// `badge-info=subscriber/22` would be parsed as `{ subscriber => 22 }`
-///
-/// @param tags The tags of the IRC message
-/// @returns A map of badge-names to their values
-std::unordered_map<QString, QString> parseBadgeInfoTag(const QVariantMap &tags);
-
-/// @brief Parses the `badges` tag of an IRC message
-///
-/// The `badges` tag contains a comma separated list of key-value elements which
-/// make up the name and version of each badge.
-///
-/// **Example**:
-/// `badges=broadcaster/1,subscriber/18` would be parsed as
-/// `[(broadcaster, 1), (subscriber, 18)]`
-///
-/// @param tags The tags of the IRC message
-/// @returns A list of badges (name and version)
-std::vector<Badge> parseBadgeTag(const QVariantMap &tags);
-
-/// @brief Parses Twitch emotes in an IRC message
-///
-/// @param tags The tags of the IRC message
-/// @param content The message text. This might be shortened due to skipping
-///                content at the start. `messageOffset` describes this offset.
-/// @param messageOffset The offset of `content` compared to the original
-///                      message text. Used for calculating indices into the
-///                      message. An offset of 3, for example, indicates that
-///                      `content` excludes the first three characters of the
-///                      original message (`@a foo` (original message) -> `foo`
-///                      (content)).
-/// @returns A list of emotes and their positions
-std::vector<TwitchEmoteOccurrence> parseTwitchEmotes(const QVariantMap &tags,
-                                                     const QString &content,
-                                                     int messageOffset);
-
-}  // namespace chatterino

src/providers/twitch/TwitchIrcServer.cpp

@@ -312,7 +312,7 @@ void TwitchIrcServer::initialize()
             postToThread([chan, action] {
                 MessageBuilder msg(action);
                 msg->flags.set(MessageFlag::PubSub);
-                chan->addOrReplaceTimeout(msg.release(), QTime::currentTime());
+                chan->addOrReplaceTimeout(msg.release());
             });
         });
 

src/providers/twitch/api/Helix.cpp

@@ -1909,7 +1909,7 @@ void Helix::updateChatSettings(
 
 void Helix::onFetchChattersSuccess(
     std::shared_ptr<HelixChatters> finalChatters, QString broadcasterID,
-    QString moderatorID, size_t maxChattersToFetch,
+    QString moderatorID, int maxChattersToFetch,
     ResultCallback<HelixChatters> successCallback,
     FailureCallback<HelixGetChattersError, QString> failureCallback,
     HelixChatters chatters)
@@ -2022,7 +2022,7 @@ void Helix::fetchChatters(
 
 void Helix::onFetchModeratorsSuccess(
     std::shared_ptr<std::vector<HelixModerator>> finalModerators,
-    QString broadcasterID, size_t maxModeratorsToFetch,
+    QString broadcasterID, int maxModeratorsToFetch,
     ResultCallback<std::vector<HelixModerator>> successCallback,
     FailureCallback<HelixGetModeratorsError, QString> failureCallback,
     HelixModerators moderators)
@@ -2459,7 +2459,7 @@ void Helix::sendWhisper(
 
 // https://dev.twitch.tv/docs/api/reference#get-chatters
 void Helix::getChatters(
-    QString broadcasterID, QString moderatorID, size_t maxChattersToFetch,
+    QString broadcasterID, QString moderatorID, int maxChattersToFetch,
     ResultCallback<HelixChatters> successCallback,
     FailureCallback<HelixGetChattersError, QString> failureCallback)
 {

src/providers/twitch/api/Helix.hpp

@@ -1073,7 +1073,7 @@ public:
     // This will follow the returned cursor and return up to `maxChattersToFetch` chatters
     // https://dev.twitch.tv/docs/api/reference#get-chatters
     virtual void getChatters(
-        QString broadcasterID, QString moderatorID, size_t maxChattersToFetch,
+        QString broadcasterID, QString moderatorID, int maxChattersToFetch,
         ResultCallback<HelixChatters> successCallback,
         FailureCallback<HelixGetChattersError, QString> failureCallback) = 0;
 
@@ -1417,7 +1417,7 @@ public:
     // This will follow the returned cursor and return up to `maxChattersToFetch` chatters
     // https://dev.twitch.tv/docs/api/reference#get-chatters
     void getChatters(
-        QString broadcasterID, QString moderatorID, size_t maxChattersToFetch,
+        QString broadcasterID, QString moderatorID, int maxChattersToFetch,
         ResultCallback<HelixChatters> successCallback,
         FailureCallback<HelixGetChattersError, QString> failureCallback) final;
 
@@ -1505,7 +1505,7 @@ protected:
     // Recursive boy
     void onFetchChattersSuccess(
         std::shared_ptr<HelixChatters> finalChatters, QString broadcasterID,
-        QString moderatorID, size_t maxChattersToFetch,
+        QString moderatorID, int maxChattersToFetch,
         ResultCallback<HelixChatters> successCallback,
         FailureCallback<HelixGetChattersError, QString> failureCallback,
         HelixChatters chatters);
@@ -1520,7 +1520,7 @@ protected:
     // Recursive boy
     void onFetchModeratorsSuccess(
         std::shared_ptr<std::vector<HelixModerator>> finalModerators,
-        QString broadcasterID, size_t maxModeratorsToFetch,
+        QString broadcasterID, int maxModeratorsToFetch,
         ResultCallback<std::vector<HelixModerator>> successCallback,
         FailureCallback<HelixGetModeratorsError, QString> failureCallback,
         HelixModerators moderators);

src/singletons/Settings.hpp

@@ -212,10 +212,6 @@ public:
     BoolSetting useCustomFfzVipBadges = {
         "/appearance/badges/useCustomFfzVipBadges", true};
     BoolSetting showBadgesSevenTV = {"/appearance/badges/seventv", true};
-    QSizeSetting lastPopupSize = {
-        "/appearance/lastPopup/size",
-        {300, 500},
-    };
 
     /// Behaviour
     BoolSetting allowDuplicateMessages = {"/behaviour/allowDuplicateMessages",

src/singletons/WindowManager.cpp

@@ -195,7 +195,6 @@ void WindowManager::updateWordTypeMask()
     flags.set(settings->animateEmotes ? MEF::BitsAnimated : MEF::BitsStatic);
 
     // badges
-    flags.set(MEF::BadgeSharedChannel);
     flags.set(settings->showBadgesGlobalAuthority ? MEF::BadgeGlobalAuthority
                                                   : MEF::None);
     flags.set(settings->showBadgesPredictions ? MEF::BadgePredictions

src/singletons/helper/GifTimer.cpp

@@ -26,7 +26,6 @@ void GIFTimer::initialize()
 
     QObject::connect(&this->timer, &QTimer::timeout, [this] {
         if (getSettings()->animationsWhenFocused &&
-            this->openOverlayWindows_ == 0 &&
             QApplication::activeWindow() == nullptr)
         {
             return;

src/singletons/helper/GifTimer.hpp

@@ -18,21 +18,9 @@ public:
         return this->position_;
     }
 
-    void registerOpenOverlayWindow()
-    {
-        this->openOverlayWindows_++;
-    }
-
-    void unregisterOpenOverlayWindow()
-    {
-        assert(this->openOverlayWindows_ >= 1);
-        this->openOverlayWindows_--;
-    }
-
 private:
     QTimer timer;
     long unsigned position_{};
-    size_t openOverlayWindows_ = 0;
 };
 
 }  // namespace chatterino

src/util/Helpers.cpp

@@ -1,6 +1,5 @@
 #include "util/Helpers.hpp"
 
-#include "Application.hpp"
 #include "providers/twitch/TwitchCommon.hpp"
 
 #include <QDirIterator>
@@ -302,16 +301,4 @@ QString unescapeZeroWidthJoiner(QString escaped)
     return escaped;
 }
 
-QLocale getSystemLocale()
-{
-#ifdef CHATTERINO_WITH_TESTS
-    if (getApp()->isTest())
-    {
-        return {QLocale::English};
-    }
-#endif
-
-    return QLocale::system();
-}
-
 }  // namespace chatterino