Repository navigation
Resolve open Dependabot alerts by upgrading vulnerable dependencies - #41
Merged
emiehling merged 3 commits intoNov 3, 2026
Merged
Conversation
- Upgrade the locked versions of pillow, virtualenv, pyjwt, vllm, mcp, jupyterlab, notebook, tornado, filelock, urllib3, certifi, dnspython, multidict, nbconvert, nltk, opentelemetry-api, accelerate, and pytest past their advisories - Raise the vllm floor to 0.32.0 and trl[vllm] to 1.17.0, and the pytest dev pin to >=9.0.3,<10.0.0 - Install mergekit from PyPI (>=0.1.5,<0.2.0), which relaxes its accelerate, pydantic, and safetensors pins; drop the `merging` extra conflicts and add `merging` to `all` - Remove the `ppo` control, since trl removed `PPOTrainer` in 1.13.0 - Compile json-schema constraints with compact separators on the Hugging Face path, matching the vLLM engine's output under `disable_any_whitespace` Two setuptools advisories (GHSA-xmj3-9gf7-h52w, GHSA-h35f-9h28-mq5c) remain open because every patched vllm release requires setuptools<81. Signed-off-by: Erik Miehling <emiehling@gmail.com>
`trust_remote_code` now defaults to False for CPO. The default encoder (`nomic-ai/nomic-embed-text-v1.5`) loads through the `NomicBertModel` class in transformers. Its Hub modeling code calls `get_extended_attention_mask`, which transformers 5.17 removed. Signed-off-by: Erik Miehling <emiehling@gmail.com>
…n vLLM 0.29+ `steerability-challenge` now includes IBM/vLLM-Hook#58, which patches the launcher app builder that `vllm serve` calls since vLLM 0.29. Without it, `/v1/hook/capabilities` and `/v1/hook/artifacts` return 404 and the `vllm-serve` backend with `hook_plugin=True` cannot start. Signed-off-by: Erik Miehling <emiehling@gmail.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
uv.lockto versions that resolve 116 of the 118 open Dependabot alerts:pillow13.0.0,virtualenv21.15.2,pyjwt2.15.1,vllm0.32.0,mcp2.3.0,jupyterlab4.6.5,notebook7.6.4,tornado6.6,filelock4.2.7,urllib32.8.0,certifi2026.10.23,dnspython2.9.1,multidict6.9.1,nbconvert7.17.2,nltk3.10.4,opentelemetry-api1.45.2,accelerate1.15.0, andpytest9.2.0vllmextra's floor tovllm>=0.32.0andtrl[vllm]>=1.17.0(trl releases before 1.17 cap vLLM below the patched versions), and thepytestdev pin to>=9.0.3,<10.0.0mergekitfrom PyPI (>=0.1.5,<0.2.0) instead of the pinned git commit. Release 0.1.5 relaxes theaccelerate~=1.6.0,pydantic~=2.10, andsafetensors~=0.5pins, which removes the reason for themergingextra's[tool.uv]conflictsmergingconflicts and addmergingto theallextra. The lock now resolves one version of each package across every extra (one torch, 2.13.0)ppocontrol and its tests, since trl removedPPOTrainerin 1.13.0",",":") on the Hugging Face path. vLLM 0.32 passes these separators to xgrammar underdisable_any_whitespace, and the Hugging Face path now produces the same output ({"ok":true}rather than{"ok": true})trust_remote_codetoFalse. The default encoder (nomic-ai/nomic-embed-text-v1.5) loads through theNomicBertModelclass in transformers, since its Hub modeling code callsget_extended_attention_mask, which transformers 5.17 removed. The embeddings are identical to those from the Hub code on transformers 5.16vllm-hook-pluginstob479aedonsteerability-challenge(Register the hook routes on the vLLM launcher app builder IBM/vLLM-Hook#58). vLLM 0.29 moved the app builder thatvllm servecalls, and the earlier plugin commit left/v1/hook/capabilitiesand/v1/hook/artifactsunregistered, which made thevllm-servebackend withhook_plugin=Truefail to startAGENTS.md, the installation page, and the controls concept page describe theallextra and the TRL wrappers withoutppoTwo
setuptoolsadvisories remain open (GHSA-h35f-9h28-mq5c and GHSA-xmj3-9gf7-h52w). Every patched vLLM release requiressetuptools<81, and both fixes are in 83.0.0 or later. The first affects sdist builds on macOS and the second affects legacy wheel installation on Windows.Tests: the full suite passes on CPU (
-m "not network") and on an A100 80GB with thenetworktests included (vLLM 0.32.0 offline engine and plugin engine). A linear and a TIES merge through themergekitcontrol run with mergekit 0.1.5. All 34 example notebooks run end to end on A100/H100 GPUs, includingrecipes/vllm_serve.ipynbagainst the locked plugin commit.