Skip to content

Commit 5cc7296

Browse files
committed
Cleanup error messages
1 parent c4cabf6 commit 5cc7296

File tree

1 file changed

+16
-5
lines changed

1 file changed

+16
-5
lines changed

src/index.ts

Lines changed: 16 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -58,10 +58,14 @@ const DEFAULT_USER_AGENT = createUserAgentFromPkgJson(rootPkgJson)
5858
class ResponseError extends Error {
5959
response: IncomingMessage
6060
constructor(response: IncomingMessage, message: string = '') {
61+
const statusCode = response.statusCode ?? 'Unknown'
62+
const statusMessage = response.statusMessage ?? 'No status message'
6163
super(
62-
`Socket API ${message || 'request failed'}: ${response.statusCode} - ${response.statusMessage}`
64+
`Socket API - ${message || 'Request failed'}: ${statusCode} - ${statusMessage}`
6365
)
66+
this.name = 'ResponseError'
6467
this.response = response
68+
Error.captureStackTrace(this, ResponseError)
6569
}
6670
}
6771

@@ -180,7 +184,9 @@ async function createUploadRequest(
180184
// Ensure a new line after file content.
181185
req.write('\r\n')
182186
} else {
183-
throw new TypeError('Invalid multipart part: expected string or stream')
187+
throw new TypeError(
188+
'Socket API - Invalid multipart part, expected string or stream'
189+
)
184190
}
185191
}
186192
} finally {
@@ -219,8 +225,11 @@ async function getResponseJson(
219225
}
220226
try {
221227
return JSON.parse(data)
222-
} catch {
223-
throw new Error(`Socket API returned an invalid JSON response: ${data}`)
228+
} catch (e) {
229+
throw new Error(
230+
`Socket API - Invalid JSON response:\n${data}\n→ ${(e as Error)?.message || 'Unknown error'}`,
231+
{ cause: e }
232+
)
224233
}
225234
}
226235

@@ -344,7 +353,9 @@ export class SocketSdk {
344353
}
345354
const statusCode = error.response.statusCode
346355
if (statusCode! >= 500) {
347-
throw new Error('Socket API returned an error', { cause: error })
356+
throw new Error(`Socket API server error (${statusCode})`, {
357+
cause: error
358+
})
348359
}
349360
return {
350361
success: false as const,

0 commit comments

Comments
 (0)