Skip to content

feat: add atuin doctor - #1796

Merged
ellie merged 10 commits into
mainfrom
ellie/doctor
Feb 29, 2024
Merged

feat: add atuin doctor#1796
ellie merged 10 commits into
mainfrom
ellie/doctor

Conversation

@ellie

@ellie ellie commented Feb 29, 2024

Copy link
Copy Markdown
Member

This should help a bunch with diagnosing support issues

I've built it to output a datastructure as YAML. The reason here is that in the future, we can easily build functions that will take setup information and suggest fixes for common problems.

Or even automatically fix them!

Plugin detection right now has a couple notes

  1. It only detects bash preexec or ble. This is most of what we are concerned about at the moment
  2. It relies on running env in a shell subprocess. Many env vars that are useful to detect plugins are not exported. This is the only easy way I can see right now, though it is kinda gross.

Checks

  • I am happy for maintainers to push small adjustments to this PR, to speed up the review cycle
  • I have checked that there are no existing pull requests for the same thing

@ellie ellie changed the title feat add atuin doctor feat: add atuin doctor Feb 29, 2024
@ellie
ellie marked this pull request as ready for review February 29, 2024 15:25
@ellie
ellie merged commit 6d62749 into main Feb 29, 2024
@ellie
ellie deleted the ellie/doctor branch February 29, 2024 15:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant