MLE-12345 - Refactors vector utility functions

[BillFarber]

Refactors the vector encoding and decoding functions to be standalone functions instead of methods within a class.

This change improves code organization and potentially simplifies usage by removing the need to instantiate a utility class.

[Copilot]

Pull Request Overview

This PR refactors the vector encoding/decoding utilities by extracting static methods from the VectorUtil class into standalone functions and updating tests to call those functions directly.

• Removed the VectorUtil class and turned its methods into base64_encode/base64_decode top-level functions.
• Updated imports and calls in tests to use the new functions.
• Adjusted module structure and imports in marklogic/vectors.py.

Reviewed Changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.

File	Description
tests/test_vectors.py	Replaced VectorUtil.base64_encode/decode calls with direct imports of base64_encode and base64_decode.
marklogic/vectors.py	Removed VectorUtil class, added module-level docstring, and defined base64_encode and base64_decode as standalone functions.

Comments suppressed due to low confidence (4)

marklogic/vectors.py:1

• [nitpick] The module-level docstring was originally for the VectorUtil class; consider updating it to reflect that these are module-level functions rather than a class.

"""

marklogic/vectors.py:11

• Removing the VectorUtil class is a breaking change for consumers who used VectorUtil.base64_encode/decode; consider adding a deprecated VectorUtil alias or updating documentation to guide consumers through this change.

def base64_encode(vector: List[float]) -> str:

tests/test_vectors.py:22

• There aren't tests covering error paths such as invalid version or insufficient buffer length in base64_decode; consider adding tests for these error cases to improve coverage.

    decoded = base64_decode(EXPECTED_BASE64)

marklogic/vectors.py:19

• [nitpick] The variable name 'buffer' shadows the built-in buffer type; consider renaming it to 'data_buffer' or 'buf' for clarity.

    )