fix(nanno): guard against mismatched array lengths in extractFieldsFromRecord

[Copilot]

extractFieldsFromRecord iterated over fieldNames.length while indexing into fieldPositions[i], causing ArrayIndexOutOfBoundsException when the two arrays had different lengths.

Changes

• AnnotationSourceTSV.java: Change loop bound from fieldNames.length to Math.min(fieldNames.length, fieldPositions.length), safely processing only the overlapping entries.

// Before
for (int i = 0; i < fieldNames.length; i++) {

// After
for (int i = 0; i < Math.min(fieldNames.length, fieldPositions.length); i++) {

• AnnotationSourceTSVTest.java: Add test cases for both mismatch directions — fieldNames longer than fieldPositions and vice versa.

Type of change

• Bug fix (non-breaking change which fixes an issue)

How Has This Been Tested?

• Unit tests added in AnnotationSourceTSVTest.extractFieldsFromRecord() covering both mismatch directions (more names than positions, more positions than names).

Are WDL Updates Required?

No.

Checklist:

• My code follows the style guidelines of this project
• I have performed a self-review of my own code
• I have commented my code, particularly in hard-to-understand areas
• I have made corresponding changes to the documentation
• My changes generate no new warnings
• I have added tests that prove my fix is effective or that my feature works
• New and existing unit tests pass locally with my changes

---

💡 You can make Copilot smarter by setting up custom instructions, customizing its development environment and configuring Model Context Protocol (MCP) servers. Learn more Copilot coding agent tips in the docs.