Promote staging/main to master - #140
Conversation
* Fix auto_discover mounts being dropped at startup Co-authored-by: Radhey Kalra <radheykalra901@gmail.com> * Make the auto_discover test pick a mounted directory and check it is returned Co-authored-by: Radhey Kalra <radheykalra901@gmail.com> --------- Co-authored-by: usehoplite[bot] <288093033+usehoplite[bot]@users.noreply.github.com> Co-authored-by: Radhey Kalra <radheykalra901@gmail.com>
* Make chunk_size_mb control browser upload chunks Co-authored-by: Radhey Kalra <radheykalra901@gmail.com> * Share one chunk size request across upload workers and respect cancel Co-authored-by: Radhey Kalra <radheykalra901@gmail.com> --------- Co-authored-by: usehoplite[bot] <288093033+usehoplite[bot]@users.noreply.github.com> Co-authored-by: Radhey Kalra <radheykalra901@gmail.com>
Co-authored-by: usehoplite[bot] <288093033+usehoplite[bot]@users.noreply.github.com> Co-authored-by: Radhey Kalra <radheykalra901@gmail.com>
* Parse WebSocket and job data at the boundary Co-authored-by: Radhey Kalra <radheykalra901@gmail.com> * Validate job progress range, add parser tests, log rejected frames briefly Co-authored-by: Radhey Kalra <radheykalra901@gmail.com> --------- Co-authored-by: usehoplite[bot] <288093033+usehoplite[bot]@users.noreply.github.com> Co-authored-by: Radhey Kalra <radheykalra901@gmail.com>
* Migrate jobs and websocket stores to runes Co-authored-by: Radhey Kalra <radheykalra901@gmail.com> * Migrate settings store to runes and split out appearance helpers Co-authored-by: Radhey Kalra <radheykalra901@gmail.com> * Migrate auth store to runes Co-authored-by: Radhey Kalra <radheykalra901@gmail.com> * Split the browse page into composables Co-authored-by: Radhey Kalra <radheykalra901@gmail.com> * Keep the WebSocket reconnect backoff when connect runs in an effect Co-authored-by: Radhey Kalra <radheykalra901@gmail.com> --------- Co-authored-by: usehoplite[bot] <288093033+usehoplite[bot]@users.noreply.github.com> Co-authored-by: Radhey Kalra <radheykalra901@gmail.com>
* Fix the auth API docs, drop stale spec comments, add AGENTS.md Co-authored-by: Radhey Kalra <radheykalra901@gmail.com> * Add the frontend test command to AGENTS.md Co-authored-by: Radhey Kalra <radheykalra901@gmail.com> * Correct the logout docs and scope the path rule Co-authored-by: Radhey Kalra <radheykalra901@gmail.com> --------- Co-authored-by: usehoplite[bot] <288093033+usehoplite[bot]@users.noreply.github.com> Co-authored-by: Radhey Kalra <radheykalra901@gmail.com>
* Fix --dev failing config validation Co-authored-by: Radhey Kalra <radheykalra901@gmail.com> * Isolate the dev credentials test from inherited BoxBox settings Co-authored-by: Radhey Kalra <radheykalra901@gmail.com> --------- Co-authored-by: usehoplite[bot] <288093033+usehoplite[bot]@users.noreply.github.com> Co-authored-by: Radhey Kalra <radheykalra901@gmail.com>
* Test the WebSocket reconnect backoff Co-authored-by: Radhey Kalra <radheykalra901@gmail.com> * Keep the connection mode on WebSocket retries and test the attempt limit Co-authored-by: Radhey Kalra <radheykalra901@gmail.com> --------- Co-authored-by: usehoplite[bot] <288093033+usehoplite[bot]@users.noreply.github.com> Co-authored-by: Radhey Kalra <radheykalra901@gmail.com>
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
|
Important Review skippedThis PR was authored by a bot without an assigned CodeRabbit review seat. Ask an organization administrator to assign a seat to the bot. See the seat-assignment guide. After the seat is assigned, request a review again. ⚙️ Run configurationConfiguration used: Repository UI Review profile: ASSERTIVE Plan: Advanced Run ID: You can disable this status message by setting the
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
| ReadOnly: mp.ReadOnly, | ||
| } | ||
| } | ||
| mountPoints := cfg.MountPoints |
There was a problem hiding this comment.
When auto_discover finds submounts, drive stats lists each one under its own name, and clicking its card navigates to that name. This code still gives the file service and mount guard only the configured parent mount, so they reject the discovered name instead of opening the drive. The displayed drives need paths those services can resolve.
Co-authored-by: Radhey Kalra <radheykalra901@gmail.com>
Co-authored-by: Radhey Kalra <radheykalra901@gmail.com>
Summary
staging/maincontains eight merged commits that have not reached BoxBox's default branch. This PR promotes them tomaster, bringing mount and virtual-path fixes, configurable upload chunk sizing, WebSocket/job validation and reconnect coverage, the Svelte rune-based browse/store refactor, the--devstartup fix, and related documentation into the release branch.Change type
Safety checklist
ghcr.io/jr4dh3y/boxbox:latestandv*tags; development builds usenightlyor branch-scoped tags. (No release-publishing workflows changed.)latest.Verification
go vet ./...andgo test ./...pass with Go 1.27.1.bun run lint,bun run check,bun run test(9 passed), andbun run buildpass. The build reports a Vite chunk-size warning.docker build --platform linux/amd64 --tag boxbox:staging-review .passes.Release notes
Created by openai/gpt-6-luna with Hoplite.
The PR should not merge until discovered drive cards open paths that the backend accepts.
Findings
Summary
This PR promotes backend mount and upload configuration changes, job and WebSocket handling, the Svelte rune-based browse/store refactor, the
--devstartup fix, and related tests and documentation. The discovered-drive display and its navigable paths need to agree before promotion.Diagram
%%{init: {'theme': 'neutral'}}%% flowchart LR C[Configured auto-discover parent] --> R[Roots and mount guard: parent name] C --> D[Drive stats: discovered submount names] D --> B[Browse card navigates by submount name] B --> RReviews (1) · Last reviewed commit: "Test the WebSocket reconnect backoff (#1..."