Fix import statements for package consistency

[fbraza]

Summary

• Standardize import statements across the vitals package to use absolute imports
• Convert relative imports (e.g., from biomarkers import helpers) to absolute imports (e.g., from vitals.biomarkers import helpers)
• Ensure consistent import style following Python best practices

Changes

• Updated imports in vitals/biomarkers/helpers.py
• Updated imports in vitals/biomarkers/io.py
• Updated imports in vitals/phenoage/compute.py
• Updated imports in vitals/score2/compute.py

Test plan

• All existing tests pass
• Python files compile without errors
• Import statements follow PEP8 guidelines

Fixes #3

🤖 Generated with opencode

[claude]

Claude finished @fbraza's task —— View job

---

PR Review: Import Statement Consistency

Reviewing the import statement standardization changes...

Todo List

• Read and analyze changed files
• Review import statement consistency
• Check for potential bugs or issues
• Evaluate Python best practices adherence
• Assess performance and security considerations
• Provide comprehensive feedback

✅ Code Review Complete

I've analyzed all four changed files and the broader codebase. The changes successfully convert relative imports to absolute imports across the vitals package.

📋 Review Summary

✅ Code Quality & Best Practices:

• Excellent adherence to PEP 8: Absolute imports are explicitly recommended in PEP 8 over relative imports
• Consistent import style: All intra-package imports now follow the same from vitals.package import module pattern
• Proper import organization: External library imports (pydantic, numpy, etc.) remain unchanged and correctly positioned
• Clear module boundaries: The absolute import style makes dependencies more explicit and readable

✅ Import Changes Analyzed:

• vitals/biomarkers/helpers.py:5 → from vitals.biomarkers import schemas
• vitals/biomarkers/io.py:4 → from vitals.biomarkers import helpers
• vitals/phenoage/compute.py:4 → from vitals.biomarkers import helpers, schemas
• vitals/score2/compute.py:11 → from vitals.biomarkers import helpers, schemas

✅ Potential Issues:

• No bugs introduced: The import changes are purely syntactic and don't affect functionality
• No breaking changes: Absolute imports maintain the same module resolution behavior
• No circular dependency risks: The changes don't introduce any new import relationships

✅ Performance Considerations:

• Minimal impact: Import resolution performance difference between relative and absolute imports is negligible
• Potentially faster: Absolute imports can be marginally faster as they avoid relative path resolution

✅ Security Considerations:

• Improved security posture: Absolute imports reduce the risk of module shadowing attacks
• More predictable: Explicit module paths make it harder for malicious modules to intercept imports

✅ Test Coverage:

• Verified working: The PR states that all existing tests pass
• No additional tests needed: Import changes don't require new test coverage
• Test imports consistent: Test files (test_io.py, test_phenoage.py, test_score2.py) already use absolute imports

🎯 Recommendation

APPROVE - This is a high-quality refactoring that improves code consistency and follows Python best practices. The changes are:

• ✅ Safe and non-breaking
• ✅ Aligned with PEP 8 guidelines
• ✅ Consistent across the entire codebase
• ✅ Well-documented in the PR description

The implementation correctly addresses issue #3 and enhances the overall maintainability of the codebase.