joshmedeski / sesh

Smart session manager for the terminal
MIT License
417 stars 17 forks source link

Issue on creating session using provided script. #64

Closed kuzyo closed 5 months ago

kuzyo commented 5 months ago

Thank for great cli app.

Using provided script that I have pasted in tmux.config seeing tmux popup window and able to search folders using fd, but after clicking enter on selected result app crashes, same is happening when I'm trying to select existing session.

I'm using fish shell and alacrity as terminal.

Could you please suggest any fix. Appreciate any help.

Screenshot 2024-02-03 at 22 19 44 Screenshot 2024-02-03 at 22 19 57
joshmedeski commented 5 months ago

Do you have zoxide set up?

kuzyo commented 5 months ago

yes, have zoxide and it shows results in popup window.

kuzyo commented 5 months ago

Initially thought issue with wrong syntax, but seeing that you actually using fish too. BTW installed sesh using brew.

joshmedeski commented 5 months ago

Okay, can you try running sesh connect {result} and tell me the exact error message? Unfortunately fzf doesn't do a good job presenting the actual error to the user.

I'm releasing a new version now so we'll see if that improves your experience.

kuzyo commented 5 months ago

Interesting that sesh connect {result} command is working correct.

I even can select sessions with such key bind in tmux.config, but not with full script, looks to me there is some small issue with syntax, probably something is not escaped.

bind-key "T" run-shell "sesh connect $(sesh list -tz | fzf-tmux -p 55%,60%)"
kuzyo commented 5 months ago

Sorry for confusion, long debugging session, but I have to just reload my machine and everything started working as expected.

joshmedeski commented 5 months ago

Glad you got it figured out, sesh is a work-in-progress so let me know if anything else comes up.

kuzyo commented 5 months ago

Thank you, definitely will do. Full work day without any issues. Went ahead and replaced ThePrimeagen tmux-sessionizer that I have used for the same purpose.