Skip to content

Remove support for reloading extensions - #2568

Open
Headline wants to merge 3 commits into
masterfrom
h/no-reloaad
Open

Remove support for reloading extensions #2568
Headline wants to merge 3 commits into
masterfrom
h/no-reloaad

Conversation

@Headline

@Headline Headline commented Sep 8, 2026

Copy link
Copy Markdown
Member

There are outstanding issues relating to extension loading / reloading. The feature is trivialized by meta reload - which is perfectly fine for iterating in development.

Existing extensions will need to be recompiled. Might be a good time for it considering recent V10 version. We can't simply stub everything out like I thought previously since third party plugins copied from our example would disregard UnloadExtension()'s return value. Leading to CExtension holding a dangling m_pAPI

AI Disclosure: Used an LLM to workshop, help find references in code, as well as update any documentation that needed changing.

@Kenzzer

Kenzzer commented Sep 8, 2026

Copy link
Copy Markdown
Member

Kudos on the AI disclosure. But unless I missed something, MMS doesn't have a meta reload command. There's meta retry but iirc this command is only for plugins that failed an initial load. So at the moment reloading sourcemod would involve

meta unload X
meta load sourcemod

Perhaps adding a command to unload sourcemod would be nice to have instead ? Metamod takes an ID to unload sourcemod, and I can sense this would become annoying during dev cycles.

sm unload
meta load sourcemod

would work out well, instead of having to find the ID of sourcemod through meta list, some sm extensions can bump the plugin ID numbers as well, so there's no reliable well to guess the ID other than checking. So I would introduce that command

@Headline

Headline commented Sep 9, 2026

Copy link
Copy Markdown
Member Author

Heh I saw a meta reload mentioned somewhere, can't say I actually checked! Yeah we'll definitely need to make that thanks for the note

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants