fix: merging dev changes to main branch

[Roopan-Microsoft]

Purpose

• This pull request includes several updates across multiple files, focusing on workflow enhancements, dependency updates, and bug fixes. The most important changes include adding support for a new branch in GitHub workflows, updating dependencies in multiple package.json and requirements.txt files, and handling null responses in the chat feature.

Workflow Enhancements:

• .github/workflows/build-clientadvisor.yml: Added support for the dependabotchanges branch in the push condition.
• .github/workflows/build-docker.yml: Added handling for the dependabotchanges branch to set the appropriate tag name.
• .github/workflows/build-researchassistant.yml: Added support for the dependabotchanges branch in the push condition.

Dependency Updates:

• ClientAdvisor/App/frontend/package.json: Updated several dependencies and devDependencies to their latest versions.
• ClientAdvisor/App/requirements.txt and ClientAdvisor/App/requirements-dev.txt: Updated multiple Python packages to their latest versions. [1] [2]
• ResearchAssistant/App/frontend/package.json: Updated several dependencies and devDependencies to their latest versions. [1] [2]
• ResearchAssistant/App/requirements.txt: Updated multiple Python packages to their latest versions.

Bug Fixes:

• ClientAdvisor/App/frontend/src/pages/chat/Chat.tsx: Added handling for null object responses from the History Generate Method in the chat feature.

Testing Adjustments:

• ClientAdvisor/App/frontend/src/test/setupTests.ts: Commented out API mocking setup to potentially address issues during testing.

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.

What to Check

Verify that the following are valid

• ...

Other Information

[Copilot]

Pull Request Overview

This pull request merges development changes into the main branch and updates the project configuration and dependencies while addressing a null response bug in the chat feature.

• Workflow enhancements: Added support for the "dependabotchanges" branch in three GitHub workflow files.
• Dependency updates: Upgraded multiple dependencies in package.json and requirements files.
• Bug fix: Improved handling of null responses in the chat component and adjusted testing setup.

Reviewed Changes

Copilot reviewed 7 out of 15 changed files in this pull request and generated no comments.

Show a summary per file

File	Description
ClientAdvisor/App/frontend/src/test/setupTests.ts	Commented out API mocking code to adjust test setup.
ClientAdvisor/App/frontend/src/pages/chat/Chat.tsx	Added logic for handling null responses from the History Generate Method.
.github/workflows/build-researchassistant.yml	Updated push condition to include "dependabotchanges" branch.
.github/workflows/build-docker.yml	Added branch-based tagging for "dependabotchanges".
.github/workflows/build-clientadvisor.yml	Updated push condition to include "dependabotchanges" branch.

Files not reviewed (8)

• ClientAdvisor/App/frontend/package.json: Language not supported
• ClientAdvisor/App/requirements-dev.txt: Language not supported
• ClientAdvisor/App/requirements.txt: Language not supported
• ClientAdvisor/AzureFunction/requirements.txt: Language not supported
• ClientAdvisor/Deployment/scripts/index_scripts/requirements.txt: Language not supported
• ResearchAssistant/App/frontend/package.json: Language not supported
• ResearchAssistant/App/requirements.txt: Language not supported
• ResearchAssistant/Deployment/scripts/aihub_scripts/requirements.txt: Language not supported

Comments suppressed due to low confidence (2)

ClientAdvisor/App/frontend/src/test/setupTests.ts:6

• [nitpick] If API mocking is intentionally disabled, consider either removing the commented out code to reduce clutter or adding an explanatory comment to clarify why it is disabled and how test coverage is maintained without it.

// import { server } from '../mocks/server';

ClientAdvisor/App/frontend/src/pages/chat/Chat.tsx:363

• [nitpick] Consider breaking out of the loop immediately after handling the null response to avoid an unnecessary iteration. You may insert a break statement inside this block to improve clarity and ensure that the loop terminates as soon as the response is determined to be null.

if (firstRead && done && value === undefined) {

[github-actions]

🎉 This PR is included in version 1.0.0 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

[github-actions]

🎉 This PR is included in version 1.1.0 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