chore(QTDI-2894): S6204 Stream.toList() by undx · Pull Request #1228 · Talend/component-runtime

undx · 2026-05-22T12:18:09Z

https://qlik-dev.atlassian.net/browse/QTDI-2894

Requirements

Any code change adding any logic MUST be tested through a unit test executed with the default build
Any API addition MUST be done with a documentation update if relevant

Why this PR is needed?

What does this PR adds (design/code thoughts)?

AI generated code

https://internal.qlik.dev/general/ways-of-working/code-reviews/#guidelines-for-ai-generated-code

this PR has been written with the help of GitHub Copilot or another generative AI tool

(cherry picked from commit c877724)

Copilot

Pull request overview

This PR updates Java stream terminal operations across the codebase, primarily replacing collect(Collectors.toList()) / collect(toList()) with Stream.toList() (per Sonar rule S6204), and updates a few generated documentation artifacts accordingly.

Changes:

Replace many Collectors.toList() usages with Stream.toList() and remove now-unused toList imports.
Regenerate/update documentation partials that embed generated content.
Apply the same migration in multiple test suites and tooling modules.

Reviewed changes

Copilot reviewed 152 out of 152 changed files in this pull request and generated 3 comments.

Show a summary per file

File	Description
vault-client/src/main/java/org/talend/sdk/components/vault/client/VaultClient.java	Replace `Collectors.toList()` with `Stream.toList()` in vault decryption flow.
talend-component-maven-plugin/src/main/java/org/talend/sdk/component/maven/WebsiteBuilderMojo.java	Replace `Collectors.toList()` with `Stream.toList()` and remove static import.
talend-component-maven-plugin/src/main/java/org/talend/sdk/component/maven/ScanDescriptorMojo.java	Replace `collect(toList())` with `toList()` and remove static import.
talend-component-maven-plugin/src/main/java/org/talend/sdk/component/maven/IconReporterMojo.java	Replace `collect(toList())` with `toList()` and remove static import.
talend-component-maven-plugin/src/main/java/org/talend/sdk/component/maven/DependencyConflictsReporterMojo.java	Replace `collect(toList())` with `toList()` in dependency conflict reporting.
talend-component-maven-plugin/src/main/java/org/talend/sdk/component/maven/ComponentMetadataMojo.java	Replace `collect(toList())` with `toList()` and remove static import.
talend-component-maven-plugin/src/main/java/org/talend/sdk/component/maven/ComponentDependenciesBase.java	Replace `collect(toList())` with `toList()` and remove static import.
talend-component-maven-plugin/src/main/java/org/talend/sdk/component/maven/ClasspathMojoBase.java	Replace `collect(toList())` with `toList()` and remove static import.
talend-component-maven-plugin/src/main/java/org/talend/sdk/component/maven/AsciidocMojo.java	Replace `collect(toList())` with `toList()` and remove static import.
singer-parent/component-kitap/src/test/java/org/talend/sdk/component/singer/kitap/KitapTest.java	Replace `collect(toList())` with `toList()` in tests and remove static import.
singer-parent/component-kitap/src/main/java/org/talend/sdk/component/singer/kitap/Kitap.java	Replace `collect(toList())` with `toList()` and remove static import.
sample-parent/sample/src/main/java/org/talend/sdk/component/sample/other/ComplexComponent.java	Replace `collect(toList())` with `toList()` and remove static import.
sample-parent/sample-features/checkpoint-runner/src/main/java/org/talend/sdk/component/sample/feature/checkpoint/IncrementalSequenceInput.java	Replace `collect(toList())` with `toList()` in sample feature input.
sample-parent/sample-connector/src/main/java/org/talend/sdk/component/test/connectors/service/ActionsServices.java	Replace `Collectors.toList()` with `toList()` and remove import.
remote-engine-customizer/src/main/java/org/talend/sdk/component/remoteengine/customizer/task/RemoteEngineCustomizer.java	Replace `collect(toList())` with `toList()` and remove static import.
remote-engine-customizer/src/main/java/org/talend/sdk/component/remoteengine/customizer/lang/IO.java	Replace `collect(toList())` with `toList()` and remove static import.
documentation/src/main/java/org/talend/runtime/documentation/SearchIndexation.java	Replace `collect(toList())` with `toList()` and remove static import.
documentation/src/main/java/org/talend/runtime/documentation/Github.java	Replace `collect(toList())` with `toList()` in contributor loading.
documentation/src/main/java/org/talend/runtime/documentation/Generator.java	Replace `collect(toList())` with `toList()` in Jira/changelog generation.
documentation/src/main/java/org/talend/runtime/documentation/component/source/MockTableMapper.java	Replace `collect(toList())` with `toList()` and remove static import.
documentation/src/main/java/org/talend/runtime/documentation/component/service/MockTableService.java	Replace `collect(toList())` with `toList()` and remove static import.
documentation/src/main/antora/modules/ROOT/pages/_partials/generated_rest-resources.adoc	Update generated embedded Swagger UI payload.
documentation/src/main/antora/modules/ROOT/pages/_partials/generated_contributors.adoc	Update generated contributor list/metadata.
container/nested-maven-repository/src/main/java/org/talend/sdk/component/container/maven/shade/ArtifactTransformer.java	Replace `collect(toList())` with `toList()` when building filters.
container/container-core/src/test/java/org/talend/sdk/component/dependencies/maven/MvnDependencyListLocalRepositoryResolverTest.java	Replace `collect(toList())` with `toList()` in tests and remove static import.
container/container-core/src/test/java/org/talend/sdk/component/ContainerTest.java	Replace `collect(toList())` with `toList()` in tests and remove static import.
container/container-core/src/test/java/org/talend/sdk/component/ContainerManagerTest.java	Replace `Collectors.toList()` with `toList()` and remove import.
container/container-core/src/test/java/org/talend/sdk/component/classloader/ConfigurableClassLoaderTest.java	Replace `collect(toList())` with `toList()` in tests and remove static import.
container/container-core/src/main/java/org/talend/sdk/component/container/ContainerManager.java	Replace `collect(toList())` with `toList()` in listener handling.
container/container-core/src/main/java/org/talend/sdk/component/container/Container.java	Replace `collect(toList())` with `toList()` for classpath file materialization.
container/container-core/src/main/java/org/talend/sdk/component/classloader/ConfigurableClassLoader.java	Replace `collect(toList())` with `toList()` in resource aggregation.
component-tools/src/test/java/org/talend/sdk/component/tools/validator/ExceptionValidatorTest.java	Replace `Collectors.toList()` with `toList()` and remove import.
component-tools/src/test/java/org/talend/sdk/component/tools/validator/ActionValidatorTest.java	Replace `collect(Collectors.toList())` with `toList()` in tests.
component-tools/src/test/java/org/talend/sdk/component/tools/SvgValidatorTest.java	Replace `collect(toList())` with `toList()` and remove static import.
component-tools/src/test/java/org/talend/sdk/component/tools/CarBundlerTest.java	Replace `collect(toList())` with `toList()` in tests and remove static import.
component-tools/src/main/java/org/talend/sdk/component/tools/validator/InternationalizationValidator.java	Replace multiple `collect(toList())` with `toList()` and remove static import.
component-tools/src/main/java/org/talend/sdk/component/tools/validator/HttpValidator.java	Replace `Collectors.toList()` with `toList()` and remove import.
component-tools/src/main/java/org/talend/sdk/component/tools/validator/FixedSchemaValidator.java	Replace `Collectors.toList()` with `toList()` and remove import.
component-tools/src/main/java/org/talend/sdk/component/tools/validator/DataStoreValidator.java	Replace `collect(toList())` with `toList()` and remove static import.
component-tools/src/main/java/org/talend/sdk/component/tools/validator/DatasetValidator.java	Replace `collect(toList())` with `toList()` and remove static import.
component-tools/src/main/java/org/talend/sdk/component/tools/StudioInstaller.java	Replace `collect(toList())` with `toList()` and remove static import.
component-tools/src/main/java/org/talend/sdk/component/tools/DocBaseGenerator.java	Replace `collect(toList())` with `toList()` and remove unused import.
component-tools/src/main/java/org/talend/sdk/component/tools/DitaDocumentationGenerator.java	Replace `collect(Collectors.toList())` with `toList()` and remove import.
component-tools/src/main/java/org/talend/sdk/component/tools/ComponentValidator.java	Replace `collect(toList())` with `toList()` and remove static import.
component-tools/src/main/java/org/talend/sdk/component/tools/AsciidoctorExecutor.java	Replace `collect(toList())` with `toList()` for file line collection.
component-tools-webapp/src/main/java/org/talend/sdk/component/tools/webapp/standalone/generator/StaticUiSpecGenerator.java	Replace `collect(toList())` with `toList()` and remove static import.
component-tools-webapp/src/main/java/org/talend/sdk/component/tools/webapp/standalone/generator/StaticResourceGenerator.java	Replace multiple `collect(toList())` with `toList()` in route generation.
component-studio/component-runtime-di/src/test/java/org/talend/sdk/component/runtime/di/studio/DynamicColumnsTest.java	Replace `collect(toList())` with `toList()` in tests.
component-studio/component-runtime-di/src/test/java/org/talend/sdk/component/runtime/di/record/DiRecordVisitorTest.java	Replace `Collectors.toList()` with `toList()` and remove import.
component-studio/component-runtime-di/src/test/java/org/talend/sdk/component/runtime/di/beam/components/DIBatchSimulationTest.java	Replace `collect(toList())` with `toList()` in Beam test setup.
component-studio/component-runtime-di/src/main/java/org/talend/sdk/component/runtime/di/schema/TaCoKitGuessSchema.java	Replace `collect(toList())` with `toList()` and remove static import.
component-studio/component-runtime-di/src/main/java/org/talend/sdk/component/runtime/di/beam/DelegatingUnBoundedSource.java	Replace `collect(toList())` with `toList()` and remove static import.
component-studio/component-runtime-di/src/main/java/org/talend/sdk/component/runtime/di/beam/DelegatingBoundedSource.java	Replace `collect(toList())` with `toList()` and remove static import.
component-starter-server/src/test/java/org/talend/sdk/component/starter/server/service/facet/component/ComponentGeneratorConfigurationTest.java	Replace `collect(toList())` with `toList()` in tests.
component-starter-server/src/test/java/org/talend/sdk/component/starter/server/front/apidemo/component/source/MockTableMapper.java	Replace `collect(toList())` with `toList()` and remove static import.
component-starter-server/src/test/java/org/talend/sdk/component/starter/server/front/apidemo/component/service/MockTableService.java	Replace `collect(toList())` with `toList()` in test service.
component-starter-server/src/main/java/org/talend/sdk/component/starter/server/service/openapi/OpenAPIGenerator.java	Replace nested `collect(toList())` with `toList()` in OpenAPI parsing.
component-starter-server/src/main/java/org/talend/sdk/component/starter/server/service/facet/component/ComponentGenerator.java	Replace `collect(toList())` with `toList()` when adding generated files.
component-starter-server/src/main/java/org/talend/sdk/component/starter/server/front/RRDEndpoint.java	Replace `collect(toList())` with `toList()` and remove static import.
component-server-parent/component-server/src/test/java/org/talend/sdk/component/server/tomcat/GenerateCertificateAndActivateHttpsTest.java	Replace `collect(toList())` with `toList()` and remove static import.
component-server-parent/component-server/src/test/java/org/talend/sdk/component/server/test/custom/CustomService.java	Replace `collect(toList())` with `toList()` and remove static import.
component-server-parent/component-server/src/test/java/org/talend/sdk/component/server/service/PropertiesServiceTest.java	Replace `collect(toList())` with `toList()` in tests.
component-server-parent/component-server/src/main/java/org/talend/sdk/component/server/service/VirtualDependenciesService.java	Replace `collect(toList())` with `toList()` and remove static import.
component-server-parent/component-server/src/main/java/org/talend/sdk/component/server/service/PropertyValidationService.java	Replace `collect(toList())` with `toList()` and remove static import.
component-server-parent/component-server/src/main/java/org/talend/sdk/component/server/service/ExtensionComponentMetadataManager.java	Replace `collect(toList())` with `toList()` and remove static import.
component-server-parent/component-server/src/main/java/org/talend/sdk/component/server/service/ComponentManagerService.java	Replace `collect(toList())` with `toList()` in post-deploy aggregation.
component-server-parent/component-server/src/main/java/org/talend/sdk/component/server/service/ActionsService.java	Replace `collect(toList())` with `toList()` and remove static import.
component-server-parent/component-server/src/main/java/org/talend/sdk/component/server/front/security/SecurityUtils.java	Replace `collect(toList())` with `toList()` and remove static import.
component-server-parent/component-server/src/main/java/org/talend/sdk/component/server/front/DocumentationResourceImpl.java	Replace `collect(toList())` with `toList()` when reading docs cache.
component-server-parent/component-server/src/main/java/org/talend/sdk/component/server/front/ConfigurationTypeResourceImpl.java	Replace `collect(toList())` with `toList()` and remove static import.
component-server-parent/component-server/src/main/java/org/talend/sdk/component/server/front/ComponentResourceImpl.java	Replace multiple `collect(toList())` with `toList()` in REST payload building.
component-server-parent/component-server/src/main/java/org/talend/sdk/component/server/front/BulkReadResourceImpl.java	Replace `collect(toList())` with `toList()` in bulk execution.
component-server-parent/component-server/src/main/java/org/talend/sdk/component/server/front/ActionResourceImpl.java	Replace `collect(toList())` with `toList()` in action listing and details.
component-runtime-testing/component-runtime-testing-spark/src/main/java/org/talend/sdk/component/runtime/testing/spark/internal/BaseSpark.java	Replace `collect(toList())` with `toList()` in Spark runner plumbing.
component-runtime-testing/component-runtime-junit/src/test/java/org/talend/sdk/component/junit5/ComponentExtensionTest.java	Replace `collect(toList())` with `toList()` in tests and remove static import.
component-runtime-testing/component-runtime-junit/src/test/java/org/talend/sdk/component/junit/SimpleComponentRuleTest.java	Replace `collect(toList())` with `toList()` in tests and remove static import.
component-runtime-testing/component-runtime-junit/src/main/java/org/talend/sdk/component/maven/MavenDecrypter.java	Replace `Collectors.toList()` with `toList()` and remove import.
component-runtime-testing/component-runtime-junit/src/main/java/org/talend/sdk/component/junit/environment/BaseEnvironmentProvider.java	Replace `collect(toList())` with `toList()` and remove static import.
component-runtime-testing/component-runtime-junit/src/main/java/org/talend/sdk/component/junit/BaseComponentsHandler.java	Replace `collect(toList())` with `toList()` and remove static import.
component-runtime-testing/component-runtime-beam-junit/src/test/java/org/talend/sdk/component/junit/beam/ProducerFinderEnvironmentTest.java	Replace `collect(toList())` with `toList()` in Beam tests.
component-runtime-testing/component-runtime-beam-junit/src/test/java/org/talend/sdk/component/junit/beam/ProcessorTest.java	Replace `collect(toList())` with `toList()` in assertions.
component-runtime-testing/component-runtime-beam-junit/src/main/java/org/talend/sdk/component/junit/beam/Data.java	Replace `collect(toList())` with `toList()` and remove static import.
component-runtime-manager/src/test/java/org/talend/test/CheckpointInput.java	Replace `collect(toList())` with `toList()` and remove static import.
component-runtime-manager/src/test/java/org/talend/sdk/component/runtime/manager/service/RecordServiceImplTest.java	Replace `Collectors.toList()` with `toList()` and remove import.
component-runtime-manager/src/test/java/org/talend/sdk/component/runtime/manager/ReflectionServiceTest.java	Replace `collect(toList())` with `toList()` in test expectations.
component-runtime-manager/src/test/java/org/talend/sdk/component/runtime/manager/interceptor/InterceptorTest.java	Replace `collect(toList())` with `toList()` in tests.
component-runtime-manager/src/test/java/org/talend/sdk/component/runtime/manager/ComponentManagerTest.java	Replace `collect(toList())` with `toList()` in tests.
component-runtime-manager/src/test/java/org/talend/sdk/component/runtime/manager/chain/JobTest.java	Replace `collect(toList())` with `toList()` in tests.
component-runtime-manager/src/main/java/org/talend/sdk/component/runtime/manager/service/ResolverImpl.java	Replace `collect(toList())` with `toList()` and remove static import.
component-runtime-manager/src/main/java/org/talend/sdk/component/runtime/manager/service/RecordPointerFactoryImpl.java	Replace `collect(toList())` with `toList()` and remove static import.
component-runtime-manager/src/main/java/org/talend/sdk/component/runtime/manager/service/LocalCacheService.java	Replace `Collectors.toList()` with `toList()` and remove import.
component-runtime-manager/src/main/java/org/talend/sdk/component/runtime/manager/service/http/RequestParser.java	Replace `collection.collect(toList())` with `collection.toList()` in CSV formatting.
component-runtime-manager/src/main/java/org/talend/sdk/component/runtime/manager/service/http/HttpClientFactoryImpl.java	Replace `collect(toList())` with `toList()` and remove static import.
component-runtime-manager/src/main/java/org/talend/sdk/component/runtime/manager/reflect/visibility/VisibilityService.java	Replace `collect(toList())` with `toList()` in JSON array mapping.
component-runtime-manager/src/main/java/org/talend/sdk/component/runtime/manager/reflect/ReflectionService.java	Replace `collect(toList())` with `toList()` in parameter factory mapping.
component-runtime-manager/src/main/java/org/talend/sdk/component/runtime/manager/reflect/ParameterModelService.java	Replace `collect(toList())` with `toList()` and remove static import.
component-runtime-manager/src/main/java/org/talend/sdk/component/runtime/manager/reflect/parameterenricher/UiParameterEnricher.java	Replace `collect(toList())` with `toList()` and remove static import.
component-runtime-manager/src/main/java/org/talend/sdk/component/runtime/manager/reflect/ComponentMetadataService.java	Replace `collect(toList())` with `toList()` and remove static import.
component-runtime-manager/src/main/java/org/talend/sdk/component/runtime/manager/interceptor/InterceptorHandlerFacade.java	Replace `collect(toList())` with `toList()` and remove static import.
component-runtime-manager/src/main/java/org/talend/sdk/component/runtime/manager/configuration/ConfigurationMapper.java	Replace `collect(toList())` with `toList()` and remove static import.
component-runtime-manager/src/main/java/org/talend/sdk/component/runtime/manager/ComponentManager.java	Replace multiple `collect(toList())` with `toList()` in service loading and setup.
component-runtime-manager/src/main/java/org/talend/sdk/component/runtime/manager/chain/internal/JobImpl.java	Replace `collect(toList())` with `toList()` for graph traversal lists.
component-runtime-manager/src/main/java/org/talend/sdk/component/runtime/manager/asm/ProxyGenerator.java	Replace `collect(toList())` with `toList()` and remove static import.
component-runtime-impl/src/test/java/org/talend/sdk/component/runtime/record/SchemaImplTest.java	Replace `Collectors.toList()` with `toList()` and remove import.
component-runtime-impl/src/test/java/org/talend/sdk/component/runtime/record/RecordConvertersTest.java	Replace `collect(toList())` with `toList()` in assertions.
component-runtime-impl/src/test/java/org/talend/sdk/component/runtime/record/RecordBuilderImplTest.java	Replace `Collectors.toList()` with `toList()` in tests.
component-runtime-impl/src/test/java/org/talend/sdk/component/runtime/record/OrderedMapTest.java	Replace `Collectors.toList()` with `toList()` in tests.
component-runtime-impl/src/test/java/org/talend/sdk/component/runtime/output/ProcessorImplTest.java	Replace `collect(toList())` with `toList()` in tests.
component-runtime-impl/src/test/java/org/talend/sdk/component/runtime/input/PartitionMapperImplTest.java	Replace `collect(toList())` with `toList()` in tests.
component-runtime-impl/src/main/java/org/talend/sdk/component/runtime/visitor/ModelVisitor.java	Replace `collect(toList())` with `toList()` and remove static import.
component-runtime-impl/src/main/java/org/talend/sdk/component/runtime/serialization/EnhancedObjectInputStream.java	Replace `collect(toList())` with `toList()` and remove static import.
component-runtime-impl/src/main/java/org/talend/sdk/component/runtime/record/SchemaImpl.java	Replace internal collection with `toList()` for entries ordering.
component-runtime-impl/src/main/java/org/talend/sdk/component/runtime/record/RecordImpl.java	Replace `Collectors.toList()` with `toList()` and remove import.
component-runtime-impl/src/main/java/org/talend/sdk/component/runtime/record/json/RecordJsonGenerator.java	Replace `collect(toList())` with `toList()` and remove static import.
component-runtime-impl/src/main/java/org/talend/sdk/component/runtime/output/ProcessorImpl.java	Replace `collect(toList())` with `toList()` in lifecycle method discovery.
component-runtime-design-extension/src/test/java/org/talend/sdk/component/design/extension/repository/RepositoryModelBuilderTest.java	Replace `collect(toList())` with `toList()` in tests.
component-runtime-design-extension/src/main/java/org/talend/sdk/component/design/extension/repository/RepositoryModelBuilder.java	Replace `collect(toList())` with `toList()` and remove static import.
component-runtime-design-extension/src/main/java/org/talend/sdk/component/design/extension/flows/ProcessorFlowsFactory.java	Replace `collect(toList())` with `toList()` and remove static import.
component-runtime-beam/src/test/java/org/talend/sdk/component/runtime/beam/transform/RecordBranchUnwrapperTest.java	Replace `collect(toList())` with `toList()` in tests.
component-runtime-beam/src/test/java/org/talend/sdk/component/runtime/beam/transform/RecordBranchMapperTest.java	Replace `collect(toList())` with `toList()` in tests.
component-runtime-beam/src/test/java/org/talend/sdk/component/runtime/beam/transform/RecordBranchFilterTest.java	Replace `collect(toList())` with `toList()` in tests.
component-runtime-beam/src/test/java/org/talend/sdk/component/runtime/beam/transform/avro/SchemalessJsonToIndexedRecordTest.java	Replace `collect(toList())` with `toList()` in tests.
component-runtime-beam/src/test/java/org/talend/sdk/component/runtime/beam/transform/avro/JsonToIndexedRecordTest.java	Replace `collect(toList())` with `toList()` in tests.
component-runtime-beam/src/test/java/org/talend/sdk/component/runtime/beam/transform/avro/IndexedRecordToJsonTest.java	Replace `collect(toList())` with `toList()` in tests.
component-runtime-beam/src/test/java/org/talend/sdk/component/runtime/beam/transform/AutoKVWrapperTest.java	Replace `collect(toList())` with `toList()` in tests.
component-runtime-beam/src/test/java/org/talend/sdk/component/runtime/beam/TalendIOTest.java	Replace `collect(toList())` with `toList()` in tests.
component-runtime-beam/src/test/java/org/talend/sdk/component/runtime/beam/spi/record/AvroSchemaCacheTest.java	Replace `Collectors.toList()` with `toList()` and remove import.
component-runtime-beam/src/test/java/org/talend/sdk/component/runtime/beam/spi/record/AvroRecordTest.java	Replace `Collectors.toList()` with `toList()` and remove import.
component-runtime-beam/src/test/java/org/talend/sdk/component/runtime/beam/NativeWrappedIOTest.java	Replace `collect(toList())` with `toList()` in tests.
component-runtime-beam/src/test/java/org/talend/sdk/component/runtime/beam/coder/JsonpJsonObjectCoderTest.java	Replace `collect(toList())` with `toList()` in tests.
component-runtime-beam/src/main/java/org/talend/sdk/component/runtime/beam/transform/avro/SchemalessJsonToIndexedRecord.java	Replace `collect(toList())` with `toList()` and remove static import.
component-runtime-beam/src/main/java/org/talend/sdk/component/runtime/beam/spi/record/SchemaIdGenerator.java	Replace `collect(toList())` with `toList()` and remove static import.
component-runtime-beam/src/main/java/org/talend/sdk/component/runtime/beam/spi/record/AvroSchemaConverter.java	Replace `collect(toList())` with `toList()` and remove static import.
component-runtime-beam/src/main/java/org/talend/sdk/component/runtime/beam/spi/record/AvroSchemaBuilder.java	Replace `Collectors.toList()` with `toList()` and remove import.
component-runtime-beam/src/main/java/org/talend/sdk/component/runtime/beam/spi/record/AvroSchema.java	Replace `collect(toList())` with `toList()` in schema entry extraction.
component-runtime-beam/src/main/java/org/talend/sdk/component/runtime/beam/spi/record/AvroRecord.java	Replace `collect(toList())` with `toList()` in mapping logic.
component-runtime-beam/src/main/java/org/talend/sdk/component/runtime/beam/customizer/Indices.java	Replace `collect(toList())` with `toList()` and remove static import.
component-runtime-beam/src/main/java/org/talend/sdk/component/runtime/beam/chain/impl/BeamExecutor.java	Replace `collect(toList())` with `toList()` and remove static import.
component-form/component-uispec-mapper/src/main/java/org/talend/sdk/component/form/uispec/mapper/impl/UiSpecMapperImpl.java	Replace `collect(toList())` with `toList()` and remove static import.
component-form/component-form-model/src/main/java/org/talend/sdk/component/form/model/uischema/UiSchema.java	Replace `collect(toList())` with `toList()` and remove static import.
component-form/component-form-core/src/test/java/org/talend/sdk/component/form/internal/validation/ValidationServiceTest.java	Replace `collect(toList())` with `toList()` in tests.
component-form/component-form-core/src/test/java/org/talend/sdk/component/form/api/UiSpecServiceTest.java	Replace multiple `collect(toList())` with `toList()` and remove imports.
component-form/component-form-core/src/main/java/org/talend/sdk/component/form/internal/validation/spi/ext/TypeValidation.java	Replace `collect(toList())` with `toList()` and remove static import.
component-form/component-form-core/src/main/java/org/talend/sdk/component/form/internal/converter/impl/widget/MultiSelectWidgetConverter.java	Replace `collect(toList())` with `toList()` and remove static import.
component-form/component-form-core/src/main/java/org/talend/sdk/component/form/internal/converter/impl/widget/GridLayoutWidgetConverter.java	Replace `collect(toList())` with `toList()` in tab list computation.
component-form/component-form-core/src/main/java/org/talend/sdk/component/form/internal/converter/impl/widget/DataListWidgetConverter.java	Replace `collect(toList())` with `toList()` and remove static import.
component-form/component-form-core/src/main/java/org/talend/sdk/component/form/internal/converter/impl/widget/AbstractWidgetConverter.java	Replace multiple `collect(toList())` with `toList()` in dynamic value conversion.
component-form/component-form-core/src/main/java/org/talend/sdk/component/form/internal/converter/impl/UiSchemaConverter.java	Replace `collect(toList())` with `toList()` and remove static import.
component-form/component-form-core/src/main/java/org/talend/sdk/component/form/internal/converter/impl/schema/ArrayPropertyConverter.java	Replace `collect(toList())` with `toList()` and remove static import.
component-form/component-form-core/src/main/java/org/talend/sdk/component/form/api/UiSpecService.java	Replace `collect(toList())` with `toList()` and remove static import.
component-api/src/test/java/org/talend/sdk/component/api/record/OrderedMapTest.java	Replace `Collectors.toList()` with `toList()` in tests.
component-api/src/main/java/org/talend/sdk/component/api/record/Schema.java	Replace `Collectors.toList()` with `toList()` in API defaults and parsing.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Copilot

