-
Notifications
You must be signed in to change notification settings - Fork 3
[Feature]: ValidateCommand name misleading — only validates version, not schema compliance #211
Copy link
Copy link
Open
Labels
designArchitecture/API design discussion or decision needed before implementationArchitecture/API design discussion or decision needed before implementationdocumentationImprovements or additions to documentationImprovements or additions to documentation
Milestone
Description
AI REVIEWED
Module: cli
File: cli/command/ValidateCommand.java
Severity: Low
Problem / Motivation
The validate command only checks version numbers in data files, not whether the data actually conforms to the schema at that version. The name "validate" suggests schema compliance checking.
Proposed Solution
Either:
- Rename to
check-versionor add a--version-onlyflag - Implement actual schema validation against registered types
- Update documentation to clearly state: "Validates that data files contain version information and that the version is within the expected range. Does NOT validate schema compliance."
Reactions are currently unavailable
Metadata
Metadata
Assignees
Labels
designArchitecture/API design discussion or decision needed before implementationArchitecture/API design discussion or decision needed before implementationdocumentationImprovements or additions to documentationImprovements or additions to documentation