@memtensor/memos-local-openclaw-plugin 1.0.4 → 1.0.6-beta.1

package/src/viewer/server.ts

@@ -1,7 +1,7 @@
 import http from "node:http";
 import os from "node:os";
 import crypto from "node:crypto";
-import { execSync, exec } from "node:child_process";
+import { execSync, exec, execFile } from "node:child_process";
 import fs from "node:fs";
 import path from "node:path";
 import readline from "node:readline";
@@ -22,9 +22,89 @@ import type { Logger, Chunk, PluginContext, MemosLocalConfig } from "../types";
 import { viewerHTML } from "./html";
 import { v4 as uuid } from "uuid";
 
-function normalizeTimestamp(ts: number): number {
-  if (ts < 1e12) return ts * 1000;
-  return ts;
+export interface MigrationStepFailureCounts {
+  summarization: number;
+  dedup: number;
+  embedding: number;
+}
+
+export interface MigrationStateSnapshot {
+  phase: string;
+  stored: number;
+  skipped: number;
+  merged: number;
+  errors: number;
+  processed: number;
+  total: number;
+  lastItem: any;
+  done: boolean;
+  stopped: boolean;
+  stepFailures: MigrationStepFailureCounts;
+  success: boolean;
+}
+
+function createInitialStepFailures(): MigrationStepFailureCounts {
+  return { summarization: 0, dedup: 0, embedding: 0 };
+}
+
+export function computeMigrationSuccess(state: Pick<MigrationStateSnapshot, "errors" | "stepFailures">): boolean {
+  const sf = state.stepFailures;
+  return state.errors === 0 && sf.summarization === 0 && sf.dedup === 0 && sf.embedding === 0;
+}
+
+export function createInitialMigrationState(): MigrationStateSnapshot {
+  const stepFailures = createInitialStepFailures();
+  return {
+    phase: "",
+    stored: 0,
+    skipped: 0,
+    merged: 0,
+    errors: 0,
+    processed: 0,
+    total: 0,
+    lastItem: null,
+    done: false,
+    stopped: false,
+    stepFailures,
+    success: computeMigrationSuccess({ errors: 0, stepFailures }),
+  };
+}
+
+export function applyMigrationItemToState(state: MigrationStateSnapshot, d: any): void {
+  if (d.status === "stored") state.stored++;
+  else if (d.status === "skipped" || d.status === "duplicate") state.skipped++;
+  else if (d.status === "merged") state.merged++;
+  else if (d.status === "error") state.errors++;
+
+  if (Array.isArray(d.stepFailures)) {
+    for (const step of d.stepFailures) {
+      if (step === "summarization") state.stepFailures.summarization++;
+      else if (step === "dedup") state.stepFailures.dedup++;
+      else if (step === "embedding") state.stepFailures.embedding++;
+    }
+  }
+
+  state.processed = d.index ?? state.processed + 1;
+  state.total = d.total ?? state.total;
+  state.lastItem = d;
+  state.success = computeMigrationSuccess(state);
+}
+
+/** Epoch ms for Chunk; OpenClaw SQLite may store Unix seconds or ms. */
+function normalizeTimestamp(value: unknown): number {
+  if (value == null) return Date.now();
+  if (typeof value === "string") {
+    const parsed = Date.parse(value.trim());
+    if (Number.isFinite(parsed)) return parsed;
+    const n = Number(value);
+    if (Number.isFinite(n)) return normalizeTimestamp(n);
+    return Date.now();
+  }
+  if (typeof value === "number" && Number.isFinite(value)) {
+    if (value > 0 && value < 5_000_000_000) return Math.round(value * 1000);
+    return Math.round(value);
+  }
+  return Date.now();
 }
 
 export interface ViewerServerOptions {
@@ -67,18 +147,7 @@ export class ViewerServer {
   private resetToken: string;
   private migrationRunning = false;
   private migrationAbort = false;
-  private migrationState: {
-    phase: string;
-    stored: number;
-    skipped: number;
-    merged: number;
-    errors: number;
-    processed: number;
-    total: number;
-    lastItem: any;
-    done: boolean;
-    stopped: boolean;
-  } = { phase: "", stored: 0, skipped: 0, merged: 0, errors: 0, processed: 0, total: 0, lastItem: null, done: false, stopped: false };
+  private migrationState: MigrationStateSnapshot = createInitialMigrationState();
   private migrationSSEClients: http.ServerResponse[] = [];
 
   private ppRunning = false;
@@ -491,13 +560,12 @@ export class ViewerServer {
     if (chunkIds.length > 0) {
       try {
         const placeholders = chunkIds.map(() => "?").join(",");
-        const sharedRows = db.prepare(`SELECT source_chunk_id, visibility, group_id FROM hub_memories WHERE source_chunk_id IN (${placeholders})`).all(...chunkIds) as Array<{ source_chunk_id: string; visibility: string; group_id: string | null }>;
-        for (const r of sharedRows) sharingMap.set(r.source_chunk_id, r);
-        const teamMetaRows = db.prepare(`SELECT chunk_id, visibility, group_id FROM team_shared_chunks WHERE chunk_id IN (${placeholders})`).all(...chunkIds) as Array<{ chunk_id: string; visibility: string; group_id: string | null }>;
-        for (const r of teamMetaRows) {
-          if (!sharingMap.has(r.chunk_id)) {
-            sharingMap.set(r.chunk_id, { visibility: r.visibility, group_id: r.group_id });
-          }
+        if (this.sharingRole === "hub") {
+          const sharedRows = db.prepare(`SELECT source_chunk_id, visibility, group_id FROM hub_memories WHERE source_chunk_id IN (${placeholders})`).all(...chunkIds) as Array<{ source_chunk_id: string; visibility: string; group_id: string | null }>;
+          for (const r of sharedRows) sharingMap.set(r.source_chunk_id, r);
+        } else {
+          const teamMetaRows = db.prepare(`SELECT chunk_id, visibility, group_id FROM team_shared_chunks WHERE chunk_id IN (${placeholders})`).all(...chunkIds) as Array<{ chunk_id: string; visibility: string; group_id: string | null }>;
+          for (const r of teamMetaRows) sharingMap.set(r.chunk_id, { visibility: r.visibility, group_id: r.group_id });
         }
         const localRows = db.prepare(`SELECT chunk_id, original_owner, shared_at FROM local_shared_memories WHERE chunk_id IN (${placeholders})`).all(...chunkIds) as Array<{ chunk_id: string; original_owner: string; shared_at: number }>;
         for (const r of localRows) localShareMap.set(r.chunk_id, r);
@@ -564,7 +632,7 @@ export class ViewerServer {
     const db = (this.store as any).db;
     const items = tasks.map((t) => {
       const meta = db.prepare("SELECT skill_status, owner FROM tasks WHERE id = ?").get(t.id) as { skill_status: string | null; owner: string | null } | undefined;
-      const sharedTask = db.prepare("SELECT visibility FROM hub_tasks WHERE source_task_id = ? ORDER BY updated_at DESC LIMIT 1").get(t.id) as { visibility: string } | undefined;
+      const hubTask = this.getHubTaskForLocal(t.id);
       return {
         id: t.id,
         sessionKey: t.sessionKey,
@@ -576,7 +644,7 @@ export class ViewerServer {
         chunkCount: this.store.countChunksByTask(t.id),
         skillStatus: meta?.skill_status ?? null,
         owner: meta?.owner ?? "agent:main",
-        sharingVisibility: sharedTask?.visibility ?? null,
+        sharingVisibility: hubTask?.visibility ?? null,
       };
     });
 
@@ -611,7 +679,7 @@ export class ViewerServer {
     const db = (this.store as any).db;
     const meta = db.prepare("SELECT skill_status, skill_reason FROM tasks WHERE id = ?").get(taskId) as
       { skill_status: string | null; skill_reason: string | null } | undefined;
-    const sharedTask = db.prepare("SELECT visibility, group_id FROM hub_tasks WHERE source_task_id = ? ORDER BY updated_at DESC LIMIT 1").get(taskId) as { visibility: string | null; group_id: string | null } | undefined;
+    const hubTask = this.getHubTaskForLocal(taskId);
 
     this.jsonResponse(res, {
       id: task.id,
@@ -626,9 +694,9 @@ export class ViewerServer {
       skillStatus: meta?.skill_status ?? null,
       skillReason: meta?.skill_reason ?? null,
       skillLinks,
-      sharingVisibility: sharedTask?.visibility ?? null,
-      sharingGroupId: sharedTask?.group_id ?? null,
-      hubTaskId: sharedTask ? true : false,
+      sharingVisibility: hubTask?.visibility ?? null,
+      sharingGroupId: hubTask?.group_id ?? null,
+      hubTaskId: hubTask ? true : false,
     });
   }
 
@@ -818,10 +886,9 @@ export class ViewerServer {
     if (visibility) {
       skills = skills.filter(s => s.visibility === visibility);
     }
-    const db = (this.store as any).db;
     const enriched = skills.map(s => {
-      const hub = db.prepare("SELECT visibility FROM hub_skills WHERE source_skill_id = ? ORDER BY updated_at DESC LIMIT 1").get(s.id) as { visibility: string } | undefined;
-      return { ...s, sharingVisibility: hub?.visibility ?? null };
+      const hubSkill = this.getHubSkillForLocal(s.id);
+      return { ...s, sharingVisibility: hubSkill?.visibility ?? null };
     });
     this.jsonResponse(res, { skills: enriched });
   }
@@ -839,11 +906,10 @@ export class ViewerServer {
     const relatedTasks = this.store.getTasksBySkill(skillId);
     const files = fs.existsSync(skill.dirPath) ? this.walkDir(skill.dirPath, skill.dirPath) : [];
 
-    const db = (this.store as any).db;
-    const sharedSkill = db.prepare("SELECT visibility, group_id FROM hub_skills WHERE source_skill_id = ? ORDER BY updated_at DESC LIMIT 1").get(skillId) as { visibility: string | null; group_id: string | null } | undefined;
+    const hubSkill = this.getHubSkillForLocal(skillId);
 
     this.jsonResponse(res, {
-      skill: { ...skill, sharingVisibility: sharedSkill?.visibility ?? null, sharingGroupId: sharedSkill?.group_id ?? null },
+      skill: { ...skill, sharingVisibility: hubSkill?.visibility ?? null, sharingGroupId: hubSkill?.group_id ?? null },
       versions: versions.map(v => ({
         id: v.id,
         version: v.version,
@@ -982,7 +1048,7 @@ export class ViewerServer {
                 method: "POST",
                 body: JSON.stringify({ visibility: "public", groupId: null, metadata: bundle.metadata, bundle: bundle.bundle }),
               }) as any;
-              if (hubClient.userId) {
+              if (this.sharingRole === "hub" && hubClient.userId) {
                 const existing = this.store.getHubSkillBySource(hubClient.userId, skillId);
                 this.store.upsertHubSkill({
                   id: response?.skillId ?? existing?.id ?? crypto.randomUUID(),
@@ -992,6 +1058,14 @@ export class ViewerServer {
                   bundle: JSON.stringify(bundle.bundle), qualityScore: skill.qualityScore,
                   createdAt: existing?.createdAt ?? Date.now(), updatedAt: Date.now(),
                 });
+              } else {
+                const conn = this.store.getClientHubConnection();
+                this.store.upsertTeamSharedSkill(skillId, {
+                  hubSkillId: String(response?.skillId ?? ""),
+                  visibility: "public",
+                  groupId: null,
+                  hubInstanceId: conn?.hubInstanceId ?? "",
+                });
               }
               hubSynced = true;
               this.log.info(`Skill "${skill.name}" published to Hub`);
@@ -1000,7 +1074,8 @@ export class ViewerServer {
                 method: "POST",
                 body: JSON.stringify({ sourceSkillId: skillId }),
               });
-              if (hubClient.userId) this.store.deleteHubSkillBySource(hubClient.userId, skillId);
+              if (this.sharingRole === "hub" && hubClient.userId) this.store.deleteHubSkillBySource(hubClient.userId, skillId);
+              else this.store.deleteTeamSharedSkill(skillId);
               hubSynced = true;
               this.log.info(`Skill "${skill.name}" unpublished from Hub`);
             }
@@ -1271,7 +1346,8 @@ export class ViewerServer {
                 createdAt: existing?.createdAt ?? Date.now(), updatedAt: Date.now(),
               });
             } else if (hubClient.userId) {
-              this.store.upsertTeamSharedChunk(chunkId, { hubMemoryId: memoryId, visibility: "public", groupId: null });
+              const conn = this.store.getClientHubConnection();
+              this.store.upsertTeamSharedChunk(chunkId, { hubMemoryId: memoryId, visibility: "public", groupId: null, hubInstanceId: conn?.hubInstanceId ?? "" });
             }
             hubSynced = true;
           } else {
@@ -1284,7 +1360,7 @@ export class ViewerServer {
               await hubRequestJson(hubClient.hubUrl, hubClient.userToken, "/api/v1/hub/memories/unshare", {
                 method: "POST", body: JSON.stringify({ sourceChunkId: chunkId }),
               });
-              if (hubClient.userId) this.store.deleteHubMemoryBySource(hubClient.userId, chunkId);
+              if (this.sharingRole === "hub" && hubClient.userId) this.store.deleteHubMemoryBySource(hubClient.userId, chunkId);
               this.store.deleteTeamSharedChunk(chunkId);
               hubSynced = true;
             } catch (err) { this.log.warn(`Failed to unshare memory from team: ${err}`); }
@@ -1297,7 +1373,7 @@ export class ViewerServer {
               await hubRequestJson(hubClient.hubUrl, hubClient.userToken, "/api/v1/hub/memories/unshare", {
                 method: "POST", body: JSON.stringify({ sourceChunkId: chunkId }),
               });
-              if (hubClient.userId) this.store.deleteHubMemoryBySource(hubClient.userId, chunkId);
+              if (this.sharingRole === "hub" && hubClient.userId) this.store.deleteHubMemoryBySource(hubClient.userId, chunkId);
               this.store.deleteTeamSharedChunk(chunkId);
               hubSynced = true;
             } catch (err) { this.log.warn(`Failed to unshare memory from team: ${err}`); }
@@ -1351,21 +1427,24 @@ export class ViewerServer {
                 chunks: chunks.map((c) => ({ id: c.id, hubTaskId: refreshedTask.id, sourceTaskId: refreshedTask.id, sourceChunkId: c.id, role: c.role, content: c.content, summary: c.summary, kind: c.kind, groupId: null, visibility: "public", createdAt: c.createdAt ?? Date.now() })),
               }),
             });
-            if (hubClient.userId) {
+            const hubTaskId = String((response as any)?.taskId ?? "");
+            if (this.sharingRole === "hub" && hubClient.userId) {
               const existing = this.store.getHubTaskBySource(hubClient.userId, taskId);
               this.store.upsertHubTask({
-                id: (response as any)?.taskId ?? existing?.id ?? crypto.randomUUID(),
+                id: hubTaskId || existing?.id || crypto.randomUUID(),
                 sourceTaskId: taskId, sourceUserId: hubClient.userId, title: refreshedTask.title ?? "",
                 summary: refreshedTask.summary ?? "", groupId: null, visibility: "public",
                 createdAt: existing?.createdAt ?? Date.now(), updatedAt: Date.now(),
               });
             }
+            const conn = this.store.getClientHubConnection();
+            this.store.markTaskShared(taskId, hubTaskId, chunks.length, "public", null, conn?.hubInstanceId ?? "");
             hubSynced = true;
           }
           if (!isLocalShared) {
             const originalOwner = task.owner;
             const db = (this.store as any).db;
-            db.prepare("INSERT INTO local_shared_tasks (task_id, hub_task_id, original_owner, shared_at) VALUES (?, ?, ?, ?) ON CONFLICT(task_id) DO UPDATE SET original_owner = excluded.original_owner, shared_at = excluded.shared_at").run(taskId, "", originalOwner, Date.now());
+            db.prepare("INSERT INTO local_shared_tasks (task_id, hub_task_id, original_owner, hub_instance_id, shared_at) VALUES (?, ?, ?, ?, ?) ON CONFLICT(task_id) DO UPDATE SET original_owner = excluded.original_owner, hub_instance_id = excluded.hub_instance_id, shared_at = excluded.shared_at").run(taskId, "", originalOwner, "", Date.now());
             db.prepare("UPDATE tasks SET owner = 'public' WHERE id = ?").run(taskId);
           }
         }
@@ -1385,7 +1464,8 @@ export class ViewerServer {
             await hubRequestJson(hubClient.hubUrl, hubClient.userToken, "/api/v1/hub/tasks/unshare", {
               method: "POST", body: JSON.stringify({ sourceTaskId: taskId }),
             });
-            if (hubClient.userId) this.store.deleteHubTaskBySource(hubClient.userId, taskId);
+            if (this.sharingRole === "hub" && hubClient.userId) this.store.deleteHubTaskBySource(hubClient.userId, taskId);
+            else this.store.downgradeTeamSharedTaskToLocal(taskId);
             hubSynced = true;
           } catch (err) { this.log.warn(`Failed to unshare task from team: ${err}`); }
         }
@@ -1397,7 +1477,8 @@ export class ViewerServer {
               await hubRequestJson(hubClient.hubUrl, hubClient.userToken, "/api/v1/hub/tasks/unshare", {
                 method: "POST", body: JSON.stringify({ sourceTaskId: taskId }),
               });
-              if (hubClient.userId) this.store.deleteHubTaskBySource(hubClient.userId, taskId);
+              if (this.sharingRole === "hub" && hubClient.userId) this.store.deleteHubTaskBySource(hubClient.userId, taskId);
+              else if (!isLocalShared) this.store.unmarkTaskShared(taskId);
               hubSynced = true;
             } catch (err) { this.log.warn(`Failed to unshare task from team: ${err}`); }
           }
@@ -1454,16 +1535,20 @@ export class ViewerServer {
               method: "POST",
               body: JSON.stringify({ visibility: "public", groupId: null, metadata: bundle.metadata, bundle: bundle.bundle }),
             });
-            if (hubClient.userId) {
+            const hubSkillId = String((response as any)?.skillId ?? "");
+            if (this.sharingRole === "hub" && hubClient.userId) {
               const existing = this.store.getHubSkillBySource(hubClient.userId, skillId);
               this.store.upsertHubSkill({
-                id: (response as any)?.skillId ?? existing?.id ?? crypto.randomUUID(),
+                id: hubSkillId || existing?.id || crypto.randomUUID(),
                 sourceSkillId: skillId, sourceUserId: hubClient.userId,
                 name: skill.name, description: skill.description, version: skill.version,
                 groupId: null, visibility: "public",
                 bundle: JSON.stringify(bundle.bundle), qualityScore: skill.qualityScore,
                 createdAt: existing?.createdAt ?? Date.now(), updatedAt: Date.now(),
               });
+            } else {
+              const conn = this.store.getClientHubConnection();
+              this.store.upsertTeamSharedSkill(skillId, { hubSkillId, visibility: "public", groupId: null, hubInstanceId: conn?.hubInstanceId ?? "" });
             }
             hubSynced = true;
           }
@@ -1480,7 +1565,8 @@ export class ViewerServer {
             await hubRequestJson(hubClient.hubUrl, hubClient.userToken, "/api/v1/hub/skills/unpublish", {
               method: "POST", body: JSON.stringify({ sourceSkillId: skillId }),
             });
-            if (hubClient.userId) this.store.deleteHubSkillBySource(hubClient.userId, skillId);
+            if (this.sharingRole === "hub" && hubClient.userId) this.store.deleteHubSkillBySource(hubClient.userId, skillId);
+            else this.store.deleteTeamSharedSkill(skillId);
             hubSynced = true;
           } catch (err) { this.log.warn(`Failed to unpublish skill from team: ${err}`); }
         }
@@ -1492,7 +1578,8 @@ export class ViewerServer {
               await hubRequestJson(hubClient.hubUrl, hubClient.userToken, "/api/v1/hub/skills/unpublish", {
                 method: "POST", body: JSON.stringify({ sourceSkillId: skillId }),
               });
-              if (hubClient.userId) this.store.deleteHubSkillBySource(hubClient.userId, skillId);
+              if (this.sharingRole === "hub" && hubClient.userId) this.store.deleteHubSkillBySource(hubClient.userId, skillId);
+              else this.store.deleteTeamSharedSkill(skillId);
               hubSynced = true;
             } catch (err) { this.log.warn(`Failed to unpublish skill from team: ${err}`); }
           }
@@ -1506,29 +1593,53 @@ export class ViewerServer {
     });
   }
 
+  private get sharingRole(): string | undefined {
+    return this.ctx?.config?.sharing?.role;
+  }
+
+  private isCurrentClientHubInstance(hubInstanceId?: string): boolean {
+    if (this.sharingRole !== "client") return true;
+    const scopedHubInstanceId = String(hubInstanceId ?? "");
+    if (!scopedHubInstanceId) return true;
+    const currentHubInstanceId = this.store.getClientHubConnection()?.hubInstanceId ?? "";
+    if (!currentHubInstanceId) return true;
+    return scopedHubInstanceId === currentHubInstanceId;
+  }
+
   private getHubMemoryForChunk(chunkId: string): any {
-    const db = (this.store as any).db;
-    const hub = db.prepare("SELECT * FROM hub_memories WHERE source_chunk_id = ? LIMIT 1").get(chunkId);
-    if (hub) return hub;
+    if (this.sharingRole === "hub") {
+      const db = (this.store as any).db;
+      return db.prepare("SELECT * FROM hub_memories WHERE source_chunk_id = ? LIMIT 1").get(chunkId);
+    }
     const ts = this.store.getTeamSharedChunk(chunkId);
-    if (ts) {
-      return {
-        source_chunk_id: chunkId,
-        visibility: ts.visibility,
-        group_id: ts.groupId,
-      };
+    if (ts && this.isCurrentClientHubInstance(ts.hubInstanceId)) {
+      return { source_chunk_id: chunkId, visibility: ts.visibility, group_id: ts.groupId };
     }
     return undefined;
   }
 
   private getHubTaskForLocal(taskId: string): any {
-    const db = (this.store as any).db;
-    return db.prepare("SELECT * FROM hub_tasks WHERE source_task_id = ? LIMIT 1").get(taskId);
+    if (this.sharingRole === "hub") {
+      const db = (this.store as any).db;
+      return db.prepare("SELECT * FROM hub_tasks WHERE source_task_id = ? LIMIT 1").get(taskId);
+    }
+    const shared = this.store.getLocalSharedTask(taskId);
+    if (shared && shared.hubTaskId && this.isCurrentClientHubInstance(shared.hubInstanceId)) {
+      return { source_task_id: taskId, visibility: shared.visibility, group_id: shared.groupId };
+    }
+    return undefined;
   }
 
   private getHubSkillForLocal(skillId: string): any {
-    const db = (this.store as any).db;
-    return db.prepare("SELECT * FROM hub_skills WHERE source_skill_id = ? LIMIT 1").get(skillId);
+    if (this.sharingRole === "hub") {
+      const db = (this.store as any).db;
+      return db.prepare("SELECT * FROM hub_skills WHERE source_skill_id = ? LIMIT 1").get(skillId);
+    }
+    const ts = this.store.getTeamSharedSkill(skillId);
+    if (ts && this.isCurrentClientHubInstance(ts.hubInstanceId)) {
+      return { source_skill_id: skillId, visibility: ts.visibility, group_id: ts.groupId };
+    }
+    return undefined;
   }
 
   private handleDeleteSession(res: http.ServerResponse, url: URL): void {
@@ -1844,6 +1955,11 @@ export class ViewerServer {
           body: JSON.stringify({ teamToken, username, deviceName: hostname, reapply: true, identityKey: existingIdentityKey }),
         }) as any;
         const returnedIdentityKey = String(result.identityKey || existingIdentityKey || "");
+        let hubInstanceId = persisted?.hubInstanceId || "";
+        try {
+          const info = await hubRequestJson(hubUrl, "", "/api/v1/hub/info", { method: "GET" }) as any;
+          hubInstanceId = String(info?.hubInstanceId ?? hubInstanceId);
+        } catch { /* best-effort */ }
         this.store.setClientHubConnection({
           hubUrl,
           userId: String(result.userId || ""),
@@ -1853,6 +1969,7 @@ export class ViewerServer {
           connectedAt: Date.now(),
           identityKey: returnedIdentityKey,
           lastKnownStatus: result.status || "",
+          hubInstanceId,
         });
         this.jsonResponse(res, { ok: true, status: result.status || "pending" });
       } catch (err) {
@@ -2060,9 +2177,10 @@ export class ViewerServer {
           }),
         });
         const hubUserId = hubClient.userId;
-        if (hubUserId) {
+        const hubTaskId = String((response as any)?.taskId ?? task.id);
+        if (this.sharingRole === "hub" && hubUserId) {
           this.store.upsertHubTask({
-            id: task.id,
+            id: hubTaskId,
             sourceTaskId: task.id,
             sourceUserId: hubUserId,
             title: task.title,
@@ -2072,6 +2190,9 @@ export class ViewerServer {
             createdAt: task.startedAt ?? Date.now(),
             updatedAt: task.updatedAt ?? Date.now(),
           });
+        } else {
+          const conn = this.store.getClientHubConnection();
+          this.store.markTaskShared(task.id, hubTaskId, chunks.length, visibility, groupId, conn?.hubInstanceId ?? "");
         }
         this.jsonResponse(res, { ok: true, taskId, visibility, response });
       } catch (err) {
@@ -2094,7 +2215,9 @@ export class ViewerServer {
           body: JSON.stringify({ sourceTaskId: task.id }),
         });
         const hubUserId = hubClient.userId;
-        if (hubUserId) this.store.deleteHubTaskBySource(hubUserId, task.id);
+        if (this.sharingRole === "hub" && hubUserId) this.store.deleteHubTaskBySource(hubUserId, task.id);
+        else if (task.owner === "public") this.store.downgradeTeamSharedTaskToLocal(task.id);
+        else this.store.unmarkTaskShared(task.id);
         this.jsonResponse(res, { ok: true, taskId });
       } catch (err) {
         this.jsonResponse(res, { ok: false, error: String(err) });
@@ -2146,7 +2269,8 @@ export class ViewerServer {
             updatedAt: now,
           });
         } else if (hubClient.userId) {
-          this.store.upsertTeamSharedChunk(chunk.id, { hubMemoryId: mid, visibility, groupId });
+          const conn = this.store.getClientHubConnection();
+          this.store.upsertTeamSharedChunk(chunk.id, { hubMemoryId: mid, visibility, groupId, hubInstanceId: conn?.hubInstanceId ?? "" });
         }
         this.jsonResponse(res, { ok: true, chunkId, visibility, response });
       } catch (err) {
@@ -2167,8 +2291,8 @@ export class ViewerServer {
           body: JSON.stringify({ sourceChunkId: chunkId }),
         });
         const hubUserId = hubClient.userId;
