Skip to content

feat: Determine server info - #51

Closed
mikew wants to merge 1 commit into
mainfrom
new-branch-1755284642
Closed

feat: Determine server info#51
mikew wants to merge 1 commit into
mainfrom
new-branch-1755284642

Conversation

@mikew

@mikew mikew commented Aug 15, 2025

Copy link
Copy Markdown
Owner

Part of what's holding back Windows report support is that nvrh assumes a bash/unix like environment.

Most of it isn't so bad, it expects /tmp to exist, but the main command to start the remote neovim instance relies on a fairly bash-compatiable shell, since it uses $SHELL -i -c '...'

This PR adds some groundwork for determining some server info (namely the shell and temp directory), but it's a chicken an egg problem, since it relies on starting the remote neovim instance before it can begin doing that.

I think there's maybe some path forward where we can just run nvim -u NONE --headless -listen :SOME_PORT

  • ports because they work on both Windows and Unix
  • -u NONE because we're just going to run some lua

But this expects nvim to exist in the users PATH, which is why I've been using $SHELL -i because ... guess who has two thumbs and nvim isn't in my PATH until fully loading an interactive shell ...

@mikew mikew closed this Aug 16, 2025
@mikew
mikew deleted the new-branch-1755284642 branch August 16, 2025 16:09
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