Merge pull request #190 from zardoy/develop

Author: zardoy

.eslintrc.json

@@ -33,7 +33,8 @@
         "unicorn/switch-case-braces": "off",
         "@typescript-eslint/consistent-type-imports": "off",
         "@typescript-eslint/ban-types": "off",
-        "sonarjs/prefer-single-boolean-return": "off"
+        "sonarjs/prefer-single-boolean-return": "off",
+        "unicorn/no-typeof-undefined": "off" // todo disable globally
     },
     "overrides": [
         {

typescript/src/getPatchedNavTree.ts

@@ -10,7 +10,7 @@ type AdditionalFeatures = Record<'arraysTuplesNumberedItems', boolean>
 
 const getPatchedNavModule = (additionalFeatures: AdditionalFeatures): { getNavigationTree(...args) } => {
     // what is happening here: grabbing & patching NavigationBar module contents from actual running JS
-    const tsServerPath = __TS_SEVER_PATH__ === undefined ? require.main!.filename : __TS_SEVER_PATH__
+    const tsServerPath = typeof __TS_SEVER_PATH__ === 'undefined' ? require.main!.filename : __TS_SEVER_PATH__
     // current lib/tsserver.js
     const mainScript = nodeModules!.fs.readFileSync(tsServerPath, 'utf8')
     type PatchData = {

typescript/src/specialCommands/handle.ts

@@ -132,6 +132,9 @@ export default (
         let targetNode: undefined | ts.Node | [number, number]
         if (ts.isIdentifier(node) && node.parent) {
             node = node.parent
+            if (ts.isJsxExpression(node)) node = node.parent
+            if (ts.isJsxAttributeLike(node)) node = node.parent
+            if (ts.isJsxAttributes(node)) node = node.parent
             if (ts.isPropertyAssignment(node)) {
                 targetNode = node.initializer
             } else if ('body' in node) {
@@ -140,6 +143,10 @@ export default (
                 const pos = node.end
                 targetNode = [pos, pos]
             }
+            if (ts.isJsxClosingElement(node) || ts.isJsxClosingFragment(node)) {
+                const { pos } = node
+                targetNode = [pos, pos]
+            }
         }
 
         if (!targetNode) {