create-cpp-tests

Create a new cpp_tests entry for validating a C++ interface vtable layout against binary reference YAMLs. Creates the .cpp test file in cpp_tests/ and appends a configs/<GAMEVER>.yaml entry under cpp_

SKILL.md

GitHub

Create cpp_tests for Interface Vtable Validation

Create a cpp_tests/<interface_lowercase>.cpp test file and append a matching configs/<GAMEVER>.yaml entry so that run_cpp_tests.py can compile the header with clang, dump the vtable layout, and compare it against reference YAML files from the binary analysis.

Resolve GAMEVER from the user's explicit request or CS2VIBE_GAMEVER, set the edit target to configs/$GAMEVER.yaml, and stop if that exact file does not exist.

When to Use

  • User asks to create cpp_tests for an interface (e.g., "create cpp_tests for ILoopMode")
  • User provides or references a header file from s2sdk/

Inputs

The user will provide some or all of:

Field Required Description Example
Interface name Yes The abstract class to validate ILoopMode
Header path Yes Path to the header in s2sdk s2sdk/public/iloopmode.h
Alias symbols No Concrete class name(s) used in binary YAML files CLoopModeGame
Reference modules No Modules where vtable YAMLs live client, server

Step-by-Step Procedure

Step 1: Read the target header

Read the header file to understand:

  • What #include directives it uses
  • What types are referenced in virtual method signatures
  • Whether it inherits from another interface (e.g., IAppSystem)

Step 2: Identify compilation dependencies

Check if the header's transitive includes will compile cleanly with the standard include set:

  • s2sdk/game/shared
  • s2sdk/public
  • s2sdk/public/tier0
  • s2sdk/public/tier1

Common issues to watch for:

  • Missing types referenced in method signatures (e.g., CSplitScreenSlot needs <tier1/convar.h>)
  • Heavy transitive includes that pull in protobuf or other unavailable deps (e.g., eiface.h -> protobuf chain)

For heavy transitive includes, pre-define include guards to block them and forward-declare/stub the minimum needed types. See the inetworkmessages.cpp and inetworksystem.cpp examples for this technique:

// Example: blocking heavy include chains
#define EIFACE_H
#define INETCHANNEL_H
enum NetChannelBufType_t : int8 {};

Step 3: Create the cpp test file

Create cpp_tests/<interface_lowercase>.cpp following this template:

#include <tier0/platform.h>
#undef RESTRICT
#define RESTRICT

// Add any extra includes needed for types in the interface signatures
// Add any include-guard stubs to block heavy transitive includes

#include <path/to/interface_header.h>

InterfaceName * instanceptr();

int main() {

    instanceptr()->SomeMethod();

    return 0;
}

Key rules:

  • Always start with the platform.h + RESTRICT preamble
  • The extern function declaration (e.g., ILoopMode * loopmode();) forces the compiler to emit vtable info
  • main() must call at least one virtual method to trigger vtable layout dump
  • Pick a simple void-returning method with no complex args for the call in main()

Step 4: Determine alias_symbols and reference_modules

If the user didn't provide these, discover them:

  1. alias_symbols: Search the tracked source-owned vtable artifacts:

    bin_artifacts/<GAMEVER>/**/*<ClassName>*vtable*
    

    The vtable_class field in those YAMLs gives the alias symbol (typically the concrete class name like CLoopModeGame for ILoopMode).

  2. reference_modules: The subdirectories under bin_artifacts/<GAMEVER>/ where vtable YAMLs exist (e.g., client, server, engine, networksystem).

Step 5: Append configs/<GAMEVER>.yaml entry

Append a new entry at the end of the cpp_tests: section in configs/<GAMEVER>.yaml:

  - name: {InterfaceName}_MSVC
    symbol: {InterfaceName}
    alias_symbols:                          # omit this block if no aliases
      - {ConcreteClassName}
    cpp: cpp_tests/{interface_lowercase}.cpp
    headers:
    - {header_path} # Used by the fix-cppheaders SKILL
    target: x86_64-pc-windows-msvc
    include_directories:
      - s2sdk/game/shared
      - s2sdk/public
      - s2sdk/public/tier0
      - s2sdk/public/tier1
    defines:
      - COMPILER_MSVC=1
      - COMPILER_MSVC64=1
      - _MSVC_STL_USE_ABORT_AS_DOOM_FUNCTION
    additional_compiler_options:
      - fms-extensions
      - fms-compatibility
      - Xclang
      - fdump-vtable-layouts
    reference_modules:
      - {module1} # bin_artifacts/{gamever}/{module1}/{AliasOrSymbol}_*.{platform}.yaml
      - {module2}

Notes:

  • include_directories, defines, and additional_compiler_options are always the same standard set
  • reference_modules comments should document the YAML file naming pattern
  • If no alias_symbols, the reference YAML files use the symbol name directly

Step 6: Commit Changes to dev

After validation passes, ensure the delivery branch is dev. Never commit directly to main. If the local dev branch exists, switch to it. Otherwise, switch to main first and create dev from main:

if git show-ref --verify --quiet refs/heads/dev; then
  git switch dev
else
  git switch main
  git switch -c dev
fi

If any branch switch fails, stop and report the error. Review git status --short, then explicitly stage only the new test and its config entry:

git add -- cpp_tests/{interface_lowercase}.cpp configs/<GAMEVER>.yaml
git diff --cached --name-only

Never use git add -A. Stop if the staged-path list contains anything unrelated to this task. Commit only the staged task changes using the repository commit format:

git commit -m "test(cpp-tests): add {InterfaceName} vtable validation" -m "Co-Authored-By: Codex <codex@openai.com>"

Do not call /create-pr, push the branch, or open a pull request unless the user separately requests it. Finish by reporting the commit hash and the validation results.

Checklist

  • New cpp test follows the platform/RESTRICT preamble and calls a virtual method
  • configs/<GAMEVER>.yaml entry contains the correct symbol, aliases, header, and reference modules
  • The current branch is dev (created from main when it did not already exist)
  • Only the new test and config entry are explicitly staged and committed
  • /create-pr was not called; no push or PR was performed without a separate user request

Reference: Existing Examples

Test Name Interface Header Has Aliases Has Include Stubs Reference Modules
IGameSystem_MSVC IGameSystem igamesystem.h No No server, client
INetworkMessages_MSVC INetworkMessages inetworkmessages.h CNetworkMessages Yes (EIFACE_H, INETCHANNEL_H) networksystem, engine, server, client
ILoopMode_MSVC ILoopMode iloopmode.h CLoopModeGame No (just extra convar.h include) client, server