helm-M-x-execute-command: Wrong type argument: commandp, <command> \ \

drewverlee commented 1 year ago

Description :octocat:

Pressing hot keys "SPC SPC" aka spacemacs/helm-M-x-fuzzy-matching, then selecting a emacs command (e.g cider-inspect-last-result) does nothing, when i expected it to issue the command. Instead, i just get this in my message buffer

helm-M-x-execute-command: Wrong type argument: commandp, cider-inspect-last-result\ \

Note, this happens for all commands, not just the cider one. E.g

helm-M-x-execute-command: Wrong type argument: commandp, spacemacs/toggle-debug-on-error\ \

And, if i just use the keyboard short cut for the command, it does work. I think the issue is the double backlashes at the end of the command.

Reproduction guide :beetle:

Start Emacs
Use emacs normally for a bit. including successfully calling spacemacs/helm-M-x-fuzzy-matching.
call spacemacs/helm-M-x-fuzzy-matching.

Observed behaviour: :eyes: :broken_heart: the command called with spacemacs/helm-M-x-fuzzy-matching doesn't happen.

Expected behaviour: :heart: :smile: the command called with spacemacs/helm-M-x-fuzzy-matching does happen.

System Info :computer:

OS: gnu/linux
Emacs: 28.1
Spacemacs: 0.999.0
Spacemacs branch: develop (rev. e4b20f797)
Graphic display: t
Running in daemon: nil
Distribution: spacemacs
Editing style: vim
Completion: helm

Layers:

