ci: fix better-sqlite3 version

[l2ysho]

No description provided.

[barjin]

It seems that more of the E2E tests have failed (below is grep "fatal" of the failing CI job).

Fwiw all of them seem to have issues with the better-sqlite installation

[l2ysho]

It seems that more of the E2E tests have failed (below is grep "fatal" of the failing CI job).

Fwiw all of them seem to have issues with the `better-sqlite` installation

Lets wait for https://github.com/apify/crawlee/actions/runs/24891314970

[l2ysho]

@patrikbraborec @DaveHanns tagging you so you are in the loop. This PR resolve (I hope) e2e tests problem in crawlee where we use CLI beta. https://github.com/apify/crawlee/actions/workflows/test-e2e.yml

[B4nan]

all e2e tests are using the local storage, i dont think this is enough to fix the e2e tests

[l2ysho]

all e2e tests are using the local storage, i dont think this is enough to fix the e2e tests

yop, it failed right now.

[vladfrangu]

I feel like we should have fixed it in the module but sure if it works

[l2ysho]

I feel like we should have fixed it in the module but sure if it works

seems https://github.com/apify/apify-storage-local-js was updated ~5 month ago, we just missing release. Locally it is working with "@apify/storage-local": "^2.3.1-beta.1"

I will try it in this PR with this version just to check e2e tests before CLI release.

[B4nan]

Yeah, we bumped it some time ago, but I am not sure if we want to ship a new version, this is breaking, so it would have to be a major bump, and I would prefer to reserve that for the v4 rewrite (especially since nobody asked for this).

edit: on the other hand, its on v2, so if we ship v3 with the deps update, we would align the next major with crawlee...

[l2ysho]

Yeah, I bumped it some time ago, but I am not sure if we want to ship a new version, this is breaking, so it would have to be a major bump, and I would prefer to reserve that for the v4 rewrite (especially since nobody asked for this).

Well yes, but now @apify/storage-local works only with node@20 which is not maintained

[B4nan]

Hmm, good point, we can ship that, especially given what I said above.

But let's bump other deps too, not just this one. I am sure there will be more things with a major bump.

[l2ysho]

I am sure there will be more things with a major bump.

There are some, I can do it.