Address feedback. Minor cleanup.

packages/labs/analyzer/src/lib/analyzer.ts

@@ -68,6 +68,15 @@ export class Analyzer implements AnalyzerInterface {
   }
 }
 
+/**
+ * Extracts a `ts.ParsedCommandLine` (essentially, the key bits of a
+ * `tsconfig.json`) from the analyzer's `ts.Program`.
+ *
+ * The `ts.getOutputFileNames()` function must be passed a
+ * `ts.ParsedCommandLine`; since not all usages of the analyzer create the
+ * program directly from a tsconfig (plugins get passed the program only),
+ * this allows backing the `ParsedCommandLine` out of an existing program.
+ */
 export const getCommandLineFromProgram = (analyzer: Analyzer) => {
   const compilerOptions = analyzer.program.getCompilerOptions();
   const commandLine = ts.parseJsonConfigFileContent(

packages/labs/analyzer/src/lib/references.ts

@@ -5,7 +5,6 @@
  */
 
 import ts from 'typescript';
-import path from 'path';
 import {DiagnosticsError} from './errors.js';
 import {AnalyzerInterface, Reference} from './model.js';
 import {getModule} from './javascript/modules.js';
@@ -45,6 +44,7 @@ export function getReferenceForSymbol(
   location: ts.Node,
   analyzer: AnalyzerInterface
 ): Reference {
+  const {path} = analyzer;
   const {name} = symbol;
   // TODO(kschaaf): Do we need to check other declarations? The assumption is
   // that even with multiple declarations (e.g. because of class interface +