Remove unused RequestValidator class

[tnorling]

Removes unused RequestValidator class

[Copilot]

Pull Request Overview

This PR removes the unused RequestValidator class and its tests, inlines redirect URI validation in AuthorizationCodeClient, and cleans up related error codes and API metadata.

• Deletes RequestValidator and its spec file.
• Inlines redirectUri validation in AuthorizationCodeClient.
• Removes invalidPromptValue entries from error codes, messages, and API review.

Reviewed Changes

Copilot reviewed 7 out of 7 changed files in this pull request and generated no comments.

Show a summary per file

File	Description
lib/msal-common/test/request/RequestValidator.spec.ts	Removed tests covering RequestValidator
lib/msal-common/src/request/RequestValidator.ts	Deleted the entire RequestValidator implementation
lib/msal-common/src/error/ClientConfigurationErrorCodes.ts	Removed invalidPromptValue constant
lib/msal-common/src/error/ClientConfigurationError.ts	Removed messages related to invalidPromptValue
lib/msal-common/src/client/AuthorizationCodeClient.ts	Inlined redirect URI validation, removed import
lib/msal-common/apiReview/msal-common.api.md	Cleaned up API metadata for removed prompt error code
change/@azure-msal-common-76b2aadc-a06c-4ad6-b941-94031039267f.json	Bump patch version and note removal of RequestValidator

Comments suppressed due to low confidence (2)

lib/msal-common/src/client/AuthorizationCodeClient.ts:252

• Add unit tests for this new inline redirect URI validation branch to verify that it throws the correct error (redirectUriEmpty) when request.redirectUri is missing.

            if (!request.redirectUri) {

lib/msal-common/src/client/AuthorizationCodeClient.ts:252

• The PR deletes the RequestValidator class but only inlines validateRedirectUri. The other validation methods (validatePrompt, validateClaims, validateCodeChallengeParams) have been removed without replacement—ensure any remaining callers are updated or migrate those validation checks.

            if (!request.redirectUri) {

[sameerag]

Do we know why we are not using them anymore? Approving, just curious. They look very superficial checks anyway not much value add.

[tnorling]

Do we know why we are not using them anymore? Approving, just curious. They look very superficial checks anyway not much value add.

Some of these checks are still being done, just not with these helpers. They got refactored out at some point