Refactor SAV module to follow MultiQC best practices

[ewels]

Summary

• Refactors the SAV module to follow MultiQC module development best practices
• Adds missing required components (add_data_source, add_software_version, write_data_file, ModuleNoSamplesFound)
• Adds general statistics columns for key metrics
• Improves code quality with semantic color scales and defensive checks

Test plan

• Verify multiqc --strict passes on MiSeq test data
• Verify multiqc --strict passes on NovaSeq6000 test data
• Verify ruff check passes
• Verify pre-commit run --all-files passes

Detailed changes

Critical fixes

• Add ModuleNoSamplesFound exception when no data found
• Add add_data_source() and add_software_version() calls
• Add write_data_file() call at end of module init
• Export SAVModule class from __init__.py

High priority fixes

• Change anchor from "SAV" to "sav" (lowercase per convention)
• Add href to Illumina InterOp GitHub repository
• Improve info string (remove leading -)

Medium priority improvements

• Add comprehensive class docstring documenting supported sequencers
• Add scale attributes to all HEADERS entries for semantic coloring
• Add general stats columns (% >= Q30, Yield, % PF, Error Rate)

Low priority improvements

• Support multiple sequencing runs (removed single-run limitation)
• Add defensive checks with .get() defaults in _clusters_lane_plot()
• Update function type hints to use SAVModule type

🤖 Generated with Claude Code

[ewels]

See example reports from the actions CI run: https://github.com/MultiQC/MultiQC_SAV/actions/runs/20332832779/artifacts/4909854845

Looks pretty good I think! I haven't taken a detailed look at the code changes, but they look sane.

What do you think @matthdsm ?

[matthdsm]

Looking good! I see some missing features that were previously there, but we can fix that in another PR.

[matthdsm]

Thanks for making quick work of this! ❤️