[BUG] defaultToEmptyContainer not working with Maven plugin

[jpfinne]

Bug Report Checklist

• Have you provided a full/minimal spec to reproduce the issue?
• Have you validated the input using an OpenAPI validator?
• Have you tested with the latest master to confirm the issue still exists?
• Have you searched for related issues/PRs?
• What's the actual output vs expected output?
• [Optional] Sponsorship to speed up the bug fix or feature request (example)

Description

#21269 introduces a new global additionalProperty: defaultToEmptyContainer

Unfortunately it only works when using additionalProperties.
It does not work with configOptions in maven. The plugin requires cliOptions. See

openapi-generator/modules/openapi-generator-maven-plugin/src/main/java/org/openapitools/codegen/plugin/CodeGenMojo.java

Line 976 in e948355

for (CliOption langCliOption : config.cliOptions()) {

openapi-generator version

7.14.0

Steps to reproduce

Configure maven

<configOptions>
        <defaultToEmptyContainer>array|undefined</defaultToEmptyContainer>
</configOptions>

Related issues/PRs

#21269

Suggest a fix

Add cliOptions to the supported generators (maybe all by default?)

cliOptions.add(CliOption.newString(DEFAULT_TO_EMPTY_CONTAINER, "description..."));

Update CodeGenMojo to put all configOptions to additionalProperties:

    if (configOptions != null) {
        for (Map.Entry<?, ?> configOptionsEntry : configOptions.entrySet()) {
            input.getConfig().additionalProperties().put(configOptionsEntry.getKey().toString(), configOptionsEntry.getValue());
        }
    }

[wing328]

Unfortunately it only works when using additionalProperties.

so are you saying maven plugin won't work with additionalProperties at all as one use case is that users provide custom input via additionalProperties which will be consumed in customized templates?

[wing328]

update: filed #22869 to add the cli option to all java generators.

not sure why it's not working via additional properties in maven plugin but adding the cli option should be good for now