Skip to content

fix: Dev to main merge - #1171

Merged
Prajwal-Microsoft merged 5 commits into
mainfrom
dev
Oct 7, 2026
Merged

Prajwal-Microsoft merged 5 commits into
mainfrom
dev

Conversation

@Dhruvkumar-Microsoft

Copy link
Copy Markdown
Contributor

Purpose

This pull request updates the urllib3 dependency from version 2.7.0 to 2.8.0 across multiple Python project modules. This ensures consistency and brings in the latest fixes and improvements from the urllib3 library.

Dependency updates:

  • Updated urllib3 from 2.7.0 to 2.8.0 in src/App/pyproject.toml to ensure the application uses the latest version.
  • Updated urllib3 from 2.7.0 to 2.8.0 in src/backend/pyproject.toml for improved security and compatibility.
  • Updated urllib3 from 2.7.0 to 2.8.0 in src/mcp_server/pyproject.toml to maintain alignment across services.

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

@github-actions

github-actions Bot commented Oct 5, 2026 •

Copy link
Copy Markdown

Coverage

Coverage Report •
FileStmtsMissCoverMissing
common/utils
   team_utils.py114992%135–136, 138–144
TOTAL383555285% 

Tests Skipped Failures Errors Time
841 0 💤 0 ❌ 0 🔥 8.724s ⏱️

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot review overview

🟡 Changes recommended

The App Docker installation bypasses the updated manifest and lockfile, leaving urllib3 2.8.0 unenforced in production.

Review effort: Balanced
Findings: 1 Medium severity

Open (1)
What changed in this PR

Aligns urllib3 dependencies across the App, backend, and MCP server.

Changes:

  • Updates urllib3 pins from 2.7.0 to 2.8.0.
  • Refreshes lockfiles, also upgrading AnyIO and PyJWT.
File Description
src/​mcp_server/​uv.lock Updates locked dependencies and artifacts.
src/​mcp_server/​pyproject.toml Pins urllib3 to 2.8.0.
src/​backend/​uv.lock Updates locked dependencies and artifacts.
src/​backend/​pyproject.toml Pins urllib3 to 2.8.0.
src/​App/​uv.lock Updates locked dependencies and artifacts.
src/​App/​pyproject.toml Pins urllib3 to 2.8.0.

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

Comment thread src/App/pyproject.toml
fix: resolved the intermittent HR issue related to not passing the cross team check gate
Copilot AI balanced review requested due to automatic review settings October 6, 2026 11:50

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot review overview

🔵 Needs a closer look

The reported App installation path bypasses the updated dependency files, leaving the intended urllib3 version unenforced.

Review effort: Balanced
Findings: None

Resolved since last review (1)

@Prajwal-Microsoft
Prajwal-Microsoft merged commit 6cd9635 into main Oct 7, 2026
13 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.

4 participants