Add xSV validation module

[mattldawson]

Description of PR purpose/changes

This PR adds a validation option for character-separated value files. It uses the xsv-validate.sh shell script that was included in the container build scripts with PR #203

Minor Update

• Modified the Dockerfile to install the qsv and xml_file_splitter packages for the hardware the image is built for, and to add a missing qsv dependency.

List any dependencies that are required for this change.

xml-validate.sh and qsv (already present in Docker build)

Testing Instructions

Build and run containerized tests as usual.

• Tests pass locally and in GitHub Actions

Dev Checklist:

• My submission follows the AI Covenant principles
• 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
• I have run Ruff format to format my code
• I have run Ruff check and fixed any errors that it uncovered

[Copilot]

Pull request overview

Adds an xSV (CSV/TSV/etc.) validation adapter to the cdm_data_loaders.validation package that shells out to xsv-validate.sh (qsv-backed), along with unit + e2e-style tests and the necessary marker/workflow/documentation updates to gate tool-dependent tests.

Changes:

• Introduces cdm_data_loaders.validation.xsv.validate() plus Status/ValidationResults to run xsv-validate.sh and parse optional summary output.
• Adds extensive tests (mocked subprocess + requires_xsv integration-style coverage) and registers the requires_xsv pytest marker.
• Updates docs, CI test selection, and Docker image dependencies/binary installation to support running the validator.

Reviewed changes

Copilot reviewed 6 out of 6 changed files in this pull request and generated 3 comments.

Show a summary per file

File	Description
src/cdm_data_loaders/validation/xsv.py	New validation adapter that invokes xsv-validate.sh and returns structured results (optionally from summary JSON).
tests/validation/test_xsv.py	New unit + tool-gated integration-style tests for argument building, status mapping, and end-to-end behavior.
pyproject.toml	Registers requires_xsv marker for pytest.
README.md	Documents how to exclude requires_xsv tests when the tool isn’t installed.
Dockerfile	Installs arch-appropriate qsv/xml_file_splitter and adds a missing runtime dependency.
.github/workflows/tests.yml	Excludes requires_xsv tests from the “local tests” job filter.

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

[codecov]

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 69.15%. Comparing base (c01480f) to head (7e9a020).
⚠️ Report is 3 commits behind head on develop.

Additional details and impacted files

@@             Coverage Diff             @@
##           develop     #205      +/-   ##
===========================================
+ Coverage    68.57%   69.15%   +0.58%     
===========================================
  Files           73       75       +2     
  Lines         4462     4546      +84     
===========================================
+ Hits          3060     3144      +84     
  Misses        1402     1402              

Files with missing lines	Coverage Δ
src/cdm_data_loaders/validation/xsv.py	100.00% <100.00%> (ø)

---

Continue to review full report in Codecov by Harness.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update ae0a5ce...7e9a020. Read the comment docs.

🚀 New features to boost your workflow:

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

[ialarmedalien]

this is great, thanks!

[ialarmedalien]

All of the validation here immediately makes me think it might be easier to use a pydantic object to capture the fields so you get the validation "for free" when the object is instantiated. A pydantic object can be dumped as json or a dictionary, so depending on how the object is set up, it might be possible to have it output the args for xiv-validate in a form that will require minimal wrangling to convert to turn into args.

[mattldawson]

Great suggestion! I took a stab at this and was able to remove all the validation code from the function, and I think it's a lot cleaner this way. I wasn't able to figure out an equally clean way to generate the xsv-validate args, though. Anything I considered just ended up needing a lot more code. Let me know what you think. I'm happy to make more revisions if needed.

[ialarmedalien]

I think it looks fine as-is, so not worth spending the extra time and effort on.

[ialarmedalien]

Looks great!