chore: prepare GAX for monorepo

.gitattributes

@@ -1,8 +1,5 @@
 .gitattributes export-ignore
 .gitignore export-ignore
 .github export-ignore
-dev export-ignore
 tests export-ignore
-codecov.yml export-ignore
-ruleset.xml export-ignore
-.travis.yml export-ignore
+/*.dist export-ignore

README.md

@@ -73,21 +73,13 @@ be found for Mac or Windows.
 3.  Run tests.
 
     ```sh
-    > composer test
+    > vendor/bin/phpunit
     ```
 
-4.  Updating dependencies after changing `composer.json`:
+4.  Run static analysis tools
 
     ```sh
-    > composer update
-    `
-    ```
-
-5.  Formatting source:
-
-    ```sh
-    > composer cs-lint
-    > composer cs-fix
+    > phpstan -c phpstan.neon.dist
     ```
 
 ## License

composer.json

@@ -34,17 +34,9 @@
     },
     "autoload-dev": {
         "psr-4": {
-            "Google\\ApiCore\\Dev\\": "dev/src",
             "Google\\ApiCore\\Tests\\": "tests",
-            "GPBMetadata\\Google\\": "metadata/Google",
             "Google\\Showcase\\": "tests/Conformance/src",
             "GPBMetadata\\Google\\Showcase\\": "tests/Conformance/metadata"
         }
-    },
-    "scripts": {
-        "regenerate-test-protos": "dev/sh/regenerate-test-protos.sh",
-        "test": "./vendor/bin/phpunit",
-        "cs-lint": "vendor/bin/php-tools cs-fixer googleapis/gax-php --ref $(git rev-parse --abbrev-ref HEAD) --flags \"\\-n --dry-run\"",
-        "cs-fix": "vendor/bin/php-tools cs-fixer googleapis/gax-php --ref $(git rev-parse --abbrev-ref HEAD) --flags \"\\-n\""
     }
 }

phpunit.xml.dist

@@ -1,8 +1,5 @@
 <?xml version="1.0" encoding="UTF-8"?>
 <phpunit xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance" bootstrap="tests/bootstrap.php" xsi:noNamespaceSchemaLocation="https://schema.phpunit.de/9.3/phpunit.xsd" colors="true">
-  <php>
-    <env name="GOOGLE_APPLICATION_CREDENTIALS" value="tests/Unit/testdata/json-key-file.json"/>
-  </php>
   <coverage>
     <include>
       <directory suffix=".php">src/ApiCore</directory>

src/ClientOptionsTrait.php

@@ -152,8 +152,7 @@ private function buildClientOptions(array|ClientOptions $options)
             $options['logger'] = ApplicationDefaultCredentials::getDefaultLogger();
         }
 
-        if (
-            $options['logger'] !== null
+        if ($options['logger'] !== null
             && $options['logger'] !== false
             && !$options['logger'] instanceof LoggerInterface
         ) {

src/OperationResponse.php

@@ -101,7 +101,7 @@ class OperationResponse
      * OperationResponse constructor.
      *
      * @param string $operationName
-     * @param object $operationsClient
+     * @param object|null $operationsClient
      * @param array $options {
      *                       Optional. Options for configuring the operation response object.
      *

src/ResourceTemplate/AbsoluteResourceTemplate.php

@@ -54,7 +54,7 @@
 class AbsoluteResourceTemplate implements ResourceTemplateInterface
 {
     private RelativeResourceTemplate $resourceTemplate;
-    /** @var string|bool */
+    /** @var string */
     private $verb;
 
     /**

src/ResourceTemplate/Segment.php

@@ -51,8 +51,8 @@ class Segment
     private ?string $value;
     private ?string $key;
     private ?RelativeResourceTemplate $template;
-    private ?string $stringRepr;
-    private ?string $separator;
+    private string $stringRepr;
+    private string $separator;
 
     /**
      * Segment constructor.

src/Serializer.php

@@ -37,6 +37,17 @@
 use Google\Protobuf\DescriptorPool;
 use Google\Protobuf\FieldDescriptor;
 use Google\Protobuf\Internal\Message;
+use Google\Rpc\BadRequest;
+use Google\Rpc\DebugInfo;
+use Google\Rpc\ErrorInfo;
+use Google\Rpc\Help;
+use Google\Rpc\LocalizedMessage;
+use Google\Rpc\PreconditionFailure;
+use Google\Rpc\QuotaFailure;
+use Google\Rpc\RequestInfo;
+use Google\Rpc\ResourceInfo;
+use Google\Rpc\RetryInfo;
+use Google\Rpc\Status;
 use RuntimeException;
 
 /**
@@ -180,7 +191,7 @@ public static function decodeMetadata(array $metadata, ?array &$errors = null)
         $result = [];
         // If metadata contains a "status" bin, use that instead
         if (isset($metadata['grpc-status-details-bin'])) {
-            $status = new \Google\Rpc\Status();
+            $status = new Status();
             $status->mergeFromString($metadata['grpc-status-details-bin'][0]);
             foreach ($status->getDetails() as $any) {
                 if (isset(KnownTypes::TYPE_URLS[$any->getTypeUrl()])) {
@@ -212,7 +223,18 @@ public static function decodeMetadata(array $metadata, ?array &$errors = null)
                 if (self::hasBinaryHeaderSuffix($key)) {
                     if (isset(KnownTypes::BIN_TYPES[$key])) {
                         $class = KnownTypes::BIN_TYPES[$key];
-                        /** @var Message $message */
+                        /**
+                         * @var BadRequest
+                         *    | DebugInfo
+                         *    | ErrorInfo
+                         *    | Help
+                         *    | LocalizedMessage
+                         *    | PreconditionFailure
+                         *    | QuotaFailure
+                         *    | RequestInfo
+                         *    | ResourceInfo
+                         *    | RetryInfo $message
+                         */
                         $message = new $class();
                         try {
                             $message->mergeFromString($value);
@@ -470,12 +492,10 @@ private function checkFieldRepeated(FieldDescriptor $field): bool
         if (method_exists($field, 'isRepeated')) {
             return $field->isRepeated();
         }
-
         if (method_exists($field, 'getLabel')) {
             return $field->getLabel() === GPBLabel::REPEATED;
         }
-
-        throw new \Exception('No field repeated method avaialble');
+        throw new \LogicException('unable to check field repeated');
     }
 
     /**

src/Transport/Rest/JsonStreamDecoder.php

@@ -92,7 +92,7 @@ public function __construct(StreamInterface $stream, string $decodeType, array $
     public function decode()
     {
         try {
-            foreach ($this->_decode() as $response) {
+            foreach ($this->doDecode() as $response) {
                 yield $response;
             }
         } catch (RuntimeException $re) {
@@ -112,7 +112,7 @@ public function decode()
     /**
      * @return \Generator
      */
