refactor: add PHP 7.4+ property typehints to all class properties (#133)

Add missing typehints to all class properties across the codebase,
replacing PHPDoc-only type annotations with native PHP property type
declarations.

Changes include:
- Added typehints to all properties in main classes (Har, Log, Entry,
Request, Response, Cache, Content, etc.)
- Added typehints to all trait properties (CommentTrait, NameTrait,
BodySizeTrait, etc.)
- Used nullable types (?Type) for properties that can be null
- Set explicit default values where needed
- Used -1.0 (float) for timing properties with -1 defaults
- Updated comparisons in Timings class to use -1.0 for consistency

All tests pass successfully after these changes.

Co-authored-by: Claude <[email protected]>

Author: deviantintegral

src/Cache.php

@@ -18,21 +18,17 @@ final class Cache
      * beforeRequest [object, optional] - State of a cache entry before the
      * request. Leave out this field if the information is not available.
      *
-     * @var CacheState
-     *
      * @Serializer\Type("Deviantintegral\Har\CacheState")
      */
-    private $beforeRequest;
+    private ?CacheState $beforeRequest = null;
 
     /**
      * afterRequest [object, optional] - State of a cache entry after the
      * request. Leave out this field if the information is not available.
      *
-     * @var CacheState
-     *
      * @Serializer\Type("Deviantintegral\Har\CacheState")
      */
-    private $afterRequest;
+    private ?CacheState $afterRequest = null;
 
     public function hasBeforeRequest(): bool
     {

src/CacheState.php

@@ -19,29 +19,23 @@ final class CacheState
     /**
      * lastAccess [string] - The last time the cache entry was opened.
      *
-     * @var string
-     *
      * @Serializer\Type("string")
      */
-    private $lastAccess;
+    private string $lastAccess;
 
     /**
      * eTag [string] - Etag.
      *
-     * @var string
-     *
      * @Serializer\Type("string")
      */
-    private $eTag;
+    private string $eTag;
 
     /**
      * hitCount [number] - The number of times the cache entry has been opened.
      *
-     * @var int
-     *
      * @Serializer\Type("integer")
      */
-    private $hitCount = 0;
+    private int $hitCount = 0;
 
     public function getLastAccess(): string
     {

src/Content.php

@@ -20,42 +20,34 @@ final class Content
      * response.bodySize if there is no compression and bigger when the content
      * has been compressed.
      *
-     * @var int
-     *
      * @Serializer\Type("integer")
      */
-    private $size;
+    private int $size;
 
     /**
      * compression [number, optional] - Number of bytes saved. Leave out this
      * field if the information is not available.
      *
-     * @var int
-     *
      * @Serializer\Type("integer")
      */
-    private $compression;
+    private int $compression;
 
     /**
      * Number of bytes saved. Leave out this field if the information is not
      * available.
      *
-     * @var int
-     *
      * @Serializer\Type("integer")
      */
-    private $number;
+    private ?int $number = null;
 
     /**
      * Encoding used for response text field e.g "base64". Leave out this field
      * if the text field is HTTP decoded (decompressed & unchunked), than
      * trans-coded from its original character set into UTF-8.
      *
-     * @var string
-     *
      * @Serializer\Type("string")
      */
-    private $encoding;
+    private ?string $encoding = null;
 
     public function getCompression(): int
     {

src/Cookie.php

@@ -21,47 +21,37 @@ final class Cookie
     /**
      * The name of the cookie.
      *
-     * @var string
-     *
      * @Serializer\Type("string")
      */
-    private $cookie;
+    private string $cookie;
 
     /**
      * The path pertaining to the cookie.
      *
-     * @var string
-     *
      * @Serializer\Type("string")
      */
-    private $path;
+    private string $path;
 
     /**
      * The host of the cookie.
      *
-     * @var string
-     *
      * @Serializer\Type("string")
      */
-    private $domain;
+    private string $domain;
 
     /**
      * Set to true if the cookie is HTTP only, false otherwise.
      *
-     * @var bool
-     *
      * @Serializer\Type("boolean")
      */
-    private $httpOnly;
+    private ?bool $httpOnly = null;
 
     /**
      * True if the cookie was transmitted over ssl, false otherwise.
      *
-     * @var bool
-     *
      * @Serializer\Type("boolean")
      */
-    private $secure;
+    private ?bool $secure = null;
 
     public function getCookie(): string
     {

src/Creator.php

@@ -19,11 +19,9 @@ class Creator
     use NameTrait;
 
     /**
-     * @var string
-     *
      * @Serializer\Type("string")
      */
-    private $version;
+    private string $version;
 
     public function getVersion(): string
     {

src/Entry.php

@@ -22,78 +22,60 @@ final class Entry
      * Reference to the parent page. Leave out this field if the application
      * does not support grouping by pages.
      *
-     * @var string
-     *
      * @Serializer\Type("string")
      */
-    private $pageref;
+    private string $pageref;
 
     /**
      * Total elapsed time of the request in milliseconds. This is the sum of all
      * timings available in the timings object (i.e. not including -1 values).
      *
-     * @var float
-     *
      * @Serializer\Type("float")
      */
-    private $time;
+    private float $time;
 
     /**
      * Detailed info about the request.
      *
-     * @var Request
-     *
      * @Serializer\Type("Deviantintegral\Har\Request")
      */
-    private $request;
+    private Request $request;
 
     /**
      * Detailed info about the response.
      *
-     * @var Response
-     *
      * @Serializer\Type("Deviantintegral\Har\Response")
      */
-    private $response;
+    private Response $response;
 
     /**
      * Info about cache usage.
      *
-     * @var Cache
-     *
      * @Serializer\Type("Deviantintegral\Har\Cache")
      */
-    private $cache;
+    private Cache $cache;
 
     /**
-     * @var Timings
-     *
      * @Serializer\Type("Deviantintegral\Har\Timings")
      */
-    private $timings;
+    private Timings $timings;
 
     /**
-     * @var string
-     *
      * @Serializer\Type("string")
      */
-    private $serverIPAddress;
+    private string $serverIPAddress;
 
     /**
-     * @var string
-     *
      * @Serializer\Type("string")
      */
-    private $connection;
+    private string $connection;
 
     /**
      * Detailed info about the request.
      *
-     * @var Initiator
-     *
      * @Serializer\Type("Deviantintegral\Har\Initiator")
      */
-    private $_initiator;
+    private ?Initiator $_initiator = null;
 
     public function getPageref(): string
     {

src/Har.php

@@ -9,11 +9,9 @@
 final class Har
 {
     /**
-     * @var Log
-     *
      * @Serializer\Type("Deviantintegral\Har\Log")
      */
-    private $log;
+    private Log $log;
 
     public function getLog(): Log
     {

src/Initiator.php

@@ -10,8 +10,6 @@
 final class Initiator
 {
     /**
-     * @var string
-     *
      * @Serializer\Type("string")
      *
      * parser:
@@ -24,25 +22,21 @@ final class Initiator
      *   Some other process or action, such as navigating to a page via a link
      *   or entering a URL in the address bar.
      */
-    private $type;
+    private ?string $type = null;
 
     /**
      * URL of the entry that initiated this request.
      *
-     * @var \Psr\Http\Message\UriInterface
-     *
      * @Serializer\Type("Psr\Http\Message\UriInterface")
      */
-    private $url;
+    private ?\Psr\Http\Message\UriInterface $url = null;
 
     /**
      * Line number that initiated this request.
      *
-     * @var int
-     *
      * @Serializer\Type("integer")
      */
-    private $lineNumber;
+    private ?int $lineNumber = null;
 
     public function getType(): ?string
     {

src/Log.php

@@ -22,39 +22,33 @@ class Log
     public const ISO_8601_MICROSECONDS = 'Y-m-d\TH:i:s.uT';
 
     /**
-     * @var string
-     *
      * @Serializer\Type("string")
      */
-    private $version;
+    private string $version;
 
     /**
-     * @var Creator
-     *
      * @Serializer\Type("Deviantintegral\Har\Creator")
      */
-    private $creator;
+    private Creator $creator;
 
     /**
-     * @var Browser
-     *
      * @Serializer\Type("Deviantintegral\Har\Browser")
      */
-    private $browser;
+    private Browser $browser;
 
     /**
      * @var Page[]
      *
      * @Serializer\Type("array<Deviantintegral\Har\Page>")
      */
-    private $pages;
+    private array $pages;
 
     /**
      * @var Entry[]
      *
      * @Serializer\Type("array<integer, Deviantintegral\Har\Entry>")
      */
-    private $entries;
+    private array $entries;
 
     public function getVersion(): string
     {

src/Page.php

@@ -14,25 +14,19 @@ final class Page
     use StartedDateTimeTrait;
 
     /**
-     * @var string
-     *
      * @Serializer\Type("string")
      */
-    private $id;
+    private string $id;
 
     /**
-     * @var string
-     *
      * @Serializer\Type("string")
      */
-    private $title;
+    private string $title;
 
     /**
-     * @var PageTimings
-     *
      * @Serializer\Type("Deviantintegral\Har\PageTimings")
      */
-    private $pageTimings;
+    private PageTimings $pageTimings;
 
     public function getId(): string
     {