@geravant/sinain 1.22.1 → 1.22.4

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@geravant/sinain",
-  "version": "1.22.1",
+  "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;
@@ -1286,11 +1343,15 @@ function renderKnowledgeUiV2(): string {
   const peerHost = process.env.SINAIN_PEERJS_HOST || "";  // empty = peerjs.com cloud default
   const inlineMax = parseInt(process.env.SINAIN_SHARE_INLINE_MAX_BYTES || "6000");
   const ttlHours = parseInt(process.env.SINAIN_SHARE_TTL_HOURS || "24");
-  // Public URL of the share-redirector (docs/share.html in the repo). Browsers
-  // preserve URL fragments through redirects without sending them to the
-  // server, so the bundle bytes never touch this CDN.
+  // Public URL of the share-redirector. Self-hosted on the existing
+  // sinain.duckdns.org Caddy + Let's Encrypt setup that already fronts the
+  // OpenClaw gateway. We control the host, headers, and reliability — no
+  // CDN policy quirks (jsDelivr serves gh-path HTML as text/plain;
+  // raw.githack.com returns 403 with body). Browsers preserve URL fragments
+  // through redirects without sending them to the server, so bundle bytes
+  // in #bundle=… never touch our host either.
   const shareBaseUrl = process.env.SINAIN_SHARE_BASE_URL
-    || "https://cdn.jsdelivr.net/gh/anthillnet/sinain-hud@main/docs/share.html";
+    || "https://sinain.duckdns.org/share.html";
   return KNOWLEDGE_UI_V2_HTML
     .replace(/__SHARE_PEERJS_HOST__/g, JSON.stringify(peerHost))
     .replace(/__SHARE_INLINE_MAX_BYTES__/g, String(inlineMax))