ItemSearchResults __construct fails, if the response misses the pagination information about selling-partner-api

[jlevers]

This is the purpose of the modifications.json file, which is documented here. I'd use that to handle this issue. If you need more guidance after taking a look at those links, let me know :)

[KhorneHoly]

This is the purpose of the modifications.json file, which is documented here. I'd use that to handle this issue. If you need more guidance after taking a look at those links, let me know :)

Thanks, didn't noticed that, sorry. Created a pull request.

Note: The generation method touched multiple dozens of files, while one one did in fact change.
I needed to revert a lot of changes done by the process.

All of that changes were changes in formatting. Spaces to intend were changed, the placement of the curly brackets was changed a bit, spaces within the Docblock were changed a bit.

Example constructor before the generation:

public function __construct(
        protected array $marketplaceIds,
        protected ?array $identifiers = null,
        protected ?string $identifiersType = null,
        protected ?array $includedData = null,
        protected ?string $locale = null,
        protected ?string $sellerId = null,
        protected ?array $keywords = null,
        protected ?array $brandNames = null,
        protected ?array $classificationIds = null,
        protected ?int $pageSize = null,
        protected ?string $pageToken = null,
        protected ?string $keywordsLocale = null,
    ) {}

After generation:

public function __construct(
		protected array $marketplaceIds,
		protected ?array $identifiers = null,
		protected ?string $identifiersType = null,
		protected ?array $includedData = null,
		protected ?string $locale = null,
		protected ?string $sellerId = null,
		protected ?array $keywords = null,
		protected ?array $brandNames = null,
		protected ?array $classificationIds = null,
		protected ?int $pageSize = null,
		protected ?string $pageToken = null,
		protected ?string $keywordsLocale = null,
	) {}

Also, the changes we had done to the Deserialies.php Trait were reverted back to the faulty state. The file doesn't state that it is auto generated, but it changes with the schema:generate command.

I'm a bit in a hurry right now, so I can't dig deeper into this, but I wanted to let you know about that. Maybe there's something missing to unify the generation.

[jlevers]

I think we resolved in this in the PR, but those changes were because the current flow assumes composer lint will be run after codegen. Maybe I'll update the build process to automatically run the linter.