Skip to content

Runtime Bindings Inspection #219

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
kunitoki wants to merge 15 commits into
base: master
Choose a base branch
from
Open

Runtime Bindings Inspection #219

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
15 commits
Select commit Hold shift + click to select a range
d6a93cd
Inspection first check in
kunitoki Apr 7, 2026
a2b0a32
Fix inspection
kunitoki Apr 7, 2026
3f65917
Merge branch 'master' into dev/inspect
kunitoki Apr 7, 2026
0c1de9d
Move tools and expand to more test runs
kunitoki Apr 7, 2026
7e9ddf0
Fix github actions
kunitoki Apr 7, 2026
c47b829
Fix warnings
kunitoki Apr 7, 2026
586f7bc
More cleanups
kunitoki Apr 7, 2026
ac79168
Disable inspection when no REFLECT is defined
kunitoki Apr 7, 2026
0da524f
More docs
kunitoki Apr 7, 2026
572d45e
Better manual
kunitoki Apr 7, 2026
7f92ff6
Update Manual.md
kunitoki Apr 7, 2026
1e458c2
Update Manual.md
kunitoki Apr 7, 2026
9d28764
Fix amalgama
kunitoki Apr 7, 2026
e035279
fix: LuaTableVisitor nested namespace handling with state stack
Copilot Apr 7, 2026
c0c14d7
Updated headers and manual
kunitoki Apr 7, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
6 changes: 3 additions & 3 deletions .github/workflows/amalgamate.yml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -6,8 +6,8 @@ on:
- master
paths:
- "**/workflows/amalgamate.yml"
- "**/Source/**"
- "**/amalgamate.py"
- "**/Source/**/*.h"
- "**/Tools/amalgamate.py"

jobs:
run:
Expand All @@ -23,7 +23,7 @@ jobs:
python-version: 3.9

- name: Create amalgamation file
run: python amalgamate.py
run: python Tools/amalgamate.py

- name: Commit changes
uses: EndBug/add-and-commit@v10
Expand Down
4 changes: 4 additions & 0 deletions .github/workflows/build_asan.yml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -53,8 +53,10 @@ jobs:
run: |
cmake --build . --config $BUILD_TYPE --parallel $(nproc) --target \
LuaBridgeTests${{ matrix.lua.suffix }} \
LuaBridgeTests${{ matrix.lua.suffix }}Reflect \
LuaBridgeTests${{ matrix.lua.suffix }}LuaC \
LuaBridgeTests${{ matrix.lua.suffix }}Noexcept \
LuaBridgeTests${{ matrix.lua.suffix }}ReflectNoexcept \
LuaBridgeTests${{ matrix.lua.suffix }}LuaCNoexcept

- name: Test Lua ${{ matrix.lua.version }}
Expand All @@ -63,6 +65,8 @@ jobs:
ASAN_OPTIONS: detect_leaks=0:detect_odr_violation=0
run: |
./LuaBridgeTests${{ matrix.lua.suffix }}
./LuaBridgeTests${{ matrix.lua.suffix }}Reflect
./LuaBridgeTests${{ matrix.lua.suffix }}LuaC
./LuaBridgeTests${{ matrix.lua.suffix }}Noexcept
./LuaBridgeTests${{ matrix.lua.suffix }}ReflectNoexcept
./LuaBridgeTests${{ matrix.lua.suffix }}LuaCNoexcept
28 changes: 23 additions & 5 deletions .github/workflows/build_linux.yml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -53,16 +53,20 @@ jobs:
run: |
cmake --build . --config $BUILD_TYPE --parallel $(nproc) --target \
LuaBridgeTests${{ matrix.lua.suffix }} \
LuaBridgeTests${{ matrix.lua.suffix }}Reflect \
LuaBridgeTests${{ matrix.lua.suffix }}LuaC \
LuaBridgeTests${{ matrix.lua.suffix }}Noexcept \
LuaBridgeTests${{ matrix.lua.suffix }}ReflectNoexcept \
LuaBridgeTests${{ matrix.lua.suffix }}LuaCNoexcept

