[autocomplete] More accurate path to node in ASTManager

Fixes: #426 Do not suggest anythin in string nodes

Author: mohsen1

app/scripts/services/ast-manager.js

@@ -189,17 +189,22 @@ SwaggerEditor.service('ASTManager', function ASTManager() {
       // and end columns we found the node
       if (start.line === line && end.line === line &&
         start.column <= row  && end.column >= row) {
+
         result = path;
 
+        return result;
+
       // if this node is a map, loop through and recurse both keys and value
       } else if (current.tag === MAP_TAG) {
+
         current.value.forEach(function (keyValuePair) {
           recurse(path, keyValuePair[0]); // key
           recurse(path.concat(keyValuePair[0].value), keyValuePair[1]); // value
         });
 
       // if this node is a sequence, loop through values and recurse
       } else if (current.tag === SEQ_TAG) {
+
         current.value.forEach(function (value, index) {
           recurse(path.concat(index), value);
         });

app/scripts/services/autocomplete.js

@@ -60,8 +60,14 @@ SwaggerEditor.service('Autocomplete', function ($rootScope, snippets,
    *   in the YAML document
   */
   function getPathForPosition(pos) {
-    // pos.column is 1 more because the character is already inserted
-    var path = ASTManager.pathForPosition(pos.row, pos.column - 1);
+
+    // we are subtracting 2 from row.column because:
+    //  1. the position object is 1 base index, but ASTManager works with 0 base
+    //     indexes
+    //  2. the already inserted character should not be counted for getting the
+    //     position. We want the path up to node that we're editing, not the the
+    //     node we're adding (if any)
+    var path = ASTManager.pathForPosition(pos.row, pos.column - 2);
 
     return path;
   }