chore(editor): Switch back to highlight-symbol package

The auto-highlight-symbol package has a nicer interactive editing mode,
but it messes with the visual presentation for selected region and other
things. So let's go back to the less fancy but more reliable
highlight-symbol package.

Also let's properly disable both symbol highlighting packages in golang
and dart, where lsp-mode provides a better and more intelligent symbol
highlighting feature.

core/siren-core-modules.el

@@ -25,7 +25,7 @@
 
 ;; Editor
 (require 'siren-amx)
-(require 'siren-auto-highlight-symbol)
+(require 'siren-highlight-symbol)
 (require 'siren-browse-kill-ring)
 (require 'siren-display-fill-column)
 (require 'siren-display-indentation)

modules/languages/siren-dart.el

@@ -8,7 +8,6 @@
 
 (require 'siren-company)
 (require 'siren-folding)
-(require 'siren-highlight-symbol)
 (require 'siren-lsp)
 (require 'siren-projectile)
 
@@ -25,8 +24,12 @@
 
   :init
   (defun siren-dart-mode-setup ()
+    (when (fboundp 'highlight-symbol-mode)
+      (highlight-symbol-mode -1))
+    (when (fboundp 'auto-highlight-symbol-mode)
+      (auto-highlight-symbol-mode -1))
+
     (company-mode +1)
-    (highlight-symbol-mode -1)
     (lsp)
     (siren-folding)
     (subword-mode +1))

modules/languages/siren-golang.el

@@ -9,7 +9,6 @@
 (require 'siren-company)
 (require 'siren-flycheck)
 (require 'siren-folding)
-(require 'siren-highlight-symbol)
 (require 'siren-lsp)
 (require 'siren-projectile)
 
@@ -37,7 +36,11 @@
     (add-hook 'before-save-hook #'lsp-format-buffer t t)
     (add-hook 'before-save-hook #'lsp-organize-imports t t)
 
-    (highlight-symbol-mode -1)
+    (when (fboundp 'highlight-symbol-mode)
+      (highlight-symbol-mode -1))
+    (when (fboundp 'auto-highlight-symbol-mode)
+      (auto-highlight-symbol-mode -1))
+
     (company-mode +1)
     (lsp-deferred)
     (siren-folding)