chore(compass-editor): replace autocompleter with the one from devtools-shared (#4818)

* chore(compass-editor): replace autocompleter with the one from devtools-shared

* chore(compass-editor): update mongodb-constants; use types from package

Author: gribnoysup

package-lock.json

@@ -45,7 +45,7 @@
     "@mongodb-js/compass-logging": "^1.1.8",
     "@mongodb-js/compass-utils": "^0.3.4",
     "@mongodb-js/explain-plan-helper": "^1.1.1",
-    "@mongodb-js/mongodb-constants": "^0.6.0",
+    "@mongodb-js/mongodb-constants": "^0.6.11",
     "@mongodb-js/mongodb-redux-common": "^2.0.10",
     "bson": "^5.2.0",
     "compass-preferences-model": "^2.12.0",
@@ -106,7 +106,7 @@
     "@mongodb-js/compass-logging": "^1.1.8",
     "@mongodb-js/compass-utils": "^0.3.4",
     "@mongodb-js/explain-plan-helper": "^1.1.1",
-    "@mongodb-js/mongodb-constants": "^0.6.0",
+    "@mongodb-js/mongodb-constants": "^0.6.11",
     "@mongodb-js/mongodb-redux-common": "^2.0.10",
     "bson": "^5.2.0",
     "compass-preferences-model": "^2.12.0",

packages/compass-aggregations/package.json

@@ -76,9 +76,8 @@
     "@codemirror/view": "^6.7.1",
     "@lezer/highlight": "^1.1.3",
     "@mongodb-js/compass-components": "^1.12.0",
-    "@mongodb-js/mongodb-constants": "^0.6.0",
+    "@mongodb-js/mongodb-constants": "^0.6.11",
     "polished": "^4.2.2",
-    "prettier": "^2.7.1",
-    "semver": "^7.5.4"
+    "prettier": "^2.7.1"
   }
 }

packages/compass-editor/package.json

@@ -1,10 +1,8 @@
 import { expect } from 'chai';
-import type { Completion } from './autocompleter';
-import { wrapField } from './autocompleter';
 import { completer } from './autocompleter';
 
 describe('completer', function () {
-  const simpleCompletions: Completion[] = [
+  const simpleCompletions = [
     { value: 'foo', version: '0.0.0', meta: 'stage' },
     { value: 'Foo', version: '0.0.0', meta: 'stage' },
     { value: 'bar', version: '1.0.0', meta: 'accumulator' },
@@ -67,27 +65,4 @@ describe('completer', function () {
       { value: 'bar', description: 'Int32' },
     ]);
   });
-
-  describe('wrapField', function () {
-    it('should leave identifier as-is if its roughly valid', function () {
-      expect(wrapField('foo')).to.eq('foo');
-      expect(wrapField('bar_buz')).to.eq('bar_buz');
-      expect(wrapField('$something')).to.eq('$something');
-      expect(wrapField('_or_other')).to.eq('_or_other');
-      expect(wrapField('number1')).to.eq('number1');
-    });
-
-    it("should wrap field in quotes when it's rougly not a valid js identifier", function () {
-      expect(wrapField('123foobar')).to.eq('"123foobar"');
-      expect(wrapField('bar@buz')).to.eq('"bar@buz"');
-      expect(wrapField('foo bar')).to.eq('"foo bar"');
-      expect(wrapField('with.a.dot')).to.eq('"with.a.dot"');
-      expect(wrapField('bla; process.exit(1); var foo')).to.eq(
-        '"bla; process.exit(1); var foo"'
-      );
-      expect(wrapField('quotes"in"the"middle')).to.eq(
-        '"quotes\\"in\\"the\\"middle"'
-      );
-    });
-  });
 });