Add getVersion method to the Environment class

Author: Kaspiman

.github/ISSUE_TEMPLATE/1_Bug_report.md

@@ -14,10 +14,10 @@ labels: Bug
 
 ### Additional Info
 
-| Q                | A
-|------------------| ---
-| Package Version  | x.y.z <!-- Please set the package version -->
-| PHP version      | x.y.z <!-- Please set the PHP version -->
-| Operating system | Linux <!-- Please set your OS -->
+| Q                | A                                             |
+|------------------|-----------------------------------------------|
+| Package Version  | x.y.z <!-- Please set the package version --> |
+| PHP version      | x.y.z <!-- Please set the PHP version -->     |
+| Operating system | Linux <!-- Please set your OS -->             |
 
 <!-- Optional: any other context about the problem: log messages, screenshots, etc. -->

.github/PULL_REQUEST_TEMPLATE.md

@@ -1,10 +1,10 @@
-| Q             | A
-| ------------- | ---
-| Bugfix?       | ✔️/❌
-| Breaks BC?    | ✔️/❌ <!-- please update "xxx Impact Changes" section in CHANGELOG.md file -->
-| New feature?  | ✔️/❌ <!-- please update "Other Features" section in CHANGELOG.md file -->
-| Issues        | #... <!-- prefix each issue number with "#" symbol, no need to create an issue if none exist, explain below instead -->
-| Docs PR       | spiral/docs#... <!-- prefix each issue number with "spiral/docs#", required only for new features -->
+| Q            | A                                                                                                                       |
+|--------------|-------------------------------------------------------------------------------------------------------------------------|
+| Bugfix?      | ✔️/❌                                                                                                                    |
+| Breaks BC?   | ✔️/❌ <!-- please update "xxx Impact Changes" section in CHANGELOG.md file -->                                           |
+| New feature? | ✔️/❌ <!-- please update "Other Features" section in CHANGELOG.md file -->                                               |
+| Issues       | #... <!-- prefix each issue number with "#" symbol, no need to create an issue if none exist, explain below instead --> |
+| Docs PR      | spiral/docs#... <!-- prefix each issue number with "spiral/docs#", required only for new features -->                   |
 
 <!-- Please, replace this notice by a short description of your feature/bugfix.
 This will help people understand your PR and can be used as a start for the documentation. -->

src/Environment.php

@@ -9,9 +9,10 @@
 /**
  * @psalm-import-type ModeType from Mode
  * @psalm-type EnvironmentVariables = array{
- *      RR_MODE?:   ModeType|string,
- *      RR_RELAY?:  string,
- *      RR_RPC?:    string,
+ *      RR_MODE?:    ModeType|string,
+ *      RR_RELAY?:   string,
+ *      RR_RPC?:     string,
+ *      RR_VERSION?: string,
  * }|array<string, string>
  * @see Mode
  */
@@ -27,7 +28,7 @@ public function __construct(
 
     public function getMode(): string
     {
-        return $this->get('RR_MODE', '');
+        return $this->get('RR_MODE');
     }
 
     public function getRelayAddress(): string
@@ -40,6 +41,11 @@ public function getRPCAddress(): string
         return $this->get('RR_RPC', 'tcp://127.0.0.1:6001');
     }
 
+    public function getVersion(): string
+    {
+        return $this->get('RR_VERSION');
+    }
+
     /**
      * @template TDefault of string
      *

src/EnvironmentInterface.php

@@ -12,6 +12,7 @@
  *
  * @psalm-import-type ModeType from Mode
  * @see Mode
+ * @method string getVersion()
  */
 interface EnvironmentInterface
 {

src/Worker.php

@@ -276,11 +276,10 @@ public static function createFromEnvironment(
         );
     }
 
-    private function sendProcessId(): static
+    private function sendProcessId(): void
     {
         $frame = new Frame($this->encode(['pid' => \getmypid()]), [], Frame::CONTROL);
         $this->sendFrame($frame);
-        return $this;
     }
 
     private function haveToPing(): void

tests/Unit/EnvironmentTest.php

@@ -45,15 +45,23 @@ public function testGetRPCAddressWithValue(): void
         $this->assertEquals('rpc_value', $env->getRPCAddress());
     }
 
+    public function testGetVersionWithValue(): void
+    {
+        $env = new Environment(['RR_VERSION' => '2024.1.3']);
+        $this->assertEquals('2024.1.3', $env->getVersion());
+    }
+
     public function testFromGlobals(): void
     {
         $_ENV['RR_MODE'] = 'global_mode';
         $_SERVER['RR_RELAY'] = 'global_relay';
+        $_SERVER['RR_VERSION'] = 'global_version';
 
         $env = Environment::fromGlobals();
 
         $this->assertEquals('global_mode', $env->getMode());
         $this->assertEquals('global_relay', $env->getRelayAddress());
+        $this->assertEquals('global_version', $env->getVersion());
         $this->assertEquals('tcp://127.0.0.1:6001', $env->getRPCAddress());
     }
 }