From 6ebd90a946c9d7082fb9e35c29e5e6d9ba6e0b54 Mon Sep 17 00:00:00 2001
From: William Carroll <wpcarro@gmail.com>
Date: Tue, 4 Sep 2018 14:55:41 -0400
Subject: [PATCH] Support searching node_modules/.bin

Adds a package that allows Emacs to searching through a projects
node_modules executables when resolving a binary like eslint, prettier
etc. This was being hacked together before by relying on explicit paths
to executables. This is a more durable solution.

Also includes some packages related to LSP for Javascript, which I
haven't been able to get working yet.
---
 emacs.d/wpc/functions.el               | 11 ----
 emacs.d/wpc/packages/wpc-javascript.el | 81 ++++++++++++++------------
 2 files changed, 44 insertions(+), 48 deletions(-)

diff --git a/emacs.d/wpc/functions.el b/emacs.d/wpc/functions.el
index 29886b304..d9c4a7960 100644
--- a/emacs.d/wpc/functions.el
+++ b/emacs.d/wpc/functions.el
@@ -103,17 +103,6 @@
   (evil-window-vsplit)
   (find-file "~/.emacs.d/init.el"))
 
-(defun wpc/set-flow-executable ()
-  (interactive)
-  (let* ((root (locate-dominating-file buffer-file-name  "node_modules/flow-bin"))
-         (executable (car (file-expand-wildcards
-                           (concat root "node_modules/flow-bin/*osx*/flow")))))
-    (setq-local company-flow-executable executable)
-    ;; These are not necessary for this package, but a good idea if you use
-    ;; these other packages
-    (setq-local flow-minor-default-binary executable)
-    (setq-local flycheck-javascript-flow-executable executable)))
-
 (defun wpc/jump-to-parent-file ()
   "Jumps to a React store or component's parent file. Useful for store or index file."
   (interactive)
diff --git a/emacs.d/wpc/packages/wpc-javascript.el b/emacs.d/wpc/packages/wpc-javascript.el
index 5dcd5937b..431d31885 100644
--- a/emacs.d/wpc/packages/wpc-javascript.el
+++ b/emacs.d/wpc/packages/wpc-javascript.el
@@ -1,4 +1,4 @@
-;;; javascript.el --- My Javascript preferences -*- lexical-binding: t -*-
+;;; wpc-javascript.el --- My Javascript preferences -*- lexical-binding: t -*-
 ;; Author: William Carroll <wpcarro@gmail.com>
 
 ;;; Commentary:
@@ -6,11 +6,17 @@
 
 ;;; Code:
 
-;; Helper functions
-(defun wpc/indium-setup (url)
-  "Setup the indium environment using URL."
-  (indium-eval (format "window.dispatchEvent(new CustomEvent('patch', {detail: {url: %s}}" url)))
+;; Constants
+(defconst wpc/js-hooks
+  '(js-mode-hook js2-mode-hook rjsx-mode)
+  "All of the commonly used hooks for Javascript buffers.")
 
+(defconst wpc/frontend-hooks
+  (-insert-at 0 'css-mode-hook wpc/js-hooks)
+  "All of the commonly user hooks for frontend development.")
+
+
+;; Helper functions
 (defun wpc/insert-flow-annotation ()
   "Insert a flow type annotation to the beginning of a buffer."
   (interactive)
@@ -18,15 +24,15 @@
     (goto-char (point-min))
     (insert "// @flow\n")))
 
+;; frontend indentation settings
+(setq js-indent-level 2
+      css-indent-offset 2)
+
 ;; ;; javascript
 ;; (evil-leader/set-key-for-mode 'rjsx-mode "t" #'wpc/toggle-between-js-test-and-module)
 ;; (evil-leader/set-key-for-mode 'rjsx-mode "x" #'wpc/toggle-between-js-component-and-store)
 ;; (evil-leader/set-key-for-mode 'rjsx-mode "u" #'wpc/jump-to-parent-file)
 
-;; javascript setup
-(use-package indium
-  :hook (indium-update-script-source . wpc/indium-setup))
-
 ;; javascript text objects
 (quelpa '(evil-text-objects-javascript
           :fetcher github
@@ -40,35 +46,21 @@
   :config
   (evil-leader/set-key-for-mode 'rjsx-mode "F" #'wpc/insert-flow-annotation))
 
-(use-package company-flow
-  :after (company)
-  :hook (rjsx-mode . wpc/set-flow-executable)
-  :config
-  (add-to-list 'company-flow-modes 'rjsx-mode)
-  (add-to-list 'company-backends 'company-flow))
+;; Shouldn't need this once LSP is setup properly
+;; (use-package company-flow
+;;   :after (company)
+;;   :config
+;;   (add-to-list 'company-flow-modes 'rjsx-mode)
+;;   (add-to-list 'company-backends 'company-flow))
 
-(use-package flycheck-flow
-  :after (flycheck)
-  :config
-  (flycheck-add-mode 'javascript-flow 'rjsx-mode)
-  (flycheck-add-mode 'javascript-flow 'flow-minor-mode)
-  (flycheck-add-mode 'javascript-eslint 'flow-minor-mode)
-  (flycheck-add-next-checker 'javascript-flow 'javascript-eslint))
-
-;; front-end indentation
-(setq js-indent-level 2
-      css-indent-offset 2)
-
-;; eslint integration with flycheck
-(setq flycheck-javascript-eslint-executable "~/urbint/grid-front-end/node_modules/.bin/eslint")
-
-;; JS autoformatting
-(use-package prettier-js
-  :after (rjsx-mode)
-  :ghook ('(rjsx-mode-hook
-            js2-mode-hook
-            json-mode-hook
-            css-mode-hook)))
+;; Shouldn't need this once LSP is setup properly
+;; (use-package flycheck-flow
+;;   :after (flycheck)
+;;   :config
+;;   (flycheck-add-mode 'javascript-flow 'rjsx-mode)
+;;   (flycheck-add-mode 'javascript-flow 'flow-minor-mode)
+;;   (flycheck-add-mode 'javascript-eslint 'flow-minor-mode)
+;;   (flycheck-add-next-checker 'javascript-flow 'javascript-eslint))
 
 ;; JSX highlighting
 (use-package rjsx-mode
@@ -83,5 +75,20 @@
   (setq js2-mode-show-parse-errors nil
         js2-mode-show-strict-warnings nil))
 
+;; JS autoformatting
+(use-package prettier-js
+  :after (rjsx-mode)
+  :config
+  (general-add-hook wpc/frontend-hooks #'prettier-js-mode))
+
+(use-package add-node-modules-path
+  :config
+  (general-add-hook wpc/js-hooks #'add-node-modules-path))
+
+;; LSP support
+(use-package lsp-javascript-flow
+  :config
+  (general-add-hook wpc/js-hooks #'lsp-javascript-flow-enable))
+
 (provide 'wpc-javascript)
 ;;; wpc-javascript.el ends here