feat: Cross-language patch support, real test suites, and a Windows-CI overhaul

.gitattributes

@@ -1,16 +1,16 @@
-# Treat all files in this repo as binary, with no git magic updating
-# line endings. This produces predictable results in different environments.
-#
-# Windows users contributing to this repo will need to use a modern version
-# of git and editors capable of LF line endings.
-#
-# Windows .bat files are known to have multiple bugs when run with LF
-# endings, and so they are checked in with CRLF endings, with a test
-# to catch problems. (See https://github.com/golang/go/issues/37791.)
-#
-# You can prevent accidental CRLF line endings from entering the repo
-# via PR/MR checks.
-#
-# See https://github.com/golang/go/issues/9281.
-# See https://adaptivepatchwork.com/2012/03/01/mind-the-end-of-your-line.
+# Treat all files in this repo as binary, with no git magic updating
+# line endings. This produces predictable results in different environments.
+#
+# Windows users contributing to this repo will need to use a modern version
+# of git and editors capable of LF line endings.
+#
+# Windows .bat files are known to have multiple bugs when run with LF
+# endings, and so they are checked in with CRLF endings, with a test
+# to catch problems. (See https://github.com/golang/go/issues/37791.)
+#
+# You can prevent accidental CRLF line endings from entering the repo
+# via PR/MR checks.
+#
+# See https://github.com/golang/go/issues/9281.
+# See https://adaptivepatchwork.com/2012/03/01/mind-the-end-of-your-line.
 * -text

.github/workflows/release-cpp.yml

@@ -2,7 +2,7 @@ name: Release CPP
 
 on:
   release:
-    types: [ published ]
+    types: [published]
   workflow_dispatch:
 
 jobs:
@@ -13,8 +13,8 @@ jobs:
       'cmd/protoc-gen-cpp-tableau-loader/')
     strategy:
       matrix:
-        goos: [ linux, darwin, windows ]
-        goarch: [ amd64, arm64 ]
+        goos: [linux, darwin, windows]
+        goarch: [amd64, arm64]
         exclude:
           - goos: linux
             goarch: arm64
@@ -28,7 +28,8 @@ jobs:
       - name: Install Go
         uses: actions/setup-go@v5
         with:
-          go-version: "1.24.x"
+          go-version-file: go.mod
+          cache-dependency-path: go.sum
 
       - name: Download dependencies
         run: |

.github/workflows/release-csharp.yml

@@ -2,7 +2,7 @@ name: Release C#
 
 on:
   release:
-    types: [ published ]
+    types: [published]
   workflow_dispatch:
 
 jobs:
@@ -13,8 +13,8 @@ jobs:
       'cmd/protoc-gen-csharp-tableau-loader/')
     strategy:
       matrix:
-        goos: [ linux, darwin, windows ]
-        goarch: [ amd64, arm64 ]
+        goos: [linux, darwin, windows]
+        goarch: [amd64, arm64]
         exclude:
           - goos: linux
             goarch: arm64
@@ -28,7 +28,8 @@ jobs:
       - name: Install Go
         uses: actions/setup-go@v5
         with:
-          go-version: "1.24.x"
+          go-version-file: go.mod
+          cache-dependency-path: go.sum
 
       - name: Download dependencies
         run: |

.github/workflows/release-go.yml

@@ -2,7 +2,7 @@ name: Release Go
 
 on:
   release:
-    types: [ published ]
+    types: [published]
   workflow_dispatch:
 jobs:
   release:
@@ -12,8 +12,8 @@ jobs:
       'cmd/protoc-gen-go-tableau-loader/')
     strategy:
       matrix:
-        goos: [ linux, darwin, windows ]
-        goarch: [ amd64, arm64 ]
+        goos: [linux, darwin, windows]
+        goarch: [amd64, arm64]
         exclude:
           - goos: linux
             goarch: arm64
@@ -27,7 +27,8 @@ jobs:
       - name: Install Go
         uses: actions/setup-go@v5
         with:
-          go-version: "1.24.x"
+          go-version-file: go.mod
+          cache-dependency-path: go.sum
 
       - name: Download dependencies
         run: |

.github/workflows/testing-cpp.yml

@@ -3,7 +3,7 @@ name: Testing C++
 # Trigger on pushes, PRs (excluding documentation changes), and nightly.
 on:
   push:
-    branches: [ master, main ]
+    branches: [master, main]
   pull_request:
   schedule:
     - cron: 0 0 * * * # daily at 00:00
@@ -16,16 +16,13 @@ jobs:
   test:
     strategy:
       matrix:
-        os: [ ubuntu-latest, windows-latest ]
-        go-version: [ 1.24.x ]
-        protobuf-version: [ "32.0", "3.19.3" ]
+        os: [ubuntu-latest, windows-latest]
+        protobuf-version: ["32.0", "3.19.3"]
         include:
           - os: ubuntu-latest
             init_script: bash init.sh
-            run_loader: ./bin/loader
           - os: windows-latest
             init_script: cmd /c init.bat
-            run_loader: .\bin\loader.exe
 
     name: test (${{ matrix.os }}, protobuf ${{ matrix.protobuf-version }})
     runs-on: ${{ matrix.os }}
@@ -35,26 +32,33 @@ jobs:
       - name: Checkout Code
         uses: actions/checkout@v6
         with:
