Add separate build script

[Timshel]

Hey

A change to move part of the build process out of the docker file in an external script.

As for why:

• It can help with testing by allowing to run locally, but this is limited by the rather strict node version checking of npm ci.
• It's mainly for the Playwright PR in Vaultwarden to be able to build aany commit without having to work with the bw_web_builds/Dockerfile.

Additionally, removed the chown node logic since I had no issues when running the build.

[BlackDex]

What is wrong with all the other scripts already there?

The same can be done with:

# Using make (separate commands)
VAULT_VERSION=v2024.5.1 make checkout
VAULT_VERSION=v2024.5.1 make patch-web-vault
VAULT_VERSION=v2024.5.1 make build

# Using make, just one command
VAULT_VERSION=v2024.5.1 make full

# Or using the script directly
VAULT_VERSION=v2024.5.1 ./scripts/checkout_web_vault.sh
VAULT_VERSION=v2024.5.1 ./scripts/patch_web_vault.sh 
VAULT_VERSION=v2024.5.1 ./scripts/build_web_vault.sh 

[Timshel]

Sorry was working on the Playwright PR and did not take time to reread the bw_web_builds docs :(.
And since the build_web_vault.sh was not used in the Dockerfile did not realise it would work.

[BlackDex]

Ah, no prob. Glad this can be solved with the current script already :).
It has been Wednesday already ;).

[Timshel]

Are you aware of any reason why the checkout_web_vault and build_web_vault are not used in the Docker image ?

They are more verbose and might make more check but since the intermediate image is discarded anyway should not have much impact ?

[BlackDex]

Um, there was a reason, not sure from the top of my head right now.
I would have to take a better look, but there was a good reason i think (if not, then we could change that too of course)