Skip to content

Conversation

Copy link

Copilot AI commented Nov 23, 2025

Addresses feedback from PR #20 regarding duplicate JSDoc comment blocks in libs/Jwt.ts.

Changes

  • Removed duplicate JSDoc block at lines 74-81 for the verify method
  • Retained the complete JSDoc block (lines 82-93) which includes @throws and @example tags

The verify method now has a single, complete documentation block:

/**
 * Online verification that performs a Keycloak server `userinfo` call
 * to make sure the token is still valid on the server-side. If the
 * call completes successfully the token is accepted and returned as a
 * `Token` wrapper for callers to inspect claims.
 *
 * @param accessToken - The JWT string to verify via Keycloak server
 * @returns A Promise resolving to a `Token` instance when userinfo succeeds
 * @throws {AxiosError} When the `userinfo` endpoint returns a non-2xx response
 * @example
 * const token = await jwt.verify('ey...')
 */
async verify(accessToken: string): Promise<Token> {

💬 We'd love your input! Share your thoughts on Copilot coding agent in our 2 minute survey.

Co-authored-by: jkyberneees <4096860+jkyberneees@users.noreply.github.com>
@jkyberneees jkyberneees marked this pull request as ready for review November 23, 2025 14:33
@jkyberneees jkyberneees merged commit 86ea476 into v5.1 Nov 23, 2025
@jkyberneees jkyberneees deleted the copilot/sub-pr-20-another-one branch November 23, 2025 14:33
Copilot AI changed the title [WIP] Fix unit and integration tests for Keycloak functionality Remove duplicate JSDoc block in Jwt.verify method Nov 23, 2025
Copilot AI requested a review from jkyberneees November 23, 2025 14:33
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.

2 participants