TypeError: OpenAI::client(): Argument #1 ($apiKey) must be of type string, null given..........

Hi @psf_

I have created MR!3 for the above issue, Please review it once.

Thank you !!

Hi, thx by the patch.

I have some notes:

- In "src/OpenAIClientWrapper.php" you were erased all comments, it's are required by code styling that we have comments.
- In the protected method getApiToken() we can't send any status message, because that will spawn the user with messages. This method is called so many times.

ankitv18 → changed the visibility of the branch 1.1.x to hidden.

@psf_ please review the MR!5 I guess we could use the ternary operator to handle the null value passed as token.

In this line:
$token = empty($token) ? $this->getApiToken() : '';

"$this->getApiToken()" can return a NULL value if the user don't configure the module.

I change your commit with:

public function getClient(string $token = ''): Client {
    if (empty($token)) {
      $token = $this->getApiToken();
    }
    $token = !$token ? '' : $token;
    return \OpenAI::client($token);
  }

But I don't tested it.

I can implement the same logic with combination or ternary and null coalescing operator.

I see it OK, but I don't tested it.

I'm so busy, when I get free I will try it, or apply it if anyother tested it.

Thx so much