feat(outfitter): add doctor command for environment validation

[galligan]

Implements doctor command that validates environment readiness:

• Checks Bun version meets minimum requirement (1.2.0)
• Validates required commands (git, node) are available
• Reports pass/warn/fail status with clear formatting
• Returns exit code 1 if any checks fail

Co-Authored-By: Claude Opus 4.5 noreply@anthropic.com

Resolves #57

[galligan]

• fix(errors): avoid TaggedError extends warnings #106 : 2 dependent PRs (#107 , #116 )
• chore(release): align package versions to 0.1.0 #105
• docs: add README and CHANGELOG for testing, kit, outfitter #95
• feat(templates): add daemon project template #94
• feat(templates): add MCP server project template #93
• feat(templates): add CLI project template #92
• fix(cli): implement confirmDestructive prompts #104
• fix(cli): implement createCLI and command builder #103
• feat(kit): add @outfitter/kit meta-package #91
• docs(daemon): add README and CHANGELOG #90
• fix(daemon): fix locking module bugs (GREEN phase) #89
• test(daemon): add locking module tests (RED phase) #88
• feat(daemon): add errors, platform, and locking module stubs #87
• feat(outfitter): add doctor command for environment validation #86 👈 (View in Graphite)
• feat(outfitter): add init command with project scaffolding #85
• feat(daemon): add IPC communication and health checks #84
• feat(daemon): add types and lifecycle management #83
• feat(index): add FTS5 full-text search implementation #82
• feat(index): add types for FTS5 full-text search #81
• feat(testing): add MCP and CLI test harnesses #80
• feat(testing): add test fixture utilities #79
• feat(mcp): add MCP server implementation with typed handlers #78
• feat(mcp): add types and interfaces for MCP server #77
• feat(ui): add Shape discriminated union and render() #76
• feat(ui): add createTokens semantic color layer #75
• feat(state): add pagination helpers #74
• feat(logging): add DEFAULT_PATTERNS for common secret redaction #73
• feat(ui): add formatRelative() for human-readable timestamps #72
• feat(state): add encodeCursor/decodeCursor for pagination #71
• feat(types): add hashId() for deterministic short identifiers #70
• feat(contracts): add Logger.child() and fix AssertionError category #69
• feat(config): centralize environment variable access with Zod validation #42
• fix(logging): enable redaction by default when global rules exist #41
• docs(ui): add README, CHANGELOG, and TSDoc comments #39
• docs(state): add README, CHANGELOG, and TSDoc comments #38
• docs(file-ops): add README, CHANGELOG, and TSDoc comments #37
• docs(logging): add README, CHANGELOG, and TSDoc comments #36
• docs(config): add README, CHANGELOG, and TSDoc comments #35
• fix(cli): enforce cursor TTL #114
• fix(tests): remove placeholder tests #113
• chore(gitattributes): prefer ours for bun lockfiles #115
• main

This stack of pull requests is managed by Graphite. Learn more about stacking.

[greptile-apps]

_{6 files reviewed, 2 comments}

_{Edit Code Review Agent Settings | Greptile}

[galligan]

Restacked on p3-23 and resolved init.test conflict by keeping the expanded init suite and forcing non-interactive (isTTY false) during tests. Pre-push tests ran clean.

[galligan]

Updated doctor version check to use Bun.semver.satisfies with a single MIN_BUN_VERSION (1.3.6 per SPEC/packageManager). Tests remain aligned to 1.3.6.

[galligan]

Restacked after downstack update (formatRelative test stabilization); no additional changes in this PR.

[galligan]

Checked greptile notes: Bun version check now uses Bun.semver.satisfies and the minimum requirement is 1.3.6 (tests reflect it). Restacked and resubmitted.

[galligan]

Bun requirement is now consistently 1.3.6 in tests/docs, and doctor uses Bun.semver.satisfies for the check.

[galligan]

Merge activity

• Jan 24, 1:54 PM UTC: A user started a stack merge that includes this pull request via Graphite.
• Jan 24, 2:51 PM UTC: Graphite rebased this pull request as part of a merge.
• Jan 24, 2:51 PM UTC: @galligan merged this pull request with Graphite.