Skip to content

ci: update twelvelabs requirement from <1.0.0,>=0.4.0 to >=0.4.0,<2.0.0 - #560

Open
dependabot[bot] wants to merge 1 commit into
mainfrom
dependabot/pip/twelvelabs-gte-0.4.0-and-lt-2.0.0
Open

dependabot[bot] wants to merge 1 commit into
mainfrom
dependabot/pip/twelvelabs-gte-0.4.0-and-lt-2.0.0

Conversation

@dependabot

@dependabot dependabot Bot commented on behalf of github Aug 3, 2026

Copy link
Copy Markdown
Contributor

Updates the requirements on twelvelabs to permit the latest version.

@dependabot dependabot Bot added dependencies Pull requests that update a dependency file python Pull requests that update python code labels Aug 3, 2026
@dependabot
dependabot Bot requested a review from a team as a code owner August 3, 2026 13:45
@dependabot
dependabot Bot requested a review from chaynabors August 3, 2026 13:45
@dependabot dependabot Bot added dependencies Pull requests that update a dependency file python Pull requests that update python code labels Aug 3, 2026
@dependabot
dependabot Bot force-pushed the dependabot/pip/twelvelabs-gte-0.4.0-and-lt-2.0.0 branch from 92e418b to 72e1e86 Compare August 3, 2026 15:29
Updates the requirements on twelvelabs to permit the latest version.

---
updated-dependencies:
- dependency-name: twelvelabs
  dependency-version: 1.3.1
  dependency-type: direct:development
...

Signed-off-by: dependabot[bot] <support@github.com>
@dependabot
dependabot Bot force-pushed the dependabot/pip/twelvelabs-gte-0.4.0-and-lt-2.0.0 branch from 72e1e86 to 7326990 Compare August 19, 2026 23:20
@yonib05
yonib05 requested review from mehtarac and removed request for chaynabors August 23, 2026 15:36

@mehtarac mehtarac left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

@strandly-the-agent is this safe to merge?

@strandly-the-agent

Copy link
Copy Markdown
Contributor

mehtarac Not safe to merge as-is. Widening to <2.0.0 lets resolvers pick twelvelabs 1.x (latest 1.3.3), which is a full SDK rewrite that breaks both tools using it — the failing CI on this PR is exactly that, not flake.

I verified locally against twelvelabs 1.3.3 (head 7326990):

  • search_video.py:55from twelvelabs.models.search import SearchDataModuleNotFoundError: No module named 'twelvelabs.models'. This is the CI failure: hatch test installs the twelvelabs extra (pyproject.toml:126), so test collection dies on import (macOS 3.13 failed first, siblings were cancelled by fail-fast).
  • chat_video.py:202,315TwelveLabs(api_key)TypeError (1.x constructor is keyword-only), and client.task no longer exists (now client.tasks with a different interface).

Options: close this and add twelvelabs to dependabot's major-version ignores (or comment @dependabot ignore this major version), or treat it as a real migration — port chat_video/search_video to the 1.x API in the same PR. The range bump alone can't ship.

Verification detail
  • Reviewed head 7326990e (only change: <1.0.0<2.0.0 in the dev and twelvelabs extras of pyproject.toml).
  • Repro: fresh venv, pip install twelvelabs==1.3.3 -e . pytest, then pytest tests/test_chat_video.py tests/test_search_video.pyERROR tests/test_search_video.py (collection ImportError above), matching the failed call-test-lint / Unit Tests - Python 3.13 - macOS job.
  • 1.x API surface checked via inspection: TwelveLabs.__init__(self, *, api_key=None, ...); instance has tasks/search/analyze, no task; twelvelabs.models package removed.
  • 0.x line is fine: current floor >=0.4.0 with <1.0.0 resolves to 0.4.11 and passes on main.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

dependencies Pull requests that update a dependency file python Pull requests that update python code

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants