Skip to content
  • stuxf's avatar
    a6c30b30
    build: migrate packaging, CI, and Docker from Poetry to uv (#25007) · a6c30b30
    stuxf authored
    
    
    * build: migrate packaging metadata to uv
    
    * ci: move automation and local tooling to uv
    
    * docker: migrate image builds and runtime setup to uv
    
    * docs: update install and deployment guidance for uv
    
    * chore: align auxiliary scripts and tests with uv
    
    * test: harden test_litellm isolation
    
    * fix: keep release and health check images self-contained
    
    * build: pin uv tooling and health check deps
    
    * test: isolate bedrock image request formatting from suite state
    
    * test: cover sandbox executor requirements flow
    
    * ci: fix circleci no-op command steps
    
    * ci: fix circleci publish workflow parsing
    
    * fix: stabilize remaining uv migration CI checks
    
    * ci: increase matrix test timeout headroom
    
    * fix: restore published docker and license coverage
    
    * fix: restore proxy runtime build parity
    
    * fix: restore proxy extras parity and venv migrations
    
    * ci: persist uv path across circleci steps
    
    * fix: keep psycopg binary in default test env
    
    * docker: preserve prisma cache across stages
    
    * test: run local proxy checks through uv python
    
    * build: restore runtime deps moved into ci
    
    * build: refresh uv lock after upstream merge
    
    * fix: restore module import in test_check_migration after merge
    
    The conflict resolution imported only the function but the test body
    references check_migration as a module throughout.
    
    Co-Authored-By: default avatarClaude Opus 4.6 (1M context) <noreply@anthropic.com>
    
    * fix: revert dependency promotions, remove nodejs-wheel-binaries, fix Docker layer caching
    
    - Move google-generativeai, Pillow, tenacity back to ci group (they are
      lazily imported and bloat the base SDK install needlessly)
    - Remove nodejs-wheel-binaries from extra_proxy and proxy-dev (redundant
      in Docker where system Node.js is already installed via apk)
    - Remove all nodejs-wheel node replacement and venv npm patching blocks
      from Dockerfiles since the wheel is no longer installed
    - Add --no-default-groups to CodSpeed benchmark workflow so the benchmark
      environment matches the old minimal pip install footprint
    - Apply standard uv two-phase Docker pattern: copy metadata first, install
      deps (cached layer), then copy source and install project
    - Replace CircleCI enterprise no-op with proper uv sync command
    
    Co-Authored-By: default avatarClaude Opus 4.6 (1M context) <noreply@anthropic.com>
    
    * chore: regenerate uv.lock after removing nodejs-wheel-binaries
    
    Co-Authored-By: default avatarClaude Opus 4.6 (1M context) <noreply@anthropic.com>
    
    * fix(ci): use cache/restore instead of cache to prevent cache poisoning
    
    The old workflow used actions/cache/restore (read-only). The uv migration
    changed it to actions/cache (read-write), which zizmor flags as a cache
    poisoning risk. Restore the safer read-only variant.
    
    Co-Authored-By: default avatarClaude Opus 4.6 (1M context) <noreply@anthropic.com>
    
    * fix(ci): disable setup-uv built-in cache to silence cache-poisoning alert
    
    The setup-uv action enables caching by default, which zizmor flags as a
    cache poisoning risk. Disable it since we already use a read-only
    cache/restore step.
    
    Co-Authored-By: default avatarClaude Opus 4.6 (1M context) <noreply@anthropic.com>
    
    * fix(ci): disable setup-uv cache in publish workflow
    
    Silences zizmor cache-poisoning alert. Publishing workflow runs
    infrequently on protected branches so caching adds no real benefit.
    
    Co-Authored-By: default avatarClaude Opus 4.6 (1M context) <noreply@anthropic.com>
    
    * fix(test): remove duplicate verbose_logger mock in test_check_migration
    
    The logger was patched twice — first via mocker.patch() then via
    mocker.patch.object(autospec=True). The second call fails because
    autospec cannot inspect an already-mocked attribute. Remove the
    redundant first patch.
    
    Co-Authored-By: default avatarClaude Opus 4.6 (1M context) <noreply@anthropic.com>
    
    * fix(ci): free disk space before Docker build in test-server-root-path
    
    The Dockerfile.non_root build ran out of disk on the CI runner. Remove
    Android SDK, .NET, Boost, and GHC toolchains (~12GB) to free space.
    
    Co-Authored-By: default avatarClaude Opus 4.6 (1M context) <noreply@anthropic.com>
    
    ---------
    
    Co-authored-by: default avatarClaude Opus 4.6 (1M context) <noreply@anthropic.com>
    a6c30b30
    build: migrate packaging, CI, and Docker from Poetry to uv (#25007)
    stuxf authored
    
    
    * build: migrate packaging metadata to uv
    
    * ci: move automation and local tooling to uv
    
    * docker: migrate image builds and runtime setup to uv
    
    * docs: update install and deployment guidance for uv
    
    * chore: align auxiliary scripts and tests with uv
    
    * test: harden test_litellm isolation
    
    * fix: keep release and health check images self-contained
    
    * build: pin uv tooling and health check deps
    
    * test: isolate bedrock image request formatting from suite state
    
    * test: cover sandbox executor requirements flow
    
    * ci: fix circleci no-op command steps
    
    * ci: fix circleci publish workflow parsing
    
    * fix: stabilize remaining uv migration CI checks
    
    * ci: increase matrix test timeout headroom
    
    * fix: restore published docker and license coverage
    
    * fix: restore proxy runtime build parity
    
    * fix: restore proxy extras parity and venv migrations
    
    * ci: persist uv path across circleci steps
    
    * fix: keep psycopg binary in default test env
    
    * docker: preserve prisma cache across stages
    
    * test: run local proxy checks through uv python
    
    * build: restore runtime deps moved into ci
    
    * build: refresh uv lock after upstream merge
    
    * fix: restore module import in test_check_migration after merge
    
    The conflict resolution imported only the function but the test body
    references check_migration as a module throughout.
    
    Co-Authored-By: default avatarClaude Opus 4.6 (1M context) <noreply@anthropic.com>
    
    * fix: revert dependency promotions, remove nodejs-wheel-binaries, fix Docker layer caching
    
    - Move google-generativeai, Pillow, tenacity back to ci group (they are
      lazily imported and bloat the base SDK install needlessly)
    - Remove nodejs-wheel-binaries from extra_proxy and proxy-dev (redundant
      in Docker where system Node.js is already installed via apk)
    - Remove all nodejs-wheel node replacement and venv npm patching blocks
      from Dockerfiles since the wheel is no longer installed
    - Add --no-default-groups to CodSpeed benchmark workflow so the benchmark
      environment matches the old minimal pip install footprint
    - Apply standard uv two-phase Docker pattern: copy metadata first, install
      deps (cached layer), then copy source and install project
    - Replace CircleCI enterprise no-op with proper uv sync command
    
    Co-Authored-By: default avatarClaude Opus 4.6 (1M context) <noreply@anthropic.com>
    
    * chore: regenerate uv.lock after removing nodejs-wheel-binaries
    
    Co-Authored-By: default avatarClaude Opus 4.6 (1M context) <noreply@anthropic.com>
    
    * fix(ci): use cache/restore instead of cache to prevent cache poisoning
    
    The old workflow used actions/cache/restore (read-only). The uv migration
    changed it to actions/cache (read-write), which zizmor flags as a cache
    poisoning risk. Restore the safer read-only variant.
    
    Co-Authored-By: default avatarClaude Opus 4.6 (1M context) <noreply@anthropic.com>
    
    * fix(ci): disable setup-uv built-in cache to silence cache-poisoning alert
    
    The setup-uv action enables caching by default, which zizmor flags as a
    cache poisoning risk. Disable it since we already use a read-only
    cache/restore step.
    
    Co-Authored-By: default avatarClaude Opus 4.6 (1M context) <noreply@anthropic.com>
    
    * fix(ci): disable setup-uv cache in publish workflow
    
    Silences zizmor cache-poisoning alert. Publishing workflow runs
    infrequently on protected branches so caching adds no real benefit.
    
    Co-Authored-By: default avatarClaude Opus 4.6 (1M context) <noreply@anthropic.com>
    
    * fix(test): remove duplicate verbose_logger mock in test_check_migration
    
    The logger was patched twice — first via mocker.patch() then via
    mocker.patch.object(autospec=True). The second call fails because
    autospec cannot inspect an already-mocked attribute. Remove the
    redundant first patch.
    
    Co-Authored-By: default avatarClaude Opus 4.6 (1M context) <noreply@anthropic.com>
    
    * fix(ci): free disk space before Docker build in test-server-root-path
    
    The Dockerfile.non_root build ran out of disk on the CI runner. Remove
    Android SDK, .NET, Boost, and GHC toolchains (~12GB) to free space.
    
    Co-Authored-By: default avatarClaude Opus 4.6 (1M context) <noreply@anthropic.com>
    
    ---------
    
    Co-authored-by: default avatarClaude Opus 4.6 (1M context) <noreply@anthropic.com>
After you've reviewed these contribution guidelines, you'll be all set to contribute to this project.
Loading