@reshotdev/screenshot 0.0.1-beta.2 → 0.0.1-beta.20

package/src/lib/api-client.js

@@ -3,15 +3,80 @@ const axios = require("axios");
 const FormData = require("form-data");
 const fs = require("fs");
 
-const baseUrl =
-  process.env.RESHOT_API_BASE_URL ||
-  process.env.DOCSYNC_API_BASE_URL ||
-  "http://localhost:3000/api";
+const PRODUCTION_API_URL = "https://reshot.dev/api";
+
+function summarizeApiBody(body) {
+  if (body == null) return "";
+  if (typeof body === "string") return body.trim();
+  if (typeof body !== "object") return String(body);
+
+  const candidates = [
+    body.message,
+    body.error?.message,
+    body.error,
+    body.reason,
+    body.details,
+  ].filter(Boolean);
+
+  for (const candidate of candidates) {
+    if (typeof candidate === "string" && candidate.trim()) {
+      return candidate.trim();
+    }
+  }
+
+  try {
+    return JSON.stringify(body);
+  } catch {
+    return String(body);
+  }
+}
+
+function createApiError(kind, endpoint, error) {
+  const status = error.response?.status || null;
+  const statusText = error.response?.statusText || "";
+  const bodySummary = summarizeApiBody(error.response?.data);
+  const baseMessage = bodySummary || error.message || "Unknown API error";
+  const statusLabel = status ? `${status}${statusText ? ` ${statusText}` : ""}` : "request failed";
+  const wrapped = new Error(`${kind} (${statusLabel}) at ${endpoint}: ${baseMessage}`);
+  wrapped.response = error.response;
+  wrapped.code = error.code;
+  wrapped.reshot = {
+    kind,
+    endpoint,
+    status,
+    statusText,
+    bodySummary,
+  };
+  return wrapped;
+}
 
 function getApiBaseUrl() {
-  return baseUrl;
+  // 1. Explicit env var override (for CI or local dev)
+  if (process.env.RESHOT_API_BASE_URL) {
+    return process.env.RESHOT_API_BASE_URL;
+  }
+
+  // 2. Read from settings.json (set during auth/setup)
+  try {
+    const path = require("path");
+    const settingsPath = path.join(process.cwd(), ".reshot", "settings.json");
+    if (fs.existsSync(settingsPath)) {
+      const settings = JSON.parse(fs.readFileSync(settingsPath, "utf8"));
+      if (settings.platformUrl) {
+        return settings.platformUrl.replace(/\/+$/, "") + "/api";
+      }
+    }
+  } catch {
+    // Settings don't exist yet (first auth) — fall through to default
+  }
+
+  // 3. Default to production
+  return PRODUCTION_API_URL;
 }
 
+// Resolved once at module load — all API calls use this
+const baseUrl = getApiBaseUrl();
+
 /**
  * Sleep helper for retry delays
  */
@@ -54,6 +119,17 @@ async function withRetry(fn, options = {}) {
         throw error;
       }
 
+      // Respect Retry-After header on 429 rate limit responses
+      const retryAfterHeader = error.response?.headers?.["retry-after"];
+      if (retryAfterHeader && statusCode === 429) {
+        const retryMs = (parseInt(retryAfterHeader, 10) || 5) * 1000;
+        console.log(
+          `  ⚠ Rate limited (attempt ${attempt}/${maxRetries}), retrying in ${retryMs / 1000}s...`,
+        );
+        await sleep(retryMs);
+        continue;
+      }
+
       // Calculate delay with exponential backoff + jitter
       const delay = Math.min(
         initialDelay * Math.pow(2, attempt - 1) + Math.random() * 1000,
@@ -190,20 +266,7 @@ async function publishAssetsV1(apiKey, metadata, assets) {
     });
     return response.data;
   } catch (error) {
-    if (error.response) {
-      const status = error.response.status;
-      const errorMsg = error.response.data?.error || error.message;
-
-      // Create an error that preserves the response for auth detection
-      const err = new Error(
-        status === 401 || status === 403
-          ? `Authentication failed: ${errorMsg}`
-          : `Failed to publish assets: ${errorMsg}`,
-      );
-      err.response = error.response;
-      throw err;
-    }
-    throw new Error(`Failed to publish assets: ${error.message}`);
+    throw createApiError("publish_ingest_failure", `${baseUrl}/v1/publish`, error);
   }
 }
 
@@ -586,6 +649,34 @@ async function publishTransactional(apiKey, payload) {
   return response.data;
 }
 