Pull request overview

Copilot reviewed 153 out of 155 changed files in this pull request and generated no new comments.

Copilot

Pull request overview

Copilot reviewed 163 out of 165 changed files in this pull request and generated 1 comment.

sonar-rnd · 2026-05-22T16:24:56Z

Quality Gate failed

0.00% Coverage on New Code (is less than 80.00%)
10 New Issues (is greater than 0)

Project ID: org.talend.sdk.component:component-runtime

View in SonarQube

Copilot

Pull request overview

Copilot reviewed 163 out of 165 changed files in this pull request and generated 2 comments.

        public List<Entry> getCurrentEntries() {
            if (this.providedSchema != null) {
-                return Collections.unmodifiableList(this.providedSchema.getAllEntries().collect(Collectors.toList()));
+                return Collections.unmodifiableList(this.providedSchema.getAllEntries().toList());
            }
-            return this.entries.streams().collect(Collectors.toList());
+            return this.entries.streams().toList();
        }


    @JsonbTransient
    public List<Entry> getEntriesOrdered() {
-        return getAllEntries().sorted(entriesOrder).collect(toList());
+        return getAllEntries().sorted(entriesOrder).toList();
    }


chore(QTDI-2894): S6204 Stream.toList()

ffe2f53

(cherry picked from commit c877724)

