WIP: Initial parts of Steam Input API support by RobertCochran · Pull Request #1695 · redeclipse/base

RobertCochran · 2026-01-28T06:36:34Z

WIP for Steam Input support, as requested on Discord.

There are several things that don't work, aren't implemented the way I'm supposed to, etc.

Perhaps the most important thing to note is that I broke the build for the Red Eclipse server. That needs addressing but it's not short-term important.

RobertCochran · 2026-01-28T06:37:14Z

bin/amd64/game_actions_967460.vdf

Steam Input action definitions file. Steam reads this to know what actions are available and what kind of actions they are. This is still WIP.

RobertCochran · 2026-01-28T06:38:05Z

src/engine/cdpi.cpp

        ISteamUserStats *stats = NULL;
        ISteamClient *client = NULL, *sclient = NULL;
        ISteamGameServer *serv = NULL;
+	ISteamInput *input = NULL;


There are a lot of places where the whitespace is inconsistent. Sorry. I'll be sure it's fixed when I am ready to actually get this PR reviewed.

RobertCochran · 2026-01-28T06:39:16Z

src/game/game.cpp

            if(d)
            {
-                float scale = (focus == player1 && inzoom() && zoomsensitivity > 0 ? (1.f-((zoomlevel+1)/float(zoomlevels+2)))*zoomsensitivity : 1.f)*sensitivity;
+                float scale = zoomsens()*sensitivity;


The zoom scaling code got factored out into a helper function so that it could be used inside the controller code.

RobertCochran · 2026-01-28T06:39:35Z

src/game/game.h

    f += (bf-f)*amt;
 }

+#define mousesens(a,b,c) ((float(a)/float(b))*c)


This macro moved here to the header so that it could be used inside the controller code.

Jigoku · 2026-01-28T08:31:41Z

Perhaps the most important thing to note is that I broke the build for the Red Eclipse server. That needs addressing but it's not short-term important.

For this, Looks like you need to add game/controller.o to CLIENT_OBJS in the Makefile so that the compiler can link it 👍

RobertCochran · 2026-02-03T08:17:39Z

There's been some more stuff since I posted

RobertCochran · 2026-02-03T08:22:29Z

config/ui/game/settings.cfg

Settings menu item for glyph prompts. 'Automatic' is probably what most people want (dynamically changes glyphs based on last input), but - as is considered good practice - there are options to explicitly lock glyphs if needed. Because Steam Input handles different hardware types for us, the only glyph distinction we need make is KB/M vs controller. This is technically an interface thing, but I feel like it makes more sense to have it together with the controls options.

KNOWN ISSUE 1: The box for this is weirdly not as wide as the other boxes in the menu. I probably copied a setting I didn't intend.
KNOWN ISSUE 2: This box needs to be hidden for non-Steam builds since it will be connected to a variable that will do nothing without SIAPI support.

It may not be worth fiddling with this much because I still need to add a button to open up the controller configurator interface provided by Steam.

RobertCochran · 2026-02-03T08:24:02Z

config/keymap.cfg

+// Steam Input API controller actions
+keymap -20 SIAPI_PRIMARY
+keymap -21 SIAPI_SECONDARY
+keymap -22 SIAPI_RELOAD
+keymap -23 SIAPI_USE
+keymap -24 SIAPI_JUMP
+keymap -25 SIAPI_WALK
+keymap -26 SIAPI_CROUCH
+keymap -27 SIAPI_SPECIAL
+keymap -28 SIAPI_DROP
+keymap -29 SIAPI_AFFINITY
+keymap -30 SIAPI_DASH
+keymap -31 SIAPI_NEXT_WEAPON
+keymap -32 SIAPI_PREVIOUS_WEAPON
+keymap -33 SIAPI_PRIMARY_WEAPON
+keymap -34 SIAPI_SECONDARY_WEAPON
+keymap -35 SIAPI_WHEEL_SELECT
+keymap -36 SIAPI_CHANGE_LOADOUT
+keymap -37 SIAPI_SCOREBOARD
+keymap -38 SIAPI_SUICIDE
+keymap -39 SIAPI_MENU


I added the SIAPI actions to the keymap, as requested. Because only the SIAPI support code should hit these 'keys' in the keymap and there is an enum to map the codes to nice names in the code, this keymap range can be moved theoretically at any time without risk of breaking anything.

RobertCochran · 2026-02-03T08:25:00Z

config/setup.cfg

