re implements vector and changes zip

[Kashargul]

About the PR

Re-implements the vector functions, using the built ins and moving the zip function to mapping to have it consistent.

Best to look over the code please.

We will now always fill vectors up with undefined if any is longer than the other. The image below was the common test case beforehand, which now also has the undefined and fill up to have it behave the same no matter which vector is the longer one.

Swapping the test case to have the larger vector as the first vector on original code:

Why's this needed?

We've multiple UIs relying on zip and vector, so we should be able to import them from tgui core

[tgstation-ci]

Thank you for contributing to tgui-core! The workflow 'Chromatic Security' requires repository secrets and will not run without approval. Maintainers can add the CI Cleared label to allow it to run. Note that any changes to chromaitc-security.yml and chromatic.yml will not be reflected.

[jlsnow301]

Wondering if we should add tests for these in the repo. Node has built in testing now, maybe we could make use of it?

[Kashargul]

Wondering if we should add tests for these in the repo. Node has built in testing now, maybe we could make use of it?

actually not a bad idea.