@thegetty/quire-cli 1.0.0-rc.44 → 1.0.0-rc.46

package/src/commands/doctor.test.js

@@ -1,1409 +0,0 @@
-import test from 'ava'
-import sinon from 'sinon'
-import esmock from 'esmock'
-
-test.beforeEach((t) => {
-  t.context.sandbox = sinon.createSandbox()
-
-  // Create mock logger (still used by outputJson for file-write messages)
-  t.context.mockLogger = {
-    info: t.context.sandbox.stub(),
-    error: t.context.sandbox.stub(),
-    warn: t.context.sandbox.stub(),
-    debug: t.context.sandbox.stub(),
-  }
-})
-
-test.afterEach.always((t) => {
-  t.context.sandbox.restore()
-  // Reset process.exitCode to prevent tests that exercise failed checks
-  // from causing AVA to exit with a non-zero code
-  process.exitCode = undefined
-})
-
-/**
- * Helper to create a DoctorCommand with mocked dependencies
- */
-async function createMockedDoctorCommand(sandbox, mockSections) {
-  return esmock('./doctor.js', {
-    '#lib/doctor/index.js': {
-      runAllChecksWithSections: sandbox.stub().resolves(mockSections),
-      checkSections: [],
-      SECTION_NAMES: ['environment', 'project', 'outputs'],
-      CHECK_IDS: ['os', 'cli', 'node', 'npm', 'git', 'project', 'deps', '11ty', 'data', 'build', 'pdf', 'epub'],
-    },
-  })
-}
-
-/**
- * Helper to stub console methods and return the stubs
- *
- * Nota bene: doctor outputHuman writes directly to console (not the logger)
- * to avoid the [quire] prefix on diagnostic output.
- */
-function stubConsole(sandbox) {
-  return {
-    log: sandbox.stub(console, 'log'),
-    warn: sandbox.stub(console, 'warn'),
-    error: sandbox.stub(console, 'error'),
-  }
-}
-
-/**
- * Helper to collect all calls from a stub as strings
- */
-function getCalls(stub) {
-  return stub.getCalls().map((call) => call.args[0])
-}
-
-/**
- * Helper to check if any stub call matches a pattern
- */
-function calledWithMatch(stub, pattern) {
-  return getCalls(stub).some((arg) => arg != null && pattern.test(String(arg)))
-}
-
-// ─────────────────────────────────────────────────────────────────────────────
-// Human-readable output tests
-//
-// Nota bene: these tests are serial because they stub global console methods.
-// Doctor outputHuman bypasses the logger to avoid the [quire] prefix.
-// ─────────────────────────────────────────────────────────────────────────────
-
-test.serial('doctor command should run all diagnostic checks with sections', async (t) => {
-  const { sandbox, mockLogger } = t.context
-  const consoleStubs = stubConsole(sandbox)
-
-  const mockSections = [
-    {
-      section: 'Environment',
-      results: [
-        { name: 'Node.js version', ok: true, message: 'v22.0.0' },
-        { name: 'npm available', ok: true, message: null },
-        { name: 'Git available', ok: true, message: null },
-      ],
-    },
-    {
-      section: 'Project',
-      results: [
-        { name: 'Quire project', ok: true, message: 'Detected via .quire' },
-        { name: 'Dependencies', ok: true, message: null },
-        { name: 'Build status', ok: true, message: 'Build output is up to date' },
-      ],
-    },
-  ]
-
-  const DoctorCommand = await esmock('./doctor.js', {
-    '#lib/doctor/index.js': {
-      runAllChecksWithSections: sandbox.stub().resolves(mockSections),
-    },
-  })
-
-  const command = new DoctorCommand()
-  command.logger = mockLogger
-
-  await command.action([], {}, command)
-
-  t.true(consoleStubs.log.called, 'should write to console')
-  t.true(
-    calledWithMatch(consoleStubs.log, /Running diagnostic checks/),
-    'should display diagnostic header'
-  )
-  t.true(
-    calledWithMatch(consoleStubs.log, /Environment/),
-    'should display Environment section header'
-  )
-  t.true(
-    calledWithMatch(consoleStubs.log, /Project/),
-    'should display Project section header'
-  )
-})
-
-test.serial('doctor command should report failed checks', async (t) => {
-  const { sandbox, mockLogger } = t.context
-  const consoleStubs = stubConsole(sandbox)
-
-  const mockSections = [
-    {
-      section: 'Environment',
-      results: [
-        { name: 'Node.js version', ok: true, message: 'v22.0.0' },
-        {
-          name: 'npm available',
-          ok: false,
-          message: 'npm not found in PATH',
-          remediation: 'Install Node.js to get npm',
-          docsUrl: 'https://quire.getty.edu/docs-v1/install-uninstall/',
-        },
-      ],
-    },
-  ]
-
-  const DoctorCommand = await createMockedDoctorCommand(sandbox, mockSections)
-
-  const command = new DoctorCommand()
-  command.logger = mockLogger
-
-  await command.action([], {}, command)
-
-  t.true(consoleStubs.error.called, 'should write errors to stderr')
-  t.true(
-    calledWithMatch(consoleStubs.error, /1 check failed/),
-    'should report failure count'
-  )
-})
-
-test.serial('doctor command should display remediation guidance for failed checks', async (t) => {
-  const { sandbox, mockLogger } = t.context
-  const consoleStubs = stubConsole(sandbox)
-
-  const mockSections = [
-    {
-      section: 'Environment',
-      results: [
-        {
-          name: 'npm available',
-          ok: false,
-          message: 'npm not found in PATH',
-          remediation: 'Install Node.js to get npm',
-          docsUrl: 'https://quire.getty.edu/docs-v1/install-uninstall/',
-        },
-      ],
-    },
-  ]
-
-  const DoctorCommand = await createMockedDoctorCommand(sandbox, mockSections)
-
-  const command = new DoctorCommand()
-  command.logger = mockLogger
-
-  await command.action([], {}, command)
-
-  // Failed checks output goes through console.error as a single joined string
-  t.true(
-    calledWithMatch(consoleStubs.error, /How to fix/),
-    'should display "How to fix" header'
-  )
-  t.true(
-    calledWithMatch(consoleStubs.error, /Install Node.js/),
-    'should display remediation text'
-  )
-  t.true(
-    calledWithMatch(consoleStubs.error, /Documentation:/),
-    'should display documentation label'
-  )
-  t.true(
-    calledWithMatch(consoleStubs.error, /quire\.getty\.edu/),
-    'should display documentation URL'
-  )
-})
-
-test.serial('doctor command should display warnings with warning indicator', async (t) => {
-  const { sandbox, mockLogger } = t.context
-  const consoleStubs = stubConsole(sandbox)
-
-  const mockSections = [
-    {
-      section: 'Project',
-      results: [
-        {
-          name: 'Build status',
-          ok: false,
-          level: 'warn',
-          message: 'Build output is 1 hour older than source files',
-          remediation: 'Run quire build to regenerate',
-        },
-      ],
-    },
-  ]
-
-  const DoctorCommand = await esmock('./doctor.js', {
-    '#lib/doctor/index.js': {
-      runAllChecksWithSections: sandbox.stub().resolves(mockSections),
-    },
-  })
-
-  const command = new DoctorCommand()
-  command.logger = mockLogger
-
-  await command.action([], {}, command)
-
-  // Warnings go through console.warn
-  t.true(consoleStubs.warn.called, 'should write warnings to stderr')
-  t.true(
-    calledWithMatch(consoleStubs.warn, /⚠/),
-    'should display warning indicator'
-  )
-  t.true(
-    calledWithMatch(consoleStubs.warn, /All checks passed with 1 warning/),
-    'should report passed with warning count'
-  )
-})
-
-test.serial('doctor command should report all checks passed when healthy', async (t) => {
-  const { sandbox, mockLogger } = t.context
-  const consoleStubs = stubConsole(sandbox)
-
-  const mockSections = [
-    {
-      section: 'Environment',
-      results: [
-        { name: 'Node.js version', ok: true, message: 'v22.0.0' },
-        { name: 'npm available', ok: true, message: null },
-        { name: 'Git available', ok: true, message: null },
-      ],
-    },
-  ]
-
-  const DoctorCommand = await esmock('./doctor.js', {
-    '#lib/doctor/index.js': {
-      runAllChecksWithSections: sandbox.stub().resolves(mockSections),
-    },
-  })
-
-  const command = new DoctorCommand()
-  command.logger = mockLogger
-
-  await command.action([], {}, command)
-
-  t.true(
-    calledWithMatch(consoleStubs.log, /All checks passed/),
-    'should report all checks passed'
-  )
-})
-
-test.serial('doctor command should display check messages', async (t) => {
-  const { sandbox, mockLogger } = t.context
-  const consoleStubs = stubConsole(sandbox)
-
-  const mockSections = [
-    {
-      section: 'Environment',
-      results: [
-        { name: 'Node.js version', ok: true, message: 'v22.0.0 (>= 22 required)' },
-      ],
-    },
-  ]
-
-  const DoctorCommand = await esmock('./doctor.js', {
-    '#lib/doctor/index.js': {
-      runAllChecksWithSections: sandbox.stub().resolves(mockSections),
-    },
-  })
-
-  const command = new DoctorCommand()
-  command.logger = mockLogger
-
-  await command.action([], {}, command)
-
-  t.true(
-    calledWithMatch(consoleStubs.log, /v22\.0\.0/),
-    'should display check message'
-  )
-})
-
-// ─────────────────────────────────────────────────────────────────────────────
-// Exit code tests
-// ─────────────────────────────────────────────────────────────────────────────
-
-test.serial('doctor command should set exitCode 1 when checks fail', async (t) => {
-  const { sandbox, mockLogger } = t.context
-  stubConsole(sandbox)
-
-  const savedExitCode = process.exitCode
-
-  try {
-    const mockSections = [
-      {
-        section: 'Environment',
-        results: [
-          {
-            name: 'npm available',
-            ok: false,
-            message: 'npm not found in PATH',
-            remediation: 'Install Node.js to get npm',
-          },
-        ],
-      },
-    ]
-
-    const DoctorCommand = await createMockedDoctorCommand(sandbox, mockSections)
-
-    const command = new DoctorCommand()
-    command.logger = mockLogger
-
-    await command.action([], {}, command)
-
-    t.is(process.exitCode, 1, 'should set process.exitCode to 1')
-  } finally {
-    process.exitCode = savedExitCode
-  }
-})
-
-test.serial('doctor command should not set exitCode when all checks pass', async (t) => {
-  const { sandbox, mockLogger } = t.context
-  stubConsole(sandbox)
-
-  const savedExitCode = process.exitCode
-  process.exitCode = undefined
-
-  try {
-    const mockSections = [
-      {
-        section: 'Environment',
-        results: [
-          { name: 'Node.js version', ok: true, message: 'v22.0.0' },
-          { name: 'npm available', ok: true, message: null },
-        ],
-      },
-    ]
-
-    const DoctorCommand = await createMockedDoctorCommand(sandbox, mockSections)
-
-    const command = new DoctorCommand()
-    command.logger = mockLogger
-
-    await command.action([], {}, command)
-
-    t.is(process.exitCode, undefined, 'should not set process.exitCode on success')
-  } finally {
-    process.exitCode = savedExitCode
-  }
-})
-
-test.serial('doctor command should not set exitCode when only warnings exist', async (t) => {
-  const { sandbox, mockLogger } = t.context
-  const consoleStubs = stubConsole(sandbox)
-
-  const savedExitCode = process.exitCode
-  process.exitCode = undefined
-
-  try {
-    const mockSections = [
-      {
-        section: 'Project',
-        results: [
-          {
-            name: 'Build status',
-            ok: false,
-            level: 'warn',
-            message: 'Build output is stale',
-            remediation: 'Run quire build',
-          },
-        ],
-      },
-    ]
-
-    const DoctorCommand = await createMockedDoctorCommand(sandbox, mockSections)
-
-    const command = new DoctorCommand()
-    command.logger = mockLogger
-
-    await command.action([], {}, command)
-
-    t.is(process.exitCode, undefined, 'should not set process.exitCode for warnings')
-    t.true(
-      calledWithMatch(consoleStubs.warn, /All checks passed with 1 warning/),
-      'should report passed with warning count'
-    )
-  } finally {
-    process.exitCode = savedExitCode
-  }
-})
-
-// ─────────────────────────────────────────────────────────────────────────────
-// Summary count tests
-// ─────────────────────────────────────────────────────────────────────────────
-
-test.serial('doctor command should display plural error count', async (t) => {
-  const { sandbox, mockLogger } = t.context
-  const consoleStubs = stubConsole(sandbox)
-
-  const mockSections = [
-    {
-      section: 'Environment',
-      results: [
-        { name: 'npm', ok: false, message: 'not found' },
-        { name: 'Git', ok: false, message: 'not found' },
-      ],
-    },
-  ]
-
-  const DoctorCommand = await createMockedDoctorCommand(sandbox, mockSections)
-
-  const command = new DoctorCommand()
-  command.logger = mockLogger
-
-  await command.action([], {}, command)
-
-  t.true(
-    calledWithMatch(consoleStubs.error, /2 checks failed/),
-    'should use plural for multiple errors'
-  )
-})
-
-test.serial('doctor command should display errors and warnings together', async (t) => {
-  const { sandbox, mockLogger } = t.context
-  const consoleStubs = stubConsole(sandbox)
-
-  const mockSections = [
-    {
-      section: 'Environment',
-      results: [
-        { name: 'npm', ok: false, message: 'not found' },
-        { name: 'Git', ok: false, level: 'warn', message: 'outdated' },
-        { name: 'Prince', ok: false, level: 'warn', message: 'not found' },
-      ],
-    },
-  ]
-
-  const DoctorCommand = await createMockedDoctorCommand(sandbox, mockSections)
-
-  const command = new DoctorCommand()
-  command.logger = mockLogger
-
-  await command.action([], {}, command)
-
-  t.true(
-    calledWithMatch(consoleStubs.error, /1 check failed, 2 warnings/),
-    'should display both error and warning counts'
-  )
-})
-
-test.serial('doctor command should display plural warning count', async (t) => {
-  const { sandbox, mockLogger } = t.context
-  const consoleStubs = stubConsole(sandbox)
-
-  const mockSections = [
-    {
-      section: 'Project',
-      results: [
-        { name: 'Build', ok: false, level: 'warn', message: 'stale' },
-        { name: 'PDF', ok: false, level: 'warn', message: 'stale' },
-      ],
-    },
-  ]
-
-  const DoctorCommand = await esmock('./doctor.js', {
-    '#lib/doctor/index.js': {
-      runAllChecksWithSections: sandbox.stub().resolves(mockSections),
-    },
-  })
-
-  const command = new DoctorCommand()
-  command.logger = mockLogger
-
-  await command.action([], {}, command)
-
-  t.true(
-    calledWithMatch(consoleStubs.warn, /All checks passed with 2 warnings/),
-    'should use plural for multiple warnings'
-  )
-})
-
-// ─────────────────────────────────────────────────────────────────────────────
-// N/A (not applicable) state tests
-// ─────────────────────────────────────────────────────────────────────────────
-
-test.serial('doctor command should display N/A checks with open circle indicator', async (t) => {
-  const { sandbox, mockLogger } = t.context
-  const consoleStubs = stubConsole(sandbox)
-
-  const mockSections = [
-    {
-      section: 'Outputs',
-      results: [
-        {
-          name: 'Build status',
-          ok: true,
-          level: 'na',
-          message: 'No build output yet (run quire build)',
-        },
-        {
-          name: 'PDF output',
-          ok: true,
-          level: 'na',
-          message: 'No PDF output (run quire pdf to generate)',
-        },
-      ],
-    },
-  ]
-
-  const DoctorCommand = await esmock('./doctor.js', {
-    '#lib/doctor/index.js': {
-      runAllChecksWithSections: sandbox.stub().resolves(mockSections),
-    },
-  })
-
-  const command = new DoctorCommand()
-  command.logger = mockLogger
-
-  await command.action([], {}, command)
-
-  // N/A checks should use ○ indicator
-  t.true(
-    calledWithMatch(consoleStubs.log, /○/),
-    'should display open circle indicator for N/A checks'
-  )
-})
-
-test.serial('doctor command should not count N/A checks as failures in summary', async (t) => {
-  const { sandbox, mockLogger } = t.context
-  const consoleStubs = stubConsole(sandbox)
-
-  const mockSections = [
-    {
-      section: 'Outputs',
-      results: [
-        { name: 'Build status', ok: true, level: 'na', message: 'No build output' },
-        { name: 'PDF output', ok: true, level: 'na', message: 'No PDF output' },
-        { name: 'EPUB output', ok: true, level: 'na', message: 'No EPUB output' },
-      ],
-    },
-  ]
-
-  const DoctorCommand = await esmock('./doctor.js', {
-    '#lib/doctor/index.js': {
-      runAllChecksWithSections: sandbox.stub().resolves(mockSections),
-    },
-  })
-
-  const command = new DoctorCommand()
-  command.logger = mockLogger
-
-  await command.action([], {}, command)
-
-  // Should report all checks passed (N/A checks don't count as failures)
-  t.true(
-    calledWithMatch(consoleStubs.log, /All checks passed/),
-    'should report all checks passed when only N/A checks exist'
-  )
-  // Should include N/A count in summary
-  t.true(
-    calledWithMatch(consoleStubs.log, /3 not applicable/),
-    'should display N/A count in summary'
-  )
-})
-
-test.serial('doctor command should not count N/A checks as failures when mixed with errors', async (t) => {
-  const { sandbox, mockLogger } = t.context
-  const consoleStubs = stubConsole(sandbox)
-
-  const mockSections = [
-    {
-      section: 'Environment',
-      results: [
-        { name: 'npm', ok: false, message: 'not found' },
-      ],
-    },
-    {
-      section: 'Outputs',
-      results: [
-        { name: 'Build status', ok: true, level: 'na', message: 'No build output' },
-        { name: 'PDF output', ok: true, level: 'na', message: 'No PDF output' },
-      ],
-    },
-  ]
-
-  const DoctorCommand = await createMockedDoctorCommand(sandbox, mockSections)
-
-  const command = new DoctorCommand()
-  command.logger = mockLogger
-
-  await command.action([], {}, command)
-
-  // Should only report 1 error (N/A checks not counted as failures)
-  t.true(
-    calledWithMatch(consoleStubs.error, /1 check failed/),
-    'should only count actual errors, not N/A checks'
-  )
-  // Should include N/A count in summary
-  t.true(
-    calledWithMatch(consoleStubs.error, /2 not applicable/),
-    'should display N/A count alongside error count'
-  )
-})
-
-test.serial('doctor command should route N/A checks to stdout', async (t) => {
-  const { sandbox, mockLogger } = t.context
-  const consoleStubs = stubConsole(sandbox)
-
-  const mockSections = [
-    {
-      section: 'Outputs',
-      results: [
-        { name: 'PDF output', ok: true, level: 'na', message: 'No PDF output' },
-      ],
-    },
-  ]
-
-  const DoctorCommand = await esmock('./doctor.js', {
-    '#lib/doctor/index.js': {
-      runAllChecksWithSections: sandbox.stub().resolves(mockSections),
-    },
-  })
-
-  const command = new DoctorCommand()
-  command.logger = mockLogger
-
-  await command.action([], {}, command)
-
-  // N/A should go through console.log (stdout), not console.warn or console.error
-  const logCalls = getCalls(consoleStubs.log)
-  const hasNaInLog = logCalls.some((arg) => arg && arg.includes('○') && arg.includes('PDF output'))
-  t.true(hasNaInLog, 'N/A checks should be routed to stdout via console.log')
-})
-
-// ─────────────────────────────────────────────────────────────────────────────
-// Verbose flag tests
-// ─────────────────────────────────────────────────────────────────────────────
-
-test.serial('doctor command should display details when --verbose is passed', async (t) => {
-  const { sandbox, mockLogger } = t.context
-  const consoleStubs = stubConsole(sandbox)
-
-  const mockSections = [
-    {
-      section: 'Environment',
-      results: [
-        { name: 'npm', ok: true, message: '10.2.4', details: '/usr/local/bin/npm' },
-        { name: 'Git', ok: true, message: '2.43.0', details: '/usr/bin/git' },
-      ],
-    },
-  ]
-
-  const DoctorCommand = await esmock('./doctor.js', {
-    '#lib/doctor/index.js': {
-      runAllChecksWithSections: sandbox.stub().resolves(mockSections),
-    },
-  })
-
-  const command = new DoctorCommand()
-  command.logger = mockLogger
-
-  await command.action([], { verbose: true }, command)
-
-  // Should display details for each check
-  t.true(
-    calledWithMatch(consoleStubs.log, /\/usr\/local\/bin\/npm/),
-    'should display npm path in verbose mode'
-  )
-  t.true(
-    calledWithMatch(consoleStubs.log, /\/usr\/bin\/git/),
-    'should display git path in verbose mode'
-  )
-})
-
-test.serial('doctor command should not display details when --verbose is not passed', async (t) => {
-  const { sandbox, mockLogger } = t.context
-  const consoleStubs = stubConsole(sandbox)
-
-  const mockSections = [
-    {
-      section: 'Environment',
-      results: [
-        { name: 'npm', ok: true, message: '10.2.4', details: '/usr/local/bin/npm' },
-      ],
-    },
-  ]
-
-  const DoctorCommand = await esmock('./doctor.js', {
-    '#lib/doctor/index.js': {
-      runAllChecksWithSections: sandbox.stub().resolves(mockSections),
-    },
-  })
-
-  const command = new DoctorCommand()
-  command.logger = mockLogger
-
-  await command.action([], {}, command)
-
-  // Should not display details path
-  const allCalls = getCalls(consoleStubs.log)
-  const hasPathInOutput = allCalls.some((arg) => arg && arg.includes('/usr/local/bin/npm'))
-  t.false(hasPathInOutput, 'should not display npm path without verbose flag')
-})
-
-test.serial('doctor command should handle checks without details in verbose mode', async (t) => {
-  const { sandbox, mockLogger } = t.context
-  const consoleStubs = stubConsole(sandbox)
-
-  const mockSections = [
-    {
-      section: 'Environment',
-      results: [
-        { name: 'Node.js version', ok: true, message: 'v22.0.0' }, // no details field
-      ],
-    },
-  ]
-
-  const DoctorCommand = await esmock('./doctor.js', {
-    '#lib/doctor/index.js': {
-      runAllChecksWithSections: sandbox.stub().resolves(mockSections),
-    },
-  })
-
-  const command = new DoctorCommand()
-  command.logger = mockLogger
-
-  // Should not throw when details is undefined
-  await t.notThrowsAsync(async () => {
-    await command.action([], { verbose: true }, command)
-  })
-
-  t.true(calledWithMatch(consoleStubs.log, /v22\.0\.0/), 'should still display message')
-})
-
-test.serial('doctor command should display symbol key in verbose mode', async (t) => {
-  const { sandbox, mockLogger } = t.context
-  const consoleStubs = stubConsole(sandbox)
-
-  const mockSections = [
-    {
-      section: 'Environment',
-      results: [
-        { name: 'Node.js version', ok: true, message: 'v22.0.0' },
-      ],
-    },
-  ]
-
-  const DoctorCommand = await esmock('./doctor.js', {
-    '#lib/doctor/index.js': {
-      runAllChecksWithSections: sandbox.stub().resolves(mockSections),
-    },
-  })
-
-  const command = new DoctorCommand()
-  command.logger = mockLogger
-
-  await command.action([], { verbose: true }, command)
-
-  t.true(
-    calledWithMatch(consoleStubs.log, /Key:.*✓.*passed.*✗.*failed.*⚠.*warning.*○.*not applicable.*not yet generated/),
-    'should display symbol key'
-  )
-})
-
-test.serial('doctor command should display symbol key in default mode', async (t) => {
-  const { sandbox, mockLogger } = t.context
-  const consoleStubs = stubConsole(sandbox)
-
-  const mockSections = [
-    {
-      section: 'Environment',
-      results: [
-        { name: 'Node.js version', ok: true, message: 'v22.0.0' },
-      ],
-    },
-  ]
-
-  const DoctorCommand = await esmock('./doctor.js', {
-    '#lib/doctor/index.js': {
-      runAllChecksWithSections: sandbox.stub().resolves(mockSections),
-    },
-  })
-
-  const command = new DoctorCommand()
-  command.logger = mockLogger
-
-  await command.action([], {}, command)
-
-  const allCalls = getCalls(consoleStubs.log)
-  const hasKey = allCalls.some((arg) => arg && arg.includes('Key:'))
-  t.true(hasKey, 'should display symbol key in default mode')
-})
-
-// ─────────────────────────────────────────────────────────────────────────────
-// JSON output tests
-// ─────────────────────────────────────────────────────────────────────────────
-
-test.serial('doctor command should output valid JSON when --json flag is used', async (t) => {
-  const { sandbox, mockLogger } = t.context
-
-  const mockSections = [
-    {
-      section: 'Environment',
-      results: [
-        { id: 'node', name: 'Node.js version', ok: true, message: 'v22.0.0' },
-        { id: 'npm', name: 'npm', ok: true, message: '10.2.4', details: '/usr/local/bin/npm' },
-      ],
-    },
-  ]
-
-  // Capture console.log output
-  const consoleLogStub = sandbox.stub(console, 'log')
-
-  const DoctorCommand = await esmock('./doctor.js', {
-    '#lib/doctor/index.js': {
-      runAllChecksWithSections: sandbox.stub().resolves(mockSections),
-    },
-  })
-
-  const command = new DoctorCommand()
-  command.logger = mockLogger
-
-  await command.action([], { json: true }, command)
-
-  // Should call console.log with JSON
-  t.true(consoleLogStub.calledOnce, 'should call console.log once')
-
-  const output = consoleLogStub.firstCall.args[0]
-  const parsed = JSON.parse(output)
-
-  t.truthy(parsed.summary, 'should have summary object')
-  t.is(parsed.summary.passed, 2, 'should count passed checks')
-  t.is(parsed.summary.failed, 0, 'should count failed checks')
-  t.truthy(parsed.results, 'should have results array')
-  t.is(parsed.results.length, 2, 'should include all results')
-  t.is(parsed.results[0].id, 'node', 'check should have id')
-  t.is(parsed.results[0].status, 'passed', 'check should have status')
-})
-
-test.serial('doctor command JSON output should set exitCode 1 when checks fail', async (t) => {
-  const { sandbox, mockLogger } = t.context
-
-  const consoleLogStub = sandbox.stub(console, 'log')
-  const savedExitCode = process.exitCode
-
-  try {
-    const mockSections = [
-      {
-        section: 'Environment',
-        results: [
-          { id: 'node', name: 'Node.js version', ok: false, message: 'v18.0.0 found' },
-        ],
-      },
-    ]
-
-    const DoctorCommand = await esmock('./doctor.js', {
-      '#lib/doctor/index.js': {
-        runAllChecksWithSections: sandbox.stub().resolves(mockSections),
-      },
-    })
-
-    const command = new DoctorCommand()
-    command.logger = mockLogger
-
-    await command.action([], { json: true }, command)
-
-    t.is(process.exitCode, 1, 'should set process.exitCode to 1')
-
-    const output = consoleLogStub.firstCall.args[0]
-    const parsed = JSON.parse(output)
-    t.is(parsed.summary.failed, 1, 'should count failed check')
-  } finally {
-    process.exitCode = savedExitCode
-  }
-})
-
-test.serial('doctor command JSON output should include remediation for failed checks', async (t) => {
-  const { sandbox, mockLogger } = t.context
-
-  const consoleLogStub = sandbox.stub(console, 'log')
-
-  const mockSections = [
-    {
-      section: 'Environment',
-      results: [
-        {
-          id: 'node',
-          name: 'Node.js version',
-          ok: false,
-          message: 'v18.0.0 found',
-          remediation: 'Install Node.js 22+',
-          docsUrl: 'https://example.com/docs',
-        },
-      ],
-    },
-  ]
-
-  const DoctorCommand = await esmock('./doctor.js', {
-    '#lib/doctor/index.js': {
-      runAllChecksWithSections: sandbox.stub().resolves(mockSections),
-    },
-  })
-
-  const command = new DoctorCommand()
-  command.logger = mockLogger
-
-  await command.action([], { json: true }, command)
-
-  const output = consoleLogStub.firstCall.args[0]
-  const parsed = JSON.parse(output)
-
-  t.is(parsed.results[0].remediation, 'Install Node.js 22+', 'should include remediation')
-  t.is(parsed.results[0].docsUrl, 'https://example.com/docs', 'should include docsUrl')
-})
-
-// ─────────────────────────────────────────────────────────────────────────────
-// --errors and --warnings filter tests
-// ─────────────────────────────────────────────────────────────────────────────
-
-test.serial('doctor command --errors flag should show only failed checks', async (t) => {
-  const { sandbox, mockLogger } = t.context
-  const consoleStubs = stubConsole(sandbox)
-
-  const mockSections = [
-    {
-      section: 'Environment',
-      results: [
-        { id: 'node', name: 'Node.js version', ok: true, message: 'v22.0.0' },
-        { id: 'npm', name: 'npm', ok: false, message: 'not found' },
-        { id: 'git', name: 'Git', ok: false, level: 'warn', message: 'old version' },
-      ],
-    },
-  ]
-
-  const DoctorCommand = await esmock('./doctor.js', {
-    '#lib/doctor/index.js': {
-      runAllChecksWithSections: sandbox.stub().resolves(mockSections),
-    },
-  })
-
-  const command = new DoctorCommand()
-  command.logger = mockLogger
-
-  await command.action([], { errors: true }, command)
-
-  // Should only show failed checks (not passed or warnings)
-  const errorCalls = getCalls(consoleStubs.error)
-  const hasNpm = errorCalls.some((arg) => arg && arg.includes('npm'))
-  t.true(hasNpm, 'should show failed npm check')
-
-  // Should not show passed checks in output
-  const logCalls = getCalls(consoleStubs.log)
-  const hasNode = logCalls.some((arg) => arg && arg.includes('Node.js') && arg.includes('✓'))
-  t.false(hasNode, 'should not show passed Node.js check')
-})
-
-test.serial('doctor command --warnings flag should show only warning checks', async (t) => {
-  const { sandbox, mockLogger } = t.context
-  const consoleStubs = stubConsole(sandbox)
-
-  const mockSections = [
-    {
-      section: 'Environment',
-      results: [
-        { id: 'node', name: 'Node.js version', ok: true, message: 'v22.0.0' },
-        { id: 'cli', name: 'CLI version', ok: false, level: 'warn', message: 'update available' },
-        { id: 'npm', name: 'npm', ok: false, message: 'not found' },
-      ],
-    },
-  ]
-
-  const DoctorCommand = await esmock('./doctor.js', {
-    '#lib/doctor/index.js': {
-      runAllChecksWithSections: sandbox.stub().resolves(mockSections),
-    },
-  })
-
-  const command = new DoctorCommand()
-  command.logger = mockLogger
-
-  await command.action([], { warnings: true }, command)
-
-  // Should show warning checks
-  const warnCalls = getCalls(consoleStubs.warn)
-  const hasCli = warnCalls.some((arg) => arg && arg.includes('CLI version'))
-  t.true(hasCli, 'should show CLI version warning')
-
-  // Should not show passed checks
-  const logCalls = getCalls(consoleStubs.log)
-  const hasNode = logCalls.some((arg) => arg && arg.includes('Node.js') && arg.includes('✓'))
-  t.false(hasNode, 'should not show passed Node.js check')
-})
-
-test.serial('doctor command should show "No failed checks found" when --errors finds none', async (t) => {
-  const { sandbox, mockLogger } = t.context
-  const consoleStubs = stubConsole(sandbox)
-
-  const mockSections = [
-    {
-      section: 'Environment',
-      results: [
-        { id: 'node', name: 'Node.js version', ok: true, message: 'v22.0.0' },
-        { id: 'npm', name: 'npm', ok: true, message: '10.2.4' },
-      ],
-    },
-  ]
-
-  const DoctorCommand = await esmock('./doctor.js', {
-    '#lib/doctor/index.js': {
-      runAllChecksWithSections: sandbox.stub().resolves(mockSections),
-    },
-  })
-
-  const command = new DoctorCommand()
-  command.logger = mockLogger
-
-  await command.action([], { errors: true }, command)
-
-  t.true(
-    calledWithMatch(consoleStubs.log, /No failed checks found/),
-    'should show no failed checks message'
-  )
-})
-
-test.serial('doctor command should show "No warnings found" when --warnings finds none', async (t) => {
-  const { sandbox, mockLogger } = t.context
-  const consoleStubs = stubConsole(sandbox)
-
-  const mockSections = [
-    {
-      section: 'Environment',
-      results: [
-        { id: 'node', name: 'Node.js version', ok: true, message: 'v22.0.0' },
-      ],
-    },
-  ]
-
-  const DoctorCommand = await esmock('./doctor.js', {
-    '#lib/doctor/index.js': {
-      runAllChecksWithSections: sandbox.stub().resolves(mockSections),
-    },
-  })
-
-  const command = new DoctorCommand()
-  command.logger = mockLogger
-
-  await command.action([], { warnings: true }, command)
-
-  t.true(
-    calledWithMatch(consoleStubs.log, /No warnings found/),
-    'should show no warnings message'
-  )
-})
-
-// ─────────────────────────────────────────────────────────────────────────────
-// --json [file] file output tests
-// ─────────────────────────────────────────────────────────────────────────────
-
-test.serial('doctor command --json <file> should write JSON to file', async (t) => {
-  const { sandbox, mockLogger } = t.context
-
-  const mockSections = [
-    {
-      section: 'Environment',
-      results: [
-        { id: 'node', name: 'Node.js version', ok: true, message: 'v22.0.0' },
-      ],
-    },
-  ]
-
-  // Mock fs.writeFileSync
-  const writeFileSyncStub = sandbox.stub()
-
-  const DoctorCommand = await esmock('./doctor.js', {
-    'node:fs': {
-      writeFileSync: writeFileSyncStub,
-    },
-    '#lib/doctor/index.js': {
-      runAllChecksWithSections: sandbox.stub().resolves(mockSections),
-    },
-  })
-
-  const command = new DoctorCommand()
-  command.logger = mockLogger
-
-  // When --json has a value, options.json is the filename string
-  await command.action([], { json: 'report.json' }, command)
-
-  t.true(writeFileSyncStub.calledOnce, 'should call writeFileSync once')
-  t.true(
-    writeFileSyncStub.firstCall.args[0].endsWith('report.json'),
-    'should write to specified file'
-  )
-
-  // Verify the content is valid JSON
-  const writtenContent = writeFileSyncStub.firstCall.args[1]
-  t.notThrows(() => JSON.parse(writtenContent.trim()), 'should write valid JSON')
-
-  t.true(
-    mockLogger.info.calledWith(sinon.match(/Results written to/)),
-    'should log success message'
-  )
-})
-
-test.serial('doctor command --json (no file) should write to stdout', async (t) => {
-  const { sandbox, mockLogger } = t.context
-
-  const mockSections = [
-    {
-      section: 'Environment',
-      results: [
-        { id: 'node', name: 'Node.js version', ok: true, message: 'v22.0.0' },
-      ],
-    },
-  ]
-
-  const consoleLogStub = sandbox.stub(console, 'log')
-
-  const DoctorCommand = await esmock('./doctor.js', {
-    '#lib/doctor/index.js': {
-      runAllChecksWithSections: sandbox.stub().resolves(mockSections),
-    },
-  })
-
-  const command = new DoctorCommand()
-  command.logger = mockLogger
-
-  // When --json is used without a value, options.json is true
-  await command.action([], { json: true }, command)
-
-  t.true(consoleLogStub.calledOnce, 'should write to stdout')
-  t.notThrows(() => JSON.parse(consoleLogStub.firstCall.args[0]), 'should output valid JSON')
-})
-
-test.serial('doctor command --json <file> should not write to stdout', async (t) => {
-  const { sandbox, mockLogger } = t.context
-
-  const mockSections = [
-    {
-      section: 'Environment',
-      results: [
-        { id: 'node', name: 'Node.js version', ok: true, message: 'v22.0.0' },
-      ],
-    },
-  ]
-
-  const consoleLogStub = sandbox.stub(console, 'log')
-  const writeFileSyncStub = sandbox.stub()
-
-  const DoctorCommand = await esmock('./doctor.js', {
-    'node:fs': {
-      writeFileSync: writeFileSyncStub,
-    },
-    '#lib/doctor/index.js': {
-      runAllChecksWithSections: sandbox.stub().resolves(mockSections),
-    },
-  })
-
-  const command = new DoctorCommand()
-  command.logger = mockLogger
-
-  await command.action([], { json: 'report.json' }, command)
-
-  t.false(consoleLogStub.called, 'should not write to stdout when file is specified')
-  t.true(writeFileSyncStub.calledOnce, 'should write to file instead')
-})
-
-// ─────────────────────────────────────────────────────────────────────────────
-// --quiet flag tests
-// ─────────────────────────────────────────────────────────────────────────────
-
-test.serial('doctor command --quiet alone should produce no output on success', async (t) => {
-  const { sandbox, mockLogger } = t.context
-
-  const mockSections = [
-    {
-      section: 'Environment',
-      results: [
-        { id: 'node', name: 'Node.js version', ok: true, message: 'v22.0.0' },
-      ],
-    },
-  ]
-
-  const consoleLogStub = sandbox.stub(console, 'log')
-
-  const DoctorCommand = await esmock('./doctor.js', {
-    '#lib/doctor/index.js': {
-      runAllChecksWithSections: sandbox.stub().resolves(mockSections),
-    },
-  })
-
-  const command = new DoctorCommand()
-  command.logger = mockLogger
-
-  // --quiet alone (no --json) should produce no output
-  await command.action([], { quiet: true }, command)
-
-  t.false(consoleLogStub.called, 'should not write to stdout')
-  t.false(mockLogger.info.called, 'should not log any info messages')
-})
-
-test.serial('doctor command --quiet alone should set exitCode 1 on failures', async (t) => {
-  const { sandbox, mockLogger } = t.context
-
-  const consoleLogStub = sandbox.stub(console, 'log')
-  const savedExitCode = process.exitCode
-
-  try {
-    const mockSections = [
-      {
-        section: 'Environment',
-        results: [
-          { id: 'npm', name: 'npm', ok: false, message: 'not found' },
-        ],
-      },
-    ]
-
-    const DoctorCommand = await esmock('./doctor.js', {
-      '#lib/doctor/index.js': {
-        runAllChecksWithSections: sandbox.stub().resolves(mockSections),
-      },
-    })
-
-    const command = new DoctorCommand()
-    command.logger = mockLogger
-
-    // --quiet alone should still set exitCode 1 on failure
-    await command.action([], { quiet: true }, command)
-
-    t.is(process.exitCode, 1, 'should set process.exitCode to 1')
-    t.false(consoleLogStub.called, 'should not output anything')
-    t.false(mockLogger.info.called, 'should not log any info messages')
-  } finally {
-    process.exitCode = savedExitCode
-  }
-})
-
-test.serial('doctor command --quiet --json should suppress stdout JSON output', async (t) => {
-  const { sandbox, mockLogger } = t.context
-
-  const mockSections = [
-    {
-      section: 'Environment',
-      results: [
-        { id: 'node', name: 'Node.js version', ok: true, message: 'v22.0.0' },
-      ],
-    },
-  ]
-
-  const consoleLogStub = sandbox.stub(console, 'log')
-
-  const DoctorCommand = await esmock('./doctor.js', {
-    '#lib/doctor/index.js': {
-      runAllChecksWithSections: sandbox.stub().resolves(mockSections),
-    },
-  })
-
-  const command = new DoctorCommand()
-  command.logger = mockLogger
-
-  // --quiet with --json should suppress JSON stdout output
-  await command.action([], { quiet: true, json: true }, command)
-
-  t.false(consoleLogStub.called, 'should not write JSON to stdout in quiet mode')
-})
-
-test.serial('doctor command --quiet --json <file> should write to file silently', async (t) => {
-  const { sandbox, mockLogger } = t.context
-
-  const mockSections = [
-    {
-      section: 'Environment',
-      results: [
-        { id: 'node', name: 'Node.js version', ok: true, message: 'v22.0.0' },
-      ],
-    },
-  ]
-
-  const writeFileSyncStub = sandbox.stub()
-
-  const DoctorCommand = await esmock('./doctor.js', {
-    'node:fs': {
-      writeFileSync: writeFileSyncStub,
-    },
-    '#lib/doctor/index.js': {
-      runAllChecksWithSections: sandbox.stub().resolves(mockSections),
-    },
-  })
-
-  const command = new DoctorCommand()
-  command.logger = mockLogger
-
-  await command.action([], { quiet: true, json: 'report.json' }, command)
-
-  t.true(writeFileSyncStub.calledOnce, 'should write to file')
-  t.false(
-    mockLogger.info.calledWith(sinon.match(/Results written to/)),
-    'should not log success message in quiet mode'
-  )
-})
-
-// ─────────────────────────────────────────────────────────────────────────────
-// --reset flag tests
-// ─────────────────────────────────────────────────────────────────────────────
-
-test.serial('doctor command --reset should clear build status', async (t) => {
-  const { sandbox, mockLogger } = t.context
-
-  const clearStatusStub = sandbox.stub()
-  const runAllChecksStub = sandbox.stub().resolves([])
-
-  const DoctorCommand = await esmock('./doctor.js', {
-    '#lib/conf/build-status.js': {
-      clearStatus: clearStatusStub,
-    },
-    '#lib/doctor/index.js': {
-      runAllChecksWithSections: runAllChecksStub,
-      checkSections: [],
-      SECTION_NAMES: ['environment', 'project', 'outputs'],
-      CHECK_IDS: ['os', 'cli', 'node', 'npm', 'git', 'project', 'deps', '11ty', 'data', 'build', 'pdf', 'epub'],
-    },
-  })
-
-  const command = new DoctorCommand()
-  command.logger = mockLogger
-
-  await command.action([], { reset: true }, command)
-
-  t.true(clearStatusStub.calledOnce, 'should call clearStatus')
-  t.is(clearStatusStub.firstCall.args[0], process.cwd(), 'should clear status for current directory')
-  t.true(
-    mockLogger.info.calledWith(sinon.match(/Build status cleared/)),
-    'should log confirmation message'
-  )
-})
-
-test.serial('doctor command --reset should not run checks', async (t) => {
-  const { sandbox, mockLogger } = t.context
-
-  const clearStatusStub = sandbox.stub()
-  const runAllChecksStub = sandbox.stub().resolves([])
-
-  const DoctorCommand = await esmock('./doctor.js', {
-    '#lib/conf/build-status.js': {
-      clearStatus: clearStatusStub,
-    },
-    '#lib/doctor/index.js': {
-      runAllChecksWithSections: runAllChecksStub,
-      checkSections: [],
-      SECTION_NAMES: ['environment', 'project', 'outputs'],
-      CHECK_IDS: ['os', 'cli', 'node', 'npm', 'git', 'project', 'deps', '11ty', 'data', 'build', 'pdf', 'epub'],
-    },
-  })
-
-  const command = new DoctorCommand()
-  command.logger = mockLogger
-
-  await command.action([], { reset: true }, command)
-
-  t.false(runAllChecksStub.called, 'should not run diagnostic checks')
-})
-
-test.serial('doctor command --reset --quiet should suppress output', async (t) => {
-  const { sandbox, mockLogger } = t.context
-
-  const clearStatusStub = sandbox.stub()
-
-  const DoctorCommand = await esmock('./doctor.js', {
-    '#lib/conf/build-status.js': {
-      clearStatus: clearStatusStub,
-    },
-    '#lib/doctor/index.js': {
-      runAllChecksWithSections: sandbox.stub().resolves([]),
-      checkSections: [],
-      SECTION_NAMES: ['environment', 'project', 'outputs'],
-      CHECK_IDS: ['os', 'cli', 'node', 'npm', 'git', 'project', 'deps', '11ty', 'data', 'build', 'pdf', 'epub'],
-    },
-  })
-
-  const command = new DoctorCommand()
-  command.logger = mockLogger
-
-  await command.action([], { reset: true, quiet: true }, command)
-
-  t.true(clearStatusStub.calledOnce, 'should still clear status')
-  t.false(mockLogger.info.called, 'should not log any messages in quiet mode')
-})