fix: fix suggestion query (#449)

Author: x1unix

web/src/components/features/workspace/CodeEditor/autocomplete/symbols/parse.test.ts

@@ -37,7 +37,33 @@ describe('parseQuery', () => {
     })
   })
 
+  test('should match function call argument', () => {
+    testParseExpression('(foo.bar', {
+      packageName: 'foo',
+      value: 'bar',
+    })
+  })
+
+  test('should match values near operators', () => {
+    testParseExpression('+ foo', {
+      value: 'foo',
+    })
+  })
+
+  test('should match pointer types', () => {
+    testParseExpression('(t *testing.T', {
+      packageName: 'testing',
+      value: 'T',
+    })
+  })
+
   test('should omit unmatched long expressions', () => {
     testParseExpression('foo.bar.baz', null)
   })
+
+  test('should not match inside string statement', () => {
+    testParseExpression('" foo', null)
+    testParseExpression('"foo', null)
+    testParseExpression('`foo', null)
+  })
 })

web/src/components/features/workspace/CodeEditor/autocomplete/symbols/parse.ts

@@ -1,5 +1,7 @@
 // Matches package (and method name)
-const queryRegexp = /(^|\s)([a-z0-9_]+)(\.([a-z0-9_]+)?)?$/i
+//
+// See: parse.test.ts
+const queryRegexp = /(?<!["`])(^|\s|\(|\{|\*|&|\+|-|\/|\||=)([a-z0-9_]+)(\.([a-z0-9_]+)?)?$/i
 
 export const parseExpression = (expr: string) => {
   queryRegexp.lastIndex = 0 // Reset regex state

web/src/components/features/workspace/CodeEditor/autocomplete/symbols/provider.ts

@@ -48,6 +48,7 @@ export class GoSymbolsCompletionItemProvider extends CacheBasedCompletionProvide
       .trim()
 
     const query = parseExpression(val)
+    console.log('expr', query)
     if (!query) {
       return null
     }