@astralform/js 5.1.0 → 6.0.0

package/dist/index.d.cts

@@ -1400,11 +1400,12 @@ declare class ChatSession {
     /**
      * Rename a conversation, server first.
      *
-     * Deliberately NOT optimistic, unlike the delete below. A failed delete is
-     * self-correcting (the row is still there on the next page fetch), but a
-     * failed rename that had already been written locally would leave the
-     * sidebar showing a title the server never accepted — and nothing refetches
-     * a conversation that is already in the loaded list.
+     * Server first, like the delete below: a failed rename written locally would
+     * leave the sidebar showing a title the server never accepted, and nothing
+     * refetches a conversation that is already in the loaded list. (The delete
+     * used to be the counter-example here — it dropped the row whatever the
+     * server said, on the theory that a failed one was self-correcting. It was
+     * not: the row stayed deleted locally and alive on the server.)
      *
      * Mirrors the `title_generated` path: the entry in `conversations` is
      * mutated in place, which is what every consumer of the list reads.
@@ -1444,7 +1445,21 @@ declare class LLMNotConfiguredError extends AstralformError {
     constructor(message?: string);
 }
 declare class ServerError extends AstralformError {
-    constructor(message?: string);
+    /**
+     * The HTTP status, when this came from a response.
+     *
+     * Every status except 401 and 429 collapses into this one class, so without
+     * it a caller cannot tell "the thing is already gone" (404) from "the server
+     * broke" (500) or "this is not yours" (403) — the message is the only other
+     * signal and it is prose. `deleteConversation` is the case that forced it:
+     * it read EVERY failure as already-deleted and dropped the conversation
+     * locally regardless, so a 500 looked exactly like success and the row came
+     * back on the next device.
+     *
+     * Undefined when a ServerError is constructed without a response.
+     */
+    readonly status?: number;
+    constructor(message?: string, status?: number);
 }
 declare class ConnectionError extends AstralformError {
     constructor(message?: string);

package/dist/index.d.ts

@@ -1400,11 +1400,12 @@ declare class ChatSession {
     /**
      * Rename a conversation, server first.
      *
-     * Deliberately NOT optimistic, unlike the delete below. A failed delete is
-     * self-correcting (the row is still there on the next page fetch), but a
-     * failed rename that had already been written locally would leave the
-     * sidebar showing a title the server never accepted — and nothing refetches
-     * a conversation that is already in the loaded list.
+     * Server first, like the delete below: a failed rename written locally would
+     * leave the sidebar showing a title the server never accepted, and nothing
+     * refetches a conversation that is already in the loaded list. (The delete
+     * used to be the counter-example here — it dropped the row whatever the
+     * server said, on the theory that a failed one was self-correcting. It was
+     * not: the row stayed deleted locally and alive on the server.)
      *
      * Mirrors the `title_generated` path: the entry in `conversations` is
      * mutated in place, which is what every consumer of the list reads.
@@ -1444,7 +1445,21 @@ declare class LLMNotConfiguredError extends AstralformError {
     constructor(message?: string);
 }
 declare class ServerError extends AstralformError {
-    constructor(message?: string);
+    /**
+     * The HTTP status, when this came from a response.
+     *
+     * Every status except 401 and 429 collapses into this one class, so without
+     * it a caller cannot tell "the thing is already gone" (404) from "the server
+     * broke" (500) or "this is not yours" (403) — the message is the only other
+     * signal and it is prose. `deleteConversation` is the case that forced it:
+     * it read EVERY failure as already-deleted and dropped the conversation
+     * locally regardless, so a 500 looked exactly like success and the row came
+     * back on the next device.
+     *
+     * Undefined when a ServerError is constructed without a response.
+     */
+    readonly status?: number;
+    constructor(message?: string, status?: number);
 }
 declare class ConnectionError extends AstralformError {
     constructor(message?: string);

package/dist/index.js

@@ -26,9 +26,10 @@ var LLMNotConfiguredError = class extends AstralformError {
   }
 };
 var ServerError = class extends AstralformError {
-  constructor(message = "Internal server error") {
+  constructor(message = "Internal server error", status) {
     super(message, "server_error");
     this.name = "ServerError";
+    if (status !== void 0) Object.assign(this, { status });
   }
 };
 var ConnectionError = class extends AstralformError {
@@ -58,6 +59,16 @@ function generateId() {
     return v.toString(16);
   });
 }
+function snakeToCamel(str) {
+  return str.replace(/_([a-z])/g, (_, c) => c.toUpperCase());
+}
+function camelizeKeys(obj) {
+  const result = {};
+  for (const key of Object.keys(obj)) {
+    result[snakeToCamel(key)] = obj[key];
+  }
+  return result;
+}
 
 // src/rate-limit.ts
 var DEFAULT_MESSAGE = "Rate limit exceeded";
@@ -208,7 +219,7 @@ async function* streamJobSSE(options) {
       case 429:
         throw createRateLimitErrorFromHttp(response, rawText);
       default:
-        throw new ServerError(text || `HTTP ${response.status}`);
+        throw new ServerError(text || `HTTP ${response.status}`, response.status);
     }
   }
   if (!response.body) {
@@ -479,7 +490,10 @@ var AstralformClient = class {
         throw createRateLimitErrorFromHttp(response, text);
       default: {
         const safeText = text ? sanitizeErrorText(text) : "";
-        throw new ServerError(safeText || `HTTP ${response.status}`);
+        throw new ServerError(
+          safeText || `HTTP ${response.status}`,
+          response.status
+        );
       }
     }
   }
