[TASK] Add native type declarations for atRuleName()

Part of #811

Author: oliverklee

CHANGELOG.md

@@ -32,7 +32,7 @@ Please also have a look at our
 - Make all non-private properties `@internal` (#886)
 - Use more native type declarations and strict mode
   (#641, #772, #774, #778, #804, #841, #873, #875, #891, #922, #923, #933, #958,
-  #964, #967, #1000, #1044, #1134, #1136, #1137, #1139, #1140, #1141)
+  #964, #967, #1000, #1044, #1134, #1136, #1137, #1139, #1140, #1141, #1145)
 - Add visibility to all class/interface constants (#469)
 
 ### Deprecated

src/CSSList/AtRuleBlockList.php

@@ -13,7 +13,7 @@
 class AtRuleBlockList extends CSSBlockList implements AtRule
 {
     /**
-     * @var string
+     * @var non-empty-string
      */
     private $type;
 
@@ -23,6 +23,7 @@ class AtRuleBlockList extends CSSBlockList implements AtRule
     private $arguments;
 
     /**
+     * @param non-empty-string $type
      * @param int<0, max> $lineNumber
      */
     public function __construct(string $type, string $arguments = '', int $lineNumber = 0)
@@ -32,6 +33,9 @@ public function __construct(string $type, string $arguments = '', int $lineNumbe
         $this->arguments = $arguments;
     }
 
+    /**
+     * @return non-empty-string
+     */
     public function atRuleName(): string
     {
         return $this->type;

src/CSSList/KeyFrame.php

@@ -7,10 +7,14 @@
 use Sabberworm\CSS\OutputFormat;
 use Sabberworm\CSS\Property\AtRule;
 
+/**
+ * Note: When constructing an instance of this class, `setVendorKeyFrame()` must be called directly after construction
+ * for the instance to be valid.
+ */
 class KeyFrame extends CSSList implements AtRule
 {
     /**
-     * @var string|null
+     * @var non-empty-string
      */
     private $vendorKeyFrame;
 
@@ -20,15 +24,15 @@ class KeyFrame extends CSSList implements AtRule
     private $animationName;
 
     /**
-     * @param string $vendorKeyFrame
+     * @param non-empty-string $vendorKeyFrame
      */
     public function setVendorKeyFrame($vendorKeyFrame): void
     {
         $this->vendorKeyFrame = $vendorKeyFrame;
     }
 
     /**
-     * @return string|null
+     * @return non-empty-string
      */
     public function getVendorKeyFrame()
     {
@@ -74,9 +78,9 @@ public function isRootList(): bool
     }
 
     /**
-     * @return string|null
+     * @return non-empty-string
      */
-    public function atRuleName()
+    public function atRuleName(): string
     {
         return $this->vendorKeyFrame;
     }

src/Property/AtRule.php

@@ -20,9 +20,9 @@ interface AtRule extends Renderable, Commentable
     public const BLOCK_RULES = 'media/document/supports/region-style/font-feature-values';
 
     /**
-     * @return string|null
+     * @return non-empty-string
      */
-    public function atRuleName();
+    public function atRuleName(): string;
 
     /**
      * @return string|null

src/Property/CSSNamespace.php

@@ -101,7 +101,7 @@ public function setPrefix($prefix): void
     }
 
     /**
-     * @return string
+     * @return non-empty-string
      */
     public function atRuleName(): string
     {

src/Property/Charset.php

@@ -84,6 +84,9 @@ public function render(OutputFormat $outputFormat): string
         return "{$outputFormat->comments($this)}@charset {$this->charset->render($outputFormat)};";
     }
 
+    /**
+     * @return non-empty-string
+     */
     public function atRuleName(): string
     {
         return 'charset';

src/Property/Import.php

@@ -86,6 +86,9 @@ public function render(OutputFormat $outputFormat): string
             . ($this->mediaQuery === null ? '' : ' ' . $this->mediaQuery) . ';';
     }
 
+    /**
+     * @return non-empty-string
+     */
     public function atRuleName(): string
     {
         return 'import';

src/RuleSet/AtRuleSet.php

@@ -16,7 +16,7 @@
 class AtRuleSet extends RuleSet implements AtRule
 {
     /**
-     * @var string
+     * @var non-empty-string
      */
     private $type;
 
@@ -26,7 +26,7 @@ class AtRuleSet extends RuleSet implements AtRule
     private $arguments;
 
     /**
-     * @param string $type
+     * @param non-empty-string $type
      * @param string $arguments
      * @param int<0, max> $lineNumber
      */
@@ -38,9 +38,9 @@ public function __construct($type, $arguments = '', int $lineNumber = 0)
     }
 
     /**
-     * @return string
+     * @return non-empty-string
      */
-    public function atRuleName()
+    public function atRuleName(): string
     {
         return $this->type;
     }