@ragable/sdk 0.6.6 → 0.6.8

package/dist/index.d.mts

@@ -898,9 +898,13 @@ declare class RagableBrowserDatabaseClient<Database extends RagableDatabase = De
     constructor(options: RagableBrowserClientOptions, ragableAuth?: RagableAuth | null);
     /** @internal Called by RagableBrowser to share the Transport instance. */
     _setTransport(transport: Transport): void;
-    from<TableName extends RagableTableNames<Database>>(table: TableName, databaseInstanceId?: string): PostgrestTableApi<Database, TableName>;
+    /**
+     * PostgREST table access. Instance field so `client.database.from` is always an own,
+     * enumerable function (avoids rare prototype/bundler issues).
+     */
+    from: <TableName extends RagableTableNames<Database>>(table: TableName, databaseInstanceId?: string) => PostgrestTableApi<Database, TableName>;
     private toUrl;
-    query<Row extends Record<string, unknown> = Record<string, unknown>>(params: BrowserSqlQueryParams): Promise<PostgrestResult<BrowserSqlQueryResult<Row>>>;
+    query: <Row extends Record<string, unknown> = Record<string, unknown>>(params: BrowserSqlQueryParams) => Promise<PostgrestResult<BrowserSqlQueryResult<Row>>>;
     private baseHeaders;
 }
 declare class RagableBrowserAgentsClient {
@@ -922,7 +926,7 @@ declare class RagableBrowser<Database extends RagableDatabase = DefaultRagableDa
     private readonly _ragableAuth;
     constructor(options: RagableBrowserClientOptions);
     /** Delegates to `database.from()`. Kept for back-compat — prefer `database.from()`. */
-    from<TableName extends RagableTableNames<Database>>(table: TableName, databaseInstanceId?: string): PostgrestTableApi<Database, TableName>;
+    from: <TableName extends RagableTableNames<Database>>(table: TableName, databaseInstanceId?: string) => PostgrestTableApi<Database, TableName>;
     destroy(): void;
 }
 declare function createBrowserClient<Database extends RagableDatabase = DefaultRagableDatabase, AuthUser extends Record<string, unknown> = Record<string, unknown>>(options: RagableBrowserClientOptions): RagableBrowser<Database, AuthUser>;

package/dist/index.d.ts

@@ -898,9 +898,13 @@ declare class RagableBrowserDatabaseClient<Database extends RagableDatabase = De
     constructor(options: RagableBrowserClientOptions, ragableAuth?: RagableAuth | null);
     /** @internal Called by RagableBrowser to share the Transport instance. */
     _setTransport(transport: Transport): void;
-    from<TableName extends RagableTableNames<Database>>(table: TableName, databaseInstanceId?: string): PostgrestTableApi<Database, TableName>;
+    /**
+     * PostgREST table access. Instance field so `client.database.from` is always an own,
+     * enumerable function (avoids rare prototype/bundler issues).
+     */
+    from: <TableName extends RagableTableNames<Database>>(table: TableName, databaseInstanceId?: string) => PostgrestTableApi<Database, TableName>;
     private toUrl;
-    query<Row extends Record<string, unknown> = Record<string, unknown>>(params: BrowserSqlQueryParams): Promise<PostgrestResult<BrowserSqlQueryResult<Row>>>;
+    query: <Row extends Record<string, unknown> = Record<string, unknown>>(params: BrowserSqlQueryParams) => Promise<PostgrestResult<BrowserSqlQueryResult<Row>>>;
     private baseHeaders;
 }
 declare class RagableBrowserAgentsClient {
@@ -922,7 +926,7 @@ declare class RagableBrowser<Database extends RagableDatabase = DefaultRagableDa
     private readonly _ragableAuth;
     constructor(options: RagableBrowserClientOptions);
     /** Delegates to `database.from()`. Kept for back-compat — prefer `database.from()`. */
-    from<TableName extends RagableTableNames<Database>>(table: TableName, databaseInstanceId?: string): PostgrestTableApi<Database, TableName>;
+    from: <TableName extends RagableTableNames<Database>>(table: TableName, databaseInstanceId?: string) => PostgrestTableApi<Database, TableName>;
     destroy(): void;
 }
 declare function createBrowserClient<Database extends RagableDatabase = DefaultRagableDatabase, AuthUser extends Record<string, unknown> = Record<string, unknown>>(options: RagableBrowserClientOptions): RagableBrowser<Database, AuthUser>;

package/dist/index.js

