Decrease psalm error level (#85)

Author: keelerm84

composer.json

@@ -24,7 +24,7 @@
         "kevinrob/guzzle-cache-middleware": "^3",
         "phpunit/php-code-coverage": "^9",
         "phpunit/phpunit": "^9",
-        "vimeo/psalm": "4.9.2"
+        "vimeo/psalm": "^4.0"
     },
     "suggest": {
         "guzzlehttp/guzzle": "(^7) Required when using GuzzleEventPublisher or the default FeatureRequester",

psalm.xml

@@ -1,6 +1,6 @@
 <?xml version="1.0"?>
 <psalm
-    errorLevel="4"
+    errorLevel="2"
     resolveFromConfigFile="true"
     xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance"
     xmlns="https://getpsalm.org/schema/config"

src/LaunchDarkly/FeatureFlagsState.php

@@ -21,7 +21,7 @@ class FeatureFlagsState implements \JsonSerializable
     /** @var array */
     protected $_flagValues;
 
-    /** @var array */
+    /** @var array<string, array{debugEventsUntilDate?: int|null, reason?: EvaluationReason, trackEvents?: true, variation?: int|null, version?: int}> **/
     protected $_flagMetadata;
 
     /**

src/LaunchDarkly/FeatureRequester.php

@@ -33,7 +33,7 @@ public function getSegment(string $key): ?Segment;
     /**
      * Gets all feature flags.
      *
-     * @return array|null The decoded FeatureFlags, or null if missing
+     * @return array<string, FeatureFlag>|null The decoded FeatureFlags, or null if missing
      */
     public function getAllFeatures(): ?array;
 }

src/LaunchDarkly/Impl/Events/EventFactory.php

@@ -58,7 +58,7 @@ public function newEvalEvent(
         if ($prereqOfFlag) {
             $e['prereqOf'] = $prereqOfFlag->getKey();
         }
-        if (($addExperimentData || $this->_withReasons) && $detail->getReason()) {
+        if (($addExperimentData || $this->_withReasons)) {
             $e['reason'] = $detail->getReason()->jsonSerialize();
         }
         if ($user->getAnonymous()) {
@@ -88,7 +88,7 @@ public function newDefaultEvent(FeatureFlag $flag, LDUser $user, EvaluationDetai
         if ($flag->getDebugEventsUntilDate()) {
             $e['debugEventsUntilDate'] = $flag->getDebugEventsUntilDate();
         }
-        if ($this->_withReasons && $detail->getReason()) {
+        if ($this->_withReasons) {
             $e['reason'] = $detail->getReason()->jsonSerialize();
         }
         if ($user->getAnonymous()) {
@@ -111,7 +111,7 @@ public function newUnknownFlagEvent(string $key, LDUser $user, EvaluationDetail
             'default' => $detail->getValue()
         ];
         // the following properties are handled separately so we don't waste bandwidth on unused keys
-        if ($this->_withReasons && $detail->getReason()) {
+        if ($this->_withReasons) {
             $e['reason'] = $detail->getReason()->jsonSerialize();
         }
         if ($user->getAnonymous()) {

src/LaunchDarkly/Impl/Events/EventProcessor.php

@@ -25,11 +25,14 @@ class EventProcessor
     /** @var int */
     private $_capacity;
 
-    public function __construct(string $sdkKey, array $options = [])
+    /**
+     * @psalm-param array{capacity: int} $options
+     */
+    public function __construct(string $sdkKey, array $options)
     {
         $this->_eventPublisher = $this->getEventPublisher($sdkKey, $options);
         $this->_eventSerializer = new EventSerializer($options);
-      
+
         $this->_capacity = $options['capacity'];
     }
 

src/LaunchDarkly/Impl/Integrations/FeatureRequesterBase.php

@@ -125,7 +125,7 @@ public function getSegment(string $key): ?Segment
     /**
      * Gets all features
      *
-     * @return array|null The decoded FeatureFlags, or null if missing
+     * @return array<string, FeatureFlag>|null The decoded FeatureFlags, or null if missing
      */
     public function getAllFeatures(): ?array
     {

src/LaunchDarkly/Impl/Integrations/FileDataFeatureRequester.php

@@ -55,7 +55,7 @@ public function getSegment(string $key): ?Segment
     /**
      * Gets all feature flags
      *
-     * @return array|null The decoded FeatureFlags, or null if missing
+     * @return array<string, FeatureFlag>|null The decoded FeatureFlags, or null if missing
      */
     public function getAllFeatures(): ?array
     {

src/LaunchDarkly/Impl/Integrations/GuzzleEventPublisher.php

@@ -49,7 +49,6 @@ public function __construct(string $sdkKey, array $options = [])
     public function publish(string $payload): bool
     {
         $client = new Client(['base_uri' => $this->_eventsUri]);
-        $response = null;
 
         try {
             $options = $this->_requestOptions;

src/LaunchDarkly/Impl/Integrations/GuzzleFeatureRequester.php

@@ -111,7 +111,7 @@ public function getSegment(string $key): ?Segment
     /**
      * Gets all features from a likely cached store
      *
-     * @return array|null The decoded FeatureFlags, or null if missing
+     * @return array<string, FeatureFlag>|null The decoded FeatureFlags, or null if missing
      */
     public function getAllFeatures(): ?array
     {