@astralform/js 5.0.0 → 6.0.0

package/README.md

@@ -265,8 +265,13 @@ const id = await session.createNewConversation();
 // Switch to an existing conversation (loads messages from backend)
 await session.switchConversation("conversation-id");
 
-// Delete a conversation
-await session.deleteConversation("conversation-id");
+// Delete a conversation. Rejects if the server refused it — a 404 counts as
+// already deleted and resolves, anything else leaves the conversation in place.
+try {
+  await session.deleteConversation("conversation-id");
+} catch (err) {
+  // Still there: tell the user rather than removing it from your own UI.
+}
 
 // Edit and resend from a checkpoint
 await session.resendFromCheckpoint("message-id", "Updated message");
@@ -357,7 +362,7 @@ import {
   AuthenticationError, // 401 — invalid API key
   RateLimitError, // 429 — rate limit exceeded
   LLMNotConfiguredError, // LLM provider not set up
-  ServerError, // 5xx or unexpected errors
+  ServerError, // 5xx or unexpected errors — carries `status` when it came from a response
   ConnectionError, // Network failures
   StreamAbortedError, // Stream cancelled via disconnect()
 } from "@astralform/js";

package/dist/index.cjs

@@ -73,9 +73,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 {
@@ -105,6 +106,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";
@@ -255,7 +266,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) {
@@ -526,7 +537,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
+        );
       }
     }
   }
@@ -563,13 +577,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`);
@@ -594,13 +602,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)}`);
@@ -613,14 +615,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
@@ -647,12 +642,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`;
@@ -723,6 +713,11 @@ var AstralformClient = class {
       // undefined so it matches the `url?: string` type and consumers that
       // check `!== undefined` never receive a null.
       url: raw.url ?? void 0,
+      // This mapping is an ALLOWLIST — a field the API returns and this
+      // function does not name is dropped silently, and no type error says so.
+      // `content_url` shipped that way and was invisible to every consumer.
+      contentUrl: raw.content_url ?? void 0,
+      posterUrl: raw.poster_url ?? void 0,
       createdAt: raw.created_at
     };
   }
@@ -764,13 +759,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
@@ -779,14 +768,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) {
@@ -823,13 +805,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`);
@@ -2217,11 +2193,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.
@@ -2237,7 +2214,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)) {