@@ -2263,103 +2263,107 @@ var RagableBrowserDatabaseClient = class {
     this.ragableAuth = ragableAuth;
     __publicField(this, "fetchImpl");
     __publicField(this, "_transport", null);
-    this.fetchImpl = bindFetch(options.fetch);
-  }
-  /** @internal Called by RagableBrowser to share the Transport instance. */
-  _setTransport(transport) {
-    this._transport = transport;
-  }
-  from(table, databaseInstanceId) {
-    const id = databaseInstanceId?.trim() || this.options.databaseInstanceId?.trim() || "";
-    const ragableAuth = this.ragableAuth;
-    const opts = this.options;
-    const transport = this._transport;
-    const fetchImpl = this.fetchImpl;
-    const pgFetch = async (params) => {
-      if (!params.databaseInstanceId?.trim()) {
-        throw new RagableError(
-          "database.from() requires databaseInstanceId in client options or as the second argument",
-          400,
-          { code: "SDK_MISSING_DATABASE_INSTANCE_ID" }
-        );
-      }
-      const gid = requireAuthGroupId(opts);
-      const token = await resolveDatabaseAuthBearer(opts, ragableAuth);
-      const baseUrl = normalizeBrowserApiBase(opts.baseUrl);
-      const qs = params.searchParams.toString();
-      const url = `${baseUrl}/auth-groups/${gid}/data/rest/${params.table}${qs ? `?${qs}` : ""}`;
-      const headers = new Headers(opts.headers);
-      headers.set("Authorization", `Bearer ${token}`);
-      headers.set("X-Database-Instance-Id", params.databaseInstanceId);
-      if (params.body !== void 0) {
-        headers.set("Content-Type", "application/json");
-      }
-      if (params.headers) {
-        for (const [k, v] of Object.entries(params.headers)) {
-          headers.set(k, v);
+    /**
+     * PostgREST table access. Instance field so `client.database.from` is always an own,
+     * enumerable function (avoids rare prototype/bundler issues).
+     */
+    __publicField(this, "from", (table, databaseInstanceId) => {
+      const id = databaseInstanceId?.trim() || this.options.databaseInstanceId?.trim() || "";
+      const ragableAuth = this.ragableAuth;
+      const opts = this.options;
+      const transport = this._transport;
+      const fetchImpl = this.fetchImpl;
+      const pgFetch = async (params) => {
+        if (!params.databaseInstanceId?.trim()) {
+          throw new RagableError(
+            "database.from() requires databaseInstanceId in client options or as the second argument",
+            400,
+            { code: "SDK_MISSING_DATABASE_INSTANCE_ID" }
+          );
         }
-      }
-      if (transport) {
-        return transport.execute({
-          url,
+        const gid = requireAuthGroupId(opts);
+        const token = await resolveDatabaseAuthBearer(opts, ragableAuth);
+        const baseUrl = normalizeBrowserApiBase(opts.baseUrl);
+        const qs = params.searchParams.toString();
+        const url = `${baseUrl}/auth-groups/${gid}/data/rest/${params.table}${qs ? `?${qs}` : ""}`;
+        const headers = new Headers(opts.headers);
+        headers.set("Authorization", `Bearer ${token}`);
+        headers.set("X-Database-Instance-Id", params.databaseInstanceId);
+        if (params.body !== void 0) {
+          headers.set("Content-Type", "application/json");
+        }
+        if (params.headers) {
+          for (const [k, v] of Object.entries(params.headers)) {
+            headers.set(k, v);
+          }
+        }
+        if (transport) {
+          return transport.execute({
+            url,
+            method: params.method,
+            headers,
+            body: params.body !== void 0 ? JSON.stringify(params.body) : void 0,
+            signal: params.signal,
+            idempotencyKey: params.idempotencyKey
+          });
+        }
+        return fetchImpl(url, {
           method: params.method,
           headers,
           body: params.body !== void 0 ? JSON.stringify(params.body) : void 0,
-          signal: params.signal,
-          idempotencyKey: params.idempotencyKey
+          signal: params.signal
         });
-      }
-      return fetchImpl(url, {
-        method: params.method,
-        headers,
-        body: params.body !== void 0 ? JSON.stringify(params.body) : void 0,
-        signal: params.signal
+      };
+      return new PostgrestTableApi(pgFetch, id, table);
+    });
+    __publicField(this, "query", async (params) => {
+      return asPostgrestResponse(async () => {
+        const gid = requireAuthGroupId(this.options);
+        const token = await resolveDatabaseAuthBearer(this.options, this.ragableAuth);
+        const databaseInstanceId = params.databaseInstanceId?.trim() || this.options.databaseInstanceId?.trim();
+        if (!databaseInstanceId) {
+          throw new RagableError(
+            "database.query requires databaseInstanceId in the request or on createBrowserClient({ databaseInstanceId })",
+            400,
+            { code: "SDK_MISSING_DATABASE_INSTANCE_ID" }
+          );
+        }
+        const headers = this.baseHeaders();
+        headers.set("Authorization", `Bearer ${token}`);
+        headers.set("Content-Type", "application/json");
+        const readOnly = (this.options.dataAuth ?? "user") === "publicAnon" ? true : params.readOnly !== false;
+        const response = await this.fetchImpl(
+          this.toUrl(`/auth-groups/${gid}/data/query`),
+          {
+            method: "POST",
+            headers,
+            body: JSON.stringify({
+              databaseInstanceId,
+              sql: params.sql,
+              ...params.params !== void 0 ? { params: params.params } : {},
+              readOnly,
+              ...params.timeoutMs !== void 0 ? { timeoutMs: params.timeoutMs } : {},
+              ...params.rowLimit !== void 0 ? { rowLimit: params.rowLimit } : {}
+            })
+          }
+        );
+        const payload = await parseMaybeJsonBody(response);
+        if (!response.ok) {
+          const message = extractErrorMessage(payload, response.statusText);
+          throw new RagableError(message, response.status, payload);
+        }
+        return payload;
       });
-    };
-    return new PostgrestTableApi(pgFetch, id, table);
+    });
+    this.fetchImpl = bindFetch(options.fetch);
+  }
+  /** @internal Called by RagableBrowser to share the Transport instance. */
+  _setTransport(transport) {
+    this._transport = transport;
   }
   toUrl(path) {
     return `${normalizeBrowserApiBase(this.options.baseUrl)}${path.startsWith("/") ? path : `/${path}`}`;
   }
-  async query(params) {
-    return asPostgrestResponse(async () => {
-      const gid = requireAuthGroupId(this.options);
-      const token = await resolveDatabaseAuthBearer(this.options, this.ragableAuth);
-      const databaseInstanceId = params.databaseInstanceId?.trim() || this.options.databaseInstanceId?.trim();
-      if (!databaseInstanceId) {
-        throw new RagableError(
-          "database.query requires databaseInstanceId in the request or on createBrowserClient({ databaseInstanceId })",
-          400,
-          { code: "SDK_MISSING_DATABASE_INSTANCE_ID" }
-        );
-      }
-      const headers = this.baseHeaders();
-      headers.set("Authorization", `Bearer ${token}`);
-      headers.set("Content-Type", "application/json");
-      const readOnly = (this.options.dataAuth ?? "user") === "publicAnon" ? true : params.readOnly !== false;
-      const response = await this.fetchImpl(
-        this.toUrl(`/auth-groups/${gid}/data/query`),
-        {
-          method: "POST",
-          headers,
-          body: JSON.stringify({
-            databaseInstanceId,
-            sql: params.sql,
-            ...params.params !== void 0 ? { params: params.params } : {},
-            readOnly,
-            ...params.timeoutMs !== void 0 ? { timeoutMs: params.timeoutMs } : {},
-            ...params.rowLimit !== void 0 ? { rowLimit: params.rowLimit } : {}
-          })
-        }
-      );
-      const payload = await parseMaybeJsonBody(response);
-      if (!response.ok) {
-        const message = extractErrorMessage(payload, response.statusText);
-        throw new RagableError(message, response.status, payload);
-      }
-      return payload;
-    });
-  }
   baseHeaders() {
     return new Headers(this.options.headers);
   }
@@ -2410,6 +2414,10 @@ var RagableBrowser = class {
     __publicField(this, "database");
     __publicField(this, "transport");
     __publicField(this, "_ragableAuth");
+    /** Delegates to `database.from()`. Kept for back-compat — prefer `database.from()`. */
+    __publicField(this, "from", (table, databaseInstanceId) => {
+      return this.database.from(table, databaseInstanceId);
+    });
     this.transport = new Transport({
       fetch: options.fetch,
       headers: options.headers,
@@ -2444,10 +2452,6 @@ var RagableBrowser = class {
     );
     this.database._setTransport(this.transport);
   }
-  /** Delegates to `database.from()`. Kept for back-compat — prefer `database.from()`. */
-  from(table, databaseInstanceId) {
-    return this.database.from(table, databaseInstanceId);
-  }
   destroy() {
     this._ragableAuth?.destroy();
   }
@@ -2531,6 +2535,11 @@ function createClient(urlOrOptions, browserOptions) {
     });
   }
   if (isServerClientOptions(urlOrOptions)) {
+    if (typeof urlOrOptions === "object" && urlOrOptions !== null && "organizationId" in urlOrOptions && typeof urlOrOptions.organizationId === "string") {
+      console.warn(
+        "[@ragable/sdk] createClient: `apiKey` is set, so the server client is returned. It has no `database` or `auth` \u2014 only `agents` and `shift`. For `database.from()` / `auth.*`, use the browser client without `apiKey` (e.g. createClient(ragableUrl, { organizationId, authGroupId, databaseInstanceId, ... }))."
+      );
+    }
     return new Ragable(urlOrOptions);
   }
   if (typeof urlOrOptions === "object" && urlOrOptions !== null && "organizationId" in urlOrOptions && typeof urlOrOptions.organizationId === "string") {