cli/pkg/doctor/cmd_checks.go
Snider 9931593f9d refactor(cli): move commands from cmd/ to pkg/ with self-registration
Implements defence in depth through build variants - only compiled code
exists in the binary. Commands now self-register via cli.RegisterCommands()
in their init() functions, mirroring the i18n.RegisterLocales() pattern.

Structure changes:
- cmd/{ai,build,ci,dev,docs,doctor,go,php,pkg,sdk,setup,test,vm}/ → pkg/*/cmd_*.go
- cmd/core_dev.go, cmd/core_ci.go → cmd/variants/{full,ci,php,minimal}.go
- Added pkg/cli/commands.go with RegisterCommands API
- Updated pkg/cli/runtime.go to attach registered commands

Build variants:
- go build           → full (21MB, all 13 command groups)
- go build -tags ci  → ci (18MB, build/ci/sdk/doctor)
- go build -tags php → php (14MB, php/doctor)
- go build -tags minimal → minimal (11MB, doctor only)

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
2026-01-30 21:55:55 +00:00

101 lines
2.6 KiB
Go

package doctor
import (
"os/exec"
"strings"
"github.com/host-uk/core/pkg/i18n"
)
// check represents a tool check configuration
type check struct {
name string
description string
command string
args []string
versionFlag string
}
// requiredChecks returns tools that must be installed
func requiredChecks() []check {
return []check{
{
name: i18n.T("cmd.doctor.check.git.name"),
description: i18n.T("cmd.doctor.check.git.description"),
command: "git",
args: []string{"--version"},
versionFlag: "--version",
},
{
name: i18n.T("cmd.doctor.check.gh.name"),
description: i18n.T("cmd.doctor.check.gh.description"),
command: "gh",
args: []string{"--version"},
versionFlag: "--version",
},
{
name: i18n.T("cmd.doctor.check.php.name"),
description: i18n.T("cmd.doctor.check.php.description"),
command: "php",
args: []string{"-v"},
versionFlag: "-v",
},
{
name: i18n.T("cmd.doctor.check.composer.name"),
description: i18n.T("cmd.doctor.check.composer.description"),
command: "composer",
args: []string{"--version"},
versionFlag: "--version",
},
{
name: i18n.T("cmd.doctor.check.node.name"),
description: i18n.T("cmd.doctor.check.node.description"),
command: "node",
args: []string{"--version"},
versionFlag: "--version",
},
}
}
// optionalChecks returns tools that are nice to have
func optionalChecks() []check {
return []check{
{
name: i18n.T("cmd.doctor.check.pnpm.name"),
description: i18n.T("cmd.doctor.check.pnpm.description"),
command: "pnpm",
args: []string{"--version"},
versionFlag: "--version",
},
{
name: i18n.T("cmd.doctor.check.claude.name"),
description: i18n.T("cmd.doctor.check.claude.description"),
command: "claude",
args: []string{"--version"},
versionFlag: "--version",
},
{
name: i18n.T("cmd.doctor.check.docker.name"),
description: i18n.T("cmd.doctor.check.docker.description"),
command: "docker",
args: []string{"--version"},
versionFlag: "--version",
},
}
}
// runCheck executes a tool check and returns success status and version info
func runCheck(c check) (bool, string) {
cmd := exec.Command(c.command, c.args...)
output, err := cmd.CombinedOutput()
if err != nil {
return false, ""
}
// Extract first line as version
lines := strings.Split(strings.TrimSpace(string(output)), "\n")
if len(lines) > 0 {
return true, strings.TrimSpace(lines[0])
}
return true, ""
}