Skip to content

Conversation

@devalog
Copy link
Collaborator

@devalog devalog commented Oct 16, 2025

No description provided.

@github-actions
Copy link
Contributor

@devalog devalog changed the title (api definitions) Document OpenAPI OAuth (api definitions) Document OpenAPI OAuth and clarify parameters Oct 16, 2025
@github-actions
Copy link
Contributor

```

## Multiple security schemes
## OAuth client credentials
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

📝 [vale] reported by reviewdog 🐶
[FernStyles.Headings] 'OAuth client credentials' should use sentence-style capitalization.

<ParamField path="expires-in" type="string" required={false}>
The property name for the expires in property in the response.
<ParamField path="response-properties.expires-in" type="string" required={false}>
The response field name for token expiration time in seconds (e.g., `"expiresIn"`, `"expires_in"`). When present, the SDK automatically refreshes tokens before expiration.
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

⚠️ [vale] reported by reviewdog 🐶
[FernStyles.Current] Avoid time-relative terms like 'present' that become outdated

<ParamField path="type" type="literal<'client-credentials'>" required={true}>
The OAuth flow type. Currently only `"client-credentials"` is supported.
<ParamField path="type" type="'client-credentials'" required={true}>
The OAuth 2.0 grant type. Currently only `"client-credentials"` is supported.
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

⚠️ [vale] reported by reviewdog 🐶
[FernStyles.Current] Avoid time-relative terms like 'Currently' that become outdated

@@ -1,6 +1,6 @@
Configuration for the token refresh endpoint.
Specifies the endpoint that exchanges a refresh token for a new access token. When configured, the SDK automatically uses this endpoint to renew expired tokens without re-sending credentials. If not configured, the SDK will re-authenticate using `get-token` when tokens expire.
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

⚠️ [vale] reported by reviewdog 🐶
[FernStyles.Current] Avoid time-relative terms like 'new' that become outdated

<ParamField path="access-token" type="string" required={false}>
The property name for the access token in the response.
<ParamField path="response-properties.access-token" type="string" required={false}>
The response field name for the new access token (e.g., `"accessToken"`, `"access_token"`).
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

⚠️ [vale] reported by reviewdog 🐶
[FernStyles.Current] Avoid time-relative terms like 'new' that become outdated

<ParamField path="expires-in" type="string" required={false}>
The property name for the expires in property in the response.
<ParamField path="response-properties.expires-in" type="string" required={false}>
The response field name for the new token's expiration time in seconds (e.g., `"expiresIn"`, `"expires_in"`).
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

⚠️ [vale] reported by reviewdog 🐶
[FernStyles.Current] Avoid time-relative terms like 'new' that become outdated

<ParamField path="refresh-token" type="string" required={false}>
The property name for the refresh token in the response.
<ParamField path="response-properties.refresh-token" type="string" required={false}>
The response field name if your API issues a new refresh token with each refresh (token rotation).
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

⚠️ [vale] reported by reviewdog 🐶
[FernStyles.Current] Avoid time-relative terms like 'new' that become outdated

@github-actions
Copy link
Contributor

@devalog devalog merged commit 1f9bb23 into main Oct 29, 2025
2 checks passed
@devalog devalog deleted the devalog/oauth branch October 29, 2025 15:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants