Skip to content

Add doctor command for environment checks#184

Open
baily-zhang wants to merge 6 commits intolocalstack:mainfrom
baily-zhang:feat/doctor-command
Open

Add doctor command for environment checks#184
baily-zhang wants to merge 6 commits intolocalstack:mainfrom
baily-zhang:feat/doctor-command

Conversation

@baily-zhang
Copy link
Copy Markdown

Motivation

Users need a read-only way to diagnose config, Docker, auth, and emulator connectivity from the CLI.

Changes

  • add a new lstk doctor command with read-only config, Docker, auth, and emulator checks
  • add timeout handling, per-check limits, concurrent emulator checks, and TUI support
  • add unit and integration coverage for doctor success, warning, and failure scenarios

Tests

  • go test ./cmd/... ./internal/...
  • cd test/integration && LSTK_KEYRING=file go test -count=1 -run TestDoctor ./...

@baily-zhang baily-zhang marked this pull request as ready for review April 6, 2026 22:33
Copy link
Copy Markdown

@claude claude bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Claude Code Review

This pull request is from a fork — automated review is disabled. A repository maintainer can comment @claude review to run a one-time review.

@gtsiolis
Copy link
Copy Markdown
Member

gtsiolis commented Apr 7, 2026

Thanks for contributing, @baily-zhang! ❤️

The doctor command looks like a good first iteration.

Heads-up: The team is sorting out a contribution policy process for this repository, we will follow up here as soon as we have clarity on next steps. Thanks for your patience. ⏳

@baily-zhang
Copy link
Copy Markdown
Author

Thanks for the update, and no worries. Glad to hear the first iteration looks useful. Happy to wait, and I’m available to make follow-up changes if needed once the process is in place.

@baily-zhang
Copy link
Copy Markdown
Author

iTerm2 2026-04-06 7 22 29 PM Btw, here's what it looks like in different scenarios

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants