Fix level 3 type errors

Author: theodorejb

composer.json

@@ -1,6 +1,6 @@
 {
     "name": "devtheorem/php-handlebars",
-    "description": "An extremely fast PHP implementation of Handlebars.",
+    "description": "A fast, spec-compliant PHP implementation of Handlebars.",
     "keywords": [
         "Handlebars",
         "PHP",
@@ -23,9 +23,9 @@
         "php": ">=8.2"
     },
     "require-dev": {
-        "friendsofphp/php-cs-fixer": "^3.74",
+        "friendsofphp/php-cs-fixer": "^3.75",
         "jbboehr/handlebars-spec": "dev-master",
-        "phpstan/phpstan": "^2.1.11",
+        "phpstan/phpstan": "^2.1.12",
         "phpunit/phpunit": "^11.5"
     },
     "autoload": {

composer.lock

@@ -1,5 +1,5 @@
 parameters:
-	level: 2
+	level: 3
 	paths:
 		- src
 		- tests

phpstan.neon

@@ -216,7 +216,7 @@ protected static function getVariableName(Context $context, array $var, ?array $
     /**
      * Return compiled PHP code for a handlebars token
      *
-     * @param array<string,array|bool> $info parsed information
+     * @param array{bool, array, array, string} $info parsed information
      */
     protected static function compileToken(Context $context, array $info): string
     {
@@ -240,7 +240,7 @@ protected static function compileToken(Context $context, array $info): string
                 return static::compileLookup($context, $vars, $raw);
             }
             if ($vars[0][0] === 'log') {
-                return static::compileLog($context, $vars, $raw);
+                return static::compileLog($context, $vars);
             }
         }
 
@@ -326,7 +326,7 @@ protected static function blockCustomHelper(Context $context, array $vars, bool
      * @param array $vars parsed arguments list
      * @param string|null $match should also match to this operator
      */
-    protected static function blockEnd(Context $context, array &$vars, ?string $match = null): string
+    protected static function blockEnd(Context $context, array $vars, ?string $match = null): string
     {
         $pop = $context->stack[count($context->stack) - 1];
 
@@ -475,9 +475,8 @@ protected static function doElse(Context $context, array $vars)
      * Return compiled PHP code for a handlebars log token
      *
      * @param array<bool|int|string|array> $vars parsed arguments list
-     * @param bool $raw is this {{{ token or not
      */
-    protected static function compileLog(Context $context, array &$vars, bool $raw): string
+    protected static function compileLog(Context $context, array &$vars): string
     {
         array_shift($vars);
         $v = static::getVariableNames($context, $vars);

src/Compiler.php

@@ -10,7 +10,7 @@ public function __construct(
         public readonly \Closure $fn,
         public readonly \Closure $inverse,
         public readonly int $blockParams,
-        public array|string|null &$scope,
+        public mixed &$scope,
         public array &$data,
     ) {}
 

src/HelperOptions.php

@@ -144,9 +144,9 @@ protected static function getExpression(string $v, Context $context, int|string
      *
      * @param array<string> $token preg_match results
      *
-     * @return array<bool|int|array> Return parsed result
+     * @return array{bool,array} Return parsed result
      */
-    public static function parse(array &$token, Context $context): array
+    public static function parse(array $token, Context $context): array
     {
         $vars = static::analyze($token[Token::POS_INNERTAG], $context);
         if ($token[Token::POS_OP] === '>') {
@@ -171,25 +171,26 @@ public static function parse(array &$token, Context $context): array
     /**
      * Get partial name from "foo" or [foo] or \'foo\'
      *
-     * @param array<bool|int|array> $vars parsed token
+     * @param array<bool|int|string> $vars parsed token
      * @param int $pos position of partial name
      *
      * @return array<string>|null Return one element partial name array
      */
-    public static function getPartialName(array &$vars, int $pos = 0): ?array
+    public static function getPartialName(array $vars, int $pos = 0): ?array
     {
-        if (!isset($vars[$pos])) {
+        if (!isset($vars[$pos]) || preg_match(SafeString::IS_SUBEXP_SEARCH, $vars[$pos])) {
             return null;
         }
-        return preg_match(SafeString::IS_SUBEXP_SEARCH, $vars[$pos]) ? null : [preg_replace('/^("(.+)")|(\\[(.+)\\])|(\\\\\'(.+)\\\\\')$/', '$2$4$6', $vars[$pos])];
+        assert(is_string($vars[$pos]));
+        return [preg_replace('/^("(.+)")|(\\[(.+)\\])|(\\\\\'(.+)\\\\\')$/', '$2$4$6', $vars[$pos])];
     }
 
     /**
      * Parse a subexpression then return parsed result.
      *
      * @param string $expression the full string of a sub expression
      *
-     * @return array<bool|int|array> Return parsed result
+     * @return array<string|int|array> Return parsed result
      */
     public static function subexpression(string $expression, Context $context): array
     {
@@ -332,7 +333,7 @@ protected static function detectQuote(string $string): ?array
      *
      * @param string $token preg_match results
      *
-     * @return array<bool|int|array> Return parsed result
+     * @return list<string> Return parsed result
      */
     protected static function analyze(string $token, Context $context): array
     {