Closed FelipeCortez closed 5 months ago
Thanks!
Looks like you've found what seems quite clearly a bug in sesman-current-session
(https://github.com/vspinu/sesman) - with an easy fix (observe the defcustom)
@vspinu do you agree?
(@FelipeCortez you should be able to redefine sesman-current-session
in the meantime)
@vemv just did!
(defun my-sesman-current-session (system &optional cxt-types)
(if sesman-use-friendly-sessions
(or (car (sesman--linked-sessions system 'sort cxt-types))
(car (sesman--friendly-sessions system 'sort)))
(car (sesman--linked-sessions system 'sort cxt-types))))
(advice-add 'sesman-current-session :override #'my-sesman-current-session)
makes it work as expected.
I've commit access for Sesman, so if you file a PR there I can merge it.
@bbatsov thanks! here's the PR: https://github.com/vspinu/sesman/pull/28
updated sesman to the latest version with the fix and this seems to be working now. thanks for the assistance!
Expected behavior
When setting up
sesman-use-friendly-sessions
tonil
, use only explicitly linked sessions. Switching projects should make the modeline saycider[not connected]
and evals shouldn't eval.Actual behavior
Cider always picks up friendly sessions regardless of the value of
sesman-use-friendly-sessions
Steps to reproduce the problem
Environment & Version information
Emacs version
29.1
Operating system
macOS Sonoma 14.2.1
More details
the Cider docs say
but can you really? I set this to nil after realizing that projects I didn't jack-into were picking up a REPL. then I noticed
cider-current-repl
callscider-repls
, and thensesman-current-session
singular, which issesman-current-sessions
plural seems to respectuse-friendly-sessions
:originally posted in the Clojurians slack: https://clojurians.slack.com/archives/C0617A8PQ/p1713215082795399