chore(snc): resolve final build-finish error (#5186)

reorder an `if` statement such that a strictNullCheck violation for
`diagnostic.absFilePath` being of type `string|undefined` where
`isRemoteUrl` only accepts `string`s is reported. since `isRemoteUrl`
returns `false` for falsy values with no side effects, this was deemed
a safe change to make

src/compiler/build/build-finish.ts

@@ -186,7 +186,7 @@ const cleanupUpdateMsg = (logger: d.Logger, msg: string, fileNames: string[]) =>
  */
 const cleanDiagnosticsRelativePath = (config: d.Config, diagnostics: ReadonlyArray<d.Diagnostic>): void => {
   diagnostics.forEach((diagnostic) => {
-    if (!diagnostic.relFilePath && !isRemoteUrl(diagnostic.absFilePath) && diagnostic.absFilePath && config.rootDir) {
+    if (!diagnostic.relFilePath && diagnostic.absFilePath && !isRemoteUrl(diagnostic.absFilePath) && config.rootDir) {
       diagnostic.relFilePath = relative(config.rootDir, diagnostic.absFilePath);
     }
   });