@cplieger/deadset-ts 7.0.0-dev.1 → 7.0.0-dev.2

package/src/findings/completion.ts

@@ -84,6 +84,35 @@ function reachabilityClasses(input: EmitterInput): (id: string) => Confidence {
   };
 }
 
+/**
+ * The confidence ceiling of a test of dead code: the lowest reachability class among the
+ * target symbols it references, because its deadness is theirs, whether or not they share
+ * its component. Every other declaration has none below `certain`.
+ */
+function testCeilings(
+  input: EmitterInput,
+  classOf: (id: string) => Confidence,
+): (id: string) => Confidence {
+  const union = input.swept.matrix.union;
+  const tests = new Set(
+    input.swept.sweep.candidates.filter((one) => one.testOfDeadCode).map((one) => one.id),
+  );
+  return (id) => {
+    let lowest: Confidence = "certain";
+    if (!tests.has(id)) {
+      return lowest;
+    }
+    for (const edge of union.out[union.at(id)] ?? []) {
+      const target = union.symbols[edge.to];
+      const testCode = union.test[edge.to] === true || union.support[edge.to] === true;
+      if (target !== undefined && !testCode && union.subject[edge.to] === true) {
+        lowest = capped(lowest, classOf(target.id));
+      }
+    }
+    return lowest;
+  };
+}
+
 /**
  * The component each dead declaration of the run falls in, and none for a live one, with
  * its members listed in the canonical order where the run lists a component in full.
@@ -154,6 +183,7 @@ export function completed(
   );
   const candidates = new Map(swept.sweep.candidates.map((candidate) => [candidate.id, candidate]));
   const classOf = reachabilityClasses(input);
+  const ceilingOf = testCeilings(input, classOf);
   const componentOf = findingComponents(swept, input.config.reporters.cascade);
   let minted = swept.sweep.components.length;
   const dead = new Set(swept.sweep.components.map((component) => component.id));
@@ -212,6 +242,11 @@ export function completed(
     const reachabilityClass =
       judged === undefined ? (finding.reachabilityClass ?? "certain") : classOf(judged.id);
     const generated = input.files.generated?.has(finding.position.path) === true;
+    const fixability = generated ? "none" : row.fixability;
+    const lastUses =
+      fixability === "deletable" && candidate !== undefined
+        ? input.dependencies.lastUses.get(candidate.id)
+        : undefined;
     return {
       code: finding.code,
       kind: row.name,
@@ -219,7 +254,10 @@ export function completed(
       position: finding.position,
       symbol: finding.symbol,
       reachabilityClass,
-      confidence: capped(reachabilityClass, row.maxClass),
+      confidence: capped(
+        capped(reachabilityClass, row.maxClass),
+        judged === undefined ? "certain" : ceilingOf(judged.id),
+      ),
       ...(candidate === undefined ? {} : { livenessRelation: candidate.relation }),
       testOnly: judged !== undefined && counts?.production === 0 && counts.test > 0,
       generated,
@@ -231,10 +269,13 @@ export function completed(
           ? swept.matrix.configurations
           : (swept.matrix.heldIn[at] ?? swept.matrix.configurations)),
       consumersLoaded: input.boundary.consumers.loaded,
-      fixability: generated ? "none" : row.fixability,
+      fixability,
       severity: severityOf(input, finding.code, row.defaultSeverity),
       message: finding.message,
-      details: finding.details ?? {},
+      details:
+        lastUses === undefined
+          ? (finding.details ?? {})
+          : { ...finding.details, removesLastUseOf: lastUses },
     };
   });
   return componentCapped(done, dead);
@@ -287,6 +328,7 @@ export function dialsOf(
 ): Dials<CompletedFinding> {
   const minConfidence = input.config.analysis.minConfidence;
   const classOf = reachabilityClasses(input);
+  const ceilingOf = testCeilings(input, classOf);
   const components = new Set(
     findings
       .filter((finding) => finding.component.root && dialed(finding, minConfidence))
@@ -300,7 +342,7 @@ export function dialsOf(
         dialed(
           {
             severity: severityOf(input, code, row.defaultSeverity),
-            confidence: capped(classOf(id), row.maxClass),
+            confidence: capped(capped(classOf(id), row.maxClass), ceilingOf(id)),
           },
           minConfidence,
         )

package/src/findings/emitters.ts

@@ -67,7 +67,7 @@ function markupKeys(input: EmitterInput): ReadonlySet<string> {
 }
 
 /** What the suppressions of a run swept under their marks are decided against. */
