Remove redundant doctor command in favor of apply --dry-run

[Copilot]

doctor was effectively a dedicated wrapper around dry-run behavior. This change removes that duplicate command surface and makes apply --dry-run the single diagnostic path.

• CLI surface

  • removes the doctor subcommand from the root command
  • keeps dry-run diagnostics on apply, with --verbose as the richer human-readable mode

• Engine cleanup

  • removes DoctorOptions, DoctorReport, and Engine.Doctor()
  • relies on existing Apply(..., DryRun: true) metadata/output instead of maintaining a parallel execution path

• Docs

  • removes the doctor section from the README
  • updates apply flag docs to point users to --dry-run --verbose for diagnostics

• Tests

  • replaces doctor-specific CLI and engine assertions with equivalent apply --dry-run coverage
  • keeps compatibility-hint and include-resolution checks on the dry-run path

Example:

# before
git-worktreeinclude doctor --from auto --include .worktreeinclude

# after
git-worktreeinclude apply --dry-run --verbose --from auto --include .worktreeinclude

[Copilot]

Pull request overview

Removes the redundant doctor CLI surface and consolidates diagnostics into the existing apply --dry-run --verbose path, simplifying both CLI and engine APIs.

Changes:

• Removed doctor subcommand from the CLI and documentation; positioned apply --dry-run --verbose as the diagnostic workflow.
• Removed Engine.Doctor() and associated types, relying on Apply(..., DryRun: true) metadata instead.
• Updated integration/unit tests to validate the dry-run diagnostic path instead of doctor.

Reviewed changes

Copilot reviewed 6 out of 6 changed files in this pull request and generated 1 comment.

Show a summary per file

File	Description
README.md	Removes doctor docs and points diagnostics users to apply --dry-run --verbose.
internal/engine/engine.go	Deletes DoctorOptions/DoctorReport and Engine.Doctor(), consolidating on Apply.
internal/engine/engine_integration_test.go	Replaces Doctor() assertions with Apply(..., DryRun: true) coverage for metadata/hints.
internal/cli/cli.go	Removes doctor command wiring/implementation and codedOrDefault helper.
internal/cli/cli_unit_test.go	Drops unit test for the removed codedOrDefault helper.
internal/cli/cli_integration_test.go	Replaces doctor CLI tests with apply --dry-run --verbose equivalents.

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.