Add cross-platform CI matrix and Windows loader support

Author: tazarov

.github/workflows/ci.yml

@@ -0,0 +1,70 @@
+name: CI
+
+on:
+  push:
+    branches:
+      - main
+  pull_request:
+
+permissions:
+  contents: read
+
+jobs:
+  build-test-lint:
+    name: Build/Test/Lint (${{ matrix.os }})
+    runs-on: ${{ matrix.os }}
+    strategy:
+      fail-fast: false
+      matrix:
+        os:
+          - ubuntu-latest
+          - macos-latest
+          - windows-latest
+
+    steps:
+      - name: Checkout
+        uses: actions/checkout@v4
+
+      - name: Set up Go
+        uses: actions/setup-go@v5
+        with:
+          go-version-file: go.mod
+          cache: true
+
+      - name: Set up Rust
+        uses: dtolnay/rust-toolchain@stable
+        with:
+          components: clippy
+
+      - name: Cache Rust dependencies
+        uses: Swatinem/rust-cache@v2
+        with:
+          workspaces: "shim -> target"
+
+      - name: Build Rust shim (debug)
+        working-directory: shim
+        run: cargo build --locked
+
+      - name: Run Go tests
+        shell: pwsh
+        run: |
+          $libName = switch ("${{ runner.os }}") {
+            "Linux" { "libchroma_go_shim.so" }
+            "macOS" { "libchroma_go_shim.dylib" }
+            "Windows" { "chroma_go_shim.dll" }
+            default { throw "Unsupported OS: ${{ runner.os }}" }
+          }
+
+          $env:CHROMA_LIB_PATH = [System.IO.Path]::GetFullPath((Join-Path $env:GITHUB_WORKSPACE "shim/target/debug/$libName"))
+          Write-Host "CHROMA_LIB_PATH=$env:CHROMA_LIB_PATH"
+          go test -v ./...
+
+      - name: Run Go lint
+        uses: golangci/golangci-lint-action@v8
+        with:
+          version: latest
+          args: --timeout=30m ./...
+
+      - name: Run Rust lint
+        working-directory: shim
+        run: cargo clippy --locked -- -D warnings

README.md

@@ -185,6 +185,15 @@ make test-all      # Run both Go and Rust tests
 make test-release  # Run Go tests with release build
 ```
 
+## CI
+
+GitHub Actions runs a cross-platform matrix (`ubuntu-latest`, `macos-latest`, `windows-latest`) on pushes to `main` and pull requests. Each matrix job runs:
+
+1. `cargo build --locked` in `shim/`
+2. `go test -v ./...` with platform-specific `CHROMA_LIB_PATH`
+3. `golangci-lint run ./...`
+4. `cargo clippy --locked -- -D warnings` in `shim/`
+
 ### Benchmarks
 
 ```bash

go.mod

@@ -7,4 +7,7 @@ require (
 	github.com/pkg/errors v0.9.1
 )
 
-require github.com/leanovate/gopter v0.2.11
+require (
+	github.com/leanovate/gopter v0.2.11
+	golang.org/x/sys v0.6.0
+)

go.sum

@@ -406,6 +406,7 @@ golang.org/x/sys v0.0.0-20210630005230-0f9fa26af87c/go.mod h1:oPkhp1MJrh7nUepCBc
 golang.org/x/sys v0.0.0-20220520151302-bc2c85ada10a/go.mod h1:oPkhp1MJrh7nUepCBck5+mAzfO9JrbApNNgaTdGDITg=
 golang.org/x/sys v0.0.0-20220722155257-8c9f86f7a55f/go.mod h1:oPkhp1MJrh7nUepCBck5+mAzfO9JrbApNNgaTdGDITg=
 golang.org/x/sys v0.5.0/go.mod h1:oPkhp1MJrh7nUepCBck5+mAzfO9JrbApNNgaTdGDITg=
+golang.org/x/sys v0.6.0 h1:MVltZSvRTcU2ljQOhs94SXPftV6DCNnZViHeQps87pQ=
 golang.org/x/sys v0.6.0/go.mod h1:oPkhp1MJrh7nUepCBck5+mAzfO9JrbApNNgaTdGDITg=
 golang.org/x/term v0.0.0-20201126162022-7de9c90e9dd1/go.mod h1:bj7SfCRtBDWHUb9snDiAeCFNEtKQo2Wmx5Cou7ajbmo=
 golang.org/x/term v0.0.0-20210927222741-03fcf44c2211/go.mod h1:jbD1KX2456YbFQfuXm/mYQcufACuNUgVhRMnK/tPxf8=

library.go

@@ -3,23 +3,17 @@ package chroma
 import (
 	"os"
 
-	"github.com/ebitengine/purego"
 	"github.com/pkg/errors"
 )
 
 const envLibPath = "CHROMA_LIB_PATH"
 
-func loadLibrary(path string) (uintptr, error) {
+func resolveLibraryPath(path string) (string, error) {
 	if path == "" {
 		path = os.Getenv(envLibPath)
 	}
 	if path == "" {
-		return 0, errors.New("library path not specified and CHROMA_LIB_PATH not set")
+		return "", errors.New("library path not specified and CHROMA_LIB_PATH not set")
 	}
-
-	libHandle, err := purego.Dlopen(path, purego.RTLD_NOW|purego.RTLD_GLOBAL)
-	if err != nil || libHandle == 0 {
-		return 0, errors.Wrapf(err, "failed to load library: %s", path)
-	}
-	return libHandle, nil
+	return path, nil
 }

library_unix.go

@@ -0,0 +1,21 @@
+//go:build !windows
+
+package chroma
+
+import (
+	"github.com/ebitengine/purego"
+	"github.com/pkg/errors"
+)
+
+func loadLibrary(path string) (uintptr, error) {
+	resolvedPath, err := resolveLibraryPath(path)
+	if err != nil {
+		return 0, err
+	}
+
+	libHandle, err := purego.Dlopen(resolvedPath, purego.RTLD_NOW|purego.RTLD_GLOBAL)
+	if err != nil || libHandle == 0 {
+		return 0, errors.Wrapf(err, "failed to load library: %s", resolvedPath)
+	}
+	return libHandle, nil
+}

library_windows.go

@@ -0,0 +1,26 @@
+//go:build windows
+
+package chroma
+
+import (
+	"golang.org/x/sys/windows"
+
+	"github.com/pkg/errors"
+)
+
+func loadLibrary(path string) (uintptr, error) {
+	resolvedPath, err := resolveLibraryPath(path)
+	if err != nil {
+		return 0, err
+	}
+
+	handle, err := windows.LoadLibrary(resolvedPath)
+	if err != nil {
+		return 0, errors.Wrapf(err, "failed to load library: %s", resolvedPath)
+	}
+	if handle == 0 {
+		return 0, errors.Errorf("failed to load library: %s", resolvedPath)
+	}
+
+	return uintptr(handle), nil
+}