best-practices/no-unused-parameter wrongly thrown on used parameter

[giorgiopogliani]

🐞 Describe the Bug

The notice is best-practices/no-unused-parameter wrongly thrown on a used parameter parameter. The parameter is used inside the array access square brackets.

🔄 Steps to Reproduce

1. mago lint the code example
2. the $key variable is actually used but reported as unused

⚙️ Configuration (mago.toml)

empty

📜 Command Output

note[best-practices/no-unused-parameter]: Parameter `$key` is never used.
   ┌─ test.php:9:25
   │
 9 │ ╭     public function set(string $key, string $value, ?string $default = null): void
   │                           ^^^^^^^^^^^ Parameter `$key` is declared here.
10 │ │     {
11 │ │         $this->data[$key] = $value ?? $default;
12 │ │     }
   │ ╰─────'
   │
   = This parameter is declared but not used within the method.
   = Help: Consider prefixing the parameter with an underscore (`_`) to indicate that it is intentionally unused, or remove it if it is not needed.

note: found 1 issues: 1 note(s)
 = 1 issues contain auto-fix suggestions

📂 PHP Code Sample (If Applicable)

<?php

declare(strict_types=1);

class Test
{
    private array $data = [];

    public function set(string $key, string $value, ?string $default = null): void
    {
        $this->data[$key] = $value ?? $default;
    }
}

🖥️ Operating System

macOS

📦 How did you install Mago?

Cargo (cargo install mago)

📝 Additional Context

No response