fix: unchanged files sometimes have no Angular information for strings when first opened

The `TextStorage` in TS server calls `useScriptVersionCacheIfValidOrOpen` in many places
when dealing with a `ScriptInfo`. One of the conditions in that function
is to switch to version _if the script is open_. This change in version
results in an identity change for the `SourceFile` which is problematic
for the compiler because we store references to string literals for
inline templates, template URLs, and style URLs. These references will
not be valid if the `SourceFile` changed identity.

To ensure that the compiler is aware of the change, we mark the project
as dirty when a text document is opened. This will cause the project to
call `updateGraph`, determine that the file changed versions, and create
a new program. This is done so that the Angular compiler can reprocess
the file.

This also appears to be one of, if not the only issue, that's currently
causing the e2e tests to be flaky.

Author: atscott

server/src/session.ts

@@ -636,6 +636,11 @@ export class Session {
         return;
       }
       if (project.languageServiceEnabled) {
+        // The act of opening a file can cause the text storage to switchToScriptVersionCache for
+        // version tracking, which results in an identity change for the source file. We need to
+        // ensure that the compiler is aware of this and recompiles the file because we store
+        // references to some string expressions (inline templates, style/template urls).
+        project.markAsDirty();
         // Show initial diagnostics
         this.requestDiagnosticsOnOpenOrChangeFile(filePath, `Opening ${filePath}`);
       }