feat: add resources view in react - #40
Merged
Merged
Conversation
Ports the Resources view to React: tokenizer inventory, sequential download queue (Download all queues every uncached tokenizer, a failure doesn't clear the queue, only Cancel does), and per-directory storage stats, backed by the app's only sanctioned network path. Makes Cancel actually reliable in both directions: the button is now disabled until the async job id it needs actually exists, and HF downloads (a single opaque blocking call with no chunked progress hook) now poll cancellation on a background thread instead of always reporting success regardless of a cancel request. A running scan and a running download can no longer overlap in either direction, so neither can silently revert the other's state. Refresh failures surface after the first successful scan instead of going silent, a stale open-folder failure clears once a later one succeeds, and the completed-download splice keeps the per-cache storage figures in step with the aggregate total instead of only updating one of them. Also fixes the gated-HF-repo hint to point at Settings, normalizes font smoothing so small muted text doesn't read heavier in light theme than dark, and stops treating a tokenizer's cache path as an uppercase label instead of a file path.
Phantom-VK
changed the title
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
1 participant
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.
Summary
Test plan
npx tsc --noEmitcleannpx vitest run— 97 passingnpm run buildcleanpytest— 325 passing (new HF-cancel-mid-download test)