@ReflectionHint types has problem with inner static class

[stefan-langenmaier]

Describe the bug

The @ReflectionHint(types = { Result.class }) does not seem to work for inner static classes.

To Reproduce
Steps to reproduce the behavior:

1. Create a file with the following content

///usr/bin/env jbang "$0" "$@" ; exit $?
//JAVA 21+
//DEPS io.goodforgod:graalvm-hint-processor:1.1.0
//DEPS io.goodforgod:graalvm-hint-annotations:1.1.0
//DEPS com.fasterxml.jackson.dataformat:jackson-dataformat-yaml:2.16.0


import static java.lang.System.*;

import com.fasterxml.jackson.core.JsonProcessingException;
import com.fasterxml.jackson.databind.JsonMappingException;
import com.fasterxml.jackson.databind.ObjectMapper;
import com.fasterxml.jackson.dataformat.yaml.YAMLFactory;

import io.goodforgod.graalvm.hint.annotation.ReflectionHint;

public class TestYaml {

    public static void main(String... args) throws JsonMappingException, JsonProcessingException {
        out.println("Hello World");

        String document = "hello: 25";

        ObjectMapper objectMapper = new ObjectMapper(new YAMLFactory());
        Result r = objectMapper.readValue(document, Result.class);

        out.println(r.hello);
    }

    public static class Result {
        public Integer hello;
    }
    
    //@ReflectionHint(typeNames = {"TestYaml$Result"})
    @ReflectionHint(types = { Result.class })
    public class ReflectionConfig {}

}

2. Execute it with jbang it should display

Hello World
25

3. when exporting the script jbang export native --force --verbose TestSnakeYaml.java the execution of the binary yields the following error

Hello World
Exception in thread "main" com.fasterxml.jackson.databind.exc.InvalidDefinitionException: Cannot construct instance of `TestYaml$Result`: cannot deserialize from Object value (no delegate- or property-based Creator): this appears to be a native image, in which case you may need to configure reflection for the class that is to be deserialized
 at [Source: (StringReader); line: 1, column: 1]
        at com.fasterxml.jackson.databind.DeserializationContext.reportBadDefinition(DeserializationContext.java:1887)

4. If the comment is switched to the typeNames it works.

Expected behavior
Should work in both cases.

[GoodforGod]

Hello @stefan-langenmaier , I'll look into it, thanks!

[GoodforGod]

What GraalVM version are you using for this example?

[stefan-langenmaier]

I'm using the community version for JDK 21: https://github.com/graalvm/graalvm-ce-builds/releases/tag/jdk-21.0.1

[stefan-langenmaier]

typeNames =

[{
  "name": "Config",
  "allDeclaredConstructors": true,
  "allDeclaredFields": true,
  "allDeclaredMethods": true
},
{
  "name": "Config$Backup",
  "allDeclaredConstructors": true,
  "allDeclaredFields": true,
  "allDeclaredMethods": true
},
{
  "name": "Config$Snapshots",
  "allDeclaredConstructors": true,
  "allDeclaredFields": true,
  "allDeclaredMethods": true
},
{
  "name": "Config$Volume",
  "allDeclaredConstructors": true,
  "allDeclaredFields": true,
  "allDeclaredMethods": true
},
{
  "name": "Config$Volume$Keep",
  "allDeclaredConstructors": true,
  "allDeclaredFields": true,
  "allDeclaredMethods": true
}]

types =

[{
  "name": "Config",
  "allDeclaredConstructors": true,
  "allDeclaredFields": true,
  "allDeclaredMethods": true
},
{
  "name": "Config.Backup",
  "allDeclaredConstructors": true,
  "allDeclaredFields": true,
  "allDeclaredMethods": true
},
{
  "name": "Config.Keep",
  "allDeclaredConstructors": true,
  "allDeclaredFields": true,
  "allDeclaredMethods": true
},
{
  "name": "Config.Snapshots",
  "allDeclaredConstructors": true,
  "allDeclaredFields": true,
  "allDeclaredMethods": true
},
{
  "name": "Config.Volume",
  "allDeclaredConstructors": true,
  "allDeclaredFields": true,
  "allDeclaredMethods": true
}]