Repository navigation
chore(deps): bump huggingface/doc-builder/.github/workflows/upload_pr_documentation.yml from cf20b09f274383f0d91a7055c5e43f0a2ab3d1a3 to 9f9bb430a712b5650d0e43e510656098ecdd75d4 - #1223
Conversation
…_documentation.yml Bumps [huggingface/doc-builder/.github/workflows/upload_pr_documentation.yml](https://github.com/huggingface/doc-builder) from cf20b09f274383f0d91a7055c5e43f0a2ab3d1a3 to 9f9bb430a712b5650d0e43e510656098ecdd75d4. - [Release notes](https://github.com/huggingface/doc-builder/releases) - [Changelog](https://github.com/huggingface/doc-builder/blob/main/RELEASE.md) - [Commits](huggingface/doc-builder@cf20b09...9f9bb43) --- updated-dependencies: - dependency-name: huggingface/doc-builder/.github/workflows/upload_pr_documentation.yml dependency-version: 9f9bb430a712b5650d0e43e510656098ecdd75d4 dependency-type: direct:production ... Signed-off-by: dependabot[bot] <support@github.com>
|
The docs for this PR live here. All of your documentation changes will be reflected on that endpoint. The docs are available until 30 days after the last update. |
There was a problem hiding this comment.
Alignment Review Report
Scope: one-line change in .github/workflows/upload_pr_documentation.yml bumping the pinned reusable-workflow SHA huggingface/doc-builder/.github/workflows/upload_pr_documentation.yml from cf20b09 -> 9f9bb43. No src/, tests/, or envs/ code is touched.
Automated Checks
- Lint (
.claude/hooks/lint.sh): exits non-zero, but only from pre-existing formatting drift insrc//tests//envs/(ruff-format: "56 files would be reformatted, 846 already formatted"; usort flags several more). This matches the known clean-tree lint state documented inAGENTS.md. usort/ruff do not scan workflow YAML, and I confirmed the changed file is not among the flagged paths. No new lint issues are introduced by this PR. - Debug code (
.claude/hooks/check-debug.sh): only pre-existingprint/TODOhits insrc/(docstrings, CLI output, a local test harness). None are in this diff. Clean.
Supply-chain verification (the substance of this PR)
This pin feeds repo secrets (an HF doc-build push token + comment-bot credentials) into the reusable workflow, so I verified the new revision against huggingface/doc-builder via the GitHub API:
9f9bb43exists and is the current tip ofdoc-builder@main(compare 9f9bb43...main=identical), so the trailing# maincomment is accurate.- It was authored by
hf-security-analysis[bot]— "fix(ci): harden workflow files flagged on #830 (#832)", i.e. a security-hardening commit. - The consumed file
upload_pr_documentation.ymlis unchanged between old and new; the 3-commit drift only touchesdependabot.yml,build_main_documentation.yml, andbuild_pr_documentation.yml. - The full 40-char SHA pin is preserved (best practice, not a mutable tag).
Net result: legitimate, low-risk, and positive for security.
Open RFCs Context
Active RFCs — 000/001/002/003/005/008/012 (In Review) and 010/011 (Draft) — cover core APIs, abstractions, rewards, agentic harnesses, validation, world-modeling, and discovery. None relate to docs-build CI, so there is no overlap with this change.
Tier 1: Fixes Required
None. The pre-existing lint drift is out of scope for a dependency bump — reformatting 50+ unrelated files in this PR would be inappropriate scope creep.
Tier 2: Alignment Discussion
Principle conflicts: None identified — no change to the Gymnasium-style API, client/server separation, reward placement, MCP boundary, or type/serialization contracts.
RFC conflicts: None identified.
Non-blocking note
The sibling pins in build_documentation.yml and build_pr_documentation.yml still reference the old cf20b09. Dependabot opens one PR per referenced workflow, so companion PRs should follow; interim mixed SHAs across these independent reusable workflows are harmless.
Summary
- 0 mechanical issues to fix (all lint/debug findings are pre-existing and out-of-scope)
- 0 alignment points for human review
- 0 RFC conflicts
- Verified safe bump to the current
doc-builder@mainsecurity-hardening commit.
Sent by Cursor Automation: Pre-review
| jobs: | ||
| build: | ||
| uses: huggingface/doc-builder/.github/workflows/upload_pr_documentation.yml@cf20b09f274383f0d91a7055c5e43f0a2ab3d1a3 # main | ||
| uses: huggingface/doc-builder/.github/workflows/upload_pr_documentation.yml@9f9bb430a712b5650d0e43e510656098ecdd75d4 # main |
There was a problem hiding this comment.
Verified this new pin against huggingface/doc-builder:
9f9bb43is the current tip ofdoc-builder@main(compare 9f9bb43...main=identical), so the# maincomment is accurate.- It is a security-hardening commit by
hf-security-analysis[bot](fix(ci): harden workflow files flagged on #830 (#832)). - The consumed
upload_pr_documentation.ymlis unchanged vs the oldcf20b09; onlydependabot.ymland the twobuild_*workflows differ across the 3-commit gap. - Full 40-char SHA pin preserved.
This workflow forwards repo secrets (HF push token + comment-bot credentials) to the reusable workflow, so the pin update matters — and it checks out as safe.
…-github/workflows/upload_pr_documentation.yml-9f9bb430a712b5650d0e43e510656098ecdd75d4 Co-authored-by: benjamin.burtenshaw <benjamin.burtenshaw@huggingface.co>
There was a problem hiding this comment.
APPROVE at 5f212718 (refreshed onto main ca4219d2).
Upstream verification
huggingface/doc-builder@9f9bb430a712b5650d0e43e510656098ecdd75d4 is the current tip of doc-builder's main (compare reports identical), and the commit is fix(ci): harden workflow files flagged on #830 (#832).
I diffed the referenced reusable workflow at both pins: .github/workflows/upload_pr_documentation.yml is byte-identical between cf20b09f and 9f9bb430 (sha256 742ca4cd53ff471b at both). So this PR moves the pin forward without changing a single line of the privileged workflow that receives hf_token, comment_bot_app_id, and comment_bot_secret_pem. The secrets: and with: contract our caller passes is unchanged.
Workflow-only; no wheel or sdist input changes, so this is not 0.6.0 package cargo. All 11 exact-head checks are green and the branch is up to date with main.
Sent by Cursor Automation: Release
Keep draft #1211 exact-main after workflow-only Dependabot merge. Co-authored-by: benjamin.burtenshaw <benjamin.burtenshaw@huggingface.co>
OpenEnv 0.6.0 Breaking change - `--llm-endpoint` / OpenAIClient now take a full base URL; the implicit port 8000 is gone. `http://localhost` means port 80; use `http://localhost:8000` or `--llm-port 8000` for the old behavior (#1189, landed via #1237). New - NovitaSandboxProvider: run an OpenEnv server in a Novita AI sandbox over wss://, from a registry image or a local Dockerfile; install with `pip install openenv[novita]` (#1191). - RFC 008 Level 2 validation building blocks: manifest-v2, report-v2 and runtime-plan schemas, the severity-v2 policy, and DockerValidationProvider (#1178, #1179). The `openenv validate` CLI is unchanged in this release. Fixes - A failed `new_session()` no longer leaves the provider's container or sandbox running (#1145). Repository and environment images (not in the wheel) - coding_env `additional_imports` now extends the default safe-import allowlist (#1147). - Environment lockfile security updates: anyio 4.14.2 and soupsieve 2.9.2 (#1196, #1197, #1201, #1202). - CI hardening: per-job GITHUB_TOKEN scopes and SHA-pinned actions (#1221, #1223, #1225, #1226, #1228, #1239, #1240); validation-lab toolchain pins (#1229, #1230, #1231); docs navigation and Miles integration docs (#1219, #1220). Known, accepted for this release: Novita provider debt (unbraced $ARG ordering in Dockerfile flattening, a tbench2 example readiness leak); fixes follow in #1235. Validated: exact-head CI 13/13 on 0034202; TestPyPI 0.6.0.dev141 (run 35854754769) byte-identical to the release wheel apart from Version; clean wheel/sdist install, CLI, and Echo reset/step smoke.


Bumps huggingface/doc-builder/.github/workflows/upload_pr_documentation.yml from cf20b09f274383f0d91a7055c5e43f0a2ab3d1a3 to 9f9bb430a712b5650d0e43e510656098ecdd75d4.
Changelog
Sourced from huggingface/doc-builder/.github/workflows/upload_pr_documentation.yml's changelog.
Commits
9f9bb43fix(ci): harden workflow files flagged on #830 (#832)17ccdf1chore: enable Dependabot weekly GitHub Actions bumps (#790)47c6b58fix(ci): pin the doc-builder checkout to the caller's pinned revision (#830)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
Only changes which reusable workflow SHA runs for PR documentation upload; no application or secrets logic changes in this repo.
Overview
Updates the Upload PR Documentation workflow to call a newer pinned revision of
huggingface/doc-builder's reusableupload_pr_documentation.ymlworkflow.The upstream bump includes CI hardening for flagged workflow files and pinning the doc-builder checkout to the same revision the caller pins, so PR doc upload behavior should stay aligned with the chosen SHA while picking up those fixes.
Reviewed by Cursor Bugbot for commit 5f21271. Bugbot is set up for automated code reviews on this repo. Configure here.