feat(cli): rename diagnose to doctor (keep deprecated alias until 0.3.0)#81
Merged
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Renames the environment health-check command from
diagnosetodoctor, following the established CLI convention (flutter doctor,brew doctor,expo doctor).diagnosestays as a deprecated alias, so this is non-breaking and can ship in 0.2.x; the alias is scheduled for removal at 0.3.0 (recorded in the PRD).Changes
doctor;diagnosekept as an alias (command.Aliases).--helpshowsdiagnose, doctor.DiagnoseQuery/DiagnoseQueryHandler/DiagnoseError/DiagnoseCommand→Doctor*(files renamed viagit mv).DiagnosticCheck/DiagnosticStatusare unchanged — the doctor runs diagnostic checks.Tests (TDD)
New
DoctorCommandTests(written Red-first, 3 cases): command is nameddoctor;diagnoseis registered as an alias; parsingdiagnoseresolves to the doctor command. Full suite green (692 tests).Verification
ivicli doctorandivicli diagnoseboth run the health checks on the real binary.diagnosereferences in code or docs except the deprecation notes and an unrelated English verb in ADR 0017.