Support hiding attached session

joshmedeski commented 9 months ago

As a user, I want to optionally flag if the attached session should be shown.

The tmux processSessions function needs to always return the attached session to prevent https://github.com/joshmedeski/sesh/issues/46
I don't think this should be a global config (just a CLI flag)
Make it something like --hide-attached for now (not sure if there's an alias that would be clear)
Hide the zoxide result that matches the attached session? (will cause a bug if someone chooses it)

markfeinstein commented 9 months ago

I can pick this up.

GravityShark0 commented 2 months ago

It appears that this flag (and --json but i dont really care) is unimplemented in sesh v2.

#currently in a session named 6
sesh list --hide-attached | head -n 1
# 6
sesh --version
# sesh version 2.0.2

I think I've found where it is printing, and I don't think it's filtering for --hide-attached https://github.com/joshmedeski/sesh/blob/9380ad3b039279d6141a3cdb6e36d4430490cbb5/seshcli/list.go#L62-L68 Even though connecting to the opened session no longer gives an error. I still think there is a benefit of removing some of the clutter during the selection of sessions.

joshmedeski commented 2 months ago

Something went wrong with the implementation, I need to revisit this.

joshmedeski / sesh