Skip to content

Land the pytest Dependabot bump and clear nltk/pygments advisories - #16

Merged
dxdc merged 2 commits into
mainfrom
claude/quirky-pasteur-2v0az3
Oct 25, 2026
Merged

dxdc merged 2 commits into
mainfrom
claude/quirky-pasteur-2v0az3

Conversation

@dxdc

@dxdc dxdc commented Oct 25, 2026

Copy link
Copy Markdown
Owner

Covers #15, plus two vulnerable packages Dependabot has not opened PRs for.

#15: pytest 9.0.2 -> 9.0.3

Merged as is. The branch was current with main, so it fast-forwards and #15 closes as merged when this lands.

nltk and pygments

Running pip-audit over every package in uv.lock flags two more:

nltk is a runtime dependency, and CI installs from pyproject.toml rather than the lock. So its floor goes up to the first fixed release, the same way #13 handled 3.9.4. pygments comes in only through pytest, so it is a lock-only bump.

Package Before After
nltk 3.9.4 3.10.4 (adds defusedxml)
pygments 2.19.2 2.21.0

pip-audit over the new lock reports no known vulnerabilities.

One nltk 3.10 behavior change

nltk.download() now refuses to fetch through an HTTP proxy, as SSRF protection. 3.9.4 downloads fine behind one. This does not affect generate.yml: it installs nltk unpinned through g2p-en and does not read pyproject.toml, and GitHub-hosted runners do not go through a proxy. It would only matter for someone running the corpus download behind a corporate proxy.

Verification

  • ruff check and ruff format --check on src/ and tests/ pass
  • pytest: 113 passed
  • uv lock --check passes

Generated by Claude Code

dependabot Bot and others added 2 commits April 14, 2026 03:01
Bumps the uv group with 1 update in the / directory: [pytest](https://github.com/pytest-dev/pytest).


Updates `pytest` from 9.0.2 to 9.0.3
- [Release notes](https://github.com/pytest-dev/pytest/releases)
- [Changelog](https://github.com/pytest-dev/pytest/blob/main/CHANGELOG.rst)
- [Commits](pytest-dev/pytest@9.0.2...9.0.3)

---
updated-dependencies:
- dependency-name: pytest
  dependency-version: 9.0.3
  dependency-type: direct:production
  dependency-group: uv
...

Signed-off-by: dependabot[bot] <support@github.com>
pip-audit over uv.lock flags two packages that Dependabot has not opened
PRs for:

  - nltk 3.9.4: 50 advisories fixed in 3.10.0-3.10.3 (among them
    GHSA-m42h-3232-vpv3, GHSA-qvv7-cg9c-w4x3, GHSA-fg7f-2386-8897)
  - pygments 2.19.2: GHSA-5239-wwwm-4pmq, fixed in 2.20.0

nltk is a runtime dependency and CI installs from pyproject.toml rather
than the lock, so raise its floor to the first fixed release, as #13 did
for 3.9.4. pygments comes in only through pytest, so it is a lock-only
bump.

  nltk      3.9.4  -> 3.10.4 (adds defusedxml)
  pygments  2.19.2 -> 2.21.0

Four more nltk advisories list no fix version (PYSEC-2026-597,
PYSEC-2026-3740, PYSEC-2026-3955, PYSEC-2026-4318), but none of them
lists 3.10.4 as affected, and pip-audit over the new lock reports no
known vulnerabilities.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01GmdbKVGsk81rJRHkUYAjNz
@dxdc
dxdc merged commit 209c752 into main Oct 25, 2026
7 checks passed
@dxdc
dxdc deleted the claude/quirky-pasteur-2v0az3 branch October 26, 2026 06:17
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.

2 participants