Skip to content

fix(deps): resolve Dependabot vulnerabilities - #3

Merged
zhanghanduo merged 3 commits into
mainfrom
fix/dependabot-vuls
Aug 24, 2026
Merged

fix(deps): resolve Dependabot vulnerabilities#3
zhanghanduo merged 3 commits into
mainfrom
fix/dependabot-vuls

Conversation

@dq-ai-dev

Copy link
Copy Markdown
Collaborator

Summary

  • refresh the complete dependency lock across all optional extras
  • enforce patched-version floors for direct and transitive vulnerable packages
  • keep the Anthropic Bedrock request override typed across the httpx to httpx2 transport migration
  • resolve all 79 currently reported Dependabot alerts when merged

Verification

  • 1,702 tests passed
  • Ruff passed
  • Pyright passed with 0 errors
  • framework and evaluation import smoke tests passed
  • symbol and lazy-export checks passed
  • Docker configuration validation passed
  • uv pip check reports all 159 installed packages compatible
  • pip-audit reports no known vulnerabilities

@zhanghanduo zhanghanduo left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Thanks for putting together the dependency security refresh. I found one blocking compatibility regression and one coverage gap:

  1. Blocking: uv.lock upgrades openai from 2.29.0 to 2.54.0, but the current config/bootstrap path can pass a cached empty api_key directly to AsyncOpenAI. The lint-and-smoke job reproduces this in test_benchmark_session_bootstraps_team_runtime: OpenAIError: Missing credentials (1701 passed, 1 failed). Please either make the runtime compatible with the new SDK (for example, normalize an empty key to None and/or reload config when environment values change) or retain a compatible OpenAI version, then rerun the full suite. A regression test for the cached-empty-key -> env-set-later path would be valuable.

  2. Non-blocking coverage: the Anthropic 1.0 / httpx2 Bedrock override has no focused test. Please add a unit test that exercises _prepare_request and verifies that Authorization: Bearer is injected with the locked Anthropic transport.

The branch also currently conflicts with main in uv.lock, so the lock should be regenerated after rebasing and CI rerun.

@dq-ai-dev

Copy link
Copy Markdown
Collaborator Author

Addressed both review findings in fe1946f:

  • OpenAIClient now normalizes an empty API key to None, preserving the OpenAI 2.54 environment fallback when config was cached before OPENAI_API_KEY was set.
  • The benchmark bootstrap test now deterministically covers that cached-empty-key ordering.
  • Added a focused Anthropic Bedrock test using the locked httpx2 transport and verifying Authorization: Bearer injection.
  • Synced the branch after main was merged; the lockfile is consistent.

Verification: 1,703 tests passed; Ruff passed; Pyright passed with 0 errors; both import-smoke stages, symbol closure, and lazy-export checks passed.

@dq-ai-dev
dq-ai-dev requested a review from zhanghanduo August 24, 2026 09:56

@zhanghanduo zhanghanduo left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

LGTM

@zhanghanduo
zhanghanduo merged commit 1fccae6 into main Aug 24, 2026
4 checks passed
@zhanghanduo
zhanghanduo deleted the fix/dependabot-vuls branch August 24, 2026 10:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

2 participants