@geravant/sinain 1.22.3 → 1.22.4

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@geravant/sinain",
-  "version": "1.22.3",
+  "version": "1.22.4",
   "description": "Ambient intelligence that sees what you see, hears what you hear, and acts on your behalf",
   "type": "module",
   "bin": {

package/sinain-core/src/server.ts

@@ -896,30 +896,43 @@ async function renderEntityPage(entity) {
 
   // Auto-import path for share links — runs BEFORE the local existence check
   // so a recipient with no prior data on this entity gets the page populated.
+  // We KEEP the hash on failure so a refresh retries; only strip on success.
+  let shareError = null;
+  let shareMode = null;
   if (location.hash.startsWith("#bundle=")) {
+    shareMode = "bundle";
     try {
       const json = await ungzipBase64(location.hash.slice("#bundle=".length));
       await api("/knowledge/concepts/import?conflict=merge", {
         method: "POST", headers: {"Content-Type": "application/json"}, body: json
       });
       showToast("✓ Concept imported");
+      history.replaceState({}, "", location.pathname);  // strip on success only
     } catch (e) {
-      showToast("Import failed: " + (e.message || "decode error"));
+      shareError = (e && e.message) || "decode error";
     }
-    history.replaceState({}, "", location.pathname);  // strip hash
   } else if (location.hash.startsWith("#peer=")) {
+    shareMode = "peer";
     const token = location.hash.slice("#peer=".length);
-    history.replaceState({}, "", location.pathname);  // strip early — keeps refresh sane
     try {
       await ShareManager.connectAsRecipient(token);
       showToast("✓ Concept imported via peer");
+      history.replaceState({}, "", location.pathname);  // strip on success only
     } catch (e) {
-      showToast("Peer share failed: " + (e.message || "unreachable"));
+      shareError = (e && e.message) || "unreachable";
     }
   }
 
   const page = await api("/knowledge/page?entity=" + encodeURIComponent(entity));
   if (!page.ok || page.fact_count === 0) {
+    // If a share-import was attempted and failed AND we have no local data,
+    // render an explicit share-failed view rather than the generic
+    // MissingConcept page — the user needs to know the share didn't land,
+    // not just that the entity is absent.
+    if (shareError && page.fact_count === 0) {
+      renderShareFailed(entity, shareMode, shareError, root);
+      return;
+    }
     if (page.fact_count === 0) {
       renderMissingConcept(entity, root);
       return;
@@ -1184,6 +1197,50 @@ function showUndoToast(factId, undoToken, sourceEntity) {
   setTimeout(() => { if ($("#undoToast")) root.innerHTML = ""; }, ms);
 }
 
+// ── Share-failed landing ──────────────────────────────────────────────────
+// Shown when a #bundle= or #peer= share-link couldn't be imported AND no
+// local data exists for the entity. Distinct from MissingConcept because
+// the user explicitly tried to load shared data — they need to know that
+// transfer failed, not just that the entity isn't here.
+function renderShareFailed(entity, mode, errorMsg, root) {
+  document.title = "Share couldn't load · Sinain";
+  const isPeer = mode === "peer";
+  root.innerHTML = \`
+    <h1>Share couldn't be loaded</h1>
+    <div class="error-block" style="margin-bottom: 16px;">
+      <strong>\${esc(entity)}</strong> — \${isPeer ? "couldn't receive bundle from sender" : "couldn't decode bundle from URL"}.
+      <br><br>
+      <code>\${esc(errorMsg)}</code>
+    </div>
+    \${isPeer ? \`
+    <p style="color: var(--fg-dim); line-height: 1.5;">
+      Likely causes:
+    </p>
+    <ul style="color: var(--fg-dim); line-height: 1.6;">
+      <li>The sender's tab is closed (peer share needs sender's sinain-core open).</li>
+      <li>The share has expired or was revoked.</li>
+      <li>Network/NAT blocked the WebRTC connection (about 10–20% of pairs need a TURN relay).</li>
+    </ul>
+    <p style="color: var(--fg-dim);">
+      Ask the sender to reopen sinain (their peer registration auto-resumes), then click <strong>Retry</strong>.
+    </p>\` : \`
+    <p style="color: var(--fg-dim); line-height: 1.5;">
+      The link's <code>#bundle=…</code> portion may have been truncated by a chat tool — some preview tools strip URL fragments.
+      Ask the sender to re-share, ideally pasted as a code block so the URL stays intact.
+    </p>\`}
+    <div class="actions" style="margin-top: 20px; display: flex; gap: 10px;">
+      <button class="primary" onclick="location.reload()">Retry</button>
+      <button onclick="window._proceedWithoutShare('\${esc(entity)}')">Continue without share</button>
+    </div>
+  \`;
+  // Continue-without-share clears the hash so the next render takes the
+  // normal MissingConcept path (which has the file-drop dropzone).
+  window._proceedWithoutShare = (eid) => {
+    history.replaceState({}, "", "/knowledge/ui/entity/" + encodeURIComponent(eid));
+    renderMissingConcept(eid, root);
+  };
+}
+
 // ── Missing concept landing ───────────────────────────────────────────────
 function renderMissingConcept(entity, root) {
   document.title = "Missing · " + entity;