- name: Test Lua ${{ matrix.lua.version }}
working-directory: ${{runner.workspace}}/build/Tests
run: |
./LuaBridgeTests${{ matrix.lua.suffix }}
./LuaBridgeTests${{ matrix.lua.suffix }}Reflect
./LuaBridgeTests${{ matrix.lua.suffix }}LuaC
./LuaBridgeTests${{ matrix.lua.suffix }}Noexcept
./LuaBridgeTests${{ matrix.lua.suffix }}ReflectNoexcept
./LuaBridgeTests${{ matrix.lua.suffix }}LuaCNoexcept

luajit:
Expand Down Expand Up @@ -90,13 +94,17 @@ jobs:
run: |
cmake --build . --config $BUILD_TYPE --parallel $(nproc) --target \
LuaBridgeTestsLuaJIT \
LuaBridgeTestsLuaJITNoexcept
LuaBridgeTestsLuaJITReflect \
LuaBridgeTestsLuaJITNoexcept \
LuaBridgeTestsLuaJITReflectNoexcept

- name: Test LuaJIT
working-directory: ${{runner.workspace}}/build/Tests
run: |
./LuaBridgeTestsLuaJIT
./LuaBridgeTestsLuaJITReflect
./LuaBridgeTestsLuaJITNoexcept
./LuaBridgeTestsLuaJITReflectNoexcept

luau:
runs-on: ubuntu-latest
Expand All @@ -118,11 +126,16 @@ jobs:

- name: Build Luau
working-directory: ${{runner.workspace}}/build
run: cmake --build . --config $BUILD_TYPE --parallel $(nproc) --target LuaBridgeTestsLuau
run: |
cmake --build . --config $BUILD_TYPE --parallel $(nproc) --target \
LuaBridgeTestsLuau \
LuaBridgeTestsLuauReflect

- name: Test Luau
working-directory: ${{runner.workspace}}/build/Tests
run: ./LuaBridgeTestsLuau
run: |
./LuaBridgeTestsLuau
./LuaBridgeTestsLuauReflect

ravi:
runs-on: ubuntu-latest
Expand All @@ -144,8 +157,13 @@ jobs:

- name: Build Ravi
working-directory: ${{runner.workspace}}/build
run: cmake --build . --config $BUILD_TYPE --parallel $(nproc) --target LuaBridgeTestsRavi
run: |
cmake --build . --config $BUILD_TYPE --parallel $(nproc) --target \
LuaBridgeTestsRavi \
LuaBridgeTestsRaviReflect

- name: Test Ravi
working-directory: ${{runner.workspace}}/build/Tests
run: ./LuaBridgeTestsRavi
run: |
./LuaBridgeTestsRavi
./LuaBridgeTestsRaviReflect
28 changes: 23 additions & 5 deletions .github/workflows/build_macos.yml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -50,16 +50,20 @@ jobs:
run: |
cmake --build . --config $BUILD_TYPE --parallel $(nproc) --target \
LuaBridgeTests${{ matrix.lua.suffix }} \
LuaBridgeTests${{ matrix.lua.suffix }}Reflect \
LuaBridgeTests${{ matrix.lua.suffix }}LuaC \
LuaBridgeTests${{ matrix.lua.suffix }}Noexcept \
LuaBridgeTests${{ matrix.lua.suffix }}ReflectNoexcept \
LuaBridgeTests${{ matrix.lua.suffix }}LuaCNoexcept

- name: Test Lua ${{ matrix.lua.version }}
working-directory: ${{runner.workspace}}/build/Tests
run: |
./LuaBridgeTests${{ matrix.lua.suffix }}
./LuaBridgeTests${{ matrix.lua.suffix }}Reflect
./LuaBridgeTests${{ matrix.lua.suffix }}LuaC
./LuaBridgeTests${{ matrix.lua.suffix }}Noexcept
./LuaBridgeTests${{ matrix.lua.suffix }}ReflectNoexcept
./LuaBridgeTests${{ matrix.lua.suffix }}LuaCNoexcept

luajit:
Expand All @@ -82,13 +86,17 @@ jobs:
run: |
cmake --build . --config $BUILD_TYPE --parallel $(nproc) --target \
LuaBridgeTestsLuaJIT \
LuaBridgeTestsLuaJITNoexcept
LuaBridgeTestsLuaJITReflect \
LuaBridgeTestsLuaJITNoexcept \
LuaBridgeTestsLuaJITReflectNoexcept

