Add support for WWW-Authenticate header (SCA associations management)

[gcatanese]

PR to add support for WWW-Authenticate:

• update templates to NOT include header in the signature
• add support for the header in RequestOptions, same approach used for IdempotencyKey
• generate services/models and add tests
• add test to verify response headers are available when the API returns 401 (ApiException)

[Copilot]

Pull Request Overview

This pull request adds support for the WWW-Authenticate header in request handling, enabling SCA (Strong Customer Authentication) associations management functionality.

Key Changes:

• Updated Mustache templates to exclude header parameters from method signatures and include required query parameters in Javadoc
• Added wwwAuthenticateHeader support to RequestOptions with getter, setter, and builder pattern method
• Generated new service classes and models for SCA device and association management
• Corrected parameter ordering in transfer limits API methods

Reviewed Changes

Copilot reviewed 41 out of 41 changed files in this pull request and generated 18 comments.

Show a summary per file

File	Description
templates-v7/libraries/jersey3/api_summary_overload.mustache	Modified Javadoc generation to include path and required query params separately
templates-v7/libraries/jersey3/api_overload.mustache	Updated to exclude header and body params from required params list
src/main/java/com/adyen/model/RequestOptions.java	Added WWW-Authenticate header field with getter/setter and addAdditionalServiceHeader helper
src/main/java/com/adyen/httpclient/AdyenHttpClient.java	Added logic to set WWW-Authenticate header from RequestOptions
src/main/java/com/adyen/constants/ApiConstants.java	Added WWW_AUTHENTICATE_HEADER constant
src/main/java/com/adyen/service/balanceplatform/ScaAssociationManagementApi.java	New service API for managing SCA associations
src/main/java/com/adyen/service/balanceplatform/ScaDeviceManagementApi.java	New service API for managing SCA devices
src/main/java/com/adyen/service/balanceplatform/TransferLimits*.java	Fixed parameter ordering for consistency
src/main/java/com/adyen/model/balanceplatform/*	Generated models for SCA entities, devices, associations, and requests/responses
src/test/java/com/adyen/BalancePlatformTest.java	Added comprehensive tests for SCA management operations
src/test/java/com/adyen/httpclient/ClientTest.java	Added tests for WWW-Authenticate header and addAdditionalServiceHeader helper
src/test/resources/mocks/balancePlatform/*.json	Added mock response files for SCA operations

---

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

[Copilot]

Redundant imports: java.util.* already includes ArrayList and List, so the explicit imports on lines 19-20 are unnecessary.

Suggested change

	import java.util.ArrayList;
	import java.util.List;

[Copilot]

Redundant imports: java.util.* already includes ArrayList and List, so the explicit imports on lines 19-20 are unnecessary.

Suggested change

	import java.util.ArrayList;
	import java.util.List;

[Copilot]

Redundant imports: java.util.* already includes ArrayList and List, so the explicit imports on lines 19-20 are unnecessary.

Suggested change

	import java.util.ArrayList;
	import java.util.List;

[Copilot]

Redundant imports: java.util.* already includes ArrayList and List, so the explicit imports on lines 19-20 are unnecessary.

Suggested change

	import java.util.ArrayList;
	import java.util.List;

[Copilot]

Redundant imports: java.util.* already includes ArrayList and List, so the explicit imports on lines 19-20 are unnecessary.

Suggested change

	import java.util.ArrayList;
	import java.util.List;

[Copilot]

Unused wildcard import: java.util.* is imported but no types from this package are used in the class except Objects, which should be imported explicitly.

Suggested change

	import java.util.*;
	import java.util.Objects;

[Copilot]

Unused wildcard import: java.util.* is imported but no types from this package are used in the class except Objects, which should be imported explicitly.

Suggested change

import java.util.*;

[Copilot]

Unused wildcard import: java.util.* is imported but no types from this package are used in the class except Objects, which should be imported explicitly.

Suggested change

import java.util.*;

[Copilot]

Unused wildcard import: java.util.* is imported but no types from this package are used in the class except Objects, which should be imported explicitly.

Suggested change

	import java.util.*;
	import java.util.Objects;

[Copilot]

Unused wildcard import: java.util.* is imported but no types from this package are used in the class except Objects, which should be imported explicitly.

Suggested change

	import java.util.*;
	import java.util.Objects;

[gcatanese]

/gemini review

[gemini-code-assist]

Code Review

This pull request introduces support for the WWW-Authenticate header, which is now configurable through RequestOptions. It also adds new services for SCA (Strong Customer Authentication) association and device management, complete with new models and comprehensive tests. Additionally, the code generation templates have been updated, resulting in cleaner Javadocs and more consistent method signatures across the API.

My review focuses on the impact of these changes. I've identified a few public methods where parameter order has changed, which constitutes a breaking change. I've also noted a minor usability issue in one of the new service methods regarding pagination parameters. The rest of the changes, including the new features and tests, are well-implemented and consistent with the existing codebase.

[gemini-code-assist]

Code Review

This pull request introduces support for the WWW-Authenticate header, which has been correctly implemented in RequestOptions and AdyenHttpClient. A significant portion of the changes involves the addition of auto-generated model and service classes for SCA association management. The PR also includes several valuable fixes and improvements, such as removing duplicate Javadoc parameters and enhancing code generation templates for better clarity and consistency. I've noted a few breaking changes in method signatures within the TransferLimitsBalanceAccountLevelApi and TransferLimitsBalancePlatformLevelApi classes, where parameter orders were swapped to improve consistency with API endpoint paths. These have been flagged as medium-severity issues due to their impact on existing users of the library. The new tests for the SCA functionality are well-written and provide good coverage.

[galesky-a]

LGTM, as we talked offline, there is a tradeoff in devx by not making the header explicitly required on the builder for SCA. I think this PR still lays the groundwork for that anyway.