Skip to content

docs: Sync MSAL Node and Angular docs with v5 SDK changes#84

Open
Dickson-Mwendia wants to merge 1 commit intoMicrosoftDocs:mainfrom
Dickson-Mwendia:docs/v5-sync-msal-node-angular
Open

docs: Sync MSAL Node and Angular docs with v5 SDK changes#84
Dickson-Mwendia wants to merge 1 commit intoMicrosoftDocs:mainfrom
Dickson-Mwendia:docs/v5-sync-msal-node-angular

Conversation

@Dickson-Mwendia
Copy link
Collaborator

Summary

This PR synchronizes the MSAL JavaScript Learn documentation with the latest v5 SDK changes for both MSAL Node and MSAL Angular. Changes were identified through an automated audit using the DocGuard skill pipeline (content comparison, API change detection, and freshness analysis).

Changes

MSAL Node

  • node/configuration.md — Removed deprecated \proxyUrl, \customAgentOptions, and \skipAuthorityMetadataCache\ options; moved \protocolMode\ to System Config; added migration note
  • node/faq.md — Fixed incorrect claim that interactive flow is unsupported (\�cquireTokenInteractive()\ exists in v5); updated proxy and custom agent FAQs; added Node 20+ requirement
  • node/v5-migration.md (new) — Migration guide from MSAL Node v3 to v5 with all breaking changes
  • node/managed-identity.md (new) — Guide for \ManagedIdentityApplication\ (system-assigned and user-assigned)

MSAL Angular

  • angular/configuration.md — Removed deprecated \storeAuthStateInCookie\ and misplaced
    avigateToLoginRequestUrl; added v5 upgrade guide reference
  • angular/redirects.md — Added standalone component patterns, \handleRedirectObservable\ options section; removed legacy IE detection code; updated sample references
  • angular/msal-interceptor.md — Added strict matching section documenting the v5 default behavior change for \protectedResourceMap\
  • angular/v4-v5-upgrade-guide.md (new) — Migration guide from MSAL Angular v4 to v5 (Angular 19, strictMatching, inject tokens)
  • angular/v3-v4-upgrade-guide.md (new) — Migration guide from MSAL Angular v3 to v4

Infrastructure

  • TOC.yml — Added entries for all 4 new pages
  • Added #Customer intent\ metadata to all modified and new files

Audit Findings Addressed

  • 7 CRITICAL fixes (removed deprecated APIs, corrected false claims, filled documentation gaps)
  • 3 HIGH fixes (stale configuration, missing features)
  • 4 MEDIUM fixes (metadata, terminology updates)

Validation

  • All changes cross-referenced against SDK source at \AzureAD/microsoft-authentication-library-for-js@dev\
  • Customer intent statements placed inside YAML frontmatter per Learn standards
  • No auto-generated reference docs modified

- Remove deprecated proxyUrl, customAgentOptions, skipAuthorityMetadataCache from Node config
- Fix FAQ incorrectly claiming interactive flow is unsupported
- Add strict matching documentation to MSAL Interceptor
- Remove IE detection code and storeAuthStateInCookie from Angular docs
- Add standalone component patterns to Angular redirects guide
- Add handleRedirectObservable options documentation
- Add customer intent metadata to all modified files
- New: MSAL Node v3-to-v5 migration guide
- New: Managed identity guide for MSAL Node
- New: MSAL Angular v3-to-v4 upgrade guide
- New: MSAL Angular v4-to-v5 upgrade guide
- Update TOC.yml with new page entries

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
@learn-build-service-prod
Copy link

PoliCheck Scan Report

The following report lists PoliCheck issues in PR files. Before you merge the PR, you must fix all severity-1 and severity-2 issues. The AI Review Details column lists suggestions for either removing or replacing the terms. If you find a false positive result, mention it in a PR comment and include this text: #policheck-false-positive. This feedback helps reduce false positives in future scans.

✅ No issues found

More information about PoliCheck

Information: PoliCheck | Severity Guidance | Term
For any questions: Try searching the learn.microsoft.com contributor guides or post your question in the Learn support channel.

@learn-build-service-prod
Copy link

Learn Build status updates of commit 38773db:

⚠️ Validation status: warnings

File Status Preview URL Details
msal-javascript-conceptual/angular/v4-v5-upgrade-guide.md ⚠️Warning View Details
msal-javascript-conceptual/angular/v3-v4-upgrade-guide.md 💡Suggestion View Details
msal-javascript-conceptual/angular/configuration.md ✅Succeeded View
msal-javascript-conceptual/angular/msal-interceptor.md ✅Succeeded View
msal-javascript-conceptual/angular/redirects.md ✅Succeeded View
msal-javascript-conceptual/node/configuration.md ✅Succeeded View
msal-javascript-conceptual/node/faq.md ✅Succeeded View
msal-javascript-conceptual/node/managed-identity.md ✅Succeeded View
msal-javascript-conceptual/node/v5-migration.md ✅Succeeded View
msal-javascript-conceptual/TOC.yml ✅Succeeded View

msal-javascript-conceptual/angular/v4-v5-upgrade-guide.md

  • Line 19, Column 16: [Warning: file-not-found - See documentation] Invalid file link: '../browser/v4-migration.md'.
  • Line 0, Column 0: [Suggestion: filename-incomplete - See documentation] The filename contains an incomplete word, misspelled word, acronym, or abbreviation that is disallowed.

msal-javascript-conceptual/angular/v3-v4-upgrade-guide.md

  • Line 0, Column 0: [Suggestion: filename-incomplete - See documentation] The filename contains an incomplete word, misspelled word, acronym, or abbreviation that is disallowed.

For more details, please refer to the build report.

Note: Your PR may contain errors or warnings or suggestions unrelated to the files you changed. This happens when external dependencies like GitHub alias, Microsoft alias, cross repo links are updated. Please use these instructions to resolve them.

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.

1 participant