[jnigen] Config API update

[liamappelbe]

Update JNIgen's config API to align it with the proposed FFIgen API.

• Group classes, sourcePath, classPath, summarizerOptions, mavenDownloads, and androidSdkConfig into Input
• Group import and hide into SymbolImports, which lives in Input.imports
• OutputConfig renamed to Output, and still contains dartConfig -> dart and symbolsConfig -> symbols. Now also contains preamble, generateStubs, and format.
• nonNullAnnotations and nullableAnnotations now grouped under NullabilityAnnotations.
• Flip isExcluded to isIncluded.
• Make a bunch of nullable fields non-null, if null is functionally identical to a default value.
• For the fields where null is functionally different, document that. Also, document any other undocumented fields.

Sorry for the crazy number of changed files. It's mostly due to the version bump. There aren't any real bindings changes. Important files are:

• pkgs/jnigen/lib/src/config/config_types.dart, containing the new API
• Any of the generation scripts, eg pkgs/jni/tool/generate_jni_bindings.dart, if you want to see the new API in action

#2062

[github-actions]

Package publishing

If you have publishing permissions, you can use the links below to publish the changes after merging this PR.

Package	Version	Status	Publish tag (post-merge)
package:code_assets	1.3.0-wip	WIP (no publish necessary)
package:data_assets	0.20.0	already published at pub.dev
package:ffi	2.2.0	already published at pub.dev
package:hooks	2.2.0-wip	WIP (no publish necessary)
package:hooks_runner	1.6.2-wip	WIP (no publish necessary)
package:jni_flutter	1.0.2	already published at pub.dev
package:jni_util	1.0.0	already published at pub.dev
package:native_toolchain_c	0.19.4-wip	WIP (no publish necessary)
package:record_use	1.1.0	ready to publish	record_use-v1.1.0
package:swift2objc	0.3.0-wip	WIP (no publish necessary)
package:swiftgen	0.1.3	already published at pub.dev

Documentation at https://github.com/dart-lang/ecosystem/wiki/Publishing-automation.

[github-actions]

PR Health

API leaks ✔️

The following packages contain symbols visible in the public API, but not exported by the library. Export these symbols or remove them from your publicly visible API.

Package	Leaked API symbol	Leaking sources

This check can be disabled by tagging the PR with skip-leaking-check.

Breaking changes ✔️

Package	Change	Current Version	New Version	Needed Version	Looking good?
jni	None	1.0.3	1.0.3	1.0.3	✔️
jni_flutter	None	1.0.2	1.0.2	1.0.2	✔️

This check can be disabled by tagging the PR with skip-breaking-check.

[dcharkes]

Helicopter view level question: We can't put this as include on visitClass right? Because this goes into the summarizer which is a different stage in the pipeline.

I think the readme of both FFIgen and JNIgen would benefit from mermaid diagrams showing the pipeline and what in the config is configuring which part of the pipeline. (Feel free to file an issue to do this later. Mermaid diagrams are rendered on GitHub, I don't know about pub, and probably not in dartdoc either.)

Another super high level question, should options only passed to the summarizer be nested in the summarizer options? That might make it more clear for users.

[dcharkes]

We can't use dot shorthands to skip typing the constructor name can we?

[dcharkes]

We dropped Config suffix everywhere, this still has a config suffix.

[goderbauer]

I noticed some others still have the config suffix as well, e.g. AndroidSdkConfig.