sync: public source for 0.2.5 - #64
Merged
Merged
Conversation
Generated by tools/sync_public.py from the internal tree at 72282cf7e. Only allow-listed paths are included; tuned quant profiles and internal tooling are excluded by rule.
This was referenced Sep 17, 2026
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.
Source sync for the 0.2.5 release, generated by the internal
public-syncworkflow's allow-list (
tools/sync_public.py) from internalmain.10 files changed, 0 new.
include/baseRT/baseRT.h, the Swiftheader, both Rust binding manifests,
base-convert/Cargo.toml+Cargo.lock,bindings/node/package.json,bindings/python/setup.py,bindings/python/baseRT/__init__.py.base-convert/crates/base-convert/src/hub.rs— theconvert-on-pull fix. The resumable range path (every file ≥ 32 MB, i.e. every
safetensors shard) parked its result in
blobs/and wrote nosnapshots/<rev>/pointer, so conversion failed withno .safetensors shardson a pull that had downloaded every byte. Now linked into the snapshot, with
the link target absolutised (a relative models root would otherwise dangle,
since a symlink target resolves relative to the link's directory) and the
repo-supplied filename routed through
cache::id_to_relpathso a traversingentry cannot place a link outside the snapshot.
The tuned quant profiles under
base-convert/profiles/are held back by theallow-list, as usual.
Overlap with #56
This carries the same fix as #56 (from @matts-path, for #54), which was
developed independently. The version here adds the absolutise and
path-traversal hardening plus two regression tests. The maintainers should
decide how to reconcile the two — this PR does not close #56.
Verification
The allow-list output was applied to a clean clone and built before this branch
was pushed (the sync workflow gates on a release build). Locally the same tree
builds and
cargo testpasses with 0 failing suites, including the two newlink_into_snapshottests.