Fix JVM crash when accessing array-backed RecordId keys

[emmanuel-keller]

Summary

Fixes #140.

This fixes a JVM crash when accessing array- or object-backed RecordId keys through RecordId.getId(). The Java native declarations for Id.getArray(...) and Id.getObject(...) expected Java objects, while the JNI implementation returns raw native pointers (jlong). The JVM could then interpret a native pointer as an object reference, causing a SIGSEGV when calling methods like len() or get(0).

Changes

• Change Id native array/object accessors to return long native pointers.
• Wrap those pointers as Array / Object on the Java side, matching the existing Value accessor pattern.
• Add regression coverage for array-backed RecordId keys from query results.
• Add direct coverage for array- and object-backed RecordId key accessors.

Testing

./gradlew test --tests com.surrealdb.ValueTypesTests.recordIdArrayKeyAccessorsFromQuery --tests com.surrealdb.TypeTests.testRecordIdWithArrayKey --tests com.surrealdb.TypeTests.testRecordIdWithObjectKey
git diff --check

[claude]

Claude Code Review

This repository is configured for manual code reviews. Comment @claude review to trigger a review and subscribe this PR to future pushes, or @claude review once for a one-time review.

_{Tip: disable this comment in your organization's Code Review settings.}