((auto-completion :variables auto-completion-enable-help-tooltip t auto-completion-enable-snippets-in-popup t auto-completion-enable-sort-by-usage t auto-completion-idle-delay 0.0 auto-completion-minimum-prefix-length 1)
(clojure :variables clojure-backend 'lsp clojure-enable-kaocha-runner t cider-repl-display-help-banner nil cider-print-fn 'puget clojure-indent-style 'align-arguments clojure-align-forms-automatically t clojure-toplevel-inside-comment-form t cider-result-overlay-position 'at-point cider-overlays-use-font-lock t cider-repl-buffer-size-limit 100 nrepl-use-ssh-fallback-for-remote-hosts t clojure-enable-sayid t cider-auto-select-error-buffer nil cider-inspector-auto-select-buffer nil cider-stacktrace-default-filters
      '(project))
colors command-log csv
(docker :variables docker-dockerfile-backend 'lsp)
emacs-lisp emoji
(git :variables git-magit-status-fullscreen t magit-diff-refine-hunk 'all git-enable-magit-todos-plugin t magit-diff-refine-ignore-whitespace t magit-after-save-refresh-status t magit-after-save-refresh-buffers t)
graphviz helm html javascript json
(lsp :variables lsp-headerline-breadcrumb-enable t lsp-headerline-breadcrumb-segments
  '(symbols)
  lsp-ui-peek-enable t lsp-semantic-tokens-enable t lsp-treemacs-error-list-current-project-only t lsp-idle-delay 0.2 lsp-eldoc-enable-hover nil lsp-ui-doc-enable nil lsp-ui-sideline-enable nil treemacs-space-between-root-nodes nil lsp-completion-mode nil)
markdown multiple-cursors
(ranger :variables ranger-show-preview t ranger-show-hidden t ranger-cleanup-eagerly t ranger-cleanup-on-disable t ranger-ignored-extensions
     '("mkv" "flv" "iso" "mp4"))
(shell :variables shell-default-shell 'vterm shell-default-term-shell "/usr/bin/zsh" shell-default-height 30 shell-default-position 'bottom)
(spacemacs-layouts :variables spacemacs-layouts-restrict-spc-tab t persp-autokill-buffer-on-remove 'kill-weak)
(spacemacs-modeline :variables doom-modeline-height 12 doom-modeline-bar-width 0 doom-modeline-buffer-state-icon t doom-modeline-major-mode-color-icon t doom-modeline-buffer-file-name-style 'relative-to-project doom-modeline-display-default-persp-name t doom-modeline-minor-modes nil doom-modeline-modal nil doom-modeline-buffer-encoding nil)
spell-checking
(syntax-checking :variables syntax-checking-use-original-bitmaps t)
(treemacs :variables treemacs-indentation 1 treemacs-use-filewatch-mode t treemacs-use-follow-mode t)
(unicode-fonts :variables unicode-fonts-enable-ligatures t unicode-fonts-ligature-modes
            '(prog-mode))
(version-control :variables version-control-diff-tool 'diff-hl version-control-global-margin t)
(yaml :variables yaml-enable-lsp t)
search-engine erc helpful
(languagetool :variables langtool-default-language "en-US" langtool-language-tool-jar "/home/drewverlee/language-tool/LanguageTool-6.0/languagetool-commandline.jar")
slack
(eww :variables eww-search-prefix "https://www.google.com/search?q="))

System configuration features: ACL CAIRO DBUS FREETYPE GIF GLIB GMP GNUTLS GPM GSETTINGS HARFBUZZ JPEG JSON LCMS2 LIBOTF LIBSELINUX LIBSYSTEMD LIBXML2 M17N_FLT MODULES NATIVE_COMP NOTIFY INOTIFY PDUMPER PNG RSVG SECCOMP SOUND THREADS TIFF TOOLKIT_SCROLL_BARS X11 XDBE XIM XPM GTK3 ZLIB

Backtrace :paw_prints:

Debugger entered--Lisp error: (wrong-type-argument commandp cider-inspect-last-result\ \ )
  call-interactively(cider-inspect-last-result\ \  record nil)
  command-execute(cider-inspect-last-result\ \  record)
  helm-M-x-execute-command(cider-inspect-last-result\ \ )
  helm-execute-selection-action-1()
  helm-execute-selection-action()
  helm-internal((((name . "Emacs Commands history") (init #f(compiled-function () #<bytecode -0x430f5f468c703ba>)) (candidates . helm-candidates-in-buffer) (cleanup . helm-M-x--unwind-forms) (keymap keymap (keymap ...) keymap (29 . helm-M-x-toggle-short-doc) (21 . helm-M-x-universal-argument) keymap (127 . helm-mode-delete-char-backward-maybe) (27 keymap ...) (C-return . helm-cr-empty-string) keymap (C-iso-lefttab . helm-follow-action-backward) (26 . helm-select-action) (tab . helm-execute-persistent-action) (33554440 . describe-key) (33554443 . helm-follow-action-backward) (33554442 . helm-follow-action-forward) (17 . ace-jump-helm-line) (f12 . #f(comp

lebensterben commented 1 year ago

helm layer hasn't been changed for 7 months.

Does this issue start to appear recently?

drewverlee commented 1 year ago

@lebensterben yes. I have run into it 3 times in the last .. idk 5 days? I have to restart emacs each time.

lebensterben commented 1 year ago

I don't think this is caused by Spacemacs.

Have you updated all packages?

drewverlee commented 1 year ago

@lebensterben yes. I regularly update my packages all at once. So when this bug was occurring all the packages were relatively up to date (within days).

Why don't you think it's caused by spacemacs? Your early comment seems to imply it's not caused by helm either as it hasn't changed. I'm probably confused but if use the emacs fn pkg-info-package-version on helm, i get 20230501.1445, which implies to me, that helm changed 3 days ago.

I assume it's helm related because helm is featured predominantly in the stacktrace. Can you share with me what you gather from that stack trace? Any quick intuition would be helpful.

lebensterben commented 1 year ago

helm changed. but Spacemacs' completion layer hasn't changed for more than half year.

drewverlee commented 1 year ago

@lebensterben. given spacemacs hasn't changed on this front, your are right to assume atm it's not the issue. Should we close this? I'll re-open the issue with helm and see if anyone there has any ideas. Ty for your time :)

lebensterben commented 1 year ago

I am still confused why the extra "\"s are appended to correct command names. But I don't see the error myself.

drewverlee commented 1 year ago

@lebensterben me to, that seems to be the problem. Next time it happens, ill try to dig deeper. Maybe it never will!

smile13241324 commented 1 year ago

Hi :),

as there has been no activity on this issue since May I would like to close it. @drewverlee feel free to reopen it when the issue reappears.

mfg Maxi

dmg46664 commented 10 months ago

@drewverlee This has been happening to me. Please update if you have further insight? 🙏

drewverlee commented 10 months ago

@drewverlee This has been happening to me. Please update if you have further insight? 🙏

I'm not running into this any more but as so often with these things i can't recall why and when it stopped unfortunately.

syl20bnr / spacemacs