A usage of `doto` that I overlooked for years

doto.clj

;; let's say you want to perform an action only when a state is true and returns true/false depending on the action success
;; typically I would write

(if @open ; check state
  (do
    (.put q bytes) ; action
    true)
  false)
  
;; or
(let [v @state]
  (when v (.put q bytes))
  v)
  
;; turns out one can just write:
(doto @state (when (.put q bytes)))

Should it be added as a rule to https://github.com/jonase/kibit? :-)

It doesn't seem right.

I mean, you can do it but, should you?