fix: dev to main - Private Networking for Web Apps, Improve CI Test Reporting, and Strengthen Error Handling

[Pavan-Microsoft]

Purpose

This pull request introduces several improvements and fixes across infrastructure, CI/CD workflows, and Python scripts. The main focus areas are enhanced private networking support for Azure Web Apps, improved test reporting in GitHub Actions, more robust and explicit error handling in Python scripts, and minor code cleanups.

Infrastructure and Deployment Enhancements:

• Added private DNS zone and private endpoint support for Azure Web Apps in both infra/main.bicep and infra/main_custom.bicep, enabling backend web apps to be deployed with private networking and disabling public network access when private networking is enabled. Related app settings are also conditionally set based on networking mode. [1] [2] [3] [4] [5] [6] [7] [8]
• Updated the compiled ARM template infra/main.json to reflect these changes, including DNS zone indices and dependency ordering. [1] [2] [3] [4]

CI/CD and Test Reporting:

• Enhanced the GitHub Actions workflow (.github/workflows/test.yml) to generate and post pytest coverage comments on pull requests, improving visibility into test coverage for changed files. [1] [2]

Python Script Improvements:

• Replaced generic except: blocks with more explicit except Exception: across several scripts for better error handling and clarity. Added comments to clarify when exceptions are expected and safe to ignore. [1] [2] [3] [4] [5] [6] [7]
• Minor code cleanups in scripts, including removing unused imports and variable renaming for clarity in file uploads. [1] [2]

Bug Fixes:

• Removed unnecessary variable initializations in utility functions to streamline code in docs/workshop/docs/workshop/Challenge-5/python/utility.py. [1] [2] [3]

Other:

• Updated version constraints in azure.yaml for dependencies.

Let me know if you have questions about any of these changes or want to dive deeper into a specific area!

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.

[github-actions]

Coverage Report •

File	Stmts	Miss	Cover	Missing
src/api/common/database
cosmosdb_service.py	107	2	98%	31, 177
sqldb_service.py	187	22	88%	18–26, 28–29, 62–63, 65–70, 195, 294, 334
src/api/services
chat_service.py	172	12	93%	59–60, 223–226, 230, 233, 244–245, 265–266
TOTAL	1305	147	88%

Tests	Skipped	Failures	Errors	Time
163	0 💤	0 ❌	0 🔥	6.571s ⏱️

[Copilot]

Pull request overview

This PR enhances deployment/security (private networking for Azure Web Apps), improves CI feedback (pytest coverage PR comments), strengthens Python exception handling, and refactors the React frontend to Redux Toolkit with a new fetch-based HTTP client + updated chat/history/citation flows.

Changes:

• Add private endpoint + private DNS zone support for Azure Web Apps, with conditional public access/app settings for WAF/private deployments.
• Improve GitHub Actions backend test reporting by publishing pytest coverage/comments to PRs.
• Refactor frontend state from a custom context/reducer to Redux Toolkit; add new utilities (HTTP client, retry/cache helpers, chat streaming hook) and update components accordingly.

Reviewed changes

Copilot reviewed 75 out of 84 changed files in this pull request and generated 4 comments.

Show a summary per file

