chore: dev to main merge by Ragini-Microsoft · Pull Request #914 · microsoft/Conversation-Knowledge-Mining-Solution-Accelerator

Ragini-Microsoft · 2026-05-14T10:06:52Z

Purpose

This pull request updates documentation to reflect a migration from Azure Container Apps to Azure App Service for hosting the application's web front-end and backend APIs. It also updates related security and troubleshooting guidance to align with this change. The most important changes are grouped below.

Service Architecture and Deployment Documentation:

Replaces references to Azure Container Apps with Azure App Service as the primary hosting platform for the web application and backend API in README.md and documents/DeploymentGuide.md. Updates include descriptions, pricing, and links to relevant Azure documentation. [1] [2]

Security Guidance:

Updates security recommendations to use Virtual Networks or access restrictions for Azure App Service, replacing previous guidance for Azure Container Apps.

Troubleshooting Documentation:

Removes troubleshooting steps and quota error documentation related to Azure Container Apps, including regional environment limits and managed environment deletion errors, and adds/updates guidance for Azure App Service capacity and related issues in documents/TroubleShootingSteps.md. [1] [2]

Does this introduce a breaking change?

Yes
No

Golden Path Validation

I have tested the primary workflows (the "golden path") to ensure they function correctly without errors.

Deployment Validation

I have validated the deployment process successfully and all services are running as expected with this change.

Remove unconditional hideCitation() call from onHandlePanelStates so toggling one panel no longer closes the other. Citation is still correctly dismissed on conversation switch and history clear. Fixes #43310 Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

…ge account and added e2eEncryption in app service

Co-authored-by: Copilot <copilot@github.com>

Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>

Use an async Azure credential for the async CosmosClient in HistoryService.init_cosmosdb_client. The previous sync ManagedIdentityCredential, when paired with azure.cosmos.aio.CosmosClient, could fail to refresh tokens cleanly after an idle period, causing DELETE /history/delete_all to return 500. Single-conversation delete was unaffected because it bypasses the query_items async iterator path. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

Co-authored-by: Copilot <copilot@github.com>

…Conversation-Knowledge-Mining-Solution-Accelerator into clean-unusedresources

…ions

Move CitationPanel outside the flex layout container (.main-container) and render it as a fixed-position overlay. This prevents the two panels from competing for flex space, which caused: - Closing Chat History to also close Citation panel - Opening Citation to push Chat History off-screen Fixes AB#43310 Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

fix: Added data collection rules for VM, added double encryption for storage account and added e2eEncryption in app service

Address review feedback: CitationPanel is now a flex participant inside .main-container using .citation-panel-wrapper (flex: 0 0 20%) instead of a fixed-position overlay. This reserves layout space and avoids blocking clicks on underlying panels. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

…istory Citation panel is now a fixed overlay that dynamically offsets its position when Chat History is open, preventing click-blocking while maintaining full panel independence. - Overlay shifts left by Chat History width when both panels are open - Panels remain fully independent (opening/closing one never affects the other) Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

The simple overlay approach is the cleanest solution - Citation floats independently without any coupling to the flex layout. Tested and verified that all 4 panels (Dashboard, Chat, Citation, Chat History) work correctly side by side. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

- Extract --header-height CSS custom property to avoid 6vh duplication - Add pointer-events: none on overlay container to prevent dead zones - Use <section> with aria-label for citation overlay accessibility Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

Copilot

Pull request overview

Copilot reviewed 14 out of 16 changed files in this pull request and generated 3 comments.

Files not reviewed (1)

src/App/package-lock.json: Language not supported

…ential Addresses Copilot review feedback on PR #915 (merged into dev). The two helpers get_azure_credential_async (async fn) and get_async_azure_credential (sync fn) differ only by word order, which is easy to confuse at call sites. Renaming the sync builder to build_async_azure_credential makes the sync-vs-async distinction explicit: build_ implies sync construction, async describes the returned credential type. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

- Use flex-based citation-panel-wrapper for consistent height alignment - Add box-sizing: border-box to include padding/border in height calc - Prevent horizontal overflow on main-container Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

Citation should close when Chat panel is hidden since citations originate from chat responses. ChatHistory toggle remains independent. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

