Skip to content

Promote staging/main to master - #140

Merged
jR4dh3y merged 10 commits into
masterfrom
staging/main
Oct 1, 2026
Merged

jR4dh3y merged 10 commits into
masterfrom
staging/main

Conversation

@usehoplite

@usehoplite usehoplite Bot commented Oct 1, 2026 •

Copy link
Copy Markdown
Contributor

Summary

staging/main contains eight merged commits that have not reached BoxBox's default branch. This PR promotes them to master, bringing mount and virtual-path fixes, configurable upload chunk sizing, WebSocket/job validation and reconnect coverage, the Svelte rune-based browse/store refactor, the --dev startup fix, and related documentation into the release branch.

Change type

  • Backend / Go
  • Frontend / Svelte
  • Website / docs
  • Docker / release pipeline
  • Configuration or deployment

Safety checklist

  • I kept stable release publishing limited to ghcr.io/jr4dh3y/boxbox:latest and v* tags; development builds use nightly or branch-scoped tags. (No release-publishing workflows changed.)
  • I did not make branch/test images replace latest.
  • I considered upgrade, rollback, permissions, storage, and configuration impact for self-hosted deployments. No data migration is introduced; the default browser upload chunk changes from 5 MiB to 10 MiB and remains configurable, while the 10 GiB upload limit is unchanged.
  • I did not commit secrets, private paths, generated local files, or environment-specific config.
  • I updated directly affected docs or examples, or this change does not require docs.

Verification

  • Backend Go tests are covered, or this change does not affect backend code. go vet ./... and go test ./... pass with Go 1.27.1.
  • Frontend Svelte checks/build are covered, or this change does not affect the frontend. bun run lint, bun run check, bun run test (9 passed), and bun run build pass. The build reports a Vite chunk-size warning.
  • Website build is covered, or this change does not affect the website. Website source is unchanged.
  • Docker image build is covered without pushing, or this change does not affect Docker packaging. docker build --platform linux/amd64 --tag boxbox:staging-review . passes.

Release notes

  • User-facing change described for release notes.
  • No release note needed.

Created by openai/gpt-6-luna with Hoplite.

RetriggerConfidence Score: 4/5

The PR should not merge until discovered drive cards open paths that the backend accepts.

Findings

  1. P1 Discovered drives cannot open ▶

Summary

This PR promotes backend mount and upload configuration changes, job and WebSocket handling, the Svelte rune-based browse/store refactor, the --dev startup 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 --> R
Loading

Reviews (1) · Last reviewed commit: "Test the WebSocket reconnect backoff (#1..."

usehoplite Bot and others added 8 commits October 1, 2026 13:45
* 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>
@vercel

vercel Bot commented Oct 1, 2026 •

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated
box-box Ready Ready Preview Oct 1, 2026 9:32am UTC

@coderabbitai

coderabbitai Bot commented Oct 1, 2026 •

Copy link
Copy Markdown
Contributor

Important

Review skipped

This 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 configuration

Configuration used: Repository UI

Review profile: ASSERTIVE

Plan: Advanced

Run ID: 82b54f71-4eb0-4a26-8683-543ce4da4fa5

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

  • Autopilot · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

Autopilot is currently an internal CodeRabbit preview.


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.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

Comment thread backend/cmd/server/main.go Outdated
ReadOnly: mp.ReadOnly,
}
}
mountPoints := cfg.MountPoints

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

P1 Discovered drives cannot open

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>
@jR4dh3y
jR4dh3y merged commit 289e4aa into master Oct 1, 2026
12 checks passed

This branch was successfully deployed

1 active deployment
Preview — a4145c49 Deployed Oct 1, 2026 by vercel[bot]
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant