Add JavaDocs to PixiOpts and update tests for new default pixi image

[pditommaso]

Summary

• Add comprehensive JavaDoc documentation to PixiOpts.java including class-level description and documentation for all fields, constants, and constructors
• Update test assertions in PixiHelperTest, ContainerHelperTest, and TemplateUtilsTest to use the new default pixi image (public.cr.seqera.io/wave/pixi:0.61.0-noble)

Test plan

• All pixi-related tests pass with ./gradlew :test --tests 'io.seqera.wave.util.PixiHelperTest' --tests 'io.seqera.wave.util.TemplateUtilsTest.*Pixi*' --tests 'io.seqera.wave.util.ContainerHelperTest.*pixi*'

🤖 Generated with Claude Code

[pditommaso]

@claude review

[claude]

Claude finished @pditommaso's task —— View job

---

PR Review Complete ✅

• Read and analyze PixiOpts.java JavaDoc additions
• Review test file updates for new pixi image
• Check for code quality and documentation standards
• Provide comprehensive review feedback

Summary

This PR successfully adds comprehensive JavaDoc documentation and updates tests for the new default pixi image. The changes are well-implemented and consistent.

JavaDoc Review (PixiOpts.java)

Strengths:

• ✅ Excellent class-level documentation explaining Pixi's purpose and the class role
• ✅ All public constants properly documented with clear descriptions
• ✅ Field documentation is concise and descriptive
• ✅ Constructor documentation includes parameter details with supported keys
• ✅ Follows standard JavaDoc conventions and formatting
• ✅ Author annotation included

Minor suggestions:

• Consider adding @since tag to indicate when this class was introduced
• The constructor parameter documentation could specify the expected types for better type safety guidance

Test Updates Review

All test assertions correctly updated to use the new default pixi image public.cr.seqera.io/wave/pixi:0.61.0-noble:

Files updated:

• ✅ PixiHelperTest.groovy (lines 44, 65)
• ✅ ContainerHelperTest.groovy (line 630)
• ✅ TemplateUtilsTest.groovy (lines 355, 386, 420, 650)

Validation:

• ✅ All image references are consistent across test files
• ✅ No hardcoded references to old image versions remain
• ✅ Tests maintain existing functionality while using updated image

Code Quality

• ✅ No security, performance, or maintainability issues identified
• ✅ Changes are focused and don't introduce unnecessary modifications
• ✅ Consistent with existing codebase patterns and style
• ✅ The constant DEFAULT_PIXI_IMAGE centralizes the image version, making future updates easier

Recommendation

APPROVE - This PR is ready to merge. The JavaDoc additions significantly improve code documentation, and the test updates are thorough and consistent. All changes align with best practices and maintain backward compatibility.