File	Description
tests/e2e-test/tests/test_telecom_smoke_tc.py	Removes unused variables in e2e telecom smoke tests.
tests/e2e-test/pages/loginPage.py	Ensures BasePage initialization via super().__init__.
tests/e2e-test/pages/KMGenericPage.py	Ensures BasePage initialization via super().__init__.
tests/e2e-test/pages/HomePage.py	Ensures BasePage initialization via super().__init__.
tests/e2e-test/config/constants.py	Reformat + env loading for e2e constants (URL/API_URL trimming).
tests/e2e-test/base/base.py	Cleanup in BasePage helper (remove unused vars/imports).
src/tests/api/services/test_history_service.py	Removes unused import from unit tests.
src/tests/api/services/test_chat_service.py	Removes unused import from unit tests.
src/tests/api/helpers/test_utils.py	Test cleanup (remove unused mocks/imports).
src/tests/api/common/logging/test_event_utils.py	Removes unused test import.
src/tests/api/common/database/test_cosmosdb_service.py	Updates test expectations for delete_messages return value.
src/tests/api/auth/test_auth_utils.py	Removes unused import from auth tests.
src/tests/api/api/test_api_routes.py	Removes unused imports and formatting cleanup.
src/api/services/chat_service.py	Adds clarifying comment for best-effort DB close exception handling.
src/api/requirements.txt	Updates backend dependency pins.
src/api/common/database/sqldb_service.py	Fixes connection return path; narrows exception handling to Exception.
src/api/common/database/cosmosdb_service.py	Ensures delete_messages consistently returns a list.
src/App/src/utils/messageUtils.ts	Adds message helpers (chart query detection, citation parsing).
src/App/src/utils/chartUtils.ts	Adds chart parsing + sentiment color helper.
src/App/src/utils/apiUtils.ts	Adds retry/cache/debounce/throttle helpers and error response helper.
src/App/src/types/AppTypes.ts	Cleans up unused types/imports.
src/App/src/state/useAppContext.tsx	Removes legacy context hook.
src/App/src/state/store.ts	Adds Redux Toolkit store configuration.
src/App/src/state/slices/dashboardSlice.ts	Adds Redux slice for dashboard/filter/chart state.
src/App/src/state/slices/citationSlice.ts	Adds Redux slice for citation panel state.
src/App/src/state/slices/chatSlice.ts	Adds Redux slice for chat state and message updates.
src/App/src/state/slices/chatHistorySlice.ts	Adds Redux slice + thunks for chat history list/read/rename/delete.
src/App/src/state/slices/appSlice.ts	Adds Redux slice for app config, conversation selection, and spinner state.
src/App/src/state/hooks.ts	Adds typed Redux hooks (useAppDispatch, useAppSelector).
src/App/src/state/AppReducer.tsx	Removes legacy reducer implementation.
src/App/src/state/AppProvider.tsx	Removes legacy context provider.
src/App/src/state/ActionConstants.tsx	Removes legacy action constants.
src/App/src/logo.svg	Removes unused default CRA logo asset.
src/App/src/index.tsx	Wires Redux <Provider store={store}> instead of AppProvider.
src/App/src/hooks/useChatApi.ts	Adds new chat streaming + persistence hook built on Redux + new API layer.
src/App/src/hooks/useAutoScroll.ts	Adds auto-scroll hook for chat UI.
src/App/src/configs/Utils.tsx	Avoids mutating inputs; tightens error parsing behavior.
src/App/src/components/Citations/Citations.tsx	Updates citations rendering to Redux-based state updates and memoization.
src/App/src/components/Citations/AnswerParser.tsx	Simplifies answer parsing (returns citations/markdown directly).
src/App/src/components/CitationPanel/CitationPanel.tsx	Updates panel close behavior to Redux; removes raw HTML rendering.
src/App/src/components/CitationPanel/CitationPanel.css	CSS cleanup/formatting.
src/App/src/components/ChatHistoryPanel/ChatHistoryPanel.tsx	Refactors to Redux selectors and removes legacy context usage.
src/App/src/components/ChatHistoryListItemGroups/ChatHistoryListItemGroups.tsx	Refactors to Redux selectors and improves observer cleanup.
src/App/src/components/ChatHistoryListItemCell/ChatHistoryListItemCell.tsx	Refactors rename/delete flows to Redux thunks; fixes citation cleanup on delete.
src/App/src/components/ChatHistory/ChatHistory.tsx	Removes unused legacy chat history placeholder component.
src/App/src/components/ChatHistory/ChatHistory.css	Removes CSS for deleted legacy component.
src/App/src/components/ChatChart/ChatChart.tsx	Safer canvas/parent handling for Chart.js + resize observer.
src/App/src/components/Chat/ChatMessageItem.tsx	Adds unified renderer for chat messages, charts, citations, typing indicator.
src/App/src/components/Chat/Chat.css	Removes commented-out CSS.
src/App/src/components/ChartFilter/ChartFilter.tsx	Refactors filter state updates to Redux slice; improves menu list rendering.
src/App/src/components/Chart/Chart.tsx	Refactors chart/filter fetching to Redux, consolidates fetch logic, adds fallbacks.
src/App/src/chartComponents/WordCloudChart.tsx	Updates re-render triggers and resize observer handling.
src/App/src/chartComponents/HorizontalBarChart.tsx	Fixes effect dependency list for chart rendering.
src/App/src/chartComponents/DonutChart.tsx	Improves ref usage + safe cleanup.
src/App/src/api/httpClient.ts	Adds fetch wrapper with interceptors, query params, timeout, and auth header injection.
src/App/src/api/api.ts	Migrates API calls to new httpClient + retry/cache helpers; improves parsing/fallbacks.
src/App/src/Assets/km_logo.png	Asset touched in PR metadata (no textual diff shown).
src/App/src/Assets/Sparkle.png	Asset touched in PR metadata (no textual diff shown).
src/App/src/Assets/Reset-icon.svg	Removes unused reset icon SVG asset.
src/App/src/Assets/ContosoImg.png	Asset touched in PR metadata (no textual diff shown).
src/App/package.json	Adds Redux dependencies and bumps frontend package versions.
src/App/nginx.conf	Adds custom nginx config including API proxy include + security headers.
src/App/env.sh	Generates nginx proxy config at startup when BACKEND_API_HOST is set.
src/App/WebApp.Dockerfile	Installs custom nginx.conf and ensures proxy conf exists at runtime.
infra/scripts/validate_bicep_params.py	Adds clarifying comment for JSON decode error handling.
infra/scripts/index_scripts/requirements.txt	Updates pypdf pin.
infra/scripts/index_scripts/04_cu_process_custom_data.py	Narrows exception handling; removes unused vars; avoids unused upload result.
infra/scripts/index_scripts/03_cu_process_data_text.py	Narrows exception handling; removes unused vars; avoids unused upload result.
infra/scripts/index_scripts/02_create_cu_template_text.py	Narrows exception handling with explanation.
infra/scripts/index_scripts/02_create_cu_template_audio.py	Narrows exception handling with explanation.
infra/scripts/fabric_scripts/create_fabric_items.py	Removes unused imports; narrows exception handling; minor cleanup.
infra/main_custom.bicep	Adds Web Apps private DNS zone + private endpoint support; conditional app settings.
infra/main.json	Updates compiled ARM template to reflect Bicep changes.
infra/main.bicep	Adds Web Apps private DNS zone + private endpoint support; conditional app settings.
docs/workshop/docs/workshop/Challenge-5/python/utility.py	Replaces bare except: with except Exception: and removes unnecessary initializations.
azure.yaml	Updates required version constraints (removes bicep constraint line).
.github/workflows/test.yml	Adds junit XML output and posts pytest coverage comments to PRs.

Files not reviewed (1)

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

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

[github-actions]

🎉 This PR is included in version 3.22.0 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