{"prompt":"You are comparing an AI-suggested code fix with actual changes made in a commit.\n\nYour task: Determine if the commit contains the essence of the AI fix and provide a confidence level.\n\n## AI-Suggested Fix\n\n**What it fixes:** @forgerock/davinci-suites:e2e-ci--src/basic.test.ts\n\n**Description:** fix(e2e): update signoff check to use waitForURL after SDK upgrade\n\n**Reasoning:** The PR upgraded `@forgerock/javascript-sdk` from `4.7.0` to `4.9.0` by aligning the version to the new pnpm catalog entry (`4.9.0`).\n\nIn SDK 4.9.0, `OAuth2Client.endSession()` was changed so that when a `logoutRedirectUri` option is present, the sign-off request is made via `window.location.assign(signoffUrl)` rather than `fetch(signoffUrl)`. The `davinci-app` passes `logoutRedirectUri: window.location.origin + '/'` to `FRUser.logout()`, which triggers this branch.\n\nThe original test used `page.\n\n**Suggested Changes (diff):**\n```diff\ndiff --git a/e2e/davinci-suites/src/basic.test.ts b/e2e/davinci-suites/src/basic.test.ts\nindex 1da585d..f567bc1 100644\n--- a/e2e/davinci-suites/src/basic.test.ts\n+++ b/e2e/davinci-suites/src/basic.test.ts\n@@ -40,14 +40,9 @@ test('Test happy paths on test page', async ({ page }) => {\n return true;\n }\n });\n- const signoff = page.waitForResponse((response) => {\n- if (response.url().includes('/signoff') && response.status() === 302) {\n- return true;\n- }\n- });\n await logoutButton.click();\n await revokeCall;\n- await signoff;\n+ await page.waitForURL('http://localhost:5829/');\n await expect(page.getByText('Username/Password Form')).toBeVisible();\n });\n test('ensure query params passed to start are sent off in authorize call', async ({ page }) => {\n\n```\n\n## Actual Commit Changes\n\n**Commit Diff:**\n```diff\ndiff --git a/e2e/davinci-app/main.ts b/e2e/davinci-app/main.ts\nindex b5e3e2da8c..c9787f6e48 100644\n--- a/e2e/davinci-app/main.ts\n+++ b/e2e/davinci-app/main.ts\n@@ -6,8 +6,9 @@\n */\n import './style.css';\n \n-import { Config, FRUser, TokenManager } from '@forgerock/javascript-sdk';\n import { davinci } from '@forgerock/davinci-client';\n+import { oidc } from '@forgerock/oidc-client';\n+import type { OidcConfig } from '@forgerock/oidc-client/types';\n import type {\n CustomLogger,\n DaVinciConfig,\n@@ -85,6 +86,11 @@ const urlParams = new URLSearchParams(window.location.search);\n \n (async () => {\n const davinciClient: DavinciClient = await davinci({ config, logger, requestMiddleware });\n+ const oidcResult = await oidc({ config: config as OidcConfig });\n+ if ('error' in oidcResult) {\n+ throw new Error(`Failed to initialize oidc client: ${oidcResult.error}`);\n+ }\n+ const oidcClient = oidcResult;\n const protectApi = protect({ envId: '02fb4743-189a-4bc7-9d6c-a919edfe6447' });\n const continueToken = urlParams.get('continueToken');\n const formEl = document.getElementById('form') as HTMLFormElement;\n@@ -98,10 +104,6 @@ const urlParams = new URLSearchParams(window.location.search);\n \n if (continueToken) {\n resumed = await davinciClient.resume({ continueToken });\n- } else {\n- // the current davinci-config has a slightly\n- // different middleware type than the old legacy config\n- await Config.setAsync(config as any);\n }\n \n function renderComplete() {\n@@ -140,25 +142,26 @@ const urlParams = new URLSearchParams(window.location.search);\n \n const tokenBtn = document.getElementById('tokensButton') as HTMLButtonElement;\n tokenBtn.addEventListener('click', async () => {\n- tokens = await TokenManager.getTokens({ query: { code, state } });\n+ tokens = await oidcClient.token.exchange(code, state);\n \n console.log(tokens);\n \n+ const accessTokenValue = tokens && 'accessToken' in tokens ? tokens.accessToken : '';\n const tokenPreEl = document.getElementById('accessTokenContainer') as HTMLPreElement;\n tokenPreEl.innerHTML = `\n ${tokens?.accessToken}\n+ >${accessTokenValue}\n `;\n });\n \n const loginBtn = document.getElementById('logoutButton') as HTMLButtonElement;\n loginBtn.addEventListener('click', async () => {\n- await FRUser.logout({ logoutRedirectUri: `${window.location.origin}/` });\n+ await oidcClient.user.logout();\n \n- //window.location.reload();\n+ window.location.reload();\n });\n }\n \ndiff --git a/e2e/davinci-app/package.json b/e2e/davinci-app/package.json\nindex 4dfc5cee73..af986f5398 100644\n--- a/e2e/davinci-app/package.json\n+++ b/e2e/davinci-app/package.json\n@@ -12,7 +12,7 @@\n },\n \"dependencies\": {\n \"@forgerock/davinci-client\": \"workspace:*\",\n- \"@forgerock/javascript-sdk\": \"4.7.0\",\n+ \"@forgerock/oidc-client\": \"workspace:*\",\n \"@forgerock/protect\": \"workspace:*\",\n \"@forgerock/sdk-logger\": \"workspace:*\"\n },\ndiff --git a/e2e/davinci-app/tsconfig.app.json b/e2e/davinci-app/tsconfig.app.json\nindex 9027cd9d88..93f398976d 100644\n--- a/e2e/davinci-app/tsconfig.app.json\n+++ b/e2e/davinci-app/tsconfig.app.json\n@@ -18,6 +18,9 @@\n {\n \"path\": \"../../packages/protect/tsconfig.lib.json\"\n },\n+ {\n+ \"path\": \"../../packages/oidc-client/tsconfig.lib.json\"\n+ },\n {\n \"path\": \"../../packages/davinci-client/tsconfig.lib.json\"\n }\ndiff --git a/e2e/davinci-suites/src/basic.test.ts b/e2e/davinci-suites/src/basic.test.ts\nindex 1da585de47..43002ae465 100644\n--- a/e2e/davinci-suites/src/basic.test.ts\n+++ b/e2e/davinci-suites/src/basic.test.ts\n@@ -40,14 +40,8 @@ test('Test happy paths on test page', async ({ page }) => {\n return true;\n }\n });\n- const signoff = page.waitForResponse((response) => {\n- if (response.url().includes('/signoff') && response.status() === 302) {\n- return true;\n- }\n- });\n await logoutButton.click();\n await revokeCall;\n- await signoff;\n await expect(page.getByText('Username/Password Form')).toBeVisible();\n });\n test('ensure query params passed to start are sent off in authorize call', async ({ page }) => {\ndiff --git a/packages/davinci-client/api-report/davinci-client.api.md b/packages/davinci-client/api-report/davinci-client.api.md\nindex 289c8a5276..b96fc65b97 100644\n--- a/packages/davinci-client/api-report/davinci-client.api.md\n+++ b/packages/davinci-client/api-report/davinci-client.api.md\n@@ -178,7 +178,7 @@ export interface CollectorErrors {\n }\n \n // @public (undocumented)\n-export type Collectors = FlowCollector | PasswordCollector | TextCollector | SingleSelectCollector | IdpCollector | SubmitCollector | ActionCollector<'ActionCollector'> | SingleValueCollector<'SingleValueCollector'> | MultiSelectCollector | DeviceAuthenticationCollector | DeviceRegistrationCollector | PhoneNumberCollector | ReadOnlyCollector | ValidatedTextCollector | ProtectCollector | PollingCollector | FidoRegistrationCollector | FidoAuthenticationCollector | QrCodeCollector | AgreementCollector | UnknownCollector;\n+export type Collectors = FlowCollector | PasswordCollector | TextCollector | SingleSelectCollector | IdpCollector | SubmitCollector | ActionCollector<'ActionCollector'> | SingleValueCollector<'SingleValueCollector'> | MultiSelectCollector | DeviceAuthenticationCollector | DeviceRegistrationCollector | PhoneNumberCollector | PhoneNumberExtensionCollector | ReadOnlyCollector | ValidatedTextCollector | ProtectCollector | PollingCollector | FidoRegistrationCollector | FidoAuthenticationCollector | QrCodeCollector | AgreementCollector | UnknownCollector;\n \n // @public\n export type CollectorValueType = T extends {\n@@ -212,7 +212,7 @@ export type CollectorValueType = T extends {\n } ? string[] : string | string[] | PhoneNumberInputValue | FidoRegistrationInputValue | FidoAuthenticationInputValue;\n \n // @public (undocumented)\n-export type ComplexValueFields = DeviceAuthenticationField | DeviceRegistrationField | PhoneNumberField | FidoRegistrationField | FidoAuthenticationField | PollingField;\n+export type ComplexValueFields = DeviceAuthenticationField | DeviceRegistrationField | PhoneNumberField | PhoneNumberExtensionField | FidoRegistrationField | FidoAuthenticationField | PollingField;\n \n // @public (undocumented)\n export interface ContinueNode {\n@@ -1035,7 +1035,7 @@ export type InferNoValueCollectorType = T exten\n export type InferSingleValueCollectorType = T extends 'TextCollector' ? TextCollector : T extends 'SingleSelectCollector' ? SingleSelectCollector : T extends 'ValidatedTextCollector' ? ValidatedTextCollector : T extends 'PasswordCollector' ? PasswordCollector : SingleValueCollectorWithValue<'SingleValueCollector'> | SingleValueCollectorNoValue<'SingleValueCollector'>;\n \n // @public (undocumented)\n-export type InferValueObjectCollectorType = T extends 'DeviceAuthenticationCollector' ? DeviceAuthenticationCollector : T extends 'DeviceRegistrationCollector' ? DeviceRegistrationCollector : T extends 'PhoneNumberCollector' ? PhoneNumberCollector : ObjectOptionsCollectorWithObjectValue<'ObjectValueCollector'> | ObjectOptionsCollectorWithStringValue<'ObjectValueCollector'>;\n+export type InferValueObjectCollectorType = T extends 'DeviceAuthenticationCollector' ? DeviceAuthenticationCollector : T extends 'DeviceRegistrationCollector' ? DeviceRegistrationCollector : T extends 'PhoneNumberCollector' ? PhoneNumberCollector : T extends 'PhoneNumberExtensionCollector' ? PhoneNumberExtensionCollector : ObjectOptionsCollectorWithObjectValue<'ObjectValueCollector'> | ObjectOptionsCollectorWithStringValue<'ObjectValueCollector'>;\n \n // @public (undocumented)\n export type InitFlow = () => Promise;\n@@ -1170,8 +1170,8 @@ value: Record;\n }, string>;\n \n // @public\n-export const nodeCollectorReducer: Reducer<(TextCollector | SingleSelectCollector | ValidatedTextCollector | PasswordCollector | MultiSelectCollector | DeviceAuthenticationCollector | DeviceRegistrationCollector | PhoneNumberCollector | IdpCollector | SubmitCollector | FlowCollector | QrCodeCollectorBase | AgreementCollector | ReadOnlyCollector | UnknownCollector | ProtectCollector | FidoRegistrationCollector | FidoAuthenticationCollector | PollingCollector | ActionCollector<\"ActionCollector\"> | SingleValueCollector<\"SingleValueCollector\">)[]> & {\n- getInitialState: () => (TextCollector | SingleSelectCollector | ValidatedTextCollector | PasswordCollector | MultiSelectCollector | DeviceAuthenticationCollector | DeviceRegistrationCollector | PhoneNumberCollector | IdpCollector | SubmitCollector | FlowCollector | QrCodeCollectorBase | AgreementCollector | ReadOnlyCollector | UnknownCollector | ProtectCollector | FidoRegistrationCollector | FidoAuthenticationCollector | PollingCollector | ActionCollector<\"ActionCollector\"> | SingleValueCollector<\"SingleValueCollector\">)[];\n+export const nodeCollectorReducer: Reducer<(TextCollector | SingleSelectCollector | ValidatedTextCollector | PasswordCollector | MultiSelectCollector | PhoneNumberExtensionCollector | DeviceAuthenticationCollector | DeviceRegistrationCollector | PhoneNumberCollector | IdpCollector | SubmitCollector | FlowCollector | QrCodeCollectorBase | AgreementCollector | ReadOnlyCollector | UnknownCollector | ProtectCollector | FidoRegistrationCollector | FidoAuthenticationCollector | PollingCollector | ActionCollector<\"ActionCollector\"> | SingleValueCollector<\"SingleValueCollector\">)[]> & {\n+ getInitialState: () => (TextCollector | SingleSelectCollector | ValidatedTextCollector | PasswordCollector | MultiSelectCollector | PhoneNumberExtensionCollector | DeviceAuthenticationCollector | DeviceRegistrationCollector | PhoneNumberCollector | IdpCollector | SubmitCollector | FlowCollector | QrCodeCollectorBase | AgreementCollector | ReadOnlyCollector | UnknownCollector | ProtectCollector | FidoRegistrationCollector | FidoAuthenticationCollector | PollingCollector | ActionCollector<\"ActionCollector\"> | SingleValueCollector<\"SingleValueCollector\">)[];\n };\n \n // @public (undocumented)\n@@ -1283,10 +1283,10 @@ export type ObjectValueAutoCollectorTypes = 'ObjectValueAutoCollector' | 'FidoRe\n export type ObjectValueCollector = ObjectOptionsCollectorWithObjectValue | ObjectOptionsCollectorWithStringValue | ObjectValueCollectorWithObjectValue;\n \n // @public (undocumented)\n-export type ObjectValueCollectors = DeviceAuthenticationCollector | DeviceRegistrationCollector | PhoneNumberCollector | ObjectOptionsCollectorWithObjectValue<'ObjectSelectCollector'> | ObjectOptionsCollectorWithStringValue<'ObjectSelectCollector'>;\n+export type ObjectValueCollectors = DeviceAuthenticationCollector | DeviceRegistrationCollector | PhoneNumberCollector | PhoneNumberExtensionCollector | ObjectOptionsCollectorWithObjectValue<'ObjectSelectCollector'> | ObjectOptionsCollectorWithStringValue<'ObjectSelectCollector'>;\n \n // @public\n-export type ObjectValueCollectorTypes = 'DeviceAuthenticationCollector' | 'DeviceRegistrationCollector' | 'PhoneNumberCollector' | 'ObjectOptionsCollector' | 'ObjectValueCollector' | 'ObjectSelectCollector';\n+export type ObjectValueCollectorTypes = 'DeviceAuthenticationCollector' | 'DeviceRegistrationCollector' | 'PhoneNumberCollector' | 'PhoneNumberExtensionCollector' | 'ObjectOptionsCollector' | 'ObjectValueCollector' | 'ObjectSelectCollector';\n \n // @public (undocumented)\n export interface ObjectValueCollectorWithObjectValue, OV = Record> {\n@@ -1328,13 +1328,68 @@ export type PasswordCollector = SingleValueCollectorNoValue<'PasswordCollector'>\n // @public (undocumented)\n export type PhoneNumberCollector = ObjectValueCollectorWithObjectValue<'PhoneNumberCollector', PhoneNumberInputValue, PhoneNumberOutputValue>;\n \n+// @public (undocumented)\n+export interface PhoneNumberExtensionCollector {\n+ // (undocumented)\n+ category: 'ObjectValueCollector';\n+ // (undocumented)\n+ error: string | null;\n+ // (undocumented)\n+ id: string;\n+ // (undocumented)\n+ input: {\n+ key: string;\n+ value: PhoneNumberExtensionInputValue;\n+ type: string;\n+ validation: (ValidationRequired | ValidationPhoneNumber)[] | null;\n+ };\n+ // (undocumented)\n+ name: string;\n+ // (undocumented)\n+ output: {\n+ key: string;\n+ label: string;\n+ type: string;\n+ extensionLabel: string;\n+ value: PhoneNumberExtensionOutputValue;\n+ };\n+ // (undocumented)\n+ type: 'PhoneNumberExtensionCollector';\n+}\n+\n+// @public (undocumented)\n+export type PhoneNumberExtensionField = PhoneNumberField & {\n+ showExtension: boolean;\n+ extensionLabel: string;\n+};\n+\n+// @public (undocumented)\n+export interface PhoneNumberExtensionInputValue {\n+ // (undocumented)\n+ countryCode: string;\n+ // (undocumented)\n+ extension: string;\n+ // (undocumented)\n+ phoneNumber: string;\n+}\n+\n+// @public (undocumented)\n+export interface PhoneNumberExtensionOutputValue {\n+ // (undocumented)\n+ countryCode?: string;\n+ // (undocumented)\n+ extension?: string;\n+ // (undocumented)\n+ phoneNumber?: string;\n+}\n+\n // @public (undocumented)\n export type PhoneNumberField = {\n type: 'PHONE_NUMBER';\n key: string;\n label: string;\n- defaultCountryCode: string | null;\n required: boolean;\n+ defaultCountryCode: string | null;\n validatePhoneNumber: boolean;\n };\n \n@@ -1724,7 +1779,7 @@ export type UnknownField = Record;\n // @public (undocumented)\n export const updateCollectorValues: ActionCreatorWithPayload< {\n id: string;\n-value: string | string[] | PhoneNumberInputValue | FidoRegistrationInputValue | FidoAuthenticationInputValue;\n+value: string | string[] | PhoneNumberInputValue | PhoneNumberExtensionInputValue | FidoRegistrationInputValue | FidoAuthenticationInputValue;\n index?: number;\n }, string>;\n \ndiff --git a/packages/davinci-client/api-report/davinci-client.types.api.md b/packages/davinci-client/api-report/davinci-client.types.api.md\nindex 05f38634dc..78e36b50a9 100644\n--- a/packages/davinci-client/api-report/davinci-client.types.api.md\n+++ b/packages/davinci-client/api-report/davinci-client.types.api.md\n@@ -178,7 +178,7 @@ export interface CollectorErrors {\n }\n \n // @public (undocumented)\n-export type Collectors = FlowCollector | PasswordCollector | TextCollector | SingleSelectCollector | IdpCollector | SubmitCollector | ActionCollector<'ActionCollector'> | SingleValueCollector<'SingleValueCollector'> | MultiSelectCollector | DeviceAuthenticationCollector | DeviceRegistrationCollector | PhoneNumberCollector | ReadOnlyCollector | ValidatedTextCollector | ProtectCollector | PollingCollector | FidoRegistrationCollector | FidoAuthenticationCollector | QrCodeCollector | AgreementCollector | UnknownCollector;\n+export type Collectors = FlowCollector | PasswordCollector | TextCollector | SingleSelectCollector | IdpCollector | SubmitCollector | ActionCollector<'ActionCollector'> | SingleValueCollector<'SingleValueCollector'> | MultiSelectCollector | DeviceAuthenticationCollector | DeviceRegistrationCollector | PhoneNumberCollector | PhoneNumberExtensionCollector | ReadOnlyCollector | ValidatedTextCollector | ProtectCollector | PollingCollector | FidoRegistrationCollector | FidoAuthenticationCollector | QrCodeCollector | AgreementCollector | UnknownCollector;\n \n // @public\n export type CollectorValueType = T extends {\n@@ -212,7 +212,7 @@ export type CollectorValueType = T extends {\n } ? string[] : string | string[] | PhoneNumberInputValue | FidoRegistrationInputValue | FidoAuthenticationInputValue;\n \n // @public (undocumented)\n-export type ComplexValueFields = DeviceAuthenticationField | DeviceRegistrationField | PhoneNumberField | FidoRegistrationField | FidoAuthenticationField | PollingField;\n+export type ComplexValueFields = DeviceAuthenticationField | DeviceRegistrationField | PhoneNumberField | PhoneNumberExtensionField | FidoRegistrationField | FidoAuthenticationField | PollingField;\n \n // @public (undocumented)\n export interface ContinueNode {\n@@ -1032,7 +1032,7 @@ export type InferNoValueCollectorType = T exten\n export type InferSingleValueCollectorType = T extends 'TextCollector' ? TextCollector : T extends 'SingleSelectCollector' ? SingleSelectCollector : T extends 'ValidatedTextCollector' ? ValidatedTextCollector : T extends 'PasswordCollector' ? PasswordCollector : SingleValueCollectorWithValue<'SingleValueCollector'> | SingleValueCollectorNoValue<'SingleValueCollector'>;\n \n // @public (undocumented)\n-export type InferValueObjectCollectorType = T extends 'DeviceAuthenticationCollector' ? DeviceAuthenticationCollector : T extends 'DeviceRegistrationCollector' ? DeviceRegistrationCollector : T extends 'PhoneNumberCollector' ? PhoneNumberCollector : ObjectOptionsCollectorWithObjectValue<'ObjectValueCollector'> | ObjectOptionsCollectorWithStringValue<'ObjectValueCollector'>;\n+export type InferValueObjectCollectorType = T extends 'DeviceAuthenticationCollector' ? DeviceAuthenticationCollector : T extends 'DeviceRegistrationCollector' ? DeviceRegistrationCollector : T extends 'PhoneNumberCollector' ? PhoneNumberCollector : T extends 'PhoneNumberExtensionCollector' ? PhoneNumberExtensionCollector : ObjectOptionsCollectorWithObjectValue<'ObjectValueCollector'> | ObjectOptionsCollectorWithStringValue<'ObjectValueCollector'>;\n \n // @public (undocumented)\n export type InitFlow = () => Promise;\n@@ -1167,8 +1167,8 @@ value: Record;\n }, string>;\n \n // @public\n-export const nodeCollectorReducer: Reducer<(TextCollector | SingleSelectCollector | ValidatedTextCollector | PasswordCollector | MultiSelectCollector | DeviceAuthenticationCollector | DeviceRegistrationCollector | PhoneNumberCollector | IdpCollector | SubmitCollector | FlowCollector | QrCodeCollectorBase | AgreementCollector | ReadOnlyCollector | UnknownCollector | ProtectCollector | FidoRegistrationCollector | FidoAuthenticationCollector | PollingCollector | ActionCollector<\"ActionCollector\"> | SingleValueCollector<\"SingleValueCollector\">)[]> & {\n- getInitialState: () => (TextCollector | SingleSelectCollector | ValidatedTextCollector | PasswordCollector | MultiSelectCollector | DeviceAuthenticationCollector | DeviceRegistrationCollector | PhoneNumberCollector | IdpCollector | SubmitCollector | FlowCollector | QrCodeCollectorBase | AgreementCollector | ReadOnlyCollector | UnknownCollector | ProtectCollector | FidoRegistrationCollector | FidoAuthenticationCollector | PollingCollector | ActionCollector<\"ActionCollector\"> | SingleValueCollector<\"SingleValueCollector\">)[];\n+export const nodeCollectorReducer: Reducer<(TextCollector | SingleSelectCollector | ValidatedTextCollector | PasswordCollector | MultiSelectCollector | PhoneNumberExtensionCollector | DeviceAuthenticationCollector | DeviceRegistrationCollector | PhoneNumberCollector | IdpCollector | SubmitCollector | FlowCollector | QrCodeCollectorBase | AgreementCollector | ReadOnlyCollector | UnknownCollector | ProtectCollector | FidoRegistrationCollector | FidoAuthenticationCollector | PollingCollector | ActionCollector<\"ActionCollector\"> | SingleValueCollector<\"SingleValueCollector\">)[]> & {\n+ getInitialState: () => (TextCollector | SingleSelectCollector | ValidatedTextCollector | PasswordCollector | MultiSelectCollector | PhoneNumberExtensionCollector | DeviceAuthenticationCollector | DeviceRegistrationCollector | PhoneNumberCollector | IdpCollector | SubmitCollector | FlowCollector | QrCodeCollectorBase | AgreementCollector | ReadOnlyCollector | UnknownCollector | ProtectCollector | FidoRegistrationCollector | FidoAuthenticationCollector | PollingCollector | ActionCollector<\"ActionCollector\"> | SingleValueCollector<\"SingleValueCollector\">)[];\n };\n \n // @public (undocumented)\n@@ -1280,10 +1280,10 @@ export type ObjectValueAutoCollectorTypes = 'ObjectValueAutoCollector' | 'FidoRe\n export type ObjectValueCollector = ObjectOptionsCollectorWithObjectValue | ObjectOptionsCollectorWithStringValue | ObjectValueCollectorWithObjectValue;\n \n // @public (undocumented)\n-export type ObjectValueCollectors = DeviceAuthenticationCollector | DeviceRegistrationCollector | PhoneNumberCollector | ObjectOptionsCollectorWithObjectValue<'ObjectSelectCollector'> | ObjectOptionsCollectorWithStringValue<'ObjectSelectCollector'>;\n+export type ObjectValueCollectors = DeviceAuthenticationCollector | DeviceRegistrationCollector | PhoneNumberCollector | PhoneNumberExtensionCollector | ObjectOptionsCollectorWithObjectValue<'ObjectSelectCollector'> | ObjectOptionsCollectorWithStringValue<'ObjectSelectCollector'>;\n \n // @public\n-export type ObjectValueCollectorTypes = 'DeviceAuthenticationCollector' | 'DeviceRegistrationCollector' | 'PhoneNumberCollector' | 'ObjectOptionsCollector' | 'ObjectValueCollector' | 'ObjectSelectCollector';\n+export type ObjectValueCollectorTypes = 'DeviceAuthenticationCollector' | 'DeviceRegistrationCollector' | 'PhoneNumberCollector' | 'PhoneNumberExtensionCollector' | 'ObjectOptionsCollector' | 'ObjectValueCollector' | 'ObjectSelectCollector';\n \n // @public (undocumented)\n export interface ObjectValueCollectorWithObjectValue, OV = Record> {\n@@ -1325,13 +1325,68 @@ export type PasswordCollector = SingleValueCollectorNoValue<'PasswordCollector'>\n // @public (undocumented)\n export type PhoneNumberCollector = ObjectValueCollectorWithObjectValue<'PhoneNumberCollector', PhoneNumberInputValue, PhoneNumberOutputValue>;\n \n+// @public (undocumented)\n+export interface PhoneNumberExtensionCollector {\n+ // (undocumented)\n+ category: 'ObjectValueCollector';\n+ // (undocumented)\n+ error: string | null;\n+ // (undocumented)\n+ id: string;\n+ // (undocumented)\n+ input: {\n+ key: string;\n+ value: PhoneNumberExtensionInputValue;\n+ type: string;\n+ validation: (ValidationRequired | ValidationPhoneNumber)[] | null;\n+ };\n+ // (undocumented)\n+ name: string;\n+ // (undocumented)\n+ output: {\n+ key: string;\n+ label: string;\n+ type: string;\n+ extensionLabel: string;\n+ value: PhoneNumberExtensionOutputValue;\n+ };\n+ // (undocumented)\n+ type: 'PhoneNumberExtensionCollector';\n+}\n+\n+// @public (undocumented)\n+export type PhoneNumberExtensionField = PhoneNumberField & {\n+ showExtension: boolean;\n+ extensionLabel: string;\n+};\n+\n+// @public (undocumented)\n+export interface PhoneNumberExtensionInputValue {\n+ // (undocumented)\n+ countryCode: string;\n+ // (undocumented)\n+ extension: string;\n+ // (undocumented)\n+ phoneNumber: string;\n+}\n+\n+// @public (undocumented)\n+export interface PhoneNumberExtensionOutputValue {\n+ // (undocumented)\n+ countryCode?: string;\n+ // (undocumented)\n+ extension?: string;\n+ // (undocumented)\n+ phoneNumber?: string;\n+}\n+\n // @public (undocumented)\n export type PhoneNumberField = {\n type: 'PHONE_NUMBER';\n key: string;\n label: string;\n- defaultCountryCode: string | null;\n required: boolean;\n+ defaultCountryCode: string | null;\n validatePhoneNumber: boolean;\n };\n \n@@ -1721,7 +1776,7 @@ export type UnknownField = Record;\n // @public (undocumented)\n export const updateCollectorValues: ActionCreatorWithPayload< {\n id: string;\n-value: string | string[] | PhoneNumberInputValue | FidoRegistrationInputValue | FidoAuthenticationInputValue;\n+value: string | string[] | PhoneNumberInputValue | PhoneNumberExtensionInputValue | FidoRegistrationInputValue | FidoAuthenticationInputValue;\n index?: number;\n }, string>;\n \ndiff --git a/pnpm-lock.yaml b/pnpm-lock.yaml\nindex f0d61d2af8..52f5f2f574 100644\n--- a/pnpm-lock.yaml\n+++ b/pnpm-lock.yaml\n@@ -315,9 +315,9 @@ importers:\n '@forgerock/davinci-client':\n specifier: workspace:*\n version: link:../../packages/davinci-client\n- '@forgerock/javascript-sdk':\n- specifier: 4.7.0\n- version: 4.7.0\n+ '@forgerock/oidc-client':\n+ specifier: workspace:*\n+ version: link:../../packages/oidc-client\n '@forgerock/protect':\n specifier: workspace:*\n version: link:../../packages/protect\n@@ -1848,9 +1848,6 @@ packages:\n '@noble/hashes':\n optional: true\n \n- '@forgerock/javascript-sdk@4.7.0':\n- resolution: {integrity: sha512-0wpy2/ii9F9yKI3r+huqQtp6bVAeajf2+Llq25dvkfxQX19FKKi9KPPMF7JTVti6heYHyo36lxweB7xerB5UTQ==}\n-\n '@forgerock/javascript-sdk@4.9.0':\n resolution: {integrity: sha512-xNE4LMIFYvvPpsq04RqSGnZ+zT+LjOdHePPj6K4uukqewBaZMDmynozkOTtCFmWui7gqANuhf2arbdo3J5uTVQ==}\n \n@@ -9850,14 +9847,6 @@ snapshots:\n optionalDependencies:\n '@noble/hashes': 1.8.0\n \n- '@forgerock/javascript-sdk@4.7.0':\n- dependencies:\n- '@reduxjs/toolkit': 2.10.1\n- immer: 10.2.0\n- transitivePeerDependencies:\n- - react\n- - react-redux\n-\n '@forgerock/javascript-sdk@4.9.0':\n dependencies:\n '@reduxjs/toolkit': 2.10.1\n\n```\n\n## Instructions\n\nRate how similar the commit is to the AI fix on a scale of 0-10.\n\n**Scoring Guide:**\n- 10: Nearly identical changes (same files, same logic, same fixes)\n- 8-9: Very similar (same core changes, minor differences in implementation)\n- 6-7: Similar intent (addresses same issue, somewhat different approach)\n- 4-5: Some overlap (touches same area, but different changes)\n- 2-3: Minimal similarity (different files or completely different changes)\n- 0-1: No similarity (unrelated changes)\n\n**Criteria:**\n- Focus on: same files modified, similar logic changes, same bug being fixed\n- The commit may have additional changes beyond the AI fix - that's okay\n- Small differences in implementation are okay if the intent is the same\n\nProvide your score (0-10) and a brief reasoning (2-3 sentences) explaining your assessment.","response":"{\"score\":4,\"reasoning\":\"The commit does include the core fix suggested by the AI for the basic.test.ts file - it removes the `page.waitForResponse()` call waiting for the signoff response and removes the `await signoff;` line. However, the commit does NOT include the replacement `await page.waitForURL('http://localhost:5829/');` that was suggested in the AI fix. Instead, the commit simply removes the signoff wait entirely and adds `await expect(page.getByText('Username/Password Form')).toBeVisible();` as the next assertion. Additionally, the commit makes extensive changes to davinci-app/main.ts (replacing javascript-sdk with oidc-client) and updates package.json dependencies, which go beyond the scope of the suggested test fix. The test file change is only partially aligned with the AI suggestion.\"}","model":"claude-haiku-4-5-20251001","cipeFixId":"69fa575e322793cb01269c1b","headSha":"d474593db0c1acb16c0cfc53c5488feb9cab5f1f","timestamp":1778616878412}