Update reports.proto (#2678)

* Update `reports.proto`

* update generated proto

* avoid branch prefix

* fix rector config

* fix PHPStan

---------

Co-authored-by: spawnia <spawnia@users.noreply.github.com>
Co-authored-by: Benedikt Franke <benedikt.franke@mll.com>

Author: github-actions[bot]

.github/workflows/update-reports-proto.yml

@@ -24,5 +24,5 @@ jobs:
         with:
           title: Update `reports.proto`
           commit-message: Update `reports.proto`
-          branch: patch/update-reports-proto
+          branch: update-reports-proto
           delete-branch: true

rector.php

@@ -47,7 +47,6 @@
         Rector\PHPUnit\CodeQuality\Rector\Class_\NarrowUnusedSetUpDefinedPropertyRector::class, // falsely removes $this->schema assignments in some tests
         Rector\PHPUnit\PHPUnit100\Rector\StmtsAwareInterface\WithConsecutiveRector::class, // messes up our custom withConsecutive replacement
         Rector\PHPUnit\PHPUnit60\Rector\ClassMethod\AddDoesNotPerformAssertionToNonAssertingTestRector::class, // does not recognize mockResolver
-        Rector\PHPUnit\CodeQuality\Rector\MethodCall\AssertCountWithZeroToAssertEmptyRector::class, // Allows garbage
         Rector\PHPUnit\CodeQuality\Rector\MethodCall\AssertEmptyNullableObjectToAssertInstanceofRector::class, // Makes assertions more brittle
     ]);
     $rectorConfig->paths([

src/Schema/Directives/HideDirective.php

@@ -26,7 +26,7 @@ public function __construct()
         assert($app instanceof Application);
 
         $environment = $app->environment();
-        assert(is_string($environment), 'Calling this method without parameters returns the current value of the environment.');
+        assert(is_string($environment), 'Calling this method without parameters returns the current value of the environment.'); // @phpstan-ignore function.alreadyNarrowedType,function.alreadyNarrowedType (dynamic type known only with the latest Larastan version)
 
         $this->env = $environment;
     }

src/Tracing/FederatedTracing/Proto/Metadata/Reports.php

@@ -589,6 +589,8 @@ message QueryMetadata {
   string signature = 2;
   // (Optional) Persisted query ID that was used to request this operation.
   string pq_id = 3;
+  // The query id. Must be present for all operations. If missing, operations will appear in the insights page without a signature and possibly without a name.
+  string operation_id = 4;
 }
 
 message ExtendedReferences {