Skip to content

fix(deps): resolve 13 Dependabot security alerts#1284

Merged
jeremyeder merged 2 commits intoambient-code:mainfrom
jeremyeder:deps/security-fixes
Apr 10, 2026
Merged

fix(deps): resolve 13 Dependabot security alerts#1284
jeremyeder merged 2 commits intoambient-code:mainfrom
jeremyeder:deps/security-fixes

Conversation

@jeremyeder
Copy link
Copy Markdown
Contributor

@jeremyeder jeremyeder commented Apr 10, 2026

Summary

Remaining alerts (3)

Alerts #144, #145, #146 (fastmcp 2.14.3 → 3.2.0) are blocked on upstream: mcp-atlassian 0.21.1 pins fastmcp<2.15.0,>=2.13.0. The CVEs affect fastmcp's OpenAPI provider and OAuth proxy — not in our code path, but the alerts will stay open until mcp-atlassian releases a version compatible with fastmcp 3.x.

Test plan

  • Frontend: 614 tests pass (npx vitest run)
  • Runner: 543 tests pass (uv run pytest tests/)
  • CI passes

🤖 Generated with Claude Code

Summary by CodeRabbit

  • Chores

    • Bumped frontend and runtime dependency minimums to newer patch releases.
  • Refactor

    • Code formatting and parameter/layout reflows across several modules for readability.
  • Tests

    • Cleaned up and reformatted unit tests, removing unused imports and improving fixture readability.

@coderabbitai
Copy link
Copy Markdown
Contributor

coderabbitai bot commented Apr 10, 2026

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

Run ID: aea36975-5fa5-4f0d-a358-3c343520ea87

📥 Commits

Reviewing files that changed from the base of the PR and between a83af40 and 9197987.

⛔ Files ignored due to path filters (2)
  • components/frontend/package-lock.json is excluded by !**/package-lock.json, !**/package-lock.json
  • components/runners/ambient-runner/uv.lock is excluded by !**/*.lock
📒 Files selected for processing (7)
  • components/frontend/package.json
  • components/runners/ambient-runner/ambient_runner/endpoints/model.py
  • components/runners/ambient-runner/ambient_runner/mlflow_observability.py
  • components/runners/ambient-runner/ambient_runner/observability.py
  • components/runners/ambient-runner/pyproject.toml
  • components/runners/ambient-runner/tests/test_mcp_config.py
  • scripts/generate-loading-tips.py
✅ Files skipped from review due to trivial changes (5)
  • components/frontend/package.json
  • components/runners/ambient-runner/ambient_runner/endpoints/model.py
  • components/runners/ambient-runner/ambient_runner/observability.py
  • components/runners/ambient-runner/ambient_runner/mlflow_observability.py
  • components/runners/ambient-runner/tests/test_mcp_config.py
🚧 Files skipped from review as they are similar to previous changes (1)
  • components/runners/ambient-runner/pyproject.toml

📝 Walkthrough

Walkthrough

Multiple formatting and minor refactor changes across the ambient-runner and scripts, plus two patch dependency bumps: next from 16.2.2→16.2.3 and aiohttp constraint from >=3.13.3→>=3.13.4. No behavioral or API changes introduced.

Changes

Cohort / File(s) Summary
Frontend Dependencies
components/frontend/package.json
Bumped next from 16.2.2 to 16.2.3 (patch update).
Runner Dependencies
components/runners/ambient-runner/pyproject.toml
Raised aiohttp minimum constraint from >=3.13.3 to >=3.13.4.
Ambient Runner — formatting/refactor
components/runners/ambient-runner/ambient_runner/endpoints/model.py, components/runners/ambient-runner/ambient_runner/mlflow_observability.py, components/runners/ambient-runner/ambient_runner/observability.py
Signature and expression reformatting, multi-line parameter/tuple assignments, and import/alias reorderings only; no logic or API changes.
Ambient Runner — tests
components/runners/ambient-runner/tests/test_mcp_config.py
Removed unused imports and reformatted test JSON fixtures to multi-line structures; test logic and assertions unchanged.
Scripts — formatting
scripts/generate-loading-tips.py
Refactored function signatures and reformatted subprocess/print invocations and list literals to multi-line forms; no behavioral changes.
🚥 Pre-merge checks | ✅ 5 | ❌ 1

❌ Failed checks (1 warning)

Check name Status Explanation Resolution
Docstring Coverage ⚠️ Warning Docstring coverage is 71.43% which is insufficient. The required threshold is 80.00%. Write docstrings for the functions missing them to satisfy the coverage threshold.
✅ Passed checks (5 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed Title follows Conventional Commits format with type(scope): description pattern and accurately describes the main change—dependency updates addressing security alerts.
Performance And Algorithmic Complexity ✅ Passed PR introduces zero algorithmic changes. All modifications are formatting adjustments, import reordering, and dependency updates. Existing loops remain unchanged.
Security And Secret Handling ✅ Passed Security-focused PR updates dependencies (next 16.2.3, aiohttp ≥3.13.4) addressing 13 CVEs with no hardcoded secrets, safe subprocess calls, path validation, and proper sensitive data masking.
Kubernetes Resource Safety ✅ Passed Pull request modifies only dependency files and Python code; no Kubernetes resource manifests are changed.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
✨ Simplify code
  • Create PR with simplified code

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

Ambient Code Bot and others added 2 commits April 10, 2026 14:58
Bump next 16.2.2 → 16.2.3 (CVE: DoS via Server Components)
Bump aiohttp >=3.13.3 → >=3.13.4 (9 CVEs: header injection, SSRF, DoS)
Bump cryptography 46.0.5 → 46.0.7 (buffer overflow)
Bump lupa 2.6 → 2.7 (sandbox escape / RCE)

3 fastmcp alerts remain open — mcp-atlassian pins fastmcp<2.15.0,
blocking upgrade to 3.2.0. Waiting on upstream release.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Move import above module-level assignment to fix E402, apply ruff
formatting fixes to upstream files.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@jeremyeder jeremyeder force-pushed the deps/security-fixes branch from a83af40 to 9197987 Compare April 10, 2026 19:00
@jeremyeder jeremyeder disabled auto-merge April 10, 2026 19:05
@jeremyeder jeremyeder merged commit 2389565 into ambient-code:main Apr 10, 2026
29 of 31 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.

1 participant