Skip to content

Streamline sidecar documentation with table format and remove duplication#3564

Merged
jmprieur merged 7 commits intofeature/doc-modernizationfrom
copilot/streamline-sidecar-documentation
Oct 20, 2025
Merged

Streamline sidecar documentation with table format and remove duplication#3564
jmprieur merged 7 commits intofeature/doc-modernizationfrom
copilot/streamline-sidecar-documentation

Conversation

Copy link
Contributor

Copilot AI commented Oct 19, 2025

Overview

This PR streamlines the sidecar documentation by establishing clear separation of concerns between README.md and index.md, improving navigation clarity and eliminating content duplication.

Changes

README.md - Restructured as Navigation Hub

Transformed README.md into a pure navigation hub using table-based formatting for improved clarity:

Before:

  • Bullet-point lists for documentation structure, scenarios, and resources
  • Narrative style mixing navigation with explanatory text

After:

  • Clean table format with Document/Description columns for core documentation
  • Scenario table with Scenario/Description columns providing clear context for each guide
  • Resources table with Resource/Link columns for external references
  • Updated Microsoft documentation URLs from deprecated docs.microsoft.com to current learn.microsoft.com domain

index.md - Removed Duplicate Content

Eliminated the redundant "Common Scenarios" section that duplicated the scenario links already present in README.md:

Before:

  • "Quick Links" section for core documentation
  • "Common Scenarios" section listing all 9 scenario guides (duplicate of README.md)

After:

  • "Quick Links" section preserved for core documentation references
  • Removed duplicate scenario links (14 lines)
  • Maintains focus on overview, architecture, and key concepts

Benefits

  • Clear Separation of Concerns: README.md serves as the navigation hub while index.md provides conceptual overview and architecture
  • Eliminates Redundancy: Scenario guides are now listed only in README.md, reducing maintenance burden
  • Improved Scannability: Table format makes it easier to quickly find relevant documentation
  • Enhanced User Experience: Users can navigate more efficiently with organized, tabular navigation
  • Modernized Links: Updated to current Microsoft Learn documentation URLs

Impact

Documentation-only changes with no code or functional impact. All links remain functional and markdown formatting is preserved.

Original prompt

This PR streamlines the sidecar documentation by establishing clear separation of concerns between README.md and index.md. The README.md is restructured as a pure navigation hub with table format, and the index.md has its duplicate "Common Scenarios" section removed to eliminate redundancy. The changes improve clarity, eliminate duplication, and enhance user experience.


✨ Let Copilot coding agent set things up for you — coding agent works faster and does higher quality work when set up for your repo.

Copilot AI and others added 2 commits October 19, 2025 17:10
…tion

Co-authored-by: jmprieur <13203188+jmprieur@users.noreply.github.com>
…main

Co-authored-by: jmprieur <13203188+jmprieur@users.noreply.github.com>
Copilot AI changed the title [WIP] Improve sidecar documentation structure and clarity Streamline sidecar documentation with table format and remove duplication Oct 19, 2025
Copilot AI requested a review from jmprieur October 19, 2025 17:14
Updated the README to enhance clarity and structure, including new sections and improved descriptions for scenarios and resources.
Updated the Microsoft Entra Identity Sidecar documentation to enhance clarity and detail on token acquisition, validation, and integration. Added benefits and use cases for the sidecar approach.
@jmprieur jmprieur marked this pull request as ready for review October 20, 2025 00:54
@jmprieur jmprieur requested a review from a team as a code owner October 20, 2025 00:54
Copy link
Collaborator

@jmprieur jmprieur left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@jmprieur jmprieur merged commit 2b92121 into feature/doc-modernization Oct 20, 2025
1 check passed
@jmprieur jmprieur deleted the copilot/streamline-sidecar-documentation branch October 20, 2025 00:54
jmprieur added a commit that referenced this pull request Jan 3, 2026
* Adding wiki archive

* Create README.md with revised documentation content

* Adding README, and quickstars for web api and web app, co-authored with GitHub Copilot

* Update the qs with auth scheme

