@crosspost/sdk 0.1.5 → 0.1.7

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@crosspost/sdk",
-  "version": "0.1.5",
+  "version": "0.1.7",
   "description": "SDK for interacting with the Crosspost API",
   "type": "module",
   "main": "dist/index.cjs",

package/src/api/auth.ts

@@ -1,8 +1,8 @@
 import type {
+  ApiResponse,
   AuthRevokeResponse,
   AuthStatusResponse,
   ConnectedAccountsResponse,
-  EnhancedApiResponse,
   NearAuthorizationResponse,
   Platform,
 } from '@crosspost/types';
@@ -57,8 +57,8 @@ export class AuthApi {
   async loginToPlatform(
     platform: Platform,
     options?: { successUrl?: string; errorUrl?: string },
-  ): Promise<EnhancedApiResponse<any>> { // TODO: Refine response type based on actual API
-    return makeRequest<EnhancedApiResponse<any>>(
+  ): Promise<ApiResponse<any>> { // TODO: Refine response type based on actual API
+    return makeRequest<ApiResponse<any>>(
       'POST',
       `/auth/${platform}/login`,
       this.options,
@@ -71,8 +71,8 @@ export class AuthApi {
    * @param platform The target platform.
    * @returns A promise resolving with the refresh response.
    */
-  async refreshToken(platform: Platform): Promise<EnhancedApiResponse<any>> { // TODO: Refine response type
-    return makeRequest<EnhancedApiResponse<any>>(
+  async refreshToken(platform: Platform): Promise<ApiResponse<any>> { // TODO: Refine response type
+    return makeRequest<ApiResponse<any>>(
       'POST',
       `/auth/${platform}/refresh`,
       this.options,
@@ -84,8 +84,8 @@ export class AuthApi {
    * @param platform The target platform.
    * @returns A promise resolving with the profile refresh response.
    */
-  async refreshProfile(platform: Platform): Promise<EnhancedApiResponse<any>> { // TODO: Refine response type
-    return makeRequest<EnhancedApiResponse<any>>(
+  async refreshProfile(platform: Platform): Promise<ApiResponse<any>> { // TODO: Refine response type
+    return makeRequest<ApiResponse<any>>(
       'POST',
       `/auth/${platform}/refresh-profile`,
       this.options,

package/src/api/post.ts

@@ -14,7 +14,6 @@ import type {
   UnlikePostRequest,
   UnlikePostResponse,
 } from '@crosspost/types';
-import { ApiError, ApiErrorCode } from '@crosspost/types';
 import { makeRequest, type RequestOptions } from '../core/request.ts';
 
 /**
@@ -93,10 +92,9 @@ export class PostApi {
    * @returns A promise resolving with the like response.
    */
   async likePost(request: LikePostRequest): Promise<LikePostResponse> {
-    // API endpoint uses postId in the path
     return makeRequest<LikePostResponse>(
       'POST',
-      `/api/post/like/${request.postId}`,
+      `/api/post/like`,
       this.options,
       request,
     );
@@ -108,10 +106,9 @@ export class PostApi {
    * @returns A promise resolving with the unlike response.
    */
   async unlikePost(request: UnlikePostRequest): Promise<UnlikePostResponse> {
-    // API endpoint uses postId in the path
     return makeRequest<UnlikePostResponse>(
       'DELETE',
-      `/api/post/like/${request.postId}`,
+      `/api/post/like`,
       this.options,
       request,
     );
@@ -123,18 +120,9 @@ export class PostApi {
    * @returns A promise resolving with the delete response.
    */
   async deletePost(request: DeletePostRequest): Promise<DeletePostResponse> {
-    // API endpoint uses postId in the path, assuming the first post ID for the URL
-    const postId = request.posts[0]?.postId || '';
-    if (!postId) {
-      throw new ApiError(
-        'Post ID is required for deletion path',
-        ApiErrorCode.VALIDATION_ERROR,
-        400,
-      );
-    }
     return makeRequest<DeletePostResponse>(
       'DELETE',
-      `/api/post/${postId}`,
+      `/api/post`,
       this.options,
       request,
     );

package/src/api/system.ts

@@ -1,4 +1,4 @@
-import type { EndpointRateLimitResponse, RateLimitResponse } from '@crosspost/types';
+import type { ApiResponse, EndpointRateLimitResponse, RateLimitResponse } from '@crosspost/types';
 import { makeRequest, type RequestOptions } from '../core/request.ts';
 
 /**
@@ -45,8 +45,8 @@ export class SystemApi {
    * Gets the health status of the API
    * @returns A promise resolving with the health status
    */
-  async getHealthStatus(): Promise<{ status: string }> {
-    return makeRequest<{ status: string }>(
+  async getHealthStatus(): Promise<ApiResponse<{ status: string }>> {
+    return makeRequest<ApiResponse<{ status: string }>>(
       'GET',
       '/health',
       this.options,

package/src/core/request.ts

@@ -1,6 +1,11 @@
-import { ApiError, ApiErrorCode } from '@crosspost/types';
+import { ApiErrorCode, type StatusCode } from '@crosspost/types';
 import { createAuthToken, type NearAuthData } from 'near-sign-verify';
-import { apiWrapper, createNetworkError, handleErrorResponse } from '../utils/error.ts';
+import {
+  apiWrapper,
+  createNetworkError,
+  CrosspostError,
+  handleErrorResponse,
+} from '../utils/error.ts';
 
 /**
  * Options for making a request to the API
@@ -56,10 +61,6 @@ export async function makeRequest<T>(
       url += `?${queryString}`;
     }
   }
-  // Check if authentication data is available
-  if (!options.nearAuthData) {
-    throw ApiError.unauthorized('Authentication required. Please provide NEAR signature.');
-  }
 
   // Create a context object for error enrichment
   const context = {
@@ -97,32 +98,30 @@ export async function makeRequest<T>(
         try {
           responseData = await response.json();
         } catch (jsonError) {
-          // If JSON parsing fails, throw a specific error or handle based on status
+          // If JSON parsing fails, did API throw an error?
           if (!response.ok) {
-            throw new ApiError(
+            throw new CrosspostError(
               `API request failed with status ${response.status} and non-JSON response`,
               ApiErrorCode.NETWORK_ERROR,
-              response.status as any,
+              response.status as StatusCode,
               { originalStatusText: response.statusText },
             );
           }
-          // If response was ok but JSON failed, maybe it was an empty 204 response?
-          if (response.status === 204) return {} as T; // Handle No Content
           // Otherwise, throw a custom error
-          throw new ApiError(
+          throw new CrosspostError(
             `Failed to parse JSON response: ${
               jsonError instanceof Error ? jsonError.message : String(jsonError)
             }`,
             ApiErrorCode.INTERNAL_ERROR,
-            response.status as any,
+            response.status as StatusCode,
           );
         }
 
         if (!response.ok) {
           lastError = handleErrorResponse(responseData, response.status);
-          // Retry only on 5xx errors or potentially recoverable errors if defined
-          const shouldRetry = response.status >= 500 ||
-            (lastError instanceof ApiError && lastError.recoverable);
+          // Only retry rate limit errors
+          const shouldRetry = lastError instanceof CrosspostError &&
+            lastError.code === ApiErrorCode.RATE_LIMITED;
           if (shouldRetry && attempt < options.retries) {
             await new Promise((resolve) => setTimeout(resolve, 1000 * Math.pow(2, attempt))); // Exponential backoff
             continue; // Retry
@@ -130,22 +129,24 @@ export async function makeRequest<T>(
           throw lastError; // Throw error if not retrying or retries exhausted
         }
 
-        // Handle cases where API indicates failure within a 2xx response
-        if (
-          responseData && typeof responseData === 'object' && 'success' in responseData &&
-          !responseData.success && responseData.error
-        ) {
-          lastError = handleErrorResponse(responseData, response.status);
-          // Decide if this specific type of "successful" response with an error payload should be retried
-          const shouldRetry = lastError instanceof ApiError && lastError.recoverable;
-          if (shouldRetry && attempt < options.retries) {
-            await new Promise((resolve) => setTimeout(resolve, 1000 * Math.pow(2, attempt))); // Exponential backoff
-            continue; // Retry
+        // Handle response based on success flag
+        if (responseData && typeof responseData === 'object' && 'success' in responseData) {
+          if (responseData.success) {
+            // Success response - return the data
+            return responseData.data as T;
+          } else {
+            // Error response - handle with our error utilities
+            lastError = handleErrorResponse(responseData, response.status);
+            // Only retry rate limit errors
+            const shouldRetry = lastError instanceof CrosspostError &&
+              lastError.code === ApiErrorCode.RATE_LIMITED;
+            if (shouldRetry && attempt < options.retries) {
+              await new Promise((resolve) => setTimeout(resolve, 1000 * Math.pow(2, attempt))); // Exponential backoff
+              continue; // Retry
+            }
+            throw lastError;
           }
-          throw lastError;
         }
-
-        return responseData as T;
       } catch (error) {
         clearTimeout(timeoutId); // Clear timeout on error
         lastError = error instanceof Error ? error : new Error(String(error)); // Store the error
@@ -159,7 +160,7 @@ export async function makeRequest<T>(
         }
 
         // If it's not a known ApiError/PlatformError, wrap it
-        if (!(error instanceof ApiError)) {
+        if (!(error instanceof CrosspostError)) {
           throw createNetworkError(error, url, options.timeout);
         }
 
@@ -169,6 +170,6 @@ export async function makeRequest<T>(
 
     // Should not be reachable if retries >= 0, but needed for type safety
     throw lastError ||
-      new ApiError('Request failed after multiple retries', ApiErrorCode.INTERNAL_ERROR, 500);
+      new CrosspostError('Request failed after multiple retries', ApiErrorCode.INTERNAL_ERROR, 500);
   }, context);
 }

package/src/index.ts

@@ -1,30 +1,20 @@
-/**
- * @crosspost/sdk
- * SDK for interacting with the Crosspost API
- */
-
-// Export main client
 export { CrosspostClient } from './core/client.ts';
 export type { CrosspostClientConfig } from './core/config.ts';
 
-// Export API modules for advanced usage
 export { ActivityApi } from './api/activity.ts';
 export { AuthApi } from './api/auth.ts';
 export { PostApi } from './api/post.ts';
 export { SystemApi } from './api/system.ts';
 
-// Export utility functions
 export {
   apiWrapper,
   createNetworkError,
   enrichErrorWithContext,
-  ERROR_CATEGORIES,
   getErrorDetails,
   getErrorMessage,
   handleErrorResponse,
   isAuthError,
   isContentError,
-  isErrorOfCategory,
   isMediaError,
   isNetworkError,
   isPlatformError,