Skip to content

[fix] Resolve all open vulnerabilities#4283

Merged
junaway merged 21 commits intorelease/v0.99.2from
chore/fix-vulnerabilities-20260506
May 6, 2026
Merged

[fix] Resolve all open vulnerabilities#4283
junaway merged 21 commits intorelease/v0.99.2from
chore/fix-vulnerabilities-20260506

Conversation

@junaway
Copy link
Copy Markdown
Contributor

@junaway junaway commented May 6, 2026

No description provided.

Copilot AI review requested due to automatic review settings May 6, 2026 14:15
@vercel
Copy link
Copy Markdown

vercel Bot commented May 6, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
agenta-documentation Ready Ready Preview, Comment May 6, 2026 2:42pm

Request Review

@dosubot dosubot Bot added the size:S This PR changes 10-29 lines, ignoring generated files. label May 6, 2026
@junaway junaway changed the base branch from main to chore/bump-py-deps-20260506 May 6, 2026 14:15
@coderabbitai
Copy link
Copy Markdown

coderabbitai Bot commented May 6, 2026

Important

Review skipped

Auto reviews are disabled on base/target branches other than the default branch.

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro Plus

Run ID: d7a45b56-0b79-4f2e-a34a-eace4c377370

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • 🔍 Trigger review
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch chore/fix-vulnerabilities-20260506

Comment @coderabbitai help to get the list of available commands and usage tips.

@junaway junaway changed the base branch from chore/bump-py-deps-20260506 to main May 6, 2026 14:15
@junaway junaway changed the title [chore] Resolve all open vulnerabilities [fix] Resolve all open vulnerabilities May 6, 2026
@dosubot dosubot Bot added the dependencies Dependencies label May 6, 2026
@junaway junaway changed the base branch from main to release/v0.99.2 May 6, 2026 14:19
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

This PR focuses on addressing reported vulnerabilities by updating dependency constraints and lockfiles across the web (pnpm) and Python (uv) parts of the repo, plus tightening a few npm/pnpm override rules.

Changes:

  • Updated web dependencies/overrides (notably axios, next, postcss, uuid) and regenerated web/pnpm-lock.yaml.
  • Updated Python dependency pins/locks (notably agenta, daytona, pytest, pytest-asyncio, OTel libs) across api/, services/, and sdk/.
  • Removed some legacy test requirements files and example lockfiles, and adjusted GitHub Actions cache keys accordingly.

Reviewed changes

Copilot reviewed 19 out of 28 changed files in this pull request and generated 4 comments.

Show a summary per file
File Description
web/pnpm-lock.yaml Regenerated pnpm lockfile with updated overrides and resolved versions (axios/next/postcss/uuid, etc.).
web/packages/agenta-ui/package.json Tightened uuid dependency range to ^11.1.1.
web/packages/agenta-shared/package.json Bumped uuid to ^11.1.1.
web/packages/agenta-playground/package.json Bumped uuid to ^11.1.1.
web/packages/agenta-playground-ui/package.json Bumped uuid to ^11.1.1.
web/packages/agenta-entity-ui/package.json Tightened uuid dependency range to ^11.1.1.
web/packages/agenta-entities/package.json Tightened uuid dependency range to ^11.1.1.
web/package.json Added overrides for postcss and next, bumped axios override.
web/oss/package.json Bumped next and uuid versions for OSS web app.
web/ee/package.json Bumped uuid for EE web app.
services/uv.lock Updated Python lockfile with newer agenta and related deps.
services/pyproject.toml Bumped agenta, pytest, pytest-asyncio version ranges.
sdk/uv.lock Updated SDK Python lockfile; bumped daytona, pytest, pytest-asyncio.
sdk/pyproject.toml Bumped daytona, pytest, pytest-asyncio version ranges.
sdk/oss/tests/requirements.txt Removed legacy test requirements file.
sdk/oss/tests/legacy/new_tests/requirements.test.txt Updated legacy test requirements (pytest-asyncio, python-dotenv).
examples/node/observability-vercel-ai/package-lock.json Removed example npm lockfile.
examples/node/observability-opentelemetry/pnpm-lock.yaml Removed example pnpm lockfile.
docs/package.json Added deps/overrides for vulnerable transitive packages (lodash/postcss/follow-redirects).
api/uv.lock Updated API Python lockfile (agenta/pytest/pytest-asyncio/otel, etc.).
api/pyproject.toml Bumped agenta, pytest, pytest-asyncio version ranges.
api/oss/tests/requirements.txt Removed legacy test requirements file.
api/oss/tests/legacy/requirements.test.txt Updated legacy test requirements (pytest-asyncio, python-dotenv).
.github/workflows/44-railway-tests.yml Updated uv cache keys (but workflow still references removed requirements files).
.github/workflows/12-check-unit-tests.yml Updated uv cache keys to stop hashing removed requirements files.
Files not reviewed (4)
  • docs/pnpm-lock.yaml: Language not supported
  • examples/node/observability-opentelemetry/pnpm-lock.yaml: Language not supported
  • examples/node/observability-vercel-ai/package-lock.json: Language not supported
  • web/pnpm-lock.yaml: Language not supported

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

Comment thread web/pnpm-lock.yaml Outdated
Comment thread web/pnpm-lock.yaml Outdated
Comment thread .github/workflows/44-railway-tests.yml
Comment thread .github/workflows/44-railway-tests.yml
Copy link
Copy Markdown

