refactor: dev to main - Foundry Roles and sanitize CU output for Unicode integrity by Pavan-Microsoft · Pull Request #922 · microsoft/Conversation-Knowledge-Mining-Solution-Accelerator

Pavan-Microsoft · 2026-05-21T07:19:39Z

Purpose

This pull request introduces several improvements and new features to the deployment workflows and documentation for local development. The main highlights include enhanced tagging of Azure resource groups, the addition of scalability options for WAF deployments, and updates to documentation to reflect recent changes in the frontend stack and configuration requirements.

Key changes:

Azure Resource Group Tagging

All deployment workflows now automatically add or update an Owner tag (with the GitHub actor's name) when creating or updating Azure resource groups, improving traceability and resource management. This applies to both new and existing resource groups. [1] [2]

WAF Scalability Feature

Added a new enable_scalability option to relevant workflows (deploy-orchestrator.yml, deploy-v2.yml, job-azure-deploy.yml, job-deploy-linux.yml, job-deploy-windows.yml) to allow users to enable or disable scalability features specifically for WAF deployments. This includes input validation and parameter propagation throughout the workflows. [1] [2] [3] [4] [5] [6] [7] [8] [9] [10] [11] [12] [13] [14] [15]

Workflow Input and Description Updates

Improved descriptions for several workflow inputs to clarify their purpose, updated Azure region options, and refined parameter names for clarity (e.g., distinguishing between AI and non-AI Azure regions, clarifying resource ID and web app URL inputs). [1] [2]

Documentation Updates

Updated the local development setup documentation to reflect the migration to Vite-powered React + TypeScript for the frontend, clarified .env configuration, updated Azure role assignment instructions, and improved layout configuration documentation. [1] [2] [3] [4] [5] [6]

These changes collectively improve deployment automation, resource governance, configuration clarity, and developer onboarding.

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.

…and configurations - Changed project structure to support Vite as the build tool. - Updated package.json to replace react-scripts with Vite and adjusted scripts accordingly. - Removed public/index.html as it is no longer needed with Vite. - Updated environment variable access from process.env to import.meta.env. - Modified TypeScript configuration to target ES2020 and updated module resolution. - Removed reportWebVitals and adjusted testing setup for Vitest. - Added Vite configuration file for server and build settings.

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

…nal ports and environment variable prefixes

…ion-Knowledge-Mining-Solution-Accelerator into km-gen-depdh

…v.production for API base URL

Agent-Logs-Url: https://github.com/microsoft/Conversation-Knowledge-Mining-Solution-Accelerator/sessions/ded6036c-43b4-462c-b7a0-f8c5483b6a1f Co-authored-by: Dhanushree-Microsoft <250931221+Dhanushree-Microsoft@users.noreply.github.com>

…ion-Knowledge-Mining-Solution-Accelerator into km-gen-depdh

…edge-Mining-Solution-Accelerator into km-gen-depdh # Conflicts: # src/App/package-lock.json

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

refactor: Updated Foundry Roles name

…Setup.md

…ion-Knowledge-Mining-Solution-Accelerator into km-gen-depdh

… and setup scripts

The Content Understanding analyzeBinary API (v2025-11-01) intermittently corrupts Unicode characters by stripping the high byte (e.g. U+2019 -> U+0019). This causes apostrophes and quotes to render as box characters in the Citation Panel. Added _sanitize_cu_output() to map known corrupted control characters back to their intended Unicode equivalents after CU processing, before saving to Search/SQL. Fixes AB#43310 Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

…ping Agent-Logs-Url: https://github.com/microsoft/Conversation-Knowledge-Mining-Solution-Accelerator/sessions/50f2a621-4990-4aca-8dd8-cfa5ea31ce6f Co-authored-by: Yamini-Microsoft <191316559+Yamini-Microsoft@users.noreply.github.com>

…gram visuals

…smap

Copilot

Pull request overview

Copilot reviewed 38 out of 40 changed files in this pull request and generated 2 comments.

github-actions · 2026-05-26T07:49:26Z

🎉 This PR is included in version 3.23.0 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

Dhanushree-Microsoft and others added 24 commits May 11, 2026 15:55

Potential fix for pull request finding

ef3fc30

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

Potential fix for pull request finding

0a77286

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

Potential fix for pull request finding

3a1e7f8

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

Potential fix for pull request finding

b78c2f8

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

docs: update Local Development Setup for Vite migration, adjust termi…

da7e601

…nal ports and environment variable prefixes

Merge branch 'km-gen-depdh' of https://github.com/microsoft/Conversat…

053af08

…ion-Knowledge-Mining-Solution-Accelerator into km-gen-depdh

chore: update package dependencies and remove unused entries; add .en…

bbc6dc7

…v.production for API base URL

chore: add 'dev' script to package.json for Vite development

b1cab0d

Merge branch 'km-gen-depdh' of https://github.com/microsoft/Conversat…

f5d50b6

…ion-Knowledge-Mining-Solution-Accelerator into km-gen-depdh

Merge branch 'dev' of https://github.com/microsoft/Conversation-Knowl…

b09a51c

…edge-Mining-Solution-Accelerator into km-gen-depdh # Conflicts: # src/App/package-lock.json

fix: uncomment metadata section in azure.yaml

a676d83

Potential fix for pull request finding

6676241

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

Updated Foundry Roles name

cd469db

Merge pull request #919 from microsoft/PSL-US-43670

0861906

refactor: Updated Foundry Roles name

fix: update frontend description and port numbers in LocalDevelopment…

65fb133

…Setup.md

Merge branch 'km-gen-depdh' of https://github.com/microsoft/Conversat…

69d9d7f

…ion-Knowledge-Mining-Solution-Accelerator into km-gen-depdh

fix: update environment variable names to VITE_API_BASE_URL in README…

29479a1

… and setup scripts

Enhance email template, add RG owner tag, scalability input

e9f9903

feat: add ENABLE_SCALABILITY output to Azure deploy job

d806651

fix: Sanitize CU output to prevent Unicode corruption in Citation Panel

e081526

Pavan-Microsoft temporarily deployed to production May 21, 2026 07:19 — with GitHub Actions Inactive

Pavan-Microsoft temporarily deployed to production May 21, 2026 07:20 — with GitHub Actions Inactive

Pavan-Microsoft temporarily deployed to production May 21, 2026 07:21 — with GitHub Actions Inactive

docs: downmerge to dev from main- Update and improve architecture dia…

f2c2e3d

…gram visuals

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

Prajwal-Microsoft temporarily deployed to production May 22, 2026 11:25 — with GitHub Actions Inactive

Dhanushree-Microsoft mentioned this pull request May 22, 2026

fix: configure Vite CSS modules to match CRA class naming convention #930

Closed

fix: hoist sanitize mapping to module constant, fix U+001E em-dash mi…

e68f85f

…smap

Avijit-Microsoft temporarily deployed to production May 22, 2026 11:41 — with GitHub Actions Inactive

Avijit-Microsoft temporarily deployed to production May 22, 2026 11:43 — with GitHub Actions Inactive

Avijit-Microsoft temporarily deployed to production May 22, 2026 11:44 — with GitHub Actions Inactive

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

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

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

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

Ritesh-Microsoft temporarily deployed to production May 25, 2026 07:56 — with GitHub Actions Inactive

Ritesh-Microsoft temporarily deployed to production May 25, 2026 07:57 — with GitHub Actions Inactive

Ritesh-Microsoft and others added 3 commits May 25, 2026 16:50

cp-e2e-fix-dev

29f993b

ckm-copilot-comments-fixed

da2cb6c

test: ckm chat history locator updated

8972abf

Copilot AI review requested due to automatic review settings May 25, 2026 14:58

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

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

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

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

Copilot AI reviewed May 25, 2026

View reviewed changes

Comment thread tests/e2e-test/pages/HomePage.py

Comment thread src/App/package.json

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

Ritesh-Microsoft temporarily deployed to production May 26, 2026 04:25 — with GitHub Actions Inactive

Ritesh-Microsoft temporarily deployed to production May 26, 2026 04:27 — with GitHub Actions Inactive

Ritesh-Microsoft temporarily deployed to production May 26, 2026 04:50 — with GitHub Actions Inactive

Avijit-Microsoft approved these changes May 26, 2026

View reviewed changes

Prajwal-Microsoft approved these changes May 26, 2026

View reviewed changes

Prajwal-Microsoft merged commit be1f7ed into main May 26, 2026
27 checks passed

github-actions Bot added the released label May 26, 2026

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

refactor: dev to main - Foundry Roles and sanitize CU output for Unicode integrity#922

refactor: dev to main - Foundry Roles and sanitize CU output for Unicode integrity#922
Prajwal-Microsoft merged 38 commits into
mainfrom
dev

Pavan-Microsoft commented May 21, 2026 •

edited

Loading

Uh oh!

Copilot AI left a comment

Uh oh!

Uh oh!

Uh oh!

Uh oh!

github-actions Bot commented May 26, 2026

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

12 participants

Conversation

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

Uh oh!

Purpose

Azure Resource Group Tagging

WAF Scalability Feature

Workflow Input and Description Updates

Documentation Updates

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!

github-actions Bot commented May 26, 2026

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

12 participants

Pavan-Microsoft commented May 21, 2026 •

edited

Loading