Skip to content

fix(deps): bump typer-slim to match typer in uv.lock#169

Merged
georgeh0 merged 1 commit into
mainfrom
g/fix-typer-slim-lock-mismatch
May 15, 2026
Merged

fix(deps): bump typer-slim to match typer in uv.lock#169
georgeh0 merged 1 commit into
mainfrom
g/fix-typer-slim-lock-mismatch

Conversation

@georgeh0
Copy link
Copy Markdown
Member

Summary

  • Bump typer-slim from 0.21.1 to 0.24.0 in uv.lock. The mismatch with typer==0.24.1 caused install-order-dependent file clobbering in shared typer/ namespace, which manifested on pre-commit (windows-latest, 3.11) as ImportError: cannot import name 'HAS_SHELLINGHAM' from 'typer.core' during pytest collection (see PR docs(readme): document OpenAI-compatible endpoint configuration #168 CI).
  • Since typer-slim>=0.22.0 is a meta-package that just depends on typer (no typer/*.py of its own), the namespace collision goes away.

Test plan

  • CI must turn green on pre-commit (windows-latest, 3.11) again — this is the failure mode the change fixes.
  • Locally verified: fresh uv sync from scratch installs cleanly, from cocoindex_code import cli succeeds, and tests/test_cli_helpers.py + tests/test_e2e.py (49 tests) pass on macOS / Python 3.14.4.

The lockfile pinned typer==0.24.1 alongside typer-slim==0.21.1, two
versions that both ship into the `typer/` namespace and overwrite each
other on install. When the install order leaves typer-slim's stale
`_completion_shared.py` next to typer's new `core.py`, importing typer
fails with

  ImportError: cannot import name 'HAS_SHELLINGHAM' from 'typer.core'

which is what broke pre-commit (windows-latest, 3.11) on PR #168.

Since typer-slim 0.22.0 it's been a meta-package that just depends on
`typer` and ships no `typer/*.py` of its own, so upgrading typer-slim
to 0.24.0 removes the conflict.
@georgeh0 georgeh0 merged commit 4392229 into main May 15, 2026
4 checks passed
@georgeh0 georgeh0 deleted the g/fix-typer-slim-lock-mismatch branch May 15, 2026 18:54
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