-export interface Suppressed {
+interface Suppressed {
   /** The same run swept with no mark, which is where a kind about liveness reports a marked declaration. */
   readonly unmarked: EmitterInput;
   /** The run's suppression records, in reading order. */
@@ -75,13 +75,13 @@ export interface Suppressed {
 }
 
 /** What the emitter table answers about one run. */
-export interface DecidedFindings {
+interface DecidedFindings {
   /** Every finding the run reports, each family's in table order. */
   readonly findings: readonly CompletedFinding[];
   /** One evaluation per declared edge side of this language, ordered by edge, then by side. */
   readonly evaluations: readonly EdgeEvaluation<CompletedFinding>[];
   /** What every suppression record did, in reading order. */
-  readonly ledger: Ledger<CompletedFinding>;
+  readonly ledger: Ledger;
   /** Every finding a record could name: the run's with no mark, then the run's as swept. */
   readonly would: readonly CompletedFinding[];
   /** The findings the minimum confidence withheld, by confidence. */
@@ -198,8 +198,3 @@ function withheldOf(input: EmitterInput, candidates: readonly CompletedFinding[]
   }
   return counts;
 }
-
-/** Every finding the run reports, each family's in table order. */
-export function findingsOf(input: EmitterInput): readonly CompletedFinding[] {
-  return decidedFindings(input).findings;
-}

package/src/findings/self-check.ts

@@ -1,8 +1,10 @@
 import { REPOSITORY_DOCUMENT } from "../config.ts";
 import type { Finding, FindingDetails, SuppressionEntry } from "../finding.ts";
 import type { InventorySymbol } from "../inventory.ts";
+import { FIXED_SEVERITY_CODES } from "../kinds.ts";
 import type { Position } from "../position.ts";
 import {
+  isRowCode,
   STALE_SUPPRESSION,
   SUPPRESSION_WITHOUT_REASON,
   type Mechanism,
@@ -58,14 +60,6 @@ interface UnmatchedDeclaration {
   readonly document: string;
 }
 
-/** A run with nothing for the self-check family to report. */
-export const NO_SELF_CHECK: SelfCheckFacts = {
-  refusals: [],
-  unmatchedRoots: [],
-  rootsDocument: "",
-  unmatchedDeclarations: [],
-};
-
 /** The words a message names a mechanism by. */
 const WRITTEN: Readonly<Record<Mechanism, string>> = {
   inline: "inline directive",
@@ -122,7 +116,7 @@ function writtenEntry(refused: Refusal): SuppressionEntry {
 }
 
 /** Each file of the run by its path, as the reference of the file it is. */
-export type FileRefs = (path: string) => string | undefined;
+type FileRefs = (path: string) => string | undefined;
 
 /**
  * The reference a finding about a suppression names: the symbol it names or bound, else
@@ -133,10 +127,7 @@ function suppressionRef(symbol: string, path: string, files: FileRefs): string {
 }
 
 /** One finding per refused suppression: the grammar's reason refusal and its scope refusal. */
-export function refusedSuppressions(
-  refusals: readonly Refusal[],
-  files: FileRefs,
-): readonly Finding[] {
+function refusedSuppressions(refusals: readonly Refusal[], files: FileRefs): readonly Finding[] {
   return refusals.map((refused) =>
     aboutSuppression(
       refused.reported,
@@ -190,7 +181,19 @@ export function staleSuppressions(
     const reports = [...new Set(records.flatMap((one) => one.reports))]
       .filter((code) => !codes.includes(code))
       .sort();
-    const instead = reports.length === 0 ? "" : `, and its declaration reports ${spelled(reports)}`;
+    const rows = codes.filter(isRowCode);
+    const settable = rows.filter((code) => !FIXED_SEVERITY_CODES.includes(code));
+    const remedy =
+      settable.length === 0
+        ? "fix what the finding names instead"
+        : `fix what the finding names or set the severity of ${spelled(settable)} instead`;
+    const declared = reports.length === 0 ? "" : `its declaration reports ${spelled(reports)}`;
+    const instead =
+      rows.length > 0
+        ? `, because ${spelled(rows)} ${rows.length === 1 ? "reports a row" : "report rows"} of a document, which no suppression can name; ${remedy}${declared === "" ? "" : `; ${declared}`}`
+        : declared === ""
+          ? ""
+          : `, and ${declared}`;
     return {
       code: STALE_SUPPRESSION,
       mechanism: first.mechanism,

package/src/findings-pass.ts

@@ -20,7 +20,7 @@ import type { Provenance, Recorded } from "./suppress-file.ts";
 import { EXIT_CLEAN, EXIT_FINDINGS } from "./verbs/verb.ts";
 
 /** The counts a report's totals carry beside its findings. */
-export interface PassTotals {
+interface PassTotals {
   readonly suppressionsInEffect: number;
   readonly reasonsRecorded: number;
   readonly staleSuppressions: number;
@@ -40,7 +40,7 @@ export interface PassResult {
   readonly edgeEvaluations: readonly EdgeEvaluation<CompletedFinding>[];
   readonly totals: PassTotals;
   /** What every suppression record did, in reading order. */
-  readonly ledger: Ledger<CompletedFinding>;
+  readonly ledger: Ledger;
 }
 
 /** The exit code of a report holding at least one pending finding. */

package/src/graph.ts

@@ -87,13 +87,13 @@ export interface Edge {
 }
 
 /** How many references one declaration carries, split by the classification of the file that made each. */
-export interface Counts {
+interface Counts {
   readonly production: number;
   readonly test: number;
 }
 
 /** One root the inventory holds, at the position of the declaration it names. */
-export interface Rooted {
+interface Rooted {
   readonly at: number;
   readonly kind: RootKind;
 }

package/src/html-entries.ts

@@ -16,7 +16,7 @@ import { dirnamePath, joinPath, relativePath, resolvePath } from "./paths.ts";
 import type { SourceFiles } from "./source-files.ts";
 
 /** One file a script loads, and the value or specifier that named it. */
-export interface HTMLEntry {
+interface HTMLEntry {
   readonly file: SourceFile;
   readonly source: string;
 }

package/src/indexed-access.ts

@@ -271,7 +271,10 @@ export function indexedAccessReferences<Brand>(
   const carried = new Map<Generic, Carried[]>();
   const names = new Set<string>();
   const sites: Site[] = [];
-  /** Calls passing a literal argument whose callee a package binding heads, which a dependency's generic may fix. */
+  /**
+   * Calls passing a literal argument whose callee a package binding heads, or that call a
+   * method, which a dependency's generic may fix.
+   */
   const literalCalls: Site[] = [];
 
   const carry = (generic: Generic, one: Carried): boolean => {
@@ -331,7 +334,12 @@ export function indexedAccessReferences<Brand>(
             (argument) => isStringLiteral(argument) || isNumericLiteral(argument),
           );
           const head = rootName(callee);
-          if (literal && isCallExpression(node) && head !== undefined && imported.has(head)) {
+          const method = isPropertyAccessExpression(callee);
+          if (
+            literal &&
+            isCallExpression(node) &&
+            (method || (head !== undefined && imported.has(head)))
+          ) {
             literalCalls.push(site);
           }
         }

package/src/inventory.ts

@@ -139,8 +139,6 @@ export interface InventoryCost {
 
 /** One project's declarations, in position order, and what reading them cost. */
 export interface Inventory {
-  /** The compiler configuration the project was opened from. */
-  readonly configFile: string;
   readonly symbols: readonly InventorySymbol[];
   /**
    * Each declaring node's {@link nodeKey} mapped to the identifier of the declaration
@@ -154,20 +152,6 @@ export interface Inventory {
    */
   readonly declarations: ReadonlyMap<string, string>;
   readonly cost: InventoryCost;
-  /**
-   * The module specifiers of the bare star re-exports the project's files carry, in
-   * the order they were read. A bare star re-export declares nothing here: the names
-   * it carries forward are the declarations of the module it names, so it is a
-   * reference to that module and not a declaration of this one.
-   */
-  readonly starReExports: readonly string[];
-  /**
-   * The number of member declarations that were left out because the file declaring
-   * them is not one of the target's own files. Such a declaration belongs to another
-   * program, so the target's inventory does not name it, and the count is reported so
-   * that the omission is a number rather than a silence.
-   */
-  readonly outsideOwnFiles: number;
   /**
    * The declarations whose module, namespace or container table the checker did not
    * answer, in position order: whether each is exported, and which members a table
@@ -534,9 +518,7 @@ export function inventory<Brand>(
   const byNode = new Map<string, Building>();
   const exportScopes: ExportScope[] = [];
   const containers: Container[] = [];
-  const starReExports: string[] = [];
   const modules: (readonly [Building, string])[] = [];
-  let outsideOwnFiles = 0;
 
   const keep = (record: Building): Building => {
     building.push(record);
@@ -782,14 +764,10 @@ export function inventory<Brand>(
     }
   }
 
-  function walkExportDeclaration(
-    node: { readonly exportClause?: Node; readonly moduleSpecifier?: Node },
-    owner: Building,
-  ): void {
+  function walkExportDeclaration(node: { readonly exportClause?: Node }, owner: Building): void {
     const clause = node.exportClause;
+    // A bare star re-export declares nothing: the names it carries are the named module's.
     if (clause === undefined) {
-      const specifier = node.moduleSpecifier;
-      starReExports.push(specifier === undefined ? "" : literalTextOf(specifier));
       return;
     }
     if (isNamespaceExport(clause)) {
@@ -876,7 +854,7 @@ export function inventory<Brand>(
         memberTables += 1;
         const table = project.queries.membersOf(symbol);
         if (isAnswered(table)) {
-          outsideOwnFiles += readTable(container, table, seen, false);
+          readTable(container, table, seen, false);
         } else {
           unansweredOwners.add(container.owner);
         }
@@ -885,7 +863,7 @@ export function inventory<Brand>(
         memberTables += 1;
         const table = project.queries.exportsOf(symbol);
         if (isAnswered(table)) {
-          outsideOwnFiles += readTable(container, table, seen, container.isClass);
+          readTable(container, table, seen, container.isClass);
         } else {
           unansweredOwners.add(container.owner);
         }
@@ -905,8 +883,7 @@ export function inventory<Brand>(
     table: ReadonlyMap<string, TSSymbol>,
     seen: Map<string, Building>,
     areStatic: boolean,
-  ): number {
-    let outside = 0;
+  ): void {
     for (const [, member] of table) {
       const declarations = member.declarations
         .map((handle) => project.declarationAt(handle))
@@ -916,7 +893,6 @@ export function inventory<Brand>(
       }
       const own = declarations.filter((held) => ownFiles.has(held.node.getSourceFile().fileName));
       if (own.length === 0) {
-        outside += declarations.length;
         continue;
       }
       // A merged symbol's table holds the members every one of its declarations writes,
@@ -925,7 +901,6 @@ export function inventory<Brand>(
         keepMember(container, held.node, seen, areStatic);
       }
     }
-    return outside;
   }
 
   /**
@@ -1017,14 +992,11 @@ export function inventory<Brand>(
     .sort((a, b) => byPosition(a.position, b.position) || (a.ref < b.ref ? -1 : 1));
 
   return {
-    configFile: project.configFile,
     symbols,
     declarations: new Map(
       [...byNode].map(([key, record]) => [key, positionKey(record.position)] as const),
     ),
     cost: { batches: 1, exportTables, memberTables },
-    starReExports,
-    outsideOwnFiles,
     unanswered: symbols
       .filter((symbol) => underUnanswered(byId.get(symbol.id), unansweredOwners))
       .map((symbol) => symbol.id),

package/src/json-document.ts

@@ -100,7 +100,7 @@ class Walk {
 }
 
 /** What a walk of one document found. */
-export interface DocumentWalk {
+interface DocumentWalk {
   /** The offset of the brace opening each object of the document's array member, in order. */
   readonly opened: readonly number[];
   /** The first member one object writes twice, at any depth, as the offset of its name. */

package/src/manifest.ts

@@ -15,7 +15,7 @@ const MANIFEST = "package.json";
 const LEGACY_MEMBERS = ["main", "module", "types"] as const;
 
 /** What reaches one entry point the manifest names. */
-export type EntryRole =
+type EntryRole =
   /** A consumer importing the package, which is a caller the analysis cannot see. */
   | "import"
   /** A shell running the package's command, which is a caller that exists. */

package/src/namespace-objects.ts

@@ -34,7 +34,7 @@ import {
 } from "@typescript/native/unstable/ast";
 
 /** What one use of a namespace object reads beyond the names the checker resolves. */
-export type NamespaceUse =
+type NamespaceUse =
   /** Every export of the module `specifier` names. */
   | { readonly kind: "whole"; readonly specifier: Node }
   /** The export a literal index names, the literal being the node to resolve. */

package/src/pnpm-workspace.ts

@@ -6,7 +6,7 @@
  */
 
 /** What the document says about `packages`. */
-export type PnpmPackages =
+type PnpmPackages =
   | { readonly kind: "listed"; readonly patterns: readonly string[] }
   /** The document declares no `packages` key. */
   | { readonly kind: "absent" }

package/src/references.ts

@@ -78,7 +78,7 @@ import { handedOn, valuesOf } from "./value-flow.ts";
  * `decorator` names the declaration a decorator expression is attached to, which the
  * decorator receives when the class is defined.
  */
-export type Use = "read" | "write" | "evaluation" | "decorator";
+type Use = "read" | "write" | "evaluation" | "decorator";
 
 /**
  * Which path answered for one reference: `batch`, a file's batched lookup; `resolved-symbol`, the
@@ -88,7 +88,7 @@ export type Use = "read" | "write" | "evaluation" | "decorator";
  * or a value's position reads; `override`, the override a base call dispatches to; `keyed`, the
  * member a symbol key selects; `by-name`, each declaration so spelled, where the checker failed.
  */
-export type Resolution =
+type Resolution =
   | "batch"
   | "resolved-symbol"
   | "shorthand"
@@ -189,8 +189,6 @@ export interface ReferenceCost {
 
 /** One project's references, in position order, and what reading them cost. */
 export interface References {
-  /** The compiler configuration the project was opened from. */
-  readonly configFile: string;
   readonly references: readonly Reference[];
   /**
    * Every file the rules classified as a test file, by its path below the target root,
@@ -214,6 +212,8 @@ export interface ReferenceOptions {
    * Each pattern is a rule of its own, which {@link testFileRulesOf} counts.
    */
   readonly testFiles: readonly string[];
+  /** The files, absolute, that are test files beside those the patterns name. */
+  readonly testFileNames?: ReadonlySet<string>;
   /**
    * The consumer whose files the pass walks instead of the target's. Its files are the
    * program's own files below its root and not below the target root, each position is
@@ -1225,7 +1225,9 @@ export function references<Brand>(
   const testFile = new Map<SourceFile, boolean>();
   for (const { file } of walked) {
     const path = renderPosition(file, root, 0).path;
-    const test = patterns.some((pattern) => pattern.test(path));
+    const test =
+      patterns.some((pattern) => pattern.test(path)) ||
+      options.testFileNames?.has(file.fileName) === true;
     testFile.set(file, test);
     if (test) {
       tests.push(path);
@@ -1569,7 +1571,6 @@ export function references<Brand>(
   );
 
   return {
-    configFile: project.configFile,
     references:
       consumer === undefined
         ? found

package/src/report.ts

@@ -30,14 +30,14 @@ export const ANALYZER_NAME = "deadset-ts";
 /** The language this analyzer claims. */
 export const LANGUAGE = "ts";
 
-export interface WirePosition {
+interface WirePosition {
   readonly path: string;
   readonly line: number;
   readonly column: number;
   readonly end_line: number;
 }
 
-export interface WireDetails {
+interface WireDetails {
   readonly narrower_visibility?: string;
   readonly implementations?: readonly {
     readonly ref: string;
@@ -54,6 +54,7 @@ export interface WireDetails {
     readonly reason?: string;
   };
   readonly overlap?: readonly string[];
+  readonly removes_last_use_of?: readonly string[];
 }
 
 /** One finding as the finding schema spells it. */
@@ -93,7 +94,7 @@ export interface WireFinding {
   readonly details: WireDetails;
 }
 
-export interface WireEdgeEvaluation {
+interface WireEdgeEvaluation {
   readonly edge: string;
   readonly side: string;
   readonly symbol: string;
@@ -178,7 +179,7 @@ interface ReportConsumer {
 }
 
 /** One consumer the scope declared and the run did not load, with why. */
-export interface UnavailableConsumer {
+interface UnavailableConsumer {
   readonly id: string;
   readonly reason: string;
 }
@@ -297,6 +298,9 @@ function wireDetails(details: FindingDetails): WireDetails {
           },
         }),
     ...(details.overlap === undefined ? {} : { overlap: details.overlap }),
+    ...(details.removesLastUseOf === undefined
+      ? {}
+      : { removes_last_use_of: details.removesLastUseOf }),
   };
 }
 

package/src/roots.ts

@@ -81,8 +81,6 @@ export interface Root {
 
 /** One project's roots, and every configured string that named nothing in it. */
 export interface Roots {
-  /** The compiler configuration the project was opened from. */
-  readonly configFile: string;
   /**
    * Every root of this project, ordered by the site of the declaration it names,
    * then by kind, then by the string that named it. A declaration that is a root for
@@ -432,7 +430,6 @@ export function roots<Brand>(
   });
 
   return {
-    configFile: project.configFile,
     liveUnderReachability: ordered,
     unmatched,
     unanswered: answered ? api.guessed : held.symbols.map((symbol) => symbol.id),

package/src/run.ts

@@ -1,6 +1,5 @@
 import { ConfigError, REPOSITORY_DOCUMENT, type Inputs } from "./config.ts";
 import { DiscoveryError, renderDiagnostic } from "./discover.ts";
-import { EMITTERS } from "./findings/emitters.ts";
 import type { Host } from "./host.ts";
 import { joinPath, resolvePath } from "./paths.ts";
 import { readScope, ScopeError, scopeForDir, type Scope } from "./scope.ts";
@@ -264,7 +263,6 @@ function invoke(
         });
         return opened;
       },
-      emitters: EMITTERS,
     });
   } catch (error: unknown) {
     opened?.close();

package/src/schema.ts

@@ -15,7 +15,7 @@ import { ConfigError } from "./config.ts";
  * - `object` is one setting written as an object: its member names are declared
  *   and a source supplies the object whole.
  */
-export type KeyKind = "leaf" | "section" | "map" | "list" | "object";
+type KeyKind = "leaf" | "section" | "map" | "list" | "object";
 
 /**
  * One node of the closed key list. `members` holds a section's declared members,
@@ -259,7 +259,7 @@ export function malformed(label: string, path: string, detail: string): ConfigEr
  * A refusal of a key the closed key list does not declare, naming the nearest key
  * it does.
  */
-export function unimplementedKey(label: string, path: string): ConfigError {
+function unimplementedKey(label: string, path: string): ConfigError {
   const nearest = nearestKey(path);
   let hint = "";
   if (nearest === path) {

package/src/scope.ts

@@ -4,8 +4,8 @@ import { dirnamePath, isAbsolutePath, joinPath, resolvePath } from "./paths.ts";
 import { checkDocument, type KeyNode } from "./schema.ts";
 
 /** The roles a scope document declares for a module. */
-export const ROLE_TARGET = "target";
-export const ROLE_CONSUMER = "consumer";
+const ROLE_TARGET = "target";
+const ROLE_CONSUMER = "consumer";
 
 /**
  * A scope document above this length is refused rather than decoded. It names one

package/src/session.ts

@@ -417,7 +417,6 @@ export function ownedAs<Brand>(
 
 /** Every diagnostic set a project is refused for carrying an error in. */
 export interface ProjectDiagnostics {
-  readonly configFile: string;
   readonly syntactic: readonly Diagnostic[];
   readonly semantic: readonly Diagnostic[];
   readonly configParsing: readonly Diagnostic[];
@@ -431,7 +430,6 @@ export interface ProjectDiagnostics {
 /** The three diagnostic sets one project carries, read before any analysis. */
 export function diagnosticsOf<Brand>(project: ProjectView<Brand>): ProjectDiagnostics {
   return {
-    configFile: project.configFile,
     syntactic: project.program.getSyntacticDiagnostics(),
     semantic: project.program.getSemanticDiagnostics(),
     configParsing: project.program.getConfigFileParsingDiagnostics(),

package/src/stores.ts

@@ -37,7 +37,7 @@ export interface Stores {
 }
 
 /** What one configuration contributes to the run's stores. */
-export interface ConfigurationStores {
+interface ConfigurationStores {
   readonly references: readonly Reference[];
   readonly accessors: readonly string[];
 }

package/src/suppress-inline.ts

@@ -37,7 +37,7 @@ const NO_REASON = new RegExp(DIRECTIVE_EXPRESSIONS.noReason);
 const DIRECTIVE_FORM = "// deadset:ignore DS0000[,DS0000...] -- <reason>";
 
 /** What the grammar's decision procedure makes of one comment. */
-export type Directive =
+type Directive =
   | { readonly is: "none" }
   | { readonly is: "directive"; readonly codes: readonly string[]; readonly reason: string }
   | { readonly is: "no-reason"; readonly codes: readonly string[] }
@@ -64,7 +64,7 @@ export function classifyComment(text: string): Directive {
 }
 
 /** One `//` line comment: the offset of its first solidus, and its text to the end of its line. */
-export interface LineComment {
+interface LineComment {
   readonly offset: number;
   readonly text: string;
 }
@@ -123,7 +123,7 @@ export function tokenStart(text: string, from: number): number {
  * syntax tree holds are skipped, so a `//` inside a string, a template or a regular
  * expression starts nothing.
  */
-export function lineComments(file: SourceFile): readonly LineComment[] {
+function lineComments(file: SourceFile): readonly LineComment[] {
   const text = file.text;
   const spans: { readonly start: number; readonly end: number }[] = [];
   const visit = (node: Node): void => {