Made token refresh code resilient to missing id_token

[etraut-openai]

This PR does the following:

1. Changes try_refresh_token to handle the case where the endpoint returns a response without an id_token. The OpenID spec indicates that this field is optional and clients should not assume it's present.
2. Changes the attempt_stream_responses to propagate token refresh errors rather than silently ignoring them.
3. Fixes a typo in a couple of error messages (unrelated to the above, but something I noticed in passing) - "reconnect" should be spelled without a hyphen.

This PR does not implement the additional suggestion from @pakrym-oai that we should sign out when receiving refresh_token_expired from the refresh endpoint. Leaving this as a follow-on because I'm undecided on whether this should be implemented in try_refresh_token or its callers.

[etraut-openai]

@codex review

[chatgpt-codex-connector]

Codex Review: Something went wrong. Try again later by commenting “@codex review”.

An unknown error occurred

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

• Open a pull request for review
• Mark a draft as ready
• Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

[chatgpt-codex-connector]

Codex Review: Something went wrong. Try again later by commenting “@codex review”.

An unknown error occurred

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

• Open a pull request for review
• Mark a draft as ready
• Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

[pakrym-oai]

Approved too early.

[pakrym-oai]

Will this throw for API keys? We have this in refresh_token

    pub async fn refresh_token(&self) -> Result<String, std::io::Error> {
        tracing::info!("Refreshing token");

        let token_data = self
            .get_current_token_data()
            .ok_or(std::io::Error::other("Token data is not available."))?;

[pakrym-oai]

Yep:

[pakrym-oai]

We should also prefix the message.

[etraut-openai]

Good catch!