fix(downloads): automatic really is automatic, and a selection downloads all of it

Two bugs in what shipped last, and both were mine.

Automatic mode still opened Save As, because with no folder granted the
code fell through to the file picker — while the documentation said it
would use the browser's own download folder. It does that now. Over
512 MB it still asks, since getting there means holding the file in
memory and a tab will not survive a 40 GB blob; Settings is where to stop
it asking again.

Selecting two files downloaded one. They were started without awaiting,
so each asked the browser for a save dialog at once, and a browser allows
exactly one — the rest were rejected and the errors went nowhere. They
are awaited one at a time now, which serializes the dialogs and not the
transfers: each call returns as soon as its transfer is registered.

Then the adjustments. The transfers widget offers Open on a finished
download that went into a granted folder — the bytes go to a new tab, and
that is the whole of what a page can do: no browser lets one start a
desktop application or show a file manager, so the folder half of that
request cannot be built and the guide says so.

The Files toolbar was four controls of three different heights in a row.
It is three groups now — what you can add, where you are, what you can do
with what is here — on one baseline, with icons from the set and a gap
between the dots and the word Actions. Chat comes first among the tabs
and is the one you land on. The three Discover entries in the sidebar
have icons. And a link in a chat message becomes a link: built as an
element and never as markup, http and https only, so `javascript:` is not
one message away from running here.

Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>

1 files changed, 30 insertions, 0 deletions

diff --git a/packages/meshbay-hub/tests/test_spa_ordering.py b/packages/meshbay-hub/tests/test_spa_ordering.py
index 04c5159..79d7c9a 100644
--- a/packages/meshbay-hub/tests/test_spa_ordering.py
+++ b/packages/meshbay-hub/tests/test_spa_ordering.py
@@ -199,3 +199,33 @@ def test_the_files_panel_no_longer_carries_its_own_progress_bars():
     app = APP.read_text()
     for gone in ("setUlState", "setDlState", "dl-bar"):
         assert gone not in app, f"{gone} survived the move to the transfer widget"
+
+
+# ── Downloading a selection ─────────────────────────────────────────────────
+
+def test_a_multi_file_download_waits_for_each_picker():
+    """
+    A browser allows one file picker at a time. Firing every download at once
+    meant the first opened a dialog and the rest were rejected — two files
+    selected, one file downloaded.
+    """
+    app = APP.read_text()
+    block = app[app.index("${selectedFiles.length > 0 && html`"):]
+    block = block[:block.index("`}")]
+    assert "await downloadFile(e)" in block, (
+        "downloads are fired without awaiting again; only the first will ask "
+        "for a save location and the others will be rejected")
+
+
+def test_links_in_chat_are_built_as_elements_not_markup():
+    """
+    A message is something another member wrote. It becomes an anchor element,
+    never HTML, and only for http(s) — otherwise javascript: would be one
+    message away from running here.
+    """
+    app = APP.read_text()
+    fn = app[app.index("function linkify("):]
+    fn = fn[:fn.index("\nfunction ", 1)]
+    assert "innerHTML" not in fn and "dangerouslySetInnerHTML" not in fn
+    assert 'rel="noopener noreferrer"' in fn
+    assert "https?" in app[app.index("const URL_RE"):app.index("function linkify(")]