- name: Test LuaJIT
working-directory: ${{runner.workspace}}/build/Tests
run: |
./LuaBridgeTestsLuaJIT
./LuaBridgeTestsLuaJITReflect
./LuaBridgeTestsLuaJITNoexcept
./LuaBridgeTestsLuaJITReflectNoexcept

luau:
runs-on: macos-latest
Expand All @@ -107,11 +115,16 @@ jobs:

- name: Build Luau
working-directory: ${{runner.workspace}}/build
run: cmake --build . --config $BUILD_TYPE --parallel $(nproc) --target LuaBridgeTestsLuau
run: |
cmake --build . --config $BUILD_TYPE --parallel $(nproc) --target \
LuaBridgeTestsLuau \
LuaBridgeTestsLuauReflect

- name: Test Luau
working-directory: ${{runner.workspace}}/build/Tests
run: ./LuaBridgeTestsLuau
run: |
./LuaBridgeTestsLuau
./LuaBridgeTestsLuauReflect

ravi:
runs-on: macos-latest
Expand All @@ -130,8 +143,13 @@ jobs:

- name: Build Ravi
working-directory: ${{runner.workspace}}/build
run: cmake --build . --config $BUILD_TYPE --parallel $(nproc) --target LuaBridgeTestsRavi
run: |
cmake --build . --config $BUILD_TYPE --parallel $(nproc) --target \
LuaBridgeTestsRavi \
LuaBridgeTestsRaviReflect

- name: Test Ravi
working-directory: ${{runner.workspace}}/build/Tests
run: ./LuaBridgeTestsRavi
run: |
./LuaBridgeTestsRavi
./LuaBridgeTestsRaviReflect
4 changes: 4 additions & 0 deletions .github/workflows/build_tsan.yml_
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -53,8 +53,10 @@ jobs:
run: |
cmake --build . --config $BUILD_TYPE --parallel $(nproc) --target \
LuaBridgeTests${{ matrix.lua.suffix }} \
LuaBridgeTests${{ matrix.lua.suffix }}Reflect \
LuaBridgeTests${{ matrix.lua.suffix }}LuaC \
LuaBridgeTests${{ matrix.lua.suffix }}Noexcept \
LuaBridgeTests${{ matrix.lua.suffix }}ReflectNoexcept \
LuaBridgeTests${{ matrix.lua.suffix }}LuaCNoexcept

- name: Test Lua ${{ matrix.lua.version }}
Expand All @@ -63,6 +65,8 @@ jobs:
TSAN_OPTIONS: halt_on_error=1:print_stacktrace=1
run: |
./LuaBridgeTests${{ matrix.lua.suffix }}
./LuaBridgeTests${{ matrix.lua.suffix }}Reflect
./LuaBridgeTests${{ matrix.lua.suffix }}LuaC
./LuaBridgeTests${{ matrix.lua.suffix }}Noexcept
./LuaBridgeTests${{ matrix.lua.suffix }}ReflectNoexcept
./LuaBridgeTests${{ matrix.lua.suffix }}LuaCNoexcept
4 changes: 4 additions & 0 deletions .github/workflows/build_ubsan.yml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -53,8 +53,10 @@ jobs:
run: |
cmake --build . --config $BUILD_TYPE --parallel $(nproc) --target \
LuaBridgeTests${{ matrix.lua.suffix }} \
LuaBridgeTests${{ matrix.lua.suffix }}Reflect \
LuaBridgeTests${{ matrix.lua.suffix }}LuaC \
LuaBridgeTests${{ matrix.lua.suffix }}Noexcept \
LuaBridgeTests${{ matrix.lua.suffix }}ReflectNoexcept \
LuaBridgeTests${{ matrix.lua.suffix }}LuaCNoexcept

- name: Test Lua ${{ matrix.lua.version }}
Expand All @@ -63,6 +65,8 @@ jobs:
UBSAN_OPTIONS: halt_on_error=1:print_stacktrace=1
run: |
./LuaBridgeTests${{ matrix.lua.suffix }}
./LuaBridgeTests${{ matrix.lua.suffix }}Reflect
./LuaBridgeTests${{ matrix.lua.suffix }}LuaC
./LuaBridgeTests${{ matrix.lua.suffix }}Noexcept
./LuaBridgeTests${{ matrix.lua.suffix }}ReflectNoexcept
./LuaBridgeTests${{ matrix.lua.suffix }}LuaCNoexcept
24 changes: 20 additions & 4 deletions .github/workflows/build_windows.yml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -52,17 +52,21 @@ jobs:
run: |
cmake --build . --config $BUILD_TYPE --parallel 4 --target \
LuaBridgeTests${{ matrix.lua.suffix }} \
LuaBridgeTests${{ matrix.lua.suffix }}Reflect \
LuaBridgeTests${{ matrix.lua.suffix }}LuaC \
LuaBridgeTests${{ matrix.lua.suffix }}Noexcept \
LuaBridgeTests${{ matrix.lua.suffix }}ReflectNoexcept \
LuaBridgeTests${{ matrix.lua.suffix }}LuaCNoexcept

- name: Test Lua ${{ matrix.lua.version }}
working-directory: ${{runner.workspace}}/build/Tests/Release
shell: bash
run: |
./LuaBridgeTests${{ matrix.lua.suffix }}.exe
./LuaBridgeTests${{ matrix.lua.suffix }}Reflect.exe
./LuaBridgeTests${{ matrix.lua.suffix }}LuaC.exe
./LuaBridgeTests${{ matrix.lua.suffix }}Noexcept.exe
./LuaBridgeTests${{ matrix.lua.suffix }}ReflectNoexcept.exe
./LuaBridgeTests${{ matrix.lua.suffix }}LuaCNoexcept.exe

luajit:
Expand All @@ -87,14 +91,18 @@ jobs:
run: |
cmake --build . --config $BUILD_TYPE --parallel 4 --target \
LuaBridgeTestsLuaJIT \
LuaBridgeTestsLuaJITNoexcept
LuaBridgeTestsLuaJITReflect \
LuaBridgeTestsLuaJITNoexcept \
LuaBridgeTestsLuaJITReflectNoexcept

- name: Test LuaJIT
working-directory: ${{runner.workspace}}/build/Tests/Release
shell: bash
run: |
./LuaBridgeTestsLuaJIT.exe
./LuaBridgeTestsLuaJITReflect.exe
./LuaBridgeTestsLuaJITNoexcept.exe
./LuaBridgeTestsLuaJITReflectNoexcept.exe

luau:
runs-on: windows-latest
Expand All @@ -115,12 +123,17 @@ jobs:
- name: Build Luau
working-directory: ${{runner.workspace}}/build
shell: bash
run: cmake --build . --config $BUILD_TYPE --parallel 4 --target LuaBridgeTestsLuau
run: |
cmake --build . --config $BUILD_TYPE --parallel 4 --target \
LuaBridgeTestsLuau \
LuaBridgeTestsLuauReflect

- name: Test Luau
working-directory: ${{runner.workspace}}/build/Tests/Release
shell: bash
run: ./LuaBridgeTestsLuau.exe
run: |
./LuaBridgeTestsLuau.exe
./LuaBridgeTestsLuauReflect.exe

ravi:
runs-on: windows-latest
Expand All @@ -141,7 +154,10 @@ jobs:
- name: Build Ravi
working-directory: ${{runner.workspace}}/build
shell: bash
run: cmake --build . --config $BUILD_TYPE --parallel 4 --target LuaBridgeTestsRavi
run: |
cmake --build . --config $BUILD_TYPE --parallel 4 --target \
LuaBridgeTestsRavi \
LuaBridgeTestsRaviReflect

- name: Test Ravi
working-directory: ${{runner.workspace}}/build/Tests/Release
Expand Down
30 changes: 24 additions & 6 deletions .github/workflows/coverage.yml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -55,16 +55,20 @@ jobs:
run: |
cmake --build . --config $BUILD_TYPE --parallel $(nproc) --target \
LuaBridgeTests${{ matrix.lua.suffix }} \
LuaBridgeTests${{ matrix.lua.suffix }}Reflect \
LuaBridgeTests${{ matrix.lua.suffix }}LuaC \
LuaBridgeTests${{ matrix.lua.suffix }}Noexcept \
LuaBridgeTests${{ matrix.lua.suffix }}ReflectNoexcept \
LuaBridgeTests${{ matrix.lua.suffix }}LuaCNoexcept

