pmtiles-swarm 0.99.0 → 0.99.2

package/CHANGELOG.md

@@ -7,6 +7,34 @@
 ### 🐞 Bug fixes
 - _...Add new stuff here..._
 
+## 0.99.2
+### ✨ Features and improvements
+
+### 🐞 Bug fixes
+- **The stack editor named a pinned archive by its infohash.** The list of saved stacks was fixed in
+  0.98.3, but the editor has its own row and kept showing `33394361a935…` — where the menu that
+  offered the archive a moment earlier had shown its filename. The editor works on the recipe, and
+  a pinned source in a recipe is an infohash and nothing else, so the row had nothing else to
+  print. It remembers the names the source menu already fetched now, and the infohash is on hover.
+
+## 0.99.1
+### ✨ Features and improvements
+
+### 🐞 Bug fixes
+- **The sidecar stops segfaulting, which is what left a library half loaded after a restart.**
+  `pmtiles-torrent` 0.11.1. The traceback 0.99.0 made possible named the same frame every time: the
+  alert pump, inside `session.wait_for_alert()`. That call hands back a borrowed pointer to an
+  alert which the next `pop_alerts()` — the following statement — invalidates, and nothing ever
+  read the return value; it was only a way to sleep. The pump waits on its own stop event now.
+
+  This is the cause behind the `not loaded` archives, not another symptom of them. Each crash took
+  whatever restore had handed over before it, because the replacement sidecar holds none of it, so
+  the archives restored first were the ones that came back missing. The node stops that leaving a
+  mark either way — it hands them back — but they should now not be dropped in the first place.
+
+  Sidecar shutdown is quicker too: the pump answers the stop at once, where the old wait had to be
+  sat through to the end of its timeout.
+
 ## 0.99.0
 ### ✨ Features and improvements
 - **A sidecar crash now names the frame it happened in.** `pmtiles-torrent` 0.11.0 arms

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "pmtiles-swarm",
-  "version": "0.99.0",
+  "version": "0.99.2",
   "description": "BitTorrent distribution for PMTiles map archives: create torrents, watch folders, publish and subscribe to RSS feeds, and seed through qBittorrent or an embedded client",
   "type": "module",
   "main": "src/index.js",
@@ -49,7 +49,7 @@
     "maplibre-gl": "^6.2.0",
     "parse-torrent": "^11.0.24",
     "pmtiles": "^4.4.1",
-    "pmtiles-torrent": "^0.11.0",
+    "pmtiles-torrent": "^0.11.1",
     "webtorrent": "^3.0.21"
   },
   "engines": {

package/src/web/index.html

@@ -8152,6 +8152,25 @@ Every piece is hashed against the ` +
        * one over an infohash pins to a build that will eventually be retired.
        * @returns {Promise<void>} - Resolves once filled.
        */
+      /**
+       * Archive names by infohash, learned while filling the source menu.
+       *
+       * The editor works on the recipe, and a pinned source in a recipe is an
+       * infohash and nothing else -- so a row drawn from the draft alone can
+       * only show forty characters of hex, which is what it used to do. The
+       * menu that offered the archive had its name a moment earlier; this is
+       * where that is kept so the row can use it.
+       */
+      const archiveNames = new Map();
+
+      /**
+       * What to call a pinned archive in the editor.
+       * @param {string} infoHash - The archive the recipe names.
+       * @returns {string} - Its name, or the short hash where none is known.
+       */
+      const archiveLabel = (infoHash) =>
+        archiveNames.get(infoHash) ?? shortHash(infoHash) ?? '';
+
       const fillStackSourceChoices = async () => {
         const select = $('stack-add-source');
         const taken = new Set(
@@ -8212,6 +8231,14 @@ Every piece is hashed against the ` +
           // missing from a menu the operator is looking at. This is the same
           // endpoint the Archives view reads, and it answers a bare array.
           const held = await api('/api/torrents');
+          // Every archive, before the menu is filtered down. The one the
+          // editor most needs a name for is the one already in the draft,
+          // which is exactly what `taken` removes below.
+          for (const entry of held ?? []) {
+            if (entry.infoHash && entry.name) {
+              archiveNames.set(entry.infoHash, entry.name);
+            }
+          }
           // The same rule the tile route enforces, because a stack source is
           // read through exactly that path: a complete MBTiles belongs here,
           // and one still arriving does not -- a stack naming it would answer
@@ -8549,12 +8576,16 @@ Every piece is hashed against the ` +
           <div class="card tight">
             <div class="row">
               <span class="muted">${index}</span>
-              <code>${escapeHtml(
+              <code${
+                source.archive
+                  ? ` title="${escapeHtml(source.archive)}"`
+                  : ''
+              }>${escapeHtml(
                 remote
                   ? (source.url || 'an address').replace(/^https?:\/\//, '')
                   : (source.stack ??
                       source.category ??
-                      shortHash(source.archive) ??
+                      (source.archive ? archiveLabel(source.archive) : '') ??
                       ''),
               )}</code>
               <span class="sub" title="${