Generate valid phpdoc param tags (#1457)

* Fix the phpdoc of the RequestContext

* Generate valid phpdoc param tags

When an array shape key starts with a `@` like `@region`, it must be
quoted in the phpdoc as the unquoted one would be interpreted as the
next phpdoc tag.

* Use an unsealed shape for the type in Input

This avoids reports about passing extra keys from child classes in
Psalm.

* Allow passing null as the region in the input

The base input class already accept null values in addition to omitted
values for the region.

* Update the psalm baseline

* Disallow invalid docblocks in Psalm as well

Author: stof

src/CodeCommitClient.php

@@ -86,8 +86,7 @@ class CodeCommitClient extends AbstractApi
      *   repositoryName: string,
      *   repositoryDescription?: string,
      *   tags?: array<string, string>,
-     *
-     *   @region?: string,
+     *   '@region'?: string|null,
      * }|CreateRepositoryInput $input
      *
      * @throws RepositoryNameExistsException
@@ -139,8 +138,7 @@ public function createRepository($input): CreateRepositoryOutput
      *
      * @param array{
      *   repositoryName: string,
-     *
-     *   @region?: string,
+     *   '@region'?: string|null,
      * }|DeleteRepositoryInput $input
      *
      * @throws RepositoryNameRequiredException
@@ -176,8 +174,7 @@ public function deleteRepository($input): DeleteRepositoryOutput
      * @param array{
      *   repositoryName: string,
      *   blobId: string,
-     *
-     *   @region?: string,
+     *   '@region'?: string|null,
      * }|GetBlobInput $input
      *
      * @throws RepositoryNameRequiredException
@@ -223,8 +220,7 @@ public function getBlob($input): GetBlobOutput
      * @param array{
      *   repositoryName?: string,
      *   branchName?: string,
-     *
-     *   @region?: string,
+     *   '@region'?: string|null,
      * }|GetBranchInput $input
      *
      * @throws RepositoryNameRequiredException
@@ -268,8 +264,7 @@ public function getBranch($input = []): GetBranchOutput
      * @param array{
      *   repositoryName: string,
      *   commitId: string,
-     *
-     *   @region?: string,
+     *   '@region'?: string|null,
      * }|GetCommitInput $input
      *
      * @throws RepositoryNameRequiredException
@@ -319,8 +314,7 @@ public function getCommit($input): GetCommitOutput
      *   afterPath?: string,
      *   MaxResults?: int,
      *   NextToken?: string,
-     *
-     *   @region?: string,
+     *   '@region'?: string|null,
      * }|GetDifferencesInput $input
      *
      * @throws RepositoryNameRequiredException
@@ -375,8 +369,7 @@ public function getDifferences($input): GetDifferencesOutput
      *   nextToken?: string,
      *   sortBy?: SortByEnum::*,
      *   order?: OrderEnum::*,
-     *
-     *   @region?: string,
+     *   '@region'?: string|null,
      * }|ListRepositoriesInput $input
      *
      * @throws InvalidSortByException
@@ -404,8 +397,7 @@ public function listRepositories($input = []): ListRepositoriesOutput
      * @param array{
      *   repositoryName: string,
      *   triggers: RepositoryTrigger[],
-     *
-     *   @region?: string,
+     *   '@region'?: string|null,
      * }|PutRepositoryTriggersInput $input
      *
      * @throws RepositoryDoesNotExistException

src/Input/CreateRepositoryInput.php

@@ -51,8 +51,7 @@ final class CreateRepositoryInput extends Input
      *   repositoryName?: string,
      *   repositoryDescription?: string,
      *   tags?: array<string, string>,
-     *
-     *   @region?: string,
+     *   '@region'?: string|null,
      * } $input
      */
     public function __construct(array $input = [])

src/Input/DeleteRepositoryInput.php

@@ -24,8 +24,7 @@ final class DeleteRepositoryInput extends Input
     /**
      * @param array{
      *   repositoryName?: string,
-     *
-     *   @region?: string,
+     *   '@region'?: string|null,
      * } $input
      */
     public function __construct(array $input = [])

src/Input/GetBlobInput.php

@@ -34,8 +34,7 @@ final class GetBlobInput extends Input
      * @param array{
      *   repositoryName?: string,
      *   blobId?: string,
-     *
-     *   @region?: string,
+     *   '@region'?: string|null,
      * } $input
      */
     public function __construct(array $input = [])

src/Input/GetBranchInput.php

@@ -29,8 +29,7 @@ final class GetBranchInput extends Input
      * @param array{
      *   repositoryName?: string,
      *   branchName?: string,
-     *
-     *   @region?: string,
+     *   '@region'?: string|null,
      * } $input
      */
     public function __construct(array $input = [])

src/Input/GetCommitInput.php

@@ -34,8 +34,7 @@ final class GetCommitInput extends Input
      * @param array{
      *   repositoryName?: string,
      *   commitId?: string,
-     *
-     *   @region?: string,
+     *   '@region'?: string|null,
      * } $input
      */
     public function __construct(array $input = [])

src/Input/GetDifferencesInput.php

@@ -76,8 +76,7 @@ final class GetDifferencesInput extends Input
      *   afterPath?: string,
      *   MaxResults?: int,
      *   NextToken?: string,
-     *
-     *   @region?: string,
+     *   '@region'?: string|null,
      * } $input
      */
     public function __construct(array $input = [])

src/Input/ListRepositoriesInput.php

@@ -42,8 +42,7 @@ final class ListRepositoriesInput extends Input
      *   nextToken?: string,
      *   sortBy?: SortByEnum::*,
      *   order?: OrderEnum::*,
-     *
-     *   @region?: string,
+     *   '@region'?: string|null,
      * } $input
      */
     public function __construct(array $input = [])

src/Input/PutRepositoryTriggersInput.php

@@ -35,8 +35,7 @@ final class PutRepositoryTriggersInput extends Input
      * @param array{
      *   repositoryName?: string,
      *   triggers?: RepositoryTrigger[],
-     *
-     *   @region?: string,
+     *   '@region'?: string|null,
      * } $input
      */
     public function __construct(array $input = [])