Warn if the record type is ERROR

[Randgalt]

This can happen with circular/nested references. It appears to be a bug in the JDK compiler. You can work around it by using the FQPN so this warning instructs to do that.

Relates to #269

[Randgalt]

attn @brandondahler

[brandondahler]

There's a few additional places that we should probably emit this warning at.

Specifically:

• InternalDeconstructorProcessor::buildRecordComponents where we are resolving executableElement.getReturnType()'s class name -

  record-builder/record-builder-processor/src/main/java/io/soabase/recordbuilder/processor/InternalDeconstructorProcessor.java

  Line 230 in 543453a

  TypeName typeName = TypeName.get(executableElement.getReturnType());

• InternalDeconstructorProcessor::validateParameter where we are resolving the typeParameter's class name -

  record-builder/record-builder-processor/src/main/java/io/soabase/recordbuilder/processor/InternalDeconstructorProcessor.java

  Line 332 in 543453a

  return new ValidatedParameter(TypeName.get(typeParameter),

• InternalRecordInterfaceProcessor::getRecordComponents where we are validating element.getReturnType() to be used in the record parameter definition and alternative method name return type (those two usages use ClassName.get on it) -

  record-builder/record-builder-processor/src/main/java/io/soabase/recordbuilder/processor/InternalRecordInterfaceProcessor.java

  Line 179 in 543453a

  if (!element.getParameters().isEmpty() || element.getReturnType().getKind() == TypeKind.VOID) {

[brandondahler]

Tests for those looks straightforward:

ChildDeconstructorMethod.java

package io.soabase.recordbuilder.test.circular.nested;

import io.soabase.recordbuilder.core.RecordBuilder;
import java.util.function.Consumer;

public class ChildDeconstructorMethod {
    private final io.soabase.recordbuilder.test.circular.ParentBuilder parentBuilder;

    public ChildDeconstructorMethod(io.soabase.recordbuilder.test.circular.ParentBuilder parentBuilder) {
        this.parentBuilder = parentBuilder;
    }

    @RecordBuilder.Deconstructor
    public void deconstructor(Consumer<io.soabase.recordbuilder.test.circular.ParentBuilder> parentBuilder) {
        parentBuilder.accept(this.parentBuilder);
    }
}

ChildDeconstructorRecord.java

package io.soabase.recordbuilder.test.circular.nested;

import io.soabase.recordbuilder.core.RecordBuilder.Deconstructor;
import io.soabase.recordbuilder.core.RecordBuilder.DeconstructorAccessor;

@Deconstructor
public record ChildDeconstructorRecord(@DeconstructorAccessor io.soabase.recordbuilder.test.circular.ParentBuilder parentBuilder) {

}

ChildInterface.java

package io.soabase.recordbuilder.test.circular.nested;

import io.soabase.recordbuilder.core.RecordBuilder;

@RecordInterface
public interface ChildInterface {
    io.soabase.recordbuilder.test.circular.ParentBuilder parentBuilder();
}

For posterity/licensing - the code snippets above are good to be released under the Apache 2.0 license -- I omitted the license header for conciseness.