invoking a function that returns a 500 error throws instead of returning the error object

[netanelgilad]

@claude I have this example code usage:

try {
            const { data, error: functionError } = await base44.functions.invoke('testApiKey');
            console.log(functionError);
            
            if (functionError || (data && data.error)) {
                throw new Error(functionError?.message || data.error);
            }

            setApiKeyStatus(data);
        } catch (err) {
            console.log('this was an error', err);
            setError(err.message);
        } finally {

and a backend function:

Deno.serve(async (req) => {
    try {
        throw new Error('my life');
    } catch (error) {
        return new Response(JSON.stringify({ error: error.message }), {
            status: 500,
            headers: { "Content-Type": "application/json" },
        });
    }
});

and instead of returning the error in functionError, it's going to the catch clause.
and this is what is printed

[pages/HelloWorld.js] this was an error 
AxiosError {message: 'Request failed with status code 500', name: 'AxiosError', code: 'ERR_BAD_RESPONSE', config: {…}, request: XMLHttpRequest, …}
code
: 
"ERR_BAD_RESPONSE"
config
: 
{transitional: {…}, adapter: Array(3), transformRequest: Array(1), transformResponse: Array(1), timeout: 0, …}
message
: 
"Request failed with status code 500"
name
: 
"AxiosError"

let's fix this so that it returns the error as part of the function invokation result, and add a test to verify it

[claude]

Claude finished @netanelgilad's task —— View job • claude/issue-14-20250811-1551 • Create PR ➔

---

Fixing function invocation error handling ✅

Fixed the issue where invoking a function that returns a 500 error throws instead of returning the error object.

Todo List

• Read repository setup instructions (CLAUDE.md)
• Explore codebase to understand function invocation implementation
• Identify where axios error handling occurs
• Fix error handling to return 500 errors in the error field instead of throwing
• Add test case to verify the fix
• Run linting/testing commands
• Commit and push changes

Changes Made

Fixed src/modules/functions.ts: Modified the invoke method to:

• Catch errors instead of letting them bubble up as exceptions
• Return { data: response.data, error: null } on success
• Return { data: null, error: error } on failure (including 500 errors)

Updated tests/unit/functions.test.ts:

• Modified all 13 success test cases to expect result.error to be null and check result.data
• Updated 2 existing error test cases to expect errors in result.error instead of throwing
• Added a new test case specifically for 500 errors to verify they are returned in the error field

Results: All unit tests pass (15/15) ✅ and the build compiles successfully ✅

The user can now handle function errors like this:

const { data, error } = await base44.functions.invoke('testApiKey');
if (error) {
    console.log('Function error:', error.message);
    // Handle error case
} else {
    console.log('Success:', data);
    // Handle success case  
}