From 5ea2c3a9289abf59239b2d36927c711589a75952 Mon Sep 17 00:00:00 2001 From: Gaurav Chadha Date: Thu, 27 Nov 2025 14:44:40 +0530 Subject: [PATCH 1/3] pydantic-parsing-error-handling Signed-off-by: Gaurav Chadha --- apps/api/src/controllers/v1/extract-status.ts | 2 +- apps/api/src/controllers/v2/extract-status.ts | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/apps/api/src/controllers/v1/extract-status.ts b/apps/api/src/controllers/v1/extract-status.ts index 8a3e27386..4a61cdc7a 100644 --- a/apps/api/src/controllers/v1/extract-status.ts +++ b/apps/api/src/controllers/v1/extract-status.ts @@ -116,7 +116,7 @@ export async function extractStatusController( success: status === "failed" ? false : true, data, status, - error: extract?.error ?? undefined, + error: typeof extract?.error === "string" ? extract.error : undefined, expiresAt: (await getExtractExpiry(req.params.jobId)).toISOString(), steps: extract?.showSteps ? extract.steps : undefined, llmUsage: extract?.showLLMUsage ? extract.llmUsage : undefined, diff --git a/apps/api/src/controllers/v2/extract-status.ts b/apps/api/src/controllers/v2/extract-status.ts index 3d9ae5111..f640a14b0 100644 --- a/apps/api/src/controllers/v2/extract-status.ts +++ b/apps/api/src/controllers/v2/extract-status.ts @@ -116,7 +116,7 @@ export async function extractStatusController( success: status === "failed" ? false : true, data, status, - error: extract?.error ?? undefined, + error: typeof extract?.error === "string" ? extract.error : undefined, expiresAt: (await getExtractExpiry(req.params.jobId)).toISOString(), steps: extract?.showSteps ? extract.steps : undefined, llmUsage: extract?.showLLMUsage ? extract.llmUsage : undefined, From 5eefc3c2b22f670b5657ac0a3d8bc96896a9ce0f Mon Sep 17 00:00:00 2001 From: Gaurav Chadha Date: Thu, 27 Nov 2025 14:59:07 +0530 Subject: [PATCH 2/3] stringify-error Signed-off-by: Gaurav Chadha --- apps/api/src/controllers/v1/extract-status.ts | 8 +++++++- apps/api/src/controllers/v2/extract-status.ts | 8 +++++++- 2 files changed, 14 insertions(+), 2 deletions(-) diff --git a/apps/api/src/controllers/v1/extract-status.ts b/apps/api/src/controllers/v1/extract-status.ts index 4a61cdc7a..c232d0f1c 100644 --- a/apps/api/src/controllers/v1/extract-status.ts +++ b/apps/api/src/controllers/v1/extract-status.ts @@ -116,7 +116,13 @@ export async function extractStatusController( success: status === "failed" ? false : true, data, status, - error: typeof extract?.error === "string" ? extract.error : undefined, + error: (() => { + if (typeof extract?.error === "string") return extract.error; + if (extract?.error && typeof extract.error === "object") { + return extract.error.message || extract.error.error || JSON.stringify(extract.error); + } + return undefined; + })(), expiresAt: (await getExtractExpiry(req.params.jobId)).toISOString(), steps: extract?.showSteps ? extract.steps : undefined, llmUsage: extract?.showLLMUsage ? extract.llmUsage : undefined, diff --git a/apps/api/src/controllers/v2/extract-status.ts b/apps/api/src/controllers/v2/extract-status.ts index f640a14b0..f68081d54 100644 --- a/apps/api/src/controllers/v2/extract-status.ts +++ b/apps/api/src/controllers/v2/extract-status.ts @@ -116,7 +116,13 @@ export async function extractStatusController( success: status === "failed" ? false : true, data, status, - error: typeof extract?.error === "string" ? extract.error : undefined, + error: (() => { + if (typeof extract?.error === "string") return extract.error; + if (extract?.error && typeof extract.error === "object") { + return extract.error.message || extract.error.error || JSON.stringify(extract.error); + } + return undefined; + })(), expiresAt: (await getExtractExpiry(req.params.jobId)).toISOString(), steps: extract?.showSteps ? extract.steps : undefined, llmUsage: extract?.showLLMUsage ? extract.llmUsage : undefined, From bd9b46979d27e4ae5fb401e060d9aa82c9ec7f3d Mon Sep 17 00:00:00 2001 From: Gaurav Chadha <65453826+Chadha93@users.noreply.github.com> Date: Thu, 27 Nov 2025 15:27:14 +0530 Subject: [PATCH 3/3] Update apps/api/src/controllers/v1/extract-status.ts Co-authored-by: cubic-dev-ai[bot] <191113872+cubic-dev-ai[bot]@users.noreply.github.com> --- apps/api/src/controllers/v1/extract-status.ts | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/apps/api/src/controllers/v1/extract-status.ts b/apps/api/src/controllers/v1/extract-status.ts index c232d0f1c..95f33446c 100644 --- a/apps/api/src/controllers/v1/extract-status.ts +++ b/apps/api/src/controllers/v1/extract-status.ts @@ -119,7 +119,7 @@ export async function extractStatusController( error: (() => { if (typeof extract?.error === "string") return extract.error; if (extract?.error && typeof extract.error === "object") { - return extract.error.message || extract.error.error || JSON.stringify(extract.error); + return typeof extract.error.message === "string" ? extract.error.message : (typeof extract.error.error === "string" ? extract.error.error : JSON.stringify(extract.error)); } return undefined; })(),