Remove unused evaluateSchema CLI command #3613
Conversation
![devin-ai-integration[bot]](https://avatars.githubusercontent.com/in/811515?s=60&v=4){kind=small}
- Remove src/cli/evaluateSchema.ts CLI script that was redundant - Remove 'evaluateSchema' script entry from package.json - Preserve internal evaluateSchema function used by evaluateSchemaMulti and tests - All tests continue to pass, functionality remains intact The evaluateSchemaMulti command provides the same functionality and can handle both single and multiple dataset evaluations, making the standalone CLI command unnecessary. Co-Authored-By: noritaka.ikeda@route06.co.jp <noritaka.ikeda@route06.co.jp>
🤖 Devin AI EngineerI'll be helping with this pull request! Here's what you should know: ✅ I will automatically:
Note: I can only respond to comments from users who have write access to this repository. ⚙️ Control Options:
|
|
|
The latest updates on your projects. Learn more about Vercel for GitHub.
2 Skipped Deployments
|
|
Important Review skippedBot user detected. To trigger a single review, invoke the You can disable this status message by setting the Comment |
|
Updates to Preview Branch (devin/1758881170-remove-unused-evaluateSchema-cli) ↗︎
Tasks are run on every commit but only new migration files are pushed.
View logs for this Workflow Run ↗︎. |
Issue
Why is this change needed?
This change removes the redundant
evaluateSchemaCLI command from the schema-bench package. The functionality provided by this command is fully covered byevaluateSchemaMulti, which can handle both single and multiple dataset evaluations. This simplifies the codebase by removing duplicate functionality that was not being used in practice.What was changed?
src/cli/evaluateSchema.ts(44 lines)"evaluateSchema"script entryevaluateSchemafunction inevaluation.tsremains intact and continues to be used byevaluateSchemaMultiand testsMigration path
Instead of:
Use:
Human Review Checklist
evaluateSchemafunction inevaluation.tsis still accessible and working (tests pass but worth double-checking)evaluateSchemaMultican handle single dataset evaluation as intendedSession Info: Implemented by noritaka.ikeda@route06.co.jp
Link to Devin run: https://app.devin.ai/sessions/631dd75b8e6f4640986f39ad5990dbe2