-          submodules: recursive
+          submodules: true
 
       - name: Install Go
         uses: actions/setup-go@v6
         with:
-          go-version: ${{ matrix.go-version }}
+          go-version-file: go.mod
+          cache-dependency-path: go.sum
           cache: true
 
       - name: Install dependencies (Ubuntu)
         if: runner.os == 'Linux'
-        run: sudo apt-get update && sudo apt-get install -y cmake ninja-build
-
-      - name: Install dependencies (Windows)
-        if: runner.os == 'Windows'
-        run: choco install cmake ninja -y
+        run: sudo apt-get update && sudo apt-get install -y ninja-build
 
       - name: Setup MSVC (Windows)
         if: runner.os == 'Windows'
         uses: ilammy/msvc-dev-cmd@v1
 
+      - name: Cache protobuf install
+        id: cache-protobuf
+        uses: actions/cache@v4
+        with:
+          path: |
+            third_party/_submodules/protobuf/.build/_install
+          key: protobuf-${{ matrix.os }}-${{ matrix.protobuf-version }}-${{ hashFiles('init.sh', 'init.bat', '.gitmodules') }}
+          restore-keys: |
+            protobuf-${{ matrix.os }}-${{ matrix.protobuf-version }}-
+
       - name: Init submodules and build protobuf
         run: ${{ matrix.init_script }}
         env:
@@ -87,13 +91,12 @@ jobs:
 
       - name: CMake Configure
         working-directory: test/cpp-tableau-loader
-        run: cmake -S . -B build -G Ninja -DCMAKE_BUILD_TYPE=Debug
-          -DCMAKE_CXX_STANDARD=17
+        run: cmake -S . -B build -G Ninja -DCMAKE_BUILD_TYPE=Debug -DCMAKE_CXX_STANDARD=17
 
       - name: CMake Build
         working-directory: test/cpp-tableau-loader
         run: cmake --build build --parallel
 
-      - name: Run loader
+      - name: Run tests
         working-directory: test/cpp-tableau-loader
-        run: ${{ matrix.run_loader }}
+        run: ctest --test-dir build --output-on-failure

.github/workflows/testing-csharp.yml

@@ -3,7 +3,7 @@ name: Testing C#
 # Trigger on pushes, PRs (excluding documentation changes), and nightly.
 on:
   push:
-    branches: [ master, main ]
+    branches: [master, main]
   pull_request:
   schedule:
     - cron: 0 0 * * * # daily at 00:00
@@ -16,9 +16,8 @@ jobs:
   test:
     strategy:
       matrix:
-        os: [ ubuntu-latest, windows-latest ]
-        go-version: [ 1.24.x ]
-        protobuf-version: [ "32.0", "3.19.3" ]
+        os: [ubuntu-latest, windows-latest]
+        protobuf-version: ["32.0", "3.19.3"]
 
     name: test (${{ matrix.os }}, protobuf ${{ matrix.protobuf-version }})
     runs-on: ${{ matrix.os }}
@@ -33,7 +32,8 @@ jobs:
       - name: Install Go
         uses: actions/setup-go@v6
         with:
-          go-version: ${{ matrix.go-version }}
+          go-version-file: go.mod
+          cache-dependency-path: go.sum
           cache: true
 
       - name: Install .NET SDK
@@ -66,10 +66,6 @@ jobs:
         working-directory: test/csharp-tableau-loader
         run: buf generate ..
 
-      - name: Build
+      - name: Test
         working-directory: test/csharp-tableau-loader
-        run: dotnet build
-
-      - name: Run loader
-        working-directory: test/csharp-tableau-loader
-        run: dotnet run
+        run: dotnet test --nologo --logger "console;verbosity=normal"

.github/workflows/testing-go.yml

@@ -3,7 +3,7 @@ name: Testing Go
 # Trigger on pushes, PRs (excluding documentation changes), and nightly.
 on:
   push:
-    branches: [ master, main ]
+    branches: [master, main]
   pull_request:
   schedule:
     - cron: 0 0 * * * # daily at 00:00
@@ -16,9 +16,8 @@ jobs:
   test:
     strategy:
       matrix:
-        os: [ ubuntu-latest, windows-latest ]
-        go-version: [ 1.24.x ]
-        protobuf-version: [ "32.0", "3.19.3" ]
+        os: [ubuntu-latest, windows-latest]
+        protobuf-version: ["32.0", "3.19.3"]
 
     name: test (${{ matrix.os }}, protobuf ${{ matrix.protobuf-version }})
     runs-on: ${{ matrix.os }}
@@ -33,7 +32,8 @@ jobs:
       - name: Install Go
         uses: actions/setup-go@v6
         with:
-          go-version: ${{ matrix.go-version }}
+          go-version-file: go.mod
+          cache-dependency-path: go.sum
           cache: true
 
       - name: Install Buf
@@ -54,10 +54,6 @@ jobs:
         run: go test -v -timeout 30m -race ./... -coverprofile=coverage.txt
           -covermode=atomic
 
-      - name: Run loader
-        working-directory: test/go-tableau-loader
-        run: go run .
-
       - name: Upload coverage reports to Codecov
         uses: codecov/codecov-action@v5
         with:

.gitignore

@@ -47,3 +47,6 @@ coverage.txt
 !test/testdata/bin/
 test/csharp-tableau-loader/protoconf
 
+# C# Dev Kit language service cache (VS Code)
+*.lscache
+