fix: remove guards because of workingDirectory

Author: aminya

lib/grammar-utils.js

@@ -61,6 +61,18 @@ const GrammarUtils = {
     return ["-c", command]
   },
 
+  /** get workingDirectory */
+  workingDirectory() {
+    const activePane = atom.workspace.getActivePaneItem()
+    if (activePane && activePane.buffer && activePane.buffer.file && activePane.buffer.file.getParent) {
+      const parent = activePane.buffer.file.getParent()
+      if (parent && parent.getPath) {
+        return parent.getPath()
+      }
+    }
+    return process.cwd()
+  },
+
   // Public: Get the Java helper object
   //
   // Returns an {Object} which assists in preparing java + javac statements

lib/grammars/c.js

@@ -1,11 +1,5 @@
 "use babel"
 
-/*
- * decaffeinate suggestions:
- * DS102: Remove unnecessary code created because of implicit returns
- * DS205: Consider reworking code to avoid use of IIFEs
- * Full docs: https://github.com/decaffeinate/decaffeinate/blob/master/docs/suggestions.md
- */
 import path from "path"
 import GrammarUtils from "../grammar-utils"
 const { OperatingSystem, command } = GrammarUtils

lib/grammars/doc.js

@@ -1,11 +1,5 @@
 "use babel"
 
-/*
- * decaffeinate suggestions:
- * DS102: Remove unnecessary code created because of implicit returns
- * DS103: Rewrite code to no longer use __guard__, or convert again using --optional-chaining
- * Full docs: https://github.com/decaffeinate/decaffeinate/blob/master/docs/suggestions.md
- */
 import { shell } from "electron"
 import GrammarUtils from "../grammar-utils"
 
@@ -30,18 +24,7 @@ const DOT = {
 const gnuplot = {
   "File Based": {
     command: "gnuplot",
-    workingDirectory: __guardMethod__(
-      __guardMethod__(
-        __guard__(
-          __guard__(atom.workspace.getActivePaneItem(), (x1) => x1.buffer),
-          (x) => x.file
-        ),
-        "getParent",
-        (o1) => o1.getParent()
-      ),
-      "getPath",
-      (o) => o.getPath()
-    ),
+    workingDirectory: GrammarUtils.workingDirectory(),
     args({ filepath }) {
       return ["-p", filepath]
     },
@@ -117,17 +100,6 @@ const Sass = {
 
 const SCSS = Sass
 
-function __guardMethod__(obj, methodName, transform) {
-  if (typeof obj !== "undefined" && obj !== null && typeof obj[methodName] === "function") {
-    return transform(obj, methodName)
-  } else {
-    return undefined
-  }
-}
-function __guard__(value, transform) {
-  return typeof value !== "undefined" && value !== null ? transform(value) : undefined
-}
-
 const Docs = {
   DOT,
   GNUPlot: gnuplot,

lib/grammars/index.js

@@ -3,7 +3,6 @@
 /*
  * decaffeinate suggestions:
  * DS102: Remove unnecessary code created because of implicit returns
- * DS103: Rewrite code to no longer use __guard__, or convert again using --optional-chaining
  * DS205: Consider reworking code to avoid use of IIFEs
  * Full docs: https://github.com/decaffeinate/decaffeinate/blob/master/docs/suggestions.md
  */
@@ -153,18 +152,7 @@ const OtherGrammars = {
   Go: {
     "File Based": {
       command: "go",
-      workingDirectory: __guardMethod__(
-        __guardMethod__(
-          __guard__(
-            __guard__(atom.workspace.getActivePaneItem(), (x1) => x1.buffer),
-            (x) => x.file
-          ),
-          "getParent",
-          (o1) => o1.getParent()
-        ),
-        "getPath",
-        (o) => o.getPath()
-      ),
+      workingDirectory: GrammarUtils.workingDirectory(),
       args({ filepath }) {
         if (filepath.match(/_test.go/)) {
           return ["test", ""]
@@ -611,14 +599,3 @@ const OtherGrammars = {
   },
 }
 export default OtherGrammars
-
-function __guardMethod__(obj, methodName, transform) {
-  if (typeof obj !== "undefined" && obj !== null && typeof obj[methodName] === "function") {
-    return transform(obj, methodName)
-  } else {
-    return undefined
-  }
-}
-function __guard__(value, transform) {
-  return typeof value !== "undefined" && value !== null ? transform(value) : undefined
-}