Next.js build fails due to this regex

[lightyaer]

Next.js build fails when you upgrade to 14.0.3+ versions.

Upon searching I found this issue in next.js repo - https://github.com/vercel/next.js/issues/59540 which points that a regex change which fixes the build issue.

In the latest version of libsql-client-ts 0.4.0-pre.10, the fix has been implemented, but,

• some weirdness with string literals and regex parsing OR
• how next.js build process(rust) parses this lib OR
• voodoo magic

Fails the build with a Collecting Page Data error.

Using a string concatenation solves this.

Also, tagging another issue from this repo

• https://github.com/tursodatabase/libsql-client-ts/issues/142

Fixes #142 Fixes vercel/next.js#59540

[penberg]

@giovannibenussi @notrab please review

[giovannibenussi]

The code in general looks good but has a few issues that are reflected in the tests. For example, we got the error "URL_PARAM_NOT_SUPPORTED: Unknown URL query parameter \"1\"" in the test below, which means that we're doing something wrong in the parsing. I'll leave further comments in the code if I find something that could be causing those issues and take another look once they're fixed!

test("URL scheme incompatible with ?tls", () => {
    const urls = [
        "ws://localhost?tls=1",
        "wss://localhost?tls=0",
        "http://localhost?tls=1",
        "https://localhost?tls=0",
    ];
    for (const url of urls) {
        expect(() => createClient({url}))
            .toThrow(expect.toBeLibsqlError("URL_INVALID", /TLS/));
    }
});

[penberg]

Fixed by https://github.com/tursodatabase/libsql-client-ts/pull/172 so closing this PR.