@inline-chat/realtime-sdk 0.0.6 → 0.0.8

package/dist/realtime/protocol-client.d.ts

@@ -65,7 +65,7 @@ export declare class ProtocolClient {
     private trySendPendingRpcRequest;
 }
 export declare class ProtocolClientError extends Error {
-    constructor(code: "not-authorized" | "not-connected" | "rpc-error" | "stopped" | "timeout", details?: {
+    constructor(code: "not-authorized" | "not-connected" | "rpc-error" | "stopped" | "timeout" | "invalid-rpc-method", details?: {
         code?: number;
         message?: string;
     });

package/dist/realtime/protocol-client.js

@@ -3,6 +3,11 @@ import { AsyncChannel } from "../utils/async-channel.js";
 import { PingPongService } from "./ping-pong.js";
 const emptyRpcInput = { oneofKind: undefined };
 const defaultRpcTimeoutMs = 30_000;
+const assertValidRpcMethod = (method) => {
+    if (typeof method !== "number" || !Number.isInteger(method) || method <= 0) {
+        throw new ProtocolClientError("invalid-rpc-method", { message: `Invalid rpc method: ${String(method)}` });
+    }
+};
 export class ProtocolClient {
     events = new AsyncChannel();
     transport;
@@ -52,6 +57,7 @@ export class ProtocolClient {
     }
     async sendRpc(method, input = emptyRpcInput) {
         this.ensureOpenForRpc();
+        assertValidRpcMethod(method);
         const message = this.wrapMessage({
             oneofKind: "rpcCall",
             rpcCall: { method, input },
@@ -60,6 +66,7 @@ export class ProtocolClient {
         return message.id;
     }
     async callRpc(method, input = emptyRpcInput, options) {
+        assertValidRpcMethod(method);
         const message = this.wrapMessage({
             oneofKind: "rpcCall",
             rpcCall: { method, input },

package/dist/sdk/inline-sdk-client.js

@@ -175,18 +175,28 @@ export class InlineSdkClient {
         }
         const peerId = this.inputPeerFromTarget(params, "sendMessage");
         const media = params.media != null ? toInputMedia(params.media) : undefined;
-        const result = await this.invoke(Method.SEND_MESSAGE, {
-            oneofKind: "sendMessage",
-            sendMessage: {
-                peerId,
-                ...(hasText ? { message: params.text } : {}),
-                ...(media != null ? { media } : {}),
-                ...(params.replyToMsgId != null ? { replyToMsgId: asInlineId(params.replyToMsgId, "replyToMsgId") } : {}),
-                ...(params.parseMarkdown != null ? { parseMarkdown: params.parseMarkdown } : {}),
-                ...(params.entities != null ? { entities: params.entities } : {}),
-                ...(params.sendMode === "silent" ? { sendMode: MessageSendMode.MODE_SILENT } : {}),
-            },
-        });
+        let result;
+        try {
+            result = await this.invoke(Method.SEND_MESSAGE, {
+                oneofKind: "sendMessage",
+                sendMessage: {
+                    peerId,
+                    ...(hasText ? { message: params.text } : {}),
+                    ...(media != null ? { media } : {}),
+                    ...(params.replyToMsgId != null ? { replyToMsgId: asInlineId(params.replyToMsgId, "replyToMsgId") } : {}),
+                    ...(params.parseMarkdown != null ? { parseMarkdown: params.parseMarkdown } : {}),
+                    ...(params.entities != null ? { entities: params.entities } : {}),
+                    ...(params.sendMode === "silent" ? { sendMode: MessageSendMode.MODE_SILENT } : {}),
+                },
+            });
+        }
+        catch (error) {
+            const target = "chatId" in params ? `chat:${String(params.chatId)}` : `user:${String(params.userId)}`;
+            const mediaKind = params.media?.kind ?? "none";
+            const textLen = hasText ? params.text.length : 0;
+            const detail = extractErrorMessage(error);
+            throw new Error(`sendMessage: request failed (${detail}; target=${target}; media=${mediaKind}; textLen=${textLen}; replyTo=${params.replyToMsgId != null ? String(params.replyToMsgId) : "none"})`, { cause: error });
+        }
         const messageId = extractFirstMessageId(result.sendMessage.updates);
         return { messageId };
     }
@@ -196,9 +206,14 @@ export class InlineSdkClient {
         const fileName = normalizeUploadFileName(params.fileName, params.type);
         const fileContentType = resolveUploadContentType(params.type, params.contentType);
         form.set("file", toUploadMultipartFile(params.file, fileName, fileContentType), fileName);
+        const fileSize = getBinaryInputSize(params.file);
+        let thumbnailName;
+        let thumbnailContentType;
+        let thumbnailSize;
         if (params.thumbnail != null) {
-            const thumbnailName = normalizeUploadFileName(params.thumbnailFileName, "photo");
-            const thumbnailContentType = resolveUploadContentType("photo", params.thumbnailContentType);
+            thumbnailName = normalizeUploadFileName(params.thumbnailFileName, "photo");
+            thumbnailContentType = resolveUploadContentType("photo", params.thumbnailContentType);
+            thumbnailSize = getBinaryInputSize(params.thumbnail);
             form.set("thumbnail", toUploadMultipartFile(params.thumbnail, thumbnailName, thumbnailContentType), thumbnailName);
         }
         if (params.type === "video") {
@@ -209,29 +224,56 @@ export class InlineSdkClient {
             form.set("height", String(height));
             form.set("duration", String(duration));
         }
-        const response = await this.fetchImpl(resolveUploadFileUrl(this.httpBaseUrl), {
-            method: "POST",
-            headers: {
-                authorization: `Bearer ${this.options.token}`,
-            },
-            body: form,
+        const uploadUrl = resolveUploadFileUrl(this.httpBaseUrl);
+        const requestContext = describeUploadContext({
+            type: params.type,
+            fileName,
+            fileContentType,
+            fileSize,
+            ...(thumbnailName ? { thumbnailName } : {}),
+            ...(thumbnailContentType ? { thumbnailContentType } : {}),
+            ...(thumbnailSize != null ? { thumbnailSize } : {}),
+            ...(params.type === "video"
+                ? {
+                    width: params.width ?? defaultVideoWidth,
+                    height: params.height ?? defaultVideoHeight,
+                    duration: params.duration ?? defaultVideoDuration,
+                }
+                : {}),
+            uploadUrl: uploadUrl.toString(),
         });
+        let response;
+        try {
+            response = await this.fetchImpl(uploadUrl, {
+                method: "POST",
+                headers: {
+                    authorization: `Bearer ${this.options.token}`,
+                },
+                body: form,
+            });
+        }
+        catch (error) {
+            const detail = extractErrorMessage(error);
+            throw new Error(`uploadFile: network request failed (${detail}; ${requestContext})`, {
+                cause: error,
+            });
+        }
         const payload = await parseJsonResponse(response);
         if (!response.ok) {
             const detail = describeUploadFailure(payload);
-            throw new Error(`uploadFile: request failed with status ${response.status}${detail ? ` (${detail})` : ""}`);
+            throw new Error(`uploadFile: request failed with status ${response.status}${detail ? ` (${detail})` : ""}; ${requestContext}`);
         }
         if (!isRecord(payload) || payload.ok !== true) {
             const detail = describeUploadFailure(payload);
-            throw new Error(`uploadFile: API error${detail ? ` (${detail})` : ""}`);
+            throw new Error(`uploadFile: API error${detail ? ` (${detail})` : ""}; ${requestContext}`);
         }
         const result = payload.result;
         if (!isRecord(result)) {
-            throw new Error("uploadFile: malformed success payload");
+            throw new Error(`uploadFile: malformed success payload; ${requestContext}`);
         }
         const fileUniqueId = typeof result.fileUniqueId === "string" ? result.fileUniqueId.trim() : "";
         if (!fileUniqueId) {
-            throw new Error("uploadFile: response missing fileUniqueId");
+            throw new Error(`uploadFile: response missing fileUniqueId; ${requestContext}`);
         }
         const photoId = parseOptionalBigInt(result.photoId, "photoId");
         const videoId = parseOptionalBigInt(result.videoId, "videoId");
@@ -699,6 +741,40 @@ function sanitizeUploadFileName(raw) {
     const noQuery = leaf.split(/[?#]/, 1)[0] ?? leaf;
     return noQuery.trim();
 }
+function getBinaryInputSize(input) {
+    if (input instanceof Blob)
+        return input.size;
+    if (input instanceof Uint8Array)
+        return input.byteLength;
+    return input.byteLength;
+}
+function describeUploadContext(params) {
+    const parts = [
+        `type=${params.type}`,
+        `fileName=${params.fileName}`,
+        `fileContentType=${params.fileContentType}`,
+        `fileSize=${params.fileSize}`,
+        `uploadUrl=${params.uploadUrl}`,
+    ];
+    if (params.thumbnailName)
+        parts.push(`thumbnailName=${params.thumbnailName}`);
+    if (params.thumbnailContentType)
+        parts.push(`thumbnailContentType=${params.thumbnailContentType}`);
+    if (params.thumbnailSize != null)
+        parts.push(`thumbnailSize=${params.thumbnailSize}`);
+    if (params.width != null)
+        parts.push(`width=${params.width}`);
+    if (params.height != null)
+        parts.push(`height=${params.height}`);
+    if (params.duration != null)
+        parts.push(`duration=${params.duration}`);
+    return parts.join(", ");
+}
+function extractErrorMessage(error) {
+    if (error instanceof Error)
+        return error.message;
+    return String(error);
+}
 function normalizePositiveInt(value, field) {
     if (value == null)
         return undefined;

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@inline-chat/realtime-sdk",
-  "version": "0.0.6",
+  "version": "0.0.8",
   "private": false,
   "type": "module",
   "files": [
@@ -20,7 +20,7 @@
     "test": "vitest run --coverage"
   },
   "dependencies": {
-    "@inline-chat/protocol": "^0.0.2",
+    "@inline-chat/protocol": "^0.0.3",
     "ws": "^8.18.3"
   },
   "devDependencies": {