* add the plan (co authored with Copilot

* Adding credentials (with copilot)

* Credentials

* Add main overview for calling downstream APIs

* Add comprehensive documentation for Microsoft Entra Identity Sidecar (#3535)

* Initial plan

* Add core sidecar documentation files

Co-authored-by: jmprieur <13203188+jmprieur@users.noreply.github.com>

* Add remaining core documentation files

Co-authored-by: jmprieur <13203188+jmprieur@users.noreply.github.com>

* Add all scenario documentation files

Co-authored-by: jmprieur <13203188+jmprieur@users.noreply.github.com>

* Address PR feedback: Convert diagram to mermaid, update wording, add validation scenario

Co-authored-by: jmprieur <13203188+jmprieur@users.noreply.github.com>

* Update docs/sidecar/reverse-proxy.md

* Address PR review feedback: Update agent identity docs, improve examples, fix terminology

Co-authored-by: jmprieur <13203188+jmprieur@users.noreply.github.com>

* Update docs/sidecar/faq.md

* Replace ASCII diagram with mermaid diagram in comparison.md

Co-authored-by: jmprieur <13203188+jmprieur@users.noreply.github.com>

* Apply suggestions from code review

Co-authored-by: Keegan <Keegan.Caruso@microsoft.com>

* Update docs/sidecar/comparison.md

* Update docs/sidecar/comparison.md

* Address comprehensive PR review feedback

Co-authored-by: jmprieur <13203188+jmprieur@users.noreply.github.com>

* Remove Service Mesh Integration section from security.md

Co-authored-by: jmprieur <13203188+jmprieur@users.noreply.github.com>

* Update network security guidance to use Kestrel configuration instead of network policies

Co-authored-by: jmprieur <13203188+jmprieur@users.noreply.github.com>

* Add input validation guidance and incident response plan reference

Co-authored-by: jmprieur <13203188+jmprieur@users.noreply.github.com>

---------

Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com>
Co-authored-by: jmprieur <13203188+jmprieur@users.noreply.github.com>
Co-authored-by: Jean-Marc Prieur <jmprieur@microsoft.com>
Co-authored-by: Keegan <Keegan.Caruso@microsoft.com>

* Add table of contents for Sidecar documentation

* Streamline sidecar documentation with table format and remove duplication (#3564)

* Initial plan

* Streamline sidecar documentation with table format and remove duplication

Co-authored-by: jmprieur <13203188+jmprieur@users.noreply.github.com>

* Update Microsoft documentation URLs to current learn.microsoft.com domain

Co-authored-by: jmprieur <13203188+jmprieur@users.noreply.github.com>

* docs(sidecar): fix endpoints path formatting and clarify wording

* Revise README for clarity and improved navigation

Updated the README to enhance clarity and structure, including new sections and improved descriptions for scenarios and resources.

* Revise Entra Identity Sidecar documentation

Updated the Microsoft Entra Identity Sidecar documentation to enhance clarity and detail on token acquisition, validation, and integration. Added benefits and use cases for the sidecar approach.

* Update endpoints.md

---------

Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com>
Co-authored-by: jmprieur <13203188+jmprieur@users.noreply.github.com>
Co-authored-by: Jean-Marc Prieur <jmprieur@microsoft.com>

* Docs for Downstream Apis (#3558)

* Downstream Apis

* Update README.md with revised service registration pattern and documentation improvements.

* Revise README for Calling Downstream APIs

This document provides a comprehensive guide for implementing various approaches to call downstream APIs using Microsoft.Identity.Web, including decision trees, comparison tables, token acquisition patterns, and code examples for different scenarios.

* Enhance README for downstream API guidance

Updated README to include .NET applications and agent identities.

* Update README with API selection decision table

Replaced mermaid diagram with a decision table for API selection.

* Update docs/calling-downstream-apis/azure-sdks.md

* Refactor downstream API documentation

Updated the documentation for calling downstream APIs by consolidating API registration and clarifying the request methods. Removed redundant sections and improved the structure.

* Remove Azure SDK examples from documentation

Removed examples for Azure KeyVault, Azure Storage Blobs, Azure KeyVault Secrets, Azure Service Bus, Azure Tables, and Azure Cosmos DB from the documentation.

* Update custom-apis.md

* Update from-web-apis.md

* Update microsoft-graph.md

* Revise documentation modernization plan and status

Updated the documentation modernization plan with completed milestones and revised priorities. Adjusted documentation structure and clarified the status of various sections, including the addition of new documentation for sidecar and agent identities.

* Fixing broken links and updating the plan

* Finishing downstream APIs

* adding token cache

* Adding cross links with token cache

* cross links with token cache

* Updating the plan

* More articles

* Separating MSAL and OWIN

* separating OWIN and MSAL

* Add more cross links

* Adding daemon scenario

* Add gatewatsm proxies and Fix #3596

* Merge the custom-api documentation

* Removing docs we don't need

* Remove sidecar. Rename readme.md

* Adding missing docs, a script to detect broken links, and fixing a few links

* Moving design-docs out of (customer) docs

* docs: fix quickstart web API snippet typos (#3658)

* Fixing links

* Fixing broken links

* Adding missing links

* FIxing docs.

---------

Co-authored-by: Copilot <198982749+Copilot@users.noreply.github.com>
Co-authored-by: jmprieur <13203188+jmprieur@users.noreply.github.com>
Co-authored-by: Keegan <Keegan.Caruso@microsoft.com>
Co-authored-by: kz <74097723+fuguiKz@users.noreply.github.com>
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