…ential (address Copilot review on #915)

Copilot

Pull request overview

Copilot reviewed 14 out of 16 changed files in this pull request and generated 1 comment.

Files not reviewed (1)

src/App/package-lock.json: Language not supported

Yamini-Microsoft and others added 21 commits May 12, 2026 19:17

cleanup unused resources

f716105

Added data collection rules for VM, added double encryption for stora…

62ce8eb

…ge account and added e2eEncryption in app service

copilot comments

9711da3

Co-authored-by: Copilot <copilot@github.com>

fixed table structure

235d997

add extension monitoring agent configuration for VM

3aaebef

Potential fix for pull request finding

aa70100

Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>

docs: Add Azure Container Registry to prerequisites in Deployment Guide

75e453a

Co-authored-by: Copilot <copilot@github.com>

Merge branch 'clean-unusedresources' of https://github.com/microsoft/…

7f50235

…Conversation-Knowledge-Mining-Solution-Accelerator into clean-unusedresources

removed unnecessary comments

59a7aa7

refactor: Refactor codebase by removing unused scripts and configurat…

9bcc8e1

…ions

Merge branch 'dev' into psl-hb-us-43316

07f4586

rebuild main.json

5944cde

Merge pull request #910 from microsoft/psl-hb-us-43316

66ecc58

fix: Added data collection rules for VM, added double encryption for storage account and added e2eEncryption in app service

fix: added code for citation and chat history panel independence

98d88ae

Copilot AI review requested due to automatic review settings May 14, 2026 10:06

Ragini-Microsoft requested review from Avijit-Microsoft, Prajwal-Microsoft, Roopan-Microsoft, Vinay-Microsoft, aniaroramsft, brittneek, nchandhi and toherman-msft as code owners May 14, 2026 10:06

Copilot started reviewing on behalf of Avijit-Microsoft May 14, 2026 12:34 View session

Avijit-Microsoft temporarily deployed to production May 14, 2026 12:34 — with GitHub Actions Inactive

Copilot AI reviewed May 14, 2026

View reviewed changes

Comment thread README.md

Comment thread src/api/helpers/azure_credential_utils.py Outdated

Comment thread src/App/src/App.tsx Outdated

Avijit-Microsoft temporarily deployed to production May 14, 2026 13:15 — with GitHub Actions Inactive

Rafi-Microsoft mentioned this pull request May 14, 2026

refactor: rename get_async_azure_credential to build_async_azure_credential (address Copilot review on #915) #916

Merged

4 tasks

Yamini-Microsoft and others added 3 commits May 15, 2026 02:42

fix: hide citation panel when chat is hidden

c4486f2

Citation should close when Chat panel is hidden since citations originate from chat responses. ChatHistory toggle remains independent. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

refactor: rename get_async_azure_credential to build_async_azure_cred…

174a390

…ential (address Copilot review on #915)

Avijit-Microsoft temporarily deployed to production May 15, 2026 04:52 — with GitHub Actions Inactive

Avijit-Microsoft temporarily deployed to production May 15, 2026 04:53 — with GitHub Actions Inactive

Avijit-Microsoft temporarily deployed to production May 15, 2026 05:15 — with GitHub Actions Inactive

Kanchan-Microsoft temporarily deployed to production May 15, 2026 07:06 — with GitHub Actions Inactive

Kanchan-Microsoft temporarily deployed to production May 15, 2026 07:09 — with GitHub Actions Inactive

Kanchan-Microsoft temporarily deployed to production May 15, 2026 07:10 — with GitHub Actions Inactive

Kanchan-Microsoft temporarily deployed to production May 15, 2026 07:52 — with GitHub Actions Inactive

fix: Citation and Chat History panel independence (AB#43310)

715ba55

Copilot AI review requested due to automatic review settings May 15, 2026 09:19

Avijit-Microsoft temporarily deployed to production May 15, 2026 09:19 — with GitHub Actions Inactive

Avijit-Microsoft temporarily deployed to production May 15, 2026 09:20 — with GitHub Actions Inactive

Copilot started reviewing on behalf of Avijit-Microsoft May 15, 2026 09:21 View session

Avijit-Microsoft temporarily deployed to production May 15, 2026 09:21 — with GitHub Actions Inactive

Copilot AI reviewed May 15, 2026

View reviewed changes

Comment thread infra/main_custom.bicep

Avijit-Microsoft temporarily deployed to production May 15, 2026 09:42 — with GitHub Actions Inactive

Avijit-Microsoft approved these changes May 18, 2026

View reviewed changes

Prajwal-Microsoft approved these changes May 18, 2026

View reviewed changes

Prajwal-Microsoft merged commit 5f1ae0a into main May 18, 2026
23 checks passed

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

chore: dev to main merge#914

chore: dev to main merge#914
Prajwal-Microsoft merged 30 commits into
mainfrom
dev

Ragini-Microsoft commented May 14, 2026 •

edited

Loading

Uh oh!

Copilot AI left a comment

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Copilot AI left a comment

Uh oh!

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

9 participants

Conversation

Ragini-Microsoft commented May 14, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Purpose

Does this introduce a breaking change?

Golden Path Validation

Deployment Validation

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Uh oh!

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

9 participants

Ragini-Microsoft commented May 14, 2026 •

edited

Loading