@@ -516,13 +530,7 @@ var AstralformClient = class {
     const safeLimit = Math.max(1, Math.min(200, Math.floor(Number(limit))));
     const safeOffset = Math.max(0, Math.floor(Number(offset)));
     const raw = await this.get(`/v1/conversations?limit=${safeLimit}&offset=${safeOffset}`);
-    return raw.map((c) => ({
-      id: c.id,
-      title: c.title,
-      messageCount: c.message_count,
-      createdAt: c.created_at,
-      updatedAt: c.updated_at
-    }));
+    return raw.map((c) => camelizeKeys(c));
   }
   async getMessages(conversationId) {
     const raw = await this.get(`/v1/conversations/${encodeURIComponent(conversationId)}/messages`);
@@ -547,13 +555,7 @@ var AstralformClient = class {
    */
   async renameConversation(id, title) {
     const c = await this.patch(`/v1/conversations/${encodeURIComponent(id)}`, { title });
-    return {
-      id: c.id,
-      title: c.title,
-      messageCount: c.message_count,
-      createdAt: c.created_at,
-      updatedAt: c.updated_at
-    };
+    return camelizeKeys(c);
   }
   async deleteConversation(id) {
     await this.del(`/v1/conversations/${encodeURIComponent(id)}`);
@@ -566,14 +568,7 @@ var AstralformClient = class {
    */
   async getAgents() {
     const raw = await this.get("/v1/agents");
-    return raw.map((a) => ({
-      name: a.name,
-      displayName: a.display_name,
-      description: a.description,
-      isOrchestrator: a.is_orchestrator,
-      isEnabled: a.is_enabled,
-      avatarUrl: a.avatar_url
-    }));
+    return raw.map((a) => camelizeKeys(a));
   }
   /**
    * List the models the caller may pick this turn — expanded from the curated
@@ -600,12 +595,7 @@ var AstralformClient = class {
   }
   async getSkills() {
     const raw = await this.get("/v1/skills");
-    return raw.map((s) => ({
-      name: s.name,
-      displayName: s.display_name,
-      description: s.description,
-      isEnabled: s.is_enabled
-    }));
+    return raw.map((s) => camelizeKeys(s));
   }
   async getConversationEvents(conversationId, jobId) {
     let url = `/v1/conversations/${encodeURIComponent(conversationId)}/events`;
@@ -722,13 +712,7 @@ var AstralformClient = class {
   // sending them in API-key mode yields 401.
   async listTeams() {
     const raw = await this.get("/v1/teams");
-    return raw.map((t) => ({
-      id: t.id,
-      name: t.name,
-      slug: t.slug,
-      isDefault: t.is_default,
-      role: t.role
-    }));
+    return raw.map((t) => camelizeKeys(t));
   }
   /**
    * List the team-level agents (formerly "projects") the signed-in user can
@@ -737,14 +721,7 @@ var AstralformClient = class {
    */
   async listAgents(teamId) {
     const raw = await this.get(`/v1/teams/${encodeURIComponent(teamId)}/agents`);
-    return raw.map((a) => ({
-      id: a.id,
-      name: a.name,
-      teamId: a.team_id,
-      createdAt: a.created_at,
-      updatedAt: a.updated_at,
-      avatarUrl: a.avatar_url ?? null
-    }));
+    return raw.map((a) => camelizeKeys(a));
   }
   // --- Jobs API ---
   async createJob(request) {
@@ -781,13 +758,7 @@ var AstralformClient = class {
     };
     if (request.comment != null) body.comment = request.comment;
     const raw = await this.post(`/v1/jobs/${encodeURIComponent(jobId)}/feedback`, body);
-    return {
-      id: raw.id,
-      jobId: raw.job_id,
-      rating: raw.rating,
-      comment: raw.comment,
-      createdAt: raw.created_at
-    };
+    return camelizeKeys(raw);
   }
   async getActiveJob(conversationId) {
     const raw = await this.get(`/v1/conversations/${encodeURIComponent(conversationId)}/active-job`);
@@ -2175,11 +2146,12 @@ var ChatSession = class {
   /**
    * Rename a conversation, server first.
    *
-   * Deliberately NOT optimistic, unlike the delete below. A failed delete is
-   * self-correcting (the row is still there on the next page fetch), but a
-   * failed rename that had already been written locally would leave the
-   * sidebar showing a title the server never accepted — and nothing refetches
-   * a conversation that is already in the loaded list.
+   * Server first, like the delete below: a failed rename written locally would
+   * leave the sidebar showing a title the server never accepted, and nothing
+   * refetches a conversation that is already in the loaded list. (The delete
+   * used to be the counter-example here — it dropped the row whatever the
+   * server said, on the theory that a failed one was self-correcting. It was
+   * not: the row stayed deleted locally and alive on the server.)
    *
    * Mirrors the `title_generated` path: the entry in `conversations` is
    * mutated in place, which is what every consumer of the list reads.
@@ -2195,7 +2167,8 @@ var ChatSession = class {
   async deleteConversation(id) {
     try {
       await this.client.deleteConversation(id);
-    } catch {
+    } catch (err) {
+      if (!(err instanceof ServerError) || err.status !== 404) throw err;
     }
     await this.storage.deleteConversation(id);
     if (this.serverConversationIds.delete(id)) {