T09 · Insecure Skill Coding Practices
- Location
SKILL.md:119- Finding
Ambiguous comma parsing can silently corrupt clinical measurements
- Content
View full analysis
- Remediation
View remediation
Security audit
Security checks for vulnerabilities and agentic risk
This skill is not malware, but it should be reviewed because its lab-cleaning instructions can silently drop or alter clinical measurements.
Review this skill carefully before using it on real patient or research data. It should require explicit source units or trusted source mappings, preserve original values, log every conversion or dropped record, keep full-precision numeric canonical data, and treat range checks as validation warnings rather than automatic conversion or clamping rules.
SKILL.md:119Ambiguous comma parsing can silently corrupt clinical measurements
SKILL.md:140Range-based unit inference can silently apply an incorrect conversion
SKILL.md:202Uniform two-decimal formatting causes clinically significant precision loss
The skill instructs users to drop any row containing missing numeric values before harmonization, which is a destructive data-processing step not required for unit normalization. In healthcare analytics, this can silently remove large portions of patient data, bias cohorts, distort downstream models or clinical summaries, and potentially exclude vulnerable patients with partial lab panels.
The documentation first states that out-of-range values should be returned unchanged and explicitly says not to clamp, but later permits tolerance-based clamping near boundaries. This inconsistency can lead implementers to silently alter clinical measurements at decision thresholds, masking true abnormalities and producing non-reproducible transformations across systems.