error running mage test:unit in main branch

[dangogh]

First time contributor trying to run unit tests and I get the following error. Could there be something incorrect in my setup?

mage test:unit
# github.com/aquasecurity/trivy/pkg/module/wasm
../../wasm/sdk.go:146:10: cannot use uintptr(size) (value of type uintptr) as int value in assignment
../../wasm/sdk.go:147:10: cannot use uintptr(size) (value of type uintptr) as int value in assignment
../../wasm/sdk.go:163:9: cannot use uintptr(size) (value of type uintptr) as int value in struct literal
../../wasm/sdk.go:164:9: cannot use uintptr(size) (value of type uintptr) as int value in struct literal
pkg/module/testdata/analyzer/analyzer.go:1: running "tinygo": exit status 1
Error: running "go generate pkg/module/testdata/analyzer/analyzer.go" failed with exit code 1

Running on a macbook pro M3 with Sonoma 14.6.1 and go1.22.4 darwin/arm64

[dangogh]

other details:
Above is on main branch with no changes.
tinygo version 0.33.0 darwin/arm64 (using go version go1.22.4 and LLVM version 18.1.2)

[knqyf263]

TinyGo might have added any breaking change. Did you try v0.31.1?

trivy/aqua.yaml

Line 8 in 1f9fc13

- name: tinygo-org/tinygo@v0.31.1

[dangogh]

You are correct. I tried v0.31.2 and v0.32.0. The breaking change seems to be in v0.32.0.

I think the uintptr cast here should be removed -- it's unclear why it was there to start with:

trivy/pkg/module/wasm/sdk.go

Lines 146 to 147 in 37d549e

	s.Len = uintptr(size)
	s.Cap = uintptr(size)

[dangogh]

ok -- I'll start a discussion over there

[dangogh]

@knqyf263
Actually, it appears to be an intentional known breaking change to make the reflect.*Header types compatible with the main Go implementation based on these discussions:

tinygo-org/tinygo#4318
kelindar/binary#18

[knqyf263]

OK, we want to fix our code, then.

[michaeljsaenz]

is there a temp workaround to get mage test:unit passing?

[knqyf263]

v0.31.2 should work.
#7568 (comment)