-    private function _decode()
+    private function doDecode()
     {
         $decodeType = $this->decodeType;
         $str = false;

src/Transport/Rest/RestServerStreamingCall.php

@@ -55,7 +55,7 @@ class RestServerStreamingCall implements ServerStreamingCallInterface
     private array $decoderOptions;
 
     private RequestInterface $originalRequest;
-    private ?JsonStreamDecoder $decoder;
+    private JsonStreamDecoder $decoder;
     private string $decodeType;
     private ?ResponseInterface $response;
     private stdClass $status;
@@ -181,7 +181,7 @@ public function getPeer()
      */
     public function cancel()
     {
-        if (!is_null($this->decoder)) {
+        if (isset($this->decoder)) {
             $this->decoder->close();
         }
     }

src/Transport/RestTransport.php

@@ -200,7 +200,7 @@ public function startServerStreamingCall(Call $call, array $options)
         $request = $this->requestBuilder->build(
             $call->getMethod(),
             $call->getMessage()
-            // Exclude headers here because they will be added in _serverStreamRequest().
+            // Exclude headers here because they will be added in doServerStreamRequest().
         );
 
         $decoderOptions = [];
@@ -209,7 +209,7 @@ public function startServerStreamingCall(Call $call, array $options)
         }
 
         return new ServerStream(
-            $this->_serverStreamRequest(
+            $this->doServerStreamRequest(
                 $this->httpHandler,
                 $request,
                 $headers,
@@ -233,7 +233,7 @@ public function startServerStreamingCall(Call $call, array $options)
      *
      * @return RestServerStreamingCall
      */
-    private function _serverStreamRequest(
+    private function doServerStreamRequest(
         $httpHandler,
         $request,
         $headers,