build: update dependency to latest v13 release and fix errors

The latest v13 release of the language service removes the option to
disable Ivy from the `PluginConfig`. This commit also removes it from
the session configuration, but does not add a workaround to continue
supporting View Engine. That will need to be done before the v13
release.

In addition, the `esModuleInterop` option was enabled, which matches the
angular/angular repo. This addresses another build issue due to default
import/exports.

Author: atscott

integration/lsp/ivy_spec.ts

@@ -180,7 +180,7 @@ describe('Angular Ivy language server', () => {
       }))!;
       expect(response).not.toBeNull();
       expect(response.signatures.length).toEqual(1);
-      expect(response.signatures[0].label).toEqual('toString(): string');
+      expect(response.signatures[0].label).toEqual('(): string');
     });
 
     it('should show signature help with multiple arguments', async () => {
@@ -203,7 +203,7 @@ describe('Angular Ivy language server', () => {
       expect(response).not.toBeNull();
       expect(response.signatures.length).toEqual(1);
       expect(response.signatures[0].label)
-          .toEqual('substr(from: number, length?: number | undefined): string');
+          .toEqual('(from: number, length?: number | undefined): string');
       expect(response.signatures[0].parameters).not.toBeUndefined();
       expect(response.activeParameter).toBe(1);
 

integration/lsp/viewengine_spec.ts

@@ -13,7 +13,8 @@ import {APP_COMPONENT, FOO_TEMPLATE} from '../test_constants';
 
 import {createConnection, initializeServer, openTextDocument} from './test_utils';
 
-describe('Angular language server', () => {
+// TODO(atscott): re-enable when we have a solution in place to continue to support View Engine
+xdescribe('Angular language server', () => {
   jasmine.DEFAULT_TIMEOUT_INTERVAL = 10000; /* 10 seconds */
 
   let client: MessageConnection;

integration/tsconfig.json

@@ -2,14 +2,19 @@
   "extends": "../tsconfig.json",
   "compilerOptions": {
     "outDir": "../dist/integration",
-    "rootDirs": ["..", "dist"]
+    "rootDirs": [
+      "..",
+      "dist"
+    ]
   },
   "references": [
-    { "path": "../server/tsconfig.json" }
+    {
+      "path": "../server/tsconfig.json"
+    }
   ],
   "include": [
     "test_constants.ts",
     "e2e",
     "lsp"
   ]
-}
+}

package.json

@@ -183,7 +183,7 @@
     "test:syntaxes": "yarn compile:syntaxes-test && yarn build:syntaxes && jasmine dist/syntaxes/test/driver.js"
   },
   "dependencies": {
-    "@angular/language-service": "13.0.0-next.3",
+    "@angular/language-service": "13.0.0-next.12",
     "typescript": "4.4.3",
     "vscode-jsonrpc": "6.0.0",
     "vscode-languageclient": "7.0.0",

server/package.json

@@ -15,7 +15,7 @@
     "ngserver": "./bin/ngserver"
   },
   "dependencies": {
-    "@angular/language-service": "13.0.0-next.3",
+    "@angular/language-service": "13.0.0-next.12",
     "vscode-jsonrpc": "6.0.0",
     "vscode-languageserver": "7.0.0",
     "vscode-uri": "3.0.2"

server/src/session.ts

@@ -7,7 +7,6 @@
  */
 
 import {isNgLanguageService, NgLanguageService, PluginConfig} from '@angular/language-service/api';
-import * as assert from 'assert';
 import * as ts from 'typescript/lib/tsserverlibrary';
 import {promisify} from 'util';
 import * as lsp from 'vscode-languageserver/node';
@@ -146,12 +145,13 @@ export class Session {
       }
     });
 
+    // TODO(atscott): The Ivy flag was removed in v13. We need to include a legacy version (some
+    // v12) of the language service in order to continue supporting view engine.
     const pluginConfig: PluginConfig = {
       angularOnly: true,
-      ivy: options.ivy,
     };
     if (options.host.isG3) {
-      assert(options.ivy === true, 'Ivy LS must be used in google3');
+      options.ivy = true;
       pluginConfig.forceStrictTemplates = true;
     }
     projSvc.configurePlugin({

server/src/tests/tsconfig.json

@@ -1,10 +1,15 @@
 {
   "extends": "../../../tsconfig.json",
   "compilerOptions": {
-    "outDir": "../../../dist/server/tests"
+    "outDir": "../../../dist/server/tests",
+    "esModuleInterop": false
   },
   "references": [
-    { "path": "../../tsconfig.json" }
+    {
+      "path": "../../tsconfig.json"
+    }
   ],
-  "include": ["*.ts"]
-}
+  "include": [
+    "*.ts"
+  ]
+}

server/src/version_provider.ts

@@ -11,8 +11,8 @@ import * as path from 'path';
 
 import {NodeModule, resolve, Version} from '../common/resolver';
 
-const MIN_TS_VERSION = '4.2';
-const MIN_NG_VERSION = '12.0';
+const MIN_TS_VERSION = '4.3';
+const MIN_NG_VERSION = '13.0';
 const TSSERVERLIB = 'typescript/lib/tsserverlibrary';
 
 /**

tsconfig.json

@@ -4,12 +4,19 @@
     "module": "commonjs",
     "moduleResolution": "node",
     "sourceMap": true,
-    "strict": true
+    "strict": true,
+    "esModuleInterop": true
   },
   "files": [],
   "references": [
-    { "path": "./client" },
-    { "path": "./server" },
-    { "path": "./syntaxes" }
+    {
+      "path": "./client"
+    },
+    {
+      "path": "./server"
+    },
+    {
+      "path": "./syntaxes"
+    }
   ]
-}
+}

yarn.lock

@@ -52,10 +52,10 @@
     yaml "^1.10.0"
     yargs "^17.0.0"
 
-"@angular/[email protected].3":
-  version "13.0.0-next.3"
-  resolved "https://registry.yarnpkg.com/@angular/language-service/-/language-service-13.0.0-next.3.tgz#80aa3c0f0c1bfeabb04b11ba1a889461f0979cfc"
-  integrity sha512-etqLlMX0twVI2lZZhS4Kt3gEOmqq4FiAXEMSmpklJaXqKa4qVUrOqjgkNeAXr3vw/d/Knb4DVn7AQiIUoNNLgg==
+"@angular/[email protected].12":
+  version "13.0.0-next.12"
+  resolved "https://registry.yarnpkg.com/@angular/language-service/-/language-service-13.0.0-next.12.tgz#e080fc9e58af9267d9c97abad15dbef4b8c0ea18"
+  integrity sha512-YUF/XsQ192tycaOzhYuTWwMx299zIGxENqQ1AVf8rp1TNnfr6csh+7LJ42j60MN44FmYXnIrVRV2+fp3jfhyBA==
 
 "@babel/code-frame@^7.0.0":
   version "7.12.13"