undx requested a review from Copilot May 22, 2026 12:20

Copilot started reviewing on behalf of undx May 22, 2026 12:21 View session

Copilot AI reviewed May 22, 2026

View reviewed changes

undx added 4 commits May 22, 2026 15:21

chore(QTDI-2894): S6204 Stream.toList() - potential API break

17159c0

chore(QTDI-2894): S6204 Stream.toList() - restore partials

6d71def

chore(QTDI-2894): try-with-resources

3e5d2fa

chore(QTDI-2894): remove compiler level 1.8

9cd070b

undx requested a review from Copilot May 22, 2026 13:36

Copilot started reviewing on behalf of undx May 22, 2026 13:36 View session

Copilot AI reviewed May 22, 2026

View reviewed changes

undx added 3 commits May 22, 2026 16:36

chore(QTDI-2894): fix compiler level 17 tests

c96713e

chore(QTDI-2894): add safe S6204 not detected

01b1ba4

chore(QTDI-2894): add more S6204

573bcbb

undx requested a review from Copilot May 22, 2026 15:49

Copilot started reviewing on behalf of undx May 22, 2026 15:49 View session

Copilot AI reviewed May 22, 2026

View reviewed changes

Comment thread ...-server/src/test/java/org/talend/sdk/component/starter/server/front/ProjectResourceTest.java

chore(QTDI-2894): fix test for non regression

1a8b0cb

Copilot started work on behalf of undx May 22, 2026 17:05 View session

Copilot finished work on behalf of undx May 22, 2026 17:07

undx requested a review from Copilot May 22, 2026 17:28

Copilot started reviewing on behalf of undx May 22, 2026 17:28 View session

Copilot AI reviewed May 22, 2026

View reviewed changes

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

chore(QTDI-2894): S6204 Stream.toList()#1228

chore(QTDI-2894): S6204 Stream.toList()#1228
undx wants to merge 9 commits into
masterfrom
ouf/QTDI-2894_stream_tolist

undx commented May 22, 2026

Uh oh!

Copilot AI left a comment

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Copilot AI left a comment

Uh oh!

Copilot AI left a comment

Uh oh!

Uh oh!

sonar-rnd Bot commented May 22, 2026

Uh oh!

Copilot AI left a comment

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

3 participants

Conversation

undx commented May 22, 2026

Requirements

Why this PR is needed?

What does this PR adds (design/code thoughts)?

AI generated code

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Reviewed changes

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Uh oh!

Uh oh!

sonar-rnd Bot commented May 22, 2026

Quality Gate failed

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

3 participants