Skip to content

fix: dependabot package upgrades#554

Merged
Roopan-Microsoft merged 9 commits intodevfrom
feature/dependabot-package-upgrades
Apr 22, 2026
Merged

fix: dependabot package upgrades#554
Roopan-Microsoft merged 9 commits intodevfrom
feature/dependabot-package-upgrades

Conversation

@Ayaz-Microsoft
Copy link
Copy Markdown
Contributor

Purpose

  • Current Vulnerabilities count: High-4, Moderate-23

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

…npm lockfile

- Upgrade pydantic 2.12.5 -> 2.13.3 and pydantic-core 2.45.0 -> 2.46.3
  to fix incompatible pinned versions (pydantic 2.12.5 requires pydantic-core==2.41.5)
- Regenerate pnpm-lock.yaml to match updated package.json dependencies

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
@github-actions
Copy link
Copy Markdown

github-actions Bot commented Apr 21, 2026

Coverage

Coverage Report •
FileStmtsMissCoverMissing
TOTAL121016186% 
report-only-changed-files is enabled. No files were changed during this commit :)

Tests Skipped Failures Errors Time
242 0 💤 0 ❌ 0 🔥 3.627s ⏱️

- Add eslint@^9.0.0 as devDependency to fix 'Cannot find module eslint' build error
- Revert @eslint/js from ^10.0.1 to ^9.39.2 for compatibility with react-scripts 5.x
- Regenerate pnpm-lock.yaml

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Ayaz-Microsoft and others added 2 commits April 21, 2026 12:08
react-scripts 5.0.1 uses eslint-webpack-plugin@3.2.0 which passes
extensions and resolvePluginsRelativeTo options removed in eslint 9.
Using eslint 8.57.x which still supports these options.

Tested locally - build succeeds.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Resolved 9 conflicting files by keeping the latest version of each package:
- pydantic 2.13.3, pydantic-core 2.46.3 (from PR branch, newer)
- pillow 12.2.0 (from PR branch, newer)
- coverage 7.13.5, pytest-cov 7.1.0 (from dev, newer)
- pydantic-settings 2.13.1, starlette 1.0.0, uvicorn 0.42.0 (from dev, newer)
- Regenerated uv.lock files with uv lock
- Regenerated pnpm-lock.yaml with pnpm install

Tested locally - frontend build succeeds.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Regenerated uv.lock files for ContentProcessor and ContentProcessorAPI
to align with pydantic 2.13.3 after new commits landed on dev.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Updates dependency pins/lockfiles to address reported vulnerabilities across the Web (pnpm) and Python services (ContentProcessor + ContentProcessorAPI).

Changes:

  • Bump MSAL packages and several frontend transitive overrides (including follow-redirects) in the web app lockfile.
  • Adjust web dev tooling dependencies (notably @eslint/js) and add an eslint devDependency entry.
  • Update Python dependency pins (e.g., pydantic, pydantic-core, python-multipart, pillow, pytest) in both requirements.txt and pyproject.toml where applicable.

Reviewed changes

Copilot reviewed 5 out of 8 changed files in this pull request and generated no comments.

Show a summary per file
File Description
src/ContentProcessorWeb/pnpm-lock.yaml Locks updated frontend dependency graph (MSAL, ESLint-related packages, follow-redirects).
src/ContentProcessorWeb/package.json Updates frontend dev tooling deps and pnpm overrides to align with vulnerability remediation.
src/ContentProcessorAPI/requirements.txt Bumps pinned Python package versions to newer patched releases.
src/ContentProcessorAPI/pyproject.toml Updates pinned Python dependencies for the API service to match newer versions.
src/ContentProcessor/requirements.txt Bumps pinned Python package versions (incl. pillow, pydantic*, pytest).
src/ContentProcessor/pyproject.toml Updates pinned Python dependencies/dev-deps for the ContentProcessor service.
Files not reviewed (1)
  • src/ContentProcessorWeb/pnpm-lock.yaml: Language not supported
Comments suppressed due to low confidence (1)

src/ContentProcessorWeb/package.json:70

  • devDependencies contains two eslint entries (one newly added near the top and another existing later). Duplicate keys in package.json are invalid JSON and may lead to confusing/unstable dependency installs depending on the parser. Remove one of the eslint entries and keep a single version declaration.
  "devDependencies": {
    "@eslint/js": "^9.39.2",
    "eslint": "^8.57.0",
    "@testing-library/jest-dom": "^6.9.1",
    "@testing-library/react": "^16.3.2",
    "@testing-library/user-event": "^14.6.1",
    "@types/jest": "^30.0.0",
    "@types/node": "^22.15.0",
    "@types/react": "^19.2.14",
    "@types/react-dom": "^19.2.3",

    "@types/react-virtualized": "^9.22.3",
    "@types/react-window": "^2.0.0",
    "customize-cra": "^1.0.0",
    "eslint": "^8.57.0",
    "gh-pages": "^6.3.0",

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

Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Copilot reviewed 5 out of 8 changed files in this pull request and generated 1 comment.

Files not reviewed (1)
  • src/ContentProcessorWeb/pnpm-lock.yaml: Language not supported

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

Comment thread src/ContentProcessorWeb/package.json
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
@Roopan-Microsoft Roopan-Microsoft merged commit 44e0714 into dev Apr 22, 2026
6 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.

3 participants