build(deps): bump poetry from 2.2.1 to 2.3.4#20823
build(deps): bump poetry from 2.2.1 to 2.3.4#20823dependabot[bot] wants to merge 1 commit intomainfrom
Conversation
Bumps [poetry](https://github.com/python-poetry/poetry) from 2.2.1 to 2.3.4. - [Release notes](https://github.com/python-poetry/poetry/releases) - [Changelog](https://github.com/python-poetry/poetry/blob/main/CHANGELOG.md) - [Commits](python-poetry/poetry@2.2.1...2.3.4) --- updated-dependencies: - dependency-name: poetry dependency-version: 2.3.4 dependency-type: direct:production update-type: version-update:semver-minor ... Signed-off-by: dependabot[bot] <support@github.com>
🤖 Cursor Dependency AnalysisSupply-Chain Malware ReviewChecking how Poetry is pinned in this repository to ground the review. Verdict: benign Why this looks safe
Actionable note: Prefer merging 2.3.4 (or newer) over stopping at 2.3.3, since 2.3.4 fixes a regression introduced in 2.3.3 and adds the sdist path-traversal fix per the release notes you quoted. Compatibility AnalysisSearching the repository for Poetry usage and scanning for compatibility concerns. 1) Where
|
| Area | Role |
|---|---|
requirements-poetry.txt |
Pins poetry plus poetry-dynamic-versioning[plugin] and poetry-plugin-export. |
setup-poetry.sh / Setup-poetry.ps1 |
pip install -r requirements-poetry.txt into .penv. |
install.sh / Install.ps1 |
poetry env use and poetry sync (main install path). |
install-timelord.sh |
poetry show for chiavdf. |
pyproject.toml |
Poetry project layout ([tool.poetry], deps, supplemental index). |
poetry-check.py + .pre-commit-config.yaml |
poetry check --strict then poetry lock via activated.py --poetry. |
.github/workflows/upload-pypi-source.yml |
poetry check. |
.github/workflows/reflow-version.yml |
pip install poetry (unpinned) and poetry version -s. |
| Other workflows | References to poetry.lock, setup-poetry / mode: poetry in installer/test flows. |
There are no import poetry usages in Chia application code (only under .upstream-dependency, the Poetry source tree). Runtime behavior of the node/wallet does not depend on the Poetry package.
2) Overlap with 2.2.1 → 2.3.4 changes
- CLI surface you use (
sync,check,version,show,env use) stays the same in spirit; the release notes emphasize installer security, wheel perf, env detection, HTTP auth, git tag updates, and poetry-core parsing/resolution fixes—not a removed public CLI. - Meaningful intersection:
poetry lock(including inpoetry-check.py). Updated poetry-core can change how markers,!=, groups, and related constraints resolve, so the next lock refresh may produce a realpoetry.lockdiff even ifpyproject.tomlis unchanged. That is the main “API-adjacent” touchpoint for this repo. poetry syncagainst an existing lock is usually stable; risk rises when the lock is regenerated.
3) Risks / unknowns
- Lock churn: First
poetry lock(e.g. from pre-commit) might be noisier or conflict with review expectations; worth a quick eye on CI/pre-commit after merge. - Plugins:
poetry-dynamic-versioningandpoetry-plugin-exportare not pinned to exact versions inrequirements-poetry.txt(onlypoetry==…). They generally track Poetry 2.x, but any incompatibility would show up at install or plugin load time. - Drift:
reflow-version.ymlinstallspoetrywithout the same pin asrequirements-poetry.txt, so that job can use a different Poetry than local/CI install scripts unless someone aligns it. - Security notes (sdist / wheel path traversal): Relevant to Poetry’s install path when handling malicious packages; upgrading reduces that tooling risk for dev/CI.
4) Recommendation
Merge-with-caveats: Approve/merge once CI is green, with the expectation that regenerating the lockfile may show poetry-core–driven diffs and that plugin install should be smoke-tested. Optionally align reflow-version.yml with the pinned Poetry version in a follow-up for consistency.
If CI fails only on lock or plugin resolution, treat that as fix-forward (adjust pins or lockfile) rather than blocking the upgrade indefinitely—the 2.3.x security and installer fixes are a strong reason to stay current on the tooling pin.
Malware Scan Summary
- Status: warn
- Warn only mode:
true - Changed upstream files scanned:
181 - Resolution strategy:
tag_range - Changed node/vendor paths:
0 - Changed lockfiles:
1 - Resolved upstream range:
b9e5d79fc57de2f2e60973019d56662b7398440b..7c7af71ba206dadd2ff7eda19b9a4c90c4349754 - Resolved refs: from=
b9e5d79fc57de2f2e60973019d56662b7398440bto=7c7af71ba206dadd2ff7eda19b9a4c90c4349754 - Unicode findings (post-allowlist):
0 - Confusable findings (post-allowlist):
0 - IOC findings (post-allowlist):
0 - Heuristic findings (post-allowlist):
116
Top findings
tests/utils/env/test_env.py:5shell_process_spawn ::import subprocesstests/utils/env/test_env.py:146shell_process_spawn ::mocker.patch("subprocess.check_output", side_effect=KeyboardInterrupt())tests/utils/env/test_env.py:149shell_process_spawn ::subprocess.check_output.assert_called_once() # type: ignore[attr-defined]tests/utils/env/test_env.py:155shell_process_spawn ::mocker.patch("subprocess.check_call", side_effect=KeyboardInterrupt())tests/utils/env/test_env.py:159shell_process_spawn ::subprocess.check_call.assert_called_once() # type: ignore[attr-defined]tests/utils/env/test_env.py:166shell_process_spawn ::"subprocess.check_output",tests/utils/env/test_env.py:167shell_process_spawn ::side_effect=subprocess.CalledProcessError(tests/utils/env/test_env.py:173shell_process_spawn ::subprocess.check_output.assert_called_once() # type: ignore[attr-defined]tests/utils/env/test_env.py:182shell_process_spawn ::"subprocess.check_call",tests/utils/env/test_env.py:183shell_process_spawn ::side_effect=subprocess.CalledProcessError(tests/utils/env/test_env.py:190shell_process_spawn ::subprocess.check_call.assert_called_once() # type: ignore[attr-defined]tests/utils/env/test_env.py:199shell_process_spawn ::"subprocess.check_output",tests/utils/env/test_env.py:200shell_process_spawn ::side_effect=subprocess.CalledProcessError(tests/utils/env/test_env.py:206shell_process_spawn ::subprocess.check_output.assert_called_once() # type: ignore[attr-defined]tests/utils/env/test_env.py:242shell_process_spawn ::"subprocess.run",tests/utils/env/test_env.py:243shell_process_spawn ::side_effect=subprocess.CalledProcessError(tests/utils/env/python/test_python_installer.py:3shell_process_spawn ::from subprocess import CalledProcessErrortests/utils/env/test_env_manager.py:574shell_process_spawn ::"subprocess.check_output",tests/utils/env/test_env_manager.py:621shell_process_spawn ::"subprocess.check_output",tests/utils/env/test_env_manager.py:691shell_process_spawn ::"subprocess.check_output",
Coverage Report for CI Build 24744741326Coverage decreased (-0.02%) to 91.16%Details
Uncovered ChangesNo uncovered changes found. Coverage Regressions28 previously-covered lines in 7 files lost coverage.
Coverage Stats💛 - Coveralls |
Bumps poetry from 2.2.1 to 2.3.4.
Release notes
Sourced from poetry's releases.
... (truncated)
Changelog
Sourced from poetry's changelog.
... (truncated)
Commits
7c7af71release: bump version to 2.3.4e512e7ffix: refuse to write files outside the target directory during sdist extracti...506c09dperf: useos.path.abspath()instead ofPath.resolve()(#10821)3d0151arelease: bump version to 2.3.389f09aafix long path issue on Windows (#10794)e068177installer: fix path traversal (#10792)d76a2f6chore: require new poetry-core version (#10790)859d443Update init & new commands for PEP 639 (License) (#10787)2ff2845fix: pass auth via Request constructor instead of calling HTTPBasicAuth on un...286e43benv: improve error handling if.venvis not a directory but a file (#10777)Dependabot will resolve any conflicts with this PR as long as you don't alter it yourself. You can also trigger a rebase manually by commenting
@dependabot rebase.Dependabot commands and options
You can trigger Dependabot actions by commenting on this PR:
@dependabot rebasewill rebase this PR@dependabot recreatewill recreate this PR, overwriting any edits that have been made to it@dependabot show <dependency name> ignore conditionswill show all of the ignore conditions of the specified dependency@dependabot ignore this major versionwill close this PR and stop Dependabot creating any more for this major version (unless you reopen the PR or upgrade to it yourself)@dependabot ignore this minor versionwill close this PR and stop Dependabot creating any more for this minor version (unless you reopen the PR or upgrade to it yourself)@dependabot ignore this dependencywill close this PR and stop Dependabot creating any more for this dependency (unless you reopen the PR or upgrade to it yourself)Note
Low Risk
Low risk version bump limited to build tooling, though it may slightly change lock/export behavior in CI and local dev environments.
Overview
Bumps the pinned Poetry version in
requirements-poetry.txtfrom 2.2.1 to 2.3.4 to keep the repo’s Poetry tooling in sync with Dependabot.Reviewed by Cursor Bugbot for commit d3a6bea. Bugbot is set up for automated code reviews on this repo. Configure here.