@_mustachio/openauth 0.10.0 → 0.11.0

package/src/domain/token.ts

@@ -47,7 +47,8 @@ import {
   sha256,
   utf8,
 } from "./crypto"
-import { signAccessToken } from "./jwt"
+import { buildIdTokenClaims, shouldIssueIdToken } from "./id-token"
+import { signAccessToken, signIdToken } from "./jwt"
 import { validatePkce } from "./pkce"
 
 /**
@@ -88,6 +89,13 @@ export type TokenAuthCodeRequest = {
   clientSecret?: string
   /** Required if the original `/authorize` request had a `code_challenge`. */
   codeVerifier?: string
+  /**
+   * RFC 9449 §6 — the JWK thumbprint of the presented DPoP proof. Set by
+   * the HTTP layer after verifying the `DPoP:` header against this
+   * request's actual method + URI. The domain binds the issued access
+   * token to this thumbprint via `cnf.jkt`.
+   */
+  dpopJkt?: string
 }
 
 export type ExchangeCodeDeps = {
@@ -102,6 +110,11 @@ export type ExchangeCodeDeps = {
   newRefreshToken?: () => string
   /** Test override. */
   newRefreshFamily?: () => string
+  /**
+   * Host-supplied vendor scope → claim-names map. Forwarded to `mintTokens`
+   * for id_token + /userinfo scope-gating. See `IdPOptions.customScopeClaims`.
+   */
+  customScopeClaims?: Record<string, ReadonlyArray<string>>
 }
 
 export async function exchangeCode(
@@ -141,6 +154,17 @@ export async function exchangeCode(
   const authResult = await verifyClientCredentials(client, req.clientSecret)
   if (authResult) return err(authResult)
 
+  // RFC 9449 §5.2 — if the client is configured to require DPoP, a
+  // bearer-only request (no DPoP header → no `dpopJkt` threaded in) is
+  // refused with `invalid_dpop_proof` before any token is minted.
+  if (client.dpopRequired && req.dpopJkt === undefined) {
+    return err(
+      authError.invalidDpopProof(
+        `client "${client.id}" requires DPoP-bound tokens`,
+      ),
+    )
+  }
+
   // 4. Redirect URI binding.
   if (payload.appRedirectUri !== req.redirectUri) {
     return err(authError.invalidGrant("redirect_uri mismatch with auth code"))
@@ -198,11 +222,14 @@ export async function exchangeCode(
     }
   }
 
-  // 8. Mint access + refresh.
+  // 8. Mint access + refresh (+ id_token if `openid` scope was granted).
   const minted = await mintTokens({
     tenant,
     claim,
-    payload,
+    payload: {
+      ...payload,
+      ...(req.dpopJkt !== undefined ? { dpopJkt: req.dpopJkt } : {}),
+    },
     deps,
     family: (deps.newRefreshFamily ?? randomId)(),
   })
@@ -219,7 +246,30 @@ export async function mintTokens(args: {
   payload: Pick<
     CodePayload,
     "tenantId" | "clientId" | "methodId" | "methodKind" | "scopes" | "audience"
-  >
+  > & {
+    /**
+     * Present on `authorization_code` and `refresh_token` grants where an
+     * end-user actually authenticated. Absent on `client_credentials`
+     * (no end-user) — and absent means no `id_token` is emitted even if
+     * the requested scopes nominally include `openid`.
+     */
+    authTime?: number
+    /** RP's OIDC `nonce` from `/authorize`, when present. */
+    appNonce?: string
+    /**
+     * DPoP key thumbprint (RFC 9449 §6.1). When present, the access
+     * token's `cnf.jkt` claim is set and `token_type` flips to `"DPoP"`;
+     * the saved refresh-token payload's `dpopJkt` is set so refresh
+     * rotation re-enforces sender constraint.
+     */
+    dpopJkt?: string
+    /**
+     * OIDC Core §5.5 — RP-requested claims from `/authorize`. Carried
+     * into the id_token and forward across refresh rotations (§12) so
+     * later /userinfo calls keep returning the requested fields.
+     */
+    claimsRequest?: import("../types/authorization").ClaimsRequest
+  }
   family: string
   /**
    * `client_credentials` grants (RFC 6749 §4.4.3) and other paths where a
@@ -236,6 +286,11 @@ export async function mintTokens(args: {
     issuerUrl: string
     clock: () => number
     newRefreshToken?: () => string
+    /**
+     * Host-supplied vendor scope → claim-names map merged into the
+     * id_token + /userinfo scope-gating. See `IdPOptions.customScopeClaims`.
+     */
+    customScopeClaims?: Record<string, ReadonlyArray<string>>
   }
 }): Promise<Result<TokenResponse, AuthError>> {
   const { tenant, claim, payload, deps, family, skipRefresh } = args
@@ -248,12 +303,22 @@ export async function mintTokens(args: {
       ? tenant.config.refreshTtl * 1000
       : DEFAULT_REFRESH_TTL_MS
   const now = deps.clock()
-  const subjectId = await deriveSubjectId(claim)
+  // OIDC Core §8.1 — `sectorIdentifier` from the receiving client drives
+  // pairwise vs public subject derivation. Look it up off the tenant
+  // config (already loaded by the grant flow). Absent = public.
+  const receivingClient = tenant.config.clients.find(
+    (c) => c.id === payload.clientId,
+  )
+  const subjectId = await deriveSubjectId(
+    claim,
+    receivingClient?.sectorIdentifier,
+  )
 
   const keyRes = await deps.keyStore.currentSigningKey()
   if (isErr(keyRes)) return err(keyRes.error)
   const signingKey = keyRes.value
 
+  const userinfoClaimNames = Object.keys(payload.claimsRequest?.userinfo ?? {})
   const claims: AccessTokenClaims = {
     iss: deps.issuerUrl,
     sub: subjectId,
@@ -265,6 +330,9 @@ export async function mintTokens(args: {
     mkind: payload.methodKind,
     scope: payload.scopes.join(" "),
     claim,
+    ...(payload.authTime !== undefined ? { auth_time: payload.authTime } : {}),
+    ...(payload.dpopJkt !== undefined ? { cnf: { jkt: payload.dpopJkt } } : {}),
+    ...(userinfoClaimNames.length > 0 ? { uic: userinfoClaimNames } : {}),
   }
 
   let accessToken: string
@@ -295,6 +363,14 @@ export async function mintTokens(args: {
       family,
       methodId: payload.methodId,
       methodKind: payload.methodKind,
+      // Carry the original end-user auth time forward. Refresh-grant
+      // reissue uses this verbatim so `id_token.auth_time` is stable per
+      // OIDC Core §12 (refresh does not re-authenticate the user).
+      authTime: payload.authTime ?? Math.floor(now / 1000),
+      ...(payload.dpopJkt !== undefined ? { dpopJkt: payload.dpopJkt } : {}),
+      ...(payload.claimsRequest !== undefined
+        ? { claimsRequest: payload.claimsRequest }
+        : {}),
       issuedAt: now,
       expiresAt: now + refreshTtl,
     }
@@ -302,6 +378,41 @@ export async function mintTokens(args: {
     if (isErr(saved)) return err(saved.error)
   }
 
+  // OIDC id_token issuance — only when the grant carried an end-user
+  // (`authTime` present) AND `openid` scope was granted. Client-credentials
+  // never satisfies the first condition; refresh + code do.
+  let idToken: string | undefined
+  if (payload.authTime !== undefined && shouldIssueIdToken(payload.scopes)) {
+    const idClaims = await buildIdTokenClaims({
+      issuerUrl: deps.issuerUrl,
+      audience: payload.clientId,
+      subjectId,
+      claim,
+      scopes: payload.scopes,
+      authTime: payload.authTime,
+      ...(payload.appNonce !== undefined ? { appNonce: payload.appNonce } : {}),
+      now,
+      methodKind: payload.methodKind,
+      accessToken,
+      ...(payload.claimsRequest !== undefined
+        ? { claimsRequest: payload.claimsRequest }
+        : {}),
+      ...(deps.customScopeClaims !== undefined
+        ? { customScopeClaims: deps.customScopeClaims }
+        : {}),
+    })
+    try {
+      idToken = await signIdToken(
+        idClaims,
+        signingKey.privateKeyRef as Parameters<typeof signIdToken>[1],
+        signingKey.alg,
+        signingKey.kid,
+      )
+    } catch (e) {
+      return err(authError.serverError("id_token sign failed", e))
+    }
+  }
+
   await safeAudit(deps, {
     kind: "token_issued",
     tenantId: payload.tenantId,
@@ -310,14 +421,17 @@ export async function mintTokens(args: {
     methodKind: payload.methodKind,
     subjectId,
     refreshTokenIdHash: refresh ? await hashTokenForAudit(refresh) : "",
+    ...(idToken !== undefined ? { idTokenIssued: true } : {}),
+    ...(payload.dpopJkt !== undefined ? { dpopBound: true } : {}),
     timestamp: now,
   })
 
   return ok({
     access_token: accessToken,
-    token_type: "Bearer",
+    token_type: payload.dpopJkt !== undefined ? "DPoP" : "Bearer",
     expires_in: Math.floor(accessTtl / 1000),
     ...(refresh !== undefined ? { refresh_token: refresh } : {}),
+    ...(idToken !== undefined ? { id_token: idToken } : {}),
     scope: payload.scopes.join(" "),
   })
 }
@@ -336,11 +450,23 @@ export async function hashClientSecret(plain: string): Promise<string> {
 /**
  * Derive a stable subject id from the issued `SubjectClaim`. Hash inputs
  * are canonicalized so reordered `properties` keys hash identically.
+ *
+ * OIDC Core §8.1 — when `sectorIdentifier` is supplied, the derivation
+ * mixes it in so the resulting `sub` is **pairwise**: identical across
+ * clients sharing that sector, distinct across sectors. Absent =
+ * **public** subject (same `sub` for every RP).
  */
-async function deriveSubjectId(claim: SubjectClaim): Promise<string> {
+async function deriveSubjectId(
+  claim: SubjectClaim,
+  sectorIdentifier?: string,
+): Promise<string> {
   const c = claim as { type: string; properties: Record<string, unknown> }
   const ordered = canonicalize(c.properties)
-  return base64url.encode(await sha256(`${c.type}\0${ordered}`)).slice(0, 22)
+  const seed =
+    sectorIdentifier !== undefined
+      ? `${sectorIdentifier}\0${c.type}\0${ordered}`
+      : `${c.type}\0${ordered}`
+  return base64url.encode(await sha256(seed)).slice(0, 22)
 }
 
 function canonicalize(value: unknown): string {

package/src/domain/userinfo.ts

@@ -1,7 +1,23 @@
 /**
- * `/userinfo` endpoint logic (OIDC Core §5.3). Verifies the bearer access
- * token and returns the inlined `SubjectClaim` claims.
+ * `/userinfo` endpoint logic (OIDC Core §5.3). Verifies the presented
+ * access token, optionally enforces a DPoP proof binding (RFC 9449 §7),
+ * and returns the inlined `SubjectClaim` claims, scope-gated per OIDC
+ * Core §5.4.
+ *
+ * Profile / email / phone / address claims appear in the response only
+ * when (a) the access token's `scope` includes the granting scope and
+ * (b) the value exists on `SubjectClaim.properties`. Other properties
+ * (host-specific subject fields like `userId`, `roles`, …) are returned
+ * under `properties` and not subject to OIDC scope gating — they're
+ * outside §5.4's universe.
+ *
+ * DPoP at the resource server (RFC 9449 §7):
+ *  - If the access token has a `cnf.jkt` claim, the request MUST carry a
+ *    matching `DPoP:` proof whose `ath` equals SHA-256(access_token).
+ *  - If no `cnf.jkt`, the access token is plain Bearer and any DPoP
+ *    proof is silently ignored at this layer (host can still enforce).
  */
+import type { TokenStore } from "../ports/token-store"
 import type { KeyStore } from "../ports/key-store"
 import { authError, type AuthError } from "../types/error"
 import type { Result } from "../types/result"
@@ -9,13 +25,16 @@ import { err, isErr, ok } from "../types/result"
 import type { SubjectClaim } from "../types/subject"
 import type { AccessTokenClaims } from "../types/token"
 
+import { computeAth, verifyDpopProof } from "./dpop"
+import { pickScopedClaims } from "./id-token"
 import { verifyAccessToken } from "./jwt"
+import type { ScopedProfileClaims } from "../types/token"
 
-export type UserinfoResponse = {
+export type UserinfoResponse = ScopedProfileClaims & {
   sub: string
   /** Subject type discriminator (e.g. `"user"` / `"admin"`). */
   subject_type: string
-  /** Inlined claim properties. */
+  /** Inlined claim properties, host-specific and not gated by OIDC scope. */
   properties: Record<string, unknown>
   /** Scope granted to this access token. */
   scope?: string
@@ -23,11 +42,34 @@ export type UserinfoResponse = {
 
 export type UserinfoDeps = {
   keyStore: KeyStore
+  /** Required when DPoP-bound tokens are accepted. */
+  tokenStore?: TokenStore
   issuerUrl?: string
+  /**
+   * Host-supplied vendor scope → claim-names map. Merged into the
+   * scope-gated /userinfo response just as it is for id_token. See
+   * `IdPOptions.customScopeClaims`.
+   */
+  customScopeClaims?: Record<string, ReadonlyArray<string>>
+}
+
+export type UserinfoInput = {
+  /** Access token from `Authorization: Bearer ...` or `Authorization: DPoP ...`. */
+  accessToken: string
+  /** `Authorization`-scheme as presented. Defaults to `"Bearer"`. */
+  scheme?: "Bearer" | "DPoP"
+  /** Raw `DPoP:` header from the request, if any. */
+  dpopProof?: string
+  /** Canonical request URI for DPoP `htu` check (scheme + host + path). */
+  htu?: string
+  /** Request method for DPoP `htm` check, uppercase. */
+  htm?: string
+  /** Wall clock seconds for DPoP `iat` window. */
+  nowSec?: number
 }
 
 export async function userinfo(
-  bearerToken: string,
+  input: UserinfoInput,
   deps: UserinfoDeps,
 ): Promise<Result<UserinfoResponse, AuthError>> {
   const keysRes = await deps.keyStore.signingKeys()
@@ -35,22 +77,95 @@ export async function userinfo(
 
   let claims: AccessTokenClaims
   try {
-    claims = await verifyAccessToken(bearerToken, keysRes.value, {
+    claims = await verifyAccessToken(input.accessToken, keysRes.value, {
       ...(deps.issuerUrl ? { issuer: deps.issuerUrl } : {}),
     })
   } catch {
     return err(authError.invalidGrant("access token invalid or expired"))
   }
 
+  // RFC 9449 §7 — DPoP-bound tokens require a matching proof at the RS.
+  const boundJkt = claims.cnf?.jkt
+  if (boundJkt !== undefined) {
+    if (input.scheme !== "DPoP") {
+      return err(
+        authError.invalidDpopProof(
+          'access token is DPoP-bound; Authorization scheme must be "DPoP"',
+        ),
+      )
+    }
+    if (!input.dpopProof) {
+      return err(
+        authError.invalidDpopProof(
+          "access token is DPoP-bound; request is missing a DPoP proof",
+        ),
+      )
+    }
+    if (!input.htu || !input.htm || input.nowSec === undefined) {
+      return err(
+        authError.invalidDpopProof(
+          "userinfo dpop verification requires htu, htm, and nowSec",
+        ),
+      )
+    }
+    if (!deps.tokenStore) {
+      return err(
+        authError.invalidDpopProof(
+          "userinfo dpop verification requires a token-store (jti replay protection)",
+        ),
+      )
+    }
+    const expectedAth = await computeAth(input.accessToken)
+    const dpopRes = await verifyDpopProof(
+      {
+        proofJwt: input.dpopProof,
+        htu: input.htu,
+        htm: input.htm,
+        nowSec: input.nowSec,
+        expectedAth,
+      },
+      { tokenStore: deps.tokenStore },
+    )
+    if (isErr(dpopRes)) return err(dpopRes.error)
+    if (dpopRes.value.jkt !== boundJkt) {
+      return err(
+        authError.invalidDpopProof(
+          "dpop proof key does not match access token cnf.jkt",
+        ),
+      )
+    }
+  } else if (input.scheme === "DPoP") {
+    // Token isn't bound but caller used the DPoP scheme — reject so a
+    // confused-deputy can't accidentally rely on RS-side enforcement.
+    return err(
+      authError.invalidGrant(
+        "access token is not DPoP-bound; use Authorization: Bearer",
+      ),
+    )
+  }
+
   const claim = claims.claim as SubjectClaim & {
     type: string
     properties: Record<string, unknown>
   }
 
+  const scopes = claims.scope ? claims.scope.split(" ").filter(Boolean) : []
+  // OIDC Core §5.5 — claims the RP requested via `claims=...userinfo:...`
+  // bypass scope gating. Names were captured into `uic` at mint time so
+  // the resource server doesn't need to re-resolve the original /authorize
+  // request.
+  const scopedClaims = pickScopedClaims(
+    claim,
+    scopes,
+    claims.uic ?? [],
+    deps.customScopeClaims ?? {},
+  )
+
   return ok({
     sub: claims.sub,
     subject_type: claim.type,
     properties: claim.properties,
     scope: claims.scope,
+    ...scopedClaims,
   })
 }

package/src/http/context.ts

@@ -18,6 +18,7 @@ import type { AuthError } from "../types/error"
 import type {
   ExchangeAudience,
   PersistUpstreamTokens,
+  RegisterClient,
   RenderPicker,
   SuccessMapInput,
 } from "../types/idp"
@@ -51,12 +52,15 @@ export type HttpDeps = {
   persistUpstreamTokens?: PersistUpstreamTokens
   exchangeAudience?: ExchangeAudience
   renderPicker?: RenderPicker
+  registerClient?: RegisterClient
   /** Builds `TenantContext.request.custom` for every request. See `IdPOptions.buildCustomContext`. */
   buildCustomContext?: (
     req: Request,
   ) => Record<string, unknown> | Promise<Record<string, unknown>>
   clock: () => number
   cookieDefaults: CookieDefaults
+  /** See `IdPOptions.customScopeClaims`. */
+  customScopeClaims?: Record<string, ReadonlyArray<string>>
 }
 
 /** Per-request variables populated by middleware. */

package/src/http/handlers/authorize.ts

@@ -17,7 +17,10 @@ import { isErr } from "../../types/result"
 import { startAuthorize } from "../../domain/authorize"
 import { asTenantId } from "../../types/tenant"
 import { authError } from "../../types/error"
-import type { AuthorizationRequest } from "../../types/authorization"
+import type {
+  AuthorizationRequest,
+  ClaimsRequest,
+} from "../../types/authorization"
 import type { PickerContext, PickerMethod } from "../../types/picker"
 import { renderPicker as renderDefaultPicker } from "../../ui/picker"
 
@@ -28,7 +31,10 @@ import {
   authorizeRedirectErrorResponse,
   isNonRecoverable,
 } from "../errors"
-import { authorizeQuerySchema } from "../schemas/authorize"
+import {
+  authorizeQuerySchema,
+  authorizeRequestUriQuerySchema,
+} from "../schemas/authorize"
 
 export function makeAuthorizeHandler(deps: HttpDeps) {
   return async (c: HttpContext): Promise<Response> => {
@@ -41,7 +47,51 @@ export function makeAuthorizeHandler(deps: HttpDeps) {
 
     const url = new URL(c.req.url)
     const raw = Object.fromEntries(url.searchParams.entries())
-    const parsed = authorizeQuerySchema.safeParse(raw)
+
+    // RFC 9126 §4: rehydrate a PAR record when `request_uri` is present.
+    // The user-agent URL carries only `client_id` + `request_uri`; the
+    // stored params drive the rest of the request through the standard
+    // parser below.
+    let workingRaw: Record<string, string> = raw
+    if (raw.request_uri !== undefined) {
+      const parParsed = authorizeRequestUriQuerySchema.safeParse(raw)
+      if (!parParsed.success) {
+        return authorizeDirectErrorResponse(
+          authError.invalidRequest(
+            parParsed.error.issues[0]?.message ?? "invalid request",
+            parParsed.error.issues[0]?.path[0]?.toString() ?? "request",
+          ),
+        )
+      }
+      if (!deps.sessionStore.consumePar) {
+        return authorizeDirectErrorResponse(
+          authError.invalidRequest(
+            "session adapter does not support PAR",
+            "request_uri",
+          ),
+        )
+      }
+      const par = await deps.sessionStore.consumePar(parParsed.data.request_uri)
+      if (isErr(par)) {
+        return authorizeDirectErrorResponse(
+          authError.invalidRequest(
+            "unknown or expired request_uri",
+            "request_uri",
+          ),
+        )
+      }
+      if (par.value.clientId !== parParsed.data.client_id) {
+        return authorizeDirectErrorResponse(
+          authError.invalidRequest(
+            "request_uri belongs to a different client",
+            "request_uri",
+          ),
+        )
+      }
+      workingRaw = par.value.params
+    }
+
+    const parsed = authorizeQuerySchema.safeParse(workingRaw)
     if (!parsed.success) {
       const first = parsed.error.issues[0]
       const field = first?.path[0]?.toString() ?? "request"
@@ -51,6 +101,20 @@ export function makeAuthorizeHandler(deps: HttpDeps) {
     }
     const q = parsed.data
 
+    // RFC 9126 §2 per-client PAR enforcement: refuse direct `/authorize`
+    // when the client requires PAR and `request_uri` was absent.
+    if (raw.request_uri === undefined) {
+      const client = tenant.config.clients.find((c) => c.id === q.client_id)
+      if (client?.requirePushedAuthorizationRequests) {
+        return authorizeDirectErrorResponse(
+          authError.invalidRequest(
+            "this client requires Pushed Authorization Requests (RFC 9126)",
+            "request_uri",
+          ),
+        )
+      }
+    }
+
     if (q.response_type !== "code") {
       // OAuth 2.1: only `code` is supported. Implicit (`token`) is removed.
       return authorizeDirectErrorResponse({
@@ -60,6 +124,17 @@ export function makeAuthorizeHandler(deps: HttpDeps) {
       })
     }
 
+    let claimsRequest: ClaimsRequest | undefined
+    if (q.claims !== undefined) {
+      const parsedClaims = parseClaimsParameter(q.claims)
+      if (!parsedClaims.ok) {
+        return authorizeDirectErrorResponse(
+          authError.invalidRequest(parsedClaims.error, "claims"),
+        )
+      }
+      claimsRequest = parsedClaims.value
+    }
+
     const request: AuthorizationRequest = {
       tenantId: asTenantId(tenant.id),
       clientId: q.client_id,
@@ -78,6 +153,7 @@ export function makeAuthorizeHandler(deps: HttpDeps) {
       ...(q.prompt !== undefined ? { prompt: q.prompt } : {}),
       ...(q.ui_locales !== undefined ? { uiLocales: q.ui_locales } : {}),
       ...(q.nonce !== undefined ? { nonce: q.nonce } : {}),
+      ...(claimsRequest !== undefined ? { claimsRequest } : {}),
     }
 
     const result = await startAuthorize(
@@ -189,6 +265,71 @@ export function makeAuthorizeHandler(deps: HttpDeps) {
   }
 }
 
+/**
+ * Parse + validate the OIDC Core §5.5 `claims` parameter. Accepts a
+ * JSON string; verifies it's an object whose `userinfo` / `id_token`
+ * values, if present, are objects mapping claim names to `null` or
+ * `{essential?, value?, values?}` entries. Returns a discriminated
+ * `{ok, value}` shape so the caller can map failure → `invalid_request`
+ * without dragging the AuthError import into this helper.
+ */
+function parseClaimsParameter(
+  raw: string,
+): { ok: true; value: ClaimsRequest } | { ok: false; error: string } {
+  let parsed: unknown
+  try {
+    parsed = JSON.parse(raw)
+  } catch (e) {
+    return {
+      ok: false,
+      error: `claims parameter is not valid JSON: ${e instanceof Error ? e.message : String(e)}`,
+    }
+  }
+  if (!parsed || typeof parsed !== "object" || Array.isArray(parsed)) {
+    return { ok: false, error: "claims parameter must be a JSON object" }
+  }
+  const obj = parsed as Record<string, unknown>
+  const result: ClaimsRequest = {}
+  for (const section of ["userinfo", "id_token"] as const) {
+    const v = obj[section]
+    if (v === undefined) continue
+    if (!v || typeof v !== "object" || Array.isArray(v)) {
+      return {
+        ok: false,
+        error: `claims.${section} must be a JSON object`,
+      }
+    }
+    const sectionMap: Record<
+      string,
+      { essential?: boolean; value?: unknown; values?: unknown[] } | null
+    > = {}
+    for (const [name, entry] of Object.entries(v)) {
+      if (entry === null) {
+        sectionMap[name] = null
+        continue
+      }
+      if (typeof entry !== "object" || Array.isArray(entry)) {
+        return {
+          ok: false,
+          error: `claims.${section}.${name} must be null or a JSON object`,
+        }
+      }
+      const e = entry as Record<string, unknown>
+      const normalized: {
+        essential?: boolean
+        value?: unknown
+        values?: unknown[]
+      } = {}
+      if (typeof e.essential === "boolean") normalized.essential = e.essential
+      if ("value" in e) normalized.value = e.value
+      if (Array.isArray(e.values)) normalized.values = e.values
+      sectionMap[name] = normalized
+    }
+    result[section] = sectionMap
+  }
+  return { ok: true, value: result }
+}
+
 function clearFlowCookie() {
   return {
     name: "idp.flow",