Uses JS SDK from pocketbase/js-sdk

[rodrigorm]

@benallfree I was looking at dop251/goja and found out that it supports async/await syntax, what did you think about using the JS SDK from pocketbase/js-sdk instead of benallfree/pocketbase-js-sdk? Did you tried it before and not worked?

[benallfree]

Easy enough to test, just try importing the regular SDK in a hook and see what happens. Please report back :)

[rodrigorm]

Tested it and found out this will be an hard task, as all of the PocketPages core need to be rewritten to support async/away starting by the EJS render.

Easy enough to test, hard enough to implement. ;)

[benallfree]

I'm not sure I understand. Do you mean the regular pocketbase SDK works without changes in goja?

[rodrigorm]

As goja supports async/await and Promises, I think it is possible to uses pocketbase SDK, the only change you need to do is to provide an fetch function, and It can be done without forking:

const pb = new PocketBase('http://127.0.0.1:8090');

pb.beforeSend = function (url, options) {
    options.fetch = async (url, options) => {
       // Uses $http.send here
       ...
    };

    return { url, options };
};

[rodrigorm]

I was trying to do directly at the 'pocketpages-plugin-js-sdk' but I did not get it to work because I found we need to rewrite all the EJS and PocketPages plugins to support async/await too.

I will check using directly in and route hook and give you a feedback.

[benallfree]

That doesn’t make sense to me. Why would we need to rewrite?

[rodrigorm]

I will use the starter/auth as an examples:

At pb_hooks/pages/auth/login/_private/password.ejs there is this code:

const methods = pb().collection('users').listAuthMethods();

If the is changed to an async version it will need to be rewritten too:

const methods = async pb().collection('users').listAuthMethods();

So just by this change the pocketpages-plugin-ejs need to have support to async/await and it will cascade to the PocketPages hooks and so on.

[benallfree]

But forget pocket pages for the moment. Just create a fresh pb project with a hook that loads and uses the standard sdk. See if it works :)

[rodrigorm]

Ok, did not work at all, It is not possible to register and async routing route like:

routerAdd('GET', '/hello/{name}', async (e) => {
});

Logs this:

WARN the handler must a non-async function and not return a Promise

[VictorioBerra]

So goja supports it but PocketBase does not allow yet?

…

On Fri, Aug 15, 2025 at 10:35 AM Rodrigo Moyle ***@***.***> wrote: Ok, did not work at all, It is not possible to register and async routing route like: routerAdd('GET', '/hello/{name}', async (e) => {}); Logs this: WARN the handler must a non-async function and not return a Promise — Reply to this email directly, view it on GitHub <#88 (reply in thread)>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/AAWMN26BI6JJH7BKPU7JD6D3NX42NAVCNFSM6AAAAACD5ESYBWVHI2DSMVQWIX3LMV43URDJONRXK43TNFXW4Q3PNVWWK3TUHMYTIMJRHAZTMOA> . You are receiving this because you are subscribed to this thread.Message ID: ***@***.*** com>

-- *Victorio Berra* ***@***.*** 636-328-7110