+/**
+ * Batch publish metadata for multiple scenarios in one request
+ * @param {string} apiKey - API key for authentication
+ * @param {Object} payload - { commits: Array<{ metadata, assets }> }
+ * @returns {Promise<Object>}
+ */
+async function publishBatch(apiKey, payload) {
+  if (!apiKey) {
+    throw new Error("API key is required to publish");
+  }
+
+  return withRetry(
+    async () => {
+      const response = await axios.post(`${baseUrl}/v1/publish/batch`, payload, {
+        headers: {
+          "Content-Type": "application/json",
+          Authorization: `Bearer ${apiKey}`,
+        },
+        timeout: 60000,
+        maxBodyLength: Infinity,
+        maxContentLength: Infinity,
+      });
+      return response.data;
+    },
+    { maxRetries: 2, retryOn: [500, 502, 503, 504, 429, "ECONNRESET", "ETIMEDOUT"] },
+  );
+}
+
 /**
  * Check which hashes already exist in storage (for deduplication)
  * @param {string} apiKey - API key for authentication
@@ -648,7 +739,7 @@ async function getBaselines(projectId, apiKey) {
  */
 async function exportVisuals(projectId, options = {}) {
   const { format = "json", status = "approved" } = options;
-  const settings = require("./config").loadSettings();
+  const settings = require("./config").readSettings();
   const apiKey = settings?.apiKey;
 
   if (!apiKey) {
@@ -656,15 +747,23 @@ async function exportVisuals(projectId, options = {}) {
   }
 
   return withRetry(async () => {
-    const response = await axios.get(
-      `${baseUrl}/projects/${projectId}/visuals/export`,
-      {
-        params: { format, status },
-        headers: { Authorization: `Bearer ${apiKey}` },
-        timeout: 60000,
-      },
-    );
-    return response.data;
+    try {
+      const response = await axios.get(
+        `${baseUrl}/projects/${projectId}/visuals/export`,
+        {
+          params: { format, status },
+          headers: { Authorization: `Bearer ${apiKey}` },
+          timeout: 60000,
+        },
+      );
+      return response.data;
+    } catch (error) {
+      throw createApiError(
+        "export_visuals_failure",
+        `${baseUrl}/projects/${projectId}/visuals/export`,
+        error,
+      );
+    }
   });
 }
 
@@ -688,7 +787,7 @@ async function post(endpoint, data, options = {}) {
 }
 
 /**
- * DocSync: Initialize ingestion job with manifest
+ * Initialize ingestion job with manifest
  */
 async function initIngest(apiKey, projectId, manifest) {
   return withRetry(async () => {
@@ -705,7 +804,7 @@ async function initIngest(apiKey, projectId, manifest) {
 }
 
 /**
- * DocSync: Commit ingestion job after uploads complete
+ * Commit ingestion job after uploads complete
  */
 async function commitIngest(apiKey, projectId, uploadResults, git, cli) {
   return withRetry(async () => {
@@ -722,47 +821,56 @@ async function commitIngest(apiKey, projectId, uploadResults, git, cli) {
 }
 
 /**
- * DocSync: Get drift records for a project
+ * Get drift records for a project
  */
 async function getDrifts(apiKey, projectId, options = {}) {
-  return withRetry(async () => {
-    const params = new URLSearchParams();
-    if (options.status) params.set("status", options.status);
-    if (options.journeyKey) params.set("journeyKey", options.journeyKey);
+  const params = new URLSearchParams();
+  if (options.status) params.set("status", options.status);
+  if (options.journeyKey) params.set("journeyKey", options.journeyKey);
+  const endpoint = `${baseUrl}/v1/projects/${projectId}/drifts?${params.toString()}`;
 
-    const response = await axios.get(
-      `${baseUrl}/v1/projects/${projectId}/drifts?${params.toString()}`,
-      {
+  try {
+    return await withRetry(async () => {
+      const response = await axios.get(endpoint, {
         headers: { Authorization: `Bearer ${apiKey}` },
         timeout: 30000,
-      },
-    );
-    return response.data;
-  });
+      });
+      return response.data;
+    });
+  } catch (error) {
+    // Surface HTTP status + endpoint + server message instead of axios's
+    // bare "Request failed with status code N".
+    throw createApiError("Drifts request failed", endpoint, error);
+  }
 }
 
 /**
- * DocSync: Get sync jobs for a project
+ * Get sync jobs for a project
  */
 async function getSyncJobs(apiKey, projectId, options = {}) {
-  return withRetry(async () => {
-    const response = await axios.post(
-      `${baseUrl}/v1/projects/${projectId}/sync-jobs`,
-      {
-        limit: options.limit || 10,
-        status: options.status,
-      },
-      {
-        headers: { Authorization: `Bearer ${apiKey}` },
-        timeout: 30000,
-      },
-    );
-    return response.data;
-  });
+  const endpoint = `${baseUrl}/v1/projects/${projectId}/sync-jobs`;
+  try {
+    return await withRetry(async () => {
+      const response = await axios.post(
+        endpoint,
+        {
+          limit: options.limit || 10,
+          status: options.status,
+        },
+        {
+          headers: { Authorization: `Bearer ${apiKey}` },
+          timeout: 30000,
+        },
+      );
+      return response.data;
+    });
+  } catch (error) {
+    throw createApiError("Sync jobs request failed", endpoint, error);
+  }
 }
 
 /**
- * DocSync: Perform action on a drift record
+ * Perform action on a drift record
  */
 async function driftAction(apiKey, projectId, driftId, action, options = {}) {
   return withRetry(async () => {
@@ -794,22 +902,26 @@ module.exports = {
   getProjectConfig,
   postChangelogDrafts,
   getApiBaseUrl,
+  createApiError,
   syncPushAssets,
   getSyncStatus,
   // New transactional flow
   signAssets,
   uploadToPresignedUrl,
   publishTransactional,
+  publishBatch,
   checkExistingHashes,
   // Diffing support
   getBaselines,
   // Export support
   exportVisuals,
-  // DocSync
+  // Reshot
   post,
   initIngest,
   commitIngest,
   getDrifts,
   getSyncJobs,
   driftAction,
+  // Testing
+  withRetry,
 };