Skip to content

Conversation

@Roopan-Microsoft
Copy link
Collaborator

@Roopan-Microsoft Roopan-Microsoft commented Nov 17, 2025

Purpose

This pull request introduces improvements for robustness and clarity in both infrastructure scripts and the frontend UI. The main changes are grouped into two themes: enhanced fallback logic for extracting Azure deployment outputs in both PowerShell and Bash scripts, and UI updates to better reflect the intended action for team deletion in the Team Selector component. Additionally, there are minor backend and dependency updates.

Infrastructure script improvements:

  • Added a helper function (Get-DeploymentValue in PowerShell, extract_value in Bash) to extract deployment outputs with fallback support, allowing scripts to handle alternate key names if the primary key is missing. (infra/scripts/Team-Config-And-Data.ps1, infra/scripts/team_config_and_data.sh) [1] [2]
  • Updated the extraction logic in both Get-ValuesFromAzDeployment (PowerShell) and get_values_from_az_deployment (Bash) to use the new helper functions, improving reliability when output key names vary. (infra/scripts/Team-Config-And-Data.ps1, infra/scripts/team_config_and_data.sh) [1] [2]

Frontend UI updates:

  • Replaced the "More" icon/button with a "Delete" icon/button in the Team Selector component to more clearly indicate the delete action, and updated related CSS class names and styles for consistency and improved visual feedback. (src/frontend/src/components/common/TeamSelector.tsx, src/frontend/src/styles/TeamSelector.module.css) [1] [2] [3] [4]

Backend and dependency updates:

  • Set the log level for the opentelemetry.sdk logger to ERROR to reduce noise in backend logs. (src/backend/app_kernel.py)
  • Updated the fastmcp dependency version from 2.11.3 to 2.13.0 in pyproject.toml. (src/mcp_server/pyproject.toml)

Does this introduce a breaking change?

  • Yes
  • No

How to Test

  • Get the code
git clone [repo-address]
cd [repo-name]
git checkout [branch-name]
npm install
  • Test the code

What to Check

Verify that the following are valid

  • ...

Other Information

@Roopan-Microsoft Roopan-Microsoft merged commit 148922b into main Nov 24, 2025
10 checks passed
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.

8 participants