[FIX] Align python-dotenv to 1.2.2 across all packages (unbreak main after #2006)#2008
Conversation
Dependabot #2006 bumped python-dotenv to 1.2.2 in most packages but left the root project (hook-check-django-migrations group, ==1.0.1) and the unstract-connectors test group (~=1.0.0) behind, producing an unsatisfiable resolution against members that now require 1.2.2. This broke main: the test job fails to resolve (connectors:test and the root workspace) and the worker-unified image build fails with 'lockfile needs to be updated' because workspace resolution pulls the stale connectors constraint. Bump both stragglers to 1.2.2 and regenerate the affected lockfiles (root, connectors, plus the downstream library locks filesystem, tool-registry, workflow-execution that were still pinned at 1.0.1).
|
Caution Review failedPull request was closed or merged during review No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Pro Run ID: ⛔ Files ignored due to path filters (5)
📒 Files selected for processing (2)
Summary by CodeRabbit
WalkthroughThis PR updates the ChangesPython-dotenv version bump
🎯 1 (Trivial) | ⏱️ ~2 minutes 🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
|
|
| Filename | Overview |
|---|---|
| pyproject.toml | Bumps python-dotenv from ==1.0.1 to ==1.2.2 in the hook-check-django-migrations dependency group to resolve workspace-level conflict |
| unstract/connectors/pyproject.toml | Bumps python-dotenv from ~=1.0.0 to ~=1.2.2 in the test dependency group; resolves the second straggler constraint that made uv resolution unsatisfiable |
| unstract/connectors/uv.lock | Lock file regenerated; only change is python-dotenv 1.0.1 → 1.2.2 with updated sdist/wheel hashes |
| unstract/filesystem/uv.lock | Lock file regenerated; only change is python-dotenv 1.0.1 → 1.2.2 with updated sdist/wheel hashes |
| unstract/tool-registry/uv.lock | Lock file regenerated; only change is python-dotenv 1.0.1 → 1.2.2 with updated sdist/wheel hashes |
| unstract/workflow-execution/uv.lock | Lock file regenerated; only change is python-dotenv 1.0.1 → 1.2.2 with updated sdist/wheel hashes |
| uv.lock | Root lock file regenerated; changes are exclusively python-dotenv 1.0.1 → 1.2.2 across workspace members, including the specifier floor bump from >=1.0.0 to >=1.2.2 |
Flowchart
%%{init: {'theme': 'neutral'}}%%
flowchart TD
A[Dependabot #2006\nbumped python-dotenv to 1.2.2\nin most packages] --> B{Two stragglers left behind}
B --> C[pyproject.toml\nhook-check-django-migrations\npython-dotenv==1.0.1]
B --> D[connectors/pyproject.toml\ntest group\npython-dotenv~=1.0.0]
C --> E[uv resolution fails:\nunsatisfiable constraints]
D --> E
E --> F[This PR: bump both to 1.2.2\nregenerate 5 uv.lock files]
F --> G[root uv.lock]
F --> H[connectors uv.lock]
F --> I[filesystem uv.lock]
F --> J[tool-registry uv.lock]
F --> K[workflow-execution uv.lock]
G & H & I & J & K --> L[Workspace resolution\nsatisfiable again]
Reviews (1): Last reviewed commit: "[FIX] Align python-dotenv to 1.2.2 acros..." | Re-trigger Greptile
Unstract test resultsPer-group results
Critical paths
|
Resolve conflict between python-dotenv alignment (#2008) and the uv group bump. Keep python-dotenv ~=1.2.2 from main and the Dependabot bumps (requests 2.33.0, pytest 9.0.3, aiohttp 3.13.4, pypdf 6.10.2). Regenerate affected uv.lock files for workspace consistency.



What
Aligns
python-dotenvto1.2.2across every package and regenerates the affecteduv.lockfiles, fixing an unsatisfiable dependency resolution onmain.Why
Dependabot #2006 ("Bump the uv group across 10 directories") bumped
python-dotenvto1.2.2in most packages but left two stragglers behind:pyproject.toml(hook-check-django-migrationsgroup) →python-dotenv==1.0.1unstract/connectors/pyproject.toml(testgroup) →python-dotenv~=1.0.0(i.e.<1.1)These are mutually unsatisfiable with workspace members that now require
1.2.2(unstract-sdk1,unstract-workers, etc.), somainis currently broken. This surfaced on PR #2005 (and any PR branched off / merging current main) as two failed checks:test—uvresolution fails:unstract:hook-check-django-migrations depends on python-dotenv==1.0.1andunstract-connectors:test depends on python-dotenv>=1.0.0,<1.1.dev0conflict with the1.2.2requirement.build(worker-unified image) —uv sync --group deploy --lockedfails with "The lockfile at uv.lock needs to be updated" because workspace resolution pulls in the stale connectors constraint.The downstream library locks (
filesystem,tool-registry,workflow-execution) were also left pinned atpython-dotenv 1.0.1by the same bump and are regenerated here for consistency.How
pyproject.tomlpython-dotenv==1.0.1→==1.2.2unstract/connectors/pyproject.toml(testgroup)python-dotenv~=1.0.0→~=1.2.2uv.lockfor root,unstract/connectors,unstract/filesystem,unstract/tool-registry,unstract/workflow-execution(every lock diff is exclusivelypython-dotenv 1.0.1 → 1.2.2)Can this PR break any existing features. If yes, please list possible items. If no, please explain why. (PS: Admins do not merge the PR without this section filled)
No. This only aligns a single dependency (
python-dotenv) to the version (1.2.2) that the rest of the repo already uses, restoring a satisfiable resolution. No application code changes. Verified locally thatuv lock --checkpasses for every package and that the previously-failinguv sync --group deploy --locked(worker-unified) now succeeds.Database Migrations
Env Config
Relevant Docs
Related Issues or PRs
mainbreakage)Dependencies Versions
python-dotenv:1.0.1→1.2.2(root + connectors test group; locks aligned to the1.2.2already used everywhere else)Notes on Testing
uv lock --checkpasses for all packages (root, backend, prompt-service, platform-service, runner, x2text-service, tool-sidecar, workers, and allunstract/*libs)main:uv sync --group deploy --locked --no-install-project --no-dev(the worker-unified build step) failed with "lockfile needs to be updated"; with this fix it succeeds.Screenshots
Checklist
I have read and understood the Contribution Guidelines.