Skip to content

Commit e5fad7c

Browse files
author
Andrei Bratu
committed
Added catch blocks detailing the upsert error
1 parent 9277556 commit e5fad7c

1 file changed

Lines changed: 43 additions & 19 deletions

File tree

src/eval_utils/run.ts

Lines changed: 43 additions & 19 deletions
Original file line numberDiff line numberDiff line change
@@ -208,35 +208,59 @@ export async function runEval(
208208
let hlFile: FileResponse;
209209
switch (type) {
210210
case "flow": {
211-
// Be more lenient with Flow versions as they are arbitrary json
212-
if (version && !version.attributes) {
213-
version.attributes = version as Record<string, unknown>;
211+
try {
212+
// Be more lenient with Flow versions as they are arbitrary json
213+
if (version && !version.attributes) {
214+
version.attributes = version as Record<string, unknown>;
215+
}
216+
const updatedData = { ...rest, ...version } as FlowRequest;
217+
hlFile = await client.flows.upsert(updatedData);
218+
} catch (e) {
219+
throw new Error(
220+
`Error upserting the Flow associated with callable ${callable?.name}: ${e}`,
221+
);
214222
}
215-
const updatedData = { ...rest, ...version } as FlowRequest;
216-
hlFile = await client.flows.upsert(updatedData);
217223
break;
218224
}
219225
case "prompt": {
220-
hlFile = await client.prompts.upsert({
221-
...rest,
222-
...version,
223-
} as PromptRequest);
226+
try {
227+
hlFile = await client.prompts.upsert({
228+
...rest,
229+
...version,
230+
} as PromptRequest);
231+
} catch (e) {
232+
throw new Error(
233+
`Error upserting the Prompt associated with callable ${callable?.name}: ${e}`,
234+
);
235+
}
224236
break;
225237
}
226238
case "tool": {
227-
hlFile = await client.tools.upsert({
228-
...rest,
229-
...version,
230-
} as ToolRequest);
239+
try {
240+
hlFile = await client.tools.upsert({
241+
...rest,
242+
...version,
243+
} as ToolRequest);
244+
} catch (e) {
245+
throw new Error(
246+
`Error upserting the Tool associated with callable ${callable?.name}: ${e}`,
247+
);
248+
}
231249
break;
232250
}
233251
case "evaluator": {
234-
// @ts-ignore EvaluatorRequest is generated by Fern as 'unknown'
235-
// Leading to a type error here
236-
hlFile = await client.evaluators.upsert({
237-
...rest,
238-
...version,
239-
} as EvaluatorsRequest);
252+
try {
253+
// @ts-ignore EvaluatorRequest is generated by Fern as 'unknown'
254+
// Leading to a type error here
255+
hlFile = await client.evaluators.upsert({
256+
...rest,
257+
...version,
258+
} as EvaluatorsRequest);
259+
} catch (e) {
260+
throw new Error(
261+
`Error upserting the Evaluator associated with callable ${callable?.name}: ${e}`,
262+
);
263+
}
240264
break;
241265
}
242266
default:

0 commit comments

Comments
 (0)