Skip to content

refactor(models): consolidate artifact and dependency handling - #116

Merged
tulayha merged 2 commits into
mainfrom
refactor/consolidate-model-handling
Aug 13, 2026
Merged

refactor(models): consolidate artifact and dependency handling#116
tulayha merged 2 commits into
mainfrom
refactor/consolidate-model-handling

Conversation

@tulayha

@tulayha tulayha commented Aug 13, 2026

Copy link
Copy Markdown
Collaborator

Summary

  • Centralize retry, backoff, error translation, checksum verification, and preparation progress used by model providers.
  • Make model readiness require the pinned checksum instead of accepting any file at the expected path.
  • Update the locked GitPython version from 3.1.57 to 3.1.58 for its security fixes.
  • Keep provider-specific model constructors inside their capabilities.

This is internal-only. It does not change public contracts and requires no model, index, or repository rebuild.

Validation

  • uv run --no-sync ruff check .
  • uv run --no-sync pytest -q tests/test_models.py — 30 passed, 2 subtests passed.
  • $env:PYTHONPATH='src'; uv run --no-sync python -m unittest discover -s tests -q — 612 passed, 4 skipped with GitPython 3.1.58 installed.
  • uv lock --check

@tulayha tulayha changed the title fix(models): consolidate artifact handling and update GitPython refactor(models): consolidate artifact and dependency handling Aug 13, 2026
@tulayha
tulayha marked this pull request as ready for review August 13, 2026 08:50
@tulayha
tulayha merged commit 59463dc into main Aug 13, 2026
12 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