Skip to content

Commit 24fd735

Browse files
committed
fix(latex): surface extracted TeX errors on all failed compile responses
1 parent 0ade161 commit 24fd735

1 file changed

Lines changed: 13 additions & 5 deletions

File tree

apps/sim/app/api/tools/latex/route.ts

Lines changed: 13 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -162,12 +162,20 @@ async function buildCompileErrorResponse(
162162
: undefined
163163
const errorCode = typeof errorRecord?.error === 'string' ? errorRecord.error : undefined
164164
const compilationErrors = extractCompilationErrors(errorRecord?.log_files)
165+
const details = compilationErrors ? `:\n${compilationErrors}` : ''
165166

166-
if (upstreamResponse.status === 400 && errorCode) {
167-
logger.warn(`[${requestId}] LaTeX compilation failed`, { errorCode })
168-
const details = compilationErrors ? `:\n${compilationErrors}` : ''
167+
const isCompilationFailure =
168+
upstreamResponse.status >= 400 &&
169+
upstreamResponse.status < 500 &&
170+
Boolean(errorCode || compilationErrors)
171+
172+
if (isCompilationFailure) {
173+
logger.warn(`[${requestId}] LaTeX compilation failed`, {
174+
status: upstreamResponse.status,
175+
errorCode,
176+
})
169177
return NextResponse.json(
170-
{ error: `LaTeX compilation failed (${errorCode})${details}` },
178+
{ error: `LaTeX compilation failed (${errorCode || upstreamResponse.status})${details}` },
171179
{ status: 422 }
172180
)
173181
}
@@ -177,7 +185,7 @@ async function buildCompileErrorResponse(
177185
errorCode,
178186
})
179187
return NextResponse.json(
180-
{ error: `LaTeX compile service error: ${upstreamResponse.status}` },
188+
{ error: `LaTeX compile service error: ${upstreamResponse.status}${details}` },
181189
{ status: 502 }
182190
)
183191
}

0 commit comments

Comments
 (0)