Skip to content

feat(api): Add CreateMenu API function#24

Merged
Rushaway merged 8 commits intosrcdslab:masterfrom
Dolly132:main-dolly
Mar 3, 2026
Merged

feat(api): Add CreateMenu API function#24
Rushaway merged 8 commits intosrcdslab:masterfrom
Dolly132:main-dolly

Conversation

@Dolly132
Copy link
Contributor

@Dolly132 Dolly132 commented Mar 1, 2026

This PR is mainly made to make Torchlight !search command work with a menu

Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR extends the SM JSON API plugin to support creating SourceMod menus from JSON payloads, enabling external integrations (e.g., Torchlight !search) to present results as an in-game menu.

Changes:

  • Added a new API function API_CreateMenu plus a menu callback that publishes an OnMenuSelect forward event.
  • Extended request argument handling to accept JSON objects as function arguments and echo them back in the response args.
  • Bumped plugin version to 1.1.0.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 6 comments.

File Description
addons/sourcemod/scripting/include/API.inc Adds the new menu-creation API and menu selection publish callback.
addons/sourcemod/scripting/SMJSONAPI.sp Adds JSON object argument support in the dynamic request function argument handler.

Dolly132 and others added 3 commits March 2, 2026 23:19
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Copilot reviewed 2 out of 2 changed files in this pull request and generated 2 comments.

@Rushaway Rushaway merged commit c7b281c into srcdslab:master Mar 3, 2026
3 checks passed
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.

3 participants