chore(deps): bump soupsieve from 2.8.3 to 2.8.4 in /envs/browsergym_env#938
chore(deps): bump soupsieve from 2.8.3 to 2.8.4 in /envs/browsergym_env#938dependabot[bot] wants to merge 1 commit into
Conversation
Bumps [soupsieve](https://github.com/facelessuser/soupsieve) from 2.8.3 to 2.8.4. - [Release notes](https://github.com/facelessuser/soupsieve/releases) - [Commits](facelessuser/soupsieve@2.8.3...2.8.4) --- updated-dependencies: - dependency-name: soupsieve dependency-version: 2.8.4 dependency-type: indirect ... Signed-off-by: dependabot[bot] <support@github.com>
There was a problem hiding this comment.
Alignment Review Report
Two-tier review of this Dependabot change. The PR is titled as a soupsieve patch bump, but the diff scope is much larger than the title suggests, so most of this review is about that scope — not the version bump itself.
Automated Checks
- Lint (
.claude/hooks/lint.sh): FAIL on the repo baseline, but not attributable to this PR. Every flagged file is unformatted Python in other envs (opencode_env,chat_env,repl_env,terminus_env,textarena_env,coding_tools_env,jupyter_env,agent_world_model_env). This PR changes only auv.lock(not a Python file), so it introduces no lint issues. - Debug code (
.claude/hooks/check-debug.sh): FOUND on the baseline, but not attributable to this PR — allprint/TODOhits are pre-existing undersrc/. Nothing in the changed file.
Open RFCs Context
Open RFCs: 000, 001, 002, 003, 005 (In Review) and 010 (Draft). All concern the runtime abstractions, env-spec, MCP, rubrics, harnesses, and a token world model — none touch dependency management, packaging, uv locks, or PyPI registries, so none are relevant to this change.
Tier 1: Fixes Required
None. The only modified file is a uv lockfile; no lint, debug, type, import, syntax, or credential/injection issues. The soupsieve 2.8.3 → 2.8.4 patch bump itself is well-formed (valid sdist + wheel hashes on files.pythonhosted.org) and low-risk.
Tier 2: Alignment Discussion
Principle Conflicts
None identified — PRINCIPLES.md / INVARIANTS.md govern the runtime API and architecture, which this change does not touch.
RFC Conflicts
None identified.
Process & Supply-Chain Concerns
Neither is a hard-rule violation, but both look like unintended side effects of how Dependabot regenerated the lock and deserve a human decision before merge:
ALIGNMENT FLAG: PR updates a path Dependabot is configured to exclude
- Policy at stake:
.github/dependabot.ymlsetsexclude-paths: ["envs/**"]for theuvecosystem (added in #644). - The concern: this PR bumps a
uvdependency insideenvs/browsergym_env/— exactly the excluded path. Either the exclusion isn't matching as intended (may needdirectories:or a different glob) or this PR is stale. Merging it effectively validates output the config is meant to suppress. - Suggested reviewer: @burtenshaw (author of the Dependabot config in #644 and of the dependency-aggregation PR #930); cc @Darktex
ALIGNMENT FLAG: a "soupsieve patch bump" also rewrites the package index for the whole lockfile
- The concern: beyond
soupsieve, the diff flips thesourceregistry of ~57 packages (170 lines) fromhttps://pypi.registries.huggingface.tech/→https://pypi.org/simple, and bumps the lockfilerevision2 → 3. The HF registry isn't configured anywhere in-repo (no[[tool.uv.index]]), so this is an artifact of the environment where Dependabot re-resolved the lock, not an intentional migration. The repo is currently split (~25 envs still on the internal registry, ~10 already on public PyPI); merging deepens that inconsistency, and if HF build/CI infra relies on the internal index this could affect reproducibility. Artifact URLs/hashes are unchanged, so this is not a dependency-confusion risk. - Suggested action: decide whether to standardize the index across envs and/or pin Dependabot's registry so locks stop flip-flopping on each run.
- Suggested reviewer: @burtenshaw; cc @Darktex
Summary
- 0 mechanical (Tier 1) issues to fix
- 2 alignment / process points for human review
- 0 RFC conflicts
Note: uv was absent from the review environment; I installed it to run the lint hook. Because the only changed file is a lockfile, the hook results reflect the repo baseline, not this PR.
Sent by Cursor Automation: Pre-review
| @@ -1,5 +1,5 @@ | |||
| version = 1 | |||
| revision = 2 | |||
| revision = 3 | |||
There was a problem hiding this comment.
revision = 2 → 3 is a uv lockfile format bump — it means Dependabot regenerated this lock with a newer uv than the one that originally produced it. That regeneration is why all ~57 package source lines below also got rewritten in a PR that's nominally just a soupsieve patch.
| source = { registry = "https://pypi.registries.huggingface.tech/" } | ||
| sdist = { url = "https://files.pythonhosted.org/packages/7b/ae/2d9c981590ed9999a0d91755b47fc74f74de286b0f5cee14c9269041e6c4/soupsieve-2.8.3.tar.gz", hash = "sha256:3267f1eeea4251fb42728b6dfb746edc9acaffc4a45b27e19450b676586e8349", size = 118627, upload-time = "2026-01-20T04:27:02.457Z" } | ||
| version = "2.8.4" | ||
| source = { registry = "https://pypi.org/simple" } |
There was a problem hiding this comment.
This block is the intended change (soupsieve 2.8.3 → 2.8.4) and it's fine on its own.
source here also flips from the internal pypi.registries.huggingface.tech registry to public pypi.org/simple — and the same flip hits ~57 packages / 170 lines across this file. That index churn (plus the revision bump on line 2) is the real thing to review here, not the version bump. See the review summary for the full flag.
|
Aggregated into #921 ( |
|
OK, I won't notify you again about this release, but will get in touch when a new version is available. If you'd rather skip all updates until the next major or minor version, let me know by commenting If you change your mind, just re-open this PR and I'll resolve any conflicts on it. |


Bumps soupsieve from 2.8.3 to 2.8.4.
Release notes
Sourced from soupsieve's releases.
Commits
28108abLimit excessive selectorsef18872Fix test for Windowseb43976Merge commit from fork3a661b2Fix typo in pseudo-classes.md (#294)0cb533dUpdate hatchling version requirement in pyproject.toml (#290)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)You can disable automated security fix PRs for this repo from the Security Alerts page.
Note
Low Risk
Patch-level transitive dependency update in the browsergym env lockfile only; no direct code changes.
Overview
Bumps soupsieve from 2.8.3 to 2.8.4 in
envs/browsergym_env(lockfile update; transitive dependency of beautifulsoup4).2.8.4 is a patch release with selector-engine fixes: more efficient attribute matching and a cap on how many selectors are processed in a single pattern (mitigates abusive or oversized selector input). No application code changes in this repo.
Reviewed by Cursor Bugbot for commit b518bb1. Bugbot is set up for automated code reviews on this repo. Configure here.