+// Steam Input controller binds; these actions _should not be rebound_ - do it
+// through the Steam Input configuration interface
+bind SIAPI_PRIMARY          [ primary ]
+specbind SIAPI_PRIMARY      [ spectate 0 ]
+bind SIAPI_SECONDARY        [ secondary ]
+specbind SIAPI_SECONDARY    [ spectate 0 ]
+bind SIAPI_WHEEL_SELECT     [ game_hud_piemenu_open_weapsel_key ] // Currently broken
+bind SIAPI_NEXT_WEAPON      [ universaldelta 1 ]
+bind SIAPI_PREVIOUS_WEAPON  [ universaldelta -1 ]
+bind SIAPI_PRIMARY_WEAPON   [ weapon (weapload 0) 1 ]
+bind SIAPI_SECONDARY_WEAPON [ weapon (weapload 1) 1 ]
+bind SIAPI_MENU             [ uitoggle ]
+bind SIAPI_JUMP             [ jump ]
+specbind SIAPI_JUMP         [ specmodeswitch ]
+bind SIAPI_SPECIAL          [ special ]
+bind SIAPI_CROUCH           [ crouch ]
+bind SIAPI_USE              [ use ]
+specbind SIAPI_USE          [ spectate 0 ]
+bind SIAPI_RELOAD           [ reload ]
+specbind SIAPI_RELOAD       [ specmodeswitch ]
+bind SIAPI_DROP             [ drop ]
+bind SIAPI_AFFINITY         [ affinity ]
+bind SIAPI_SCOREBOARD       [ showscores ]
+bind SIAPI_CHANGE_LOADOUT   [ gameui_player_show_loadout ]
+bind SIAPI_SUICIDE          [ suicide ]
+


Bind the SIAPI actions. You are not supposed to rebind these actions in-engine; you are intended to use the Steam Input configurator tool and change things there.

RobertCochran · 2026-02-03T08:25:57Z

src/engine/rendertext.cpp

-struct textkey
-{
-    char *name, *file;
-    Texture *tex;
-    textkey() : name(NULL), file(NULL), tex(NULL) {}
-    textkey(char *n, char *f, Texture *t) : name(newstring(n)), file(newstring(f)), tex(t) {}
-    ~textkey()
-    {
-        DELETEA(name);
-        DELETEA(file);
-    }
-};


This moved to rendertext.h because I needed the struct to be available elsewhere.

RobertCochran · 2026-02-03T08:27:06Z

src/engine/rendertext.cpp

+bool shouldkeepkey(const char *str)
+{
+	bool is_siapi_textkey = controller::is_siapi_textkey(str);
+	switch (textkeyimagepreference) {
+	case tkip_automatic:
+		return controller::lastinputwassiapi ? is_siapi_textkey : !is_siapi_textkey;
+	case tkip_kbm:
+		return !is_siapi_textkey;
+	case tkip_controller:
+		return is_siapi_textkey;
+	case tkip_both:
+		return true;
+	};
+}
+


Pursuant to above variable, there is now support in the textkey system to filter out glyphs

RobertCochran · 2026-02-03T08:32:14Z

src/game/controller.cpp

+		das->tk->file = NULL; // we don't use this here
+		das->tk->name = newstring(str);
+	}
+	// We have to check if the origin has changed, and if so, reload the texture


https://partner.steamgames.com/doc/features/steam_controller/getting_started_for_devs says that

Please note that the user can change their configuration at any time. To help with this, we've ensured the ISteamInput::GetDigitalActionOrigins and ISteamInput::GetAnalogActionOrigins functions are extremely cheap to call.

When you display an onscreen prompt, don't cache the origins and keep displaying the first results. Instead, we recommend you re-gather the origins each frame, and display the matching prompts. That way, if a user decides to change their configuration as a result of seeing the prompt, when they return from the configuration screen the prompts will automatically update to match the new origins.

In order to honor this best practice, textkeys related to SIAPI actions are handled differently and not cached in the same way that regular KB/M ones are.

RobertCochran · 2026-02-03T08:33:44Z

src/game/controller.cpp

+public:
+	InputDigitalActionHandle_t handle = -1;
+	int keymap_id = -1;
+	// Should be more than one origin eventually!


SIAPI actions can have up to 8 origins (which is SIAPIese for 'the button this action is bound to'), but this implementation doesn't yet support actually using them all. It's going to require a non-trivial refactoring of the textkey rendering system to support this.

WIP: Initial parts of Steam Input API support

9c723da

RobertCochran requested a review from a team as a code owner January 28, 2026 06:36

RobertCochran commented Jan 28, 2026

View reviewed changes

qreeves marked this pull request as draft January 28, 2026 08:18

qreeves requested review from qreeves and removed request for a team January 28, 2026 08:18

qreeves added status: enhancement priority: high branch: master difficulty: hard labels Jan 28, 2026

qreeves added this to the 2.1.0 milestone Jan 28, 2026

RobertCochran added 6 commits January 28, 2026 20:09

Remove leftover debug include

6859ab8

Don't require both X and Y movement to consider it a controller move

0a13193

Move to using keymaps instead of directly calling doaction

d6d88e9

Make the handle and keymap_id public members

57e5cbd

Deduplicate function

c82ecbe

Initial glyph implementation

af30273

RobertCochran commented Feb 3, 2026

View reviewed changes

Uh oh!

Conversation

RobertCochran commented Jan 28, 2026

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Jigoku commented Jan 28, 2026

Uh oh!

RobertCochran commented Feb 3, 2026

Uh oh!

RobertCochran Feb 3, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

3 participants

RobertCochran Feb 3, 2026 •

edited

Loading