- name: Test Lua ${{ matrix.lua.version }}
working-directory: ${{runner.workspace}}/build/Tests
run: |
./LuaBridgeTests${{ matrix.lua.suffix }}
./LuaBridgeTests${{ matrix.lua.suffix }}Reflect
./LuaBridgeTests${{ matrix.lua.suffix }}LuaC
./LuaBridgeTests${{ matrix.lua.suffix }}Noexcept
./LuaBridgeTests${{ matrix.lua.suffix }}ReflectNoexcept
./LuaBridgeTests${{ matrix.lua.suffix }}LuaCNoexcept

- name: Coverage Lua ${{ matrix.lua.version }}
Expand Down Expand Up @@ -106,13 +110,17 @@ jobs:
run: |
cmake --build . --config $BUILD_TYPE --parallel $(nproc) --target \
LuaBridgeTestsLuaJIT \
LuaBridgeTestsLuaJITNoexcept
LuaBridgeTestsLuaJITReflect \
LuaBridgeTestsLuaJITNoexcept \
LuaBridgeTestsLuaJITReflectNoexcept

- name: Test LuaJIT
working-directory: ${{runner.workspace}}/build/Tests
run: |
./LuaBridgeTestsLuaJIT
./LuaBridgeTestsLuaJITReflect
./LuaBridgeTestsLuaJITNoexcept
./LuaBridgeTestsLuaJITReflectNoexcept

- name: Coverage LuaJIT
working-directory: ${{runner.workspace}}/build
Expand Down Expand Up @@ -150,11 +158,16 @@ jobs:

- name: Build Luau
working-directory: ${{runner.workspace}}/build
run: cmake --build . --config $BUILD_TYPE --parallel $(nproc) --target LuaBridgeTestsLuau
run: |
cmake --build . --config $BUILD_TYPE --parallel $(nproc) --target \
LuaBridgeTestsLuau \
LuaBridgeTestsLuauReflect

- name: Test Luau
working-directory: ${{runner.workspace}}/build/Tests
run: ./LuaBridgeTestsLuau
run: |
./LuaBridgeTestsLuau
./LuaBridgeTestsLuauReflect

- name: Coverage Luau
working-directory: ${{runner.workspace}}/build
Expand Down Expand Up @@ -192,11 +205,16 @@ jobs:

- name: Build Ravi
working-directory: ${{runner.workspace}}/build
run: cmake --build . --config $BUILD_TYPE --parallel $(nproc) --target LuaBridgeTestsRavi
run: |
cmake --build . --config $BUILD_TYPE --parallel $(nproc) --target \
LuaBridgeTestsRavi \
LuaBridgeTestsRaviReflect

- name: Test Ravi
working-directory: ${{runner.workspace}}/build/Tests
run: LD_PRELOAD=$(gcc -print-file-name=libasan.so) ./LuaBridgeTestsRavi
run: |
LD_PRELOAD=$(gcc -print-file-name=libasan.so) ./LuaBridgeTestsRavi
LD_PRELOAD=$(gcc -print-file-name=libasan.so) ./LuaBridgeTestsRaviReflect

- name: Coverage Ravi
working-directory: ${{runner.workspace}}/build
Expand Down Expand Up @@ -299,7 +317,7 @@ jobs:

#- name: Convert to Coverage TXT
# working-directory: ${{runner.workspace}}/build
# run: python3 ${{runner.workspace}}/cobertura.py coverage/cobertura.info coverage/coverage.txt
# run: python3 ${{runner.workspace}}/Tools/cobertura.py coverage/cobertura.info coverage/coverage.txt

- name: Upload Cobertura XML
uses: actions/upload-artifact@v4
Expand Down
2 changes: 1 addition & 1 deletion CMakeLists.txt
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -19,7 +19,7 @@ set (LUABRIDGE_SANITIZE "" CACHE STRING "Sanitizer to enable (address, undefined

add_subdirectory (Source)

if (LUABRIDGE_TESTING)
if (PROJECT_IS_TOP_LEVEL AND LUABRIDGE_TESTING)
include (CTest)
enable_testing()
set (gtest_force_shared_crt ON CACHE BOOL "Use /MD and /MDd" FORCE)
Expand Down
Loading
Loading