SK-3037 give the bulk tokenize and delete interfaces their own options types

[saileshwar-skyflow]

bulkTokenize and bulkDeleteTokens took TokenizeOptions and DeleteTokensOptions directly. They now take BulkTokenizeOptions and BulkDeleteTokensOptions, which extend those — the same split the requests already use, where BulkTokenizeRequest extends TokenizeRequest.

Signatures

bulkDeleteTokens(BulkDeleteTokensRequest, BulkDeleteTokensOptions)
bulkDeleteTokensAsync(BulkDeleteTokensRequest, BulkDeleteTokensOptions)
bulkTokenize(BulkTokenizeRequest, BulkTokenizeOptions)
bulkTokenizeAsync(BulkTokenizeRequest, BulkTokenizeOptions)

Parents extended rather than renamed

DeleteTokensOptions and TokenizeOptions stay put, so the unary interfaces still have their options type when they ship. To make them subclassable: the class and its Builder drop final, the constructor becomes protected, and the Builder gains a protected no-arg constructor. Nothing else about them changes.

The bulk classes add no fields today — they exist so the bulk interfaces have their own type to grow into, matching the BulkInsertRecord / InsertRecord precedent.

Why the builder overrides are required

Each bulk builder overrides interceptor(...) and build() with covariant return types. This is not cosmetic. BulkTokenizeOptions.builder() hides the parent's static builder(), so without the overrides a chained call returns a TokenizeOptions and fails to assign:

BulkTokenizeOptions options = BulkTokenizeOptions.builder().interceptor(x).build();

Same static-hiding trap already handled in BulkTokenizeRequestBuilder.

Tests

8 added to OptionsTests, for each of the two new classes:

• interceptor set, and unset (null)
• instanceof the parent, with the inherited accessor returning the same interceptor
• a chain assigned to a bulk-typed builder variable, so a regression in the covariant overrides breaks compilation rather than passing quietly

The two VaultControllerTests interceptor tests move to the new types, keeping their once-per-batch assertions intact.

693 tests pass (common 166, flowvault 527); cspell reports no issues.

🤖 Generated with Claude Code