Can you remove the count updating responsibility from printProjects and replace it with const counter = this.externalProjects.length + this.configuredProjects.length + this.inferredProjects.length. Then printProjects can be lifted out of this function entirely.

@sandersn, counter is used on line 948, though its still feasible to do something like:

function printProjects(projects: Project[], counter: number) {
  for (const project of projects) {
    ...
    counter++;
  }
  return counter;
}

And then use it like this:

let counter = 0;
counter = printProjects(this.externalProjects, counter);
counter = printProjects(this.configuredProjects, counter);
printProjects(this.inferredProjects, counter);

@sandersn, counter is used on line 948, though its still feasible to do something like:

function printProjects(projects: Project[], counter: number) {
  for (const project of projects) {
    ...
    counter++;
  }
  return counter;
}

And then use it like this:

let counter = 0;
counter = printProjects(this.externalProjects, counter);
counter = printProjects(this.configuredProjects, counter);
printProjects(this.inferredProjects, counter);

Could we instead turn Msg into a string enum? That would allow us to continue to find all references if necessary.

Why not turn this into a string enum?

There should only be one reference to each kind of message -- one each in perftrc(), info(), and err().