@tinycloud/sdk-core 2.11.0-beta.3 → 2.11.0-beta.5

package/dist/index.cjs

@@ -12266,6 +12266,60 @@ var AccountService = class {
         await this.upsertSpaceIndexQuietly(registered);
         return (0, import_sdk_services6.ok)(registered);
       },
+      /**
+       * Register multiple spaces in one KV batch write + one multi-row index
+       * write, instead of one `register()` round trip per space (TC-373).
+       *
+       * The stored records are computed exactly once and reused for both the
+       * batch attempt and any reconciliation below, so a fallback never mints a
+       * fresh timestamp for a space that already has a written record — a
+       * second `spaceRecordFromInput` call would produce a new content hash and
+       * leave the first attempt's blocks orphaned.
+       */
+      registerBatch: async (spaces) => {
+        if (spaces.length === 0) return (0, import_sdk_services6.ok)({ spaces: [] });
+        await this.config.ensureAccountSpaceHosted?.();
+        const kvResult = this.accountKV();
+        if (!kvResult.ok) return kvResult;
+        const stored = spaces.map((space) => spaceRecordFromInput(space));
+        const registered = stored.map(
+          (record) => spaceFromRecord(spaceKey(record.space_id), record)
+        );
+        const items = stored.map((record) => ({
+          key: spaceKey(record.space_id),
+          value: record
+        }));
+        const batchResult = await kvResult.data.batchPut(items);
+        if (batchResult.ok) {
+          await this.upsertSpacesIndexQuietly(registered);
+          return (0, import_sdk_services6.ok)({ spaces: registered });
+        }
+        if (!isAmbiguousBatchFailure(batchResult.error)) {
+          return accountErr(batchResult.error);
+        }
+        const reconcileFailures = [];
+        for (const record of stored) {
+          const written = await kvResult.data.put(spaceKey(record.space_id), record);
+          if (!written.ok) {
+            reconcileFailures.push(`${record.space_id}: ${written.error.message}`);
+          }
+        }
+        if (reconcileFailures.length > 0) {
+          return (0, import_sdk_services6.err)(
+            (0, import_sdk_services6.serviceError)(
+              "KV_BATCH_RECONCILE_FAILED",
+              `KV batch write for ${items.length} space(s) failed ambiguously (${batchResult.error.message}) and per-space reconciliation also failed for: ${reconcileFailures.join("; ")}`,
+              SERVICE_NAME2,
+              { cause: batchResult.error.cause, meta: { originalError: batchResult.error } }
+            )
+          );
+        }
+        console.warn(
+          `[AccountService] KV batch write for ${items.length} space(s) returned an ambiguous error and was reconciled via per-space fallback. Original error: ${batchResult.error.message}`
+        );
+        await this.upsertSpacesIndexQuietly(registered);
+        return (0, import_sdk_services6.ok)({ spaces: registered, recoveredFromBatchError: batchResult.error });
+      },
       syncAccessible: async () => {
         const listed = await this.config.getSpaces().list();
         if (!listed.ok) return accountErr(listed.error);
@@ -12652,6 +12706,47 @@ var AccountService = class {
     if (!written.ok) return accountErr(written.error);
     return (0, import_sdk_services6.ok)(void 0);
   }
+  async upsertSpacesIndexQuietly(spaces) {
+    await ignoreIndexFailure(() => this.upsertSpacesIndex(spaces));
+  }
+  /**
+   * Upsert multiple space index rows in ONE multi-row `INSERT OR REPLACE`
+   * statement (TC-373), rather than one statement per space. This matters
+   * because `IDatabaseHandle.batch()` executes its statement list as a plain
+   * loop, not a single database transaction — an N-statement batch is not
+   * atomic. A single multi-row INSERT is one SQLite statement, so it applies
+   * atomically: all rows or none.
+   */
+  async upsertSpacesIndex(spaces) {
+    if (spaces.length === 0) return (0, import_sdk_services6.ok)(void 0);
+    const dbResult = this.accountDb();
+    if (!dbResult.ok) return (0, import_sdk_services6.ok)(void 0);
+    const schema = await this.ensureAccountIndex(dbResult.data);
+    if (!schema.ok) return schema;
+    const placeholders = spaces.map(() => "(?, ?, ?, ?, ?, ?, ?, ?, ?)").join(", ");
+    const params = spaces.flatMap((space) => {
+      const updatedAt = space.updatedAt ?? (/* @__PURE__ */ new Date()).toISOString();
+      return [
+        space.spaceId,
+        space.name,
+        space.ownerDid,
+        space.type,
+        JSON.stringify(space.permissions),
+        space.status,
+        space.registeredAt ?? updatedAt,
+        updatedAt,
+        space.expiresAt?.toISOString() ?? null
+      ];
+    });
+    const written = await dbResult.data.batch([
+      {
+        sql: `INSERT OR REPLACE INTO spaces (space_id, name, owner_did, type, permissions_json, status, registered_at, updated_at, expires_at) VALUES ${placeholders}`,
+        params
+      }
+    ]);
+    if (!written.ok) return accountErr(written.error);
+    return (0, import_sdk_services6.ok)(void 0);
+  }
   async deleteSpaceIndexQuietly(spaceId) {
     await ignoreIndexFailure(() => this.deleteSpaceIndex(spaceId));
   }
@@ -12978,6 +13073,17 @@ function accountErr(error) {
 function isMissingIndexError(error) {
   return /no such table:/i.test(error.message);
 }
+var AMBIGUOUS_WRITE_STATUSES = /* @__PURE__ */ new Set([500, 502, 503, 504, 520, 522, 524]);
+function isAmbiguousBatchFailure(error) {
+  if (error.code === import_sdk_services6.ErrorCodes.NETWORK_ERROR || error.code === import_sdk_services6.ErrorCodes.TIMEOUT) {
+    return error.meta?.requestMayHaveDispatched === true;
+  }
+  if (error.code === import_sdk_services6.ErrorCodes.KV_WRITE_FAILED) {
+    const status = error.meta?.status;
+    return typeof status === "number" && AMBIGUOUS_WRITE_STATUSES.has(status);
+  }
+  return false;
+}
 async function ignoreIndexFailure(task) {
   try {
     await task();
@@ -15350,13 +15456,14 @@ async function discoverLocalTinyCloudNode(options2 = {}) {
   }
   const fetchFn = options2.fetch ?? globalThis.fetch;
   const identityStore = options2.identityStore ?? defaultLocalNodeIdentityStore;
-  const staticCandidates = [
-    {
+  const staticCandidates = [];
+  if (options2.localNodeUrl) {
+    staticCandidates.push({
       source: "local-loopback",
-      url: options2.localNodeUrl ?? DEFAULT_LOCAL_NODE_URL,
+      url: options2.localNodeUrl,
       timeoutMs: LOCAL_LOOPBACK_PROBE_TIMEOUT_MS
-    }
-  ];
+    });
+  }
   if (options2.localLinkName) {
     if (isValidDnsLabel(options2.localLinkName)) {
       staticCandidates.push({