-        if (hubUserId) this.store.deleteHubMemoryBySource(hubUserId, chunkId);
-        this.store.deleteTeamSharedChunk(chunkId);
+        if (this.sharingRole === "hub" && hubUserId) this.store.deleteHubMemoryBySource(hubUserId, chunkId);
+        else this.store.deleteTeamSharedChunk(chunkId);
         this.jsonResponse(res, { ok: true, chunkId });
       } catch (err) {
         this.jsonResponse(res, { ok: false, error: String(err) });
@@ -2213,7 +2337,7 @@ export class ViewerServer {
           }),
         });
         const hubUserId = hubClient.userId;
-        if (hubUserId) {
+        if (this.sharingRole === "hub" && hubUserId) {
           const existing = this.store.getHubSkillBySource(hubUserId, skillId);
           this.store.upsertHubSkill({
             id: (response as any)?.skillId ?? existing?.id ?? crypto.randomUUID(),
@@ -2229,6 +2353,14 @@ export class ViewerServer {
             createdAt: existing?.createdAt ?? Date.now(),
             updatedAt: Date.now(),
           });
+        } else {
+          const conn = this.store.getClientHubConnection();
+          this.store.upsertTeamSharedSkill(skillId, {
+            hubSkillId: String((response as any)?.skillId ?? ""),
+            visibility,
+            groupId,
+            hubInstanceId: conn?.hubInstanceId ?? "",
+          });
         }
         this.jsonResponse(res, { ok: true, skillId, visibility, response });
       } catch (err) {
@@ -2251,7 +2383,8 @@ export class ViewerServer {
           body: JSON.stringify({ sourceSkillId: skill.id }),
         });
         const hubUserId = hubClient.userId;
-        if (hubUserId) this.store.deleteHubSkillBySource(hubUserId, skill.id);
+        if (this.sharingRole === "hub" && hubUserId) this.store.deleteHubSkillBySource(hubUserId, skill.id);
+        else this.store.deleteTeamSharedSkill(skill.id);
         this.jsonResponse(res, { ok: true, skillId });
       } catch (err) {
         this.jsonResponse(res, { ok: false, error: String(err) });
@@ -2717,19 +2850,21 @@ export class ViewerServer {
           const isClient = newEnabled && newRole === "client";
           if (wasClient && !isClient) {
             await this.withdrawOrLeaveHub();
+            this.store.clearAllTeamSharingState();
             this.store.clearClientHubConnection();
-            this.log.info("Client hub connection cleared (sharing disabled or role changed)");
+            this.log.info("Client hub connection and team sharing state cleared (sharing disabled or role changed)");
           }
 
           if (wasClient && isClient) {
             const newClientAddr = String((merged.client as Record<string, unknown>)?.hubAddress || "");
             if (newClientAddr && oldClientHubAddress && normalizeHubUrl(newClientAddr) !== normalizeHubUrl(oldClientHubAddress)) {
               this.notifyHubLeave();
+              this.store.clearAllTeamSharingState();
               const oldConn = this.store.getClientHubConnection();
               if (oldConn) {
-                this.store.setClientHubConnection({ ...oldConn, hubUrl: normalizeHubUrl(newClientAddr), userToken: "", lastKnownStatus: "hub_changed" });
+                this.store.setClientHubConnection({ ...oldConn, hubUrl: normalizeHubUrl(newClientAddr), userToken: "", hubInstanceId: "", lastKnownStatus: "hub_changed" });
               }
-              this.log.info("Client hub connection token cleared (switched to different Hub), identity preserved");
+              this.log.info("Client hub connection and team sharing state cleared (switched to different Hub)");
             }
           }
 
@@ -2790,6 +2925,11 @@ export class ViewerServer {
       body: JSON.stringify({ teamToken, username, deviceName: hostname, identityKey: existingIdentityKey }),
     }) as any;
     const returnedIdentityKey = String(result.identityKey || existingIdentityKey || "");
+    let hubInstanceId = persisted?.hubInstanceId || "";
+    try {
+      const info = await hubRequestJson(hubUrl, "", "/api/v1/hub/info", { method: "GET" }) as any;
+      hubInstanceId = String(info?.hubInstanceId ?? hubInstanceId);
+    } catch { /* best-effort */ }
     this.store.setClientHubConnection({
       hubUrl,
       userId: String(result.userId || ""),
@@ -2799,6 +2939,7 @@ export class ViewerServer {
       connectedAt: Date.now(),
       identityKey: returnedIdentityKey,
       lastKnownStatus: result.status || "",
+      hubInstanceId,
     });
     this.log.info(`Auto-join on save: status=${result.status}, userId=${result.userId}`);
     if (result.userToken) {
@@ -2811,6 +2952,7 @@ export class ViewerServer {
     this.readBody(_req, async () => {
       try {
         await this.withdrawOrLeaveHub();
+        this.store.clearAllTeamSharingState();
         this.store.clearClientHubConnection();
 
         const configPath = this.getOpenClawConfigPath();
@@ -3196,7 +3338,8 @@ export class ViewerServer {
 
             // Install dependencies
             this.log.info(`update-install: installing dependencies...`);
-            exec(`cd ${extDir} && npm install --omit=dev --ignore-scripts`, { timeout: 120_000 }, (npmErr, npmOut, npmStderr) => {
+            const npmCmd = process.platform === "win32" ? "npm.cmd" : "npm";
+            execFile(npmCmd, ["install", "--omit=dev", "--ignore-scripts"], { cwd: extDir, timeout: 120_000 }, (npmErr, npmOut, npmStderr) => {
               if (npmErr) {
                 try { fs.rmSync(tmpDir, { recursive: true, force: true }); } catch {}
                 this.log.warn(`update-install: npm install failed: ${npmErr.message}`);
@@ -3204,25 +3347,21 @@ export class ViewerServer {
                 return;
               }
 
-              // Rebuild native modules (do not swallow errors)
-              exec(`cd ${extDir} && npm rebuild better-sqlite3`, { timeout: 60_000 }, (rebuildErr, rebuildOut, rebuildStderr) => {
+              execFile(npmCmd, ["rebuild", "better-sqlite3"], { cwd: extDir, timeout: 60_000 }, (rebuildErr, rebuildOut, rebuildStderr) => {
                 if (rebuildErr) {
                   this.log.warn(`update-install: better-sqlite3 rebuild failed: ${rebuildErr.message}`);
                   const stderr = String(rebuildStderr || "").trim();
                   if (stderr) this.log.warn(`update-install: rebuild stderr: ${stderr.slice(0, 500)}`);
-                  // Continue so postinstall.cjs can run (it will try rebuild again and show user guidance)
                 }
 
-                // Run postinstall.cjs: legacy cleanup, skill install, version marker, and optional sqlite re-check
                 this.log.info(`update-install: running postinstall...`);
-                exec(`cd ${extDir} && node scripts/postinstall.cjs`, { timeout: 180_000 }, (postErr, postOut, postStderr) => {
+                execFile(process.execPath, ["scripts/postinstall.cjs"], { cwd: extDir, timeout: 180_000 }, (postErr, postOut, postStderr) => {
                   try { fs.rmSync(tmpDir, { recursive: true, force: true }); } catch {}
 
                   if (postErr) {
                     this.log.warn(`update-install: postinstall failed: ${postErr.message}`);
                     const postStderrStr = String(postStderr || "").trim();
                     if (postStderrStr) this.log.warn(`update-install: postinstall stderr: ${postStderrStr.slice(0, 500)}`);
-                    // Still report success; plugin is updated, user can run postinstall manually if needed
                   }
 
                   // Read new version
@@ -3575,7 +3714,7 @@ export class ViewerServer {
     } else if (this.migrationState.done) {
       const evtName = this.migrationState.stopped ? "stopped" : "done";
       res.write(`event: state\ndata: ${JSON.stringify(this.migrationState)}\n\n`);
-      res.write(`event: ${evtName}\ndata: ${JSON.stringify({ ok: true })}\n\n`);
+      res.write(`event: ${evtName}\ndata: ${JSON.stringify({ ok: this.migrationState.success, ...this.migrationState })}\n\n`);
       res.end();
     } else {
       res.end();
@@ -3616,19 +3755,12 @@ export class ViewerServer {
         this.migrationSSEClients = this.migrationSSEClients.filter(c => c !== res);
       });
 
-      this.migrationAbort = false;
-      this.migrationState = { phase: "", stored: 0, skipped: 0, merged: 0, errors: 0, processed: 0, total: 0, lastItem: null, done: false, stopped: false };
+      this.migrationState = createInitialMigrationState();
 
       const send = (event: string, data: unknown) => {
         if (event === "item") {
           const d = data as any;
-          if (d.status === "stored") this.migrationState.stored++;
-          else if (d.status === "skipped" || d.status === "duplicate") this.migrationState.skipped++;
-          else if (d.status === "merged") this.migrationState.merged++;
-          else if (d.status === "error") this.migrationState.errors++;
-          this.migrationState.processed = d.index ?? this.migrationState.processed + 1;
-          this.migrationState.total = d.total ?? this.migrationState.total;
-          this.migrationState.lastItem = d;
+          applyMigrationItemToState(this.migrationState, d);
         } else if (event === "phase") {
           this.migrationState.phase = (data as any).phase;
         } else if (event === "progress") {
@@ -3641,11 +3773,13 @@ export class ViewerServer {
       this.runMigration(send, opts.sources, concurrency).finally(() => {
         this.migrationRunning = false;
         this.migrationState.done = true;
+        this.migrationState.success = computeMigrationSuccess(this.migrationState);
+        const donePayload = { ok: this.migrationState.success, ...this.migrationState };
         if (this.migrationAbort) {
           this.migrationState.stopped = true;
-          this.broadcastSSE("stopped", { ok: true, ...this.migrationState });
+          this.broadcastSSE("stopped", donePayload);
         } else {
-          this.broadcastSSE("done", { ok: true });
+          this.broadcastSSE("done", donePayload);
         }
         this.migrationAbort = false;
         const clientsToClose = [...this.migrationSSEClients];
@@ -3742,11 +3876,24 @@ export class ViewerServer {
               }
 
               try {
-                const summary = await summarizer.summarize(row.text);
+                const stepFailures: Array<"summarization" | "dedup" | "embedding"> = [];
+                let summary = "";
+                try {
+                  summary = await summarizer.summarize(row.text);
+                } catch (err) {
+                  stepFailures.push("summarization");
+                  this.log.warn(`Migration summarization failed: ${err}`);
+                }
+                if (!summary) {
+                  stepFailures.push("summarization");
+                  summary = row.text.slice(0, 200);
+                }
+
                 let embedding: number[] | null = null;
                 try {
                   [embedding] = await this.embedder.embed([summary]);
                 } catch (err) {
+                  stepFailures.push("embedding");
                   this.log.warn(`Migration embed failed: ${err}`);
                 }
 
@@ -3765,26 +3912,31 @@ export class ViewerServer {
                     }).filter(c => c.summary);
 
                     if (candidates.length > 0) {
-                      const dedupResult = await summarizer.judgeDedup(summary, candidates);
-                      if (dedupResult?.action === "DUPLICATE" && dedupResult.targetIndex) {
-                        const targetId = candidates[dedupResult.targetIndex - 1]?.chunkId;
-                        if (targetId) {
-                          dedupStatus = "duplicate";
-                          dedupTarget = targetId;
-                          dedupReason = dedupResult.reason;
-                        }
-                      } else if (dedupResult?.action === "UPDATE" && dedupResult.targetIndex && dedupResult.mergedSummary) {
-                        const targetId = candidates[dedupResult.targetIndex - 1]?.chunkId;
-                        if (targetId) {
-                          this.store.updateChunkSummaryAndContent(targetId, dedupResult.mergedSummary, row.text);
-                          try {
-                            const [newEmb] = await this.embedder.embed([dedupResult.mergedSummary]);
-                            if (newEmb) this.store.upsertEmbedding(targetId, newEmb);
-                          } catch { /* best-effort */ }
-                          dedupStatus = "merged";
-                          dedupTarget = targetId;
-                          dedupReason = dedupResult.reason;
+                      try {
+                        const dedupResult = await summarizer.judgeDedup(summary, candidates);
+                        if (dedupResult?.action === "DUPLICATE" && dedupResult.targetIndex) {
+                          const targetId = candidates[dedupResult.targetIndex - 1]?.chunkId;
+                          if (targetId) {
+                            dedupStatus = "duplicate";
+                            dedupTarget = targetId;
+                            dedupReason = dedupResult.reason;
+                          }
+                        } else if (dedupResult?.action === "UPDATE" && dedupResult.targetIndex && dedupResult.mergedSummary) {
+                          const targetId = candidates[dedupResult.targetIndex - 1]?.chunkId;
+                          if (targetId) {
+                            this.store.updateChunkSummaryAndContent(targetId, dedupResult.mergedSummary, row.text);
+                            try {
+                              const [newEmb] = await this.embedder.embed([dedupResult.mergedSummary]);
+                              if (newEmb) this.store.upsertEmbedding(targetId, newEmb);
+                            } catch { /* best-effort */ }
+                            dedupStatus = "merged";
+                            dedupTarget = targetId;
+                            dedupReason = dedupResult.reason;
+                          }
                         }
+                      } catch (err) {
+                        stepFailures.push("dedup");
+                        this.log.warn(`Migration dedup judgment failed: ${err}`);
                       }
                     }
                   }
@@ -3827,7 +3979,13 @@ export class ViewerServer {
                   preview: row.text.slice(0, 120),
                   summary: summary.slice(0, 80),
                   source: file,
+                  stepFailures,
                 });
+                if (stepFailures.length > 0) {
+                  this.log.warn(`[MIGRATION] sqlite item imported with step failures: ${stepFailures.join(",")}`);
+                } else {
+                  this.log.info("[MIGRATION] sqlite item imported successfully (all steps)");
+                }
               } catch (err) {
                 totalErrors++;
                 send("item", {
@@ -3957,11 +4115,24 @@ export class ViewerServer {
               }
 
               try {
-                const summary = await summarizer.summarize(content);
+                const stepFailures: Array<"summarization" | "dedup" | "embedding"> = [];
+                let summary = "";
+                try {
+                  summary = await summarizer.summarize(content);
+                } catch (err) {
+                  stepFailures.push("summarization");
+                  this.log.warn(`Migration summarization failed: ${err}`);
+                }
+                if (!summary) {
+                  stepFailures.push("summarization");
+                  summary = content.slice(0, 200);
+                }
+
                 let embedding: number[] | null = null;
                 try {
                   [embedding] = await this.embedder.embed([summary]);
                 } catch (err) {
+                  stepFailures.push("embedding");
                   this.log.warn(`Migration embed failed: ${err}`);
                 }
 
@@ -3980,17 +4151,22 @@ export class ViewerServer {
                     }).filter(c => c.summary);
 
                     if (candidates.length > 0) {
-                      const dedupResult = await summarizer.judgeDedup(summary, candidates);
-                      if (dedupResult?.action === "DUPLICATE" && dedupResult.targetIndex) {
-                        const targetId = candidates[dedupResult.targetIndex - 1]?.chunkId;
-                        if (targetId) { dedupStatus = "duplicate"; dedupTarget = targetId; dedupReason = dedupResult.reason; }
-                      } else if (dedupResult?.action === "UPDATE" && dedupResult.targetIndex && dedupResult.mergedSummary) {
-                        const targetId = candidates[dedupResult.targetIndex - 1]?.chunkId;
-                        if (targetId) {
-                          this.store.updateChunkSummaryAndContent(targetId, dedupResult.mergedSummary, content);
-                          try { const [newEmb] = await this.embedder.embed([dedupResult.mergedSummary]); if (newEmb) this.store.upsertEmbedding(targetId, newEmb); } catch { /* best-effort */ }
-                          dedupStatus = "merged"; dedupTarget = targetId; dedupReason = dedupResult.reason;
+                      try {
+                        const dedupResult = await summarizer.judgeDedup(summary, candidates);
+                        if (dedupResult?.action === "DUPLICATE" && dedupResult.targetIndex) {
+                          const targetId = candidates[dedupResult.targetIndex - 1]?.chunkId;
+                          if (targetId) { dedupStatus = "duplicate"; dedupTarget = targetId; dedupReason = dedupResult.reason; }
+                        } else if (dedupResult?.action === "UPDATE" && dedupResult.targetIndex && dedupResult.mergedSummary) {
+                          const targetId = candidates[dedupResult.targetIndex - 1]?.chunkId;
+                          if (targetId) {
+                            this.store.updateChunkSummaryAndContent(targetId, dedupResult.mergedSummary, content);
+                            try { const [newEmb] = await this.embedder.embed([dedupResult.mergedSummary]); if (newEmb) this.store.upsertEmbedding(targetId, newEmb); } catch { /* best-effort */ }
+                            dedupStatus = "merged"; dedupTarget = targetId; dedupReason = dedupResult.reason;
+                          }
                         }
+                      } catch (err) {
+                        stepFailures.push("dedup");
+                        this.log.warn(`Migration dedup judgment failed: ${err}`);
                       }
                     }
                   }
@@ -4010,7 +4186,12 @@ export class ViewerServer {
                 if (embedding && dedupStatus === "active") this.store.upsertEmbedding(chunkId, embedding);
 
                 totalStored++;
-                send("item", { index: idx, total: totalMsgs, status: dedupStatus === "active" ? "stored" : dedupStatus, preview: content.slice(0, 120), summary: summary.slice(0, 80), source: file, agent: agentId, role: msgRole });
+                send("item", { index: idx, total: totalMsgs, status: dedupStatus === "active" ? "stored" : dedupStatus, preview: content.slice(0, 120), summary: summary.slice(0, 80), source: file, agent: agentId, role: msgRole, stepFailures });
+                if (stepFailures.length > 0) {
+                  this.log.warn(`[MIGRATION] session item imported with step failures: ${stepFailures.join(",")}`);
+                } else {
+                  this.log.info("[MIGRATION] session item imported successfully (all steps)");
+                }
               } catch (err) {
                 totalErrors++;
                 send("item", { index: idx, total: totalMsgs, status: "error", preview: content.slice(0, 120), source: file, agent: agentId, error: String(err).slice(0, 200) });
@@ -4051,7 +4232,14 @@ export class ViewerServer {
     }
 
     send("progress", { total: totalProcessed, processed: totalProcessed, phase: "done" });
-    send("summary", { totalProcessed, totalStored, totalSkipped, totalErrors });
+    send("summary", {
+      totalProcessed,
+      totalStored,
+      totalSkipped,
+      totalErrors,
+      success: computeMigrationSuccess(this.migrationState),
+      stepFailures: this.migrationState.stepFailures,
+    });
   }
 
   // ─── Post-processing: independent task/skill generation ───