Add default implementation of iterable write on GenericWriter

[RobFaustLZ]

No description provided.

[codecov]

Codecov Report

❌ Patch coverage is 0% with 4 lines in your changes missing coverage. Please review.
✅ Project coverage is 57.92%. Comparing base (37216dd) to head (49553da).

Files with missing lines	Patch %	Lines
...n/java/com/labelzoom/api/output/GenericWriter.java	0.00%	4 Missing ⚠️

Additional details and impacted files

@@             Coverage Diff              @@
##               main      #44      +/-   ##
============================================
- Coverage     58.17%   57.92%   -0.25%     
  Complexity      207      207              
============================================
  Files            49       50       +1     
  Lines           930      934       +4     
  Branches        105      105              
============================================
  Hits            541      541              
- Misses          376      380       +4     
  Partials         13       13              

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

[augmentcode]

🤖 Augment PR Summary

Summary: Adds a bulk-write convenience method to GenericWriter and clarifies reader type-parameter naming to better convey input vs output.


Changes: Introduces GenericWriter#writeAll(Iterable<I>) as a default method delegating to write, and renames GenericReader/LabelReader generic parameters for readability.

_{🤖 Was this summary useful? React with 👍 or 👎}

[augmentcode]

Review completed. 1 suggestions posted.

Comment augment review to trigger a new review at any time.

[github-actions]

Test Results

17 tests  ±0   17 ✅ ±0   0s ⏱️ -1s
 4 suites ±0    0 💤 ±0 
 4 files   ±0    0 ❌ ±0 

Results for commit 8cff691. ± Comparison against base commit d9e84f2.

♻️ This comment has been updated with latest results.

[RobFaustLZ]

augment review

[augmentcode]

Review completed. 3 suggestions posted.

Comment augment review to trigger a new review at any time.

[augmentcode]

writeAll(Iterable<I> items) is invariant in I, so a GenericWriter<Number> can’t accept a List<Integer> here even though write(Integer) would be valid; consider whether Iterable<? extends I> is intended for this API.

_{🤖 Was this useful? React with 👍 or 👎}

[augmentcode]

writeAll will throw a NullPointerException if items is null; since this is a public default API method, it may be worth documenting the non-null contract (or otherwise handling it) to avoid surprising callers.

_{🤖 Was this useful? React with 👍 or 👎}

[augmentcode]

writeAll introduces new behavior but currently has no direct tests around it; adding a small unit test that asserts write(...) is invoked once per element (and in order) would help prevent regressions.

_{🤖 Was this useful? React with 👍 or 👎}