feat(ng-dev/ts-circular-dependencies): make golden file optional

The framework repo currently has no circular dependencies and does not
allow new ones. This change makes the golden file optional so that
framework can remove its empty golden file & associated pullapprove
config

Author: mmalerba

ng-dev/ts-circular-dependencies/config.ts

@@ -23,7 +23,7 @@ export interface CircularDependenciesTestConfig extends CircularDependenciesPars
   /** Base directory used for shortening paths in the golden file. */
   baseDir: string;
   /** Path to the golden file that is used for checking and approving. */
-  goldenFile: string;
+  goldenFile?: string;
   /** Glob that resolves source files which should be checked. */
   glob: string;
   /**
@@ -81,7 +81,7 @@ export async function loadTestConfig(configPath: string): Promise<CircularDepend
     if (!isAbsolute(config.baseDir)) {
       config.baseDir = resolveRelativePath(config.baseDir);
     }
-    if (!isAbsolute(config.goldenFile)) {
+    if (config.goldenFile && !isAbsolute(config.goldenFile)) {
       config.goldenFile = resolveRelativePath(config.goldenFile);
     }
     if (!isAbsolute(config.glob)) {

ng-dev/ts-circular-dependencies/index.ts

@@ -10,8 +10,8 @@ import {existsSync, readFileSync, writeFileSync} from 'fs';
 import {isAbsolute, relative, resolve} from 'path';
 import {Argv} from 'yargs';
 
-import ts from 'typescript';
 import {globSync} from 'fast-glob';
+import ts from 'typescript';
 
 import {green, Log, yellow} from '../utils/logging.js';
 
@@ -88,10 +88,12 @@ export function main(
 
   Log.info(green(`   Current number of cycles: ${yellow(cycles.length.toString())}`));
 
-  if (approve) {
+  if (goldenFile && approve) {
     writeFileSync(goldenFile, JSON.stringify(actual, null, 2));
     Log.info(green('✔  Updated golden file.'));
     return 0;
+  } else if (!goldenFile) {
+    Log.error(`x  Circular dependency goldens are not allowed.`);
   } else if (!existsSync(goldenFile)) {
     Log.error(`x  Could not find golden file: ${goldenFile}`);
     return 1;
@@ -116,7 +118,7 @@ export function main(
     Log.warn(`   Please rerun with "--warnings" to inspect unresolved imports.`);
   }
 
-  const expected = JSON.parse(readFileSync(goldenFile, 'utf8')) as Golden;
+  const expected = goldenFile ? (JSON.parse(readFileSync(goldenFile, 'utf8')) as Golden) : [];
   const {fixedCircularDeps, newCircularDeps} = compareGoldens(actual, expected);
   const isMatching = fixedCircularDeps.length === 0 && newCircularDeps.length === 0;
 
@@ -143,7 +145,7 @@ export function main(
 
   if (approveCommand) {
     Log.info(yellow(`   Please approve the new golden with: ${approveCommand}`));
-  } else {
+  } else if (goldenFile) {
     Log.info(
       yellow(
         `   Please update the golden. The following command can be ` +