@coderabbitai coderabbitai Bot left a comment

Choose a reason for hiding this comment

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

Actionable comments posted: 1

🧹 Nitpick comments (1)
web/packages/agenta-entity-ui/package.json (1)

32-32: 💤 Low value

Update @types/uuid from ^9.0.8 to ^11.0.0.

The uuid dependency at line 32 is now at ^11.1.1, but @types/uuid at line 55 remains at ^9.0.8. Version 11.0.0 of @types/uuid is now available and should be updated to keep type definitions aligned with the runtime library version.


ℹ️ Review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro Plus

Run ID: 5a02c3f2-534c-47ca-86bb-1e113beddeda

📥 Commits

Reviewing files that changed from the base of the PR and between a0f386e and f7cb66e.

⛔ Files ignored due to path filters (9)
  • api/uv.lock is excluded by !**/*.lock
  • docs/package-lock.json is excluded by !**/package-lock.json
  • docs/pnpm-lock.yaml is excluded by !**/pnpm-lock.yaml
  • examples/node/observability-opentelemetry/pnpm-lock.yaml is excluded by !**/pnpm-lock.yaml
  • examples/node/observability-vercel-ai/package-lock.json is excluded by !**/package-lock.json
  • examples/python/RAG_QA_chatbot/frontend/pnpm-lock.yaml is excluded by !**/pnpm-lock.yaml
  • sdk/uv.lock is excluded by !**/*.lock
  • services/uv.lock is excluded by !**/*.lock
  • web/pnpm-lock.yaml is excluded by !**/pnpm-lock.yaml
📒 Files selected for processing (19)
  • .github/workflows/12-check-unit-tests.yml
  • .github/workflows/44-railway-tests.yml
  • api/oss/tests/legacy/requirements.test.txt
  • api/oss/tests/requirements.txt
  • api/pyproject.toml
  • docs/package.json
  • sdk/oss/tests/legacy/new_tests/requirements.test.txt
  • sdk/oss/tests/requirements.txt
  • sdk/pyproject.toml
  • services/pyproject.toml
  • web/ee/package.json
  • web/oss/package.json
  • web/package.json
  • web/packages/agenta-entities/package.json
  • web/packages/agenta-entity-ui/package.json
  • web/packages/agenta-playground-ui/package.json
  • web/packages/agenta-playground/package.json
  • web/packages/agenta-shared/package.json
  • web/packages/agenta-ui/package.json
💤 Files with no reviewable changes (2)
  • api/oss/tests/requirements.txt
  • sdk/oss/tests/requirements.txt

Comment thread api/oss/tests/legacy/requirements.test.txt Outdated
@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented May 6, 2026

Railway Preview Environment

Status Destroyed (PR closed)

Updated at 2026-05-06T14:41:42.255Z

Copilot AI review requested due to automatic review settings May 6, 2026 14:40
@junaway junaway merged commit 15faddf into release/v0.99.2 May 6, 2026
11 of 12 checks passed
@coderabbitai coderabbitai Bot mentioned this pull request May 6, 2026
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 19 out of 28 changed files in this pull request and generated 1 comment.

Files not reviewed (4)
  • docs/pnpm-lock.yaml: Language not supported
  • examples/node/observability-opentelemetry/pnpm-lock.yaml: Language not supported
  • examples/node/observability-vercel-ai/package-lock.json: Language not supported
  • web/pnpm-lock.yaml: Language not supported
Comments suppressed due to low confidence (2)

sdk/pyproject.toml:58

  • sdk/run-tests.py imports from dotenv import load_dotenv, but python-dotenv is not declared in this project's dependency-groups. The test runner currently relies on a transitive install (e.g., via daytona), which is fragile and can break if upstream deps change. Add python-dotenv>=1,<2 to the dev dependency group (or to dependencies if required at runtime) so uv sync installs it explicitly.
[dependency-groups]
dev = [
    "posthog>=7,<8",
    "questionary>=2,<3",
    "setuptools>=80,<81",
    "uvicorn>=0.38,<0.39",
    "requests>=2,<3",
    "pexpect>=4,<5",
    "boto3>=1,<2",
    "click>=8,<9",
    "pytest>=9,<10",
    "pytest-asyncio>=1,<2",
    "pytest-xdist>=3,<4",
    "pytest-mock>=3,<4",
    "pytest-html>=4,<5",
]

services/pyproject.toml:31

  • services/run-tests.py imports from dotenv import load_dotenv, but python-dotenv is not declared in the dev dependency group. CI currently works only because python-dotenv happens to be pulled in transitively; please add python-dotenv>=1,<2 to the dev group to make the test runner's dependencies explicit.
[dependency-groups]
dev = [
    "watchdog[watchmedo]>=3,<4",
    "requests>=2,<3",
    "click>=8,<9",
    "pytest>=9,<10",
    "pytest-asyncio>=1,<2",
    "pytest-xdist>=3,<4",
    "pytest-mock>=3,<4",
    "pytest-html>=4,<5",
]

Comment on lines 351 to 355
pip install "uv==0.11.9"
cd api
uv sync --locked --python 3.11
uv pip install --python .venv/bin/python -r oss/tests/requirements.txt --editable ../sdk/
uv pip install --python .venv/bin/python --editable ../sdk/

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

dependencies Dependencies size:S This PR changes 10-29 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants